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

Review Summary: No summary


----------------------------------------
File: APPS/Halux/PPS/Activiteit/impl/MVX/MVXLadePersen/InfoBepaler.cls.xml
Revision Comment by Tommy Hebb on 21 April 2021, 15:06 defect
http://subversion02:8060/cru/CR-2293#c3695

  Voor alle dependencies die hierboven staan geldt:
  - $$$Inject gebruiken i.p.v. $$$Implementation (dat is de
  "oude" manier).
  - Geen instance nemen van de Context, maar de echte klassen
  newen (ook dat is oude vs nieuwe manier).
  
  En ook nog: Beetje vreemd om eerst de MaatwerkCalcApi op te
  halen en in een lokale variabele te steken (ook de echte klasse
  newen graag en dan kun je gelijk setten i.p.v. te dimmen), om
  dat dan direct daarna nog eens opnieuw te doen om de
  LadeRugUtils op te halen :-) 
  En daar bovenop... Als je de MaatwerkCalcApi gebruikt om er de
  LadeRugUtils af te halen, wat is het nut dan om een inject te
  doen van de LadeRugUtils zelf? Normaal ga je dan in je testen
  de MaatwerkCalcApi uitmocken
  (IndienMethodCall("GeefLadeRugUtils", "MVX")), dus het is
  eigenlijk de MaatwerkCalcApi die je moet injecteren en niet de
  RugUtils.

Revision Comment by Tommy Hebb on 21 April 2021, 15:22 defect
http://subversion02:8060/cru/CR-2293#c3696

  Returntype definiëren graag

Revision Comment by Tommy Hebb on 21 April 2021, 15:23 defect
http://subversion02:8060/cru/CR-2293#c3697

  MeldSynchronisatiestang? Moet die gemeld worden of zo? ;-)

Revision Comment by Tommy Hebb on 21 April 2021, 15:26 defect
http://subversion02:8060/cru/CR-2293#c3698

  Returntype vergeten ook

Revision Comment by Tommy Hebb on 21 April 2021, 15:28 defect
http://subversion02:8060/cru/CR-2293#c3699

  Zelfde als boven, moet er iets gemeld worden? ;-)

Revision Comment by Tommy Hebb on 21 April 2021, 15:29 
http://subversion02:8060/cru/CR-2293#c3700

  Macro's graag voluit in moderne code :-) 
  $Select
  $Length

----------------------------------------
File: APPS/Halux/PPS/Activiteit/impl/MVX/MVXLadePersen.cls.xml
Revision Comment by Tommy Hebb on 21 April 2021, 15:34 defect
http://subversion02:8060/cru/CR-2293#c3701

  Graag de echte newen en niet van de Context pakken

Revision Comment by Tommy Hebb on 21 April 2021, 15:35 
http://subversion02:8060/cru/CR-2293#c3702

  Macro's voluit... $Select

Revision Comment by Tommy Hebb on 21 April 2021, 15:36 
http://subversion02:8060/cru/CR-2293#c3703

  Macro's voluit... $Extract

Revision Comment by Tommy Hebb on 21 April 2021, 15:37 defect
http://subversion02:8060/cru/CR-2293#c3704

  Ook de echte newen ipv via de Context

Revision Comment by Tommy Hebb on 21 April 2021, 15:38 
http://subversion02:8060/cru/CR-2293#c3705

  Best haakjes rond zetten

Revision Comment by Tommy Hebb on 21 April 2021, 15:40 defect
http://subversion02:8060/cru/CR-2293#c3706

  Echte ProductieAPI newen

Revision Comment by Tommy Hebb on 21 April 2021, 15:40 
http://subversion02:8060/cru/CR-2293#c3707

  Grote D in Do ;-)

----------------------------------------
File: APPS/Halux/PPS/MVXPers/impl/Werkpost.cls.xml
Revision Comment by Tommy Hebb on 21 April 2021, 15:43 
http://subversion02:8060/cru/CR-2293#c3708

  %RegisteredObject mag je altijd direct vervangen door
  TECH.RegisteredObject

Revision Comment by Tommy Hebb on 21 April 2021, 16:21 
http://subversion02:8060/cru/CR-2293#c3709

  Misschien ook alvast de echte ProductApi hier ophalen ipv die
  van de Context

Revision Comment by Tommy Hebb on 21 April 2021, 16:23 defect
http://subversion02:8060/cru/CR-2293#c3710

  Returntype vergeten

----------------------------------------
File: APPS/VKP/Maatwerk/LadeRegistratie/impl/MVXDataOphaler.cls.xml
Revision Comment by Tommy Hebb on 21 April 2021, 16:25 
http://subversion02:8060/cru/CR-2293#c3711

  TECH.RegisteredObject

Revision Comment by Tommy Hebb on 21 April 2021, 16:27 defect
http://subversion02:8060/cru/CR-2293#c3712

  Variabele wordt nergens gebruikt

Revision Comment by Tommy Hebb on 21 April 2021, 16:29 defect
http://subversion02:8060/cru/CR-2293#c3713

  proper dimmen als string


---

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

Title: [ICT-2306] MVX: Persaansturing
Statement of Objectives:


State: Closed

Summary: 

Author: Peter Vercoutter
Moderator: Peter Vercoutter
Reviewers: (0 active, 1 completed*)
     Tommy Hebb   (*)