Fixed Error when air_date is "null" #174

2025-04-18T14:13:13Z

martenumberto commented

2025-04-18 14:13:13 +00:00

(Migrated from github.com)

Some Providers dont fill the "air_date" in Json or leave it "null" which results in an Empty Series.
To avoid that we guss that "air_date" is a String and Convert it to DateTime if it not equals "null"

Some Providers dont fill the "air_date" in Json or leave it "null" which results in an Empty Series. To avoid that we guss that "air_date" is a String and Convert it to DateTime if it not equals "null"

martenumberto commented

2025-04-20 13:02:59 +00:00

(Migrated from github.com)

Works, but may have following errors.

Kevinjil commented

2025-04-21 09:02:19 +00:00

(Migrated from github.com)

Thank you for your contribution. I'll keep this PR, together with #175, open until I have some time to dig a little deeper into it. I do not prefer the manual conversion needed in this PR, but if the JSON library does not properly support it I guess we will have to resort to this solution.

martenumberto commented

2025-04-21 11:31:38 +00:00

(Migrated from github.com)

Thx for the review!
My first thoughts about following errors are not related to this Issue/PR. It was a completly other issue https://github.com/Kevinjil/Jellyfin.Xtream/pull/176

Thx for the review! My first thoughts about following errors are not related to this Issue/PR. It was a completly other issue https://github.com/Kevinjil/Jellyfin.Xtream/pull/176

This pull request can be merged automatically.

You are not authorized to merge this pull request.

View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.

git fetch -u origin martenumberto/fix/series-date:martenumberto/fix/series-date

git checkout martenumberto/fix/series-date

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: Sirherobrine23/Jellyfin_Xtream#174