Mythbuntu

MythMovies plugin broken due to change at ignyte.com

Bug #230240 reported by dmandell on 2008-05-14

Affects		Status	Importance	Assigned to	Milestone
	Mythbuntu	Fix Released	Undecided	Unassigned
	mythplugins (Ubuntu)	Fix Released	Undecided	Mario Limonciello

Bug Description

Some time in the last few days the MythMovies (aka Movie Times), or at least the US grabber that gets packages with MythBuntu (ignyte) is broken.

If you run "ignyte --zip 97202 --radius 5" from the command line you'll see that ignyte.com returns:

<?xml version="1.0" encoding="utf-8"?><MovieTimes></title></head><body><h1>Object Moved</h1>This document may be found <a HREF="http://ignyte.com/webservices/ignyte.whatsshowing.webservice/moviefunctions.asmx">here</a></body></MovieTimes>

I edited ignytegrabber.cpp (in mythplugins/mythmovies/ignyte) and changed line 19 from:

QString server("www.ignyte.com");

QString server("ignyte.com");

then recompiled, the resulting binary works again.

The unfortunate part about this bug is that Mythfrontend crashes when proper xml isn't returned to MythMovies... so until this bug is fixed any time someone clicks on "Movie Times" the frontend will restart. Blech. I thought it might be better to report the bug here so it could be fixed quickly by you before too many Mythbuntu users were affected.

Related branches

lp:ubuntu/karmic/mythplugins

Revision history for this message

JamesGoben (jgoben) wrote on 2008-05-14:

Response when ignyte is ran - piped to text file. Edit (238 bytes, text/plain)

I second this. Attached is the response from the web service.
Below is the stdout from mythfrontend:

2008-05-14 18:44:11.926 MythMovies Data Grabber: Executing '/usr/bin/ignyte --zip 47715 --radius 20'
2008-05-14 18:44:12.068 Grabber Finished. Processing Data.
2008-05-14 18:44:12.068 Error parsing data from grabber: Error: tag mismatch Location Line: 1 Column 58

Revision history for this message

duckduckgoose (jeffery-merle-potter) wrote on 2008-05-15:

I third that,

Mythmovies ignyte grabber kills frontend. The ignyte server address has changed.

Related to http://svn.mythtv.org/trac/ticket/5165 using googlemovies instead of ignyte.

This ignyte bug is not reported at svn.mythtv.org, and still exist in the current codebase.

I modified my version of ignyte to accept a --server argument to protect against further address magic.

using --server ignyte.com returns proper xml listings.

propose: since mythvideos is using a command line option in the frontend, add --server argument
to binary, and set it in the interface to the correct address.

Revision history for this message

Mario Limonciello (superm1) wrote on 2008-06-01:

We'll get the crash avoidance for free in the next upload (it was in 17361), but I'll add a patch for switching the server to ignyte.com. Can you please file a bug upstream?

Revision history for this message

Mario Limonciello (superm1) wrote on 2008-06-01:

Well actually we get the fix for that for free too. It's been changed already upstream. This will all be fixed in the next upload

Revision history for this message

Mario Limonciello (superm1) wrote on 2008-06-01:

bzr #60

Changed in mythplugins:
assignee:	nobody → superm1
status:	New → Fix Committed
Changed in mythbuntu:
status:	New → Fix Committed

Revision history for this message

Launchpad Janitor (janitor) wrote on 2008-06-02:

This bug was fixed in the package mythplugins - 0.21.0+fixes17416-0ubuntu1

---------------
mythplugins (0.21.0+fixes17416-0ubuntu1) intrepid; urgency=low

  [ Mario Limonciello ]
  * New upstream checkout.
    - Fixes mythmovies stability issues (LP: #230240)
  * debian/control:
    - Build depend on 17416 of mythtv-common.
  * Update 11_ofdb_r16988_r16995 for changes that were absorbed into -fixes.

  [ Michael Haas ]
  * Don't fail if {mythgallery, mythvideo, mythmusic} can't chmod/chown their
    directories in /var/lib/. Some people have these directories mounted on NFS
    which will prevent those packages from upgrading/installing successfully.
    (LP: #220087)

-- Mario Limonciello <email address hidden> Sun, 01 Jun 2008 17:50:28 -0500