Überflüssiger Code

[to_rob2]

Mir sind diverse Stellen (Stand: SP1d) aufgefallen, an denen es überflüssigen Code in Form von mehrfachen identischen Definitionen von Texten gibt.

lang/german/admin/orders.php:

Code: PHP  [Auswählen]

define('TABLE_HEADING_STATUS', 'Status');
...
define('TABLE_HEADING_STATUS', 'Status');

lang/german/lang_german.conf

Code: PHP  [Auswählen]

[checkout_confirmation]
text_payment_info = 'Zahlungsinformation:'
...
text_payment_info = 'Zahlungsinformationen:'

lang/english/admin/orders.php:

Code: PHP  [Auswählen]

define('TABLE_HEADING_STATUS', 'Status');
...
define('TABLE_HEADING_STATUS', 'Status');
...
define('TABLE_HEADING_STATUS', 'Status');

lang/english/lang_english.conf

Code: PHP  [Auswählen]

[checkout_confirmation]
text_payment_info = 'Payment information'
...
text_payment_info = 'Payment information'

Mehrfache Definitionen in den gleichen Dateien bzw. Abschnitten führen nur zu Irritationen. Wäre schön, wenn man das bereinigen würde.

Linkback: https://www.modified-shop.org/forum/index.php?topic=21237.0

[jannemann]

Hallo to_rob2,

die Stellen, die du gefunden hast, sind kein überflüssiger Code, sondern kommen öfters vor, da für jeden Bereich im Shop [checkout_payment_address], [checkout_shipping_address] etc. diese Variable definiert werden muss.

Schöne Grüße,
Jan

[Gradler]

Also wenn es in den normalen .php Sprachdateien doppelte Definitionen gibt raus damit!

in den lang_XXX.conf Dateien nur dann, wenn gleiche Definitionen innerhalb einer Sektion [] stehen

Es wird eh nur der erste Definitionseintrag genommen