aboutsummaryrefslogtreecommitdiff
path: root/Jellyfin.Data
diff options
context:
space:
mode:
authorJPVenson <github@jpb.email>2024-12-15 14:46:40 +0000
committerJPVenson <github@jpb.email>2024-12-15 14:46:40 +0000
commita0c568bc6c9944fab8b6d1d6d420a9716bdab1e9 (patch)
tree00565ba6c1e73ddfd3278cd3356c5110699553ad /Jellyfin.Data
parent1c3196dd5f07d152b5c0ab4614ab0ca01557375d (diff)
Applied review comments
Diffstat (limited to 'Jellyfin.Data')
-rw-r--r--Jellyfin.Data/Entities/BaseItemEntity.cs6
-rw-r--r--Jellyfin.Data/Entities/BaseItemExtraType.cs2
-rw-r--r--Jellyfin.Data/Entities/BaseItemImageInfo.cs6
-rw-r--r--Jellyfin.Data/Entities/ItemValueType.cs2
-rw-r--r--Jellyfin.Data/Entities/MediaStreamInfo.cs3
-rw-r--r--Jellyfin.Data/Entities/People.cs3
6 files changed, 12 insertions, 10 deletions
diff --git a/Jellyfin.Data/Entities/BaseItemEntity.cs b/Jellyfin.Data/Entities/BaseItemEntity.cs
index 0c9020a66..33b2b6741 100644
--- a/Jellyfin.Data/Entities/BaseItemEntity.cs
+++ b/Jellyfin.Data/Entities/BaseItemEntity.cs
@@ -1,3 +1,6 @@
+#pragma warning disable CS1591 // Missing XML comment for publicly visible type or member
+#pragma warning disable CA2227 // Collection properties should be read only
+
using System;
using System.Collections.Generic;
using System.ComponentModel.DataAnnotations;
@@ -5,9 +8,6 @@ using System.ComponentModel.DataAnnotations.Schema;
namespace Jellyfin.Data.Entities;
-#pragma warning disable CS1591 // Missing XML comment for publicly visible type or member
-#pragma warning disable CA2227 // Collection properties should be read only
-
public class BaseItemEntity
{
public required Guid Id { get; set; }
diff --git a/Jellyfin.Data/Entities/BaseItemExtraType.cs b/Jellyfin.Data/Entities/BaseItemExtraType.cs
index 341697436..54aef50e4 100644
--- a/Jellyfin.Data/Entities/BaseItemExtraType.cs
+++ b/Jellyfin.Data/Entities/BaseItemExtraType.cs
@@ -1,6 +1,6 @@
+#pragma warning disable CS1591
namespace Jellyfin.Data.Entities;
-#pragma warning disable CS1591
public enum BaseItemExtraType
{
Unknown = 0,
diff --git a/Jellyfin.Data/Entities/BaseItemImageInfo.cs b/Jellyfin.Data/Entities/BaseItemImageInfo.cs
index 6390cac58..37723df11 100644
--- a/Jellyfin.Data/Entities/BaseItemImageInfo.cs
+++ b/Jellyfin.Data/Entities/BaseItemImageInfo.cs
@@ -1,8 +1,9 @@
+#pragma warning disable CA2227
+
using System;
using System.Collections.Generic;
namespace Jellyfin.Data.Entities;
-#pragma warning disable CA2227
/// <summary>
/// Enum TrailerTypes.
@@ -39,11 +40,12 @@ public class BaseItemImageInfo
/// </summary>
public int Height { get; set; }
-#pragma warning disable CA1819
+#pragma warning disable CA1819 // Properties should not return arrays
/// <summary>
/// Gets or Sets the blurhash.
/// </summary>
public byte[]? Blurhash { get; set; }
+#pragma warning restore CA1819
/// <summary>
/// Gets or Sets the reference id to the BaseItem.
diff --git a/Jellyfin.Data/Entities/ItemValueType.cs b/Jellyfin.Data/Entities/ItemValueType.cs
index 006036b40..48c5d0f30 100644
--- a/Jellyfin.Data/Entities/ItemValueType.cs
+++ b/Jellyfin.Data/Entities/ItemValueType.cs
@@ -10,9 +10,7 @@ public enum ItemValueType
/// <summary>
/// Artists.
/// </summary>
-#pragma warning disable CA1008 // Enums should have zero value. Cannot apply here.
Artist = 0,
-#pragma warning restore CA1008 // Enums should have zero value
/// <summary>
/// Album.
diff --git a/Jellyfin.Data/Entities/MediaStreamInfo.cs b/Jellyfin.Data/Entities/MediaStreamInfo.cs
index 79053652a..77816565a 100644
--- a/Jellyfin.Data/Entities/MediaStreamInfo.cs
+++ b/Jellyfin.Data/Entities/MediaStreamInfo.cs
@@ -1,9 +1,10 @@
+#pragma warning disable CS1591 // Missing XML comment for publicly visible type or member
+
using System;
using System.Diagnostics.CodeAnalysis;
namespace Jellyfin.Data.Entities;
-#pragma warning disable CS1591 // Missing XML comment for publicly visible type or member
public class MediaStreamInfo
{
public required Guid ItemId { get; set; }
diff --git a/Jellyfin.Data/Entities/People.cs b/Jellyfin.Data/Entities/People.cs
index b1834a70d..18c778b17 100644
--- a/Jellyfin.Data/Entities/People.cs
+++ b/Jellyfin.Data/Entities/People.cs
@@ -1,8 +1,9 @@
+#pragma warning disable CA2227 // Collection properties should be read only
+
using System;
using System.Collections.Generic;
namespace Jellyfin.Data.Entities;
-#pragma warning disable CA2227 // Collection properties should be read only
/// <summary>
/// People entity.