search

cmarrin / videomonkey

ffmpeg based video encoding tool for Mac OSX 10.6 and above
http://videomonkey.org
Other
38 stars 10 forks source link

Metadata for TV DB no longer works #108

Open linuxdan opened 9 years ago

linuxdan commented 9 years ago

ERROR parsing XML document: searching for TV show ID {Show ID Number}. The requested URL was not found on this server. WARNING:No appropriate metadata found for '{Name of Show}'

stuarthadley commented 9 years ago

occasionally I find its way file name is written. I did a show and had to search for it on TV DB then rename file to same as was on there and it then found all meta data. In my case it was a show remade in 2001 and without the correct name would only find original.

Must admit found that there are shows on TV DB that just can't be found. Hope that helps.

linuxdan commented 9 years ago

This does not help. Shows that I have been tagging for a while now do not work. I will provide sample file names later if you are willing to test them. Is there any way to turn on some debugging outpput or something like that ?

stuarthadley commented 9 years ago

Sorry afraid not sure about debugging output, I'm just a user who works with a lot of media files and metadata. Have had similar issues so feel your pain.

Happy to see if have same issue if you provide file names though. I do think there is something with the TV DB Api that must be causing issue but afraid just out of my knowledge.

linuxdan commented 9 years ago

Try this file name: NCIS.s12e11.mp4

I get the same error: ERROR parsing XML document: searching for TV show ID 72108. The requested URL was not found on this server. WARNING:No appropriate metadata found for 'NCIS'

…and I think VM is the problem because the episode exists on TVDB : http://www.thetvdb.com/?tab=episode&seriesid=72108&seasonid=589318&id=5064049&lid=7

cmarrin commented 9 years ago

There may be a problem with theTVDB. I just tried searching directly and got a popup error message.

Did you try just searching for NCIS from the Metadata panel?

linuxdan commented 9 years ago

I will try again when I get home.

I pulled down a copy of the source from GitHub and I was planning to walk thru the TVDB-API by hand. I looked on the TVDB forums and found a similar grumble, but it turned out the user was using older, obsolete API calls. Sounds to me like someone at TVDB recently/finally pulled the plug on this older API.

seanreiser commented 9 years ago

I know something changed at TVDB in the last week Identify needed to make some changes to support whatever changed.

cmarrin commented 9 years ago

Interesting. I knew that was going to happen, but I thought I had made the necessary changes. But I just looked and saw that the changes I made were in late 2013 and it was when MovieDB changed their API. So it's likely that the change to theTVDB API is the cause of your woes. I encourage linuxdan (or others) to look at TVDBMetadataSearcher.m. The changes would be to that file. If you have a fix, please submit a patch. I will review and land it in the source tree.

linuxdan commented 9 years ago

P.W.O.R. http://gerryanderson.wikia.com/wiki/P.W.O.R.

stuarthadley commented 9 years ago

Tried it and a few other TB shows i've been ripping. It's got to be an API issue. I think API's in general might have a few issues as have a bug with themoviedb where it returns multiple results of same name but won't let you choose any but the first version i presume because its same name.

linuxdan commented 9 years ago

Hey CMARRIN ! I am looking thru TVDBMetadataSearcher.m and all of the code and Metadata Notes.txt and I have a question: Where are you using the API key for TVDB ? I can only find mention of it in Metadata Notes.txt

cmarrin commented 9 years ago

Good question. I don't see it anywhere in the code. The one for MovieDB is there. Apparently theTVDB has never required it. Although maybe the do now and that's the problem?

linuxdan commented 9 years ago

I got your fix : http://forums.thetvdb.com/viewtopic.php?f=17&t=21545 http://www.thetvdb.com/wiki/index.php/Programmers_API

cmarrin commented 9 years ago

Excellent, and an easy fix! I will get it in as soon as I get a chance - hopefully this weekend. Thank you for tracking it down.

I guess when I was writing it, I saw some sample code that used the non-key URL and since it worked never bothered to change it.

mattbishop commented 9 years ago

Hello, any luck getting a fix in?

dash16 commented 9 years ago

Eagerly awaiting the fix, entering metadata by hand is getting to be a real pain!

cmarrin commented 9 years ago

Sorry, things have been up in the air at the Shire with some construction work. I'll try to get it in this weekend

cmarrin commented 9 years ago

Ok, I've finally gotten version 0.17 built and uploaded. If your copy doesn't autoupdate please go to the VideoMonkey menu and choose Check for Updates...

Have fun!