search

jellyfin / jellyfin-plugin-tvheadend

https://jellyfin.org
MIT License
32 stars 19 forks source link

Added Pre-Padding and Post-Padding to settings. #57

Closed lomion0815 closed 2 years ago

lomion0815 commented 2 years ago

These paddings are used when scheduling recordings in order to start seconds early and stop later. Before both values were fixed to 0 seconds.

crobibero commented 2 years ago

@lomion0815 what is the status of the TODO?

lomion0815 commented 2 years ago

Regarding "TODO: Check if an empty value results in using the DVR Profile settings.": A number must be entered in the configuration (defaults to 0). If the field is empty an error is thrown (see below). But this is not related to this patch but happens for every configuration field of type int. System.Text.Json.JsonException: The JSON value could not be converted to System.Int32. Path: $.Pre_Padding | LineNumber: 0 | BytePositionInLine: 184. ---> System.FormatException: Either the JSON value is not in a supported format, or is out of bounds for an Int32.)