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