Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
TestFiles reviewen, is in dit soort stories bijna onbegonnen werk. Ik heb zo goed en zo kwaad mogelijk de changes bekeken. Alvast bedankt om het aantal testfiles te filteren. Het geheel lijkt me we...

TestFiles reviewen, is in dit soort stories bijna onbegonnen werk. Ik heb zo goed en zo kwaad mogelijk de changes bekeken.
Alvast bedankt om het aantal testfiles te filteren.
Het geheel lijkt me wel oké te zijn.

Ook in eCon enkele commits gereviewed.

De dossiercode "LI7K" was blijkbaar een onbedoelde tussenstap (Tricky situatie met de TAOR Losse componenten) In combinatie met svn rev. 69413 is dit wel een correcte overgang. Hierbij is deze cha...

De dossiercode "LI7K" was blijkbaar een onbedoelde tussenstap (Tricky situatie met de TAOR Losse componenten)
In combinatie met svn rev. 69413 is dit wel een correcte overgang.

Hierbij is deze change nagekeken en goedgekeurd!

De meeste testfiles heb ik maar buiten de review gehouden. Een aantal zijn toch toegevoegd, bij wijze van check van het resultaat is dat, denk ik, wel ok en het is bovendien soms ook moeilijk om ui...

De meeste testfiles heb ik maar buiten de review gehouden. Een aantal zijn toch toegevoegd, bij wijze van check van het resultaat is dat, denk ik, wel ok en het is bovendien soms ook moeilijk om uit te maken a.d.h.v. de changesets of het al dan niet over code gaat of testfiles.
P.s.: Ook kleine change in Econ (zie svn).

[ICT-3120] [rvWV] PM: TAX: Front HoogteVerstelling MoventoKoppeling: release alle klanten:
[ICT-3120] [rvWV] PM: TAX: Front HoogteVerstelling MoventoKoppeling: release alle klanten:
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 )

Woohoow, het is klaar! Op Jira heb ik weer afbeeldingen gezet van hoe het er nu uitziet, inclusief voor het productiedetailblad (wat oorspronkelijk niet op het kaartje stond)

Woohoow, het is klaar! Op Jira heb ik weer afbeeldingen gezet van hoe het er nu uitziet, inclusief voor het productiedetailblad (wat oorspronkelijk niet op het kaartje stond)

[ICT-2777] [rv PVR] HLX - LBX - Pickinglijst losse onderdelen - betere aanduiding ivm afdekkap buiten...
[ICT-2777] [rv PVR] HLX - LBX - Pickinglijst losse onderdelen - betere aanduiding ivm afdekkap buiten...
Op het kaartje in Jira heb ik een afbeelding gepost van hoe het eruit ziet na de wijzigingen (of de papieren op m'n bureau mocht ge daar ooit eens passeren).

Op het kaartje in Jira heb ik een afbeelding gepost van hoe het eruit ziet na de wijzigingen (of de papieren op m'n bureau mocht ge daar ooit eens passeren).

[ICT-2588] [rvWV] TAX - nieuwe kleur W7 - Walnut medium Brown:
[ICT-2588] [rvWV] TAX - nieuwe kleur W7 - Walnut medium Brown:
[ICT-2765] [rvWV] PM: Maatwerk: TAX: Builders: KantenbandKleurVolgtLadeKleur ProjectSetting wegwerken:...
[ICT-2765] [rvWV] PM: Maatwerk: TAX: Builders: KantenbandKleurVolgtLadeKleur ProjectSetting wegwerken:...
Kleine tip : wanneer ik $$$ElkeWaarde zie staan, dan probeer ik die (als reviewer of als developer) steeds toch eens te vervangen door iets specifieker : in dit geval poging met ..IngegevenKenmerke...

Kleine tip :
wanneer ik $$$ElkeWaarde zie staan, dan probeer ik die (als reviewer of als developer) steeds toch eens te vervangen door iets specifieker : in dit geval poging met ..IngegevenKenmerken
maar dat lukte helaas niet.
Dus mag het blijven staan zoals het nu is

[ICT-2623] [rvWV] HX: TAX: Movento-koppeling: DiepteVerstelling-builder:
[ICT-2623] [rvWV] HX: TAX: Movento-koppeling: DiepteVerstelling-builder:
dto ...MetKar hier gebruikt in de impl van BuilderZonderKar? Indien de definitie/implementatie hetzelfde is voor "metKar" en "ZonderKar", dan best de klassenaam van de dto renamen naar iets gemeens...

dto ...MetKar hier gebruikt in de impl van BuilderZonderKar?
Indien de definitie/implementatie hetzelfde is voor "metKar" en "ZonderKar", dan best de klassenaam van de dto renamen naar iets gemeenschappelijk, bvb. zonder "Kar" (if you know what I mean :-D )

[ICT-2564] [rvWV] HX: TAX: Movento-koppeling: Builders:
[ICT-2564] [rvWV] HX: TAX: Movento-koppeling: Builders:
[ICT-2203] [rvTVE] PM - TAX front voor LBX - oplossing voor "OudZwart" (Z8/Z9):
[ICT-2203] [rvTVE] PM - TAX front voor LBX - oplossing voor "OudZwart" (Z8/Z9):