Index: vhUnitTest/AXimpl/Dynamics/ProductItemUpsertIn/Converter/ProductItemConverter/MappingTest.cls.xml
===================================================================
diff -u -r47144 -r47152
--- vhUnitTest/AXimpl/Dynamics/ProductItemUpsertIn/Converter/ProductItemConverter/MappingTest.cls.xml	(.../MappingTest.cls.xml)	(revision 47144)
+++ vhUnitTest/AXimpl/Dynamics/ProductItemUpsertIn/Converter/ProductItemConverter/MappingTest.cls.xml	(.../MappingTest.cls.xml)	(revision 47152)
@@ -43,6 +43,11 @@
 <Private>1</Private>
 </Property>
 
+<Property name="LoggerMock">
+<Type>vhLib.Logger</Type>
+<Private>1</Private>
+</Property>
+
 <Method name="OnBeforeOneTest">
 <FormalSpec>aTestName:%String</FormalSpec>
 <Implementation><![CDATA[
@@ -64,8 +69,10 @@
 	
 	Set ..KlasRepositoryMock = ##class(vhTest.Mock.Prod.Klas.KlasRepository).%New()
 	Do ..KlasRepositoryMock.IndienMethodCall("ZoekViaGroepen","","","").DanReturn(..GeefKlasIterator())
+	
+	Set ..LoggerMock = ##class(vhTest.Mock.vhLib.Logger).%New()
 
-	Set ..Converter = ##class(AXimpl.Dynamics.ProductItemUpsertIn.Converter.ProductItemConverter).%New(..ProductAPIMock,..ProductRolAPIMock,..LangtekstBewaarderMock,..KlasRepositoryMock)
+	Set ..Converter = ##class(AXimpl.Dynamics.ProductItemUpsertIn.Converter.ProductItemConverter).%New(..ProductAPIMock,..ProductRolAPIMock,..LangtekstBewaarderMock,..KlasRepositoryMock,..LoggerMock)
 ]]></Implementation>
 </Method>
 
@@ -185,36 +192,33 @@
 ]]></Implementation>
 </Method>
 
-<Method name="Test voorlopig exceptie indien CertClaimLevel FSCRecycledCredit is">
+<Method name="Test voorlopig ErrorMail indien CertClaimLevel FSCRecycledCredit is">
 <Implementation><![CDATA[
-	#dim Substrings As %ListOfDataTypes = ##class(TECH.ListUtils).CreateListOfDataTypes("CertClaimLevel 'FSCRecycledCredit' kan niet worden geset voor product IdentNummer")
-	Do ..ZetVerwachteExceptie(##class(Tools.Test.VerwachteExceptie).%New(##class(TECH.Exceptions.IllegalArgumentException).%ClassName(1),Substrings))
-
+	Do ..LoggerMock.VerwachtMethodCall("ErrorMail","Productsynchronizatie AX - Admin","CertClaimLevel 'FSCRecycledCredit' kan niet worden geset voor product IdentNummer")
 	Set ..ProductItem.CertClaimLevel = "FSCRecycledCredit"
 
 	Do ..Converter.ConvertProductItem(..ProductItem)
+	Do ..LoggerMock.Verifieer()
 ]]></Implementation>
 </Method>
 
-<Method name="Test voorlopig exceptie indien CertClaimLevel FSC100 is">
+<Method name="Test voorlopig Errormail indien CertClaimLevel FSC100 is">
 <Implementation><![CDATA[
-	#dim Substrings As %ListOfDataTypes = ##class(TECH.ListUtils).CreateListOfDataTypes("CertClaimLevel 'FSC100' kan niet worden geset voor product IdentNummer")
-	Do ..ZetVerwachteExceptie(##class(Tools.Test.VerwachteExceptie).%New(##class(TECH.Exceptions.IllegalArgumentException).%ClassName(1),Substrings))
-
+	Do ..LoggerMock.VerwachtMethodCall("ErrorMail","Productsynchronizatie AX - Admin","CertClaimLevel 'FSC100' kan niet worden geset voor product IdentNummer")
 	Set ..ProductItem.CertClaimLevel = "FSC100"
 
 	Do ..Converter.ConvertProductItem(..ProductItem)
+	Do ..LoggerMock.Verifieer()
 ]]></Implementation>
 </Method>
 
-<Method name="Test exceptie indien CertClaimLevel een onbekende waarde heeft">
+<Method name="Test errormail indien CertClaimLevel een onbekende waarde heeft">
 <Implementation><![CDATA[
-	#dim Substrings As %ListOfDataTypes = ##class(TECH.ListUtils).CreateListOfDataTypes("CertClaimLevel 'OngeldigClaimLevel' kan niet worden geset voor product IdentNummer")
-	Do ..ZetVerwachteExceptie(##class(Tools.Test.VerwachteExceptie).%New(##class(TECH.Exceptions.IllegalArgumentException).%ClassName(1),Substrings))
-
+	Do ..LoggerMock.VerwachtMethodCall("ErrorMail","Productsynchronizatie AX - Admin","CertClaimLevel 'OngeldigClaimLevel' kan niet worden geset voor product IdentNummer")
 	Set ..ProductItem.CertClaimLevel = "OngeldigClaimLevel"
 
 	Do ..Converter.ConvertProductItem(..ProductItem)
+	Do ..LoggerMock.Verifieer()
 ]]></Implementation>
 </Method>
 
@@ -262,14 +266,13 @@
 ]]></Implementation>
 </Method>
 
-<Method name="Test Exceptie indien indien CountableOrDivisible ongeldige waarde bevat bevat">
+<Method name="Test errormail indien indien CountableOrDivisible ongeldige waarde bevat bevat">
 <Implementation><![CDATA[
-	#dim Substrings As %ListOfDataTypes = ##class(TECH.ListUtils).CreateListOfDataTypes("CountableOrDivisible 'OngeldigeWaarde' kan niet worden geset voor product IdentNummer")
-	Do ..ZetVerwachteExceptie(##class(Tools.Test.VerwachteExceptie).%New(##class(TECH.Exceptions.IllegalArgumentException).%ClassName(1),Substrings))
-
+	Do ..LoggerMock.VerwachtMethodCall("ErrorMail","Productsynchronizatie AX - Admin","CountableOrDivisible 'OngeldigeWaarde' kan niet worden geset voor product IdentNummer")
 	Set ..ProductItem.CountableOrDivisible = "OngeldigeWaarde"
 
 	Do ..Converter.ConvertProductItem(..ProductItem)
+	Do ..LoggerMock.Verifieer()
 ]]></Implementation>
 </Method>
 
@@ -328,7 +331,7 @@
 ]]></Implementation>
 </Method>
 
-<Method name="Test WerkvloerEenheid is ST indien InventUnid STK is">
+<Method name="Test WerkvloerEenheid is ST indien InventUnitId STK is">
 <Implementation><![CDATA[
 	Set ..ProductItem.InventUnitId = "STK"
 	Do ..ProductMock.VerwachtMethodCall("ZetWerkvloerEenheid","ST").DanDoeNiks()
@@ -339,7 +342,7 @@
 ]]></Implementation>
 </Method>
 
-<Method name="Test WerkvloerEenheid is CM indien InventUnid cm is">
+<Method name="Test WerkvloerEenheid is CM indien InventUnitId cm is">
 <Implementation><![CDATA[
 	Set ..ProductItem.InventUnitId = "cm"
 	Do ..ProductMock.VerwachtMethodCall("ZetWerkvloerEenheid","CM").DanDoeNiks()
@@ -350,7 +353,7 @@
 ]]></Implementation>
 </Method>
 
-<Method name="Test WerkvloerEenheid is CM2 indien InventUnid cm2 is">
+<Method name="Test WerkvloerEenheid is CM2 indien InventUnitId cm2 is">
 <Implementation><![CDATA[
 	Set ..ProductItem.InventUnitId = "cm2"
 	Do ..ProductMock.VerwachtMethodCall("ZetWerkvloerEenheid","CM2").DanDoeNiks()
@@ -361,7 +364,7 @@
 ]]></Implementation>
 </Method>
 
-<Method name="Test WerkvloerEenheid is G indien InventUnid g is">
+<Method name="Test WerkvloerEenheid is G indien InventUnitId g is">
 <Implementation><![CDATA[
 	Set ..ProductItem.InventUnitId = "g"
 	Do ..ProductMock.VerwachtMethodCall("ZetWerkvloerEenheid","G").DanDoeNiks()
@@ -372,7 +375,7 @@
 ]]></Implementation>
 </Method>
 
-<Method name="Test WerkvloerEenheid is M indien InventUnid m is">
+<Method name="Test WerkvloerEenheid is M indien InventUnitId m is">
 <Implementation><![CDATA[
 	Set ..ProductItem.InventUnitId = "m"
 	Do ..ProductMock.VerwachtMethodCall("ZetWerkvloerEenheid","M").DanDoeNiks()
@@ -383,7 +386,7 @@
 ]]></Implementation>
 </Method>
 
-<Method name="Test WerkvloerEenheid is M2 indien InventUnid m2 is">
+<Method name="Test WerkvloerEenheid is M2 indien InventUnitId m2 is">
 <Implementation><![CDATA[
 	Set ..ProductItem.InventUnitId = "cm"
 	Do ..ProductMock.VerwachtMethodCall("ZetWerkvloerEenheid","CM").DanDoeNiks()
@@ -394,7 +397,7 @@
 ]]></Implementation>
 </Method>
 
-<Method name="Test WerkvloerEenheid is MM indien InventUnid mm is">
+<Method name="Test WerkvloerEenheid is MM indien InventUnitId mm is">
 <Implementation><![CDATA[
 	Set ..ProductItem.InventUnitId = "mm"
 	Do ..ProductMock.VerwachtMethodCall("ZetWerkvloerEenheid","MM").DanDoeNiks()
@@ -405,7 +408,7 @@
 ]]></Implementation>
 </Method>
 
-<Method name="Test WerkvloerEenheid is MM2 indien InventUnid mm2 is">
+<Method name="Test WerkvloerEenheid is MM2 indien InventUnitId mm2 is">
 <Implementation><![CDATA[
 	Set ..ProductItem.InventUnitId = "mm2"
 	Do ..ProductMock.VerwachtMethodCall("ZetWerkvloerEenheid","MM2").DanDoeNiks()
@@ -416,13 +419,13 @@
 ]]></Implementation>
 </Method>
 
-<Method name="Test Exceptie indien niet toegelaten InventUnid ">
+<Method name="Test Exceptie indien niet toegelaten InventUnitId ">
 <Implementation><![CDATA[
-
-	#dim Substrings As %ListOfDataTypes = ##class(TECH.ListUtils).CreateListOfDataTypes("InventUnidId 'niet toegelaten' kan niet geconverteerd worden voor itemnummer IdentNummer")
-	Do ..ZetVerwachteExceptie(##class(Tools.Test.VerwachteExceptie).%New(##class(TECH.Exceptions.IllegalArgumentException).%ClassName(1),Substrings))
+	Do ..LoggerMock.VerwachtMethodCall("ErrorMail","Productsynchronizatie AX - Admin","InventUnidId 'niet toegelaten' kan niet geconverteerd worden voor itemnummer IdentNummer")
 	Set ..ProductItem.InventUnitId = "niet toegelaten"
+
 	Do ..Converter.ConvertProductItem(..ProductItem)
+	Do ..LoggerMock.Verifieer()
 ]]></Implementation>
 </Method>
 
Index: AXimpl/Dynamics/ProductItemUpsertIn/Converter/ProductItemConverter.cls.xml
===================================================================
diff -u -r47144 -r47152
--- AXimpl/Dynamics/ProductItemUpsertIn/Converter/ProductItemConverter.cls.xml	(.../ProductItemConverter.cls.xml)	(revision 47144)
+++ AXimpl/Dynamics/ProductItemUpsertIn/Converter/ProductItemConverter.cls.xml	(.../ProductItemConverter.cls.xml)	(revision 47152)
@@ -23,8 +23,13 @@
 <Private>1</Private>
 </Property>
 
+<Property name="Logger">
+<Type>vhLib.Logger</Type>
+<Private>1</Private>
+</Property>
+
 <Method name="%OnNew">
-<FormalSpec>ProductAPI:DOM.PM.ProductAPI,ProductRolAPI:DOM.PM.ProductRolAPI,LangtekstBewaarder:DOM.PM.LangtekstBewaarder,KlasRepository:Prod.Klas.KlasRepository</FormalSpec>
+<FormalSpec>ProductAPI:DOM.PM.ProductAPI,ProductRolAPI:DOM.PM.ProductRolAPI,LangtekstBewaarder:DOM.PM.LangtekstBewaarder,KlasRepository:Prod.Klas.KlasRepository,Logger:vhLib.Logger</FormalSpec>
 <Private>1</Private>
 <ProcedureBlock>1</ProcedureBlock>
 <ReturnType>%Status</ReturnType>
@@ -34,17 +39,24 @@
 	set ..ProductRolAPI = $$$Inject(ProductRolAPI,##class(DOM.DomeinContext).Instance().GeefProductRolAPI())
 	Set ..LangtekstBewaarder = $$$Inject(LangtekstBewaarder,##class(DOM.PM.LangtekstBewaarder).%New())
 	Set ..KlasRepository = $$$Inject(KlasRepository,##class(Prod.Klas.KlasRepository).%New())
+	Set ..Logger = $$$Inject(Logger,##class(vhLib.Logger).%New())
 	Quit $$$OK
 ]]></Implementation>
 </Method>
 
 <Method name="ConvertProductItem">
 <FormalSpec>ProductItem:AXif.Dynamics.ProductItemIn.MessageParts.AxdEntityItem</FormalSpec>
 <Implementation><![CDATA[
-	if ..ProductAPI.BestaatIdentNr(ProductItem.ItemId) { ; Update
-		Do ..UpdateProduct(ProductItem)
-	} else { ;insert
-		Do ..MaakProduct(ProductItem)
+	
+	try {	
+		if ..ProductAPI.BestaatIdentNr(ProductItem.ItemId) { ; Update	
+			Do ..UpdateProduct(ProductItem)
+		} else { ;insert
+			Do ..MaakProduct(ProductItem)
+		}
+	} catch {
+		#dim Exception As TECH.Exceptions.Exception = ##class(TECH.ExceptionHandler).Catch(##class(TECH.Exceptions.IllegalArgumentException).%ClassName(1))
+		Do ..Logger.ErrorMail("Productsynchronizatie AX - Admin",Exception.GeefOmschrijving())
 	}
 ]]></Implementation>
 </Method>