Activity CacheAdminA_trunk2010/FLOWHALUX.mac.rou

1:35 pm

ik weet dat het er al stond, maar eigenlijk is KlantNummer = 26329 niet goed. Er zou gebruik gemaakt moeten worden van DOM.VKP.enu.Klant (daar bestaat ie ook nog niet). Wel rekening houden dat die ...

ik weet dat het er al stond, maar eigenlijk is KlantNummer = 26329 niet goed. Er zou gebruik gemaakt moeten worden van DOM.VKP.enu.Klant (daar bestaat ie ook nog niet). Wel rekening houden dat die enu een KlantID ipv KlantNummer teruggeeft. Deze kan geconverteerd worden naar KlantNummer met behulp van de LegacyPartijAPI. Boyscout-principe-gewijs mag je zo'n dingen zeker aanpassen.

8:53 am

Peter Vercoutter

More
CR-2350
started review

[ICT-2545] Klanten van Dupont op wegwerppallet, Dupont zelf niet

4:13 pm

Jelle Baele

More
CR-2266
Closed

9:38 am

Tom Vermeulen

More
CR-2266
finished reviewing

9:29 am

Tom Vermeulen

More
Re: CR-2266
commented on review

hier kunt ge ook AssertTrue (of AssertFalse) gebruiken.

1:30 pm

Tom Vermeulen

More
Re: CR-2266
commented on review

Ge kunt u hier natuurlijk afvragen of deze method op deze klasse nog zin heeft tout court. 't Is in een oude naamgeving / package. Waarschijnlijk beter om de code die deze method oproept, rechtstre...

Ge kunt u hier natuurlijk afvragen of deze method op deze klasse nog zin heeft tout court. 't Is in een oude naamgeving / package. Waarschijnlijk beter om de code die deze method oproept, rechtstreeks de ToeleveringService te laten gebruiken. Dan skippen we dit gewoon.

1:24 pm

Tom Vermeulen

More
Re: CR-2266
commented on review

ge hebt bij de oproepende klasse wel testen voorzien die checken of deze wordt opgeroepen, maar er zijn (nog) geen testen voor deze method zelf

11:25 am

Tom Vermeulen

More
Re: CR-2266
commented on review

Hier valt over te discussiëren: Is het niet beter om een gedefinieerd antwoord terug te geven, ook als het een error was? Onder normale omstandigheden wel, omdat je niet altijd weet wat de client g...

Hier valt over te discussiëren: Is het niet beter om een gedefinieerd antwoord terug te geven, ook als het een error was? Onder normale omstandigheden wel, omdat je niet altijd weet wat de client gaat doen bij een niet-geïnitialiseerde variabele. Aangezien je in dit concreet geval de client ook onder controle hebt, kan je ermee wegkomen. Persoonlijk zou ik in DnaCodeAfmeldenResponse een defaultvalue False voorzien voor de nieuwe property.

11:16 am