Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[ICT-1779] PA-UB-149: Hernoem pallet error

- magazijnlocatie mockable gemaakt

  1. … 4 more files in changeset.
[ICT-872] UAT 1467: Pakbon pas aanmaken op moment van transport

BUGFIX FactuurVerzendenEvent reference hernoemen + Consnr hernoemen

  1. … 1 more file in changeset.
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
[ICT-872] UAT 1467: Pakbon pas aanmaken op moment van transport

event/EventRaiser add methode RaisePakbonMagGeboektWordenEvent + mockable maken

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

Toeveogen Event /DOM.MAG.event.PakbonMagGeboektWordenEvent

    • -0
    • +12
    ./event/PakbonMagGeboektWordenEvent.cls.xml
  • More
  • CR-507
  • finished reviewing
[UST2812] VHIP903: PickingCompleet andere locatie voor event
[UST2812] VHIP903: PickingCompleet andere locatie voor event
[UST2812] VHIP903: PickingCompleet andere locatie voor event
    • -0
    • +12
    ./event/PickingCompleetEvent.cls.xml
  1. … 1 more file in changeset.
  • More
  • CR-479
  • finished reviewing
[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

    • -5
    • +10
    ./Inventaris/impl/StockWijziger.cls.xml
    • -0
    • +17
    ./event/EventRaiser.cls.xml
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
[UST2757] VHIP903 Na een cycle count moet er een bericht verstuurd worden met de nieuwe voorraad

- Event raisen in StockWijziger

    • -0
    • +6
    ./Inventaris/impl/StockWijziger.cls.xml
[UST2757] VHIP903 Na een cycle count moet er een bericht verstuurd worden met de nieuwe voorraad

- Nieuw event en -data

    • -0
    • +12
    ./event/StockGewijzigdEvent.cls.xml
    • -0
    • +45
    ./event/StockGewijzigdEventData.cls.xml
[ICT] EventData constructor-parameters optioneel gemaakt ter ondersteuning van de "ReInvoke" (vanuit vhLib.Logging) + Properties Required als noodzakelijk gevolg.
    • -2
    • +4
    ./event/WachtendOpPickingEventData.cls.xml
[ICT] - Properties zijn niet meer private in de event data.
    • -4
    • +0
    ./event/WachtendOpPickingEventData.cls.xml
[ICT] - Properties zijn niet meer private in de event data.
    • -4
    • +0
    ./event/PickingAnnulatieEventData.cls.xml
[UST1740] TAX FPV1 VPK: GeefData

- Mock gegenereerd voor MagazijnAPI

- parameter getyped

- cleanup, cosmetica

  1. … 1 more file in changeset.
[ICT] - Refactoring in event handling - Events hebben vanaf heden Type method
    • -11
    • +0
    ./event/PalletVerplaatstEvent.cls.xml