Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Scraping of nfo-file fails for year #1435

Closed
Omertron opened this issue Mar 15, 2015 · 9 comments
Closed

Scraping of nfo-file fails for year #1435

Omertron opened this issue Mar 15, 2015 · 9 comments

Comments

@Omertron
Copy link
Member

Original issue 1436 created by Omertron on 2010-04-23T18:31:53.000Z:

After generation, the movie Avatar gets the year 23 in the title.

The nfo-file has:
<?xml version="1.0" encoding="utf-8"?>
<movie xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
xmlns:xsd="http://www.w3.org/2001/XMLSchema">
<id>tt0499549</id>
<title>Avatar</title>
<originaltitle>Avatar</originaltitle>
<year>2009</year>
<rating>8.7</rating>
<votes>109,665</votes>

the xml-file has:
<?xml version="1.0" encoding="UTF-8"?>
<details>
<movie isExtra="false" isSet="false" isTV="false">
<id movieDatabase="imdb">tt0499549</id>
<mjbVersion>v2.0 BETA</mjbVersion>
<mjbRevision>UNKNOWN</mjbRevision>
<xmlGenerationDate>2010-04-17 14:53:03</xmlGenerationDate>
<baseFilename>avatar</baseFilename>
<title>Avatar</title>
<titleSort>Avatar</titleSort>
<originalTitle>Avatar</originalTitle>
<year index="Year_20-29_1">23</year>
<releaseDate>17-12-2009</releaseDate>
<rating>87</rating>

and the html-file is made from the xml-file, containing the year 23.

Yamj 1.9 r1604. Using the default skin.

@Omertron
Copy link
Member Author

Comment #1 originally posted by Omertron on 2010-04-23T18:35:54.000Z:

I use Ember Media Manager to pre-generate the nfo-files, and use YAMJ to generate the
jukebox. YAMJ 1.7 r1282 still works fine.

@Omertron
Copy link
Member Author

Comment #2 originally posted by Omertron on 2010-04-24T08:21:55.000Z:

I have tried this with the default skin and your provided NFO file and the year is
populated correctly.

Please update to the latest revision and see if that solves the issue

@Omertron
Copy link
Member Author

Comment #3 originally posted by Omertron on 2010-04-24T10:23:52.000Z:

attach the nfo file, maybe there is an encoding mismatch (it says utf8, but who knows
what the format really is...)

@Omertron
Copy link
Member Author

Comment #4 originally posted by Omertron on 2010-04-24T11:18:30.000Z:

Please update to the latest revision and see if that solves the issue

I thought r1604 was the latest revision ?

I've attached the nfo-file

@Omertron
Copy link
Member Author

Comment #5 originally posted by Omertron on 2010-04-24T17:26:50.000Z:

Found the issue. Ember is writing the format in DD-MM-YYYY format and YAMJ is
expecting it in YYYY-MM-DD

Working on adding support for a wider variety of date formats.

@Omertron
Copy link
Member Author

Comment #6 originally posted by Omertron on 2010-04-24T21:57:42.000Z:

this should be fixed in r1631

Please test and confirm

@Omertron
Copy link
Member Author

Comment #7 originally posted by Omertron on 2010-04-25T10:34:50.000Z:

Please test and confirm

I hope you don't mean me to test, as I have no idea how to download this.

Thanks for the fast fix !

@Omertron
Copy link
Member Author

Comment #8 originally posted by Omertron on 2010-04-25T10:37:27.000Z:

Yup, you :)

You can download the latest version from http://mediaplayersite.com/yamj_latest

@Omertron
Copy link
Member Author

Comment #9 originally posted by Omertron on 2010-04-25T19:04:01.000Z:

Tested, and it's fixed !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant