TRUNK: Verbesserungsvorschläge

[cYbercOsmOnauT]

Ja da sind die Variablen vertauscht worden bei der Umstellung. Danke... ich hab es in r2095 behoben.

[ut2k3]

Auf Sourceforge stimmt der Link zum Logo nicht mehr: "/wp-content/uploads/2010/10/modified eCommerce Shopsoftware_logo_209x43.gif" ist aktuell noch verlinkt.

[Tomcraft]

Danke für den Hinweis! Du meinst aber nur im Trac, richtig? Den Link habe ich korrigiert.

Grüße

Torsten

[ut2k3]

Genau

[speedy]

Ich würde diesen Bereich noch um geschweifte Klammern ergänzen, also das untere if / else.
Sicherlich bei jeweils einer Zeile nicht notwendig, aber um das konsequent gleich und besser lesbar zu gestalten ...

"/includes/modules/product_info.php":

Code: PHP  [Auswählen]

      } else {
        // BOF - Tomcraft - 2009-11-28 - Included xs:booster
        //$info_smarty->assign('ADD_QTY', xtc_draw_input_field('products_qty', '1', 'size="3"').' '.xtc_draw_hidden_field('products_id', $product->data['products_id']));
        //BOF - DokuMan - 2010-09-18 - suppress undefined index message
        //if(@$xsb_tx['XTB_ALLOW_USER_CHQTY']=='true'||$xsb_tx['products_id']!=$product->data['products_id'])
        if(@$xsb_tx['XTB_ALLOW_USER_CHQTY']=='true'|| @$xsb_tx['products_id']!=$product->data['products_id'])
        //EOF - DokuMan - 2010-09-18 - suppress undefined index message
          $info_smarty->assign('ADD_QTY', xtc_draw_input_field('products_qty', '1', 'size="3"').' '.xtc_draw_hidden_field('products_id', $product->data['products_id']));
        else
          $info_smarty->assign('ADD_QTY', xtc_draw_hidden_field('products_qty', '1').' '.xtc_draw_hidden_field('products_id', $product->data['products_id']));
        // EOF - Tomcraft - 2009-11-28 - Included xs:booster
      $info_smarty->assign('ADD_CART_BUTTON', xtc_image_submit('button_in_cart.gif', IMAGE_BUTTON_IN_CART));
      }

[Tomcraft]

Ich hoffe, dass ich die nun richtig gesetzt haben, siehe r2120.

Grüße

Torsten

[speedy]

Also ich hätte es so gemacht, noch 'ne dritte Meinung.

Code: PHP  [Auswählen]

    // check if customer is allowed to add to cart
    if ($_SESSION['customers_status']['customers_status_show_price'] != '0') {
      // fsk18
      if ($_SESSION['customers_status']['customers_fsk18'] == '1') {
        if ($product->data['products_fsk18'] == '0') {
          // BOF - Tomcraft - 2009-11-28 - Included xs:booster
          //$info_smarty->assign('ADD_QTY', xtc_draw_input_field('products_qty', '1', 'size="3"').' '.xtc_draw_hidden_field('products_id', $product->data['products_id']));
          //BOF - DokuMan - 2010-09-18 - suppress undefined index message
          //if(@$xsb_tx['XTB_ALLOW_USER_CHQTY']=='true'||$xsb_tx['products_id']!=$product->data['products_id'])
          if(@$xsb_tx['XTB_ALLOW_USER_CHQTY']=='true'|| @$xsb_tx['products_id']!=$product->data['products_id']) {
          //EOF - DokuMan - 2010-09-18 - suppress undefined index message
          $info_smarty->assign('ADD_QTY', xtc_draw_input_field('products_qty', '1', 'size="3"').' '.xtc_draw_hidden_field('products_id', $product->data['products_id']));
          } else {
          $info_smarty->assign('ADD_QTY', xtc_draw_hidden_field('products_qty', '1').' '.xtc_draw_hidden_field('products_id', $product->data['products_id']));
          }
          // EOF - Tomcraft - 2009-11-28 - Included xs:booster
          $info_smarty->assign('ADD_CART_BUTTON', xtc_image_submit('button_in_cart.gif', IMAGE_BUTTON_IN_CART));
        }
      } else {
        // BOF - Tomcraft - 2009-11-28 - Included xs:booster
        //$info_smarty->assign('ADD_QTY', xtc_draw_input_field('products_qty', '1', 'size="3"').' '.xtc_draw_hidden_field('products_id', $product->data['products_id']));
        //BOF - DokuMan - 2010-09-18 - suppress undefined index message
        //if(@$xsb_tx['XTB_ALLOW_USER_CHQTY']=='true'||$xsb_tx['products_id']!=$product->data['products_id'])
        if(@$xsb_tx['XTB_ALLOW_USER_CHQTY']=='true'|| @$xsb_tx['products_id']!=$product->data['products_id']) {
        //EOF - DokuMan - 2010-09-18 - suppress undefined index message
          $info_smarty->assign('ADD_QTY', xtc_draw_input_field('products_qty', '1', 'size="3"').' '.xtc_draw_hidden_field('products_id', $product->data['products_id']));
        } else {
          $info_smarty->assign('ADD_QTY', xtc_draw_hidden_field('products_qty', '1').' '.xtc_draw_hidden_field('products_id', $product->data['products_id']));
        }
        // EOF - Tomcraft - 2009-11-28 - Included xs:booster
        $info_smarty->assign('ADD_CART_BUTTON', xtc_image_submit('button_in_cart.gif', IMAGE_BUTTON_IN_CART));
      }
    }

Sieht man schon mal für was das mit den geschweiften Klammern gut ist, zur Entwirrung.

EDIT: Hab noch einen Kommentar verschoben und eine Ausrichtung geändert.

[Tomcraft]

Ich wollte es zumindest erstmal im Trunk haben, denn somit sehen auch die anderen Entwickler direkt die Änderung und werden das dann sicherlich auch noch hübsch formatieren, wenn sie die Notwendigkeit dazu sehen.

Die Klammern haben aber ganz klar gefehlt, weil ja mehr als eine Zeile Code folgt.

Grüße

Torsten

[speedy]

Ohne geschweifte Klammern wird bis zum nächsten Semikolon abgearbeitet, also die nächste Zeile.
Also zum If bzw. Else gehört dann nur die nächste Zeile.

Deshalb glaub ich eben auch, dass es so gehört:

Code: PHP  [Auswählen]

        if(@$xsb_tx['XTB_ALLOW_USER_CHQTY']=='true'|| @$xsb_tx['products_id']!=$product->data['products_id']) {
        //EOF - DokuMan - 2010-09-18 - suppress undefined index message
          $info_smarty->assign('ADD_QTY', xtc_draw_input_field('products_qty', '1', 'size="3"').' '.xtc_draw_hidden_field('products_id', $product->data['products_id']));
        } else {
          $info_smarty->assign('ADD_QTY', xtc_draw_hidden_field('products_qty', '1').' '.xtc_draw_hidden_field('products_id', $product->data['products_id']));
        }
        // EOF - Tomcraft - 2009-11-28 - Included xs:booster
        $info_smarty->assign('ADD_CART_BUTTON', xtc_image_submit('button_in_cart.gif', IMAGE_BUTTON_IN_CART));

und nicht:

Code: PHP  [Auswählen]

      if(@$xsb_tx['XTB_ALLOW_USER_CHQTY']=='true'|| @$xsb_tx['products_id']!=$product->data['products_id']) {
        //EOF - DokuMan - 2010-09-18 - suppress undefined index message
        $info_smarty->assign('ADD_QTY', xtc_draw_input_field('products_qty', '1', 'size="3"').' '.xtc_draw_hidden_field('products_id', $product->data['products_id']));
      } else {
          $info_smarty->assign('ADD_QTY', xtc_draw_hidden_field('products_qty', '1').' '.xtc_draw_hidden_field('products_id', $product->data['products_id']));
          // EOF - Tomcraft - 2009-11-28 - Included xs:booster
          $info_smarty->assign('ADD_CART_BUTTON', xtc_image_submit('button_in_cart.gif', IMAGE_BUTTON_IN_CART));
      }

Im vorherigen Beispiel (Beitrag) hatte ich das so gemacht.
Ansonsten hätte es ja einen Funktionsfehler gegeben (Warenkorb-Button würde beim If fehlen), das hätte auffallen müssen. Deshalb glaube ich auch, dass meine Variante richtig ist und aktuell so im Trunk ein Fehler ist. Welchen Sinn hätte ein Mengenfeld ohne Warenkorb Button ?

Mein Beitrag oben war deshalb nicht Kosmetik, die erfolgte dann später durch das EDIT. Es ging um die richtige Setzung der geschweiften Klammern.

[Tomcraft]

Na hoppala... ich denke, dass du Recht hast, aber da schaut bestimmt nochmal jemand drüber... ich werde zumindest jemand drum bitten.

Grüße

Torsten

[franky_n]

Hallo Zusammen,

habe den COde formatiert in r2021 und in r2022 den Button so gesetzt wie er sein müsste.

Diese if Anweisung:

Code: PHP  [Auswählen]

if(@$xsb_tx['XTB_ALLOW_USER_CHQTY']=='true'|| @$xsb_tx['products_id']!=$product->data['products_id']) {
 

bedeutet ob der Kunde ein Feld Quantity zu sehen bekommt oder ob immer ein Wert 1 versteckt an den Warenkorb übertragen wird.
Daher muß der Warenkorb Button außerhalb dieser if Schleife stehen.

Viele Grüße

Franky

[speedy]

Jupp, jetzt ist das so wie ich meinte
Nur der Kommentar (EOF) ist noch 2x falsch, der gehört auch aus der If / Else raus, wenn man sich ansieht wo der Kommentar beginnt (BOF).

Code: PHP  [Auswählen]

// EOF - Tomcraft - 2009-11-28 - Included xs:booster

Also:

Code: PHP  [Auswählen]

          } else {
          $info_smarty->assign('ADD_QTY', xtc_draw_hidden_field('products_qty', '1').' '.xtc_draw_hidden_field('products_id', $product->data['products_id']));
          }
          // EOF - Tomcraft - 2009-11-28 - Included xs:booster
          $info_smarty->assign('ADD_CART_BUTTON', xtc_image_submit('button_in_cart.gif', IMAGE_BUTTON_IN_CART));

[franky_n]

Hallo speedy,

so weit ich weiß werden die Kommentare wohl eh zum Teil raus geworfen, das wollte ich aber denjenigen überlassen die nun genau wissen was wie raus soll...
Diese Kommentare finde ich nämlich eh sehr unübersichtlich.
Aber gut das laß ich andere entscheiden...

Viele Grüße

Franky

[Modulfux]

Da stimme ich dir zu, Franky.

Wieso nicht einfach für die Trunkversion die Kommentare drin lassen und in der Release-Version sind sie entfernt.

Gruß
Ronny

[web28]

Teilweise haben wir 5 Zeilen Kommentar für eine Codezeile, das wird nach und nach überarbeitet und auf ein vernünftiges Maß reduziert.