Problem mit "MODULE_product_options"

[web0null]

ohh ja, is auch klar.

Gruß web0null

[noRiddle (revilonetz)]

Um Mißverständnissen vorzubeugen.
Du hast natürlich auch vollkommen recht mit deinem Post. 

Gruß,
noRiddle

[web0null]

Weiß schon wie du das meinst, im Prinzip sind es 2 "Fehler".

Und... ich frage mich sowieso warum nicht die ganze Datei in der If-clause steckt, denn wenn keine Optionen vorhanden sind, braucht ja auch gar nichts davon ausgeführt werden, oder hab ich etwas übersehen?? 

Gruß web0null

[noRiddle (revilonetz)]

Welche Datei meinst du ?
Der Schwachsinn ist doch vor allem auch die Abfrage

Code: PHP  [Auswählen]

{if $MODULE_product_options != ''}

in den product_info_*.html- Templates.
Denn:

• ist das Ding ja nie leer, da auch wenn kein product_options-Template gewählt wurde ja eines per default ausgewählt ist (das erste im Ordner)
  und
• sollte auch da danach gefragt werden ob "options" angelegt sind für den Artikel.
  Deshalb sagte ich ja, daß man das template da ändern müsse (wegen clear:both; für #productoptions wichtig)-
  ansonsten hat man ein leeres "div".

Gruß,
noRiddle

[web0null]

Die Datei meine ich: \includes\modules\product_attributes.php

Und wenn es dann so wie ich schrieb gemacht wäre, also alles in die If-clause, wäre die $MODULE_product_options auch leer, bzw. gar nicht vorhanden.

[noRiddle (revilonetz)]

Achso, du meinst

Code: PHP  [Auswählen]

if ($product->getAttributesCount() > 0) {

Jau, daß ist wohl das beste.

Trortzdem muß man ja in den Template-Dateien noch eine Abfrage einbauen damit keine leeren HTML-Elemente entstehen.

Gruß,
noRiddle

[web0null]

Das würde dann eh wieder passen, in den product_info´s ist es ja so drinnen:

Code: PHP  [Auswählen]

{if $MODULE_product_options !=''}
        <div class="productoptions">{$MODULE_product_options}</div>{/if}
 

Die $MODULE_product_options ist dann auch leer, und die product_options Dateien werden somit gar nicht mehr geladen.

Wenn ich jetzt nichts übersehen habe.

Gruß web0null

[noRiddle (revilonetz)]

Nöö, oder irre ich mich da ?

Code: PHP  [Auswählen]

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

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

$MODULE_product_options ist doch nie leer, da per default immer ein product_options-Template da ist, oder ?

Gruß,
noRiddle

**************
*NACHTRAG*
Ach nee, du hast recht, habe das mit den product_info-Templates verwechselt,
da ist immer das erste im Ordner ausgewählt, auch wenn man keines im Adminbereich ausgewählt hat.
Bei den product_option-Templates ist das nicht so.
Sehr gut.
Entweder man definiert auch da ein default-Tamplate oder muß noch ein clear:both; ins Template bekommen für den Fall, daß <div id="productoptions"> gar nicht  da ist.

[h-h-h]

Hallo zusammen,
könntet ihr die product_attributes.php aus dem Trunk mal testen?
Mit r3043 sollte das Problem behoben sein.
Wenn es wie gewünscht funktioniert, dann können wir es noch im nächsten ServicePack übernehmen.

Besten Gruß

h-h-h

[noRiddle (revilonetz)]

Hast du gerade 'ne Testumgebung web0null ?
Ich nämlich nicht, könnte es momentan nur an einem Life-Shop testen.

Kann es aber auch testen, nur nicht sofort, sorry h-h-h,
dauert zwei drei Tage.

Gruß,
noRiddle

[web0null]

Ja ich werde das testet.

Edit:
@h-h-h

Die product_attributes.php die du hier

könntet ihr die product_attributes.php aus dem Trunk mal testen?

verlinkt hast funktioniert (ich habe das schon lange so in einer Testumgebung).

Hinweis:
Das hier (der Link r3043)

Mit r3043 sollte das Problem behoben sein.

unterscheidet sich aber noch zu der oben verlinkten Datei, da ist nämlich die Zeile 157 noch außerhalb, der if-clause.

Ich würde meinen wenn alles in der if-clause ist wie in der ersten (product_attributes.php) ist das besser.

Gruß web0null

[h-h-h]

Hallo web0null,
vielen Dank für die Rückmeldung und den Test. 
In r3044 und r3045 wurden noch zwei Änderungen gemacht welche schon in der verlinkten product_attributes.php enthalten waren.

Besten Gruß

h-h-h

[web0null]

Kein Problem, gerne wieder wenn ich kann.

.. und ein weil du dies so schnell umgesetzt hast, das macht spaß hier.

Gruß web0null

[moos]

Danke euch Allen.
Freue mich wenn ich damit zur Verbesserung beitragen konnte. Aber noch mehr danke ich für die Hilfestellung. Wie kann ich jetzt am besten mit dem Bug umgehen ohne es Auszukommentieren?

[web0null]

@h-h-h,
 
aaaber, wenn man es noch perfekter lösen möchte, würde ich in der \includes\modules\product_info.php

Dieses:

Code: PHP  [Auswählen]

include (DIR_WS_MODULES.'product_attributes.php');

durch das Ersetzen:

Code: PHP  [Auswählen]

if ($product->getAttributesCount() > 0) {
            include (DIR_WS_MODULES.'product_attributes.php');
        }

und in der \includes\modules\product_attributes.php das:

Code: PHP  [Auswählen]

if ($product->getAttributesCount() > 0) {}

ganz entfernen.

Denn warum etwas inkludieren, was sowieso nichts macht, wenn keine Optionen vorhanden sind.

Gruß web0null