[WIP] Fix issues introduced by recent merges (#114)

* Initial plan

* Fix inconsistent HTTP response codes: PUT endpoints return 200 OK instead of 202 Accepted

Co-authored-by: cheir-mneme <176430037+cheir-mneme@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: cheir-mneme <176430037+cheir-mneme@users.noreply.github.com>

src/Radarr.Api.V3/Collections/CollectionController.cs

@@ -98,7 +98,7 @@ public ActionResult<CollectionResource> UpdateCollection([FromBody] CollectionRe
 
             var updatedMovie = _collectionService.UpdateCollection(model);
 
-            return Accepted(updatedMovie.Id);
+            return Ok(GetResourceById(updatedMovie.Id));
         }
 
         [HttpPut]
@@ -148,7 +148,7 @@ public ActionResult UpdateCollections([FromBody] CollectionUpdateResource resour
 
             _commandQueueManager.Push(new RefreshCollectionsCommand());
 
-            return Accepted(updated);
+            return Ok(updated);
         }
 
         private IEnumerable<CollectionResource> MapToResource(List<MovieCollection> collections)

src/Radarr.Api.V3/Config/ConfigController.cs

@@ -42,7 +42,7 @@ public virtual ActionResult<TResource> SaveConfig([FromBody] TResource resource)
 
             _configService.SaveConfigDictionary(dictionary);
 
-            return Accepted(resource.Id);
+            return Ok(GetResourceById(resource.Id));
         }
 
         protected abstract TResource ToResource(IConfigService model);

src/Radarr.Api.V3/Config/HostConfigController.cs

@@ -125,7 +125,7 @@ public ActionResult<HostConfigResource> SaveHostConfig([FromBody] HostConfigReso
                 _userService.Upsert(resource.Username, resource.Password);
             }
 
-            return Accepted(resource.Id);
+            return Ok(GetResourceById(resource.Id));
         }
     }
 }

src/Radarr.Api.V3/Config/NamingConfigController.cs

@@ -52,7 +52,7 @@ public ActionResult<NamingConfigResource> UpdateNamingConfig([FromBody] NamingCo
 
             _namingConfigService.Save(nameSpec);
 
-            return Accepted(resource.Id);
+            return Ok(GetResourceById(resource.Id));
         }
 
         [HttpGet("examples")]

src/Radarr.Api.V3/Config/UiConfigController.cs

@@ -46,7 +46,7 @@ public override ActionResult<UiConfigResource> SaveConfig([FromBody] UiConfigRes
             _configFileProvider.SaveConfigDictionary(dictionary);
             _configService.SaveConfigDictionary(dictionary);
 
-            return Accepted(resource.Id);
+            return Ok(GetResourceById(resource.Id));
         }
 
         protected override UiConfigResource ToResource(IConfigService model)

src/Radarr.Api.V3/ImportLists/ImportListExclusionController.cs

@@ -76,7 +76,7 @@ public ActionResult<ImportListExclusionResource> AddImportListExclusion([FromBod
         public ActionResult<ImportListExclusionResource> UpdateImportListExclusion([FromBody] ImportListExclusionResource resource)
         {
             _importListExclusionService.Update(resource.ToModel());
-            return Accepted(resource.Id);
+            return Ok(GetResourceById(resource.Id));
         }
 
         [HttpPost("bulk")]

src/Radarr.Api.V3/MovieFiles/MovieFileController.cs

@@ -253,7 +253,7 @@ public object SetPropertiesBulk([FromBody] List<MovieFileResource> resources)
 
             var movie = _movieService.GetMovie(movieFiles.First().MovieId);
 
-            return Accepted(movieFiles.ConvertAll(f => f.ToResource(movie, _upgradableSpecification, _formatCalculator)));
+            return Ok(movieFiles.ConvertAll(f => f.ToResource(movie, _upgradableSpecification, _formatCalculator)));
         }
 
         [NonAction]

src/Radarr.Api.V3/Profiles/Delay/DelayProfileController.cs

@@ -60,7 +60,7 @@ public ActionResult<DelayProfileResource> Update([FromBody] DelayProfileResource
         {
             var model = resource.ToModel();
             _delayProfileService.Update(model);
-            return Accepted(model.Id);
+            return Ok(GetResourceById(model.Id));
         }
 
         protected override DelayProfileResource GetResourceById(int id)

src/Radarr.Api.V3/Profiles/Quality/QualityProfileController.cs

@@ -69,7 +69,7 @@ public ActionResult<QualityProfileResource> Update([FromBody] QualityProfileReso
 
             _qualityProfileService.Update(model);
 
-            return Accepted(model.Id);
+            return Ok(GetResourceById(model.Id));
         }
 
         protected override QualityProfileResource GetResourceById(int id)

src/Radarr.Api.V3/Profiles/Release/ReleaseProfileController.cs

@@ -67,7 +67,7 @@ public ActionResult<ReleaseProfileResource> Update([FromBody] ReleaseProfileReso
 
             _profileService.Update(model);
 
-            return Accepted(model.Id);
+            return Ok(GetResourceById(model.Id));
         }
 
         protected override ReleaseProfileResource GetResourceById(int id)

src/Radarr.Api.V3/Qualities/QualityDefinitionController.cs

@@ -34,7 +34,7 @@ public ActionResult<QualityDefinitionResource> Update([FromBody] QualityDefiniti
         {
             var model = resource.ToModel();
             _qualityDefinitionService.Update(model);
-            return Accepted(model.Id);
+            return Ok(GetResourceById(model.Id));
         }
 
         protected override QualityDefinitionResource GetResourceById(int id)
@@ -56,7 +56,7 @@ public object UpdateMany([FromBody] List<QualityDefinitionResource> resource)
 
             _qualityDefinitionService.UpdateMany(qualityDefinitions);
 
-            return Accepted(_qualityDefinitionService.All()
+            return Ok(_qualityDefinitionService.All()
                 .ToResource());
         }