Opened 11 years ago

Closed 10 years ago

#6185 closed defect (fixed)

Can't delete not listed show in Mythweb

Reported by: mythtv@… Owned by: Rob Smith
Priority: major Milestone: unknown
Component: Plugin - MythWeb Version: 0.21
Severity: high Keywords:
Cc: Ticket locked: no

Description

If i try change a show that is not listed to don't record i get an error

Fatal error: Call to a member function rec_override() on a non-object in /usr/share/mythtv/mythweb/modules/tv/upcoming.php on line 48

If i click on that show in upcoming recordings it returns to the programguide and i get a warning: Unkonwn Program

I'm using Debian linux mythweb version: 0.21.svn20080706-0.0

Attachments (1)

mythweb-6185-handle_not_listed_recordings.patch (1.9 KB) - added by sphery <mtdean@…> 10 years ago.
Adds handling for Not Listed shows

Download all attachments as: .zip

Change History (8)

comment:1 Changed 11 years ago by Rob Smith

Owner: changed from xris to Rob Smith
Status: newaccepted

comment:2 Changed 11 years ago by Rob Smith

Owner: Rob Smith deleted
Status: acceptednew

comment:3 Changed 10 years ago by Rob Smith

Resolution: invalid
Status: newclosed

I'm not entirely sure what you're after here. If the show doesn't exist in your guide data, it shouldn't be in your upcoming recordings list after the scheduler next runs...

Changed 10 years ago by sphery <mtdean@…>

Adds handling for Not Listed shows

comment:4 Changed 10 years ago by sphery <mtdean@…>

mythweb-6185-handle_not_listed_recordings.patch adds handling of NotListed? shows to the Upcoming Recordings page.

You can create a test NotListed? recording by creating any timeslot rule, then manually modifying the record.starttime, then running a mythbackend --resched.

Because the specified show does not actually exist, the normal link used for the recording title (to /tv/detail) does not work. Therefore, I put in a(n ugly) section that checks for NotListed? and makes the link send the user to the specific rule on the tv/schedules page.

I added "Not Listed" to the English.lang translation, but did not touch any other translations.

I hope this patch is more help than work. If nothing else, it's well tested and does what it should. :)

comment:5 Changed 10 years ago by Rob Smith

Resolution: invalid
Status: closednew

comment:6 Changed 10 years ago by stuartm

Owner: set to Rob Smith
Status: newassigned

comment:7 Changed 10 years ago by Rob Smith

Resolution: fixed
Status: assignedclosed

(In [21081]) Fixes #6185, this adds in a catch for not listed recordings

Note: See TracTickets for help on using tickets.