2023 fxtransaction bugs 5 #258
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
siehe Ticket #563, Währung und Fremdwährung war vertauscht.
siehe Ticket #563, Währung und Fremdwährung waren vertauscht.
siehe Ticket #563, Währung und Fremdwährung waren vertauscht.
siehe Ticket #563, Währung und Fremdwährung waren vertauscht.
t/fxtransaction/ap_transactions.t enthält ein Mini-Framework für bin/mozilla Integration-Tests. Grobes Muster: # simuliert einen call zu $script.pl?action=$action + form_params my ($out, $err, @ret) = make_request($script, \%form_params, $action); # liest sucht alle <input> und <select> elemente die in der html ausgabe # vorkommen %form = form_from_html($out); Wird hier benutzt um simple Flows durch das programm zu simulieren.
Bei Fremdwährungszahlungen wird jetzt zusätzlich: - fx_transaction gesetzt - defaultcurrency_paid gesetzt
Es wird jetzt das fx_transaction flag durchgereicht, wenn die Zahlung im FGrontend angelegt wurde, oder in Fremdwährung in der Datenbank gefunden wurde. Interne Währungszahlungen werden initial ohne fx_transaction geladen, dann aber wie "normale" Fremdbuchungen durchgereicht
Es wird jetzt das fx_transaction flag durchgereicht, wenn die Zahlung im FGrontend angelegt wurde, oder in Fremdwährung in der Datenbank gefunden wurde. Interne Währungszahlungen werden initial ohne fx_transaction geladen, dann aber wie "normale" Fremdbuchungen durchgereicht
Es wird jetzt das fx_transaction flag durchgereicht, wenn die Zahlung im FGrontend angelegt wurde, oder in Fremdwährung in der Datenbank gefunden wurde. Interne Währungszahlungen werden initial ohne fx_transaction geladen, dann aber wie "normale" Fremdbuchungen durchgereicht Im Gegensatz zu AP/AR wird hier direkt create_links aufgerufen.
Es wird jetzt das fx_transaction flag durchgereicht, wenn die Zahlung im FGrontend angelegt wurde, oder in Fremdwährung in der Datenbank gefunden wurde. Interne Währungszahlungen werden initial ohne fx_transaction geladen, dann aber wie "normale" Fremdbuchungen durchgereicht Im Gegensatz zu AP/AR wird hier direkt create_links aufgerufen.
Dieser Commit implementiert die template Anpassungen von Geoffrey für die design40 templates
|
Erstmal wieder so "funktionsfähig wie vorher auch", Anmerkungen von Sven hier noch ergänzt: |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Rückmeldung von Sven:
Ich habe das jetzt versucht minimalinvasiv zum laufen zu kriegen. Was ich gemacht habe ist:
Getestet und hoffentlich nicht kaputt sind:
Das ist trotzdem alles fragil wie Hölle. Ich fasse die ganzen anderen Randfälle da nicht an. Unter anderem ist mir aufgefallen, dass die Preise beim Laden von Waren in Fremdwährung übersetzt werden, wenn eine Fremdwährung gesetzt wird und schon ein Wechselkurs bekannt ist, aber NICHT wenn der Wechselkurs nicht bekannt ist. Was dazu führt dass wenn man 2x exakt die gleiche Rechnung anlegt, die Artikelpreise auf einmal anders sind.
Beim rumklickern habe ich gleich noch 2 andere Bugs in der Fremdwährungsbehandlung gesehen:
Ich würde nach und nach noch mehr Tests dafür schreiben, bei der Hälfte der Fälle bin ich mir aber nichtmal sicher was das erwartete Verhalten ist. Vor allem bei Teilzahlungen, Steuern, oder Skontobuchungen.