add error handling to library changed notifier

MediaBrowser · Feb 23, 2016 · d8f41ea · d8f41ea
1 parent 659f201
commit d8f41ea
Show file tree

Hide file tree

Showing 3 changed files with 17 additions and 6 deletions.
diff --git a/MediaBrowser.Providers/People/MovieDbPersonProvider.cs b/MediaBrowser.Providers/People/MovieDbPersonProvider.cs
@@ -99,7 +99,7 @@ public async Task<IEnumerable<RemoteSearchResult>> GetSearchResults(PersonLookup
 
                     var requestCount = _requestCount;
 
-                    if (requestCount >= 10)
+                    if (requestCount >= 20)
                     {
                         //_logger.Debug("Throttling Tmdb people");
 

diff --git a/MediaBrowser.Server.Implementations/EntryPoints/LibraryChangedNotifier.cs b/MediaBrowser.Server.Implementations/EntryPoints/LibraryChangedNotifier.cs
@@ -213,8 +213,18 @@ private async void SendChangeNotifications(List<BaseItem> itemsAdded, List<BaseI
 
                 if (userSessions.Count > 0)
                 {
-                    var info = GetLibraryUpdateInfo(itemsAdded, itemsUpdated, itemsRemoved, foldersAddedTo,
-                                                    foldersRemovedFrom, id);
+                    LibraryUpdateInfo info;
+
+                    try
+                    {
+                         info = GetLibraryUpdateInfo(itemsAdded, itemsUpdated, itemsRemoved, foldersAddedTo,
+                                                        foldersRemovedFrom, id);
+                    }
+                    catch (Exception ex)
+                    {
+                        _logger.ErrorException("Error in GetLibraryUpdateInfo", ex);
+                        return;
+                    }
 
                     foreach (var userSession in userSessions)
                     {

diff --git a/MediaBrowser.WebDashboard/dashboard-ui/login.html b/MediaBrowser.WebDashboard/dashboard-ui/login.html
@@ -24,8 +24,9 @@ <h1 style="text-align: left;">${HeaderPleaseSignIn}</h1>
                 <button type="submit" data-role="none" class="clearButton">
                     <paper-button raised class="accent block"><iron-icon icon="check"></iron-icon><span>${ButtonSignIn}</span></paper-button>
                 </button>
-
-                <paper-button raised class="cancel block btnCancel" onclick="LoginPage.cancelLogin();"><iron-icon icon="close"></iron-icon><span>${ButtonCancel}</span></paper-button>
+                <div style="margin-top:.5em;">
+                    <paper-button raised class="cancel block btnCancel" onclick="LoginPage.cancelLogin();"><iron-icon icon="close"></iron-icon><span>${ButtonCancel}</span></paper-button>
+                </div>
                 <br />
                 <br />
             </form>
@@ -41,7 +42,7 @@ <h1 style="text-align: left;">${HeaderPleaseSignIn}</h1>
                 </div>
             </div>
 
-            <div class="readOnlyContent" style="margin: 0 auto 1em;">
+            <div class="readOnlyContent" style="margin: .5em auto 1em;">
                 <paper-button raised class="cancel block btnForgotPassword" onclick="LoginPage.cancelLogin();"><iron-icon icon="info"></iron-icon><span>${ButtonForgotPassword}</span></paper-button>
 
                 <div class="connectButtons" style="display:none;">