Fix for categories and cast

themoviedbapi/src/com/moviejukebox/themoviedb/tools/DOMParser.java

@@ -75,13 +75,20 @@ public class DOMParser {
                     subNode = subNodeList.item(nodeLoop);
                     if (subNode.getNodeType() == Node.ELEMENT_NODE) {
                         subElement = (Element) subNode;
-                        Category category = new Category();
+
-                        
+                        NodeList castList = subNode.getChildNodes();
-                        category.setType(DOMHelper.getValueFromElement(subElement, "type"));
+                        for (int i = 0; i < castList.getLength(); i++) {
-                        category.setUrl(DOMHelper.getValueFromElement(subElement, "url"));
+                            Node personNode = castList.item(i);
-                        category.setName(DOMHelper.getValueFromElement(subElement, "name"));
+                            if (personNode.getNodeType() == Node.ELEMENT_NODE) {
-                     
+                                subElement = (Element) personNode;
-                        movie.addCategory(category);
+                                Category category = new Category();
+
+                                category.setType(subElement.getAttribute("type"));
+                                category.setUrl(subElement.getAttribute("url"));
+                                category.setName(subElement.getAttribute("name"));
+                                movie.addCategory(category);
+                            }
+                        }
                     }
                 }
                 
@@ -109,18 +116,25 @@ public class DOMParser {
                     subNode = subNodeList.item(nodeLoop);
                     if (subNode.getNodeType() == Node.ELEMENT_NODE) {
                         subElement = (Element) subNode;
-                        Person person = new Person();
+
-                        
+                        NodeList castList = subNode.getChildNodes();
-                        person.setUrl(DOMHelper.getValueFromElement(subElement, "url"));
+                        for (int i = 0; i < castList.getLength(); i++) {
-                        person.setName(DOMHelper.getValueFromElement(subElement, "name"));
+                            Node personNode = castList.item(i);
-                        person.setJob(DOMHelper.getValueFromElement(subElement, "job"));
+                            if (personNode.getNodeType() == Node.ELEMENT_NODE) {
-                        person.setCharacter(DOMHelper.getValueFromElement(subElement, "character"));
+                                subElement = (Element) personNode;
-                        person.setId(DOMHelper.getValueFromElement(subElement, "id"));
+                                Person person = new Person();
-                     
+
-                        movie.addPerson(person);
+                                person.setUrl(subElement.getAttribute("url"));
+                                person.setName(subElement.getAttribute("name"));
+                                person.setJob(subElement.getAttribute("job"));
+                                person.setCharacter(subElement.getAttribute("character"));
+                                person.setId(subElement.getAttribute("id"));
+                                movie.addPerson(person);
+                            }
+                        }
                     }
                 }
-
+                
                 /*
                 * This processes the image elements. There are two formats to deal with:
                 * Movie.imdbLookup, Movie.getInfo & Movie.search: