This is a list of all comments for CR-1999. Review Summary: No summary ---------------------------------------- File: APPS/Halux/PPS/ALUFrees/Svg/KaderdeurElement/Beslag/Boringen.cls.xml Revision Comment by Tom Vermeulen on 04 March 2020, 08:37 http://subversion02:8060/cru/CR-1999#c3160 Vraagje voor WV: is het aan te raden om in het kader van 'nieuwe schrijfstijl' hier een BoringenIterator (als OneDimIterator ofzo) op te zetten ipv met de Array te werken, of is dat zeuren over details? Reply by Wim Vermeulen on 12 March 2020, 17:19 > Ja, ik geef ook de voorkeur aan nieuwe conventies, > namelijk : While (Iterator.HasNext() ) i.p.v. While > ($IsObject(Boring)) > > Merk op : testen met $$$HasLength() gebruiken we niet (meer) > bij objecten, mag alleen nog bij DataTypes, als %String. > Voor objecten is het correcter om te testen met $IsObject() > > Opm. 2: (naming convention) niet BoringenIterator maar > BoringIterator (enkelvoud) Revision Comment by Tom Vermeulen on 05 March 2020, 15:52 http://subversion02:8060/cru/CR-1999#c3195 fancy, kende deze syntax eigenlijk niet. ik zou een gewone && gebruikt hebben Dit wil wel zeggen dat je bij een DoorBoring ook altijd een Zijboring hebt. Klopt dat? (zie BoorType="Z" hierboven) Reply by Wim Vermeulen on 12 March 2020, 17:29 > Al opgelost? > Indien niet, dan mag je een "UnimplementedException" throwen > (voorbeeldjes via ctrl-shift-find) > Als je echt wil om deze te implementeren, dan kan deze boring > best voorgesteld worden door de combinatie van een "interne > zijboring" + een "externe zijboring". Revision Comment by Wim Vermeulen on 12 March 2020, 18:09 http://subversion02:8060/cru/CR-1999#c3219 Sam, deze implementatie gaan we in "pair reviewen" (wss rechtstreeks vanop de code). We kunnen hier immers vele kanten mee uit :-) ---------------------------------------- File: APPS/Halux/PPS/ALUFrees/Svg/KaderdeurElement/Boring/DoorBoring.cls.xml Revision Comment by Tom Vermeulen on 04 March 2020, 08:38 defect http://subversion02:8060/cru/CR-1999#c3161 Hier (en in andere XMLExport methods) moogt ge gerust een oneliner van maken: Quit ..SvgBox.XMLExport(jadajadajada) ---------------------------------------- File: APPS/Halux/PPS/ALUFrees/Svg/KaderdeurElement/Boring/RugBoring.cls.xml Revision Comment by Tom Vermeulen on 04 March 2020, 08:40 http://subversion02:8060/cru/CR-1999#c3164 normaalgezien geen commentlijnen laten staan. In dit geval wil je de inject blijkbaar niet meer gebruiken. Op zich kan het niet echt kwaad om hem te laten staan, omdat BasisElementFactory toch gedefault binnenkomt in de constructor Belangrijker: waarom niet meer injecten? Geen testen die daar gebruik van moeten maken? Revision Comment by Tom Vermeulen on 04 March 2020, 08:39 http://subversion02:8060/cru/CR-1999#c3162 #dim ipv #Dim Revision Comment by Tom Vermeulen on 04 March 2020, 08:42 defect http://subversion02:8060/cru/CR-1999#c3165 en ge moogt dit ineens in een Set steken Set CrossDiag = ##class(dinges).%New(....) Revision Comment by Tom Vermeulen on 04 March 2020, 08:39 http://subversion02:8060/cru/CR-1999#c3163 Do ipv DO ---------------------------------------- File: APPS/Halux/PPS/ALUFrees/Svg/BasisElement/CrossDiag.cls.xml Revision Comment by Tom Vermeulen on 05 March 2020, 15:47 defect http://subversion02:8060/cru/CR-1999#c3194 heb het hieronder al een paar gezegd $$$Not ipv ' Revision Comment by Wim Vermeulen on 12 March 2020, 16:56 http://subversion02:8060/cru/CR-1999#c3216 Beter : {code}  #dim sc As %Status Set sc = ..TopLine.XMLExport(...) Do ##class(TECH.Exceptions.StatusException).ThrowIndienNietOK(sc,"Error bij XmlExport van TopLine bij object "_$this) Set sc = ..BottomLine.XMLExport(...) Do ##class(TECH.Exceptions.StatusException).ThrowIndienNietOK(sc,"Error bij XmlExport van BottomLine bij object "_$this) Quit $$$OK {code} ---------------------------------------- File: APPS/Halux/PPS/ALUFrees/Svg/BasisElement/CrossVH.cls.xml Revision Comment by Tom Vermeulen on 05 March 2020, 15:47 defect http://subversion02:8060/cru/CR-1999#c3193 $$$Not ipv ' (en pas en passant de casing en spacing efkes aan (if hier en else hieronder) ---------------------------------------- File: APPS/Halux/PPS/ALUFrees/Svg/BasisElement/TCross.cls.xml Revision Comment by Tom Vermeulen on 04 March 2020, 08:48 defect http://subversion02:8060/cru/CR-1999#c3166 deze sc gaat overschreven worden door de volgende (2 lijnen lager). Dus: ofwel is ie echt zinloos en mag het weg, ofwel moet hieronder iets staan in de zin van "if sc heeft een of andere toestand, dan doe de VerticalLine-stuff" Uwen TCross-test had dat kunnen uitwijzen, als ge daar gaat asserten dat er twee lijnen aanwezig zijn Deze opmerking geldt voor de andere crossen ook Revision Comment by Tom Vermeulen on 05 March 2020, 15:46 defect http://subversion02:8060/cru/CR-1999#c3192 $$$Not ipv ' Revision Comment by Tom Vermeulen on 05 March 2020, 15:54 http://subversion02:8060/cru/CR-1999#c3196 zie opmerking bij Doorboring: dit kan gerust een oneliner worden. (idem bij andere BasisElementen) ---------------------------------------- File: vhUnitTest/APPS/Halux/PPS/ALUFrees/Svg/BasisElement/CrossDiag/Test.cls.xml Revision Comment by Tom Vermeulen on 04 March 2020, 08:28 http://subversion02:8060/cru/CR-1999#c3158 Voor alle testen hieronder: beter te testen op objectniveau ipv svg-strings: maw: een cross heeft twee horizontal lines (desnoods ook de properties van die dingen checken), maar hier hebben we het gisteren al over gehad :) ---------------------------------------- File: vhUnitTest/APPS/Halux/PPS/ALUFrees/Svg/KaderdeurElement/Beslag/Boringen/Test.cls.xml Revision Comment by Tom Vermeulen on 04 March 2020, 08:27 defect http://subversion02:8060/cru/CR-1999#c3157 deze test doet niet zo veel ---------------------------------------- File: APPS/Halux/PPS/ALUFrees/Svg/Kaderdeur.cls.xml Revision Comment by Tom Vermeulen on 04 March 2020, 08:32 http://subversion02:8060/cru/CR-1999#c3159 over het opsplitsen van deze lange method hebben we het gisteren ook gehad, dus daar ga ik nu geen epistel over schrijven (maar 't is maar voor het geval dat WV het zou lezen) --- ID: CR-1999 http://subversion02:8060/cru/CR-1999 Title: [ICT-1210] Statement of Objectives: State: Closed Summary: Author: Sam Van Hoey Moderator: Sam Van Hoey Reviewers: (0 active, 2 completed*) Tom Vermeulen (*) Wim Vermeulen (*)