[ 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