MediaElch icon indicating copy to clipboard operation
MediaElch copied to clipboard

[TMDb TV] `<status>` for Continuing and Canceled series displayed and stored incorrectly

Open mihawk90 opened this issue 1 year ago • 2 comments

Describe the bug The status for continuing and returning series are not displayed in the UI. They are stored in the NFO, however from what I can tell the stored values are incorrect.

To Reproduce

  1. Get some series with varying status, for example:
  2. Scrape all 3 series and check the status
  3. Only Castlevania will be filled as "Ended", which is correct, the other 2 will be empty
  4. Save all 3 series and check the NFO
    • :heavy_multiplication_x: 2 Broke Girls will say <status>Canceled</status>
    • :heavy_multiplication_x: Arcane will say <status>Returning Series</status>
    • :heavy_check_mark: Castlevania will say <status>Ended</status>

Expected behavior ME should store the correct status. I think this is probably a leftover from TheTVDB since from what I can tell they only have "Continuing" and "Ended" so the Scraper is probably just dumping whatever the API returns into the NFO. According to the Kodi Wiki on NFOs only "Continuing" and "Ended" are accepted. It also references the source code but I couldn't read much out of it whether it also accepts different values. I can however say that Jellyfin for one only supports these 2 and will display the year wrong if an invalid value is read from the NFO:

image image (although it doesn't always seem to do that :shrug: )

According to the TMDb docs there is also "In Production" but I don't think it makes much sense for a media manager anyway.

MediaElch Version:

  • [ ] 2.10.5-dev (nightly)
  • [x] 2.10.4 (stable)

Operating System:

  • [ ] Windows
  • [ ] macOS
  • [x] Linux (distribution: Fedora 38)

mihawk90 avatar Nov 23 '23 23:11 mihawk90

the Scraper is probably just dumping whatever the API returns into the NFO.

Correct.

I propose to map certain values to "Ended" and "Continuing".

Maps to "Ended":

  • Canceled
  • Ended

Maps to "Continuing":

  • Continuing
  • In Production

bugwelle avatar Nov 25 '23 13:11 bugwelle

Maps to "Continuing":

.. also Returning Series ;)

mihawk90 avatar Nov 26 '23 01:11 mihawk90