Backup_Repos/Radarr
1
0
Fork 0
mirror of https://github.com/Radarr/Radarr synced 2026-04-24 16:40:45 +02:00
Code Issues Projects Releases Wiki Activity

fix: add empty checks before First() in MovieFileController (#96)

Browse source 
Add guard clauses to prevent InvalidOperationException when
movieFiles list is empty in bulk update/delete operations

Co-authored-by: admin <admin@ardentleatherworks.com>
This commit is contained in:
Cody Kickertz 2025-12-21 11:44:49 -06:00 committed by GitHub
parent 0081ec2aa1
commit 9bf299c196
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
1 changed files with 16 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

16
src/Radarr.Api.V3/MovieFiles/MovieFileController.cs
View file

@ -112,6 +112,11 @@ public object SetMovieFile([FromBody] MovieFileListResource resource)
{
var movieFiles = _mediaFileService.GetMovies(resource.MovieFileIds);
if (movieFiles.Count == 0)
{
return new { };
}
foreach (var movieFile in movieFiles)
{
if (resource.Quality != null)
@ -178,6 +183,12 @@ public object DeleteMovieFiles([FromBody] MovieFileListResource resource)
}
var movieFiles = _mediaFileService.GetMovies(resource.MovieFileIds);
if (movieFiles.Count == 0)
{
return new { };
}
var movie = _movieService.GetMovie(movieFiles.First().MovieId);
foreach (var movieFile in movieFiles)
@ -194,6 +205,11 @@ public object SetPropertiesBulk([FromBody] List<MovieFileResource> resources)
{
var movieFiles = _mediaFileService.GetMovies(resources.Select(r => r.Id));
if (movieFiles.Count == 0)
{
return new { };
}
foreach (var movieFile in movieFiles)
{
var resourceMovieFile = resources.Single(r => r.Id == movieFile.Id);