search

YAMJ / yamj-v2

Yet Another Movie Jukebox (YAMJ) v2
GNU General Public License v3.0
28 stars 11 forks source link

unable to import multi episode NFO #2372

Closed Omertron closed 9 years ago

Omertron commented 9 years ago

Original issue 2373 created by Omertron on 2012-03-04T22:26:16.000Z:

What steps will reproduce the problem?

  1. create a multi episode file like S01E01E02
  2. EMM will create an NFO file that is not valid as it has 2 roots
  3. Parse with JAMY

What is the expected output? YAMJ to recognize the error and split the file in the episodes

What do you see instead? jus the error

What version/revision of YAMJ are you using? Latest drop

On what operating system? W7 x64

What skin are you using? Eversion

What device are you using YAMJ on? Networked Media Tank, PlayonHD, Other? PCH-C200

Please provide any additional information below.

[23:14:58 - Thread-6] ERROR com.moviejukebox.scanner.MovieNFOScanner - NFOScanner: Failed parsing NFO file: W:\pch-bimbi\Video\Serie Bimbi\Detective Conan\Season 01\Detective Conan S01E172E173 - Il primo caso di Shinichi.nfo. Please fix or remove it.

Message: The markup in the document following the root element must be well-formed. at com.sun.org.apache.xerces.internal.impl.XMLStreamReaderImpl.next(Unknown Source) at com.sun.xml.internal.stream.XMLEventReaderImpl.nextEvent(Unknown Source) at com.moviejukebox.scanner.MovieNFOScanner.parseTVNFO(MovieNFOScanner.java:950) at com.moviejukebox.scanner.MovieNFOScanner.parseXMLNFO(MovieNFOScanner.java:369) at com.moviejukebox.scanner.MovieNFOScanner.scan(MovieNFOScanner.java:133) at com.moviejukebox.MovieJukebox.updateMovieData(MovieJukebox.java:1974) at com.moviejukebox.MovieJukebox$3.call(MovieJukebox.java:952) at com.moviejukebox.MovieJukebox$3.call(MovieJukebox.java:936) at java.util.concurrent.FutureTask$Sync.innerRun(Unknown Source) at java.util.concurrent.FutureTask.run(Unknown Source) at java.util.concurrent.ThreadPoolExecutor$Worker.runTask(Unknown Source) at java.util.concurrent.ThreadPoolExecutor$Worker.run(Unknown Source) at java.lang.Thread.run(Unknown Source) at com.moviejukebox.tools.ThreadExecutor$ScheduledThread.run(ThreadExecutor.java:101)

Omertron commented 9 years ago

Comment #1 originally posted by Omertron on 2012-04-01T19:57:03.000Z:

http://code.google.com/p/moviejukebox/wiki/NFO_Files#XBMC_NFO_episodedetails_xml As stated in the wiki, you need a "root" element around all of the episode details

.... ....
Omertron commented 9 years ago

Comment #2 originally posted by Omertron on 2012-04-08T20:26:14.000Z:

Stuart, thank you for the clarification.

My request is to add a flag-feature so that if YAMJ founds and invalid XML with multiple roots it authomatically create a correct file adding the

aroudn the content of the original file.

This because EMM creates the worng XML for multiepisode files and is impossible to fix :(

Omertron commented 9 years ago

Comment #3 originally posted by Omertron on 2012-08-25T19:58:21.000Z:

Can you please open the issue and add a flag for this? I cannot change the (invalid) XML otherwise Ember will not work

Omertron commented 9 years ago

Comment #4 originally posted by Omertron on 2012-08-26T07:51:06.000Z:

This has been fixed in one of the later releases of YAMJ

It does not create the NFO file, but will read them fine.

Omertron commented 9 years ago

Comment #5 originally posted by Omertron on 2012-08-26T09:45:11.000Z:

Confirmed. I updated YAMJ to yamj-2.7-SNAPSHOT-r3150-bin and now works fine. Thank you for your help!