commons-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Benedikt Ritter (Updated) (JIRA)" <>
Subject [jira] [Updated] (SANDBOX-379) [BeanUtils2] Implement describe() on DefaultBeanAccessor
Date Fri, 03 Feb 2012 10:33:58 GMT


Benedikt Ritter updated SANDBOX-379:

    Attachment: SANDBOX-379v2.txt

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

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

readMethod = METHODS_REGISTRY.get( true, beanType, readMethod.getName() );

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:
>             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:!default.jspa
For more information on JIRA, see:


View raw message