[EMAIL PROTECTED] wrote:
Author: joerg
Date: Tue Nov 23 06:27:10 2004
New Revision: 106302
Modified:
cocoon/branches/BRANCH_2_1_X/src/blocks/batik/java/org/apache/cocoon/transformation/FragmentExtractorTransformer.java
Log:
fixed the way of fixing the handling for null namespace URIs ...
Modified:
cocoon/branches/BRANCH_2_1_X/src/blocks/batik/java/org/apache/cocoon/transformation/FragmentExtractorTransformer.java
Url:
http://svn.apache.org/viewcvs/cocoon/branches/BRANCH_2_1_X/src/blocks/batik/java/org/apache/cocoon/transformation/FragmentExtractorTransformer.java?view=diff&rev=106302&p1=cocoon/branches/BRANCH_2_1_X/src/blocks/batik/java/org/apache/cocoon/transformation/FragmentExtractorTransformer.java&r1=106301&p2=cocoon/branches/BRANCH_2_1_X/src/blocks/batik/java/org/apache/cocoon/transformation/FragmentExtractorTransformer.java&r2=106302
==============================================================================
---
cocoon/branches/BRANCH_2_1_X/src/blocks/batik/java/org/apache/cocoon/transformation/FragmentExtractorTransformer.java
(original)
+++
cocoon/branches/BRANCH_2_1_X/src/blocks/batik/java/org/apache/cocoon/transformation/FragmentExtractorTransformer.java
Tue Nov 23 06:27:10 2004
@@ -213,7 +213,7 @@
*/
public void startElement(String uri, String loc, String raw, Attributes a)
throws SAXException {
IMHO, adding single line would be simplier:
if (uri == null) uri = "";
And it can not be null anyway, so logging a warning is an option.
Vadim
- if ((uri == null || this.extractURI.equals(uri)) &&
this.extractElement.equals(loc)) {
+ if (((uri == null && this.extractURI.equals("")) ||
this.extractURI.equals(uri)) && this.extractElement.equals(loc)) {
extractLevel++;
fragmentID++;
if (getLogger().isDebugEnabled()) {
