v2.0.5.1 - GLS Inselzuschlagspreis wird nicht angezeigt

[Farian]

Im Warenkorb und beim Checkout wurde mir beim GLS-Versand der eingestellte Preis für den Inselzuschlag nicht anzeigt:

"inkl. Inselzuschlag: ()"

In der ../includes/modules/gls.php habe ich am Ende von Zeile 147 folgenden Code getauscht:

Code: PHP  [Auswählen]

('.$shipping_cost_extra_formatted.' '.$tax_text.')',

in

Code: PHP  [Auswählen]

('.MODULE_SHIPPING_GLS_POSTCODE_EXTRA_COST.'  EUR)',

Jetzt funzt die Anzeige mit dem im Versandmodul eingestellten Preis (13.95)

"inkl. Inselzuschlag: (13.95 EUR)"

..wer´s mal braucht. Oder gibt´s eine andere Lösung?

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

[noRiddle (revilonetz)]

...
..wer´s mal braucht. ...

Besser nicht.

Das von dir ersetzte $shipping_cost_extra_formatted ist aber doch mittels MODULE_SHIPPING_GLS_POSTCODE_EXTRA_COST definiert.
Ich vermute, daß eine der if-Clauses die die betroffenen Variablen konditional umfassen nicht erfüllt wird.
Das sollte man also genauer betrachten.

Das Modul hat imho aber auch fehleranfälligen Code. So sollte z.B.  $shipping_cost_extra_formatted auch definiert sein wenn die umfassende if-clause bei der Definition nicht zutrifft, was mom. nicht der Fall ist.

Im Übrigen ist der Pfad zur Datei nicht
../includes/modules/gls.php
sondern
/includes/modules/shipping/gls.php.

Gruß,
noRiddle

[Farian]

Danke für Deine Info. Das meine Änderung nur eine Notlösung ist, dachte ich mir schon. Bis da eine Lösung gefunden ist, benutze ich das mal so. Auch wenn GLS-Inselversand eher selten vorkommt, hat die fehlerfreie Funktion (und korrekte Pfadangabe) natürlich Vorrang.

Gruß
Farian

[noRiddle (revilonetz)]

Ich habe mich vielleicht mißverständlich ausgedrückt.
Eine der if-Clauses in der genannten Datei wird nicht erfüllt weil du etwas nicht ausgefüllt oder angegeben hast.
Ich vermute du hast keinen Steuersatz angegeben (im Code abgefragt mittels if ($this->tax_class > 0) { ).
Die andere if-Clause die greifen könnte wäre, daß du mit einem Konto testest welches keine PLZ hat die unter die Inselzuschläge fällt (siehe deine Config im Modul). Von dieser Möglichkeit gehe ich aber nicht aus.
*edit* diese zweite Möglichkeit ist nicht korrekt, sie ist aufgrund des Codes nicht möglich *end_edit*

Meine Kritik an dem Modul-Code bezieht sich darauf, daß einige Variablen gar nicht definiert sind wenn gewisse Konditionen nicht zutreffen, später jedoch aufgerufen werden.
Das hat auf dein Problem keine, oder besser, nur indirekte, Auswirkung.

Deine "Notlösung" ist keine weil sie verkehrt ist und sie und hat Folgen die das Modul nicht korrekt funktionieren lassen.
*edit* Satz korrigiert *end_edit*

Gruß,
noRiddle

[Farian]

Deine erste Vermutung mit dem fehlenden Steuersatz war der richtige Ansatz zur Lösung. Nachdem ich das eingestellt hatte, funktionierte die "Insel"-Anzeige wie vorgesehen.

Vielen Dank für Deine Hilfe
Farian