From 540b904ebaecdc935cb622ebf2e2cf87162fa3fe Mon Sep 17 00:00:00 2001 From: Luke Pulverenti Date: Sat, 6 Jul 2013 19:08:32 -0400 Subject: [PATCH] have some endpoints throw exceptions for offline items --- MediaBrowser.Api/ItemUpdateService.cs | 5 +++++ MediaBrowser.Api/LibraryService.cs | 5 +++++ 2 files changed, 10 insertions(+) diff --git a/MediaBrowser.Api/ItemUpdateService.cs b/MediaBrowser.Api/ItemUpdateService.cs index a86db896b..033f4014b 100644 --- a/MediaBrowser.Api/ItemUpdateService.cs +++ b/MediaBrowser.Api/ItemUpdateService.cs @@ -194,6 +194,11 @@ namespace MediaBrowser.Api private void UpdateItem(BaseItemDto request, BaseItem item) { + if (item.LocationType == LocationType.Offline) + { + throw new InvalidOperationException(string.Format("{0} is currently offline.", item.Name)); + } + item.Name = request.Name; // Only set the forced value if they changed it, or there's already one diff --git a/MediaBrowser.Api/LibraryService.cs b/MediaBrowser.Api/LibraryService.cs index 4042c7c24..17a448c3d 100644 --- a/MediaBrowser.Api/LibraryService.cs +++ b/MediaBrowser.Api/LibraryService.cs @@ -226,6 +226,11 @@ namespace MediaBrowser.Api var parent = item.Parent; + if (item.LocationType == LocationType.Offline) + { + throw new InvalidOperationException(string.Format("{0} is currently offline.", item.Name)); + } + if (item.LocationType == LocationType.FileSystem) { if (Directory.Exists(item.Path))