Apply review suggestions

jellyfin · Nov 8, 2024 · c84edaa · c84edaa
1 parent d31355a
commit c84edaa
Show file tree

Hide file tree

Showing 4 changed files with 7 additions and 6 deletions.
diff --git a/src/Jellyfin.Plugin.Dlna.Model/DlnaDeviceProfile.cs b/src/Jellyfin.Plugin.Dlna.Model/DlnaDeviceProfile.cs
@@ -1,3 +1,5 @@
+#pragma warning disable CA1819 // Properties should not return arrays
+
 using System;
 using System.Collections.Generic;
 using System.ComponentModel;
@@ -158,12 +160,12 @@ public class DlnaDeviceProfile : DeviceProfile
     /// <summary>
     /// Gets or sets the XmlRootAttributes.
     /// </summary>
-    public IReadOnlyList<XmlAttribute> XmlRootAttributes { get; set; } = [];
+    public XmlAttribute[] XmlRootAttributes { get; set; } = [];
 
     /// <summary>
     /// Gets or sets the ResponseProfiles.
     /// </summary>
-    public IReadOnlyList<ResponseProfile> ResponseProfiles { get; set; } = [];
+    public ResponseProfile[] ResponseProfiles { get; set; } = [];
 
     /// <summary>
     /// The supported media types.

diff --git a/src/Jellyfin.Plugin.Dlna.Model/IDeviceDiscovery.cs b/src/Jellyfin.Plugin.Dlna.Model/IDeviceDiscovery.cs
@@ -6,7 +6,6 @@ namespace Jellyfin.Plugin.Dlna.Model;
 /// <summary>
 /// Defines the <see cref="IDeviceDiscovery" /> interface.
 /// </summary>
-
 public interface IDeviceDiscovery
 {
     /// <summary>

diff --git a/src/Jellyfin.Plugin.Dlna/Didl/DidlBuilder.cs b/src/Jellyfin.Plugin.Dlna/Didl/DidlBuilder.cs
@@ -261,8 +261,8 @@ private void AddVideoResource(XmlWriter writer, BaseItem video, string deviceId,
         var contentFeatureList = ContentFeatureBuilder.BuildVideoHeader(
             _profile,
             streamInfo.Container,
-            streamInfo.TargetVideoCodec[0],
-            streamInfo.TargetAudioCodec[0],
+            streamInfo.TargetVideoCodec.FirstOrDefault(),
+            streamInfo.TargetAudioCodec.FirstOrDefault(),
             targetWidth,
             targetHeight,
             streamInfo.TargetVideoBitDepth,

diff --git a/src/Jellyfin.Plugin.Dlna/DlnaManager.cs b/src/Jellyfin.Plugin.Dlna/DlnaManager.cs
@@ -409,7 +409,7 @@ public void UpdateProfile(string profileId, DlnaDeviceProfile profile)
 
         if (profile.Id.IsNullOrEmpty())
         {
-            throw new ArgumentException("Profile id cannot be empty. ProfileId: {Id}", nameof(profile));
+            throw new ArgumentException($"Profile id cannot be empty. ProfileId: {profile.Id}", nameof(profile));
         }
 
         ArgumentException.ThrowIfNullOrEmpty(profile.Name);