CacheAdminA_trunk2010

Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
idem

idem

Kurt, ik deel hier de mening van Martijn. Volgens mij voldoende om de Utils-klasse te laten afleiden van TECH.Mockable

Kurt, ik deel hier de mening van Martijn.
Volgens mij voldoende om de Utils-klasse te laten afleiden van TECH.Mockable

ICT-944: [rvKCA] BugFix: TBX.WPPers.InfoBepaler crasht op TAX-lade
ICT-944: [rvKCA] BugFix: TBX.WPPers.InfoBepaler crasht op TAX-lade
ICT-918: Uitfaseren LBX Tipon : aanpassen Automatische Testen
ICT-918: Uitfaseren LBX Tipon : aanpassen Automatische Testen
Als er palletten meegeven zijn, en we vinden niet wat we zoeken, dan stoppen we. Als er geen palletten meegegeven zijn moeten we verder. (fallback) de test op Count is dus essentieel. en dient niet...

Als er palletten meegeven zijn, en we vinden niet wat we zoeken, dan stoppen we.
Als er geen palletten meegegeven zijn moeten we verder. (fallback)
de test op Count is dus essentieel. en dient niet om de 0 te detecteren, maar om niet 0 te detecteren.

OPGELET: Tracht zoveel mogelijk de code van deze klasse tussen caché v5 en cache2010 in sync te houden. Net voor uw wijzigingen waren deze methods redelijk goed in sync (behalve method FPalletsPerP...

OPGELET:
Tracht zoveel mogelijk de code van deze klasse tussen caché v5 en cache2010 in sync te houden.
Net voor uw wijzigingen waren deze methods redelijk goed in sync (behalve method FPalletsPerProdNr() die is ge-refactored).
Na uw wijzigingen dreigt de code uiteen te lopen en wordt het steeds lastiger voor iemand anders om nog toe te voegen, of erger nog: om te troubleshooten!

Maak er desnoods een apart kaartje van. Of overleg met Pieter en/of met mij

Early quit :-O Gelukkig gaat ie binnenkort wel weg.

Early quit :-O
Gelukkig gaat ie binnenkort wel weg.

Als reviewer ben ik verplicht om dit aan te geven : embedded sql aanpassen : beter : wegwerken en vervangen door TECH.Resultset reden: gedrag is beter onder controle bij errors, early quits, onvoo...

Als reviewer ben ik verplicht om dit aan te geven :

embedded sql aanpassen :
beter : wegwerken en vervangen door TECH.Resultset
reden: gedrag is beter onder controle bij errors, early quits, onvoorspelbaar gedrag indien 2 sql's gemixed worden (lees op andere stacklevel een andere &sql doorlopen), compile-issues.

In dit specifiek geval zou ik misschien toch ook de &sql behouden.

Zeer goed dat je deze lijn erbij zet! http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

Zeer goed dat je deze lijn erbij zet!

Count() is niet nodig, want als Count=0 , dan zal Find() ook niks vinden.

Count() is niet nodig, want als Count=0 , dan zal Find() ook niks vinden.

Waarom een interface?

Waarom een interface?

Waarom is interface?

Waarom is interface?

ICT-930: Refactoring nodig voor klassen van activiteiten. (vooral spoelbak is meer dan enkel spoelbak.)
ICT-930: Refactoring nodig voor klassen van activiteiten. (vooral spoelbak is meer dan enkel spoelbak.)
ICT-930: Refactoring nodig voor klassen van activiteiten. (vooral spoelbak is meer dan enkel spoelbak.)
ICT-930: Refactoring nodig voor klassen van activiteiten. (vooral spoelbak is meer dan enkel spoelbak.)
[ICT778] MAG: Picking: juiste exemplaren picken (voor het juiste order)
[ICT778] MAG: Picking: juiste exemplaren picken (voor het juiste order)
Commentaar bij een method : moet je /// (3 slashes) gebruiken. + typo : spEOlbak :-D

Commentaar bij een method : moet je /// (3 slashes) gebruiken.

+ typo : spEOlbak :-D

Alleen nog ##super() dus method override verwijderen

Alleen nog ##super() dus method override verwijderen