Opened 9 years ago

Closed 7 years ago

Last modified 7 years ago

#13215 closed Patch - Feature (fixed)

Patch for Finnish EIT weirdness

Reported by: dag@… Owned by: Dag Nygren <dag@…>
Priority: minor Milestone: 29.2
Component: MythTV - General Version: v29-fixes
Severity: medium Keywords: Finnish EIT
Cc: Ticket locked: no

Description

Finnish EIT info for most channels will have an added (<age limit>) at the end of the title. Additionally this can change between different episodes. This is very bad for scheduling and I created a patch for eitfixup to remove this.

The patch also removes the disturbing Film: or Elokuva: that is used in the title of some movies and makes it really hard to find them later in the listings.

Attachments (4)

finnish-eit.patch (1.7 KB ) - added by dag@… 9 years ago.
Patch for eitfixup fixing Finnish EIT
finnish-eit.2.patch (1.7 KB ) - added by dag@… 9 years ago.
Fixed patch
eitfixup.cpp.diff (1.3 KB ) - added by dag@… 9 years ago.
FIx for too loose regexp
finnish-eit.3.patch (1.8 KB ) - added by dag@… 9 years ago.
Sorry, wrong file uploaded disregard previous one. This is the real one

Download all attachments as: .zip

Change History (13)

by dag@…, 9 years ago

Attachment: finnish-eit.patch added

Patch for eitfixup fixing Finnish EIT

by dag@…, 9 years ago

Attachment: finnish-eit.2.patch added

Fixed patch

comment:1 by dag@…, 9 years ago

Fxed the patch not to react to a year in parenthesis at the end of the title

comment:2 by dag@…, 9 years ago

Still modified the regexp not even to accept (1T) nor (1S).

This is as good as we can get.

by dag@…, 9 years ago

Attachment: eitfixup.cpp.diff added

FIx for too loose regexp

by dag@…, 9 years ago

Attachment: finnish-eit.3.patch added

Sorry, wrong file uploaded disregard previous one. This is the real one

comment:3 by Stuart Auchterlonie, 9 years ago

Milestone: needs_triage29.2
Version: Unspecifiedv29-fixes

comment:4 by dagnygren, 8 years ago

Just check out 0.30-fixes and had to reapply this patch. Could someone commit it please?

comment:5 by Jarno Suni, 7 years ago

I think "Elokuva:" (or "Film:", which I did not find examples of) can help in finding films in listings by Mythtv frontend and by Kodi (which has more advanced program search). BTW Yle channels use "Kino:", "Kino Suomi:" and "Uusi Kino:" How would you distinguish movies from series and other programs without such prefixes? I think duplicate matching could be improved to be more flexible by allowing some editing of fields by user defined regular expressions before comparing them.

comment:6 by dagnygren, 7 years ago

  1. Film or Elokuva depends on your language settings as we have two official languages in Finland
  1. A title is a title in my opinion and should not be messed with by the broadcaster. There is a category in the EIT (and in Mythtv) that should be used for telling if it is a series or a movie - And a lot of other things, but the broadcasters in Finland doesn't seem aware of it. There is also a unique ID in there, but that also to have been overlooked as it is empty in Finland.What we might be able to do is to force the category to Movie when we see these weird additions to the title. Feel free to expand the patch! And I might also expand it with the Kino-stuff when it starts irritating me enough ;-)

comment:7 by Dag Nygren <dag@…>, 7 years ago

Owner: set to Dag Nygren <dag@…>
Resolution: fixed
Status: newclosed

In 753a062b1/mythtv:

Error: Processor CommitTicketReference failed
GIT backend not available

comment:8 by Dag Nygren <dag@…>, 7 years ago

In 4d9959418/mythtv:

Error: Processor CommitTicketReference failed
GIT backend not available

comment:9 by Jarno Suni, 7 years ago

Now how do you search for upcoming films in those Finnish channels? Many series are listed as films in parenthesis in guide. Can you still see the age limit on guide, for example?

Note: See TracTickets for help on using tickets.