Opened 11 years ago

Closed 11 years ago

#6336 closed enhancement (fixed)

Add fanart/banner/coverfile support to Watch Recordings

Reported by: robert.mcnamara@… Owned by: stuartm
Priority: minor Milestone: unknown
Component: MythTV - User Interface Library Version: head
Severity: medium Keywords:
Cc: Ticket locked: no

Description

Now that fanart/banner/coverfile/screenshot support exists in MythVideo?, this patch adds an "Image Hunt" function as well as the UI stuff necessary for fanart/coverfile/banner support in the PlayBackBox?. Because the metadata download stuff is a frontend functionality, this is a half-measure towards the best way to accomplish this. Ultimately I'd like to see the scheduler do a metadata download for anything scheduled which doesn't have fanart/coverfile/banner, and have it set as a DB value in recordedprogram by the time the recording occurs. That will require moving the download stuff into the backend/libraries-- maybe for .23? In the meantime, here is how the image hunt works, I'm just cutting and pasting the comments from the code as they explain it well:

+ Attempts to match image file in specified directory.<BR> + Falls back like this:<BR> + <BR> + Pushing Daisies 5.png<BR> + PushingDaisies5.png<BR> + PushingDaisiesSeason5.png<BR> + Pushing Daisies Season 5 Episode 1.png<BR> + PuShinG DaisIES s05e01.png<BR> + etc. (you get it)<BR> + <BR> + Or any permutation thereof including -,_, or . instead of space<BR> + Then, match by seriesid (for future PBB grabber):<BR> + <BR> + EP0012345.png<BR> + <BR> + Then, as a final fallback, match just title<BR> + <BR> + Pushing Daisies.png (or Pushing_Daisies.png, etc.)<BR> + <BR> + All this allows for grabber to grab an image with format:<BR> + <BR> + Title SeasonNumber?.ext or Title SeasonNum? # Epnum #.ext<BR> + or SeriesID.ext or Title.ext (without caring about cases,<BR> + spaces, dashes, periods, or underscores)<BR>

In about six weeks of using this code it has worked pretty flawlessly. In the short term this means copying a few correctly named images for your favorite shows into the fanart/banner/coverfile dirs, but I hope to solve this as mentioned above. In the meantime it's a nice visual feature that works well with fairly minimal intervention.

This splits out part of #6158. If this gets committed I will update that patch accordingly.

Attachments (2)

PBBfanart.diff (6.4 KB) - added by robert.mcnamara@… 11 years ago.
Add fanart/banner/coverfile support to PBB
PBBfanart.2.diff (6.2 KB) - added by robert.mcnamara@… 11 years ago.
Fix for one errant hunk that got in there

Download all attachments as: .zip

Change History (3)

Changed 11 years ago by robert.mcnamara@…

Attachment: PBBfanart.diff added

Add fanart/banner/coverfile support to PBB

Changed 11 years ago by robert.mcnamara@…

Attachment: PBBfanart.2.diff added

Fix for one errant hunk that got in there

comment:1 Changed 11 years ago by stuartm

Resolution: fixed
Status: newclosed

(In [20172]) Add fanart/banner/coverfile support to Watch Recordings. Patch from Robert McNamara?, see #6336 for details. Closes #6336

Note: See TracTickets for help on using tickets.