diff options
| author | Claus Vium <cvium@users.noreply.github.com> | 2022-09-09 13:27:52 +0200 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2022-09-09 13:27:52 +0200 |
| commit | 1cbf5431aa74274627082cb02349a3241cd8801b (patch) | |
| tree | 37879e5acc95a72082d162addb1789165bf4cd36 /MediaBrowser.Controller/Entities/Extensions.cs | |
| parent | c12cab6844c5253240b725e3de634fb505a23d1b (diff) | |
| parent | 5036afd69127cc6f6b0801329b2b04f21aee71f5 (diff) | |
Merge pull request #8250 from Bond-009/minor18
Minor cleanup
Diffstat (limited to 'MediaBrowser.Controller/Entities/Extensions.cs')
| -rw-r--r-- | MediaBrowser.Controller/Entities/Extensions.cs | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/MediaBrowser.Controller/Entities/Extensions.cs b/MediaBrowser.Controller/Entities/Extensions.cs index 9ce8eebe3..14534aa50 100644 --- a/MediaBrowser.Controller/Entities/Extensions.cs +++ b/MediaBrowser.Controller/Entities/Extensions.cs @@ -1,5 +1,3 @@ -#nullable disable - using System; using System.Linq; using Jellyfin.Extensions; @@ -19,9 +17,11 @@ namespace MediaBrowser.Controller.Entities /// <param name="url">Trailer URL.</param> public static void AddTrailerUrl(this BaseItem item, string url) { - if (string.IsNullOrEmpty(url)) + ArgumentNullException.ThrowIfNull(url, nameof(url)); + + if (url.Length == 0) { - throw new ArgumentNullException(nameof(url)); + throw new ArgumentException("String can't be empty", nameof(url)); } var current = item.RemoteTrailers.FirstOrDefault(i => string.Equals(i.Url, url, StringComparison.OrdinalIgnoreCase)); |
