Bug #434

EPG Problem on PL channels with encoding

Added by noem noem about 2 years ago. Updated 10 months ago.

Status:FixedStart date:04/12/2011
Priority:NormalDue date:
Assignee:Adam Sutton% Done:

0%

Category:EPG
Target version:2.13
Found in version:unknown Affected Versions:

Description

on most of the PL channels for the EPG is from Cyfra+
The coding should be:
ISO-8859-2 for source network id 113
ISO-6937 for source network id 318

Kind regards

Screen_shot_2011-04-12_at_12.55.46_PM.png - 1 (33.8 KB) noem noem, 04/12/2011 01:18 pm

Screen_shot_2011-04-12_at_12.54.58_PM.png - 2 (47.5 KB) noem noem, 04/12/2011 01:18 pm

dvb_support.c Magnifier (8.02 KB) bigtarget bigtarget, 05/12/2011 03:07 pm

dvb_support.c Magnifier - i clean printf functions (7.93 KB) bigtarget bigtarget, 05/12/2011 04:46 pm

fix_pl_epg.patch Magnifier - patch with tvheadend root (1.04 KB) bigtarget bigtarget, 05/13/2011 09:40 am

fix_polish_epg_final.patch Magnifier (1.37 KB) bigtarget bigtarget, 05/13/2011 04:00 pm

0001-polish-patch.patch Magnifier - No any configuration needed. (1.05 KB) bigtarget bigtarget, 07/05/2011 02:51 pm

Related issues

related to Feature #156: mediahighway epg and different epg encoding Rejected

History

#1 Updated by noem noem about 2 years ago

The problem is known you can get more info about it @ http://www.dvbviewer.tv/forum/topic/27458-epg-in-isoiec-6937/page__st__15

#3 Updated by Martin Mrvka about 2 years ago

If Cyfra+ is sending strings in their EPG without any charset information, it should default to ISO-6937 as per DVB spec.

This patch (https://github.com/andoma/tvheadend/pull/22) adds support to change the default charset and should solve your problem, since you can set the charset per service.

#4 Updated by Andreas Öman about 2 years ago

  • Category set to DVB
  • Status changed from New to Fixed
  • Assignee set to Andreas Öman
  • Target version set to 2.13

Fixed (thanks to Martin Mrvka) in fa941ddcb

#5 Updated by bigtarget bigtarget about 2 years ago

Hi, your patch is awesome, but please made one correct. When you change default encoding from web you have to force it in code, because it doesn't work for polish services. See atachment. When i am using my dvb_support.c I have polish all charactes with my change :) Thanks.

#6 Updated by bigtarget bigtarget about 2 years ago

bigtarget bigtarget wrote:

Hi, your patch is awesome, but please made one correct. When you change default encoding from web you have to force it in code, because it doesn't work for polish services. See atachment. When i am using my dvb_support.c I have polish all charactes with my change :) Thanks.

When you add my patch you can also close:
http://www.lonelycoder.com/redmine/issues/156

Thanks.

#8 Updated by Andreas Öman about 2 years ago

Please provide patches and not entire files.

#9 Updated by bigtarget bigtarget about 2 years ago

Here you are.

Thanks for merging in advance.

#10 Updated by bigtarget bigtarget about 2 years ago

My patch works, but i have only one problem :(

If I play channel with encoding ISO6937 and EPG is loading it should using all the time ISO6937, but now it is changing to service encoding :(

Example:
channel 1: ISO6937
channel 2: ISO8859-2

I watch channel 1 and it loads EPG for channel 2 at background but encoding is still ISO6937 not assigned to channel.

Could anyone fix it? I have no idea from where i can get wached channel id. I'm not a c++ programmer :) just JAVA and MUMPS :)

Thanks.

#11 Updated by bigtarget bigtarget about 2 years ago

I made final and working version of patch Polish EPG. Please merge it

Patch is on master of:
https://github.com/andoma/tvheadend.git
commit 77cbb4ba34aa4ede333a
tree 63f63f1ab4431dc85852
parent 1b05817d86d6b89e3336

#12 Updated by bigtarget bigtarget about 2 years ago

Hi Andreas,

Could you tell when you would like to merge it to git master?

My patch doesn't brake anythnig it is just condition for Polish EPG.

Thanks in advance.

#13 Updated by bigtarget bigtarget almost 2 years ago

Hi,

I would like to add patch with working all polish characters.
That code is hardcoded version but all characters is working correctly.

Thanks.

#14 Updated by Hein Rigolo almost 2 years ago

in order to supply patches it is preferred submit GIT pull requests with the changes in it with a reference to this ticket number.

#15 Updated by bigtarget bigtarget almost 2 years ago

When I'll have time, I will do it better now it is hard-coded: my repository:
https://github.com/bigtarget/tvheadend

#16 Updated by bigtarget bigtarget over 1 year ago

Could you pull that commit into master of tvheadend:
https://github.com/bigtarget/tvheadend/commit/363f417f441800b23e48dd28688fa2e33f58f44b

Thanks.

#19 Updated by Adam Sutton 10 months ago

  • Category changed from DVB to EPG
  • Assignee changed from Andreas Öman to Adam Sutton
  • Found in version set to unknown

Can you submit that as a pull request on main tvheadend repo (github.com/tvheadend/tvheadend) and I'll take a look.

#20 Updated by bigtarget bigtarget 10 months ago

Found in version: Build: 2.99.54.gfad45

#21 Updated by bigtarget bigtarget 10 months ago

Adam Sutton wrote:

Can you submit that as a pull request on main tvheadend repo (github.com/tvheadend/tvheadend) and I'll take a look.

Done: https://github.com/tvheadend/tvheadend/pull/107

#22 Updated by Adam Sutton 10 months ago

Thanks,

I'll try and take a look shortly.

Adam

Also available in: Atom PDF