350580 - [PATCH] importPackage Ambiguous import error fix

Status: RESOLVED FIXED

(Rhino Graveyard :: Core, defect)

Description

[Leszek Gawron]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8.0.6) Gecko/20060728 Firefox/1.5.0.6
Build Identifier: Rhino 1.6R2 (R3 also)

Originally reported at: 
http://groups.google.com/group/netscape.public.mozilla.jseng/browse_frm/thread/9551fca1f6200746/4c8cf0604bcd79ca?lnk=gst&q=ambiguous+import&rnum=1#4c8cf0604bcd79ca

quote:

Attached is a patch to fix importPackage to not import the
same package more than once.  This fixes an error that
shows up if a call to importPackage is encountered twice
before classes from the package actually get referenced.

I encountered this error while working with a flowscript
(Cocoon's name for javascript+continuations) for a set of
Cocoon forms.  Quickly starting more than one instance of
a form triggers this bug.

The patch replaces a comparison using '=' between two
NativeJavaPackage's with a comparison using string equality.
This effectively compares the package names instead of
checking for object identity.

The other thing which we might need to check is that the
two NativeJavaPackage's both use the same classloader.
I would appreciate it if somebody more knowledgeable of
Rhino internals could sanity check this patch. 


Reproducible: Couldn't Reproduce

Comment 1

[Leszek Gawron]

Attachment: patch file

this is the original patch posted by Tim Larson on 2005-03-22 to jseng news list. It is truly a one liner.

Comment 2

[Attila Szegedi]

Attachment: patch using equals() on NativeJavaPackage

Actually, this is a bit problematic, as you can have a border case where the two NativeJavaPackage objects are for the same package but loaded through different class loaders. The real solution is to implement equals() and hashCode() on the NativeJavaPackage so that it compares both the name and the class loader, see attached patch -- comments welcome.

Comment 3

[Attila Szegedi]

Committed patch to CVS HEAD

Comment 4

[Norris Boyd]

Adding target milestone of 1.6R6 based on the date this bug was resolved FIXED.