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

Review Summary: No summary


----------------------------------------
File: APPS/OV/OrderBevestiging/impl/WachtrijAfhandelaar/Base.cls.xml
Revision Comment by Jo Claes on 21 February 2017, 11:41 defect
http://subversion02:8060/cru/CR-92#c220

  Parameter gebruikt maar er wordt niks met gedaan ??

  Reply by Kurt Callebaut on 22 February 2017, 14:41
  > Wordt gebruikt in afgeleide klassen... mogelijk als indicatie
  > dat je deze zeker moet meegeven.  eventueel set verstuurder
  > naar base Class, Maar gezien dat de spy een andere
  > implementatie heeft voor verstuurder in constructor.

    Reply by Jo Claes on 22 February 2017, 16:12
    > Dan best parameter laten staan en deze in de afgeleide
    > klassen doorgeven via de ##super()

Revision Comment by Jo Claes on 21 February 2017, 11:43 defect
http://subversion02:8060/cru/CR-92#c222

  Als je dan toch bezig bent met de dependencies bloot te leggen

Revision Comment by Jo Claes on 21 February 2017, 11:43 defect
http://subversion02:8060/cru/CR-92#c223

  Constructor parameter van de logger mag weg

  Reply by Kurt Callebaut on 22 February 2017, 14:58
  > Laat maar zo.

    Reply by Jo Claes on 22 February 2017, 16:08
    > En om welke reden zouden we dat zo laten nu krijgen we
    > onhandige logging meldingen en kunnen we bepaalde flows in
    > de logging niet volgen..

Revision Comment by Jo Claes on 21 February 2017, 11:42 defect
http://subversion02:8060/cru/CR-92#c221

  Gebruikte klasse in de constructor zichtbaar maken

----------------------------------------
File: vhUnitTest/APPS/OV/OrderBevestiging/Verstuurder/Test.cls.xml
Revision Comment by Jo Claes on 21 February 2017, 11:45 defect
http://subversion02:8060/cru/CR-92#c224

  Voor de datum zou ik toch een datum formaat nemen

  Reply by Kurt Callebaut on 22 February 2017, 14:36
  > Het is de bedoeling om aan te geven welk veld er op die
  > positie komt en niet de inhoud. Als we datum gebruiken kan
  > eht om het even welke datum zijn.

----------------------------------------
File: APPS/OV/OrderBevestiging/impl/WachtrijAfhandelaar/Bruynzeel.cls.xml
Revision Comment by Jo Claes on 22 February 2017, 16:10 defect
http://subversion02:8060/cru/CR-92#c249

  als dan toch blijkt dat de super moet aanroepen moet hij wel de
  verstuurder meegeven zodoende hij juist geïnitialiseerd wordt

----------------------------------------
File: APPS/OV/OrderBevestiging/impl/WachtrijAfhandelaar/DKG.cls.xml
Revision Comment by Jo Claes on 22 February 2017, 16:13 defect
http://subversion02:8060/cru/CR-92#c251

  idem als de anderen


---

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

Title: [UST2111][DKG EDI orderbevestiging: positienummer invullen, en terug in werking brengen
Statement of Objectives:


State: Closed

Summary: 

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