Modify

Opened 12 years ago

Last modified 6 years ago

#131 reopened Bug/Fehler

Fehler im Couponmodul bei Kombination Versandkostenfrei & Prozentwert

Reported by: voodoopupp Owned by: Ronald Parcinski
Priority: hoch Milestone: modified-shop-2.1.0.0
Component: Shop Version: 2.0.2.1
Keywords: Cc:
Blocked By: Blocking:

Description

Das Couponmodul hat leider einen nicht unerheblichen Fehler:
wenn man einen Coupon mit Prozentwert und Versandkostenfrei anlegt, "verliert" das System die Prozenteinstellung.

Kleines Beispiel: wenn man einen Coupon Wert von 5% anlegt und dann ebenfalls "versandkostenfrei" einstellt werden aus den 5% dann 5 EUR.

Attachments (0)

Change History (19)

comment:1 by Torsten Riemer, 12 years ago

Ich kann das Problem auch im aktuellen Demoshop bestätigen.

comment:2 by Ronald Parcinski, 12 years ago

Owner: set to Ronald Parcinski
Status: newaccepted

Prozent Coupon(P) und Versandkostenfrei(S) geht nicht. Es müsste ein entsprechender Hinweistext ergänzt werden.

Oder ein ganz neuer Coupontyp eingeführt werden

comment:3 by Gerhard Waldemair, 12 years ago

Milestone: modified-shop-2.00modified-shop-2.10

comment:4 by Gerhard Waldemair, 11 years ago

Version: 1.062.10

comment:5 by Torsten Riemer, 9 years ago

Component: ModuleShop
Milestone: modified-shop-2.1.0.0modified-shop-2.0.2.2
Version: 2.0.1.02.0.2.1

in reply to:  2 comment:6 by Torsten Riemer, 9 years ago

Replying to web28:

Prozent Coupon(P) und Versandkostenfrei(S) geht nicht. Es müsste ein entsprechender Hinweistext ergänzt werden.

Oder ein ganz neuer Coupontyp eingeführt werden

Wieso lässt sich Versandkostenfrei mit prozentualem Coupon eigentlich nicht verbinden? geht nicht, gibt's nicht. ;-)

comment:7 by Ronald Parcinski, 9 years ago

Wie schon geschrieben:

Das geht nur mit einem neuen Coupontyp

Diese Typen gibt es:

Wert Coupon(F)
Prozent Coupon(P)
Versandkostenfrei(S)

Der (Buchstabe) wird in der DB gespeichert.

Versandkostenfrei(S) gibt es die Sonderprogrammierung das auch der Couponwert (wenn > 0) als Absolutwert mitberücksichtigt wird.

Mit einen Hinweistext das die Kombi nicht geht, gäbe es auch keine Probleme.

comment:8 by Torsten Riemer, 9 years ago

Dann wäre es wohl sinnvoll die Coupons wie folgt zu erweitern:

Wert Coupon(F)
Wert Coupon inkl. Versandkostenfrei(FS)
Prozent Coupon(P)
Prozent Coupon inkl. Versandkostenfrei(PS)
Versandkostenfrei(S)

comment:9 by Ronald Parcinski, 9 years ago

Ich sehe gerade das der Uralttext auch noch nicht geändert wurde:

Coupon für eine versandkostenfreie Lieferung. Achtung: Der Coupon Wert wird nicht berücksichtigt! Der Mindestbestellwert bleibt gültig.

Nach diesem Text würde Wertcoupon UND Versandkostenfrei ja gar nicht funktionieren.

comment:10 by Torsten Riemer, 9 years ago

Demnach gibt es zwei Möglichkeiten:

1.) Zusätzlichen Coupon Typ anlegen für Prozent Coupon mit Versandkostenfrei (beispielhaft der Buchstabe "T"), also:

Wert Coupon(F)
Prozent Coupon(P)
Prozent Coupon inkl. Versandkostenfrei(T)
Versandkostenfrei(S)

2.) Oder so wie ich in Kommentar 8 bereits schrieb wir erweitern das Feld für Coupon Typ auf 2 Zeichen und können dann den Coupon Typ auch zerlegen:

Wert Coupon(F)
Wert Coupon inkl. Versandkostenfrei(FS)
Prozent Coupon(P)
Prozent Coupon inkl. Versandkostenfrei(PS)
Versandkostenfrei(S)

Bei Lösung 2 ist dann darauf zu achten, dass beim Update der Shopsoftware alle Coupons vom Typ S, die einen zusätzlichen absoluten Wert haben auf Coupon Typ FS umgestellt werden müssen.

comment:11 by Torsten Riemer, 9 years ago

Milestone: modified-shop-2.0.2.2modified-shop-2.0.2.3

comment:12 by Ronald Parcinski, 9 years ago

Ich bin für Lösung 1) coupon_type = 'T' : coupon_amount percent and shipping_free

comment:13 by Torsten Riemer, 9 years ago

Am Ende des Tages kommt es ja auf's gleiche hinaus, richtig? Der Wert Versandkostenfrei(S) Coupon kann ja auch zusätzlich gleichzeitig noch einen Coupon-Wert enthalten, der berücksichtigt wird.

comment:14 by Ronald Parcinski, 9 years ago

Na da sind schon Unterschiede:
keine DB Erweiterung,
keine Änderung der bisherigen Typen, dadurch keine Notwendigkeit für Updates der bereits vorhandenen Coupons

Und zu guter Letzt hatte ich das 2011 bereits im dem neuen Typ 'T' umgesetzt.

Last edited 9 years ago by Torsten Riemer (previous) (diff)

comment:15 by Ronald Parcinski, 9 years ago

Resolution: fixed
Status: acceptedclosed

In 10754:

fix #131

comment:16 by Torsten Riemer, 9 years ago

In 10755:

Fix #131

comment:17 by Torsten Riemer, 9 years ago

Milestone: modified-shop-2.0.2.3modified-shop-2.0.2.4
Resolution: fixed
Status: closedreopened

Ich öffne das Ticket nochmal und verschiebe es nach 2.0.2.4, denn folgendes sollten wir noch ändern:

  • Die "popup_coupon_help.php" sollte beim Aufruf aus dem Warenkorb auch anzeigen, wenn ein Coupon Versandkostenfreiheit gewährt.
  • Im Checkout sollte "MODULE_ORDER_TOTAL_COUPON_TITLE (Rabatt Coupons)" noch erweitert werden, so dass dem Käufer klar ist, dass die entstandenen Versandkosten im Coupon gegen gerechnet werden.

comment:18 by Torsten Riemer, 8 years ago

Milestone: modified-shop-2.0.4.0

comment:19 by Gerhard Waldemair, 6 years ago

Milestone: modified-shop-2.1.0.0

Modify Ticket

Action
as reopened The owner will remain Ronald Parcinski.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.