Return-Path: Delivered-To: apmail-cocoon-dev-archive@www.apache.org Received: (qmail 69725 invoked from network); 7 Nov 2004 00:08:05 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (209.237.227.199) by minotaur-2.apache.org with SMTP; 7 Nov 2004 00:08:05 -0000 Received: (qmail 55180 invoked by uid 500); 7 Nov 2004 00:08:01 -0000 Delivered-To: apmail-cocoon-dev-archive@cocoon.apache.org Received: (qmail 55126 invoked by uid 500); 7 Nov 2004 00:08:01 -0000 Mailing-List: contact dev-help@cocoon.apache.org; run by ezmlm Precedence: bulk list-help: list-unsubscribe: list-post: Reply-To: dev@cocoon.apache.org Delivered-To: mailing list dev@cocoon.apache.org Received: (qmail 55113 invoked by uid 99); 7 Nov 2004 00:08:01 -0000 X-ASF-Spam-Status: No, hits=-0.0 required=10.0 tests=SPF_HELO_PASS,SPF_PASS X-Spam-Check-By: apache.org Received-SPF: pass (hermes.apache.org: domain of ap-cocoon-dev@m.gmane.org designates 80.91.229.2 as permitted sender) Received: from [80.91.229.2] (HELO main.gmane.org) (80.91.229.2) by apache.org (qpsmtpd/0.28) with ESMTP; Sat, 06 Nov 2004 16:07:59 -0800 Received: from list by main.gmane.org with local (Exim 3.35 #1 (Debian)) id 1CQab7-0001IT-00 for ; Sun, 07 Nov 2004 01:07:57 +0100 Received: from avo133.neoplus.adsl.tpnet.pl ([83.27.48.133]) by main.gmane.org with esmtp (Gmexim 0.1 (Debian)) id 1AlnuQ-0007hv-00 for ; Sun, 07 Nov 2004 01:07:57 +0100 Received: from eltehaem by avo133.neoplus.adsl.tpnet.pl with local (Gmexim 0.1 (Debian)) id 1AlnuQ-0007hv-00 for ; Sun, 07 Nov 2004 01:07:57 +0100 X-Injected-Via-Gmane: http://gmane.org/ To: dev@cocoon.apache.org From: Wojciech Gdela Subject: [blocks.serializers] XHtmlSerializer bugfixes and improvements Date: Sun, 07 Nov 2004 01:07:51 +0100 Organization: Inbud.Net Lines: 133 Message-ID: Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: 7bit X-Complaints-To: usenet@sea.gmane.org X-Gmane-NNTP-Posting-Host: avo133.neoplus.adsl.tpnet.pl User-Agent: Noworyta News Reader/2.9 Sender: news X-Virus-Checked: Checked X-Spam-Rating: minotaur-2.apache.org 1.6.2 0/1000/N Hello, --- Improvement --- Recently I've found discussion about IE problems with empty textarea tags in XHtml notation (ie.

)
[http://thread.gmane.org/gmane.text.xml.cocoon.devel/33345]. There are
also problems with other tags, here is a snippet of code:

<?xml version="1.0" encoding="UTF-8"?>
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN"
"http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<body>
       <a name="foo"/> This text is supposed not to be a link.
       <div style="text-decoration: underline"/> This text shouldn't be
underlined
<body>
</html>

In order to have this work in IE you have to add closing tags:

<?xml version="1.0" encoding="UTF-8"?>
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN"
"http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<body>
       <a name="foo"></a> This text is not a link.
       <div style="text-decoration: underline"></div> This text is not
underlined
<body>
</html>

So I think that o.a.c.components.serializers.XHtmlSerializer should be
extended in this way:

public void endElementImpl(String uri, String local, String qual)
    throws SAXException {
        if (uri.length() == 0) uri = XHTML1_NAMESPACE;

if (XHTML1_NAMESPACE.equals(uri)) {
            if ((local.equalsIgnoreCase("textarea")) ||
                (local.equalsIgnoreCase("script")) ||
                (local.equalsIgnoreCase("style")) ||
                (local.equalsIgnoreCase("a")) || // THIS LINE ADDED
                (local.equalsIgnoreCase("div"))) { // THIS LINE ADDED
                this.closeElement(false);
            } else if (local.equalsIgnoreCase("head")) {
                String loc = "meta";
                String qua = namespaces.qualify(XHTML1_NAMESPACE, loc,
"meta");
                String nsp[][] = new String[0][0];
                String att[][] = new String[2][ATTRIBUTE_LENGTH];

att[0][ATTRIBUTE_NSURI] = att[1][ATTRIBUTE_NSURI] = "";
                att[0][ATTRIBUTE_LOCAL] = att[0][ATTRIBUTE_QNAME] =
"http-equiv";
                att[1][ATTRIBUTE_LOCAL] = att[1][ATTRIBUTE_QNAME] =
"content";
                att[0][ATTRIBUTE_VALUE] = "Content-Type";
                att[1][ATTRIBUTE_VALUE] = this.getMimeType();

this.closeElement(false);
                this.startElementImpl(XHTML1_NAMESPACE, loc, qua, nsp,
att);
                this.endElementImpl(XHTML1_NAMESPACE, loc, qua);
            }
        }
        super.endElementImpl(uri, local, qual);
    }

--- Bugfix ---

When I first tried to use XHtmlSerializer I've got this exception
[mentioned in
http://article.gmane.org/gmane.text.xml.cocoon.user/42495]:

org.xml.sax.SAXException: Unable to map "http://www.w3.org/2000/xmlns/"

After reading http://www.w3.org/2000/xmlns/ I think that adding one line
to Namespaces constructor will help:

public Namespaces() {
        super();

this.push("", "");
        this.push("xml", "http://www.w3.org/XML/1998/namespace");
        this.push("xmlns", "http://www.w3.org/2000/xmlns/"); // THIS
LINE ADDED
        this.last = this.depth;
    }

The xmlns is a special namespace, so i think that this would be ok.

--- Possibly Bugfix ---

In Namespaces there is a method named pop:

public synchronized void pop(String prefix)
    throws SAXException {
        for (int x = this.position(prefix, pre); x < (--this.depth);
x++) {
            int k = (x + 1);
            this.pre[x] = this.pre[k];
            this.uri[x] = this.uri[k];
        }
        this.pre[this.depth] = null;
        this.uri[this.depth] = null;
        this.last--;
    }

In every iteration the field this.depth will be decremented by one, I
think that the loop should look like this:

public synchronized void pop(String prefix)
    throws SAXException {
        for (int x = this.position(prefix, pre); x < (this.depth - 1);
x++) {
            int k = (x + 1);
            this.pre[x] = this.pre[k];
            this.uri[x] = this.uri[k];
        }
        this.pre[this.depth] = null;
        this.uri[this.depth] = null;
        this.last--;
    }

By the way: why don't you simply use HashMap with prefix as key and uri
as value?

-- 
Best regards,
Wojciech Gdela <eltehaem@poczta.onet.pl>