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

Review Summary: No summary


----------------------------------------
File: APPS/Halux/PPS/ALUFrees/Svg/KaderdeurElement/Boring/BoringElementFactory.cls.xml
Revision Comment by Wim Vermeulen on 02 April 2020, 12:06 
http://subversion02:8060/cru/CR-2014#c3310

  beetje muggeziften, maar maakt de method wel iets
  overzichtlijker : 
   --> zet de 2 lijnen met Doorboring.SvgBox mee bovenaan, dan
  staan de (3) lijnen met Doorboring-object bij elkaar en de (3)
  lijnen met Circle-object staan dan ook bij elkaar.
  
  Een andere volgorde, op basis van een andere logica, is
  wellicht ook oké.

Revision Comment by Tom Vermeulen on 23 March 2020, 10:38 defect
http://subversion02:8060/cru/CR-2014#c3244

  well well well :)  (en er staan er nog een paar)

----------------------------------------
File: vhUnitTest/APPS/Halux/PPS/ALUFrees/Svg/KaderdeurElement/Boring/BoringELementFactory/Test.cls.xml
Revision Comment by Wim Vermeulen on 02 April 2020, 11:50 
http://subversion02:8060/cru/CR-2014#c3307

  Zelfde opmerking als bij ICT-1374 CrossFactory UT : 
  DummyCirkel en (misschien) DummySvgBox  --> dit laatste is
  mogelijk een beetje tricky :-P

Revision Comment by Wim Vermeulen on 02 April 2020, 11:53 
http://subversion02:8060/cru/CR-2014#c3308

  Zelfde opmerking als bij ICT-1374 CrossFactory UT : 
  Welgemikte getallen kiezen : bvb MaakBoring (60, 40, 5,
  Kleur.blauw)
  
  Een essentiele verantwoordelijkheid van de UT is om bij
  (x,y)-coordinaten een foutieve (y,x)-switch te detecteren.

----------------------------------------
File: APPS/Halux/PPS/ALUFrees/Svg/KaderdeurElement/Boring/DoorBoring.cls.xml
Revision Comment by Tom Vermeulen on 23 March 2020, 10:40 
http://subversion02:8060/cru/CR-2014#c3245

  zie opmerking in kaartje van CrossFactory

----------------------------------------
File: vhUnitTest/APPS/Halux/PPS/ALUFrees/Svg/KaderdeurElement/Boring/DoorBoring/Test.cls.xml
Revision Comment by Tom Vermeulen on 23 March 2020, 10:42 defect
http://subversion02:8060/cru/CR-2014#c3246

  deze test gaat niet echt uitgevoerd worden. Ofwel wel iets
  (nuttigs) laten doen, ofwel wegsmijten (idem voor andere
  boringtesten)

  Reply by Wim Vermeulen on 02 April 2020, 12:04
  > Deze klasse mag idd weg.


---

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

Title: [ICT-1367] SVG-030 Aanmaken BoringElementFactory
Statement of Objectives:


State: Closed

Summary: 

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