Activity CacheAdminA_trunk2010/vhTest/Fake/APPS/Halux/Docs/impl/LadeKlantEtiketBuilder.cls.xml

3:40 pm

Peter Vercoutter

More
Re: CR-2636
commented on review

Voor mij oke! Ik zou misschien hebben gekozen voor een andere variabele naam dan 'IsSpecialeLade' want dit geeft niet meteen aan waarvoor het zal worden gebruikt.

2:26 pm

Jelle Baele

More
CR-2636
started review

[ICT-3849] [rvPVR] Koppelen orgalux sticker aan lade code in het bewerkingscenter:

3:50 pm

Peter Vercoutter

More
Re: CR-2126
replied to Tom Vermeulen

Ik denk dat dit veel extra werk zou opleveren en niet waard zou zijn kosten/baten

11:57 am

Door de bestaande testen aan te passen heb je het gedrag waarbij de laatste parameter de default is, weggemoffeld. Je moet de originele testen laten staan en een paar nieuwe testen maken die True a...

Door de bestaande testen aan te passen heb je het gedrag waarbij de laatste parameter de default is, weggemoffeld. Je moet de originele testen laten staan en een paar nieuwe testen maken die True als laatste parameter hebben.

11:53 am

Tom Vermeulen

More
Re: CR-2126
commented on review

{$$$False} ipv 0

11:48 am

Tom Vermeulen

More
Re: CR-2126
commented on review

Kan die nieuwe property eigenlijk niet mee op de UitvoeringInfo gestoken worden? In principe staan daar nog allerlei gegevens die nodig zijn om (in dit geval) etiketten af te drukken. Op die manier...

Kan die nieuwe property eigenlijk niet mee op de UitvoeringInfo gestoken worden? In principe staan daar nog allerlei gegevens die nodig zijn om (in dit geval) etiketten af te drukken. Op die manier houden we de interface(s) proper (zeker als er ooit nog andere parameters bij moeten komen).

Dit geldt ook voor sommige andere methods/klasses in deze review

11:47 am