Activity CacheAdminA_trunk2010/vhTestFiles/IntegratieTests/Productie.Planning.ActieveBatches.Taorbox.Aansturing/Test006BinnenladeLadeKleurWalnutMediumBrown/StartBatch/Ardis/ImportAuto

3:31 pm

Tommy Hebb

More
68116
changed 19 files
root:

[ICT-2966] [rvTHB] Aanpassingen Lodder sturing

- I.g.v. IsProductMetBewerkingInSpecialWorkshop nu SpecialsPM

- VerwerkingID-prefix 'S' i.g.v. W7

=> Deel 1 van nieuwe testbestanden

./ImportAuto
- History
- Source
- Diff
./ImportAuto
- -1
- +0
./Productie_007_ImportList_15000_20141114_091500.txt
- History
- Source
- Diff
./Productie_007_ImportList_15000_20141114_091500.txt
- -8
- +0
./Productie_006_ImportFile_15000_20141114_091500.txt
- History
- Source
- Diff
./Productie_006_ImportFile_15000_20141114_091500.txt
… 16 more files in changeset.

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 )

4:29 pm