diff options
| author | Bond_009 <bond.009@outlook.com> | 2024-08-30 15:08:56 +0200 |
|---|---|---|
| committer | Bond_009 <bond.009@outlook.com> | 2024-08-30 15:08:56 +0200 |
| commit | 277830855ddd9c7fa5a5db21f02d2d2f199f6cf8 (patch) | |
| tree | 6fb7660bd24e4a05892a0c34b2d906ad7c0296fd /MediaBrowser.XbmcMetadata | |
| parent | 9644e894f0f6e1206f68a2b23aeb8065a565f9bf (diff) | |
Clean up UserDataManager
* enable nullable
* remove unused methods
* fix warnings and docs
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 f2681500b..a8800431e 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; |