This is a list of all comments for CR-1896.

Review Summary: No summary

General Comment by Jo Willems on 11 December 2019, 11:31 
http://subversion02:8060/cru/CR-1896#c2852

  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


----------------------------------------
File: AXimpl/Dynamics/InventoryOut/EventHandler/PickingCompleet.cls.xml
Revision Comment by Jo Willems on 11 December 2019, 11:46 
http://subversion02:8060/cru/CR-1896#c2855

  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

----------------------------------------
File: AXimpl/Dynamics/InventoryOut/EventHandler/PickingCompleet/PakbonMagGeboektWorden.cls.xml
Revision Comment by Jo Willems on 11 December 2019, 11:28 
http://subversion02:8060/cru/CR-1896#c2851

  zie alg opmerking coding conventions

----------------------------------------
File: AXimpl/Dynamics/InventoryOut/EventHandler/PakbonMagGeboektWorden.cls.xml
Revision Comment by Jo Willems on 11 December 2019, 12:11 
http://subversion02:8060/cru/CR-1896#c2859

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

----------------------------------------
File: AXimpl/Dynamics/InventoryOut/Converter/PakbonMagGeboektWordenConverter.cls.xml
Revision Comment by Jo Willems on 11 December 2019, 11:57 defect
http://subversion02:8060/cru/CR-1896#c2858

  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

Revision Comment by Jo Willems on 11 December 2019, 11:23 defect
http://subversion02:8060/cru/CR-1896#c2850

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

----------------------------------------
File: vhUnitTest/AXimpl/Dynamics/InventoryOut/Converter/PakbonMagGeboektWorden/Convert/Test.cls.xml
Revision Comment by Jo Willems on 11 December 2019, 11:48 
http://subversion02:8060/cru/CR-1896#c2857

  zie alg opmerking codeconventions


---

ID: CR-1896 http://subversion02:8060/cru/CR-1896

Title: [ICT-872] UAT 1467: Pakbon pas aanmaken op moment van transport
Statement of Objectives:


State: Review

Author: Laura Landuyt
Moderator: Laura Landuyt
Reviewers: (0 active, 1 completed*)
     Jo Willems   (*)