DO NOT REPLY [Bug 16736] New: - html:form Focus-Script should check for disabled input field

[bu...@apache.org]

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=16736>.
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=16736

html:form Focus-Script should check for disabled input field

           Summary: html:form Focus-Script should check for disabled input
                    field
           Product: Struts
           Version: Nightly Build
          Platform: All
        OS/Version: All
            Status: NEW
          Severity: Enhancement
          Priority: Other
         Component: Custom Tags
        AssignedTo: struts-dev@jakarta.apache.org
        ReportedBy: blochg@fmi.uni-passau.de


The <html:form>-Tag generates a script which sets focus to the input field 
specified by focus-attribute.

This script checks if the input is hidden and in that case doesn't set the 
focus (because doing so would result in an error reported by the browser).

This behaviour could easily be enhanced to check also if the input field is 
disabled. This also causes an error in the browser (tested with IE 6 SP1).

I have appended a patch that will implement the enhancement:

Index: src/share/org/apache/struts/taglib/html/FormTag.java
===================================================================
RCS file: /home/cvspublic/jakarta-
struts/src/share/org/apache/struts/taglib/html/FormTag.java,v
retrieving revision 1.43
diff -r1.43 FormTag.java
647c647
<             results.append("  if (focusControl.type != \"hidden\") {\r\n");
---
>             results.append("  if (focusControl.type != \"hidden\" && !
focusControl.disabled) {\r\n");

---------------------------------------------------------------------
To unsubscribe, e-mail: struts-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: struts-dev-help@jakarta.apache.org

[PATCH] bug #16603

[Malik Recoing <m....@almerys.com>]

Hi,

Here is my proposal of a patch for the bug report #16603.

I changed the RequestProcessor and the RequestUtils so that I can endly set
a multi-modules app with pages under WEB-INF.

But nobody seems to care about... I think I'll be glad if somebody write me
a word even to say : "Stop boring us with your stupid bug which is not, and
RTFM".

Malik.