Opened 14 years ago

Closed 14 years ago

#1293 closed patch (fixed)

Need to check if both seriesid and programid exists

Reported by: Oscar Carlsson Owned by: xris
Priority: minor Milestone: 0.20
Component: mythweb Version: head
Severity: medium Keywords:
Cc: Ticket locked: no


For "This show was recorded", both seriesid and programid need to exist in order to be able to use it for the check.

Attachments (1)

mythweb.oldrecorded.diff (750 bytes) - added by Robert Tsai <rtsai1111> 14 years ago.

Download all attachments as: .zip

Change History (9)

comment:1 Changed 14 years ago by tephra@…

Since this bug only cropped up in the weeks before .19 was released, shouldn't it be part of the .19 fixes?

see #1334 for my version of the problem.

comment:2 Changed 14 years ago by Robert Tsai <rtsai1111>

Type: defectpatch

I think the current SVN checks for programid/seriesid. Looking at ProgramInfo::ForgetHistory?, I think that oldrecorded.duplicate might need to be checked as well. See patch. This also has the (positive?) side-effect that non-AutoExpired? recordings are no longer marked as "This showing was recorded".

Changed 14 years ago by Robert Tsai <rtsai1111>

Attachment: mythweb.oldrecorded.diff added

comment:3 Changed 14 years ago by tephra@…

Hi Robert,

I tried the patch, but am still getting shows in the future displaying as "Already recorded"

Do you need anything from me? screenshots/db rows?

Cheers Dave

comment:4 Changed 14 years ago by xris

Resolution: fixed
Status: newclosed

(In [9119]) This should hopefully fix #1293

comment:5 Changed 14 years ago by tephra@…

Yep, I was just about to post this myself :) looks like the OR should have been an AND :)

can we push this change the 0.19-fixes please :)

comment:6 Changed 14 years ago by Oscar Carlsson

Next problem with this is the title/subtitle/description comparation that breaks if for example there are no subtitles. "" = "" evaluates as true thus marking the show as previously recorded..

It needs to behave the same way as myth does, if there's no sub AND desc, it can't match them.

comment:7 Changed 14 years ago by anonymous

Resolution: fixed
Status: closedreopened

comment:8 Changed 14 years ago by xris

Resolution: fixed
Status: reopenedclosed

(In [9177]) once and for all hopefully close #1293

Note: See TracTickets for help on using tickets.