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

Review Summary: No summary


----------------------------------------
File: vhUnitTest/APPS/Halux/PPS/ALUFrees/Svg/Kaderdeur/GenereerSvg.cls.xml
Revision Comment by Wim Vermeulen on 15 May 2020, 11:51 
http://subversion02:8060/cru/CR-2057#c3422

  Standaard regel voor onze UT's : naamgeving = "vhUnitTest." +
  naam van de klasse " + "." + naam van de methode "." + "Test" 
  Dus in dit geval was het bijna juist  :-) 
   -->
  vhUnitTest.APPS.Halux.PPS.ALUFrees.Svg.Kaderdeur.GenereerSvg.Test
  
  Er zijn uitzonderingen, maar hier is standaard regel zeker oké.
  Graag ook je andere UT's eens nakijken.  Aanpassen kan je best
  via : 
  {code}Do ##class(Tools.CacheSourceUtils).MoveClass("",""){code}

Revision Comment by Tom Vermeulen on 08 May 2020, 10:22 
http://subversion02:8060/cru/CR-2057#c3412

  kommaneukerij, maar ik zou "uitvoeren van methodes" niet in de
  testnaam zetten, anders kan hij misschien te lang worden en 't
  is nogal triviaal

----------------------------------------
File: vhTest/Utils/APPS/Halux/PPS/ALUFrees/Svg/Kaderdeur.cls.xml
Revision Comment by Tom Vermeulen on 08 May 2020, 10:19 defect
http://subversion02:8060/cru/CR-2057#c3410

  niet gekeken wat hierboven nog gebeurt, maar je kan best 
  w !,"standaard ....
  gebruiken. Dat ! geeft een CRLF

  Reply by Tom Vermeulen on 08 May 2020, 10:20
  > toch eens naar de rest gekeken en vraag me af wat de zin is
  > van die write (buiten tijdens development) - > weg die handel
  > :)


---

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

Title: [ICT-1458] SVG-033 Kaderdeur opsplitsen in private methods
Statement of Objectives:


State: Closed

Summary: 

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