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

Review Summary: No summary


----------------------------------------
File: vhDeploy/KCA
Revision Comment by Jo Claes on 23 February 2017, 14:31 defect
http://subversion02:8060/cru/CR-99#c259

  geen Initialen in deploy stories :D

----------------------------------------
File: APPS/EC/Winkelkar/impl/WinkelkarControleur.cls.xml
Revision Comment by Jo Claes on 23 February 2017, 14:12 defect
http://subversion02:8060/cru/CR-99#c254

  Niet nodig om een onliner af te zonderen naar een aparte method

Revision Comment by Jo Claes on 23 February 2017, 14:12 defect
http://subversion02:8060/cru/CR-99#c253

  Beter deze logica afzonderen naar een submethode kwestie om het
  toch wat leesbaarder te krijgen.

Revision Comment by Jo Claes on 23 February 2017, 14:16 defect
http://subversion02:8060/cru/CR-99#c256

  Het lijkt mij beter om deze code te herwerken zodat er met een
  MultiDim / Array kan gewerkt worden waarbij de
  Lade_Familie_Variant als key gebruikt worden.

Revision Comment by Jo Claes on 23 February 2017, 14:14 defect
http://subversion02:8060/cru/CR-99#c255

  Waarom is er voor legrabox een uitzondering door niet te
  controleren op MagKlantVerpakkingTypeBestellen ??? 
  ps : ik zie ook wel duidelijk dat die boolean hardcoded op
  $$$true wordt gezet

Revision Comment by Jo Claes on 23 February 2017, 14:22 defect
http://subversion02:8060/cru/CR-99#c257

  Beter om op de KlantInstellingenService een extra method aan te
  maken die Lade_Familie binnenkrijgt , de klantid en het
  verpakkingtype zodoende we niet per Lade_Familie extra code
  moeten toevoegen

Revision Comment by Jo Claes on 23 February 2017, 14:30 defect
http://subversion02:8060/cru/CR-99#c258

  Zelfde opmerking zoals bij de MagKlantVerpakkingTypeBestellen

----------------------------------------
File: vhUnitTest/APPS/EC/Winkelkar/WinkelkarControleur/Nederlands/TestTaorboxEnLegraboxZelfdeVerpakking.cls.xml
Revision Comment by Jo Claes on 23 February 2017, 14:33 defect
http://subversion02:8060/cru/CR-99#c260

  Testen zouden eigenlijk het aantal moeten kunnen uitmocken
  zodat de logica van de code kan getest worden onafhankelijk van
  het aantal. Anders zal er bij iedere configuratie wijzigingen
  test-resultaten wijzigen

----------------------------------------
File: APPS/EC/Winkelkar/impl/VerpakkingControleurWinkelkarVisitor.cls.xml
Revision Comment by Jo Claes on 23 February 2017, 14:10 defect
http://subversion02:8060/cru/CR-99#c252

  #dim naar het juiste type


---

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

Title: [UST2170] Vh4_SVC: TBX Probox (e.a.) min. bestelhoeveelheid
Statement of Objectives:


State: Closed

Summary: 

Author: Kurt Callebaut
Moderator: Kurt Callebaut
Reviewers: (0 active, 1 completed*)
     Jo Claes   (*)