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

http://issues.apache.org/bugzilla/show_bug.cgi?id=34660

           Summary: [Shale] Clay was not setting properties correctly
           Product: Struts
           Version: Nightly Build
          Platform: Other
        OS/Version: other
            Status: NEW
          Severity: enhancement
          Priority: P2
         Component: Shale
        AssignedTo: dev@struts.apache.org
        ReportedBy: [EMAIL PROTECTED]
                CC: [EMAIL PROTECTED]


The PropertyUtil class was not working for all properties.  This utility class 
is used by the clay component to set property values on faces components, 
converters, validators and listeners.  The utility has to take the string 
value and figure out what the target type should be.  I had pulled this from 
another project and it was just not working right with the clay component so 
I've refactored, removing unnecessary logic.  

The logic now makes some guesses on setter methods and the formal parameter 
type of the setter method.
  
For example the following properties will all result in setting the boolean:
  globalOnly
  isGlobalOnly
  escape
  isEscape
 
Two class are modified both blocking 34496 for each source file.

PropUtils.java
PropertyValueCommand.java


I had forgot this class was such a mess.  It is a very important part in 
making clay work.  Any ideas on a better approach.....

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

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

Reply via email to