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

Review Summary: No summary


----------------------------------------
File: APPS/Halux/PPS/ALUFrees/Svg/Kaderdeur/LosProfielToevoeger.cls.xml
Revision Comment by Wim Vermeulen on 02 April 2020, 12:52 
http://subversion02:8060/cru/CR-2006#c3311

  Deze lijn kan onder de if-structuur, moet niet telkens herhaald
  worden binnen de if.

Revision Comment by Wim Vermeulen on 02 April 2020, 12:53 
http://subversion02:8060/cru/CR-2006#c3312

  Indentatie (extreem rechts ?  ;-)  )

----------------------------------------
File: APPS/Halux/PPS/ALUFrees/Svg/KaderdeurElement/Beslag/BeslagBoringSpecialeVorm.cls.xml
Revision Comment by Tom Vermeulen on 10 March 2020, 17:04 defect
http://subversion02:8060/cru/CR-2006#c3213

  coordinatesservice en factory injecteren (en gebruiken in
  testen)

----------------------------------------
File: vhUnitTest/TECH/Svg/Services/CoordinatesService/MaakCoordinatenLijstVanCoordinaten/Test.cls.xml
Revision Comment by Tom Vermeulen on 19 March 2020, 14:58 defect
http://subversion02:8060/cru/CR-2006#c3236

  na opmerking van wim: testen toevoegen voor extra gedrag.
  Wat als er 0,1,2, veel coordinaten worden meegegeven?

----------------------------------------
File: TECH/Svg/Services/CoordinatesService.cls.xml
Revision Comment by Wim Vermeulen on 16 March 2020, 13:14 
http://subversion02:8060/cru/CR-2006#c3222

  Deze method is een leuke oefening (vanuit educatief standpunt),
  maar in de praktijk vind ik dit een nogal tricky interface.
  
  Als we deze willen blijven gebruiken/ondersteunen, dan is heb
  ik een review-opmerking, namelijk : deze moet een
  InvalidInput-exception smijten indien het aantal "Punten"
  oneven is.
  Let op : het tellen van Punten... is geen one-liner !
  
  Ofwel overwegen om aan te passen naar $LB(X,Y) parameters, i.e.
  {code}MaakCoordinatenLijstVanXYPunten(lbXYPunten... As %List)
  {code}

Revision Comment by Tom Vermeulen on 23 March 2020, 10:27 defect
http://subversion02:8060/cru/CR-2006#c3242

  private

Revision Comment by Tom Vermeulen on 23 March 2020, 10:26 defect
http://subversion02:8060/cru/CR-2006#c3241

  aaa, the good old wlip

Revision Comment by Tom Vermeulen on 23 March 2020, 10:32 
http://subversion02:8060/cru/CR-2006#c3243

  normaalgezien zou je heel deze if-elseif-else  kunnen vervangen
  door Quit (Totaal && ##class(TECH.Math).IsEven(Totaal))

  Reply by Sam Van Hoey on 23 March 2020, 10:43
  > Ook doen of is het dan niet leesbaar genoeg?


---

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

Title: [ICT-1340]
Statement of Objectives:


State: Closed

Summary: 

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