Problem mit "MODULE_product_options"

[moos]

Habe hier ein Fehler. Bei mir werden die Produktoptionen angezeigt obwohl keine Vorhanden.
Hoffe ich habe es deutlicht gemacht.
Ver. 1.05 SP1c



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

[noRiddle (revilonetz)]

Nee iss klaar...

[moos]

Nee iss klaar...

Folgendes steht in der product_info_v1.html.
Was mich wundert ist das es ja eine if Abfrage ist. Da ich aber kein Option habe, sollte auch wo jetzt "hallo" angzeigt wird nicht sichtbar sein. Oder verstehe ich da was falsch?!?

Vielen Dank im Voraus

Code: PHP  [Auswählen]

        </table></div>         
        {if $MODULE_product_options !=''}
  <div class="productoptions">hallo{$MODULE_product_options}</div>{/if}
<!-- BOF - Tomcraft - 2009-11-28 - Included xs:booster //-->
        {if $XTB_REDIRECT_USER_TO=='product'}<div style='width:400px;padding:10px;color:white;background-color:green;'><strong>Sie haben die eBay Auktion gewonnen!</strong><br/>Bitte w&auml;hlen Sie nun noch die gew&uuml;nschten Produkteigenschaften und legen Sie den Artikel danach in den Warenkorb, um mit der Zahlung fortzufahren und das Produkt versandfertig zu machen!</div>{/if}
<!-- EOF - Tomcraft - 2009-11-28 - Included xs:booster //-->
        {if $PRODUCTS_DESCRIPTION !=''}<div>{$PRODUCTS_DESCRIPTION}</div>{/if}

        </div>           
    </td></tr></table>

[Andre Kern]

 
Aber Du hast ja selbst Hallo reingeschrieben? Du musst im Backend Optionen anlegen damit Sie sichtbar werden.

Grüße

[moos]

Aber Du hast ja selbst Hallo reingeschrieben? Du musst im Backend Optionen anlegen damit Sie sichtbar werden.

Grüße

Danke Shiva,
da hast du Recht, habe ich aber nicht. Trotzdem werden aber dieser Bereich angzeigt (CSS Angaben). Habe nur "hallo" reingeschrieben damit man es besser sieht. Ich kann es zwar auskommentieren aber das ist ja nicht Sinn der Sache. Irgendein Tipp?

[Andre Kern]

Hast Du evtl. einen Link zum Shop? Oder ist das eine Testumgebung?

[noRiddle (revilonetz)]

moos hat schon recht, nach seinem ersten Post hatte ich allerdings auch nicht verstanden worum es geht.

Um das was da passiert zu verstehen, muß man wissen was die Smarty-Variable $MODULE_product_options genau beinhaltet.
Sie ist in /includes/modules/product_attributes.php definiert als

Code: PHP  [Auswählen]

$module = $module_smarty->fetch(CURRENT_TEMPLATE.'/module/product_options/'.$product->data['options_template']);

$info_smarty->assign('MODULE_product_options', $module);

Also ist die Variable nie leer, da das ausgewählte product_options_*.html -Template ja schon vor der foreach-Schleife Code beinhaltet.
Die if-Abfrage nach $MODULE_product_options in den product_info_*.htm -Templates ist so in der tat nicht besonders sinnvoll.

Gruß,
noRiddle

[Andre Kern]

Vielleicht wäre es dann eine einfache "Notlösung" den Background und den Border per css zu entfernen? Dann sieht man es zumindest nicht direkt oder?

Grüße

[moos]

Vielleicht wäre es dann eine einfache "Notlösung" den Background und den Border per css zu entfernen? Dann sieht man es zumindest nicht direkt oder?

Grüße

Da hast du Recht.
Ist aber nicht die "saubere" Lösung. Könnte es auch auskommentieren, wie gesagt.

Was denkst du noRiddle?

[Andre Kern]

Das musst Du ja wissen Also wenn Du es nicht brauchst wäre es in Deinem Fall ja auch OK.

Grüße

[noRiddle (revilonetz)]

Im product_options_*.html Template wird ja so abgefragt:

Code: XML  [Auswählen]

{if $options!=''}

$options ist /includes/modules/product_attributes.php so definiert:

Code: PHP  [Auswählen]

$module_smarty->assign('options', $products_options_data);

und $products_options_data ist ein Array.

Man kann aber ein Array nicht abfragen ob es leer ist indem man es auf einen leeren String testet.
Ich suche da gerade auch 'ne Lösung.
Da hast du was interessantes entdeckt, halte ich für einen Bug.

Werde die Lösung posten wenn ich eine gefunden habe.

Gruß,
noRiddle

[noRiddle (revilonetz)]

Nicht mal das geht

Code: PHP  [Auswählen]

{if is_array($options) && count($options) > 0}

Bin etwas ratlos...

Gruß,
noRiddle

[noRiddle (revilonetz)]

Geht doch, ich habe gepennt.
Also in die product_options_*.html -Templates muß statt:

Code: PHP  [Auswählen]

{if $options != ''}

das hier

Code: PHP  [Auswählen]

{if count($options) > 0}

Allerdings sollte man da noch was am Template ändern damit da nicht ein leeres <div class="productoptions"> bleibt.
Also vielleicht auch in den product_info_*.html -templates die gleiche Abfrage machen anstatt nach $MODULE_product_options abzufragen.

Gruß,
noRiddle

*****************
*NACHTRAG*
Außerdem muß zwischen Operator und Term ein Leerzeichen sein,
will sagen, wie es teilweise im Template geschrieben ist funktioniert's sowieso nicht.

Code: PHP  [Auswählen]

{if $options!=''}

SO NICHT !!

[web0null]

Hi das hatte ich auch mal.

Suche in der product_attributes.php

Code: PHP  [Auswählen]

$products_options_data = array();
if ($product->getAttributesCount() > 0) {

und ersetze es mit:

Code: PHP  [Auswählen]

// $products_options_data = array();
if ($product->getAttributesCount() > 0) {

Es wird hier zu früh gesetzt, ca. 20 Zeilen darunter ist es eh drinnen und dort ist es auch richtig

[noRiddle (revilonetz)]

Okay, das ändert aber nichts daran, daß man ein Array nicht auf leeren String abfragen kann.
Will sagen, der Bug ist ein Bug und sollte geändert werden.

Gruß,
noRiddle