Opened 8 years ago

Last modified 2 years ago

#131 reopened Bug/Fehler

Fehler im Couponmodul bei Kombination Versandkostenfrei & Prozentwert

Reported by: voodoopupp Owned by: web28
Priority: hoch Milestone: modified-shop-2.1.0.0
Component: Shop Version: 2.0.2.1

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 Changed 8 years ago by Tomcraft

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

comment:2 follow-up: Changed 8 years ago by web28

  • Owner set to web28
  • Status changed from new to accepted

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 Changed 8 years ago by GTB

  • Milestone changed from modified-shop-2.00 to modified-shop-2.10

comment:4 Changed 6 years ago by GTB

  • Version changed from 1.06 to 2.10

comment:5 Changed 4 years ago by Tomcraft

  • Component changed from Module to Shop
  • Milestone changed from modified-shop-2.1.0.0 to modified-shop-2.0.2.2
  • Version changed from 2.0.1.0 to 2.0.2.1

comment:6 in reply to: ↑ 2 Changed 4 years ago by Tomcraft

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 Changed 4 years ago by web28

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 Changed 4 years ago by Tomcraft

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 Changed 4 years ago by web28

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 Changed 4 years ago by Tomcraft

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 Changed 4 years ago by Tomcraft

  • Milestone changed from modified-shop-2.0.2.2 to modified-shop-2.0.2.3

comment:12 Changed 4 years ago by web28

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

comment:13 Changed 4 years ago by Tomcraft

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 Changed 4 years ago by web28

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 4 years ago by Tomcraft (previous) (diff)

comment:15 Changed 4 years ago by web28

  • Resolution set to fixed
  • Status changed from accepted to closed

In 10754:

fix #131

comment:16 Changed 4 years ago by Tomcraft

In 10755:

Fix #131

comment:17 Changed 4 years ago by Tomcraft

  • Milestone changed from modified-shop-2.0.2.3 to modified-shop-2.0.2.4
  • Resolution fixed deleted
  • Status changed from closed to reopened

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 Changed 3 years ago by Tomcraft

  • Milestone modified-shop-2.0.4.0 deleted

comment:19 Changed 2 years ago by GTB

  • Milestone set to modified-shop-2.1.0.0

Add Comment

Modify Ticket

Action
as reopened
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.