#1845 closed Bug/Fehler (fixed)
ot_payment speichert "&" als "&"
| Reported by: | oneQ | Owned by: | Gerhard Waldemair |
|---|---|---|---|
| Priority: | hoch | Milestone: | modified-shop-2.0.6.0 |
| Component: | Module | Version: | 2.0.5.1 |
| Keywords: | Cc: | ||
| Blocked By: | Blocking: |
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 (5)
comment:1 by , 6 years ago
| Priority: | normal → hoch |
|---|
comment:2 by , 5 years ago
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:4 by , 5 years ago
strip_tags() sollte erhalten bleiben.
Das hatte, soweit ich mich erinnere, den Grund, daß ansonsten die Voranzeige in der Box rechts zerschossen wird wenn in einem Modul HTML-tags für einen Wert benutzt werden sollten.
Gruß,
noRiddle
comment:5 by , 5 years ago
*NACHTRAG*
Vielleicht sollte man auch mal darüber nachdenken ob die Verwendung des eval() in /admin/modules.php und ~module_export.php nicht veraltet ist und man das anders lösen sollte.
Gruß,
noRiddle
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.