CacheAdminA_trunk2010

Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
En het stomme is dat ik die parameter al voorzien had, maar niet gebruik ....

En het stomme is dat ik die parameter al voorzien had, maar niet gebruik ....

constante gebruiken. als replace veranderd faalt de test niet

constante gebruiken. als replace veranderd faalt de test niet

idem als hierboven

idem als hierboven

Method verwijderd in review.

Method verwijderd in review.

wijziging is om de exceptie aan onze kant niet meer te smijten, maar te loggen. we kunnen er toch niets aan doen. De code aan ITR kant zou voorzien zijn op een Data not found exceptie, maar doet er...

wijziging is om de exceptie aan onze kant niet meer te smijten, maar te loggen.
we kunnen er toch niets aan doen.
De code aan ITR kant zou voorzien zijn op een Data not found exceptie, maar doet er op zoch niets speciaal mee.

[VRB535] HX: Klantetiket: DKG lijnref in barcode (generische maatwerkaansturing)
[VRB535] HX: Klantetiket: DKG lijnref in barcode (generische maatwerkaansturing)
't is misschien neuten, maar naar leesbaarheid toe zou ik hier een private method maken ..BepaalPalletprijs() of is dat yagni ? http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/ic...

't is misschien neuten, maar naar leesbaarheid toe zou ik hier een private method maken ..BepaalPalletprijs() of is dat yagni ?

Testje prijsverschillen nodig ?

Testje prijsverschillen nodig ?

Handig. Direct te zien bovenaan in de klasse welke prijzen gehanteerd worden.

Handig. Direct te zien bovenaan in de klasse welke prijzen gehanteerd worden.

Return type niet aanwezig

Return type niet aanwezig

Hier gebruik je de juiste dto. In de implementatiecode de verkeerde, denk ik http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

Hier gebruik je de juiste dto. In de implementatiecode de verkeerde, denk ik

Misschien de ideale moment om die ProductRepository eens te refactoren http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/wink.gif *KlantId als parameter, terwijl ...

Misschien de ideale moment om die ProductRepository eens te refactoren

  • KlantId als parameter, terwijl het eigenlijk KLNr is.
  • Return type is %String, terwijl het eigenlijk een product teruggeeft.
  • Beetje verduidelijking in naamgeving variabelen kan helpen (bv: Result)