stiftungswo / Dime

DimeERP - Timetracker, Offers & Invoices
MIT License
5 stars 1 forks source link

146 symfony upgrade - zwischenstand in master mergen #148

Closed demianh closed 6 years ago

demianh commented 6 years ago

Die deprecated Warnungen aus dem Dime Repo hab ich entfernt, so dass man einfacher auf Symfony 3+ upgraden kann. Was noch fehlt sind die Money Bundles, die nicht mehr kompatibel sind. Leider kann man die nicht einfach upgraden, wahrscheinlich muss man sie ersetzen. Ich habe dies versucht, aber leider war es nicht so einfach, da diese in die SWO Organisation geklont wurden und nicht kompatible Changes mit dem originalen Repo gemacht wurden. Ich wollte dies in meinen letzten Tagen auch nicht mehr machen, da die Testabdeckung für die Dinge wie das Moneybundle nicht gut genug sind und ich dann allenfalls ein kaputtes Dime hinterlassen würde. Man würde zwar finden, wenn der Code nicht mehr läuft (falsche Codeaufrufe, Signaturen oder Imports), aber nicht, wenn er falsch läuft bzw. die Ausgaben nicht stimmen.

Darum schlage ich vor, vorerst einfach mal die Deprecated Warning Changes in den Master zu mergen. Diese sollten eigentlich keine Probleme machen.

codecov[bot] commented 6 years ago

Codecov Report

Merging #148 into master will increase coverage by <.01%. The diff coverage is 72.41%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #148      +/-   ##
============================================
+ Coverage     72.73%   72.73%   +<.01%     
+ Complexity     1750     1723      -27     
============================================
  Files           166      166              
  Lines          5270     5215      -55     
============================================
- Hits           3833     3793      -40     
+ Misses         1437     1422      -15
Impacted Files Coverage Δ Complexity Δ
...etrackerBundle/Controller/ActivitiesController.php 83.33% <ø> (ø) 7 <0> (ø) :arrow_down:
src/Swo/CommonsBundle/Form/Type/PhoneFormType.php 0% <0%> (ø) 2 <1> (-1) :arrow_down:
...Dime/InvoiceBundle/Form/Type/CostGroupFormType.php 0% <0%> (ø) 2 <1> (-1) :arrow_down:
.../Dime/EmployeeBundle/Form/Type/HolidayFormType.php 0% <0%> (ø) 2 <1> (-1) :arrow_down:
...nvoiceBundle/Form/Type/InvoiceDiscountFormType.php 0% <0%> (ø) 2 <1> (-1) :arrow_down:
...me/TimetrackerBundle/Controller/DimeController.php 40% <0%> (ø) 5 <0> (ø) :arrow_down:
...me/InvoiceBundle/Form/Type/InvoiceItemFormType.php 0% <0%> (ø) 2 <1> (-1) :arrow_down:
...voiceBundle/Form/Type/InvoiceCostgroupFormType.php 0% <0%> (ø) 2 <1> (-1) :arrow_down:
...Dime/EmployeeBundle/Form/Type/EmployeeFormType.php 0% <0%> (ø) 2 <1> (-1) :arrow_down:
...e/TimetrackerBundle/Form/Type/ActivityFormType.php 100% <100%> (ø) 2 <1> (-1) :arrow_down:
... and 20 more

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update e97bdc1...bbf87f5. Read the comment docs.