Commit Graph

5 Commits

Author SHA1 Message Date
Linus Kardell
cd7ec988f2 Fix getId() on null in deleteConflictingEpisodeAction
Previously, it would take the guid and try to search for that in the
episode URL column, which may not find a match (or possibly even find
the wrong match).

testDoNotFailToUpdateEpisodeActionByGuidIfThereIsAnotherWithTheSameValueForEpisodeUrl
didn't catch this issue because it used the same value for episode and
guid when updating at line 84, so fix that as well. And for good
measure, give the save actions different position values, so the asserts
actually check that the saves have succeeded and they found the right
episode.
2024-01-12 20:43:01 +01:00
thrillfall
bf8fee14e3 delete conflicting episode action if current episode actions updates to same episode url 2023-02-24 20:48:53 +01:00
thrillfall
ee84d2f670 test updating episode action fails when other episode action with same episode url already exists 2023-02-24 20:42:45 +01:00
thrillfall
a6a19df999 test disambiguated episode action search 2023-02-24 19:37:50 +01:00
thrillfall
a1893061ce fix #109 search for episode action by guid only (retry with episode url if nothing found) thus avoid multiple results 2023-02-24 17:11:59 +01:00