Re: [BeanUtils] Image Button Bug (was: Reflection exceptions)

[robert burrell donkin]

spoke too early. the logic (in struts) that i thought was wrong is right 8-)

so it's good luck that craig turned up when he did.

- robert

On Wednesday, November 20, 2002, at 07:11 PM, robert burrell donkin wrote:

you can probably stop digging for a while since i think i know what the problem is.

i think i've spotted a type in the struts code that calls beanutils. you'
ll have to bear with me while i subscribe myself on stucts-dev and try to find some who isn't in LasVagas...

- robert

On Wednesday, November 20, 2002, at 07:07 PM, James Higginbotham wrote:

Thanks for the assist and pointing out a goofy bug.. I noticed Craig
wasn't active in the last 24 hours at least, so figured that was the
case.

Some more info:

1. I added a block to check to see if a nested field was "y" or "x" to
skip image tags.. That got me further
2. Then I started seeing that same IllegalAccessException on non-nested
field names..
3. I then removed the doPriviledged() call inside the
MappedPropertyDescriptor class and all is well - no exception, code
works now..
4. I then removed the check for the "y" and "x" and all is still
working, so it wasn't the image button at all - just the security
check..

More context info:

I'm running on Windows XP with JDK 1.3.0 with Jboss 3.0.4 and Jetty
4.1.3. I have no server.policy configured (i.e. the server is wide open
for now). I tried a policy, but same problem persisted. I saw a post
about this on the commons mail archive regarding reflection and anon
inner classes. It referenced a bug in the VM. As such, I'm not sure why
this is showing up for me but apparently others are using the Struts
nightly and not seeing any problems.

So, I'll run with this version of beanutils for now and wait for Craig
and/or others to provide more insight into why this is happening and
what the overall fix should be. On to fixing a javascript validator NPE
I was seeing (server side works) and all should be well.

Thanks again,
James

-----Original Message-----
From: robert burrell donkin
[mailto:[EMAIL PROTECTED]]
Sent: Wednesday, November 20, 2002 12:46 PM
To: Jakarta Commons Users List
Subject: Re: [BeanUtils] Image Button Bug (was: Reflection exceptions)


On Wednesday, November 20, 2002, at 04:25 PM, James
Higginbotham wrote:

So, having narrowed this down more since my last post, I've

found that

the BeanUtils.setProperty() method is doing a check to see

if there is

a nested property that needs to be set. Since we are using image
buttons, the name of the form element becomes GoToStep2.y, which
confuses the util and makes it try a mapped property approach. I
noticed an archive posting from 7/2000 that Craig mentions that it
could be a bug and a fix was going to be in the works (?):

craig and scott are at ApacheCon so you'll probably have to
make do with
me :)

i'm not a structs expert but i'll do my best.

<snip>

looking through your class there was only one thing that was
obviously
wrong:

    public String getUuidx(){
    	return uuidx;
    }
    public void setUuidx(){
    	this.uuidx = uuidx;
    }

this setter is obviously buggy,

i don't know whether this is the cause of the problem but i
thought that i
might was well point it out.

anyway, i'll carry on with my digging.

- robert


--
To unsubscribe, e-mail:
<mailto:commons-user-> [EMAIL PROTECTED]>
For
additional commands,
e-mail: <mailto:[EMAIL PROTECTED]>

--
To unsubscribe, e-mail:   <mailto:commons-user-
[EMAIL PROTECTED]
org>
For additional commands, e-mail: <mailto:commons-user-
[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]>