commons-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Benedikt Ritter (Issue Comment Edited) (JIRA)" <j...@apache.org>
Subject [jira] [Issue Comment Edited] (SANDBOX-397) [BeanUtils2] Replace NullPointerExceptions been thrown in DefaultBeanAccessor with NoSuchMethodEceptions
Date Sun, 26 Feb 2012 22:00:48 GMT

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

Benedikt Ritter edited comment on SANDBOX-397 at 2/26/12 9:59 PM:
------------------------------------------------------------------

I've thought about the {{Properties}} class some more and I think, that it is the right thing
to do. So I've created a patch for SANDBOX-399 that will fix the problems that {{Properties}}
caused. I've done some more clean up and I was able to eliminate the dependency between {{DefaultBeanAccessor}}
and {{PropertiesRegistry}}. {{PropertiesRegistry}} is on a lower abstraction level than {{DefautlBeanAccessor}}
so I think that is a good thing :)

If SANDBOX-399 is applied first, all test will pass when SANDBOX-397-SRPv2.txt is applied.

Regards,
Benedikt
                
      was (Author: britter):
    I've thought about the {{Properties}} class some more and I think, that it is the right
thing to do. So I've created a patch for SANDBOX-399 that will fix the problems that {{Properties}}
caused. I've done some more clean up and I was able to eliminate the dependency between {{DefaultBeanAccessor}}
and {{PropertiesRegistry}}. {{PropertiesRegistry}} is on a lower abstraction level than {{DefautlBeanAccessor}}
so I think that is a good thing :)

If SANDBOX-399 is applied first, all test will pass when SANDBOX-398-SRPv2.txt is applied.

Regards,
Benedikt
                  
> [BeanUtils2] Replace NullPointerExceptions been thrown in DefaultBeanAccessor with NoSuchMethodEceptions
> --------------------------------------------------------------------------------------------------------
>
>                 Key: SANDBOX-397
>                 URL: https://issues.apache.org/jira/browse/SANDBOX-397
>             Project: Commons Sandbox
>          Issue Type: Task
>          Components: BeanUtils2
>    Affects Versions: Nightly Builds
>            Reporter: Benedikt Ritter
>         Attachments: SANDBOX-397.txt, SANDBOX-397_SRP.txt, SANDBOX-397_SRPv2.txt
>
>
> At the moment, methods in {{DefaultBeanAccessor}} throw a {{NullPointerException}}, if
no {{PropertyDescriptor}} for a given property name can be retrieved. As discussed on the
ML (see http://markmail.org/thread/zlehclmybp5xgn5n) this behavior should be changed to throwing
{{NoSuchMethodException}}.

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

        

Mime
View raw message