Activity CacheAdminA_trunk2010/DOM/MAG/event

8:15 am

LauraL

More
59153
changed 2 files
root: CR-1896

[ICT-872] UAT 1467: Pakbon pas aanmaken op moment van transport

BUGFIX FactuurVerzendenEvent reference hernoemen + Consnr hernoemen

- -1
- +1
./EventRaiser.cls.xml
- History
- Source
- Diff
./EventRaiser.cls.xml
… 1 more file in changeset.

12:11 pm

Jo Willems

More
Re: CR-1896
commented on review

Mis een testje hier op die test of generate uitgevoerd wordt afh van config item

12:01 pm

De specs waren hier mss niet duidelijk genoeg, maar per productlijn genereer je hier identieke lijnen (wat vroeger niet het geval was vanwege de inventsync die uniek is) Al de informatie die je in ...

De specs waren hier mss niet duidelijk genoeg, maar per productlijn genereer je hier identieke lijnen (wat vroeger niet het geval was vanwege de inventsync die uniek is)
Al de informatie die je in het bericht moet steken kan je bepalen adhv bon, je hebt de lijnen niet nodig.
Je moet dus eigenlijk maar één lijn per bon voor zien.
AX kan er tegen, maar ze geen x keer de zelfde bon markeren dat geboekt mag worden.
Dit gaat de opzet van de test mocks data etc ook wat eenvoudiger maken

11:50 am

Jo Willems

More
Re: CR-1896
commented on review

Algemene opmerking code conventions volgorde van elementen van een class 1 parameters 2 properties 3 Constructor (%OnNew) 4 Public methods 5 Private methods voor Testcases onbefore* en onafter* me...

Algemene opmerking code conventions volgorde van elementen van een class
1 parameters
2 properties
3 Constructor (%OnNew)
4 Public methods
5 Private methods

voor Testcases onbefore* en onafter* methods voor de eigenlijke test methods

11:49 am

Jo Willems

More
Re: CR-1896
commented on review

zie alg opmerking codeconventions

11:47 am

Jo Willems

More
Re: CR-1896
commented on review

Meegeven van GeneratePickingCompleetResult kan nu nog. Voor de volgende fase, het uitgesteld raisen van de event heb je dat resultaat niet meer... Test op pickingcompleetresultaat dus in deze metho...

Meegeven van GeneratePickingCompleetResult kan nu nog.
Voor de volgende fase, het uitgesteld raisen van de event heb je dat resultaat niet meer...
Test op pickingcompleetresultaat dus in deze method steken.
Of anders refactoren bij implementatie 2e fase

11:30 am

Jo Willems

More
Re: CR-1896
commented on review

zie alg opmerking coding conventions

11:24 am

Jo Willems

More
Re: CR-1896
commented on review

Waarom zou je hier een spatie zetten? een spatie is niet 'niet ingevuld' Idem voor qty

Waarom zou je hier een spatie zetten? een spatie is niet 'niet ingevuld'
Idem voor qty

11:21 am

Laura Landuyt

More
CR-1896
started review

[ICT-872] UAT 1467: Pakbon pas aanmaken op moment van transport

9:52 am

LauraL

More
59069
changed 1 file
root: CR-1896

[ICT-872] UAT 1467: Pakbon pas aanmaken op moment van transport

event/EventRaiser add methode RaisePakbonMagGeboektWordenEvent + mockable maken

- -1
- +10
./EventRaiser.cls.xml
- History
- Source
- Diff
./EventRaiser.cls.xml

9:22 am

LauraL

More
59062
added 1 file
root: CR-1896

[ICT-872] UAT 1467: Pakbon pas aanmaken op moment van transport

Toeveogen Event /DOM.MAG.event.PakbonMagGeboektWordenEvent

- -0
- +12
./PakbonMagGeboektWordenEvent.cls.xml
- History
- Source
./PakbonMagGeboektWordenEvent.cls.xml

11:42 am

Tom Vermeulen

More
CR-507
Closed

3:13 pm

Wim Vermeulen

More
CR-507
finished reviewing

11:37 am

Tom Vermeulen

More
CR-507
started review

[UST2812] VHIP903: PickingCompleet andere locatie voor event

11:33 am

Tom Vermeulen

More
45188
changed 3 files
root: CR-507

[UST2812] VHIP903: PickingCompleet andere locatie voor event

- -0
- +9
./EventRaiser.cls.xml
- History
- Source
- Diff
./EventRaiser.cls.xml
- -0
- +12
./PickingCompleetEvent.cls.xml
- History
- Source
./PickingCompleetEvent.cls.xml
… 1 more file in changeset.

8:20 am

Tom Vermeulen

More
CR-479
Closed

4:36 pm

Jo Claes

More
CR-479
finished reviewing

2:34 pm

Tom Vermeulen

More
45015
changed 2 files
root: CR-479

[UST2757R] VHIP903 Na een cycle count moet er een bericht verstuurd worden met de nieuwe voorraad

- Ifke wat simpeler gemaakt voor de reviewer

- Aparte EventRaiser voorzien

- -0
- +17
./EventRaiser.cls.xml
- History
- Source
./EventRaiser.cls.xml
… 1 more file in changeset.

11:52 am

Jo Claes

More
Re: CR-479
commented on review

Normaal gezien steken we dit soort oproepen in één of andere eventraiser klasse - kwestie het in de oorspronkelijke code een onliner te hebben http://subversion02.vanhoecke.be/static/ogdo0b/2static...

Normaal gezien steken we dit soort oproepen in één of andere eventraiser klasse - kwestie het in de oorspronkelijke code een onliner te hebben

11:50 am

Jo Claes

More
Re: CR-479
commented on review

Code mag in principe een stukje lager dan heb je die if niet nodig http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif Of anders maak je er een if {} else ...

Code mag in principe een stukje lager dan heb je die if niet nodig

Of anders maak je er een if {} else {} van want nu een if met een conditionele do is iets te verwarrend

3:27 pm

Tom Vermeulen

More
CR-479
started review

[UST2757] VHIP903 Na een cycle count moet er een bericht verstuurd worden met de nieuwe voorraad

1:47 pm

Tom Vermeulen

More
44904
added 2 files
root: CR-479

[UST2757] VHIP903 Na een cycle count moet er een bericht verstuurd worden met de nieuwe voorraad

- Nieuw event en -data

- -0
- +12
./StockGewijzigdEvent.cls.xml
- History
- Source
./StockGewijzigdEvent.cls.xml
- -0
- +45
./StockGewijzigdEventData.cls.xml
- History
- Source
./StockGewijzigdEventData.cls.xml

11:37 am

Tom Vermeulen

More
44278
changed 1 file
root:

[ICT] EventData constructor-parameters optioneel gemaakt ter ondersteuning van de "ReInvoke" (vanuit vhLib.Logging) + Properties Required als noodzakelijk gevolg.