search

MathewWi / xbmc-addons

Automatically exported from code.google.com/p/xbmc-addons
0 stars 0 forks source link

XBMCEyeTvScraper error with proposed fix #47

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago 
What steps will reproduce the problem?
1.Use Version 3.0 of EyeTV
2.Scrape a standard EYE Archive that has a package name with a /
3. The scrape process generates an error part of the way through and cancels.

What is the expected output? What do you see instead?
Expect this output:
Equitrekking.Scotland.mpg
Equitrekking.Scotland.tbn
Equitrekking.Southern Spain.mpg
Equitrekking.Southern Spain.tbn
Equitrekking.Utah.mpg
Equitrekking.Utah.tbn

Instead get a list of files sorted by number or the process errors out.

The issue is that EyeTV no longer stores data in the SUBTITLE field of the
epguide file. Instead the show title is stored in the ABSTRACT field.

Fixed the issue by editing default.py on lines 503 and 504:
503 # p=re.compile('<key>SUBTITLE</key><string>(.*?)</string>')
504 p=re.compile('<key>ABSTRACT</key><string>(.*?)</string>')

This introduced a problem because the abstract sometimes contains a /
character the breaks the symbolic creation at about line 943.

Fixed this problem with the / at new line 508
508 subtitle=subtitle.replace("/", "-")

Running latest version of XBMC with latest version of XBMCEyeTVScraper on
Mac OSX 10.6.2

I suppose a more version independent fix would be to check both the
Abstract and subtitle and grab the information from the field that has info.

Original issue reported on code.google.com by davetgar...@gmail.com on 9 Feb 2010 at 3:09

Attachments:

GoogleCodeExporter commented 9 years ago 
I wrote the original, this field may be different in different 
markets/countries.
Feel free to mod, I havn't been using this for a while now, so if you want to 
run
with it, go for it, gpl and all.
Nice work on the fix, post it in the code repo, or start a new branch.

Original comment by propheadcity@gmail.com on 10 Feb 2010 at 7:47