In Bearbeitung [WAWI-50281] Schwerer Fehler in dot liquid führt zu Fehlberechnungen - Typen Deklaration Problem

  • Temporäre Senkung der Mehrwertsteuer Hier findet ihr gesammelt alle Informationen, Videos und Fragen inkl. Antworten: https://forum.jtl-software.de/threads/mehrwertsteuer-senkung-vom-01-07-31-12-2020-offizieller-diskussionthread-video.129542/

John

Sehr aktives Mitglied
3. März 2012
1.466
163
Berlin
Seit ca. Wawi 1.5.30.1 gibt es einen Bug in Dotliquid.

Code:
{% assign zahl = 100 -%}
{% assign faktor = 0.01 -%}\

{% assign ergebnis_1 = faktor | Times: zahl  -%}\
{% assign ergebnis_2 = zahl | Times: faktor  -%}\
{{ ergebnis_1 }} = {{ ergebnis_2 }}
Dieser Code liefert 1 = 0

Offenbar ist der Typ der Ergebnis Variable nur noch anhängig vom ersten Faktor. BÖSE. Ticket mache ich gleich auf...

John
 

John

Sehr aktives Mitglied
3. März 2012
1.466
163
Berlin
Addition ist ebenfalls betroffen

Code:
{% assign zahl = 100 -%}
{% assign faktor = 0.01 -%}\

{% assign ergebnis_1 = faktor | Plus: zahl  -%}\
{% assign ergebnis_2 = zahl | Plus: faktor  -%}\
{{ ergebnis_1 }} = {{ ergebnis_2 }}
100,01 = 100

...vermutlich alle Rechenarten.
 

John

Sehr aktives Mitglied
3. März 2012
1.466
163
Berlin
Es hilft auch nicht, die Variablen durch erstmalige Initialisierung auf einen passenden Datentyp zu zwingen.

Wie kann das sein? Wird der Datentyp im nachhinein auf Integer umdeklariert? Wird ja immer schlimmer!

Code:
{% assign zahl = 100 -%}
{% assign faktor = 0.01 -%}\

{% assign ergebnis_1 = 0.1 -%}
{% assign ergebnis_2 = 0.1 -%}

{{ ergebnis_1 }} = {{ ergebnis_2 }}

{% assign ergebnis_1 = faktor | Plus: zahl  -%}\
{% assign ergebnis_2 = zahl | Plus: faktor  -%}\

{{ ergebnis_1 }} = {{ ergebnis_2 }}
liefert

0,1 = 0,1

100,01 = 100
 

Dominic Nelle

Moderator
Mitarbeiter
18. Januar 2016
165
56
Code:
{% comment -%}
Hier ist "zahl" ein Integer.
{% endcomment -%}
{% assign zahl = 100 -%}
{% assign faktor = 0.01 -%}\
{% assign ergebnis_1 = faktor | Times: zahl  -%}\
{% assign ergebnis_2 = zahl | Times: faktor  -%}\
{{ ergebnis_1 }} = {{ ergebnis_2 }}

{% comment -%}
Hier ist "zahl" ein Double.
{% endcomment -%}
{% assign zahl = 100.00 -%}
{% assign faktor = 0.01 -%}\
{% assign ergebnis_1 = faktor | Times: zahl  -%}\
{% assign ergebnis_2 = zahl | Times: faktor  -%}\
{{ ergebnis_1 }} = {{ ergebnis_2 }}
So geht es ;)
 

John

Sehr aktives Mitglied
3. März 2012
1.466
163
Berlin
@Dominic Nelle danke für das Beispiel, aber es lässt sich so nicht in die Praxis übertragen, weil man nicht sicherstellen kann, daß alle Variablen einer Berechung als Double initialisiert werden.

Beispiel aus einem Workflow im Bereich Rechung
Code:
{% assign WarenWertNettoGesamt = 0 -%}\
{%- for pos in Vorgang.Auftrag.Positionen -%}\
{% assign WarenWertNettoGesamt = WarenWertNettoGesamt | Plus: pos.NettopreisGesamt  -%}\
{%- endfor -%}\

{{ WarenWertNettoGesamt }}
pos.NettopreisGesamt kommt mit Sicherheit nicht Integer daher. Trotzdem wird er als solcher aufsummiert. Das hier erzeugt im Ergebnis einen Rundungsfehler

Ich finde, den Bug eindeutig. Aktuell wird das Kommutativgesetz der Mathematik verletzt und das geht nicht.
In einer Umgebung mit expliziter Typendeklaration, ja, da bin ich selbst für korrekte Typen verantwortlich. Das ist aber in dotliquid nicht der Fall. Ich kann gar nicht deklarieren. Also müßt ihr Euch darum kümmern und zwar korrekt.

Viele Grüße

John
 

daniel.barredo.diaz

Moderator
Mitarbeiter
1. Februar 2016
21
6
Hi John,

pos.NettopreisGesamt kommt mit Sicherheit nicht Integer daher
Schau dir das Beispiel von Dominic nochmal an. Es geht in erster Linie nicht um die Variable dir wir zur Verfügung stellen.
Der von dir deklarierte "WarenWertNettoGesamt" = 0 ist der Integer der hier Probleme bereitet. Versuch die gleiche Berechnung bitte einmal mit 0.00 als initialem Wert.

weil man nicht sicherstellen kann, daß alle Variablen einer Berechung als Double initialisiert werden.
Dafür gibt es in DotLiquid einen eigenen Filter: ToDouble https://guide.jtl-software.de/jtl-wawi/vorlagen/filter-in-dotliquid/#als-zahl-todouble


Also müßt ihr Euch darum kümmern und zwar korrekt
Hier würde ich dir sogar Recht geben, wenn wir Einfluss darauf hätten. Wäre DotLiquid meine eigene Library würde ich einiges anders machen.
In der JTL-Wawi verwenden wir aktuell DotLiquid in der Version 1.7.0.0
Siehe dazu auch:
http://dotliquidmarkup.org/
https://github.com/dotliquid/dotliquid


Nach meinem Verständnis ist Liquid eine Template-Sprache für Shopify-Themes. (https://shopify.github.io/liquid/)
Die .Net-Portierung DotLiquid gibt unseren Kunden eine einfach zugängliche Möglichkeit E-Mail und Designvorlagen dynamisch zu gestalten.
Für die meisten Textoperationen in Workflows oder dem Amazon Lister ist sie auch ausreichend.

Falls du allerdings mit dem ToDouble-Filter nicht zufrieden bist gibt es immer noch die Möglichkeit auf SQL auszuweichen.

Mit DirectQuery oder DirectQueryScalar könntest du Berechnungen in SQL durchführen und das Ergebnis dann nochmal in Liquid behandeln.
Mir selbst fallen Berechnungen in SQL auch wesentlich leichter als mit umständlichen Filtern und nicht klar erkennbaren Datentypen in Liquid.


Viele Grüße,
Daniel
 

John

Sehr aktives Mitglied
3. März 2012
1.466
163
Berlin
@daniel.barredo.diaz Danke für Deine Ausführungen aber was mache ich, wenn ich einen Variable, die durch Euch als Integer bereitgestellt wird (z.B. Anzahl der Positionen im Auftrag) mit einer Fließkommazahl verrechnen muß? Bsp:

Code:
{% assign test = 0.0 -%}
{% assign test = Vorgang.Auftrag.Positionen.Anzahl | Plus: 0.01 -%}
Funktioniert nicht. Ja, ich kann die Summanden hier umdrehen. Aber das kann es ja wohl nicht sein.
Dein Vorschlag mit ToDouble erwartet als Eingabe einen String. Soll ich den Integer in einen String konvertieren, um ihn dann in einen Double zu wandeln?!?

Bitte beachte auch: Das Problem gab es z.B. mit Wawi 1.5.26.0 noch nicht. Aus meiner Sicht hat sich da ein Bug eingeschlichen, ob nun bei Euch oder in der externen Bibliothek.

Viele Berechnungen, die es hier im Forum als Beispiele gibt oder die draussen bei Kunden laufen, werden Fehler nach dem Wawi Update liefern.

Ich melde mich ja hier auch nur, weil es bei Kunden von mir gekracht hat und vorher lief.

Viele Grüße

John
 

daniel.barredo.diaz

Moderator
Mitarbeiter
1. Februar 2016
21
6
Hi John,

mir sind keine Änderungen bekannt. Die DotLiquid 1.7.0.0 verwenden wir schon ewig.
Auch an dem Feld Anzahl hat sich nichts geändert.

Viele Berechnungen, die es hier im Forum als Beispiele gibt oder die draussen bei Kunden laufen, werden Fehler nach dem Wawi Update liefern
Das wäre nicht gut. Ich werde das mal in der 1.5.26.0 nachstellen.
Du sagst dein erster Beitrag in diesem Post lieferte in der alten Version noch das richtige Ergebnis?

Btw. das Ticket finde ich leider in unserem System nicht. Könntest du nochmal darauf antworten?


Viele Grüße,
Daniel
 

T4DT.GmbH

Offizieller Servicepartner
SPBanner
6. November 2018
262
67
Hannover

John

Sehr aktives Mitglied
3. März 2012
1.466
163
Berlin
Hier noch deutlicher:

Code:
{% assign test = 0.01 -%}
{% assign test = Vorgang.Auftrag.Positionen.Anzahl | Plus: test | Plus: 0.01 -%}
{{test}}
berechnet in betroffenen Versionen falsch. Hingegen

Code:
{% assign test = 0.01 -%}
{% assign test = test | Plus: Vorgang.Auftrag.Positionen.Anzahl | Plus: 0.01 -%}
{{test}}
berechnet auch in betroffenen Versionen richtig.

@T4DT.GmbH und wie addiert man dann einen int und einen double, außer die Reihenfolge der Summanden entsprechend halten?
Btw. die Operation wird ja nicht auf dem int ausgeführt, sondern in meinem Beispiel auf der Variablen "Test", die in der ersten Zeile als Double initialisiert wurde. Durch die Verrechnung mit dem int wird sie aber vom Double zum int und das ist imho fatal.

@daniel.barredo.diaz Alle meine Beispiele habe ich eben mit 1.5.24.1 getestet. Funktioniert darin. Mit 1.5.26.x gehts imho auch noch, habe ich ber nicht mehr zur Hand. Die Änderung ist irgendwann am danach.

Ticket#2020101910002763 - offen seit gestern. Bei der Nennung oben fehlte die letzte Ziffer.

Viele Grüße

John
 
Zuletzt bearbeitet:

daniel.barredo.diaz

Moderator
Mitarbeiter
1. Februar 2016
21
6
Seit ca. Wawi 1.5.30.1 gibt es einen Bug in Dotliquid.

Code:
{% assign zahl = 100 -%}
{% assign faktor = 0.01 -%}\

{% assign ergebnis_1 = faktor | Times: zahl  -%}\
{% assign ergebnis_2 = zahl | Times: faktor  -%}\
{{ ergebnis_1 }} = {{ ergebnis_2 }}
Dieser Code liefert 1 = 0

Offenbar ist der Typ der Ergebnis Variable nur noch anhängig vom ersten Faktor. BÖSE. Ticket mache ich gleich auf...

John

Guten Morgen,

ich habe das Skript mal in der 1.5.24.1 und der zukünftigen 1.5.33.0 verglichen.

Ergebnis in 1.5.24.1: 1,00 = 1,00
Ergebnis in 1.5.33.0: 1 = 0

Für die Version 1.5.28.0 wurden die Filter in DotLiquid erweitert.
Möglicherweise hatte diese Anpassung ein paar ungewünschte Nebeneffekte. Die genaue Ursache kann ich dir noch nicht sagen.

Meine Kollegen prüfen das Problem bereits und sobald ich ein passendes Ticket habe wird hier im Thread natürlich aktualisiert.

@John Vielen Dank für den Hinweis auf das Problem!
Meine Workarounds helfen dir eventuell trotzdem erstmal weiter.
Mit
{% assign zahl = 100.00 -%}
bekomme ich aktuell 1 = 1 als Ergebnis.

Viele Grüße,
Daniel
 
Ähnliche Themen Forum Antworten Erstelldatum des Themas
Neu ziemlich genervt! Update 4 auf 5 immer wieder Fehlermeldungen in der WAWI Upgrade JTL-Shop4 auf JTL-Shop5 0
Neu Kundengruppen werden von JTL Wawi nicht nach JTL Shop 5 übertragen Onlineshop-Anbindung 1
Neu Support unserer Kunden bzgl. JTL WaWi Dienstleistung, Jobs und Ähnliches 0
Neu JTL WaWi mit VPN vom HomeOffice verbinden, aber wie? User helfen Usern - Fragen zu JTL-Wawi 2
Neu Fehler bei Verknüpfung Wawi zu SQL Datenbank von JTL Installation von JTL-Wawi 0
Neu WAWI 1.5.40.0: Paket-Status wird nicht abgerufen JTL-Track&Trace - Fehler und Bugs 0
Neu Shop ohne WaWi JTL-Shop - Ideen, Lob und Kritik 12
Neu EDI in JTL Wawi integrieren (Aufwand und Kosten) User helfen Usern - Fragen zu JTL-Wawi 0
Neu Artikel Lagerbestand stimmt bei Amazon nicht mit JTL WAWI User helfen Usern - Fragen zu JTL-Wawi 0
Neu Gast Bestellungen werden nicht an Wawi übermittelt WooCommerce-Connector 0
In Bearbeitung Einlagerungsliste für Retouren nutzen [WAWI-35018] Arbeitsabläufe in JTL-WMS / JTL-Packtisch+ 1
Beantwortet Ab wann kommt 1.6 Wawi? User helfen Usern - Fragen zu JTL-Wawi 1
Neu Farbliche Markierung in der Wawi User helfen Usern - Fragen zu JTL-Wawi 0
Neu Lokale Testumgebung DB/Wawi Installation von JTL-Wawi 0
Neu Fehler beim Update der WAWI auf eine Version ab 1.5.40 Installation von JTL-Wawi 6
Neu WAWI Stürzt ab und Backup geht nicht mehr JTL-Wawi - Fehler und Bugs 1
In Bearbeitung Fehlerhaftes Kassenbuch-Rabatte aus Wawi werden nicht berücksichtig JTL-POS - Fehler und Bugs 3
Neu Installation von JTL -WAWI - Hänge beim SQL Installation von JTL-Wawi 6
Gelöst WAWI 1.0.11.8 Artikel mit Sonderpreisen exportieren? User helfen Usern - Fragen zu JTL-Wawi 10
Neu JTL WAWI 1.4.5.40.0, JTL Shop 5 Nova-Template User helfen Usern - Fragen zu JTL-Wawi 0
Kann man Updaten wenn man die Wawi 1.6 nicht hat Upgrade JTL-Shop4 auf JTL-Shop5 1
Neu Versandart ps_wirepayment wird immer wieder an WaWi übertragen PrestaShop-Connector 1
Neu In der WAWI sind 6 Rechnungen nicht mehr aufzufinden, einfach weg User helfen Usern - Fragen zu JTL-Wawi 10
Neu WaWi-Server (W2k11) von Schadsoftware verschlüsselt / Backup auf NAS (Filesystem) vorhanden - was nun? Installation von JTL-Wawi 4
Neu Es werden Keine Paypalzahlungen mehr an die WaWi übertragen JTL-Shop - Fehler und Bugs 3
Neu Kein Zugriff auf die Wawi User helfen Usern - Fragen zu JTL-Wawi 2
Neu Datenaustausch WaWi Shop oder doch Internet Allgemeine Fragen zu JTL-Shop 1
Neu Umstellung auf JTL Wawi von Virtuemart Starten mit JTL: Projektabwicklung & Migration 0
Neu JTL shop und JTL WAWI auf rootserver einrichten Starten mit JTL: Projektabwicklung & Migration 1
Gelöst Amazon angebote lassen sich nicht dem Wawi Artikel zuordnen Amazon-Anbindung - Fehler und Bugs 1
Neu Erster Lieferscheindruck nach Start der JTL-Wawi: Fehler JTL-Wawi - Fehler und Bugs 0
In Bearbeitung Artikel werden aus der Wawi nicht in der POS aktualisiert. Allgemeine Fragen zu JTL-POS 5
Neu Lieferzeiten aus der Wawi werden bei Stücklisten nicht übernommen JTL-Shop - Fehler und Bugs 1
Neu Artikedaten WaWi neuen Mandanten User helfen Usern - Fragen zu JTL-Wawi 1
Neu WaWi Shop5 User helfen Usern - Fragen zu JTL-Wawi 4
Neu WaWi -> shop 5.0 abgleich Onlineshop-Anbindung 0
In Bearbeitung Rechnung aus JTL-POS im JTL Wawi Format drucken (Rechnung) JTL-POS - Fehler und Bugs 1
Neu JTL-Wawi 1.5.38.1 - EVO-Designvorlage - Layout von "GalerieContainer" anpassen, wo und wie? eBay-Designvorlagen - Fehler und Bugs 0
In Bearbeitung Retoure im anderen Zustand aufnehmen und barcodieren - Artikeletikett immer noch vom orginal Artikel [WAWI-33115] JTL-WMS / JTL-Packtisch+ - Fehler und Bugs 2
Neu WAWI Stücklisten JTL-Wawi - Ideen, Lob und Kritik 1
In Bearbeitung WMS Mobile App Problem bei der Anzeige für Eingabe MHD in der Horizontalen [WAWI-53300] JTL-WMS / JTL-Packtisch+ - Fehler und Bugs 4
Neu Wie erstelle ich eine XRechnung in JTL-Wawi & wo finde ich die Leitweg-ID (Video) Umfragen rund um JTL 5
In Diskussion Aktueller DHL Tracking-Link mit Variablen für JTl Wawi JTL-ShippingLabels - Ideen, Lob und Kritik 2
Neu Lager-Packliste Packliste Wawi-Picklisten Unterschied Arbeitsabläufe in JTL-Wawi 0
Neu ebay Managed PAyments - Zahlung nicht in WaWi eBay-Anbindung - Fehler und Bugs 3
In Bearbeitung WAWI-53430 - Konfigurationsgruppen bzw. Artikel JTL-POS - Fehler und Bugs 3
Beantwortet Kategorien im Online-Shop unabhänging von Wawi Allgemeine Fragen zu JTL-Shop 4
Neu JTL WAWI>German Market>WooCommerce Anleitung WooCommerce-Connector 1
Neu JTL Wawi - Shopware - Subshops mit unterschiedlichen Beschreibungen Shopware-Connector 12
Neu Bilderupload von WAWI auf Gambio Shop führt zu Fehlermeldung JTL-Wawi - Fehler und Bugs 0
Ähnliche Themen