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

- Uitbreiding TaorboxModel met kenmerk DwarsverdelingType en rol DwarsverdelingDubbelHouder

- Uitbreiden TAORKenmerken met DwarsverdelingType

- Uitbreiden HalffabClassNameFactory met halffab DwarsverdelingDubbelHouder

- Toevoegen rol DwarsverdelingDubbelHouder

- Deployklasse voor Selectiekenmerken die halffab-klasse aangemaakt heeft

  1. … 5 more files in changeset.
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-2728] [rvWV] TAX configurator : uitbreiden met een dwarsverdeling:

- Uitbreiding MaatwerkModel met rollen Dwarsverdeling en DwarsverdelingHouder

- Uitbreiding TAORKenmerken met AantalDwarsverdelingen en DwarsverdelingKleur

- Toevoeging halffabjes voor HalffabDwarsverdeling en HalffabDwarsverdelingHouder

- Uitbreiding BaseEnums met DwarsverdelingKleur

- Deployklasjes voor Selectie- en pps-kenmerken

  1. … 10 more files in changeset.
Is er een reden waarom je tests toevoegt en uitschakelt?

Is er een reden waarom je tests toevoegt en uitschakelt?

#dim zonder type is even erg als een Set

#dim zonder type is even erg als een Set

Method mag private (en bij voorkeur geen strings als methodnaam, maar UpperCamelCased) *Personality specifieke httpverzender meegeven i.p.v. extra logica in test (bepaling van httpverzender via o...
  • Method mag private (en bij voorkeur geen strings als methodnaam, maar UpperCamelCased)
  • Personality specifieke httpverzender meegeven i.p.v. extra logica in test (bepaling van httpverzender via objectapi)
Deze klasse test enkel of de dependencies opgeroepen worden: veel setup en weinig (duidelijke) asserts. Dit geeft aan dat hier logica opgesplitst zou moeten worden.

Deze klasse test enkel of de dependencies opgeroepen worden: veel setup en weinig (duidelijke) asserts. Dit geeft aan dat hier logica opgesplitst zou moeten worden.

Kan je ook omschrijven in de testnaam wat het standaard gedrag is?

Kan je ook omschrijven in de testnaam wat het standaard gedrag is?

Injecteren en testen

Injecteren en testen

Wat is de meerwaarde van deze getter-methode?

Wat is de meerwaarde van deze getter-methode?

Afzonderen en testen van TAX en LBX logica

Afzonderen en testen van TAX en LBX logica

naamgeving: ProjectSettingsApi

naamgeving: ProjectSettingsApi

Else (inconsistente casing)

Else (inconsistente casing)

Injecteren en testen

Injecteren en testen

$$$False ipv 0

$$$False ipv 0

Gebruik property IsBinnenlade

Gebruik property IsBinnenlade

Deze method heeft geen meerwaarde, je kan net zowel de property opvragen

Deze method heeft geen meerwaarde, je kan net zowel de property opvragen