Index: TECH/ClassUtils.cls.xml
===================================================================
diff -u -r2141 -r2153
--- TECH/ClassUtils.cls.xml	(.../ClassUtils.cls.xml)	(revision 2141)
+++ TECH/ClassUtils.cls.xml	(.../ClassUtils.cls.xml)	(revision 2153)
@@ -1,8 +1,6 @@
 <?xml version="1.0" encoding="UTF-8"?>
 <Export generator="Cache" version="9">
 <Class name="TECH.ClassUtils">
-<Description>
-</Description>
 <ProcedureBlock>1</ProcedureBlock>
 <Super>%RegisteredObject</Super>
 <ClassDefinitionError>0</ClassDefinitionError>
@@ -88,6 +86,11 @@
 </Method>
 
 <Method name="ValueIsValid">
+<Description><![CDATA[
+[ Review CSC -> JBL ]
+Heeft het zin om een boolean terug te geven als er excepties worden gethrowed bij result = false?
+Method zal enkel true teruggeven. Meer zinvol zou zijn om het ValidateValue() te noemen zonder resultaat (enkel excepties bij foutief).
+Zo kan iedereen er van uitgaan dat hij de return value niet moet controleren.]]></Description>
 <ClassMethod>1</ClassMethod>
 <FormalSpec><![CDATA[&Value:%CacheString,ClassName:%String]]></FormalSpec>
 <ReturnType>%Boolean</ReturnType>
@@ -116,6 +119,9 @@
 </Method>
 
 <Method name="PropertiesValid">
+<Description><![CDATA[
+[ Review CSC -> JBL ]
+Idem als hierboven.]]></Description>
 <ClassMethod>1</ClassMethod>
 <FormalSpec><![CDATA[&Object:%CacheString]]></FormalSpec>
 <ReturnType>%Boolean</ReturnType>
Index: WS/PM/KenmerkenService.cls.xml
===================================================================
diff -u -r2148 -r2153
--- WS/PM/KenmerkenService.cls.xml	(.../KenmerkenService.cls.xml)	(revision 2148)
+++ WS/PM/KenmerkenService.cls.xml	(.../KenmerkenService.cls.xml)	(revision 2153)
@@ -101,6 +101,9 @@
 	$$$chkIsObject(ProductenKenmerken)
 	
 	Set Result = ""
+	// [ Review CSC -> JBL ]
+	// If test is zinloos vermits deze altijd true zal teruggeven. Gewoon de call doen is voldoen.
+	// Zie ook commentaar in class utils klasse zelf bij de method
 	If (##class(TECH.ClassUtils).ValueIsValid(ProductenKenmerken,"APPS.PM.dto.Kenm.ProductenKenmerken")){
 		Set Result = ##class(APPS.PM.KenmerkenService).%New().BewaarProductenKenmerken(ProductenKenmerken)
 	}
Index: WS/PM/ProductService.cls.xml
===================================================================
diff -u -r2131 -r2153
--- WS/PM/ProductService.cls.xml	(.../ProductService.cls.xml)	(revision 2131)
+++ WS/PM/ProductService.cls.xml	(.../ProductService.cls.xml)	(revision 2153)
@@ -27,6 +27,9 @@
 	$$$chkIsObject(Criteria)
 	
 	Set Result = ""
+	// [ Review CSC -> JBL ]
+	// If test is zinloos vermits deze altijd true zal teruggeven. Gewoon de call doen is voldoen.
+	// Zie ook commentaar in class utils klasse zelf bij de method
 	If (##class(TECH.ClassUtils).ValueIsValid(Criteria,"APPS.PM.dto.ZoekCriteria")){
 		Set Result = ##class(APPS.PM.ProductService).%New().ZoekProducten(Criteria)
 	}
Index: DOM/common/dto/ProductData.cls.xml
===================================================================
diff -u -r2147 -r2153
--- DOM/common/dto/ProductData.cls.xml	(.../ProductData.cls.xml)	(revision 2147)
+++ DOM/common/dto/ProductData.cls.xml	(.../ProductData.cls.xml)	(revision 2153)
@@ -18,6 +18,9 @@
 </Property>
 
 <Property name="Korttekst">
+<Description><![CDATA[
+[ Review CSC -> JBL ]
+Deze moet terug weg. Code is in dit geval de korttekst, waar omschrijving de langtekst is]]></Description>
 <Type>%String</Type>
 </Property>
 
Index: APPS/PM/KenmerkenService.cls.xml
===================================================================
diff -u -r2148 -r2153
--- APPS/PM/KenmerkenService.cls.xml	(.../KenmerkenService.cls.xml)	(revision 2148)
+++ APPS/PM/KenmerkenService.cls.xml	(.../KenmerkenService.cls.xml)	(revision 2153)
@@ -27,6 +27,10 @@
 </Property>
 
 <Method name="GeefAutoVariantenVanMasters">
+<Description><![CDATA[
+[ Review CSC -> JBL ]
+Behoort dit tot het consistenter maken van de parameters?
+Ik dacht dat het de bedoeling was dat er een combinatie van ProductID/UnID werd gevraagd als input?]]></Description>
 <FormalSpec>MasterUnIDs:%ListOfDataTypes,KenmerkGroep:%String</FormalSpec>
 <ReturnType>APPS.PM.dto.Kenm.AutoVariantenVanMasters</ReturnType>
 <Implementation><![CDATA[