diff options
| author | Claus Vium <cvium@users.noreply.github.com> | 2020-11-08 10:22:05 +0100 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2020-11-08 10:22:05 +0100 |
| commit | 1823cbb0261ce8388fd39078c77de89990ed2437 (patch) | |
| tree | 494480998122f451d50d6fb55c70b2297e4cff21 | |
| parent | 9764b6b2df2da3d37f8ba79cb502b4554c73c45a (diff) | |
| parent | 19d77c99ae77339e919090bfea1244e1eba26b0c (diff) | |
Merge pull request #4412 from cvium/fix_displaypreferences_creation
Save new display preferences
| -rw-r--r-- | Jellyfin.Api/Controllers/DisplayPreferencesController.cs | 3 | ||||
| -rw-r--r-- | MediaBrowser.Controller/IDisplayPreferencesManager.cs | 6 |
2 files changed, 9 insertions, 0 deletions
diff --git a/Jellyfin.Api/Controllers/DisplayPreferencesController.cs b/Jellyfin.Api/Controllers/DisplayPreferencesController.cs index 874467c75..76f5717e3 100644 --- a/Jellyfin.Api/Controllers/DisplayPreferencesController.cs +++ b/Jellyfin.Api/Controllers/DisplayPreferencesController.cs @@ -81,6 +81,9 @@ namespace Jellyfin.Api.Controllers dto.CustomPrefs["enableNextVideoInfoOverlay"] = displayPreferences.EnableNextVideoInfoOverlay.ToString(CultureInfo.InvariantCulture); dto.CustomPrefs["tvhome"] = displayPreferences.TvHome; + // This will essentially be a noop if no changes have been made, but new prefs must be saved at least. + _displayPreferencesManager.SaveChanges(); + return dto; } diff --git a/MediaBrowser.Controller/IDisplayPreferencesManager.cs b/MediaBrowser.Controller/IDisplayPreferencesManager.cs index b35f83096..6658269bd 100644 --- a/MediaBrowser.Controller/IDisplayPreferencesManager.cs +++ b/MediaBrowser.Controller/IDisplayPreferencesManager.cs @@ -12,6 +12,9 @@ namespace MediaBrowser.Controller /// <summary> /// Gets the display preferences for the user and client. /// </summary> + /// <remarks> + /// This will create the display preferences if it does not exist, but it will not save automatically. + /// </remarks> /// <param name="userId">The user's id.</param> /// <param name="client">The client string.</param> /// <returns>The associated display preferences.</returns> @@ -20,6 +23,9 @@ namespace MediaBrowser.Controller /// <summary> /// Gets the default item display preferences for the user and client. /// </summary> + /// <remarks> + /// This will create the item display preferences if it does not exist, but it will not save automatically. + /// </remarks> /// <param name="userId">The user id.</param> /// <param name="itemId">The item id.</param> /// <param name="client">The client string.</param> |