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

Review Summary: No summary


----------------------------------------
File: APPS/EDI/Bestel/impl/BestelService/TandemboxBesteller/ProductConverter.cls.xml
Revision Comment by Jo Claes on 06 January 2017, 09:43 
http://subversion02:8060/cru/CR-11#c33

  Waarom is dit geen enumeratie ???
  
  Ik weet niet of het een piste is, maar het lijkt mij handig dat
  we enumeratie hebben van kenmerknamen in de verschillende
  delen, zodat de enumeratie naam hetzelfde is maar de waarde
  verschillend.
  
  Kunnen dan op dat soort enumeratie een extra parameter met
  values zetten zodat we waarden kunnen converteren van het ene
  naar het andere

----------------------------------------
File: vhUnitTest/APPS/EDI/Bestel/impl/BestelService/TandemboxBesteller/ProductConverter/ConverteerProduct/Test.cls.xml
Revision Comment by Jo Claes on 06 January 2017, 09:51 
http://subversion02:8060/cru/CR-11#c34

  In dit geval lijkt het me beter om in het vervolg een dummy
  TBXLadeCodeBepaler te gebruiken zodat je niet iedere keer die
  lijn met ).DanDoeNiks() hebt
  
  Wat ik wel raar vind is dat de method een waarde retourneert en
  je een DanDoeNiks() method gebruikt .. het zou misschien beter
  zijn om .DanReturn() te doen of een method aan te maken
  .DanReturnNiks()
  kwestie om daar geen verwarring in te hebben


---

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

Title: [UW324] conversie zijwandverhogingtype van ext naar edi
Statement of Objectives:


State: Closed

Summary: 

Author: Tom Vermeulen
Moderator: Tom Vermeulen
Reviewers: (0 active, 1 completed*)
     Jo Claes   (*)