You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@commons.apache.org by "Adrian Cumiskey (Commented) (JIRA)" <ji...@apache.org> on 2011/11/04 03:35:33 UTC

[jira] [Commented] (OGNL-35) [PATCH] Improvements to {List,Map,Array}PropertyAccessor and ASTProperty

    [ https://issues.apache.org/jira/browse/OGNL-35?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13143700#comment-13143700 ] 

Adrian Cumiskey commented on OGNL-35:
-------------------------------------

patch-OGNL35-v2.txt improves upon and replaces the original one.
                
> [PATCH] Improvements to {List,Map,Array}PropertyAccessor and ASTProperty
> ------------------------------------------------------------------------
>
>                 Key: OGNL-35
>                 URL: https://issues.apache.org/jira/browse/OGNL-35
>             Project: OGNL
>          Issue Type: Improvement
>            Reporter: Adrian Cumiskey
>            Priority: Minor
>         Attachments: patch-OGNL35-v2.txt, patch-OGNL35.txt
>
>
> * Flipped equals() everywhere where a variable is tested against a String constant.
> * ListPropertyAccessor: CPD fix, extracted shared method getSourceBeanMethod() from getSourceSetter() and getSourceAccessor().
> * ASTProperty: CPD fix, extracted shared method getSourceString() from toGetSourceString() and toSetSourceString().
> * ArrayPropertyAccessor: CPD fix, extracted getIndexString() from getSourceAccessor() and getSourceSetter().

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira