Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[ICT-3198] TAX - Dubbele dwarsverdeling - Caché gedeelte
[ICT-3198] TAX - Dubbele dwarsverdeling - Caché gedeelte
[ICT-3198] TAX - Dubbele dwarsverdeling - Caché gedeelte

- Multiplier voor Montage en Inkorten Dwarsverdeling juist gezet

[ICT-3198] TAX - Dubbele dwarsverdeling - Caché gedeelte

- BOL aangepast voor Dwarsverdeling: indien DwarsverdelingType = Dubbel, dan tijd * 2

    • -0
    • +23
    ./Multiplier/DwarsverdelingType.cls.xml
Ge ziet niets over 't hoofd :-D Ge hebt helemaal gelijk! Is een 1-op-1-property, dus mag automatisch geconverteerd worden. In tegenstelling tot DwarsverdelingAantal, want daar is de property-naam v...

Ge ziet niets over 't hoofd :-D Ge hebt helemaal gelijk! Is een 1-op-1-property, dus mag automatisch geconverteerd worden. In tegenstelling tot DwarsverdelingAantal, want daar is de property-naam verschillend (cq. AantalDwarsverdelingen in APPS).

Aangepast naar "Dwarsverdelingen te picken"

Aangepast naar "Dwarsverdelingen te picken"

Als opmerking in de kantlijn (dus je moet niks aanpassen) : hier moeten we eigenlijk gebruik maken van het berekende HalffabItem en daarvan de (bewerkte) lengte opvragen. Nu staat er duplicate logi...

Als opmerking in de kantlijn (dus je moet niks aanpassen) : hier moeten we eigenlijk gebruik maken van het berekende HalffabItem en daarvan de (bewerkte) lengte opvragen. Nu staat er duplicate logica.
Maar ik geloof best, zonder het na te kijken, dat in deze klasse nog code staat die vertrekt van de ingegevenKenmerken i.p.v. halffabItem.

DwarsverdelingKleur zit volgens mij "impliciet" in de ConverteerGegenereerdeProperties() hiervoor is geen "uitzondering nodig". Tijdens de uitbreidingsfase en review lijkt het me handig om te zien ...

DwarsverdelingKleur zit volgens mij "impliciet" in de ConverteerGegenereerdeProperties()
hiervoor is geen "uitzondering nodig".
Tijdens de uitbreidingsfase en review lijkt het me handig om te zien dat beide properties zijn toegevoegd. Maar eenmaal in productie (en ondersteund door automatische testen) kan dit het standaard pad volgen.
Of zie ik iets over t hoofd?

Tekst correct? Zonder kar --> geen vakken, right?

Tekst correct?
Zonder kar --> geen vakken, right?

Staat al in productie en is voordien al uitgebreid end-to-end getest.

Staat al in productie en is voordien al uitgebreid end-to-end getest.

[ICT-2728] [rvWV] TAX configurator : uitbreiden met een dwarsverdeling:
[ICT-2728] [rvWV] TAX configurator : uitbreiden met een dwarsverdeling:
[ICT-3077] TAX dwarsverdeling - kleine foutjes/aanvullingen
[ICT-3077] TAX dwarsverdeling - kleine foutjes/aanvullingen
[ICT-3077] TAX dwarsverdeling - kleine foutjes/aanvullingen

- Ik had de naam niet volledig overgenomen

[ICT-3077] TAX dwarsverdeling - kleine foutjes/aanvullingen

- Multiplier voor AantalDwarsverdelingen: deze wordt nu genomen van de IK ipv de halffab sinds deze laatste niet altijd natuurlijke getallen bevat

    • -2
    • +2
    ./Multiplier/DwarsverdelingAantal.cls.xml
[ICT-3077] TAX dwarsverdeling - kleine foutjes/aanvullingen

- KostItem Linefeeding Voormontage toegevoegd

[ICT-2728] [rvWV] TAX configurator : uitbreiden met een dwarsverdeling:

- Toevoeging Kostenplaats VoormontageDwarsverdelingEnHouder

- Toevoeging Multiplier voor DwarsverdelingAantal

- Uitbreiding Voormontage-KostBuilder met Linefeeding houders en dwarsverdeling + Inkorten en montage dwarsverdeling

    • -0
    • +18
    ./Multiplier/DwarsverdelingAantal.cls.xml
  1. … 3 more files in changeset.
FYI: Ik vind het zeker oké dat je hier de ##super hebt weggelaten. (de gegenereerde code die daar stond was eigenlijk zelfs niet correct : LadeHoogte is enum en dus niet in MM uitgedrukt http://sub...

FYI: Ik vind het zeker oké dat je hier de ##super hebt weggelaten. (de gegenereerde code die daar stond was eigenlijk zelfs niet correct : LadeHoogte is enum en dus niet in MM uitgedrukt )
De impl in deze afgeleide klasse is wel correct.

De implementatie voor "Lengte" hieronder zou ik wel in een private method steken, analoog aan GeefFrontPlaatHoogte()
Zo blijft deze method simpel leesbaar en overzichtelijk. En dan is de scope ook duidelijk, nl. dat Kleur W7 enkel impact heeft op de kenmerk "Lengte, en niet op de "Breedte" van het Front.

copy-paste foutje : "... kan niet gemaild worden." (ook al is de conditie altijd false :-P )

copy-paste foutje :
"... kan niet gemaild worden."
(ook al is de conditie altijd false :-P )

[ICT-2588] [rvWV] TAX - nieuwe kleur W7 - Walnut medium Brown:
[ICT-2588] [rvWV] TAX - nieuwe kleur W7 - Walnut medium Brown:
ICT-2807 [rv WV] TAX - Tijdelijke meerkost voor de hoogteverstelling reeds in schaduwprijs
ICT-2807 [rv WV] TAX - Tijdelijke meerkost voor de hoogteverstelling reeds in schaduwprijs
[ICT-2588] [rvWV] TAX - nieuwe kleur W7 - Walnut medium Brown:

- Leeg lijntje weg