Cloud9Developer / Jellyfin-Newsletter-Plugin

GNU General Public License v3.0
74 stars 9 forks source link

Newsletter output malformed for Shows #76

Closed solidsnake1298 closed 4 months ago

solidsnake1298 commented 7 months ago

The format/output for shows is malformed. Movies are formatted correctly.

image

image

I have not been able to get a single show to display correctly.

It also treats whole libraries like a show.

image

Movies always work correctly.

image

solidsnake1298 commented 7 months ago

An example entry from the DB.

/media/library/TV Shows/Adventure Time/S02e25.mkv|TV Shows|-1|25||https://jellyfin.domain.tld:50443/Items/24178277f1a179c637ce237a0d3834e4/Images/Primary|24178277f1a179c637ce237a0d3834e4|/var/lib/jellyfin/metadata/library/02/02953f7f18c1d6ddffc2a8d00cc275aa/poster.jpg|Series

Cloud9Developer commented 7 months ago

This is odd.. Thought I fixed this a while back, guess not. Trying to find some time to actually sit down and do some refactoring and optimization, but haven't had a chance yet

Cloud9Developer commented 4 months ago

Still in my backlog, bare with me :) I'm starting to get some free time now. I'm planning on doing a pretty large overhaul of how the plugin works honestly, so that may address this issue

Cloud9Developer commented 4 months ago

After some major code refactoring, I scanned my ENTIRE library (3900 items total), No issues with formatting! So hopefully once I push the release, you shouldn't have this issue anymore :)

TheDMV2 commented 4 months ago

It also treats whole libraries like a show.

It is still doing this for me as well (even after deleting my db yesterday). v0.6.4.2

Cloud9Developer commented 4 months ago

I believe we discovered why in matrix the other night. It has to do with not following the proper file structure. @solidsnake1298 can you confirm if your re-org worked?

solidsnake1298 commented 4 months ago

It did.

solidsnake1298 commented 4 months ago

Specifically, I did not have season folders for each season of a show.

Cloud9Developer commented 4 months ago

Pinned this for others, thanks for confirming!