DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG 
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://nagoya.apache.org/bugzilla/show_bug.cgi?id=15703>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND 
INSERTED IN THE BUG DATABASE.

http://nagoya.apache.org/bugzilla/show_bug.cgi?id=15703

Illegal scope attribute without var in "fmt:setLocale" tag





------- Additional Comments From [EMAIL PROTECTED]  2003-12-15 18:55 -------
Pierre/Jan,

The problem is not the validate() method itself, but the halpers (isFmtTag(),
isCoreTag(), etc...) used while parsing the PageData (right now, validate() is
defined in a super class - JstlBaseTLV - but the parsing code is spread throught
the sub-classes).

I think the easiest solution would be to redefine validate() in each sub-class,
setting the proper mapping between the TLV's taglib and its URI. Something like
this:

On JstlBaseTLV
--------------

private String fmtUri = null;

protected void setFmtUri( String uri ) { this.fmtUri = uri; }

protected boolean isFmtTag(String tagUri, String tagLn, String target) {
        return ( this.fmtUri != null &&
                 isTag(tagUri, tagLn, FMT, target)
              || isTag(tagUri, tagLn, FMT_EL, target)
              || isTag(tagUri, tagLn, FMT_RT, target));
    }


On JstlFmtTLV
-------------

    public synchronized ValidationMessage[] validate(
            String prefix, String uri, PageData page) {
         setFmtUri( uri );
         super.validate( prefix, uri, page );

    }

What do you think? If you're ok with that, I can do these changes on JSTL's code.

Felipe

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

Reply via email to