diff options
| author | Niels van Velzen <nielsvanvelzen@users.noreply.github.com> | 2024-09-06 21:56:42 +0200 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2024-09-06 21:56:42 +0200 |
| commit | cc2f91b331cd51ecf238db4370b98ecdaf1ad897 (patch) | |
| tree | 9acf3d94518501a279f05d2837f5cd40c5688bd3 /MediaBrowser.XbmcMetadata | |
| parent | 9afaa6ae4ddb9c7aa109604881496344d7124b92 (diff) | |
| parent | 277830855ddd9c7fa5a5db21f02d2d2f199f6cf8 (diff) | |
Merge pull request #12547 from Bond-009/userdatamanager
Clean up UserDataManager
Diffstat (limited to 'MediaBrowser.XbmcMetadata')
| -rw-r--r-- | MediaBrowser.XbmcMetadata/Parsers/BaseNfoParser.cs | 31 |
1 files changed, 20 insertions, 11 deletions
diff --git a/MediaBrowser.XbmcMetadata/Parsers/BaseNfoParser.cs b/MediaBrowser.XbmcMetadata/Parsers/BaseNfoParser.cs index f2681500b0..a8800431e1 100644 --- a/MediaBrowser.XbmcMetadata/Parsers/BaseNfoParser.cs +++ b/MediaBrowser.XbmcMetadata/Parsers/BaseNfoParser.cs @@ -306,12 +306,15 @@ namespace MediaBrowser.XbmcMetadata.Parsers break; case "watched": var played = reader.ReadElementContentAsBoolean(); - if (!string.IsNullOrWhiteSpace(nfoConfiguration.UserId)) + if (Guid.TryParse(nfoConfiguration.UserId, out var userId)) { - var user = _userManager.GetUserById(Guid.Parse(nfoConfiguration.UserId)); - userData = _userDataManager.GetUserData(user, item); - userData.Played = played; - _userDataManager.SaveUserData(user, item, userData, UserDataSaveReason.Import, CancellationToken.None); + var user = _userManager.GetUserById(userId); + if (user is not null) + { + userData = _userDataManager.GetUserData(user, item); + userData.Played = played; + _userDataManager.SaveUserData(user, item, userData, UserDataSaveReason.Import, CancellationToken.None); + } } break; @@ -320,9 +323,12 @@ namespace MediaBrowser.XbmcMetadata.Parsers && Guid.TryParse(nfoConfiguration.UserId, out var playCountUserId)) { var user = _userManager.GetUserById(playCountUserId); - userData = _userDataManager.GetUserData(user, item); - userData.PlayCount = count; - _userDataManager.SaveUserData(user, item, userData, UserDataSaveReason.Import, CancellationToken.None); + if (user is not null) + { + userData = _userDataManager.GetUserData(user, item); + userData.PlayCount = count; + _userDataManager.SaveUserData(user, item, userData, UserDataSaveReason.Import, CancellationToken.None); + } } break; @@ -331,9 +337,12 @@ namespace MediaBrowser.XbmcMetadata.Parsers && Guid.TryParse(nfoConfiguration.UserId, out var lastPlayedUserId)) { var user = _userManager.GetUserById(lastPlayedUserId); - userData = _userDataManager.GetUserData(user, item); - userData.LastPlayedDate = lastPlayed; - _userDataManager.SaveUserData(user, item, userData, UserDataSaveReason.Import, CancellationToken.None); + if (user is not null) + { + userData = _userDataManager.GetUserData(user, item); + userData.LastPlayedDate = lastPlayed; + _userDataManager.SaveUserData(user, item, userData, UserDataSaveReason.Import, CancellationToken.None); + } } break; |