search

stiftungswo / Dime

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

EditableOverviews extrahieren, Angular ControlArray verwenden #129

Closed tourn closed 6 years ago

tourn commented 6 years ago

Dies bringt etwas mehr Unterteilung in die Overviews. Das Problem aus #124, dass die Formularfelder in Overviews bei Veränderungen manchmal leer werden, sollte dadurch auch behoben werden.

Die neue abstrakte Klasse EditableOverview enthält den Code, der die Controls für das Template im Code generiert. In den Subklassen werden Felder im Template nicht wie zuvor mit [(ngModel)] und (ngModelChange), sondern mit [ngFormControl] gebunden. Überlegung dahinter ist in #124.

Connect #124

codecov[bot] commented 6 years ago

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@1bac13e). Click here to learn what that means. The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #129   +/-   ##
=========================================
  Coverage          ?   47.89%           
  Complexity        ?     1796           
=========================================
  Files             ?      171           
  Lines             ?     5452           
  Branches          ?        0           
=========================================
  Hits              ?     2611           
  Misses            ?     2841           
  Partials          ?        0
Impacted Files Coverage Δ Complexity Δ
src/Dime/InvoiceBundle/Entity/InvoiceCostgroup.php 33.33% <ø> (ø) 6 <0> (?)

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 1bac13e...ff11238. Read the comment docs.