[mythtv-commits] Ticket #6428: tmdb.pl - Check for response before parsing

[MythTV]

#6428: tmdb.pl - Check for response before parsing
----------------------------------------------+-----------------------------
 Reporter:  sphery <mtdean at thirdcontact.com>  |       Owner:  awithers
     Type:  patch                             |      Status:  new     
 Priority:  minor                             |   Milestone:  unknown 
Component:  Plugin - MythVideo                |     Version:  head    
 Severity:  medium                            |     Mlocked:  0       
----------------------------------------------+-----------------------------
 The attached patch checks the response before parsing it, providing a much
 more sensible error message when network (or server) errors occur.

 Before the patch, the message is something like:

 File does not exist:  at ./tmdb.pl line 302

 After the patch:

 Unable to contact themoviedb.org while retrieving movie list, stopped at
 ./tmdb.pl line 328.

 (or similar).  I tested the patch with an /etc/hosts entry:

 127.0.0.1 api.themoviedb.org

 but haven't yet tested the patch with a real, live network/server issue.
 Still waiting for (another) one to occur.

-- 
Ticket URL: <http://svn.mythtv.org/trac/ticket/6428>
MythTV <http://www.mythtv.org/>
MythTV