[ 
https://issues.apache.org/jira/browse/SANDBOX-379?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Benedikt Ritter updated SANDBOX-379:
------------------------------------

    Attachment: SANDBOX-379v2.txt

Patch modified. Summary:
* renamed {{BeanAccessorTestCase}} to {{DescribeTestCase}} and moved test 
methods from {{Jira157TestCase}} there
* Added static reference to {{MethodsRegistry}} in 
{{PropertyDescriptorRegistry}}
* Changed signature of {{PropertyDescrioptorsRegistry.makeMethodsAccessible()}} 
to {{void}}

Please have a close look at line 144 in {{PropertyDescriprotrsRegistry}}:

{code:java}
readMethod = METHODS_REGISTRY.get( true, beanType, readMethod.getName() );
{code}

I'm afraid that this will get us into trouble when we implement the methods for 
indexed and mapped properties (because I'm not passing any arguments).

Thank you for the inspiring discussion on this issue!
                
> [BeanUtils2] Implement describe() on DefaultBeanAccessor 
> ---------------------------------------------------------
>
>                 Key: SANDBOX-379
>                 URL: https://issues.apache.org/jira/browse/SANDBOX-379
>             Project: Commons Sandbox
>          Issue Type: Improvement
>          Components: BeanUtils2
>    Affects Versions: Nightly Builds
>            Reporter: Benedikt Ritter
>         Attachments: SANDBOX-379.txt, SANDBOX-379v2.txt
>
>
> Implement the above mentioned method an corresponding unit tests

--
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

        

Reply via email to