Activity CacheAdminA_trunk2010/vhTest/Mock/APPS/Halux/PPS/Document

10:03 am

Jelle Baele

More
CR-2652
started review

[ICT-3795] VHLOG MES M1 - (4) trigger voor verzenden productie-orders naar OBJT:

10:45 am

Jelle Baele

More
CR-2639
started review

[ICT-3795] VHLOG MES M1 - (4) trigger voor verzenden productie-orders naar OBJT:

10:41 am

Jelle Baele

More
CR-2637
started review

[ICT-3794] VHLOG MES M1 - (3) nieuwe aansturingactie 'batch naar OBJT':

4:41 pm

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.

10:08 am

Wim Vermeulen

More
Re: CR-2429
commented on review

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 )

3:13 pm

Wim Vermeulen

More
CR-2426
Closed

3:37 pm

Tommy Hebb

More
CR-2429
started review

[ICT-2588] [rvWV] TAX - nieuwe kleur W7 - Walnut medium Brown:

4:49 pm

Peter Vercoutter

More
CR-2426
finished reviewing

3:26 pm

Wim Vermeulen

More
Re: CR-2426
replied to Peter Vercoutter

Je opmerking is deels terecht, want de code is inderdaad identiek, maar eigenlijk is de context anders, en daarom heb ik de code bewust niet "samengenomen" in één method(). Om het met de woorden va...

Je opmerking is deels terecht, want de code is inderdaad identiek, maar eigenlijk is de context anders, en daarom heb ik de code bewust niet "samengenomen" in één method().
Om het met de woorden van een oud-collega te zeggen "de implementatie is TOEVALLIG hetzelfde, dus het is geen duplicate code".
In 't begin vond ik dit ook wel een vreemde uitleg, nu gebruik ik dit zelf als excuus

12:07 pm

Peter Vercoutter

More
Re: CR-2426
commented on review

GenereerKaderdeurDetailBladPdf en UpdateKaderdeurDetailBladPdf is eigenlijk duplicated code. Kunnen we hier een 'GeneerOfUpdate' (of ene andere naam, je begrijpt wel het punt http://subversion02.va...

GenereerKaderdeurDetailBladPdf en UpdateKaderdeurDetailBladPdf is eigenlijk duplicated code. Kunnen we hier een 'GeneerOfUpdate' (of ene andere naam, je begrijpt wel het punt ) van maken?