Index: vhUnitTest/WSimpl/Vhisie4/Winkelkar/WinkelkarService/Helper/VerwijderOngeldigGewordenGepersisteerdeMaatwerkLijnen/Test.cls.xml
===================================================================
diff -u -r57536 -r57673
--- vhUnitTest/WSimpl/Vhisie4/Winkelkar/WinkelkarService/Helper/VerwijderOngeldigGewordenGepersisteerdeMaatwerkLijnen/Test.cls.xml (.../Test.cls.xml) (revision 57536)
+++ vhUnitTest/WSimpl/Vhisie4/Winkelkar/WinkelkarService/Helper/VerwijderOngeldigGewordenGepersisteerdeMaatwerkLijnen/Test.cls.xml (.../Test.cls.xml) (revision 57673)
@@ -89,33 +89,29 @@
]]>
-
-
+
-
+ Do ..LijnRepositoryMock.VerwachtMethodCall("BestaatViaID",1).DanReturn($$$true)
+ Do ..LijnRepositoryMock.VerwachtMethodCall("GeefLijnIDViaID","1").DanReturn("InterneWinkelkarID|LijnID1")
+
+ Do ..WinkelkarRepositoryMock.VerwachtMethodCall("GeefWinkelkarIDViaID","DummyWinkelkarId").DanReturn("InterneWinkelkarID")
+ Do ..WinkelkarServiceMock.VerwachtMethodCall("VerwijderLijn","InterneWinkelkarID","InterneWinkelkarID|LijnID1").DanDoeNiks()
+
+ Do ..Helper.VerwijderOngeldigGewordenGepersisteerdeMaatwerkLijnen(Winkelkar)
+
+ Do ..WinkelkarRepositoryMock.Verifieer()
+ Do ..WinkelkarServiceMock.Verifieer()
+ Do ..LijnRepositoryMock.Verifieer()
+]]>
+
+
-AantalLijnen:%Integer
+AantalMaatwerkLijnen:%Integer=0,AantalProductLijnen:%Integer=0
1
WS.Vhisie4.Winkelkar.dto.Winkelkar
-
+
LijnId:%Integer
1
WS.Vhisie4.Winkelkar.dto.MaatwerkLijn
@@ -149,6 +150,23 @@
Quit DummyWinkelkarLijn
]]>
+
+
+LijnId:%Integer
+1
+WS.Vhisie4.Winkelkar.dto.ProductLijn
+
+
Index: WSimpl/Vhisie4/Winkelkar/WinkelkarService/Helper.cls.xml
===================================================================
diff -u -r57536 -r57673
--- WSimpl/Vhisie4/Winkelkar/WinkelkarService/Helper.cls.xml (.../Helper.cls.xml) (revision 57536)
+++ WSimpl/Vhisie4/Winkelkar/WinkelkarService/Helper.cls.xml (.../Helper.cls.xml) (revision 57673)
@@ -73,15 +73,17 @@
Set RequestLijnIterator = ##class(TECH.ListIterator).%New(Winkelkar.Lijnen)
While (RequestLijnIterator.HasNext()){
#dim RequestLijn as WS.Vhisie4.Winkelkar.dto.WinkelkarLijn = RequestLijnIterator.Next()
- if (RequestLijn.Ongeldig = $$$true){
- If (..LijnRepository.BestaatViaID(RequestLijn.Id)) {
- #dim LijnID As %String = ..LijnRepository.GeefLijnIDViaID(RequestLijn.Id)
- if $$$Not(##class(TECH.StringUtils).StartsWith(LijnID,WinkelkarID)) {
- Do ##class(TECH.ExceptionHandler).Throw(##class(TECH.Exceptions.InvalidInputException).%New("Winkelkarlijn met id '" _ RequestLijn.Id _ "' behoort niet tot winkelkar met id '" _ HerberekenMaatwerkRequest.Winkelkar.Id_"'"))
+ if RequestLijn.%IsA("WS.Vhisie4.Winkelkar.dto.MaatwerkLijn"){
+ if (RequestLijn.Ongeldig = $$$true){
+ If (..LijnRepository.BestaatViaID(RequestLijn.Id)) {
+ #dim LijnID As %String = ..LijnRepository.GeefLijnIDViaID(RequestLijn.Id)
+ if $$$Not(##class(TECH.StringUtils).StartsWith(LijnID,WinkelkarID)) {
+ Do ##class(TECH.ExceptionHandler).Throw(##class(TECH.Exceptions.InvalidInputException).%New("Winkelkarlijn met id '" _ RequestLijn.Id _ "' behoort niet tot winkelkar met id '" _ HerberekenMaatwerkRequest.Winkelkar.Id_"'"))
+ }
+ Do ..WinkelkarService.VerwijderLijn(WinkelkarID, LijnID)
+ } else {
+ ;de lijn bestaat niet meer, dus moeten we ze ook niet verwijderen :-)
}
- Do ..WinkelkarService.VerwijderLijn(WinkelkarID, LijnID)
- } else {
- ;de lijn bestaat niet meer, dus moeten we ze ook niet verwijderen :-)
}
}
}