From 4a4fef830eccf0629d7cf955126f0cd78867e0ee Mon Sep 17 00:00:00 2001 From: gnattu Date: Tue, 1 Apr 2025 07:43:31 +0800 Subject: Explicitly set default value for enums used in API models (#13821) Enums in response model with no nullability or default value will make the API very fragile as each extension to the enum will break the API for some clients, but a lot of enums actually do have an unknown value which should be used as a default. This set all model properties that are non-nullable using an enum that has an Unknown member in 10.10, except MediaStream.VideoRangeType which is refactored in #13277 --- MediaBrowser.Model/MediaSegments/MediaSegmentDto.cs | 2 ++ 1 file changed, 2 insertions(+) (limited to 'MediaBrowser.Model/MediaSegments') diff --git a/MediaBrowser.Model/MediaSegments/MediaSegmentDto.cs b/MediaBrowser.Model/MediaSegments/MediaSegmentDto.cs index 6e5c7885cc..d9129c3957 100644 --- a/MediaBrowser.Model/MediaSegments/MediaSegmentDto.cs +++ b/MediaBrowser.Model/MediaSegments/MediaSegmentDto.cs @@ -1,4 +1,5 @@ using System; +using System.ComponentModel; using Jellyfin.Database.Implementations.Enums; namespace MediaBrowser.Model.MediaSegments; @@ -21,6 +22,7 @@ public class MediaSegmentDto /// /// Gets or sets the type of content this segment defines. /// + [DefaultValue(MediaSegmentType.Unknown)] public MediaSegmentType Type { get; set; } /// -- cgit v1.2.3