DO NOT REPLY [Bug 12825] New: - Action "roles" attribute incorrectly parsed for comma-seperated roles

[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=12825>.
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=12825

Action "roles" attribute incorrectly parsed for comma-seperated roles

           Summary: Action "roles" attribute incorrectly parsed for comma-
                    seperated roles
           Product: Struts
           Version: 1.1 Beta 2
          Platform: PC
        OS/Version: Windows NT/2K
            Status: NEW
          Severity: Normal
          Priority: Other
         Component: Controller
        AssignedTo: struts-dev@jakarta.apache.org
        ReportedBy: howzr@lavconsulting.com


In the "action" node of the strugs-config, if you use a comma-seperated list of 
roles for the new "roles" attribute, the first letter of each role (except the 
last role) is dropped.  The bug is in:

org.apache.struts.config.ActionConfig

method: setRoles(String roles)

On line 328 of the source, it reads:

list.add(roles.substring(1, comma).trim());

The "1" needs to be a "0"

And why not use StringTokenizer instead? (not that it matters much)

--
To unsubscribe, e-mail:   <ma...@jakarta.apache.org>
For additional commands, e-mail: <ma...@jakarta.apache.org>