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

Review Summary: No summary


----------------------------------------
File: DOM/PM/Maatwerk/Calc/Common/VhConfigHelper.cls.xml
Revision Comment by Jelle Baele on 11 March 2024, 11:15 
http://subversion02:8060/cru/CR-2706#c4300

  Ik zou dit niet als constructor porperty meegeven, want nu moet
  je overal waar je de VHConfigHelper wilt gebruiken, eerst hem
  initialiseren in de code zelf. Op die manier kan je moeilijker
  de vhconfighelper injecteren.
  
  Er is wel de method 'ZetVhConfig', maar die moet je dan eerst
  aanroepen vooraleer je de andere methods kan gebruiken. Ergens
  lijkt mij dat wat zot, omdat je dan snel fouten kan maken

Revision Comment by Jelle Baele on 11 March 2024, 11:27 
http://subversion02:8060/cru/CR-2706#c4301

  ProductApi is nergens gedefinieerd als input parameter, dus
  deze inject is niet nodig

Revision Comment by Jelle Baele on 11 March 2024, 11:29 
http://subversion02:8060/cru/CR-2706#c4302

  Rol doet niets, mag weg

Revision Comment by Jelle Baele on 01 March 2024, 16:26 
http://subversion02:8060/cru/CR-2706#c4290

  In enu steken en hier ook naar verwijzen?

----------------------------------------
File: DOM/PM/Maatwerk/Calc/Common/VhConfig/Converter/HalffabItemsBoom.cls.xml
Revision Comment by Jelle Baele on 02 April 2024, 11:23 
http://subversion02:8060/cru/CR-2706#c4328

  Ik had deze in de klassenaam zelf misschien ook "converter"
  bijgezet. Nu is dit HalffabItemsboom, maar is op die manier
  niet echt duidelijk dat het om een converter gaat zonder naar
  de folder te kijken (zelfde voor LBXKenmerken)

Revision Comment by Jelle Baele on 02 April 2024, 11:15 
http://subversion02:8060/cru/CR-2706#c4326

  Samengesteld ipv SamenGesteld

Revision Comment by Jelle Baele on 02 April 2024, 11:20 
http://subversion02:8060/cru/CR-2706#c4327

  Zijn deze lijnen in commentaar nog relevant?

----------------------------------------
File: DOM/PM/Maatwerk/Calc/Common/impl/OmleidingNaarEncowayBepaler.cls.xml
Revision Comment by Jelle Baele on 02 April 2024, 11:14 
http://subversion02:8060/cru/CR-2706#c4325

  'Dotnet' -> Encoway

Revision Comment by Jelle Baele on 07 March 2024, 13:47 
http://subversion02:8060/cru/CR-2706#c4295

  Ik zou deze in een config item plaatsen

----------------------------------------
File: DOM/PM/impl/DataM/DataMLBXLadeCreator.cls.xml
Revision Comment by Jelle Baele on 01 March 2024, 16:23 
http://subversion02:8060/cru/CR-2706#c4289

  Mijn gevoel is om dit ook in een aparte klasse te plaatsen en
  onder test te steken (ook al zal deze klasse dan vrij klein
  zijn)

----------------------------------------
File: DOM/PM/impl/DataM/DataMProductCreator.cls.xml
Revision Comment by Jelle Baele on 01 March 2024, 16:19 
http://subversion02:8060/cru/CR-2706#c4287

  Talen kunnen eventueel in een aparte enu?

----------------------------------------
File: vhUnitTest/DOM/PM/Maatwerk/Calc/Common/impl/OmleidingNaarDotNetBepaler/IsMaatwerkCalculatieInEncoway/Test.cls.xml
Revision Comment by Jelle Baele on 01 March 2024, 16:12 
http://subversion02:8060/cru/CR-2706#c4283

  Variabele naam aanpassen naar Encoway ipv dotnet

----------------------------------------
File: vhUnitTest/DOM/PM/Maatwerk/Calc/Common/impl/OmleidingNaarEncowayBepaler/IsMaatwerkCalculatieInEncoway/Test.cls.xml
Revision Comment by Jelle Baele on 01 March 2024, 16:12 
http://subversion02:8060/cru/CR-2706#c4284

  Variabele naam aanpassen naar Encoway ipv dotnet


---

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

Title: [ICT-3982] Productcreatie in cache voor encowayconfiguraties
Statement of Objectives:


State: Closed

Summary: 

Author: Peter Vercoutter
Moderator: Peter Vercoutter
Reviewers: (0 active, 1 completed*)
     Jelle Baele   (*)