Activity CacheAdminA_trunk2010/vhTest/Fake/DOM/PM/impl

10:55 am

Peter Vercoutter

More
CR-2859
started review

[ICT-5441] [rvJLM] Tabletlades in vhintra onder categorie tabletlade brengen

1:58 pm

Jelle Baele

More
CR-2484
started review

[ICT-3054] [rvPVR] RVG aansturingactie: Klantetiketten

3:01 pm

Jelle Baele

More
CR-2481
started review

[ICT-3050] [rvPVR] RVG: Productiegroepbepaler en Productielijnbepaler implementeren

8:40 am

Peter Vercoutter

More
Re: CR-2465
replied to Jelle Baele

Je doet een vergelijking van de verschillen, dus je wilt dat er geen verschillen zijn, daarom empty list

4:42 pm

Jelle Baele

More
Re: CR-2465
commented on review

Vanwaar assert empty list?

4:40 pm

Jelle Baele

More
Re: CR-2465
commented on review

wlips mogen weg of het zal trakteren worden http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/wink.gif

wlips mogen weg of het zal trakteren worden

10:57 am

Peter Vercoutter

More
CR-2465
started review

[ICT-2932] INK: Productcreatie INK

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.