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 )

[ICT-2588] [rvWV] TAX - nieuwe kleur W7 - Walnut medium Brown:
[ICT-2588] [rvWV] TAX - nieuwe kleur W7 - Walnut medium Brown:
Deze method GeefPalletDraagstructuurProductId() zou eigenlijk beter verplaatst worden naar de builder zelf : private method in HFVerpakkingv001PalletDraagstructuur.cls Zo vermijd je om de waarde vi...

Deze method GeefPalletDraagstructuurProductId() zou eigenlijk beter verplaatst worden naar de builder zelf : private method in HFVerpakkingv001PalletDraagstructuur.cls
Zo vermijd je om de waarde via de constructor te moeten doorgeven.

WLIP's opkuisen

WLIP's opkuisen

De HFProboxPalletv001 wordt toegevoegd, hier in de "Verwerk verpakking Pallet" en tegelijkertijd staat er in de builder HFProboxPalletv001 dat "Aantal = 0" indien verpakking pallet. Waarom dan niet...

De HFProboxPalletv001 wordt toegevoegd, hier in de "Verwerk verpakking Pallet" en tegelijkertijd staat er in de builder HFProboxPalletv001 dat "Aantal = 0" indien verpakking pallet.
Waarom dan niet gewoon die Builder hier NIET toevoegen. Da's toch minder complex, of ni?

[ICT-1626] TAX: Palletverpakking HalffabBuilders aanmaken
[ICT-1626] TAX: Palletverpakking HalffabBuilders aanmaken
[ICT750] TAX: FPV1: Opkuis code flatpack Procom:
[ICT750] TAX: FPV1: Opkuis code flatpack Procom:
Done! UST4187

Done! UST4187

SCA: Momenteel geen aandacht besteden aan pallet-inhoud!

SCA: Momenteel geen aandacht besteden aan pallet-inhoud!

Kaartje UST4201 aangemaakt

Kaartje UST4201 aangemaakt

OnBeforeOneTest() --> zelfde opmerking als bij de UT DoosBepaler

OnBeforeOneTest() --> zelfde opmerking als bij de UT DoosBepaler

DoosBepaler als property invullen via OnBeforeOneTest() zetten --> telkens één regel minder, overzichtelijker http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smil...

DoosBepaler als property invullen via OnBeforeOneTest() zetten --> telkens één regel minder, overzichtelijker

Dezelfde method ook toevoegen voor "Rug" : VerwachtMethodCall("GeefBreedteInMM")

Dezelfde method ook toevoegen voor "Rug" : VerwachtMethodCall("GeefBreedteInMM")

De Key van PRBS is nu altijd "LOSSECOM" , hiermee loop je het risico dat sommige onderdelen zullen worden samengeteld en dus als 1 gesommeerde bouwsteen zullen getoond worden. Ook hier kan je best...

De Key van PRBS is nu altijd "LOSSECOM" , hiermee loop je het risico dat sommige onderdelen zullen worden samengeteld en dus als 1 gesommeerde bouwsteen zullen getoond worden.

Ook hier kan je best specifiek per "onderdeel" een Key toekennen (zie vhDeploy.Maatwerk.LBX.BouwsteenmappingxTAXVerpakkingen )
VPRUG / VPZIJ / VPFRO join DOO / INS / SPC / VDP / VUL

Zoals mondeling besproken : Naam meer expliciet maken : analoog aan "Front dekseldoos" - "Zijkant_Links tussenschot" , ...

Zoals mondeling besproken :

Naam meer expliciet maken :
analoog aan "Front dekseldoos" - "Zijkant_Links tussenschot" , ...