Index: vhUnitTest/AXimpl/Dynamics/ProductItemUpsertIn/Converter/ProductItemConverter/Test.cls.xml
===================================================================
diff -u -r49563 -r49691
--- vhUnitTest/AXimpl/Dynamics/ProductItemUpsertIn/Converter/ProductItemConverter/Test.cls.xml	(.../Test.cls.xml)	(revision 49563)
+++ vhUnitTest/AXimpl/Dynamics/ProductItemUpsertIn/Converter/ProductItemConverter/Test.cls.xml	(.../Test.cls.xml)	(revision 49691)
@@ -99,8 +99,6 @@
 	Do $$$AssertEquals(Converter.NieuwProductGegeven,$$$False)
 	Do $$$AssertEquals(Converter.IndexenVerwijderd,$$$True)
 	Do $$$AssertEquals(Converter.IndexenAangemaakt,$$$True)
-	Do $$$AssertFalse(Converter.ProductIsGelocked)
-	Do $$$AssertTrue(Converter.ProductIsGelockedGeweest)
 	
 	Do ProductMock.Verifieer()
 	Do ProductAPIMock.Verifieer()
@@ -138,38 +136,6 @@
 ]]></Implementation>
 </Method>
 
-<Method name="Test Exceptie indien product niet gelocked kan worden">
-<Implementation><![CDATA[
-	#dim ProductMock as vhTest.Mock.PM.DOM.Product = ##class(vhTest.Mock.DOM.PM.Product).%New()
-	Do ProductMock.IndienMethodCall("IsGenerischProduct").DanReturn($$$False)
-	Do ProductMock.IndienMethodCall("GeefID").DanReturn("ProductID")
-
-	#dim ProductAPIMock as DOM.PM.ProductAPI = ##class(vhTest.Mock.DOM.PM.ProductAPI).MockInstance()
-	Do ProductAPIMock.IndienMethodCall("BestaatIdentNr","IdentNummer").DanReturn($$$True)
-	Do ProductAPIMock.IndienMethodCall("GeefProductVolgensIdentNr","IdentNummer").DanReturn(ProductMock)
-
-	#dim ProductRolAPIMock as DOM.PM.ProductRolAPI = ##class(vhTest.Mock.DOM.PM.ProductRolAPI).MockInstance()
-	Set LangtekstBewaarderMock = ##class(vhTest.Mock.DOM.PM.LangtekstBewaarder).%New()
-	Set KlasRepositoryMock = ##class(vhTest.Mock.Prod.Klas.KlasRepository).%New()
-	Set PakketServiceMock = ##class(vhTest.Mock.DOM.PM.impl.DataM.PakketService).%New()
-	Set Converter = ##class(vhUnitTest.AXimpl.Dynamics.ProductItemUpsertIn.Converter.ProductItemConverter.ProductItemConverterFake).%New(ProductAPIMock,ProductRolAPIMock,LangtekstBewaarderMock,KlasRepositoryMock,,,PakketServiceMock)
-	Set Converter.ReturnLockException = $$$True
-	
-	#dim Envelope as AXif.Dynamics.ProductItemIn.Envelope = ..GeefEnvelope("ProductItemInUpdateKorttekst.xml")
-
-	#dim Substrings as %ListOfDataTypes = ##class(TECH.ListUtils).CreateListOfDataTypes("Product met ID 'ProductID' is reeds gelocked")
-	Do ..ZetVerwachteExceptie(##class(Tools.Test.VerwachteExceptie).%New(##class(TECH.Exceptions.LockedException).%ClassName(1),Substrings))
-
-	Do Converter.ConvertProductItem(Envelope.Body.MessageParts.MessagePart.Item.GetAt(1),"vhbe")
-	
-	Do $$$AssertEquals(Converter.DoorgestuurdNaarEWMS,$$$False)
-	Do $$$AssertEquals(Converter.NieuwProductGegeven,$$$False)
-	Do $$$AssertEquals(Converter.IndexenAangepast,$$$False)
-	Do $$$AssertFalse(Converter.ProductIsGelocked)
-	Do $$$AssertFalse(Converter.ProductIsGelockedGeweest)
-]]></Implementation>
-</Method>
-
 <Method name="Test geen verwerking indien inconsistentie in product naam over de verschillende talen">
 <Implementation><![CDATA[
 	#dim ProductAPIMock as DOM.PM.ProductAPI = ##class(vhTest.Mock.DOM.PM.ProductAPI).MockInstance()
@@ -252,8 +218,6 @@
 	Do $$$AssertEquals(Converter.NieuwProductGegeven,$$$True)
 	Do $$$AssertEquals(Converter.IndexenVerwijderd,$$$False)
 	Do $$$AssertEquals(Converter.IndexenAangemaakt,$$$True)
-	Do $$$AssertFalse(Converter.ProductIsGelocked)
-	Do $$$AssertFalse(Converter.ProductIsGelockedGeweest)
 	
 	Do ProductMock.Verifieer()
 	Do ProductAPIMock.Verifieer()
Index: vhUnitTest/AXimpl/Dynamics/ProductItemUpsertIn/Converter/ProductItemConverter/ProductItemConverterFake.cls.xml
===================================================================
diff -u -r49562 -r49691
--- vhUnitTest/AXimpl/Dynamics/ProductItemUpsertIn/Converter/ProductItemConverter/ProductItemConverterFake.cls.xml	(.../ProductItemConverterFake.cls.xml)	(revision 49562)
+++ vhUnitTest/AXimpl/Dynamics/ProductItemUpsertIn/Converter/ProductItemConverter/ProductItemConverterFake.cls.xml	(.../ProductItemConverterFake.cls.xml)	(revision 49691)
@@ -23,18 +23,6 @@
 <Type>%Boolean</Type>
 </Property>
 
-<Property name="ProductIsGelocked">
-<Type>%Boolean</Type>
-</Property>
-
-<Property name="ProductIsGelockedGeweest">
-<Type>%Boolean</Type>
-</Property>
-
-<Property name="ReturnLockException">
-<Type>%Boolean</Type>
-</Property>
-
 <Method name="GeefNieuwProduct">
 <ReturnType>DOM.PM.Product</ReturnType>
 <Implementation><![CDATA[
@@ -60,21 +48,10 @@
 	Set ..DoorgestuurdNaarEWMS = $$$False
 	Set ..IndexenAangemaakt = $$$False
 	Set ..IndexenVerwijderd = $$$False
-	Set ..ReturnLockException = $$$False
 	Quit ##super(ProductAPI,ProductRolAPI,LangtekstBewaarder,KlasRepository,Logger,MailAPI,PakketService)
 ]]></Implementation>
 </Method>
 
-<Method name="LockProduct">
-<FormalSpec>ProductID:%String</FormalSpec>
-<Private>1</Private>
-<Implementation><![CDATA[
-	Do:..ReturnLockException ##class(TECH.ExceptionHandler).Throw(##class(TECH.Exceptions.LockedException).%New("Product met ID '"_ProductID_"' is reeds gelocked"))
-	Set ..ProductIsGelocked = $$$True
-	Set ..ProductIsGelockedGeweest = $$$True
-]]></Implementation>
-</Method>
-
 <Method name="VerwijderIndexen">
 <FormalSpec>ProductID:%String</FormalSpec>
 <Private>1</Private>
@@ -88,13 +65,6 @@
 <Implementation><![CDATA[	Set ..IndexenAangemaakt = $$$True
 ]]></Implementation>
 </Method>
-
-<Method name="UnLockProduct">
-<FormalSpec>ProductID:%String</FormalSpec>
-<Private>1</Private>
-<Implementation><![CDATA[	Set ..ProductIsGelocked = $$$False
-]]></Implementation>
-</Method>
 </Class>
 
 
Index: AXimpl/Dynamics/ProductItemUpsertIn/Converter/ProductItemConverter.cls.xml
===================================================================
diff -u -r49628 -r49691
--- AXimpl/Dynamics/ProductItemUpsertIn/Converter/ProductItemConverter.cls.xml	(.../ProductItemConverter.cls.xml)	(revision 49628)
+++ AXimpl/Dynamics/ProductItemUpsertIn/Converter/ProductItemConverter.cls.xml	(.../ProductItemConverter.cls.xml)	(revision 49691)
@@ -136,64 +136,56 @@
 <FormalSpec>ProductItem:AXif.Dynamics.ProductItemIn.MessageParts.AxdEntityItem,Product:DOM.PM.Product,LegalEntity:%String</FormalSpec>
 <Private>1</Private>
 <Implementation><![CDATA[
-	try {
-		If $$$HasLength(Product.GeefID()){
-			Do ..LockProduct(Product.GeefID()) ;nieuw product heeft nog geen ID
-			Do ..VerwijderIndexen(Product.GeefID())
-		}
+	If $$$HasLength(Product.GeefID()){  ;nieuw product heeft nog geen ID
+		Do ..VerwijderIndexen(Product.GeefID())
+	}
 
-		Do Product.ZetLeverancierId(..BepaalLeverancierId(ProductItem,LegalEntity)) ;PrimaryVendorId
-		Do Product.ZetAxGesynchroniseerd($$$True)		
-		;Do Product.ZetCifPPL(..BepaalCifPPL(ProductItem)) ;ActiveCostPrice
-		; wat van AX komt is reeds de cifppl. In cache is dit echter een berekend veld. We moeten dus alles goed zetten zodat het correct berekend wordt.
-		; DataMProduct.Save roept een RECALC^PRODUKT2 op die dit doet.	
-		; geen prijs gerelateerde velden aanpassen indien product kindbouwstenen (niet zeker voor hoeveel stuks de prijs is)	
-		#dim BouwsteenIterator =  Product.GeefKindBouwstenenIterator()
-		if (($$$Not(BouwsteenIterator.HasNext())) || (..IsProductItemSOPR(ProductItem))){
-			Do Product.ZetPPL(..BepaalCifPPL(ProductItem)) 
-			Do Product.ZetGrootteOrdeAankoop(##class(DOM.PM.enu.GrootteOrde).Honderd())
-			Do Product.ZetKorting(0)
-			Do Product.ZetCifPercentage(0) ;CostInsuranceFreight
-		}
-		Do Product.ZetFSCHout(..BepaalFSCHout(ProductItem)) ;CertClaimLevel	
-		Do Product.ZetTelbaar(..BepaalTelbaar(ProductItem)) ;CountableOrDivisible
-		Do Product.ZetBarcodeWaarde(..BepaalBarcodeWaarde(ProductItem)) ;GTIN
-		Do Product.ZetWerkvloerEenheid(..BepaalWerkvloerEenheid(ProductItem)) ;InventUnitId
-		Do Product.ZetLevertermijn(..BepaalLevertermijn(ProductItem,LegalEntity)) ;LeadTimePurch
-		Do Product.ZetMinimumBestelhoeveelheid(..BepaalMinimumBestelhoeveelheid(ProductItem)) ;LowestQtyPurch
-		Do Product.ZetGewichtPerDuizendStuksInKG(..BepaalGewichtPerDuizendStuksInKG(ProductItem)) ;NetWeight
-		Do Product.ZetAantalInGrootVerpakking(..BepaalAantalInGrootVerpakking(ProductItem)) ;QtyGVP 
-		Do Product.ZetAantalInKleinVerpakking(..BepaalAantalInKleinVerpakking(ProductItem)) ;QtyKVP 
-		Do Product.ZetAantalInNormaalVerpakking(..BepaalAantalInNormaalVerpakking(ProductItem)) ;QtyNVP
-		Do Product.ZetPalletAantal(..BepaalPalletAantal(ProductItem)) ;QtyPLT 
-		Do Product.ZetStockNietStock(..BepaalStockNietStock(ProductItem)) ; ReqCovType
-		Do Product.ZetMerk(" ") ; nergens gebruikt maar nodig voor ^KPL
-		Do Product.ZetActiefStatus(..BepaalCommercieelAktief(ProductItem)) ; StoppedSales
-		if ..MagLinktypeAangepastWordenBijUpdate(ProductItem,Product) {
-			Do Product.ZetLinkType(..BepaalLinkType(ProductItem,LegalEntity)) ; Ook ReqCovType  Alleen bij creatie zetten, daarna afblijven.
-		}	
-		Do Product.ZetKorttekst(..BepaalKorttekst(ProductItem))	
-		Do ..ConvertGroepen(Product, ProductItem)	;ok
-		; nodig want anders kan de ProductId niet opgevraagd worden, nodig om admin product aan te maken indien die niet zou bestaan.
-		; via event wordt ook admin prod aangemaakt
-		Do Product.Save() 
-			
-		Do ..ConvertTranslation(Product, ##class(TECH.ListIterator).%New(ProductItem.Translation)) ; ok
+	Do Product.ZetLeverancierId(..BepaalLeverancierId(ProductItem,LegalEntity)) ;PrimaryVendorId
+	Do Product.ZetAxGesynchroniseerd($$$True)		
+	;Do Product.ZetCifPPL(..BepaalCifPPL(ProductItem)) ;ActiveCostPrice
+	; wat van AX komt is reeds de cifppl. In cache is dit echter een berekend veld. We moeten dus alles goed zetten zodat het correct berekend wordt.
+	; DataMProduct.Save roept een RECALC^PRODUKT2 op die dit doet.	
+	; geen prijs gerelateerde velden aanpassen indien product kindbouwstenen (niet zeker voor hoeveel stuks de prijs is)	
+	#dim BouwsteenIterator =  Product.GeefKindBouwstenenIterator()
+	if (($$$Not(BouwsteenIterator.HasNext())) || (..IsProductItemSOPR(ProductItem))){
+		Do Product.ZetPPL(..BepaalCifPPL(ProductItem)) 
+		Do Product.ZetGrootteOrdeAankoop(##class(DOM.PM.enu.GrootteOrde).Honderd())
+		Do Product.ZetKorting(0)
+		Do Product.ZetCifPercentage(0) ;CostInsuranceFreight
+	}
+	Do Product.ZetFSCHout(..BepaalFSCHout(ProductItem)) ;CertClaimLevel	
+	Do Product.ZetTelbaar(..BepaalTelbaar(ProductItem)) ;CountableOrDivisible
+	Do Product.ZetBarcodeWaarde(..BepaalBarcodeWaarde(ProductItem)) ;GTIN
+	Do Product.ZetWerkvloerEenheid(..BepaalWerkvloerEenheid(ProductItem)) ;InventUnitId
+	Do Product.ZetLevertermijn(..BepaalLevertermijn(ProductItem,LegalEntity)) ;LeadTimePurch
+	Do Product.ZetMinimumBestelhoeveelheid(..BepaalMinimumBestelhoeveelheid(ProductItem)) ;LowestQtyPurch
+	Do Product.ZetGewichtPerDuizendStuksInKG(..BepaalGewichtPerDuizendStuksInKG(ProductItem)) ;NetWeight
+	Do Product.ZetAantalInGrootVerpakking(..BepaalAantalInGrootVerpakking(ProductItem)) ;QtyGVP 
+	Do Product.ZetAantalInKleinVerpakking(..BepaalAantalInKleinVerpakking(ProductItem)) ;QtyKVP 
+	Do Product.ZetAantalInNormaalVerpakking(..BepaalAantalInNormaalVerpakking(ProductItem)) ;QtyNVP
+	Do Product.ZetPalletAantal(..BepaalPalletAantal(ProductItem)) ;QtyPLT 
+	Do Product.ZetStockNietStock(..BepaalStockNietStock(ProductItem)) ; ReqCovType
+	Do Product.ZetMerk(" ") ; nergens gebruikt maar nodig voor ^KPL
+	Do Product.ZetActiefStatus(..BepaalCommercieelAktief(ProductItem)) ; StoppedSales
+	if ..MagLinktypeAangepastWordenBijUpdate(ProductItem,Product) {
+		Do Product.ZetLinkType(..BepaalLinkType(ProductItem,LegalEntity)) ; Ook ReqCovType  Alleen bij creatie zetten, daarna afblijven.
+	}	
+	Do Product.ZetKorttekst(..BepaalKorttekst(ProductItem))	
+	Do ..ConvertGroepen(Product, ProductItem)	;ok
+	; nodig want anders kan de ProductId niet opgevraagd worden, nodig om admin product aan te maken indien die niet zou bestaan.
+	; via event wordt ook admin prod aangemaakt
+	Do Product.Save() 
+		
+	Do ..ConvertTranslation(Product, ##class(TECH.ListIterator).%New(ProductItem.Translation)) ; ok
 
-		#dim Filter as TECH.Filter = ##class(AXimpl.Dynamics.ProductItemUpsertIn.Converter.ExtFilter).Maak("Vend","VendGroup")
-		Do ..ConvertExtModuleTypeVend(Product, ##class(TECH.DynamicIterator).%New(##class(TECH.ListIterator).%New(ProductItem.Ext),"",Filter))
+	#dim Filter as TECH.Filter = ##class(AXimpl.Dynamics.ProductItemUpsertIn.Converter.ExtFilter).Maak("Vend","VendGroup")
+	Do ..ConvertExtModuleTypeVend(Product, ##class(TECH.DynamicIterator).%New(##class(TECH.ListIterator).%New(ProductItem.Ext),"",Filter))
 
-		Set Filter = ##class(AXimpl.Dynamics.ProductItemUpsertIn.Converter.ExtFilter).Maak("Cust")
-		Do ..ConvertExtModuleTypeCust(Product, ##class(TECH.DynamicIterator).%New(##class(TECH.ListIterator).%New(ProductItem.Ext),"",Filter))
-		
-		Do Product.Save()
-		Do ..MaakIndexen(Product.GeefID())
-		Do ..UnLockProduct(Product.GeefID())
-	} catch {
-		#dim Exception as TECH.Exception = ##class(TECH.ExceptionHandler).Catch()
-		Do ..UnLockProduct(Product.GeefID())		
-		Do ##class(TECH.ExceptionHandler).Rethrow(Exception)		
-	}
+	Set Filter = ##class(AXimpl.Dynamics.ProductItemUpsertIn.Converter.ExtFilter).Maak("Cust")
+	Do ..ConvertExtModuleTypeCust(Product, ##class(TECH.DynamicIterator).%New(##class(TECH.ListIterator).%New(ProductItem.Ext),"",Filter))
+	
+	Do Product.Save()
+	Do ..MaakIndexen(Product.GeefID())
 ]]></Implementation>
 </Method>
 
@@ -732,30 +724,6 @@
 	Quit Result
 ]]></Implementation>
 </Method>
-
-<Method name="LockProduct">
-<FormalSpec>ProductID:%String</FormalSpec>
-<Private>1</Private>
-<Implementation><![CDATA[
-	Do ..Logger.Info("AX ProductSynchronizatie","Locking product '"_ProductID_ "'")
-	Lock +^KPR(ProductID):1
-	If $$$Not($Test) {
-		#dim LockInfo as %String = $$GebruikerInfoJob^vhLock($$LOCKJOB^vhLock($Name(^KPR(ProductID))))
-		Do ##class(Tools.Wlip).%New(207).String("locked")
-		Do ##class(TECH.ExceptionHandler).Throw(##class(AXimpl.VerwerkAXBerichtLaterOpnieuwException).%New("Product met ID '"_ProductID_"' is reeds locked." _ LockInfo))
-	}
-	Do ..Logger.Info("AX ProductSynchronizatie","Locked product '"_ProductID_ "'")
-]]></Implementation>
-</Method>
-
-<Method name="UnLockProduct">
-<FormalSpec>ProductID:%String</FormalSpec>
-<Private>1</Private>
-<Implementation><![CDATA[
-	Lock -^KPR(ProductID)
-	Do ..Logger.Info("AX ProductSynchronizatie","Unlocked product '"_ProductID_ "'")
-]]></Implementation>
-</Method>
 </Class>