Activity CacheAdminA_trunk2010/BL/Legacy/FLOWHALUX.cls.xml

3:51 pm

Jelle Baele

More
69156
changed 20 files
root:

[ICT-3068] [rvPVR] Batchservice uitbreiden voor RVG:

- Wijzigingen voor Revego producten met leverancier VanHoecke terugdraaien

- -18
- +0
./FLOWHALUX.cls.xml
- History
- Source
- Diff
./FLOWHALUX.cls.xml
… 19 more files in changeset.

5:22 pm

Jelle Baele

More
69022
changed 4 files
root:

[ICT-3068] [rvPVR] Batchservice uitbreiden voor RVG:

- Logica bepalen juiste leverancier opsplitsen naar legacy klasse

- -0
- +18
./FLOWHALUX.cls.xml
- History
- Source
- Diff
./FLOWHALUX.cls.xml
… 3 more files in changeset.

4:13 pm

Tom Vermeulen

More
Re: CR-2266
commented on review

hier kunt ge ook AssertTrue (of AssertFalse) gebruiken.

3:21 pm

Jelle Baele

More
63842
changed 2 files
root: CR-2266

[ICT-2157] [rvTVE] PA-UB-217: spacesteppopup bij afmelden dnacode

- Verwerken review opmerkingen: Oproep 'HeeftToeleveringMinstensEenSST' verplaatst van Legacy klasse naar mac-routine zelf

- -11
- +0
./FLOWHALUX.cls.xml
- History
- Source
- Diff
./FLOWHALUX.cls.xml
… 1 more file in changeset.

1:30 pm

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