Bug ? in Versandmodul freeamount = versandkostenfrei

[noRiddle (revilonetz)]

Hallo Community, hallo Entwickler, hallo Team.

Folgendes ist mir in der letzten Version (1.06 rev 4642) aufgefallen:
Wenn man im Backend bei Module => Versandart freeamount aktiviert
und dort MODULE_SHIPPING_FREEAMOUNT_DISPLAY auf 'true' setzt

(Anzeige aktivieren
Möchten Sie anzeigen, wenn der Mindestbetrag zur VK-freien Lieferung nicht erreicht ist?)

sollte auf der checkout_shipping -Seite doch der Text erscheinen der für
MODULE_SHIPPING_FREEAMOUNT_TEXT_WAY in /lang/german/modules/shipping/freeamount.php
definiert ist.

Dies geschieht jedoch nicht.
Ich blicke bei den ganzen quote(), quote[] und quotes[] noch nicht ganz durch und finde den Grund deshalb nicht.
Wo ist der Fehler ?

Gruß,
noRiddle

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

[fishnet]

Mal kurz auf den Testshop geguckt:
wenn ich MODULE_SHIPPING_FREEAMOUNT_DISPLAY auf true setze, wird in der checkout shipping der kostenlose Versand eingeblendet mit dem Hinweis "ab x EUR Bestellwert versenden wir Ihre Bestellung versandkostenfrei".

Wenn ich auf false stelle, verschwindet die komplette Versandart aus der Liste, sobald der Bestellwert nicht erreicht wurde.

Sieht doch korrekt aus ?

[noRiddle (revilonetz)]

Korrekt.
Der Demo-Shop ist aber keine Referenz da er schon einige Änderungen implementiert hat.
Frage ist wie es bei einem aktuellen 1.06 rev 4642 aussieht.
Kann das mal jemand verifizieren ?

Bei mir jedenfalls wird folgender Befehl aus der /includes/modules/shipping/freeamount.php nicht beachtet:

Code: PHP  [Auswählen]

if ( $xtPrice->xtcRemoveCurr($_SESSION['cart']->show_total()) < MODULE_SHIPPING_FREEAMOUNT_AMOUNT )
        $this->quotes['error'] = sprintf(MODULE_SHIPPING_FREEAMOUNT_TEXT_WAY,$xtPrice->xtcFormat(MODULE_SHIPPING_FREEAMOUNT_AMOUNT,true,0,true));
      else
...

Ich habe aber noch nicht durchgeblickt wo die Ausgabe von quotes['error'] implementiert ist.
In der checkout_shipping.php  scheint mir etwas zu fehlen....

Gruß,
noRiddle

[WayneTsun]

Hi noRiddle,

ich kann das hier auch nachvollziehen in 1.06 rev. 4642. Bei nicht erreichtem Minimumbetrag für versandkostenfreie Lieferung wird diese Versandart bei mir im checkout_shipping gar nicht angezeigt. Entweder haben wir beide die gleichen Änderungen/Module bereits installiert, oder es ist ein Bug, der im Testshop nicht mehr existiert.

Evtl. ist der Sinn des Testshops auch nicht, diesen ohne neue Revisionsnummer zu ändern - also wenn das passiert sein sollte. 

Beste Grüße,
Wayne

[noRiddle (revilonetz)]

Danke für das Feedback.
Die Versandart soll auch nicht angezeigt werden sondern der Satz (default-Text)
"ab SUMME  Bestellwert versenden wir Ihre Bestellung versandkostenfrei"
aber das meinst du ja auch, oder ?

Was die Version im Demoshop anbelangt hast du Recht.
Allein wenn du mal unter Module => Versandart schaust sieht alles neu aus.
Die Version ist aber nach wie vor rev 4642.
Etwas verwirrend das ganze.

Gruß,
noRiddle

[WayneTsun]

Die Versandart soll auch nicht angezeigt werden sondern der Satz (default-Text)
"

Ja, das meinte ich.

ref. Revision...lässt nur vermuten/hoffen, dass es bald eine neue Revision geben wird und es schon im Testshop eingebaut wurde. - Aber richtig, etwas verwirrend ist das schon.

Beste Grüße,
Wayne

[jannemann]

Guten Abend,

ich habe das in Ticket #38 im Bug-Tracker festgehalten.

Schöne Grüße,
Jan

[noRiddle (revilonetz)]

Merci.

Gruß,
noRiddle

[Koala]

$this->quotes['error'] bezieht sich auf die Klasse freeamount.

Dort fehlt also:

Code: PHP  [Auswählen]

  class freeamount {
    var $code, $title, $description, $icon, $enabled;

    var $quotes = array();
 

Außerdem fehlt sort_order.

Also:

Code: PHP  [Auswählen]

  class freeamount {
    var $code, $title, $description, $icon, $sort_order, $enabled;

    var $quotes = array();
 

Wieso überhaupt "var".

Sollen die Module denn noch PHP 4 - kompatibel sein ?

[noRiddle (revilonetz)]

Irgendwo muß $this->quotes['error'] ja ausgegeben werden.
Das klappt aber nicht; und wo überhaupt ?,
wenn, doch dann in der checkout_shipping.php.

Der Sinn deines Posts, mit Verlaub, ist mir nicht klar.
Das löst doch das Problem nicht.

Gruß,
noRiddle

[Koala]

Das löst doch das Problem nicht.

sicher, aber wir wollen doch nicht so einen schlampigen Code mitschleppen.

Hab ein paar Sachen zu freeamount gelesen.

Hast Du das hier gesehn ?
Lieferung versandkostenfrei - FreeShippingData für xt:Commerce

Das müßte man downloaden und anpassen.

 

[fishnet]

Evtl. ist der Sinn des Testshops auch nicht, diesen ohne neue Revisionsnummer zu ändern - also wenn das passiert sein sollte. 

[Modulfux]

Außerdem fehlt sort_order.

Dann würde das aber in allen Versand- und Bezahlmodulen fehlen. Also halte ich diese Aussage für etwas vage.

Gruß
Ronny

[Koala]

ja, fehlt auch in allen.

sort_order wird beim Aufruf von $this->set_order trotzdem initialisiert.

Weiß nicht warum das immer weggelassen wurde.

Vermutlich hat da mal jemand gedacht,
daß set_order wie eine globale Variable verfügbar sei.
Aber das ist natürlich nonsense.

[noRiddle (revilonetz)]

Naja, das sind doch alles Relikte aus dem alten xtc-System.
Im Prinzip müssten alle Klassen umgeschrieben werden und Richtung OOP gehen, also private, protected, public und static Deklarationen benutzen. Dann wäre auch das deprecated var verschwunden.
Kann sich ja mal jemand dran machen .

Habe übrigens einen 1.05 1b Shop den ich betreue wo die Sache mit der Anzeige MODULE_SHIPPING_FREEAMOUNT_TEXT_WAY funktioniert.
Hatte nur noch keine Zeit das zu analysieren.
Die /checkout_shipping.php in 1.06 rev 4642 weicht deutlich ab von der der 1.05 1b.

Gruß,
noRiddle