Forum rules
Under no circumstances is spamming or advertising of any kind allowed. Do not post any abusive, obscene, vulgar, slanderous, hateful, threatening, sexually-orientated or any other material that may violate others security. Profanity or any kind of insolent behavior to other members (regardless of rank) will not be tolerated. Remember, what you don’t find offensive can be offensive to other members. Please treat each other with the kind of reverence you’d expect from other members.
Failure to comply with any of the above will result in users being banned without notice. If any further details are needed, contact: “The team” using the link at the bottom of the forum page. Thank you.
mataka80
Posts: 10
Joined: Wed Mar 11, 2009 9:04 pm

Directors element has member with empty name element

Mon Apr 27, 2009 4:44 pm

Hi

When I call GetIMDBMovieDetails with 0427392 response will contain following:

.
.
.
<member>
<name>directors</name>
<value>
<struct>
<member>
<name>_0386570</name>
<value>
<string>Oliver Hirschbiegel</string>
</value>
</member>
<member>
<name/>
<value>
<string>more</string>
</value>
</member>
</struct>
</value>
</member>
.
.
.

This causes library I use to throw exception during parsing (http://ws.apache.org/xmlrpc/xmlrpc2/). Library doesn't like empty name elements.

Is this intentional to add that "more" element without name to response or is it just accidentially there (IMDB mostly doesn't have such "more" elements).

Maybe it would be better to just drop such more elements away from responses or if such information is wanted to be delivered to client maybe it should be done separately not "as one of directors"?

User avatar
eduo
Posts: 716
Joined: Sat Feb 10, 2007 1:40 am
Location: Information Technology
Contact: ICQ Website Yahoo Messenger

Fri May 01, 2009 2:18 pm

This is clearly a bug. It's like you're getting a paged version.

Do you have the full request and full response to the XMLRPC method?
http://eduo.info/
[url=http://eduo.info/soleol/]OpenSubtitles from your desktop: SolEol for Mac/Windows/Linux[/url]
[url=http://forums.plexapp.com/index.php?showtopic=325&st=0&p=2480&#entry2480]My current episode processing work flow[/url].

mataka80
Posts: 10
Joined: Wed Mar 11, 2009 9:04 pm

Sat May 02, 2009 5:53 pm

Request:

Code: Select all

<methodCall> <methodName>GetIMDBMovieDetails</methodName> <params> <param> <value><string>d0iot4l0teqk9nmn3s99bdhsh3</string> </value> </param> <param> <value><string>0427392</string></value> </param> </params> </methodCall>


Response:

Code: Select all

<?xml version="1.0" encoding="utf-8"?> <methodResponse> <params> <param> <value> <struct> <member> <name>status</name> <value> <string>200 OK</string> </value> </member> <member> <name>data</name> <value> <struct> <member> <name>cast</name> <value> <struct> <member> <name>_0000173</name> <value> <string>Nicole Kidman</string> </value> </member> <member> <name>_0185819</name> <value> <string>Daniel Craig</string> </value> </member> <member> <name>_0000562</name> <value> <string>Jeremy Northam</string> </value> </member> <member> <name>_2044796</name> <value> <string>Jackson Bond</string> </value> </member> <member> <name>_0942482</name> <value> <string>Jeffrey Wright</string> </value> </member> <member> <name>_0001021</name> <value> <string>Veronica Cartwright</string> </value> </member> <member> <name>_0813977</name> <value> <string>Josef Sommer</string> </value> </member> <member> <name>_0922927</name> <value> <string>Celia Weston</string> </value> </member> <member> <name>_0715953</name> <value> <string>Roger Rees</string> </value> </member> <member> <name>_2049959</name> <value> <string>Eric Benjamin</string> </value> </member> <member> <name>_0283299</name> <value> <string>Susan Floyd</string> </value> </member> <member> <name>_0077678</name> <value> <string>Stephanie Berry</string> </value> </member> <member> <name>_1187340</name> <value> <string>Alexis Raben</string> </value> </member> <member> <name>_0480775</name> <value> <string>Adam LeFevre</string> </value> </member> <member> <name>_0581008</name> <value> <string>Joanna Merlin</string> </value> </member> </struct> </value> </member> <member> <name>goofs</name> <value> <string>Continuity: When Carol rushes into the house after contamination, she has black stockings on her, but when she enters the bathroom and gets undressed, the stockings are white.</string> </value> </member> <member> <name>writers</name> <value> <struct> <member> <name>_1738734</name> <value> <string>David Kajganich</string> </value> </member> <member> <name>_0278277</name> <value> <string>Jack Finney</string> </value> </member> </struct> </value> </member> <member> <name>request_from</name> <value> <string>cache</string> </value> </member> <member> <name>rating</name> <value> <string>6.0</string> </value> </member> <member> <name>trivia</name> <value> <string>While filming at the Chilean Embassy in Washington D.C., Nicole Kidman and Daniel Craig were to drive a white SUV to the entrance, stop the car and allow valets to park it. Kidman forgot to place the car in park and as she got out it began to roll down the hill. Craig swooped in from the passenger side and dove at the car, hitting the emergency brake and stopping the disaster from proceeding. This happened during heavy rain effects. more</string> </value> </member> <member> <name>cover</name> <value> <string>http://ia.media-imdb.com/images/M/MV5BMTQyMDkzMjc0Ml5BMl5BanBnXkFtZTcwNDg2MzU1MQ@@._V1._SX100_SY122_.jpg</string> </value> </member> <member> <name>awards</name> <value> <string>1 nomination</string> </value> </member> <member> <name>genres</name> <value> <array> <data> <value> <string>Horror</string> </value> <value> <string>Sci-Fi</string> </value> <value> <string>Thriller</string> </value> </data> </array> </value> </member> <member> <name>id</name> <value> <string>0427392</string> </value> </member> <member> <name>votes</name> <value> <string>22536</string> </value> </member> <member> <name>country</name> <value> <array> <data> <value> <string>USA</string> </value> <value> <string>Australia</string> </value> </data> </array> </value> </member> <member> <name>language</name> <value> <array> <data> <value> <string>English</string> </value> </data> </array> </value> </member> <member> <name>directors</name> <value> <struct> <member> <name>_0386570</name> <value> <string>Oliver Hirschbiegel</string> </value> </member> <member> <name/> <value> <string>more</string> </value> </member> </struct> </value> </member> <member> <name>duration</name> <value> <string>99 min</string> </value> </member> <member> <name>certification</name> <value> <array> <data> <value> <string>UK</string> </value> <value> <string>Taiwan</string> </value> <value> <string>Australia</string> </value> <value> <string>Sweden</string> </value> <value> <string>Germany</string> </value> <value> <string>New Zealand</string> </value> <value> <string>Hong Kong</string> </value> <value> <string>Canada</string> </value> <value> <string>USA</string> </value> <value> <string>Brazil</string> </value> <value> <string>Netherlands</string> </value> <value> <string>Japan</string> </value> <value> <string>Singapore</string> </value> <value> <string>South Korea</string> </value> <value> <string>Argentina</string> </value> <value> <string>Malaysia</string> </value> <value> <string>Finland</string> </value> <value> <string>Ireland</string> </value> <value> <string>Portugal</string> </value> </data> </array> </value> </member> <member> <name>tagline</name> <value> <string>Do not trust anyone. Do not show emotion. Do not fall asleep.</string> </value> </member> <member> <name>plot</name> <value> <string>As a Washington psychiatrist unearths the origin of an alien epidemic, she also discovers her son might be the only way it can be stopped.</string> </value> </member> <member> <name>title</name> <value> <string>The Invasion</string> </value> </member> <member> <name>aka</name> <value> <array> <data> <value> <string>Invasion of the Body Snatchers (USA) (working title)</string> </value> <value> <string>The Visiting (USA) (working title)</string> </value> </data> </array> </value> </member> <member> <name>year</name> <value> <string>2007</string> </value> </member> </struct> </value> </member> <member> <name>seconds</name> <value> <double>0.644</double> </value> </member> </struct> </value> </param> </params> </methodResponse>

User avatar
oss
Site Admin
Posts: 5890
Joined: Sat Feb 25, 2006 11:26 pm
Contact: Website

Mon Jun 22, 2009 9:13 am

should be fixed now, please confirm.

Thanks for info.

mataka80
Posts: 10
Joined: Wed Mar 11, 2009 9:04 pm

Tue Jul 14, 2009 9:21 pm

Hi

I checked case with my test case and seems that particular problem is solved. Anyway it seems that movie doesn't have directors element at all anymore. Also cast and some others are missing. Is this because of this fix or change in imdb or something else (elements seems to be missing also from other movies)?

User avatar
oss
Site Admin
Posts: 5890
Joined: Sat Feb 25, 2006 11:26 pm
Contact: Website

Sun Jul 19, 2009 9:30 pm

please try again, it should work, it was little bit outdated imdb module and also small error in code :)

thanks for reporting!

mataka80
Posts: 10
Joined: Wed Mar 11, 2009 9:04 pm

Mon Jul 20, 2009 2:09 pm

Yes. Not it works :)

Return to “Developing”

Who is online

Users browsing this forum: No registered users and 25 guests