Index: WSimpl/Vhisie4/Winkelkar/WinkelkarService/HerberekenMaatwerk.cls.xml
===================================================================
diff -u -r51013 -r57541
--- WSimpl/Vhisie4/Winkelkar/WinkelkarService/HerberekenMaatwerk.cls.xml	(.../HerberekenMaatwerk.cls.xml)	(revision 51013)
+++ WSimpl/Vhisie4/Winkelkar/WinkelkarService/HerberekenMaatwerk.cls.xml	(.../HerberekenMaatwerk.cls.xml)	(revision 57541)
@@ -28,6 +28,11 @@
 <Private>1</Private>
 </Property>
 
+<Property name="DtoWinkelkarUtils">
+<Type>WSimpl.Vhisie4.Winkelkar.WinkelkarUtils</Type>
+<Private>1</Private>
+</Property>
+
 <Property name="ExterneVerkoopService">
 <Type>APPS.EC.Verkoop.impl.Externe.VerkoopService</Type>
 <Private>1</Private>
@@ -38,13 +43,28 @@
 <Private>1</Private>
 </Property>
 
+<Property name="LijnRepository">
+<Type>WSimpl.Vhisie4.Winkelkar.WinkelkarLijnRepository</Type>
+<Private>1</Private>
+</Property>
+
+<Property name="WinkelkarService">
+<Type>APPS.EC.Winkelkar.impl.WinkelkarService</Type>
+<Private>1</Private>
+</Property>
+
+<Property name="Helper">
+<Type>WSimpl.Vhisie4.Winkelkar.WinkelkarService.Helper</Type>
+<Private>1</Private>
+</Property>
+
 <Property name="InterneOfferteId">
 <Type>%String</Type>
 <Private>1</Private>
 </Property>
 
 <Method name="%OnNew">
-<FormalSpec>WinkelkarRepository:WSimpl.Vhisie4.Winkelkar.WinkelkarRepository="",ExterneVerkoopService:APPS.EC.Verkoop.impl.Externe.VerkoopService="",Synchroniseer:WSimpl.Vhisie4.Winkelkar.WinkelkarService.Synchroniseer=""</FormalSpec>
+<FormalSpec>WinkelkarRepository:WSimpl.Vhisie4.Winkelkar.WinkelkarRepository="",ExterneVerkoopService:APPS.EC.Verkoop.impl.Externe.VerkoopService="",Synchroniseer:WSimpl.Vhisie4.Winkelkar.WinkelkarService.Synchroniseer="",LijnRepository:WSimpl.Vhisie4.Winkelkar.WinkelkarLijnRepository="",WinkelkarService:APPS.EC.Winkelkar.impl.WinkelkarService="",Helper:WSimpl.Vhisie4.Winkelkar.WinkelkarService.Helper=""</FormalSpec>
 <Private>1</Private>
 <ProcedureBlock>1</ProcedureBlock>
 <ReturnType>%Status</ReturnType>
@@ -56,8 +76,11 @@
 	
 	Set ..WinkelkarConverter = ##class(WSimpl.Vhisie4.Winkelkar.WinkelkarConverter).%New()
 	Set ..WinkelkarUtils = ##class(APPS.EC.Winkelkar.impl.WinkelkarUtils).%New()
-
+	Set ..DtoWinkelkarUtils = ##class(WSimpl.Vhisie4.Winkelkar.WinkelkarUtils).%New()
+	Set ..LijnRepository = $$$Inject(LijnRepository, ##class(WSimpl.Vhisie4.Winkelkar.WinkelkarLijnRepository).%New())
+	Set ..WinkelkarService = $$$Inject(WinkelkarService, ##class(APPS.EC.Winkelkar.impl.WinkelkarService).%New())
 	Set ..LijnCacheBuilder = ##class(WSimpl.Vhisie4.Winkelkar.WinkelkarService.HerberekenMaatwerk.LijnCacheBuilder).%New()
+	Set ..Helper = $$$Inject(Helper,##class(WSimpl.Vhisie4.Winkelkar.WinkelkarService.Helper).%New())
 
 	Quit $$$OK
 ]]></Implementation>
@@ -78,12 +101,14 @@
 		}
 		Set ..InterneOfferteId = GepersisteerdeOfferte.GeefID()
 		Set GepersisteerdeOfferte = ""
-
+		
+		Do ..Helper.VerwijderOngeldigGewordenGepersisteerdeMaatwerkLijnen(HerberekenMaatwerkRequest.Winkelkar)
+		Set HerberekenMaatwerkRequest.Winkelkar.Lijnen = ..DtoWinkelkarUtils.VerwijderOngeldigeMaatwerkLijnen(HerberekenMaatwerkRequest.Winkelkar)
+		
 		#dim NietgepersisteerdeMaatwerklijnen As APPS.EC.Winkelkar.dto.MaatwerkLijnList 
 		Set NietgepersisteerdeMaatwerklijnen = ..GeefGevalideerdeLijnenAlvorensTeHerberekenen(HerberekenMaatwerkRequest)
 		
 		#dim Offerte As APPS.EC.dto.Offerte = ..ExterneVerkoopService.GeefOfferteEnHerbereken(..InterneOfferteId)
-		
 		Set Response = ..MaakResponse(Offerte,..GeefHerberekendeLijnen(Offerte,NietgepersisteerdeMaatwerklijnen))
 	} Catch {
 		Lock -^Vhisie4.Winkelkar(HerberekenMaatwerkRequest.Winkelkar.Id)
Index: vhUnitTest/WSimpl/Vhisie4/Winkelkar/WinkelkarService/HerberekenMaatwerk/Test.cls.xml
===================================================================
diff -u -r50899 -r57541
--- vhUnitTest/WSimpl/Vhisie4/Winkelkar/WinkelkarService/HerberekenMaatwerk/Test.cls.xml	(.../Test.cls.xml)	(revision 50899)
+++ vhUnitTest/WSimpl/Vhisie4/Winkelkar/WinkelkarService/HerberekenMaatwerk/Test.cls.xml	(.../Test.cls.xml)	(revision 57541)
@@ -23,6 +23,21 @@
 <Private>1</Private>
 </Property>
 
+<Property name="LijnRepositoryMock">
+<Type>vhTest.Mock.WSimpl.Vhisie4.Winkelkar.WinkelkarLijnRepository</Type>
+<Private>1</Private>
+</Property>
+
+<Property name="WinkelkarServiceMock">
+<Type>vhTest.Mock.APPS.EC.Winkelkar.WinkelkarService</Type>
+<Private>1</Private>
+</Property>
+
+<Property name="HelperMock">
+<Type>vhTest.Mock.WSimpl.Vhisie4.Winkelkar.WinkelkarService.Helper</Type>
+<Private>1</Private>
+</Property>
+
 <Method name="OnBeforeOneTest">
 <FormalSpec>aTestName:%String</FormalSpec>
 <Implementation><![CDATA[
@@ -31,8 +46,11 @@
 	Set ..WinkelkarRepositoryMock = ##class(vhTest.Mock.WSimpl.Vhisie4.Winkelkar.WinkelkarRepository).%New()
 	Set ..VerkoopServiceMock = ##class(vhTest.Mock.APPS.EC.Verkoop.ExterneVerkoopService).%New()
 	Set ..SynchroniseerMock = ##class(vhTest.Mock.WSimpl.Vhisie4.Winkelkar.WinkelkarService.Synchroniseer).%New()
-	
-	Set ..HerberekenMaatwerk = ##class(WSimpl.Vhisie4.Winkelkar.WinkelkarService.HerberekenMaatwerk).%New(..WinkelkarRepositoryMock, ..VerkoopServiceMock, ..SynchroniseerMock)
+	Set ..LijnRepositoryMock = ##class(vhTest.Mock.WSimpl.Vhisie4.Winkelkar.WinkelkarLijnRepository).%New()
+	Set ..WinkelkarServiceMock = ##class(vhTest.Mock.APPS.EC.Winkelkar.WinkelkarService).%New()
+	Set ..HelperMock = ##class(vhTest.Mock.WSimpl.Vhisie4.Winkelkar.WinkelkarService.Helper).%New()
+		
+	Set ..HerberekenMaatwerk = ##class(WSimpl.Vhisie4.Winkelkar.WinkelkarService.HerberekenMaatwerk).%New(..WinkelkarRepositoryMock, ..VerkoopServiceMock, ..SynchroniseerMock,..LijnRepositoryMock,..WinkelkarServiceMock,..HelperMock)
 ]]></Implementation>
 </Method>
 
@@ -44,6 +62,8 @@
 	
 	#dim HerberekenMaatwerkRequest As WS.Vhisie4.Winkelkar.HerberekenMaatwerkRequest = ..GeefHerberekenMaatwerkRequest()
 
+	Do ..HelperMock.VerwachtMethodCall("VerwijderOngeldigGewordenGepersisteerdeMaatwerkLijnen",HerberekenMaatwerkRequest.Winkelkar).DanDoeNiks()
+
 	Do ..WinkelkarRepositoryMock.VerwachtMethodCall("GeefOfferteViaID", "DummyWinkelkarId").DanReturn(GepersisteerdeOfferte)
 
 	#dim Offerte As APPS.EC.dto.Offerte = ..GeefOfferte()
@@ -60,6 +80,7 @@
 	Do GepersisteerdeOfferte.Verifieer()
 	Do ..WinkelkarRepositoryMock.Verifieer()
 	Do ..VerkoopServiceMock.Verifieer()
+	Do ..HelperMock.Verifieer()
 ]]></Implementation>
 </Method>
 
@@ -72,6 +93,8 @@
 	#dim Offerte As APPS.EC.dto.Offerte = ..GeefOfferte(13.20,2)
 	#dim HerberekenMaatwerkRequest As WS.Vhisie4.Winkelkar.HerberekenMaatwerkRequest = ..GeefHerberekenMaatwerkRequest(2)
 
+	Do ..HelperMock.VerwachtMethodCall("VerwijderOngeldigGewordenGepersisteerdeMaatwerkLijnen",HerberekenMaatwerkRequest.Winkelkar).DanReturn(HerberekenMaatwerkRequest.Winkelkar.Lijnen)
+
 	Do ..WinkelkarRepositoryMock.VerwachtMethodCall("GeefOfferteViaID", "DummyWinkelkarId").DanReturn(GepersisteerdeOfferte)
 	
 	Do ..VerkoopServiceMock.VerwachtMethodCall("GeefOfferteEnHerbereken", "DummyOfferteId").DanReturn(Offerte)
@@ -94,9 +117,115 @@
 	Do ..WinkelkarRepositoryMock.Verifieer()
 	Do ..VerkoopServiceMock.Verifieer()
 	Do GepersisteerdeOfferte.Verifieer()
+	Do ..HelperMock.Verifieer()
 ]]></Implementation>
 </Method>
 
+<Method name="Test: HerberekenMaatwerk winkelkar met 2 ongeldige lijnen, verwijdert deze en geeft was nog geldig respons">
+<Implementation><![CDATA[
+	Set GepersisteerdeOfferte = ##class(vhTest.Mock.APPS.VKP.Offerte).%New()
+	Do GepersisteerdeOfferte.VerwachtMethodCall("GeefBestelStatus").DanReturn(##class(APPS.VKP.Offerte.enu.BestelStatus).Offerte())
+	Do GepersisteerdeOfferte.VerwachtMethodCall("GeefID").DanReturn("DummyOfferteId")
+	
+	#dim HerberekenMaatwerkRequest As WS.Vhisie4.Winkelkar.HerberekenMaatwerkRequest = ..GeefHerberekenMaatwerkRequest(2)
+	Set HerberekenMaatwerkRequest.Winkelkar.Lijnen.GetAt(1).Ongeldig = $$$true
+	Set HerberekenMaatwerkRequest.Winkelkar.Lijnen.GetAt(2).Ongeldig = $$$true
+	
+	Do ..HelperMock.VerwachtMethodCall("VerwijderOngeldigGewordenGepersisteerdeMaatwerkLijnen",HerberekenMaatwerkRequest.Winkelkar).DanDoeNiks()
+
+	Do ..WinkelkarRepositoryMock.VerwachtMethodCall("GeefOfferteViaID", "DummyWinkelkarId").DanReturn(GepersisteerdeOfferte)
+
+	#dim Offerte As APPS.EC.dto.Offerte = ..GeefOfferte(,0)
+	Do ..VerkoopServiceMock.VerwachtMethodCall("GeefOfferteEnHerbereken", "DummyOfferteId").DanReturn(Offerte)
+	Do ..VerkoopServiceMock.VerwachtMethodCall("GeefOfferteZonderHerberekening", "DummyOfferteId").DanReturn(Offerte)
+	
+	#dim HerberekenMaatwerkResponse As WS.Vhisie4.Winkelkar.HerberekenMaatwerkResponse 
+	Set HerberekenMaatwerkResponse = ..HerberekenMaatwerk.HerberekenMaatwerk(HerberekenMaatwerkRequest)
+	
+	Do $$$AssertEquals(HerberekenMaatwerkResponse.HerberekendeLijnen.Count(),0) 
+	Do $$$AssertEquals(HerberekenMaatwerkResponse.MeldingenAanGebruiker.Count(),0)
+	Do $$$AssertTrue(HerberekenMaatwerkResponse.WasNogGeldig)
+
+	Do GepersisteerdeOfferte.Verifieer()
+	Do ..WinkelkarRepositoryMock.Verifieer()
+	Do ..VerkoopServiceMock.Verifieer()
+	;Do ..LijnRepositoryMock.Verifieer()
+	;Do ..WinkelkarServiceMock.Verifieer()
+	Do ..HelperMock.Verifieer()
+]]></Implementation>
+</Method>
+
+<Method name="Test: HerberekenMaatwerk winkelkar met 2 lijnen waarvan 1 ongeldig, verwijdert ongeldige en geeft was nog geldig respons">
+<Implementation><![CDATA[
+	Set GepersisteerdeOfferte = ##class(vhTest.Mock.APPS.VKP.Offerte).%New()
+	Do GepersisteerdeOfferte.VerwachtMethodCall("GeefBestelStatus").DanReturn(##class(APPS.VKP.Offerte.enu.BestelStatus).Offerte())
+	Do GepersisteerdeOfferte.VerwachtMethodCall("GeefID").DanReturn("DummyOfferteId")
+	
+	#dim HerberekenMaatwerkRequest As WS.Vhisie4.Winkelkar.HerberekenMaatwerkRequest = ..GeefHerberekenMaatwerkRequest(2)
+	Set HerberekenMaatwerkRequest.Winkelkar.Lijnen.GetAt(1).Ongeldig = $$$false
+	Set HerberekenMaatwerkRequest.Winkelkar.Lijnen.GetAt(2).Ongeldig = $$$true
+	
+	Do ..HelperMock.VerwachtMethodCall("VerwijderOngeldigGewordenGepersisteerdeMaatwerkLijnen",HerberekenMaatwerkRequest.Winkelkar).DanDoeNiks()
+
+	Do ..WinkelkarRepositoryMock.VerwachtMethodCall("GeefOfferteViaID", "DummyWinkelkarId").DanReturn(GepersisteerdeOfferte)
+
+	#dim Offerte As APPS.EC.dto.Offerte = ..GeefOfferte(,1)
+	Do ..VerkoopServiceMock.VerwachtMethodCall("GeefOfferteEnHerbereken", "DummyOfferteId").DanReturn(Offerte)
+	Do ..VerkoopServiceMock.VerwachtMethodCall("GeefOfferteZonderHerberekening", "DummyOfferteId").DanReturn(Offerte)
+	
+	#dim HerberekenMaatwerkResponse As WS.Vhisie4.Winkelkar.HerberekenMaatwerkResponse 
+	Set HerberekenMaatwerkResponse = ..HerberekenMaatwerk.HerberekenMaatwerk(HerberekenMaatwerkRequest)
+	
+	Do $$$AssertEquals(HerberekenMaatwerkResponse.HerberekendeLijnen.Count(),0) 
+	Do $$$AssertEquals(HerberekenMaatwerkResponse.MeldingenAanGebruiker.Count(),0)
+	Do $$$AssertTrue(HerberekenMaatwerkResponse.WasNogGeldig)
+
+	Do GepersisteerdeOfferte.Verifieer()
+	Do ..WinkelkarRepositoryMock.Verifieer()
+	Do ..VerkoopServiceMock.Verifieer()
+	;Do ..LijnRepositoryMock.Verifieer()
+	;Do ..WinkelkarServiceMock.Verifieer()
+	Do ..HelperMock.Verifieer()
+]]></Implementation>
+</Method>
+
+<Method name="Test: HerberekenMaatwerk winkelkar met 2 lijnen waarvan 1 ongeldig, verwijdert ongeldige en geeft herrekende lijn respons">
+<Implementation><![CDATA[
+	Set GepersisteerdeOfferte = ##class(vhTest.Mock.APPS.VKP.Offerte).%New()
+	Do GepersisteerdeOfferte.VerwachtMethodCall("GeefBestelStatus").DanReturn(##class(APPS.VKP.Offerte.enu.BestelStatus).Offerte())
+	Do GepersisteerdeOfferte.VerwachtMethodCall("GeefID").DanReturn("DummyOfferteId")
+	
+	#dim HerberekenMaatwerkRequest As WS.Vhisie4.Winkelkar.HerberekenMaatwerkRequest = ..GeefHerberekenMaatwerkRequest(2)
+	Set HerberekenMaatwerkRequest.Winkelkar.Lijnen.GetAt(1).Ongeldig = $$$false
+	Set HerberekenMaatwerkRequest.Winkelkar.Lijnen.GetAt(2).Ongeldig = $$$true
+	Do ..HelperMock.VerwachtMethodCall("VerwijderOngeldigGewordenGepersisteerdeMaatwerkLijnen",HerberekenMaatwerkRequest.Winkelkar).DanDoeNiks()
+
+	Do ..WinkelkarRepositoryMock.VerwachtMethodCall("GeefOfferteViaID", "DummyWinkelkarId").DanReturn(GepersisteerdeOfferte)
+
+	#dim Offerte As APPS.EC.dto.Offerte = ..GeefOfferte(13.2,1)
+	Do ..VerkoopServiceMock.VerwachtMethodCall("GeefOfferteEnHerbereken", "DummyOfferteId").DanReturn(Offerte)
+	Do ..VerkoopServiceMock.VerwachtMethodCall("GeefOfferteZonderHerberekening", "DummyOfferteId").DanReturn(Offerte)
+	
+	#dim HerberekenMaatwerkResponse As WS.Vhisie4.Winkelkar.HerberekenMaatwerkResponse 
+	Set HerberekenMaatwerkResponse = ..HerberekenMaatwerk.HerberekenMaatwerk(HerberekenMaatwerkRequest)
+	
+	If $$$AssertEquals(HerberekenMaatwerkResponse.HerberekendeLijnen.Count(),1) {
+		Do ..AssertMaatwerkLijn(HerberekenMaatwerkResponse.HerberekendeLijnen.GetAt(1), 13.2, 26.40, $$$DummyDate)
+	}
+	
+	If $$$AssertEquals(HerberekenMaatwerkResponse.MeldingenAanGebruiker.Count(),1) {
+		Do $$$AssertEquals(HerberekenMaatwerkResponse.MeldingenAanGebruiker.GetAt(1),"DummyMelding")
+	}
+
+	Do $$$AssertFalse(HerberekenMaatwerkResponse.WasNogGeldig)
+
+	Do GepersisteerdeOfferte.Verifieer()
+	Do ..WinkelkarRepositoryMock.Verifieer()
+	Do ..VerkoopServiceMock.Verifieer()
+	Do ..HelperMock.Verifieer()
+]]></Implementation>
+</Method>
+
 <Method name="AssertMaatwerkLijn">
 <FormalSpec>HerberekendeLijn:WS.Vhisie4.Winkelkar.dto.HerberekendeMaatwerkLijn,EenheidPrijs:%Decimal,TotaalPrijs:%Decimal,Datum:%Date</FormalSpec>
 <Private>1</Private>
@@ -162,6 +291,7 @@
 	Set DummyWinkelkarLijn.Id = LijnId
 	Set DummyWinkelkarLijn.EenheidPrijs = 12.5
 	Set DummyWinkelkarLijn.TotaalPrijs = 25
+	Set DummyWinkelkarLijn.Ongeldig = $$$false
 	Set DummyWinkelkarLijn.EconConfiguratie = ##class(vhTest.Utils.ECON.PM.Maatwerk.dto.TbxKenmerken).StandaardLade().ToArray()
 	
 	Quit DummyWinkelkarLijn