This is a list of all comments for CR-2021. Review Summary: No summary ---------------------------------------- File: APPS/Halux/PPS/ALUFrees/Svg/Kaderdeur/LosProfielToevoeger.cls.xml Revision Comment by Tom Vermeulen on 25 March 2020, 12:05 defect http://subversion02:8060/cru/CR-2021#c3263 indents ongedaan maken ---------------------------------------- File: APPS/Halux/PPS/ALUFrees/Svg/Kaderdeur.cls.xml Revision Comment by Tom Vermeulen on 25 March 2020, 12:13 defect http://subversion02:8060/cru/CR-2021#c3267 Deze locatie gaat ge toch ergens in een ConfigItem moeten steken en niet zo hardcoded laten staan ---------------------------------------- File: APPS/Halux/PPS/ALUFrees/Svg/KaderdeurElement/Boring/DoorBoring.cls.xml Revision Comment by Tom Vermeulen on 25 March 2020, 12:08 defect http://subversion02:8060/cru/CR-2021#c3265 Geen code uitvoeren in een constructor (behalve het newen van andere objecten). Code afzonderen naar andere method en die dan apart oproepen. Idem bij constructors andere boringen ---------------------------------------- File: APPS/Halux/PPS/ALUFrees/Svg/KaderdeurElement/Hoekverbinder.cls.xml Revision Comment by Tom Vermeulen on 25 March 2020, 12:11 http://subversion02:8060/cru/CR-2021#c3266 elementtoevoeger en factory hier niet newen, maar gezien dat de file al outdated is en deze opmerking niet meer relevant is (kleine kaartjes, rap committen, enz enz ;) ) ---------------------------------------- File: vhUnitTest/APPS/Halux/PPS/ALUFrees/Svg/KaderdeurElement/Beslag/Beslagen/Test.cls.xml Revision Comment by Tom Vermeulen on 25 March 2020, 12:16 defect http://subversion02:8060/cru/CR-2021#c3268 Ge hebt een Assign en Act gedeelte, maar genen Assert. Deze test doet dus eigenlijk niets ---------------------------------------- File: APPS/Halux/PPS/ALUFrees/Svg/KaderdeurElement/Beslag/Boringen.cls.xml Revision Comment by Tom Vermeulen on 25 March 2020, 12:06 defect http://subversion02:8060/cru/CR-2021#c3264 klasse newen in constructor --- ID: CR-2021 http://subversion02:8060/cru/CR-2021 Title: [ICT-1349] SVG-026 Toevoegen van SvgBox.ElementToevoeger Statement of Objectives: State: Closed Summary: Author: Sam Van Hoey Moderator: Sam Van Hoey Reviewers: (0 active, 2 completed*) Tom Vermeulen (*) Wim Vermeulen (*)