Opened 4 months ago

Closed 4 months ago

#1845 closed Bug/Fehler (fixed)

ot_payment speichert "&" als "&"

Reported by: oneQ Owned by: GTB
Priority: hoch Milestone: modified-shop-2.0.5.2
Component: Module Version: 2.0.5.1

Description

Wenn man im Modul "Rabatt & Zuschlag auf Zahlungsarten" eine Kombination von % und festen Aufschlag oder nur festen Aufschlag angibt (0:0&-1.5), wird das "&-Zeichen" in HTML "&" umgewandelt
aus
DE|0:-1.9&-0.35 wird
DE|0:-1.9&-0.35 und wenn man das bei einer erneuten Aktualisierung übersieht (z.B. weil man einen Schalter vergessen hat zu ändern und das schnell nachholen will), wird daraus
DE|0:-1.9&-0.35 und der Zuschlag / Rabatt wird nicht mehr berechnet.

Im dev-Shop so auch nachgestellt.

By the way, scheint das Modul auch das einzige - oder eins der wenigen (??) - zu sein, bei dem der feste Aufschlag (als Bruttowert übernommen wird, also nicht der Netto-Wert eingegeben werden darf. Sollte evtl. noch als Hinweis dazu.

Attachments (0)

Change History (3)

comment:1 Changed 4 months ago by Tomcraft

  • Priority changed from normal to hoch

Kann ich nachvollziehen. Der Fehler kam in r12513 mit Shopversion 2.0.5.1 rev 12725.

In dem Changeset wurde in den Dateien "/admin/module_export.php" und "/admin/modules.php" an jeweils zwei Stellen ein encode_htmlspecialchars() eingefügt und jeweils ein strip_tags().
Da müssen wir nochmal alle Stellen prüfen auf mögliche Probleme mit den Modulen.

comment:2 Changed 4 months ago by oneQ

Ich bin mir nicht sicher, ob die Überschrift richtig ist. Das Team wird es schon richtig interpretiert haben. Die falsche "Umwandlung" passiert anscheinend beim Lesen aus der DB oder anschl. Anzeige in den Eingabefeldern.
Wenn die Werte richtig in die Felder geschrieben werden, wird auch richtig an die DB übergeben und der Aufschlag entsprechend richtig angezeigt und berechnet.

comment:3 Changed 4 months ago by GTB

  • Owner set to GTB
  • Resolution set to fixed
  • Status changed from new to closed

In 12812:

fix #1845 - prevent ampersand from beeing converted to html entity in modules

Add Comment

Modify Ticket

Action
as closed
The resolution will be deleted. Next status will be 'reopened'
Author


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

 
Note: See TracTickets for help on using tickets.