(after a bit of consideration) i've now committed this patch. this isn't really a bit of beanutils i've been heavily involved with, so if anyone has strong opinions that it this is wrong, i won't be upset if they want to fix it in some other way.
- robert On Sunday, May 26, 2002, at 06:17 PM, Tomas Viberg wrote: > Comments, anyone? > > Regards > / Tomas > > > ----- Original Message ----- > From: Tomas Viberg > To: [EMAIL PROTECTED] > Sent: Sunday, May 19, 2002 7:29 PM > Subject: [PATCH] BeanUtils - read only properties and null value > conversion > > > Hello, > > I humbly submit a patch for BeanUtils.java, correcting the following > errors: > > 1. When you try to populate a read-only property, the populate method > throws a NoSuchMethodException (wrapped in an InvocationTargetException), > since there is no check that the property has a setter method. > > 2. When you try to populate a primitive property with a null value, you > get a NullPointerException, since no data type conversion is applied to > null values. > > I have also patched BeanUtilsTestCase.java to include setting a primitive > property to null and attempting to set a read-only (simple) property. > > Regards, > Tomas Viberg, [EMAIL PROTECTED] > > > > -- > To unsubscribe, e-mail: <mailto:[EMAIL PROTECTED]. > org> > For additional commands, e-mail: <mailto:[EMAIL PROTECTED]. > org> > -- > To unsubscribe, e-mail: <mailto:[EMAIL PROTECTED]. > org> > For additional commands, e-mail: <mailto:[EMAIL PROTECTED]. > org> -- To unsubscribe, e-mail: <mailto:[EMAIL PROTECTED]> For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>
