Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Private methods onderaan plaatsen. (zie zelfde opm bij andere UT)

Private methods onderaan plaatsen.
(zie zelfde opm bij andere UT)

Ik heb enkele bedenkingen, die we gemakkelijker mondeling kunnen bespreken. Let's skype http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

Ik heb enkele bedenkingen, die we gemakkelijker mondeling kunnen bespreken.
Let's skype

Private methods onderaan plaatsen. Alternatief is om er een OnBeforeOneTest() van te maken, maar dat zal hier niet lukken aangezien je data meegeeft (Kenm en variant) Dus toch private en onderaan...

Private methods onderaan plaatsen.

Alternatief is om er een OnBeforeOneTest() van te maken, maar dat zal hier niet lukken aangezien je data meegeeft (Kenm en variant)

Dus toch private en onderaan
En dan zou ik er een Function van maken i.p.v. "Do ..." van maken,
Bijvoorbeeld : GeefDataMMatCreator() met dezelfde args.

Hier zou ik de "StandaardLade" method oproepen en dan enkel de properties Kleur en Materiaal overschrijven. Da's 4 lijnen code-duplicatie minder. Anderszijds wou ik opmerken dat je de kleur-variat...

Hier zou ik de "StandaardLade" method oproepen en dan enkel de properties Kleur en Materiaal overschrijven.
Da's 4 lijnen code-duplicatie minder.

Anderszijds wou ik opmerken dat je de kleur-variaties beter in de UT zelf implementeert.
De Utils.MatKenmerken klasse is niet bedoeld om alle mogelijke variaties van kleuren, die we ooit ergens (in een UT) zouden nodig hebben, hier te gaan toevoegen.
1 kleur-variatie lijkt me nog wel te kunnen in deze klasse (is al schemerzone), maar 3 toch liever niet.

Ter illustratie : in deze klasse kan bijvoorbeeld wel : TaorIndelingMat , ExtraBredeMat, ... (die zijn nu uiteraard niet nodig)

ICT-1497: Matten: kleur wordt niet weergegeven bij type Orgalux en Taor
ICT-1497: Matten: kleur wordt niet weergegeven bij type Orgalux en Taor
[UST3860] VHIP184: Sync Prod: Wissel leverancier:
[UST3860] VHIP184: Sync Prod: Wissel leverancier:
[DEF672] VHIP184: Sync Prod: BtwKode op product nodig (MAG: Leverbon maken: CalcBon error):
[DEF672] VHIP184: Sync Prod: BtwKode op product nodig (MAG: Leverbon maken: CalcBon error):
  • More
  • CR-877
  • finished reviewing
?

?

[UW420] VHIP184 : validatie op ident nummer inbouwen
[UW420] VHIP184 : validatie op ident nummer inbouwen
  • More
  • CR-724
  • finished reviewing
  • More
  • CR-692
  • finished reviewing
Over de weetniet velden heb ik nog niet teveel opmerkingen gegeven -> zullen hopelijk in de nabije toekomst WeetWelVeld1 ,WeetWelVeld2,WeetWelVeld3 worden

Over de weetniet velden heb ik nog niet teveel opmerkingen gegeven -> zullen hopelijk in de nabije toekomst WeetWelVeld1 ,WeetWelVeld2,WeetWelVeld3 worden

Lijkt mij niet echt zinvol om hier te mailen -> we gaan dubbele meldingen krijgen !?!

Lijkt mij niet echt zinvol om hier te mailen -> we gaan dubbele meldingen krijgen !?!

Naamgeving van Iterator - Verschil maken tussen de originele iterator en de FallbackItems iterator

Naamgeving van Iterator - Verschil maken tussen de originele iterator en de FallbackItems iterator

Feature Envy !?!?

Feature Envy !?!?

Betekenis van 5 !?!?

Betekenis van 5 !?!?

'k vind geen test die bewijst dat deze functionaliteit werkt

'k vind geen test die bewijst dat deze functionaliteit werkt

Het lijkt mij beter dat deze melding wordt omgedraaid , het feit dat PM op de hoogte gesteld is zou ik iets meer naar voor zetten.

Het lijkt mij beter dat deze melding wordt omgedraaid , het feit dat PM op de hoogte gesteld is zou ik iets meer naar voor zetten.

  • More
  • CR-576
  • finished reviewing