Jump to content

Reading and writing of nfo for multi-part episodes is wrong


pünktchen

Recommended Posts

GrimReaper
12 minutes ago, worthmo said:

NFO reading is the only thing enabled.

 

Attached is a copy of how media companion creates NFO files for multi-episodes.

Paw Patrol S07E01E02.nfo 9.36 kB · 0 downloads

That NFO is not formatted correctly, as <episodedetails> tag should be top-level tag, as opposed to <multiepisodenfo> in example NFO.

 image.png.c3018df1f6be53e7d14c8b52d5b4b1bd.png

As <multiepisodenfo></multiepisodenfo>is unknown tag, anything contained within that node is not being read.

You either need to disable writing of said tag in Media Companion (if feasible, not using it so wouldn't know) or use an alternate software. 

 

 

Edited by GrimReaper
  • Agree 1
Link to comment
Share on other sites

Happy2Play
5 minutes ago, worthmo said:

NFO reading is the only thing enabled.

 

Attached is a copy of how media companion creates NFO files for multi-episodes.

Paw Patrol S07E01E02.nfo 9.36 kB · 1 download

Will test but this is a different design layout.

When was <multiepisodenfo> node added?   

image.png.040d607a2341cdd07198c2558eeee60c.png

But does not follow the Kodi standards per their kb though.

image.thumb.png.e38df2bbd9e3a884dae9269235e74348.png

 

So this would be another required change if this is added. @Luke

  • Agree 1
Link to comment
Share on other sites

Happy2Play

Follow up removing the <multiepisodenfo> node.

image.thumb.png.ed9d2ec4934b80a174bf6dec6c6c52f7.png

Link to comment
Share on other sites

worthmo
4 minutes ago, Happy2Play said:

Follow up removing the <multiepisodenfo> node.

image.thumb.png.ed9d2ec4934b80a174bf6dec6c6c52f7.png

The only you thing you did was remove the <multiepisodenfo> tags?  If so that is a simple fix from my side if needed.

Link to comment
Share on other sites

Happy2Play
11 minutes ago, worthmo said:

The only you thing you did was remove the <multiepisodenfo> tags?  If so that is a simple fix from my side if needed.

Yes I only removed the top open and bottom closing <multiepisodenfo> nodes.

Link to comment
Share on other sites

3 hours ago, Happy2Play said:

Will test but this is a different design layout.

When was <multiepisodenfo> node added?   

image.png.040d607a2341cdd07198c2558eeee60c.png

But does not follow the Kodi standards per their kb though.

image.thumb.png.e38df2bbd9e3a884dae9269235e74348.png

 

So this would be another required change if this is added. @Luke

So it's proprietary to media companion?

Link to comment
Share on other sites

Happy2Play
5 minutes ago, Luke said:

So it's proprietary to media companion?

Looks like @worthmoit in discussion over there.  Keep us updated on there reply.

Media Companion / Discussion / Requests Declined: Multiple TV Episodes Format for Emby (sourceforge.net)

 

 

But has anyone heard of Kodi using <multiepisodenfo>?

Edited by Happy2Play
  • Thanks 1
Link to comment
Share on other sites

6 minutes ago, Happy2Play said:

Looks like @worthmoit in discussion over there.  Keep us updated on there reply.

Media Companion / Discussion / Requests Declined: Multiple TV Episodes Format for Emby (sourceforge.net)

 

 

But has anyone heard of Kodi using <multiepisodenfo>?

Searching the kodi codebase yields no results and also found this:

https://github.com/xbmc/xbmc/issues/16506

Granted, it's always possible that it's supported in kodi but the code is somewhere else for it.

Link to comment
Share on other sites

sydlexius

At least Media Companion took a stab at making an XML file more compliant by having only a unique root node for that...better than the garbage cooked up by whoever is responsible for that XBMC/Kodi mess.  Still, thank you @Lukefor making something bug-compatible 🤣

Link to comment
Share on other sites

2 minutes ago, sydlexius said:

At least Media Companion took a stab at making an XML file more compliant by having only a unique root node for that...better than the garbage cooked up by whoever is responsible for that XBMC/Kodi mess.  Still, thank you @Lukefor making something bug-compatible 🤣

It advertises full xbmc integration so I'm wondering how the info gets imported from those nfo's to kodi.

Link to comment
Share on other sites

sydlexius
3 minutes ago, Luke said:

It advertises full xbmc integration so I'm wondering how the info gets imported from those nfo's to kodi.

Good question, given the Kodi Wiki example for a multi-episode.nfo file shows no mention of that element.

  • Thanks 1
Link to comment
Share on other sites

worthmo

This was the response from the media companion discussion.

 

I see Kodi has set that multi-episode data in a nfo be contained in repetitive
\< episodedetails> tags
but this causes xml syntax error of:

XML Parsing Error - Only one top level element is allowed in an XML document

But Kodi also supports the original method of wrapping \< episodedetails> tag within the
\< multiepisodenfo> tag, still allowing for legitimate XML file.

Media Companion still uses the \< multiepisodenfo> tag as this does not break the xml file making it impossible to load.

As this works for Media Companion, and for Kodi, there is no reason to make any changes to MC's current method of writing multi episode nfo's

Link to comment
Share on other sites

Create an account or sign in to comment

You need to be a member in order to leave a comment

Create an account

Sign up for a new account in our community. It's easy!

Register a new account

Sign in

Already have an account? Sign in here.

Sign In Now
×
×
  • Create New...