You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@abdera.apache.org by jm...@apache.org on 2006/07/14 22:08:51 UTC

svn commit: r422009 - /incubator/abdera/java/trunk/parser/src/main/java/org/apache/abdera/parser/stax/FOMContent.java

Author: jmsnell
Date: Fri Jul 14 13:08:51 2006
New Revision: 422009

URL: http://svn.apache.org/viewvc?rev=422009&view=rev
Log:
entry.getContent() used to throw a NPE when XML or XHTML content is empty. 
Strictly speaking, this is ok, but it's not really necessary.  Fixed.

Modified:
    incubator/abdera/java/trunk/parser/src/main/java/org/apache/abdera/parser/stax/FOMContent.java

Modified: incubator/abdera/java/trunk/parser/src/main/java/org/apache/abdera/parser/stax/FOMContent.java
URL: http://svn.apache.org/viewvc/incubator/abdera/java/trunk/parser/src/main/java/org/apache/abdera/parser/stax/FOMContent.java?rev=422009&r1=422008&r2=422009&view=diff
==============================================================================
--- incubator/abdera/java/trunk/parser/src/main/java/org/apache/abdera/parser/stax/FOMContent.java (original)
+++ incubator/abdera/java/trunk/parser/src/main/java/org/apache/abdera/parser/stax/FOMContent.java Fri Jul 14 13:08:51 2006
@@ -209,9 +209,11 @@
     } else if (Type.HTML.equals(type)) {
       val = getText();
     } else if (Type.XHTML.equals(type)) {
-      val = ((FOMDiv)this.getFirstChildWithName(Constants.DIV)).getInternalValue();
+      FOMDiv div = (FOMDiv)this.getFirstChildWithName(Constants.DIV);
+      if (div != null) val = div.getInternalValue();
     } else if (Type.XML.equals(type)) {
-      val = this.getFirstElement().toString();
+      OMElement el = this.getFirstElement();
+      if (el != null) val = el.toString();
     } else if (Type.MEDIA.equals(type)) {
       val = getText();
     }