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 :-) } } }