Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Mis een testje hier op die test of generate uitgevoerd wordt afh van config item

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

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

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

zie alg opmerking codeconventions

zie alg opmerking codeconventions

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

zie alg opmerking coding conventions

zie alg opmerking coding conventions

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

[ICT-872] UAT 1467: Pakbon pas aanmaken op moment van transport
[ICT-872] UAT 1467: Pakbon pas aanmaken op moment van transport
  • More
  • CR-507
  • finished reviewing
[UST2812] VHIP903: PickingCompleet andere locatie voor event
[UST2812] VHIP903: PickingCompleet andere locatie voor event
  • More
  • CR-479
  • finished reviewing
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

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

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