Re: Bug in BeanUtils.populate()

2002-01-14 Thread Arron
the change to the code that I suggested below. If you look at the code, the bug is very obivious. -Original Message- From: Arron Bates Sent: Sun 1/13/2002 11:31 PM To: Struts Users Mailing List Cc: Subject: Re: Bug in BeanUtils.populate

Re: Bug in BeanUtils.populate()

2002-01-14 Thread Don Saxton
Builds as of a few days ago will accept implementations of java.util.List as well as the primitive arrays the spec defines. so does List work as well with your nested tags as Object[] ? -- To unsubscribe, e-mail: mailto:[EMAIL PROTECTED] For additional commands, e-mail: mailto:[EMAIL

RE: Bug in BeanUtils.populate()

2002-01-14 Thread Jason Chaffee
To: Struts Users Mailing List Subject: Re: Bug in BeanUtils.populate() The method definitions were literally cut-and-paste from the spec. There is no other code in the spec relating to indexed properties. Arron. Jason Chaffee wrote: The spec. supports the following: setFoo(int index, Object

Re: Bug in BeanUtils.populate()

2002-01-14 Thread Arron Bates
That's what made me look into it. :) Was getting feedback that the iterate:tag will stake anything that remotely smells like a collection, and I've always said that the nested tags sit on the other ones, so why didn't it work?... difference in how the properties are fetched, as they're all

Re: Bug in BeanUtils.populate()

2002-01-14 Thread Arron Bates
. I think the ones you're after are the array methods. Arron. It appears there is a bug in BeanUtils.populate() for an indexed setter of array type. It doesn't take into account that it is an indexed setter and that the second parameter is an array because

RE: Bug in BeanUtils.populate()

2002-01-14 Thread Jason Chaffee
Mailing List Subject: Re: Bug in BeanUtils.populate() Here is the spec... http://java.sun.com/products/javabeans/docs/beans.101.pdf ...where in it does it have... void setFoo(int index, Object[] array) ...? Indexed methods description starts on page 40, and go over to 41. Arron. Jason

RE: Bug in BeanUtils.populate()

2002-01-14 Thread Jason Chaffee
[mailto:[EMAIL PROTECTED]] Sent: Monday, January 14, 2002 3:03 PM To: Struts Users Mailing List Subject: Re: Bug in BeanUtils.populate() I get what you're trying to do now. Sorry about all that. If you can test your change and make a CVS diff of the changes, why don't you log a bugzilla bug in commons

Re: Bug in BeanUtils.populate()

2002-01-14 Thread Arron Bates
tested my change and it works fine. -Original Message- From: Arron Bates [mailto:[EMAIL PROTECTED]] Sent: Monday, January 14, 2002 3:03 PM To: Struts Users Mailing List Subject: Re: Bug in BeanUtils.populate() I get what you're trying to do now. Sorry about all that. If you can test your

Bug in BeanUtils.populate()

2002-01-13 Thread Jason Chaffee
It appears there is a bug in BeanUtils.populate() for an indexed setter of array type. It doesn't take into account that it is an indexed setter and that the second parameter is an array because it only checks the first parameter which is always an int for an indexed setter

Re: Bug in BeanUtils.populate()

2002-01-13 Thread Arron Bates
separate code blocks to handle both. Builds as of a few days ago will accept implementations of java.util.List as well as the primitive arrays the spec defines. I think the ones you're after are the array methods. Arron. It appears there is a bug in BeanUtils.populate() for an indexed setter

RE: Bug in BeanUtils.populate()

2002-01-13 Thread Jason Chaffee
: Arron Bates Sent: Sun 1/13/2002 11:31 PM To: Struts Users Mailing List Cc: Subject: Re: Bug in BeanUtils.populate() BeanUtils works correctly in that if you want to set against an index, you can have the following forms