Activity CacheAdminA_trunk2010/DOM/PM/Product/impl

1:58 pm

Eigenlijk zijn dat hier allemaal commits om dependency injection toe te laten op deze Creator. Zouden er dan ook niet wat testen moeten zijn die wat dependencies injecten en het gedrag van hieronde...

Eigenlijk zijn dat hier allemaal commits om dependency injection toe te laten op deze Creator. Zouden er dan ook niet wat testen moeten zijn die wat dependencies injecten en het gedrag van hieronder verifiëren?

8:01 am

Tom Vermeulen

More
Re: CR-1891
commented on review

overbodige comment?

2:35 pm

Jo Willems

More
CR-1891
started review

[ICT-860]PM: Maatwerk: ingekort ORGALUX: "ORGALUX" in bestelreferentie zetten

2:34 pm

Jo Willems

More
58910
changed 8 files
root: CR-1891

[ICT-860]PM: Maatwerk: ingekort ORGALUX: "ORGALUX" in bestelreferentie zetten

- Beetje refactoren om de constructor aan de coding conventions te laten voldoen.

- -1
- +2
./TAORMaatwerkProductCreator.cls.xml
- History
- Source
- Diff
./TAORMaatwerkProductCreator.cls.xml
… 7 more files in changeset.

9:42 am

Tommy Hebb

More
CR-1872
Closed

3:08 pm

Kurt Callebaut

More
CR-1872
finished reviewing

1:20 pm

Wim Vermeulen

More
Re: CR-1872
commented on review

Deze method voor TAX is volledig identiek http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/warning.gif aan die erboven voor LBX, op één woordje na : ..GeefProductU...

Deze method voor TAX is volledig identiek aan die erboven voor LBX, op één woordje na : ..GeefProductUpdater("TAX", ...
En ondertussen mogelijk ook nog eens t zelfde voor TBX (niet in deze review)

Zoals reeds eerder opgemerkt : Maak in de UT duidelijk waar de variatie zit, dus niet door de volledige impl te kopieren en één waarde te veranderen.
Oplossing hier is heel simpel : maak private method AssertProductUpdaterVoorLadeVariant (LadeVariant As ...enu)
en dan heb je 2 (of 3 indien TBX ook) testmethods die 1 lijntje code bevat met "LBX" of "TAX" of "TBX"

3:43 pm

Tommy Hebb

More
CR-1872
started review

[ICT-958] [rvWV] PM: Maatwerk: BOMBOL VHIP481: TAX updaten:

11:08 am

Tommy Hebb

More
58635
changed 1 file
root: CR-1872

[ICT-958] [rvWV] PM: Maatwerk: BOMBOL VHIP481: TAX updaten:

- Louter refactor

- -6
- +8
./TAORMaatwerkProductCreator.cls.xml
- History
- Source
- Diff
./TAORMaatwerkProductCreator.cls.xml

11:04 am

Tommy Hebb

More
58634
changed 1 file
root: CR-1872

[ICT-958] [rvWV] PM: Maatwerk: BOMBOL VHIP481: TAX updaten:

- Ongebruikte parameter van constructor verwijderd

- -1
- +0
./TAORMaatwerkProductCreator.cls.xml
- History
- Source
- Diff
./TAORMaatwerkProductCreator.cls.xml

5:16 pm

Tommy Hebb

More
58631
changed 2 files
root: CR-1872

[ICT-958] [rvWV] PM: Maatwerk: BOMBOL VHIP481: TAX updaten:

- Ook igv TAX wordt nu ProductData netjes doorgegeven om PRNr en Dossiercode te behouden van een bestaand product om het te verwijderen en met andere kenmerken opnieuw te maken