Bug ? in Versandmodul freeamount = versandkostenfrei

[noRiddle (revilonetz)]

Okay, ich habe den Fehler gefunden.
In /includes/classes/shipping.php gibt's ca. in Zeile 101 folgenden Code:

Code: PHP  [Auswählen]

// BOF - Tomcraft - 2011-02-01 - Paypal Express Modul
// if (is_array($quotes)) $quotes_array[] = $quotes;
if (!isset ($quotes['error'])) {
    if (is_array($quotes)) $quotes_array[] = $quotes;
}
// EOF - Tomcraft - 2011-02-01 - Paypal Express Modul

Da aber $quotes['error'] "ge-set-et" () ist wenn die Anzeige für den zu erreichenden Betrag für "versandkostenfrei" im freeamount-Modul auf "True" steht, wird das Modul nicht ins Array $quotes geladen.... pfff, oder wie soll ich's erklären...
Der "böse" Tomcraft ist also schuld 

Da ich nicht untersucht habe was der Code mit dem PayPal Express-Modul zu tun hat, weiß ich nicht ob man die if-clause einfach auskommentieren kann;
tut man es jedoch, ist alles in Ordnung was die Anzeige des zu erreichenden Betrages für "versandkostenfrei" betrifft.
Vielleicht blickt da einer mehr durch.

Gruß,
noRiddle

[WayneTsun]

Habe übrigens einen 1.05 1b Shop den ich betreue wo die Sache mit der Anzeige MODULE_SHIPPING_FREEAMOUNT_TEXT_WAY funktioniert.

Das kann ich auch bestätigen.

Beste Grüße,
Wayne

[noRiddle (revilonetz)]

Jau, da war nämlich dieser Tomcraft-Einschub - vermeintlich wegen PayPal-Express - noch nicht drin.

Habe schon das Team informiert und web28 angeschrieben wegen des Problems, bislang jedoch keine Rückmeldung.

Gruß,
noRiddle

[WayneTsun]

Die sind halt dolle im Stress, glaub ich. 

[Tomcraft]

Der Fehler wurde bereits in r4848 korrigiert.

Grüße

Torsten

[piru]

Dann nochmal die Frage,

Sollte nicht in BUGFIXES modified 1.6 ergänzt werden?

Gruß piru

[Archetim]

Der Fehler wurde bereits in r4848 korrigiert.

Grüße

Torsten

Hallo,

aus aktuellem Anlass bezüglich eines Kundenprojektes habe ich mich an diesen Artikel erinnert. Schade finde ich es nur, das es , wie früher , keine Möglichkeit mehr gibt, die Änderungen im Code einzusehen. Eventuell ist ja mehr "verbessert/angepasst" worden, als aus dem Tread ersichtlich.

Grüße
Rene

[Tomcraft]

Anbei die geänderten Dateien des Changesets r4848.

Grüße

Torsten

[Archetim]

Hi Tomcraft,

danke für die rasche Antwort. Bin leider erst jetzt dazu gekommen, nachzusehen.

Grüße
Rene