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();
}