Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[ICT-2765] [rvWV] PM: Maatwerk: TAX: Builders: KantenbandKleurVolgtLadeKleur ProjectSetting wegwerken:...
[ICT-2765] [rvWV] PM: Maatwerk: TAX: Builders: KantenbandKleurVolgtLadeKleur ProjectSetting wegwerken:...
Deze voorwaarde komt vele malen voor in de Builders (idem voor ..IsNietUitvoeringTablet() ) Kunnen we een propere en efficiente oplossing vinden waarbij die method niet telkens opnieuw in de builde...

Deze voorwaarde komt vele malen voor in de Builders (idem voor ..IsNietUitvoeringTablet() )
Kunnen we een propere en efficiente oplossing vinden waarbij die method niet telkens opnieuw in de builder moet gedefinieerd worden. Laten we dat samen eens bekijken.

Deze method komt vele malen voor in de Builders (idem voor =Servodrive) Kunnen we een propere en efficiente oplossing vinden waarbij die method niet telkens opnieuw in de builder moet gedefinieerd ...

Deze method komt vele malen voor in de Builders (idem voor =Servodrive)
Kunnen we een propere en efficiente oplossing vinden waarbij die method niet telkens opnieuw in de builder moet gedefinieerd worden. Laten we dat samen eens bekijken.

Het "tussen-niveau" ...HalffabItemBuilderBasis is bij SST eigenlijk niet nodig. Dit was wel zo bij de "Matten" omdat je MatX,MatY, ... hebt. Hier, bij SST, heb je maar 1 klasse, t.t.z. Builder en B...

Het "tussen-niveau" ...HalffabItemBuilderBasis is bij SST eigenlijk niet nodig. Dit was wel zo bij de "Matten" omdat je MatX,MatY, ... hebt.
Hier, bij SST, heb je maar 1 klasse, t.t.z. Builder en BuilderBasis vallen samen.
Maar wat mij betreft hoeft dit niet weggewerkt te worden in caché. Misschien wel bij het porten naar .NET

FYI: het was uiteindelijk de dubbele definitie van de method GeefGekoppeldIKType() die voor enige verwarring zorgde. Vandaar de review opm.

[ICT-1633] Implementatie Halffabbuilders SpaceStep
[ICT-1633] Implementatie Halffabbuilders SpaceStep
[BUS708] TAX: kantenband onderaan moet buitenkleur volgen (zijkant en rug)
[BUS708] TAX: kantenband onderaan moet buitenkleur volgen (zijkant en rug)
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

Injecteren en testen

Injecteren en testen

OfferteGenerator overschrijven eerder dan onderstaande inject

OfferteGenerator overschrijven eerder dan onderstaande inject