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

Review Summary: No summary

General Comment by Tom Vermeulen on 27 February 2020, 08:04 
http://subversion02:8060/cru/CR-1966#c3147

  ik denk dat ge uw testen vergeten committen zijt :)


----------------------------------------
File: APPS/Halux/PPS/ALUFrees/Svg/Kaderdeur.cls.xml
Revision Comment by Tom Vermeulen on 04 March 2020, 09:59 
http://subversion02:8060/cru/CR-1966#c3171

  algemene opmerking: je hebt hier in 1 commit verschillende
  dingen gedaan: enus gebruiken, kleuren veranderen, refactorke
  hier en daar. Maak er volgende keer gerust een hoop kleinere
  commitjes van

Revision Comment by Tom Vermeulen on 04 March 2020, 09:54 
http://subversion02:8060/cru/CR-1966#c3168

  misschien in een apart kaartje: al die strings "LPR", "GL", ...
  enz zouden misschien beter in een enumeratie gestoken worden.
  Er bestaan er nu al een hoop in DOM.PM.enu.KAD*

  Reply by Wim Vermeulen on 05 March 2020, 13:45
  > Ja, o.a. voor Toepassing, ProductieWijze kan je de
  > enu-waarden vinden in de excel-file :
  > E:\svn\Files\trunk\Excel\Halux_Apps_Data\KAD Res.PIItems.xls
  > Ook voor ProfielPlaats zou je een enu kunnen maken (hiervan
  > bestaat geen persistente klasse)

Revision Comment by Tom Vermeulen on 04 March 2020, 09:53 defect
http://subversion02:8060/cru/CR-1966#c3167

  geen  ' maar $$$Not   gebruiken   (staat op meerdere plaatsen
  blijkbaar)
  
  anders moet ge misschien eens ctrl-shift-f doen voor een single
  quote :)

Revision Comment by Tom Vermeulen on 04 March 2020, 09:57 defect
http://subversion02:8060/cru/CR-1966#c3169

  hiep hoi

Revision Comment by Tom Vermeulen on 04 March 2020, 09:58 
http://subversion02:8060/cru/CR-1966#c3170

  ge moet u echt niet inhouden om variabelen een langere naam te
  geven

  Reply by Wim Vermeulen on 05 March 2020, 13:51
  > Tja, een groot deel van de code heeft Sam overgenomen van
  > mijn oude code, vandaar (idem voor notaties als ... '= ... )
  > Maar ik deel de mening van Tom : gebruik gerust onze
  > "moderne" coding-conventions  :-)

Revision Comment by Tom Vermeulen on 27 February 2020, 08:03 defect
http://subversion02:8060/cru/CR-1966#c3146

  moeten deze twee methods geen private method zijn?

Revision Comment by Tom Vermeulen on 27 February 2020, 08:07 
http://subversion02:8060/cru/CR-1966#c3149

  Gingen deze "higher level" componenten ook naar een Factory of
  niet?

Revision Comment by Tom Vermeulen on 27 February 2020, 08:05 
http://subversion02:8060/cru/CR-1966#c3148

  zou het ok zijn om de ID en transformatie toe te voegen aan de
  "maak" method? Dan reduceert ge de lengte van deze methods een
  stuk en worden ze misschien gemakkelijker leesbaar? In de
  "maak" zelf wordt er namelijk ook al met de group gespeeld.

Revision Comment by Tom Vermeulen on 27 February 2020, 08:08 
http://subversion02:8060/cru/CR-1966#c3150

  Het zal waarschijnlijk iets SVG gerelateerd zijn, maar ik als
  leek weet niet wat Vcode is. Kan een andere naam de lading
  beter dekken? Of moet ik eerst maar wat meer SVG stuff leren?
  Vooral hieronder Vcode = 2 en Vcode = 1  zegt me echt niets :)

Revision Comment by Tom Vermeulen on 27 February 2020, 08:11 
http://subversion02:8060/cru/CR-1966#c3151

  Over deze method: (ik kan compleet mis zijn, 't is nog vroeg).
  Als je de inputvariabele ProfPlaats van de vorige method niet
  uit een gewone string PO, PB, ... laat bestaan, maar uit een
  lijst van strings (of een concatenate), kan je dan de
  functionaliteit van beide methods niet in 1 method steken? door
  niet met ProfPlaats="XXX" maar door met ProfPlaats [ "XXX" te
  werken. Alternatief: laat naast de vier huidige Values voor
  ProfPlaats ook een 'ALL' toe die ze alle vier toevoegt.

----------------------------------------
File: APPS/Halux/PPS/ALUFrees/Svg/KaderdeurElement/Hoekverbinder.cls.xml
Revision Comment by Tom Vermeulen on 04 March 2020, 10:02 
http://subversion02:8060/cru/CR-1966#c3173

  ge zijt uw testen vergeten committen

Revision Comment by Tom Vermeulen on 04 March 2020, 10:02 
http://subversion02:8060/cru/CR-1966#c3172

  oneliner? zie een of ander ander kaartje dat ik vandaag
  gereviewed heb


---

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

Title: [ICT-1203]  SVG-015 Verbinders toevoegen aan kaderdeur
Statement of Objectives:


State: Closed

Summary: 

Author: Sam Van Hoey
Moderator: Sam Van Hoey
Reviewers: (0 active, 2 completed*)
     Tom Vermeulen   (*)
     Wim Vermeulen   (*)