Re: svn commit: r106302 - /cocoon/branches/BRANCH_2_1_X/src/blocks/batik/java/org/apache/cocoon/transformation/FragmentExtractorTransformer.java

[Joerg Heinicke]

On 23.11.2004 17:14, Vadim Gritsenko wrote:
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.
-        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)) {
I had this in mind too, but should a transformer fix the behaviour of 
another component? In theory you can add this fix to every transformer 
or serializer.

Joerg