[OFBiz] Dev - A bug in lookup popup window?

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

[OFBiz] Dev - A bug in lookup popup window?

Bugzilla from elliot.li@gmail.com
Hello,

I've been making a lookup window that contains a tree. Although
successfully made it, I thought there is a problem.

In fieldlookup.js (code is below), who take charge of the poping up of
a lookup window, there is a lookups, of which I thought is
unnecessary, since on my Firefox there could be only one popup window
available at a time, and there's no need to track all the popup windows.

function fieldLookup1(obj_target) {
        // passing methods
        this.popup = lookup_popup1;
        this.popup2 = lookup_popup2;

        // validate input parameters
        if (!obj_target)
                return lookup_error("Error calling the field lookup: no target
control specified");
        if (obj_target.value == null)
                return cal_error("Error calling the field lookup: parameter
specified is not valid target control");
        this.target = obj_target;
       
        // register in global collections
        this.id = lookups.length;
        lookups[this.id] = this;
}

That "lookups" make a trouble on my window. At first, I don't know
there is an "id" in requestParameter, which holding the sequenceId of
"lookups", so after the user clicked the expand-collapse button and
the popup window move to another location, the "id" was lost, which
resulted that the value would always be set into the first
control(edit box) in the opener window, without any error message.

In a word, I think that "lookups" and related process codes just make
troubles, and have no use. Am I right?

Thanks for your help.

PS: I'm using Firefox, not tried on IE yet.

--
Best regards,
Yan
 
_______________________________________________
Dev mailing list
[hidden email]
http://lists.ofbiz.org/mailman/listinfo/dev