TbbcMoneyBundle und money forks ersetzen

stiftungswo / Dime

DimeERP - Timetracker, Offers & Invoices

MIT License

5 stars 1 forks source link

TbbcMoneyBundle und money forks ersetzen #151

Closed andyundso closed 5 years ago

andyundso commented 6 years ago

Genaue Unit-Tests mit Resultaten schreiben für alle Methoden, welche Abhängigkeiten zum Money Bundle aufweisen.
Zu den aktuellen Stables vom TbbcMoneyBundle und money wechseln.
Ggf. roundto5or0 erneut implementieren für die Totale.

andyundso commented 6 years ago

Folgende Klassen haben Money (oder Sachen aus dem Money-Bundle) implementiert:

ActivitiesController
Activity
ActivityFormType
Invoice
InvoiceCostgroup
InvoiceDiscount
InvoiceItem
InvoiceFormType
InvoiceItemFormType
print.pdf.twig von Invoice
InvoiceBreakdown
Offer
OfferPositionFormType
OfferFormType " OfferDiscount
OfferPosition
Project
ProjectFormType
ProjectHandler
Rate
RateFormType
ReportHandler

andyundso commented 6 years ago

Die API-Änderung des neuen money-Bundles sind nicht allzu riesig. Hauptsächlich fehlt schon die 5 Rappen-Runden Mechanik. Ich habe versucht, dies einzubauen, jedoch nicht geschafft, weil money noch zwei zusätzliche Mathematik-Bibliotheken unterstützt, welche etwas umständlich sind.

Ich habe nun einen Issue erstellt bei Money, um nachzufragen, ob es trotzdem irgendwie geht und ich es übersehen habe. Alternativ lässt sich dies schon einfügen, mit etwas mehr oder weniger Zeit.

Alternativ könnten wir auch eine andere Money-Library verwenden, welche dies bereits unterstützt. Dann müssten wir aber in zwei Schritten zuerst alle Datentypen von Money umwandeln zu normalen Integern, bevor wir die originale Money-Library endgültig entfernen können.

andyundso commented 6 years ago

Funktion kommt erst: https://github.com/moneyphp/money/issues/480

andyundso commented 5 years ago

Im Rahmen von #59 wurde eine neue InvoiceBreakdown-Klasse gemäss Instruktionen von Andi erstellt. Momentan rechnen die "Legacy-Methoden" die Sache etwas anders als in der InvoiceBreakdown-Klasse: Während im Invoice-Breakdown die Abzüge von den Positionen ohne Mehrwertsteuer gemacht werden, rechnet die Invoice-Klasse die Abzüge von den Positionen mit der Mehrwertsteuer. Selbiges auch bei der Offerte.

TO-DO:

Nochmals bei Andi verifizieren, dass das Vorgehen beim InvoiceBreakdown korrekt ist.
- Mit Andi besprechen, wie das Update eingespielt werden soll, da die Beträge der Rechnungen teilweise nicht dasselbe sein werden nach dem Update.

andyundso commented 5 years ago

Gemäss Feedback von Andi Änderungen erst nach Neujahr mergen.

Bis dahin To-Do:

Berechnung von Offerten den Rechnungen anpassen (Positionen -> Abzug -> MwSt. anstatt Positionen -> MwSt. -> Abzug).
- Legacy Methoden aus der Invoice / Offerten-Methode entfernen.

andyundso commented 5 years ago

Das neue Dime kommt komplett ohne Money-Bundle aus. Die Werte werden in Rappen gespeichert, gerundet wird am Ende des ganzen Prozesses. Die Berechnung der Offerte gemäss den Rechnungen wurde angepasst, die neuen Offer / Invoice-Models enthalten auch keine konkreten Berechnungen mehr.