Modify

Ticket #554 (closed defect: fixed)

Opened 7 years ago

Last modified 3 years ago

Match duplicates using subtitle & description needs to also check partnumber/parttotal

Reported by: jppoet@… Owned by: gigem
Priority: minor Milestone: unknown
Component: mythtv Version: head
Severity: low Keywords: scheduler duplicates
Cc: Ticket locked: no

Description (last modified by danielk) (diff)

I noticed this a long time ago, but have been waiting for a good example.

Some mini-series shows will designate every episode with the exact same Title/Subtitle/Programid?. The only difference is the (Part X of Y) in the program information.

A current example (as of 31Oct05), is "Category 7: The End of the World". If you set up this show to record with "Match duplicates using subtitle & description), the second part will not record because it think it is the same as the first part.

If Bruce or David does not want to look at doing a "quick" fix for this, I will look into it. I am barely more than a beginner with SQL, however.

John

Attachments

programid-part.patch (2.0 KB) - added by jppoet@… 3 years ago.
Fudge programids to include the part number as necessary to make the programid unique
programid-part-fixes.patch (2.0 KB) - added by jppoet@… 3 years ago.
Version for -fixes (maybe). No guarantee it still applies.
programid-part-v1.1.patch (2.5 KB) - added by jppoet@… 3 years ago.
include parttotal as part of fudged programid

Change History

comment:1 Changed 7 years ago by danielk

  • Owner changed from ijr to gigem
  • Version set to head
  • Description modified (diff)
  • Milestone set to 0.20

comment:2 Changed 6 years ago by ijr

  • Milestone changed from 0.20 to 0.21

comment:3 Changed 4 years ago by janne

  • Milestone changed from 0.21 to unknown

comment:4 Changed 3 years ago by Dibblah

  • Status changed from new to assigned

Changed 3 years ago by jppoet@…

Fudge programids to include the part number as necessary to make the programid unique

comment:5 follow-up: ↓ 6 Changed 3 years ago by jppoet@…

I have attached a patch to this ticket which causes the programid to be fudged to include the mini-series "part". This is necessary for some mini-series such as "The Last Templar", which use the same programid for all the episodes. Without this patch, myth would only record the first episode of the series.

Bruce rejected this approach a few years ago, but never came up with a better solution, So I am throwing out there again.

Changed 3 years ago by jppoet@…

Version for -fixes (maybe). No guarantee it still applies.

comment:6 in reply to: ↑ 5 Changed 3 years ago by gigem

Replying to jppoet@gmail.com:

I have attached a patch to this ticket which causes the programid to be fudged to include the mini-series "part". This is necessary for some mini-series such as "The Last Templar", which use the same programid for all the episodes. Without this patch, myth would only record the first episode of the series.

While I don't see this hack as ideal, I've yet to think of any better solution. I'd certainly rather have this than add part number into the mix everywhere duplicate checking is done.

John, do you think you should encode the total number of parts in the programid as well?

Bruce rejected this approach a few years ago, but never came up with a better solution, So I am throwing out there again.

If Bruce still doesn't like it, he can come out of hiding and restate his case.

Changed 3 years ago by jppoet@…

include parttotal as part of fudged programid

comment:7 Changed 3 years ago by jppoet@…

As requested by David...

programid-part-v1.1.patch includes parttotal as part of fudged programid.

comment:8 Changed 3 years ago by gigem

  • Status changed from assigned to closed
  • Resolution set to fixed

(In [20318]) Applied a patch from John Poet (with a fix by me) to modify the programids of some multi-part programs to make them unique. The intent is to make sure successive parts don't get marked as duplicates after an earlier part is recorded. This hack isn't ideal, but it's better, IMO, than changing the code to also compare part numbers everywhere programids are compared.

Fixes #554.

View

Add a comment

Modify Ticket

Action
as closed
The resolution will be deleted. Next status will be 'new'
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.