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

Review Summary: No summary


----------------------------------------
File: vhUnitTest/TECH/Svg/Basis/Circle/Test.cls.xml
Revision Comment by Tom Vermeulen on 30 January 2020, 11:22 defect
http://subversion02:8060/cru/CR-1944#c2965

  zoals gisteren bij een andere klasse gezegd: AssertSVG is goed
  en wel, maar je moet ook testen voorzien die de values van de
  properties zelf, voor het geval de SVG export gewijzigd wordt.

----------------------------------------
File: TECH/Svg/Basis/Circle.cls.xml
Revision Comment by Tom Vermeulen on 30 January 2020, 11:17 
http://subversion02:8060/cru/CR-1944#c2964

  zou het interessanter zijn om ipv CX, CY en R iets als
  MiddelpuntX, MiddelpuntY en Straal mee te geven? In dit geval
  vrij triviaal, maar ge moet altijd stilstaan bij het feit dat
  het nu triviaal lijkt omdat ge er mee bezig zijt, maar over een
  paar jaar is het misschien minder triviaal. Ge hebt de
  voorbeelden eens gezien van onze oude .mac routines met
  allemaal variabelenamen van 1 of 2 letters :)  De properties
  zelf mogen onaangeroerd blijven natuurlijk, ik heb het enkel
  over de variabelen die hier staan (of op een andere publieke
  method)
  (ongeacht of het al dan niet in de constructor zelf hoort te
  zitten, zie opmerking bij rectangle)

  Reply by Wim Vermeulen on 30 January 2020, 14:19
  > Akkoord met Tom.
  > bijkomende vraag/bedenking : de voluit-benamingen ook
  > gebruiken als PropertyNaam? En dus alleen de XmlName blijft
  > CX, CY, R.
  > Of bedoelde Tom enkel om de function argument namen aan te
  > passen?

    Reply by Tom Vermeulen on 31 January 2020, 10:23
    > minstens al de argumenten, maar je zou inderdaad de
    > properties zelf ook kunnen wijzigen en spelen met de
    > XmlName


---

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

Title: [ICT-1131] SVG-004 aanmaken van klasse cirkel
Statement of Objectives:


State: Closed

Summary: 

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