This is a list of all comments for CR-2023. Review Summary: No summary ---------------------------------------- File: APPS/Halux/PPS/ALUFrees/Svg/Kaderdeur.cls.xml Revision Comment by Wim Vermeulen on 02 April 2020, 16:21 http://subversion02:8060/cru/CR-2023#c3318 Deze method gaan we eens samen bekijken: - afsplitsen in meerdere private methods : bevordert de leesbaarheid, maar bovenal vermijdt dependencies tussen stukken code die niks met mekaar te maken hebben. - oproep/interface geschikt maken om vanuit Prod.GADef.Kaderdeur op te roepen. - UT's rond GenereerSvg() zou ik een andere aanpak voorstellen. Contact me :-) ---------------------------------------- File: vhUnitTest/APPS/Halux/PPS/ALUFrees/Svg/KaderdeurElement/Beslag/Boringen/VoegToeBeslagBoringen/Test.cls.xml Revision Comment by Wim Vermeulen on 02 April 2020, 16:45 http://subversion02:8060/cru/CR-2023#c3319 Ik ben nogal verward hoe deze test is opgebouwd. Mogelijk is dit veroorzaakt door misleidende naamgevingen. Ook conceptueel lijkt er iets niet te kloppen. Laten we zeggen dat de test aan het licht brengt dat de interfaces niet helemaal goed zitten. Aan deze UT's (en code) gaan we best wat knutselen. Contact me :-) --- ID: CR-2023 http://subversion02:8060/cru/CR-2023 Title: [ICT-1285] Statement of Objectives: State: Closed Summary: Author: Sam Van Hoey Moderator: Sam Van Hoey Reviewers: (0 active, 2 completed*) Tom Vermeulen (*) Wim Vermeulen (*)