$_SESSION['language'] // Lästige mehrfach Assign.

Duplicate julien

Hallo,

in der vorherige Version auch da, die mehrfach Setzung der variable:

PHP

$smarty->assign('language', $_SESSION['language']);

Bestes Beispel: die 404.ph
Linie 71 und gleich nochmal Linie 74 ??
oder
advanced_search_result.php
Linie 479, 486, 544

Da ist/wäre noch Raum zum Code cleansing, oder?
Wer der Liste braucht einfach melden.

Julien.

juergen

Hallo Julien, wäre super wenn du mir die Liste zukommen lassen würdest.

Im voraus vielen Dank

Duplicate julien

Hallo Juergen,

hier die Trefferliste.
Ich schlage vor wir? gehen die Liste durch und bestätigen später welche sicher zu entfernen sind, oder wo man die Assign am besten platzieren sollte.

Zitat

D:\xampp\htdocs\cseov22CE\404.php
71: $smarty->assign('language', $_SESSION['language']);
74: $smarty->assign('language', $_SESSION['language']);

D:\xampp\htdocs\cseov22CE\account.php
63: $smarty->assign('language', $_SESSION['language']);
121: $smarty->assign('language', $_SESSION['language']);
D:\xampp\htdocs\cseov22CE\advanced_search_result.php
479: $smarty->assign('language', $_SESSION['language']);
487: $smarty->assign('language', $_SESSION['language']);
544: $smarty->assign('language', $_SESSION['language']);
D:\xampp\htdocs\cseov22CE\checkout_payment.php
28: $smarty->assign('language', $_SESSION['language']);
328: $smarty->assign('language', $_SESSION['language']);
333: $smarty->assign('language', $_SESSION['language']);

D:\xampp\htdocs\cseov22CE\checkout_payment_address.php
279: $smarty->assign('language', $_SESSION['language']);
284: $smarty->assign('language', $_SESSION['language']);

D:\xampp\htdocs\cseov22CE\checkout_payment_iframe.php
85: $smarty->assign('language', $_SESSION['language']);
91: $smarty->assign('language', $_SESSION['language']);
D:\xampp\htdocs\cseov22CE\checkout_shipping.php
254: $smarty->assign('language', $_SESSION['language']);
260: $smarty->assign('language', $_SESSION['language']);

D:\xampp\htdocs\cseov22CE\checkout_shipping_address.php
295: $smarty->assign('language', $_SESSION['language']);
300: $smarty->assign('language', $_SESSION['language']);

D:\xampp\htdocs\cseov22CE\checkout_success.php
179: $smarty->assign('language', $_SESSION['language']);
186: $smarty->assign('language', $_SESSION['language']);

D:\xampp\htdocs\cseov22CE\cookie_usage.php
33: $smarty->assign('language', $_SESSION['language']);
47: $smarty->assign('language', $_SESSION['language']);

D:\xampp\htdocs\cseov22CE\create_account.php
293: $smarty->assign('language', $_SESSION['language']);
338: $smarty->assign('language', $_SESSION['language']);
560: $smarty->assign('language', $_SESSION['language']);
565: $smarty->assign('language', $_SESSION['language']);

D:\xampp\htdocs\cseov22CE\create_guest_account.php
288: $smarty->assign('language', $_SESSION['language']);
492: $smarty->assign('language', $_SESSION['language']);
498: $smarty->assign('language', $_SESSION['language']);
D:\xampp\htdocs\cseov22CE\gv_send.php
120: $smarty->assign('language', $_SESSION['language']);
226: $smarty->assign('language', $_SESSION['language']);
229: $smarty->assign('language', $_SESSION['language']);
D:\xampp\htdocs\cseov22CE\login.php
466: $smarty->assign('language', $_SESSION['language']);
511: $smarty->assign('language', $_SESSION['language']);
736: $smarty->assign('language', $_SESSION['language']);
D:\xampp\htdocs\cseov22CE\logoff.php
64: $smarty->assign('language', $_SESSION['language']);
70: $smarty->assign('language', $_SESSION['language']);

D:\xampp\htdocs\cseov22CE\newsletter.php
38: $smarty->assign('language', $_SESSION['language']);
328: $smarty->assign('language', $_SESSION['language']);
335: $smarty->assign('language', $_SESSION['language']);
D:\xampp\htdocs\cseov22CE\password_double_opt.php
46: $smarty->assign('language', $_SESSION['language']);
115: $smarty->assign('language', $_SESSION['language']);
161: $smarty->assign('language', $_SESSION['language']);
169: $smarty->assign('language', $_SESSION['language']);
190: $smarty->assign('language', $_SESSION['language']);
211: $smarty->assign('language', $_SESSION['language']);
219: $smarty->assign('language', $_SESSION['language']);
240: $smarty->assign('language', $_SESSION['language']);
247: $smarty->assign('language', $_SESSION['language']);

D:\xampp\htdocs\cseov22CE\paypal_checkout.php
465: $smarty->assign('language', $_SESSION['language']);
472: $smarty->assign('language', $_SESSION['language']);
D:\xampp\htdocs\cseov22CE\product_filter.php
27: $smarty->assign('language', $_SESSION['language']);
71: $smarty->assign('language', $_SESSION['language']);
330: $smarty->assign('language', $_SESSION['language']);
335: $smarty->assign('language', $_SESSION['language']);
D:\xampp\htdocs\cseov22CE\product_reviews.php
70: $smarty->assign('language', $_SESSION['language']);
76: $smarty->assign('language', $_SESSION['language']);

D:\xampp\htdocs\cseov22CE\product_reviews_info.php
63: $smarty->assign('language', $_SESSION['language']);
77: $smarty->assign('language', $_SESSION['language']);
D:\xampp\htdocs\cseov22CE\products_new.php
104: $smarty->assign('language', $_SESSION['language']);
113: $smarty->assign('language', $_SESSION['language']);

D:\xampp\htdocs\cseov22CE\reviews.php
114: $smarty->assign('language', $_SESSION['language']);
120: $smarty->assign('language', $_SESSION['language']);
D:\xampp\htdocs\cseov22CE\shop_content.php
187: $smarty->assign('language', $_SESSION['language']);
211: $smarty->assign('language', $_SESSION['language']);
227: $smarty->assign('language', $_SESSION['language']);
D:\xampp\htdocs\cseov22CE\shopping_cart.php
175: $smarty->assign('language', $_SESSION['language']);
293: $smarty->assign('language', $_SESSION['language']);

D:\xampp\htdocs\cseov22CE\specials.php
93: $smarty->assign('language', $_SESSION['language']);
99: $smarty->assign('language', $_SESSION['language']);

D:\xampp\htdocs\cseov22CE\ssl_check.php
32: $smarty->assign('language', $_SESSION['language']);
46: $smarty->assign('language', $_SESSION['language']);

D:\xampp\htdocs\cseov22CE\taglisting.php
169: $smarty->assign('language', $_SESSION['language']);
181: $smarty->assign('language', $_SESSION['language']);

Alles anzeigen

Julien.

Duplicate julien

Hallo,

ich würde fast sagen ,dass man die "language" Assign gleich nach:

Code

// create smarty elements
$smarty = new Smarty;

definieren sollte und alle andere Vorkomnisse löschen könnte da die $_SESSION in dem includes/application_top.php definiert wird und nie? innerhalb eine shoproot/*.php geändert wird.

admin, was denkst Du? Wäre es vielleicht was?

J.

juergen

Hallo Julien, bin zur Zeit unterwegs. Sehe mir das ganze aber heute Abend an.

juergen

Hallo Julien, sorry ist nicht meine Art mich verspätet zu melden.
Bin gestern nicht dazu gekommen. Werde das ganze aber am Wochenende machen.
Melde mich dann!
Vieleicht meldet sich ja Herr Kausch auch noch dazu!

admin

Ja, danke, noch mehr Arbeit Aber einiges haben wir schon bereinigt, aber recht hast Du, ist mir auch an diversen Stellen aufgefallen. Sollte performance technisch kein Problem sein, eher kosmetischer Natur. Werden das sicher mal in Ruhe durch gehen, als Fix aber ungeeignet In die aptlication_top gehört es aber definitiv nicht rein.

admin

Ach ja, warum ich da vorsichtig bin, es muss eine Reihenfolge eingehalten werden! Nicht einfach oben rein klatschen. Die Liste ist aber absolut top, da wieß ich schon mal auf Anhieb, wo die größten Sünder sind Aber wie komme ich jetzt auf Dein D:? :p

Duplicate julien

admin,

jetzt weisst Du ,dass ich lokal arbeite und Xampp nutze, Sh*t!;)
Die Liste habe ich übrigens mit PsPad Editor, mein seit Jahren lieblings Editor und natürlich gratis!
Das mit der application_top.php war von mir falsch ausgedruckt, ich wollte sagen ,dass die $_SESSION['language'] dort definiert wird UND DANN in den shoproot/*.php zu den Smarty Var weitergegeben wird, aber NIE in den shoproot/*.php modifiziert.
Deshalb mein Vorschalg gleich in den jeweiligen shoproot/*.php die Var unter dem

Zitat

// create smarty elements
$smarty = new Smarty;

zu platzieren.

By the way, aber da bin ich mir nicht so sicher, ich glaube das gleiche ist auch zu beobachten bei den

Code

$smarty->caching = false;

die mehrmals auch auftauchen (aber wie gesagt da bin ich mir nicht 100% sicher)

Übrigens Admin, Du tust ein guten Job. Weiter so.
J.