[EMAIL PROTECTED] writes:
>  +                case 1:
>  +                    {
>  +                        String doc = (String) doctypeProperty.firstElement();
>  +                        tag = "HTML";
>  +                        identifier = "-//W3C//DTD HTML 4.0 ";
>  +                        uri = "http://www.w3.org/TR/REC-html40/";;
>  +                        if (doc.equalsIgnoreCase(
>  +                             TurbineConstants.DOCUMENT_TYPE_HTML40TRANSITIONAL))
>  +                        {
>  +                            identifier += "Transitional";
                            uri += "loose";
>  +                        }
>  +                        else if (doc.equalsIgnoreCase(
>  +                                TurbineConstants.DOCUMENT_TYPE_HTML40STRICT))
>  +                        {
>  +                            uri += "strict";
>  +                        }
>  +                        else if (doc.equalsIgnoreCase(
>  +                                TurbineConstants.DOCUMENT_TYPE_HTML40FRAMESET))
>  +                        {
>  +                            identifier = "Frameset";
>  +                            uri += "frameset";

Nah. I -1 this change. This is too much specific domain knowledge in this
class. If we really want to puzzle DTD descriptions, please add a helper
class which accepts a parameter and capsulates this code. This is bound
to rot in a few months.

        Regards
                Henning

-- 
Dipl.-Inf. (Univ.) Henning P. Schmiedehausen          INTERMETA GmbH
[EMAIL PROTECTED]        +49 9131 50 654 0   http://www.intermeta.de/

Java, perl, Solaris, Linux, xSP Consulting, Web Services 
freelance consultant -- Jakarta Turbine Development  -- hero for hire

"Dominate!! Dominate!! Eat your young and aggregate! I have grotty silicon!" 
      -- AOL CD when played backwards  (User Friendly - 200-10-15)

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to