Index: WSimpl/Vhisie4/Winkelkar/WinkelkarService/VerwijderLijn.cls.xml
===================================================================
diff -u -r50446 -r54004
--- WSimpl/Vhisie4/Winkelkar/WinkelkarService/VerwijderLijn.cls.xml	(.../VerwijderLijn.cls.xml)	(revision 50446)
+++ WSimpl/Vhisie4/Winkelkar/WinkelkarService/VerwijderLijn.cls.xml	(.../VerwijderLijn.cls.xml)	(revision 54004)
@@ -18,8 +18,13 @@
 <Private>1</Private>
 </Property>
 
+<Property name="Logger">
+<Type>vhLib.Logger</Type>
+<Private>1</Private>
+</Property>
+
 <Method name="%OnNew">
-<FormalSpec>WinkelkarService:APPS.EC.Winkelkar.impl.WinkelkarService,WinkelkarRepository:WSimpl.Vhisie4.Winkelkar.WinkelkarRepository,LijnRepository:WSimpl.Vhisie4.Winkelkar.WinkelkarLijnRepository</FormalSpec>
+<FormalSpec>WinkelkarService:APPS.EC.Winkelkar.impl.WinkelkarService,WinkelkarRepository:WSimpl.Vhisie4.Winkelkar.WinkelkarRepository,LijnRepository:WSimpl.Vhisie4.Winkelkar.WinkelkarLijnRepository,Logger:vhLib.Logger</FormalSpec>
 <Private>1</Private>
 <ProcedureBlock>1</ProcedureBlock>
 <ReturnType>%Status</ReturnType>
@@ -28,6 +33,8 @@
 	Set ..WinkelkarService = $$$Inject(WinkelkarService, ##class(APPS.EC.Winkelkar.impl.WinkelkarService).%New())
 	Set ..WinkelkarRepository = $$$Inject(WinkelkarRepository,##class(WSimpl.Vhisie4.Winkelkar.WinkelkarRepository).%New())
 	Set ..LijnRepository = $$$Inject(LijnRepository, ##class(WSimpl.Vhisie4.Winkelkar.WinkelkarLijnRepository).%New())
+	Set ..Logger = $$$Inject(Logger, ##class(vhLib.Logger).%New())
+	
 	Quit $$$OK
 ]]></Implementation>
 </Method>
@@ -38,27 +45,37 @@
 	Lock +^Vhisie4.Winkelkar(VerwijderLijnRequest.WinkelkarId, VerwijderLijnRequest.LijnId):0 Else  Do ##class(TECH.ExceptionHandler).Throw(##class(TECH.Exceptions.LockedException).%New("Winkelkar '" _VerwijderLijnRequest.WinkelkarId _"' is gelocked door een andere webservice call. Probeer later opnieuw."))
 	
 	Try {
-		
-		Do ..ControleerExterneLijnId(VerwijderLijnRequest.WinkelkarId, VerwijderLijnRequest.LijnId)
-		
-		#dim WinkelkarID As %String = ..WinkelkarRepository.GeefWinkelkarIDViaID(VerwijderLijnRequest.WinkelkarId)
-		#dim LijnID As %String = ..LijnRepository.GeefLijnIDViaID(VerwijderLijnRequest.LijnId)
+		If $$$Not(..LijnRepository.BestaatViaID(VerwijderLijnRequest.LijnId)) {
+			; als de lijn niet bestaat, kunnen we daar niet veel aan doen. ITR wilt ze verwijderen, en ze bestaat bij ons al niet meer. -> geen error smijten, alleen loggen
+			#dim Melding As %String = ##class(TECH.StringUtils).BuildText("Winkelkarlijn met ExterneId '{0}' van Winkelkar met ExterneId '{1}', bestaat niet!",VerwijderLijnRequest.LijnId, VerwijderLijnRequest.WinkelkarId)
+			Set Melding = Melding _ $$$CRLF _ "Geen actie nodig, OV wordt verwittigd indien nodig bij de bestelwinkelkar indien ITR winkelkar niet overeenkomt met Cache"
+			Do ..Logger.Error("VerwijderWinkelkarLijn",Melding)
+		} else {
+			#dim WinkelkarID As %String = ..WinkelkarRepository.GeefWinkelkarIDViaID(VerwijderLijnRequest.WinkelkarId)
+			#dim LijnID As %String = ..LijnRepository.GeefLijnIDViaID(VerwijderLijnRequest.LijnId)
 
-		if $$$Not(##class(TECH.StringUtils).StartsWith(LijnID,WinkelkarID)) {
-			Do ##class(TECH.ExceptionHandler).Throw(##class(TECH.Exceptions.InvalidInputException).%New("Winkelkarlijn met id '" _ VerwijderLijnRequest.LijnId _ "' behoort niet tot winkelkar met id '" _ VerwijderLijnRequest.WinkelkarId_"'"))
-		}
+			if $$$Not(##class(TECH.StringUtils).StartsWith(LijnID,WinkelkarID)) {
+				Do ##class(TECH.ExceptionHandler).Throw(##class(TECH.Exceptions.InvalidInputException).%New("Winkelkarlijn met id '" _ VerwijderLijnRequest.LijnId _ "' behoort niet tot winkelkar met id '" _ VerwijderLijnRequest.WinkelkarId_"'"))
+			}
 
-		Do ..WinkelkarService.VerwijderLijn(WinkelkarID, LijnID)
+			Do ..WinkelkarService.VerwijderLijn(WinkelkarID, LijnID)
+		}
 	} Catch {
 		Lock -^Vhisie4.Winkelkar(VerwijderLijnRequest.WinkelkarId,VerwijderLijnRequest.LijnId)
-		Do ##class(TECH.ExceptionHandler).Rethrow()
+		
+		#dim Exception as TECH.IException = ##class(TECH.ExceptionHandler).Catch()		
+		if $$$Not(Exception.%ClassName(1) = ##class(TECH.Exceptions.DataNotFoundException).%ClassName(1)) {
+			Do ##class(TECH.ExceptionHandler).Rethrow(Exception)
+		}
+		
+
 	}
 	
 	Lock -^Vhisie4.Winkelkar(VerwijderLijnRequest.WinkelkarId,VerwijderLijnRequest.LijnId)
 ]]></Implementation>
 </Method>
 
-<Method name="ControleerExterneLijnId">
+<Method name="ControleerExterneLijnIdx">
 <FormalSpec>ExternId:%String,ExterneLijnId:%String</FormalSpec>
 <Private>1</Private>
 <Implementation><![CDATA[
Index: vhUnitTest/WSimpl/Vhisie4/Winkelkar/WinkelkarService/VerwijderLijn/Test.cls.xml
===================================================================
diff -u -r50443 -r54004
--- vhUnitTest/WSimpl/Vhisie4/Winkelkar/WinkelkarService/VerwijderLijn/Test.cls.xml	(.../Test.cls.xml)	(revision 50443)
+++ vhUnitTest/WSimpl/Vhisie4/Winkelkar/WinkelkarService/VerwijderLijn/Test.cls.xml	(.../Test.cls.xml)	(revision 54004)
@@ -5,6 +5,8 @@
 
 <Method name="Test: VerwijderLijn verwijdert lijn op de winkelkarservice met de juiste parameters">
 <Implementation><![CDATA[
+	Set LoggerMock = ##class(vhTest.Mock.vhLib.Logger).%New()
+
 	Set WinkelkarRepositoryMock = ##class(vhTest.Mock.WSimpl.Vhisie4.Winkelkar.WinkelkarRepository).%New()
 	Do WinkelkarRepositoryMock.VerwachtMethodCall("GeefWinkelkarIDViaID", "WinkelkarId").DanReturn("DummyID")
 	
@@ -15,7 +17,7 @@
 	set WinkelkarServiceMock = ##class(vhTest.Mock.APPS.EC.Winkelkar.WinkelkarService).%New()
 	Do WinkelkarServiceMock.VerwachtMethodCall("VerwijderLijn","DummyID", "DummyID||DummyLijnID").DanDoeNiks()
 	
-	Set Service = ##class(WSimpl.Vhisie4.Winkelkar.WinkelkarService.VerwijderLijn).%New(WinkelkarServiceMock, WinkelkarRepositoryMock, WinkelkarLijnRepositoryMock)
+	Set Service = ##class(WSimpl.Vhisie4.Winkelkar.WinkelkarService.VerwijderLijn).%New(WinkelkarServiceMock, WinkelkarRepositoryMock, WinkelkarLijnRepositoryMock,LoggerMock)
 	
 	Do Service.VerwijderLijn(..GeefVerwijderLijnRequest())
 
@@ -27,6 +29,8 @@
 
 <Method name="Test: Verwijderlijn gooit exceptie indien winkelkarlijn niet van opgegeven winkelkar is">
 <Implementation><![CDATA[
+	Set LoggerMock = ##class(vhTest.Mock.vhLib.Logger).%New()
+	
 	Set WinkelkarRepositoryMock = ##class(vhTest.Mock.WSimpl.Vhisie4.Winkelkar.WinkelkarRepository).%New()
 	Set WinkelkarLijnRepositoryMock = ##class(vhTest.Mock.WSimpl.Vhisie4.Winkelkar.WinkelkarLijnRepository).%New()
 
@@ -39,21 +43,22 @@
 	#dim Substrings As %ListOfDataTypes = ##class(TECH.ListUtils).CreateListOfDataTypes("Winkelkarlijn met id 'LijnId' behoort niet tot winkelkar met id 'WinkelkarId'")
 	Do ..ZetVerwachteExceptie(##class(Tools.Test.VerwachteExceptie).%New("TECH.Exceptions.InvalidInputException", Substrings))
 	
-	Set Service = ##class(WSimpl.Vhisie4.Winkelkar.WinkelkarService.VerwijderLijn).%New(,WinkelkarRepositoryMock,WinkelkarLijnRepositoryMock)
+	Set Service = ##class(WSimpl.Vhisie4.Winkelkar.WinkelkarService.VerwijderLijn).%New(,WinkelkarRepositoryMock,WinkelkarLijnRepositoryMock,LoggerMock)
 	Do Service.VerwijderLijn(VerwijderLijnRequest)
 ]]></Implementation>
 </Method>
 
 <Method name="Test: VerwijderLijn geeft foutmelding indien winkelkar niet bestaat">
 <Implementation><![CDATA[
+	Set LoggerMock = ##class(vhTest.Mock.vhLib.Logger).%New()
 	Set WinkelkarRepositoryStub = ##class(vhTest.Mock.WSimpl.Vhisie4.Winkelkar.WinkelkarRepository).%New()
 	Set InvalidInputException = ##class(TECH.Exceptions.InvalidInputException).%New("Winkelkar met id 'WinkelkarId' bestaat niet.")
 	Do WinkelkarRepositoryStub.IndienMethodCall("GeefWinkelkarIDViaID", "WinkelkarId").DanException(InvalidInputException)
 
 	Set WinkelkarLijnRepositoryStub = ##class(vhTest.Mock.WSimpl.Vhisie4.Winkelkar.WinkelkarLijnRepository).%New()
 	Do WinkelkarLijnRepositoryStub.IndienMethodCall("BestaatViaID", "LijnId").DanReturn($$$True)
 	
-	Set Service = ##class(WSimpl.Vhisie4.Winkelkar.WinkelkarService.VerwijderLijn).%New(,WinkelkarRepositoryStub, WinkelkarLijnRepositoryStub)
+	Set Service = ##class(WSimpl.Vhisie4.Winkelkar.WinkelkarService.VerwijderLijn).%New(,WinkelkarRepositoryStub, WinkelkarLijnRepositoryStub,LoggerMock)
 	
 	#dim Substrings As %ListOfDataTypes = ##class(TECH.ListUtils).CreateListOfDataTypes("Winkelkar met id 'WinkelkarId' bestaat niet.")
 	Do ..ZetVerwachteExceptie(##class(Tools.Test.VerwachteExceptie).%New("TECH.Exceptions.InvalidInputException", Substrings))
@@ -62,20 +67,28 @@
 ]]></Implementation>
 </Method>
 
-<Method name="Test: VerwijderLijn geeft foutmelding indien Lijn niet bestaat">
+<Method name="Test: VerwijderLijn geeft geen foutmelding indien Lijn niet bestaat en doet alsof de lijn verwijderd is maar logt wel">
 <Implementation><![CDATA[
+	Set LoggerMock = ##class(vhTest.Mock.vhLib.Logger).%New()
+	#dim Melding as %String = "Winkelkarlijn met ExterneId 'LijnId' van Winkelkar met ExterneId 'WinkelkarId', bestaat niet!"
+	Set Melding = Melding  _ $$$CRLF _"Geen actie nodig, OV wordt verwittigd indien nodig bij de bestelwinkelkar indien ITR winkelkar niet overeenkomt met Cache"
+	Do LoggerMock.VerwachtMethodCall("Error", "VerwijderWinkelkarLijn",Melding,0).DanDoeNiks()
+	
+	Set WinkelkarServiceMock = ##class(vhTest.Mock.APPS.EC.Winkelkar.WinkelkarService).%New()
+	
 	Set WinkelkarRepositoryStub = ##class(vhTest.Mock.WSimpl.Vhisie4.Winkelkar.WinkelkarRepository).%New()
 	Do WinkelkarRepositoryStub.IndienMethodCall("GeefWinkelkarIDViaID", "WinkelkarId").DanReturn("DummyWinkelkarID")
 	
 	Set WinkelkarLijnRepositoryStub = ##class(vhTest.Mock.WSimpl.Vhisie4.Winkelkar.WinkelkarLijnRepository).%New()
 	Do WinkelkarLijnRepositoryStub.IndienMethodCall("BestaatViaID", "LijnId").DanReturn($$$False)
 
-	Set Service = ##class(WSimpl.Vhisie4.Winkelkar.WinkelkarService.VerwijderLijn).%New(, WinkelkarRepositoryStub,WinkelkarLijnRepositoryStub)
+	Set Service = ##class(WSimpl.Vhisie4.Winkelkar.WinkelkarService.VerwijderLijn).%New(WinkelkarServiceMock, WinkelkarRepositoryStub,WinkelkarLijnRepositoryStub,LoggerMock)
 	
 	#dim Substrings As %ListOfDataTypes = ##class(TECH.ListUtils).CreateListOfDataTypes("Winkelkarlijn met ExterneId 'LijnId' van Winkelkar met ExterneId 'WinkelkarId', bestaat niet!")
-	Do ..ZetVerwachteExceptie(##class(Tools.Test.VerwachteExceptie).%New("TECH.Exceptions.DataNotFoundException", Substrings))
 	
 	Do Service.VerwijderLijn(..GeefVerwijderLijnRequest())
+	
+	Do LoggerMock.Verifieer()
 ]]></Implementation>
 </Method>