Index: vhUnitTest/DOM/PM/impl/TestIngegevenKenmerkenAPI.cls.xml =================================================================== diff -u -r1809 -r1851 --- vhUnitTest/DOM/PM/impl/TestIngegevenKenmerkenAPI.cls.xml (.../TestIngegevenKenmerkenAPI.cls.xml) (revision 1809) +++ vhUnitTest/DOM/PM/impl/TestIngegevenKenmerkenAPI.cls.xml (.../TestIngegevenKenmerkenAPI.cls.xml) (revision 1851) @@ -48,6 +48,8 @@ Do ##class(DOM.PM.Maatwerk.Calc.Common.IngegevenKenmerken).%DeleteId(TAOOKenmerken.%Id()) Do ##class(DOM.PM.Maatwerk.Calc.Common.impl.IKIDHash).%DeleteId(IKID.GeefHash()) Do ##class(DOM.PM.Maatwerk.Calc.Common.IKID).%DeleteId(IKID.GeefHash()_"||"_IKID.GeefOverflowID()) + + ; [Review CSC WV IKID] Ge kunt nog testen dat een wijziging aan het TAOOKenmerken-object hierboven resulteert in een verschillend IKID (object) ]]> Index: DOM/PM/Maatwerk/Calc/Common/IKCanonischeWaardeGenerator.cls.xml =================================================================== diff -u -r1805 -r1851 --- DOM/PM/Maatwerk/Calc/Common/IKCanonischeWaardeGenerator.cls.xml (.../IKCanonischeWaardeGenerator.cls.xml) (revision 1805) +++ DOM/PM/Maatwerk/Calc/Common/IKCanonischeWaardeGenerator.cls.xml (.../IKCanonischeWaardeGenerator.cls.xml) (revision 1851) @@ -3,6 +3,7 @@ 1 DOM.PM.Maatwerk.Calc.Common.IngegevenKenmerkenVisitor +0 %String @@ -16,9 +17,7 @@ %String - @@ -36,7 +35,6 @@ ";"_TBXKenmerken.ZijwandVerhogingKleur_";"_TBXKenmerken.ZijwandVerhogingAantal Quit $$$OK - ]]> @@ -70,7 +68,6 @@ // boringen en vullingen sorteren met comparator. Nog goede comparator te bedenken. Mogelijks gewoon soort toString maken en op string sorteren. Quit $$$OK - ]]> @@ -82,7 +79,6 @@ Do ..VerwerkIngegevenKenmerken(OLMKenmerken) Set ..CanonischeWaarde = ..CanonischeWaarde_";"_OLMKenmerken.InTeKortenPRNr_";"_OLMKenmerken.TeBekomenLengte Quit $$$OK - ]]> @@ -94,11 +90,13 @@ Do ..VerwerkIngegevenKenmerken(TAOOKenmerken) Set ..CanonischeWaarde = ..CanonischeWaarde_";"_TAOOKenmerken.Draagkracht_";"_TAOOKenmerken.LadeKleur_";"_TAOOKenmerken.LadeHoogte_";"_TAOOKenmerken.CorpusBinnenbreedte_";"_TAOOKenmerken.IsBinnenlade_";"_TAOOKenmerken.LadeDiepte Quit $$$OK - ]]> + +[Review CSC WV IKID] methodnaam is niet erg duidelijk + best een korte woordje commentaar bijvoegen, nl dat de properties van de parent-class aan de string worden toegevoegd (maar dat kan jij wel beter verwoorden) IngegevenKenmerken:DOM.PM.Maatwerk.Calc.Common.IngegevenKenmerken 1 @@ -119,7 +116,6 @@ Index: vhUnitTest/DOM/PM/impl/TestIKCanonischeWaarde.cls.xml =================================================================== diff -u -r1805 -r1851 --- vhUnitTest/DOM/PM/impl/TestIKCanonischeWaarde.cls.xml (.../TestIKCanonischeWaarde.cls.xml) (revision 1805) +++ vhUnitTest/DOM/PM/impl/TestIKCanonischeWaarde.cls.xml (.../TestIKCanonischeWaarde.cls.xml) (revision 1851) @@ -37,7 +37,7 @@ Set OLMKenmerken = ##class(DOM.PM.Maatwerk.Calc.Common.impl.OLMKenmerken).%New() // Deze houden hier geen steek, maar gewoon als test zijn ze nuttig Set OLMKenmerken.ProductieWijze = "Volledig" - Do OLMKenmerken.NietMeeleveren.Insert("Zijkant") + Do OLMKenmerken.NietMeeleveren.Insert("Zijkant") ; [Review CSC WV IKID] ik zou hier voor abstracte namen kiezen bvb. RolA en RolB Do OLMKenmerken.NietMeeleveren.Insert("Bodem") Do OLMKenmerken.Onderdelen.Insert("Rug") Do OLMKenmerken.Onderdelen.Insert("Glijder") Index: DOM/DomeinContext.cls.xml =================================================================== diff -u -r1803 -r1851 --- DOM/DomeinContext.cls.xml (.../DomeinContext.cls.xml) (revision 1803) +++ DOM/DomeinContext.cls.xml (.../DomeinContext.cls.xml) (revision 1851) @@ -205,6 +205,8 @@ +> copy-paste probleem :-P ]]> DOM.VKP.PrijsUitzAPI