[mythtv-users] tmdb grabber switch to v3

[Joey Morris]

Raymond Wagner <raymond at wagnerrp.com> wrote on Thu, Sep 19, 2013 at 11:02:41PM -0400:
> On 9/19/2013 8:17 AM, Raymond Wagner wrote:
> >On Sep 19, 2013, at 8:05, Joey Morris <rjmorris.list at zoho.com> wrote:
> >
> >>Al Bates <d2a at lineone.net> wrote on Thu, Sep 19, 2013 at 09:41:58AM +0100:
> >>>Not sure if I'm the only one whose metadata grabber stopped working
> >>>- can't see any mentions here so perhaps my system is unusual.
> >>>
> >>>TMDB recently (September 15th) retired their v2.1 api, which meant
> >>>that my myth 0.26 system setup (mythbuntu, recently updated) stopped
> >>>looking up video metadata.
> >>>
> >>>The fix (in my theme) is to go to Setup/Artwork and data sources and
> >>>switch the Movie Metadata Source from TheMovieDB.org to TheMovieDB.org
> >>>V3
> >>Thanks. I had just noticed mine stopped working but hadn't
> >>investigated yet. Changing the Movie Metadata Source worked for me,
> >>mostly. The problem is that it picks up the rating incorrectly. For
> >>example, it grabs a rating of K-18 for Reservoir Dogs. I'm in the US,
> >>so the rating should be R. On the TMDB web page for this movie
> >>(http://www.themoviedb.org/movie/500-reservoir-dogs), it looks like
> >>K-18 is the rating in Finland.
> >First guess is that everything is working as it should, but it's case sensitive.  The script wants capital locales, while MythTV wants them lowercase, so with no exact match, it just takes the first one off the top of the list.
> 
> That didn't pan out.  The code explicitly forces the country code in
> the locale to be upper case.
> 
> http://code.mythtv.org/cgit/mythtv/tree/mythtv/libs/libmythbase/mythlocale.cpp#n33
> 
> Digging further, the metadata library code was intended to support
> locale-specific certifications, but since no source at the time
> offered them, the library simply added a comment in the vain "add
> this later", and pulls the first certification returned by the
> grabber.  However, since tmdb3.py grabber does do locale sorting for
> certifications, the first one returned should be the one supplied in
> the arguments, and everything should still work.
> 
> http://code.mythtv.org/cgit/mythtv/tree/mythtv/libs/libmythmetadata/metadatacommon.cpp#n1061
> 
> Could you check your logs for a line that looks like this?
> 
> Running Grabber: /usr/local/share/mythtv/metadata/Movie/tmdb3.py -l
> en -a US -D 500

Thanks for looking into this, Raymond. My line like that is missing
the -a US argument.

Running Grabber: /usr/local/share/mythtv/metadata/Movie/tmdb3.py -l en -M Reservoir Dogs
Running Grabber: /usr/local/share/mythtv/metadata/Movie/tmdb3.py -l en -D 500

I'm running 0.26-fixes.