TRUNK: Verbesserungsvorschläge

[speedy]

Man sieht eh immer wieder, dass Kommentare rausfliegen oder zumindest nicht mehr so uferlos verschachtelt sind, sondern 1-Zeiler.
Doch bis das soweit ist, stört mich das als Perfektionist.
Nee, es ist einfach so, was jetzt nicht gemacht wird, bleibt auf unbestimmte Zeit so drin - ist kein Fehler, aber halt nicht perfekt.

Ich würde deshalb die beiden Kommentare noch ändern.

[speedy]

Sind die ganzen Änderungen der SQL-Abfragen in r2150 richtig ?

Siehe

Code: PHP  [Auswählen]

                            $orders_query_raw = "-- /admin/orders.php
                                                 select

[h-h-h]

Hi Speedy,
dies ist zur Hilfe gemacht worden, um bei Fehlermeldungen genau zu wissen in welcher Datei gesucht werden muss.

Besten Gruß

h-h-h

[speedy]

Hallo h-h-h,

ah ok, also kommt in der Endversion wieder raus.

[Tomcraft]

Nein, das bleibt drin.

Grüße

Torsten

[speedy]

In "/lang/german/modules/payment/iclear.php" und entsprechend in english fehlt das hier:

Code: PHP  [Auswählen]

  // BOF - Hendrik - 2010-08-09 - exlusion config for shipping modules
  define('MODULE_PAYMENT_ICLEAR_NEG_SHIPPING_TITLE', 'Ausschluß bei Versandmodulen');
  define('MODULE_PAYMENT_ICLEAR_NEG_SHIPPING_DESC', 'Dieses Zahlungsmodul deaktivieren wenn Versandmodul gewählt (Komma separierte Liste)');
  // EOF - Hendrik - 2010-08-09 - exlusion config for shipping modules

Bei den anderen Zahlungsmodulen ist das zumindest weiterhin.

[h-h-h]

Speedy unser fleißiger Tester, finde ich richtig klasse!!

Gute Arbeit leistest Du hier, vielen lieben Dank.

Gruß

h-h-h

[speedy]

Schön zu hören.

[Tomcraft]

Das fehlt bestimmt nicht nur in den Sprachdateien.

Grüße

Torsten

[DokuMan]

Die fehlenden Definitionen für iclear sind mit r2156 behoben.

[Tomcraft]

Cool, Dankeschön!

Grüße

Torsten

[speedy]

War ja doch 'ne Menge Holz, besten Dank - DokuMan

[Modulfux]

Wo wir doch gerade bei der iclear Sache sind, dann wäre es sinnvoll im Trunk nicht nur eine aktuelle Datei einzupflegen, sondern auch gleich überflüssigen Müll zu entsorgen.

Diese Zeilen sind überflüssig in der "checkout_iclear.php":

Code: PHP  [Auswählen]

if (sizeof($order->info['tax_groups']) > 1) {
        if ($_SESSION['customers_status']['customers_status_show_price_tax'] == 0 && $_SESSION['customers_status']['customers_status_add_tax_ot'] == 1) {
        }
} else {
}

Etwas ähnliches gilt für die "checkout_payment.php".

Wäre dieser Abschnitt:

Code: PHP  [Auswählen]

if (isset ($selection[$i]['error'])) {

} else {
  $radio_buttons++;
}

in dieser Form nicht verständlicher?

Code: PHP  [Auswählen]

if (!isset ($selection[$i]['error'])) {
  $radio_buttons++;
}

Gruß
Ronny

[speedy]

Tatsache, gut gesehen - ein Else ohne Inhalt - die finde ich auch immer sehr sinnvoll.
Wenn man hier noch irgendwann Funktionalität nachrüsten will, dann sollte man wenigstens einen Kommentar unterbringen.

[DokuMan]

Danke für den Input, die "checkout_iclear.php" habe ich von iclear.de direkt übernommen ohne nochmal gegenzuchecken.

Was mich in der Datei noch stutzig macht, ist das hier:

Code: PHP  [Auswählen]

  if ($confirmation = $payment_modules->confirmation()) {

Das kann doch gar nicht klappen?

Könnt ihr das mit r2167 nochmal gegenchecken?