Issue642483

classification
Title: Bug 628315: call non public method
Type: Severity: normal
Components: None Versions:
Milestone:
process
Status: closed Resolution: fixed
Dependencies: Superseder:
Assigned To: Nosy List: andreaskohn, cgroves, kzuberi, pedronis, ype
Priority: high Keywords: patch

Created on 2002-11-22.19:37:57 by ype, last changed 2007-01-10.20:15:40 by kzuberi.

Files
File name Uploaded Description Edit Remove
patch628315try2.txt ype, 2002-11-22.19:37:57 PyJavaClass.java diff -c against current
Messages
msg2276 (view) Author: Ype (ype) Date: 2002-11-22.19:37:57
In PyJavaClass.java:
Make sure that no more methods are called.

Factored setBeanInfoCustum(), setFields() and 
setMethods() into a new setAttributes() method.

Call setAttributes() for proxyClass (old behaviour) and for 
all interfaces (new).
setAttributes() does nothing when the given class/interface
is not public.


Doing setBeanInfoCustom() also on interfaces may be
a bit much, but why not? Currently there don't seem
to be bean like things in interfaces, but that might change.

Also removed several old lines with commented code.

I hope I got the indentation right.

msg2277 (view) Author: Samuele Pedroni (pedronis) Date: 2005-01-09.19:36:25
Logged In: YES 
user_id=61408

it seem a bit too much brute force, but yes this one of the
most annoying bugs wrt to java integration. It needs to be
reviewed in the context of the polishing of the
*Instance/*Class cluster.
msg2278 (view) Author: Charlie Groves (cgroves) Date: 2006-08-21.03:45:03
Logged In: YES 
user_id=1174327

This was fixed by making package protected methods
accessible so the patch isn't needed.
msg2279 (view) Author: Andreas Kohn (andreaskohn) Date: 2007-01-10.09:00:31
What revision exactly was this fixed in? I need to modify the local version of jython with this and ONLY this change, and cannot use another jython version?
msg2280 (view) Author: Khalid Zuberi (kzuberi) Date: 2007-01-10.20:15:40

I believe it was fixed in 2889:

 http://jython.svn.sourceforge.net/viewvc/jython?view=rev&revision=2889

If you dig around in the tracker you'll find some other but reports referring to the same issue.

- kz
History
Date User Action Args
2002-11-22 19:37:57ypecreate