Multiple lookups target problem
------------------------------- Key: OFBIZ-3845 URL: https://issues.apache.org/jira/browse/OFBIZ-3845 Project: OFBiz Issue Type: Bug Components: ALL COMPONENTS Affects Versions: jQuery Reporter: Shubham Dubey Fix For: jQuery Steps: (1) Go to any page having more than one lookup. (2) Click on any lookup and find the corrosponding information and click on tab to select the information. Any lookup we select, lookup for same lookupId is created, which is for the last lookup. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online. |
[ https://issues.apache.org/jira/browse/OFBIZ-3845?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12884675#action_12884675 ] Shubham Dubey commented on OFBIZ-3845: -------------------------------------- Seems a problem in the jquery code in htmlFormMacroLibrary.ftl , line no 520. instead of selecting the current div, the last script generated div is passed to the function. Thanks & Regards Shubham Dubey > Multiple lookups target problem > ------------------------------- > > Key: OFBIZ-3845 > URL: https://issues.apache.org/jira/browse/OFBIZ-3845 > Project: OFBiz > Issue Type: Bug > Components: ALL COMPONENTS > Affects Versions: jQuery > Reporter: Shubham Dubey > Fix For: jQuery > > > Steps: > (1) Go to any page having more than one lookup. > (2) Click on any lookup and find the corrosponding information and click on tab to select the information. > > Any lookup we select, lookup for same lookupId is created, which is for the last lookup. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online. |
In reply to this post by Nicolas Malin (Jira)
[ https://issues.apache.org/jira/browse/OFBIZ-3845?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Shubham Dubey updated OFBIZ-3845: --------------------------------- Issue Type: Sub-task (was: Bug) Parent: OFBIZ-3814 > Multiple lookups target problem > ------------------------------- > > Key: OFBIZ-3845 > URL: https://issues.apache.org/jira/browse/OFBIZ-3845 > Project: OFBiz > Issue Type: Sub-task > Components: ALL COMPONENTS > Affects Versions: jQuery > Reporter: Shubham Dubey > Fix For: jQuery > > > Steps: > (1) Go to any page having more than one lookup. > (2) Click on any lookup and find the corrosponding information and click on tab to select the information. > > Any lookup we select, lookup for same lookupId is created, which is for the last lookup. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online. |
In reply to this post by Nicolas Malin (Jira)
[ https://issues.apache.org/jira/browse/OFBIZ-3845?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12884678#action_12884678 ] Shubham Dubey commented on OFBIZ-3845: -------------------------------------- Sorry a typo, instead of div it's a lookupId which is passed. Thanks Shubham Dubey > Multiple lookups target problem > ------------------------------- > > Key: OFBIZ-3845 > URL: https://issues.apache.org/jira/browse/OFBIZ-3845 > Project: OFBiz > Issue Type: Sub-task > Components: ALL COMPONENTS > Affects Versions: jQuery > Reporter: Shubham Dubey > Fix For: jQuery > > > Steps: > (1) Go to any page having more than one lookup. > (2) Click on any lookup and find the corrosponding information and click on tab to select the information. > > Any lookup we select, lookup for same lookupId is created, which is for the last lookup. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online. |
In reply to this post by Nicolas Malin (Jira)
[ https://issues.apache.org/jira/browse/OFBIZ-3845?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12885116#action_12885116 ] Sascha Rodekamp commented on OFBIZ-3845: ---------------------------------------- Hi Shubham, ok thanks for testing. I think i know why the error occur and will fix it ASAP :) Have a good day Sascha > Multiple lookups target problem > ------------------------------- > > Key: OFBIZ-3845 > URL: https://issues.apache.org/jira/browse/OFBIZ-3845 > Project: OFBiz > Issue Type: Sub-task > Components: ALL COMPONENTS > Affects Versions: jQuery > Reporter: Shubham Dubey > Fix For: jQuery > > > Steps: > (1) Go to any page having more than one lookup. > (2) Click on any lookup and find the corrosponding information and click on tab to select the information. > > Any lookup we select, lookup for same lookupId is created, which is for the last lookup. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online. |
In reply to this post by Nicolas Malin (Jira)
[ https://issues.apache.org/jira/browse/OFBIZ-3845?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12885119#action_12885119 ] Shubham Dubey commented on OFBIZ-3845: -------------------------------------- Hi Sascha I tried to fix it earlier with the way it is implemented but was unable to do it the right way. I believe the lookup parameters are need to pass when the function is called, don't know why it is done this way >> jQuery("#" + lookupId + "_button").click( May be you can explain it better.. Thanks $ Regards Shubham Dubey > Multiple lookups target problem > ------------------------------- > > Key: OFBIZ-3845 > URL: https://issues.apache.org/jira/browse/OFBIZ-3845 > Project: OFBiz > Issue Type: Sub-task > Components: ALL COMPONENTS > Affects Versions: jQuery > Reporter: Shubham Dubey > Fix For: jQuery > > > Steps: > (1) Go to any page having more than one lookup. > (2) Click on any lookup and find the corrosponding information and click on tab to select the information. > > Any lookup we select, lookup for same lookupId is created, which is for the last lookup. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online. |
In reply to this post by Nicolas Malin (Jira)
[ https://issues.apache.org/jira/browse/OFBIZ-3845?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12885127#action_12885127 ] Sascha Rodekamp commented on OFBIZ-3845: ---------------------------------------- Hi Shubham, that's to open the lookup by clicking on the icon. First the lookup is initialized var dialogRef = jQuery("#" + lookupId).dialog(dialogOpts); with an attribute autoOpen: false The code jQuery("#" + lookupId + "_button").click( bind an event to the button which opens the lookup. You cn find an example here : [http://jqueryui.com/demos/dialog/#animated] BTW: I shortly tested on [https://localhost:8443/example/control/FormWidgetExamples] and i can't reproduce the issue. there are two lookups and both have a different ID. Can you give me a link, where you found the issue? > Multiple lookups target problem > ------------------------------- > > Key: OFBIZ-3845 > URL: https://issues.apache.org/jira/browse/OFBIZ-3845 > Project: OFBiz > Issue Type: Sub-task > Components: ALL COMPONENTS > Affects Versions: jQuery > Reporter: Shubham Dubey > Fix For: jQuery > > > Steps: > (1) Go to any page having more than one lookup. > (2) Click on any lookup and find the corrosponding information and click on tab to select the information. > > Any lookup we select, lookup for same lookupId is created, which is for the last lookup. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online. |
In reply to this post by Nicolas Malin (Jira)
[ https://issues.apache.org/jira/browse/OFBIZ-3845?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12885170#action_12885170 ] Shubham Dubey commented on OFBIZ-3845: -------------------------------------- Hi I am watching this issue on every page having more than one lookup, also on the link example/control/FormWidgetExamples. I can't reproduce it on live bcoz we don't have a demo trunk for jquery Branch. Do we ? Thanks $ Regards Shubham Dubey > Multiple lookups target problem > ------------------------------- > > Key: OFBIZ-3845 > URL: https://issues.apache.org/jira/browse/OFBIZ-3845 > Project: OFBiz > Issue Type: Sub-task > Components: ALL COMPONENTS > Affects Versions: jQuery > Reporter: Shubham Dubey > Fix For: jQuery > > > Steps: > (1) Go to any page having more than one lookup. > (2) Click on any lookup and find the corrosponding information and click on tab to select the information. > > Any lookup we select, lookup for same lookupId is created, which is for the last lookup. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online. |
In reply to this post by Nicolas Malin (Jira)
[ https://issues.apache.org/jira/browse/OFBIZ-3845?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12885180#action_12885180 ] Jacques Le Roux commented on OFBIZ-3845: ---------------------------------------- Hi Shubham, No we don't > Multiple lookups target problem > ------------------------------- > > Key: OFBIZ-3845 > URL: https://issues.apache.org/jira/browse/OFBIZ-3845 > Project: OFBiz > Issue Type: Sub-task > Components: ALL COMPONENTS > Affects Versions: jQuery > Reporter: Shubham Dubey > Fix For: jQuery > > > Steps: > (1) Go to any page having more than one lookup. > (2) Click on any lookup and find the corrosponding information and click on tab to select the information. > > Any lookup we select, lookup for same lookupId is created, which is for the last lookup. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online. |
In reply to this post by Nicolas Malin (Jira)
[ https://issues.apache.org/jira/browse/OFBIZ-3845?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12885880#action_12885880 ] Sascha Rodekamp commented on OFBIZ-3845: ---------------------------------------- Hi Shubham, i could reproduce the error. In my instance it's not the problems with the generated IDs (every lookup have a unique ID). The problem is the Request. I don't now why but the last Request which is used on the site for the last lookup will be used for every other lookup before. Freemarker renders the right Tag but the behavior of the .load method seems to be mysterious. I will have a look at this issue :-) > Multiple lookups target problem > ------------------------------- > > Key: OFBIZ-3845 > URL: https://issues.apache.org/jira/browse/OFBIZ-3845 > Project: OFBiz > Issue Type: Sub-task > Components: ALL COMPONENTS > Affects Versions: jQuery > Reporter: Shubham Dubey > Fix For: jQuery > > > Steps: > (1) Go to any page having more than one lookup. > (2) Click on any lookup and find the corrosponding information and click on tab to select the information. > > Any lookup we select, lookup for same lookupId is created, which is for the last lookup. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online. |
In reply to this post by Nicolas Malin (Jira)
[ https://issues.apache.org/jira/browse/OFBIZ-3845?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12885901#action_12885901 ] Shubham Dubey commented on OFBIZ-3845: -------------------------------------- Hi Sascha Yes there is a problem, and i did't said that lookUpId's are same for all the lookup's infact i mentioned <<Any lookup we select, lookup for same lookupId is created, which is for the last lookup.>> If it is confusing,then i would like to clarify it. Please see the associated lookupId on title bar of the lookup, that targets to the last one. Also i see a problem that sometimes lookup's are not displayed, that might be a instantaneous problem on my local instance,that i will look. Thanks for reviewing it, i will also try to have closer look at this mysterious behaviour :) > Multiple lookups target problem > ------------------------------- > > Key: OFBIZ-3845 > URL: https://issues.apache.org/jira/browse/OFBIZ-3845 > Project: OFBiz > Issue Type: Sub-task > Components: ALL COMPONENTS > Affects Versions: jQuery > Reporter: Shubham Dubey > Fix For: jQuery > > > Steps: > (1) Go to any page having more than one lookup. > (2) Click on any lookup and find the corrosponding information and click on tab to select the information. > > Any lookup we select, lookup for same lookupId is created, which is for the last lookup. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online. |
In reply to this post by Nicolas Malin (Jira)
[ https://issues.apache.org/jira/browse/OFBIZ-3845?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Sascha Rodekamp updated OFBIZ-3845: ----------------------------------- Attachment: OFBIZ-3845_MultiLookup.patch Hi, here is a fix for this lookup issue (just a blueprint for testing purposes). I know i have to prettyfy the code ... but it works now :) Cheers Sascha > Multiple lookups target problem > ------------------------------- > > Key: OFBIZ-3845 > URL: https://issues.apache.org/jira/browse/OFBIZ-3845 > Project: OFBiz > Issue Type: Sub-task > Components: ALL COMPONENTS > Affects Versions: jQuery > Reporter: Shubham Dubey > Fix For: jQuery > > Attachments: OFBIZ-3845_MultiLookup.patch > > > Steps: > (1) Go to any page having more than one lookup. > (2) Click on any lookup and find the corrosponding information and click on tab to select the information. > > Any lookup we select, lookup for same lookupId is created, which is for the last lookup. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online. |
In reply to this post by Nicolas Malin (Jira)
[ https://issues.apache.org/jira/browse/OFBIZ-3845?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Sascha Rodekamp updated OFBIZ-3845: ----------------------------------- Attachment: OFBIZ-3845_MultiLookup.patch > Multiple lookups target problem > ------------------------------- > > Key: OFBIZ-3845 > URL: https://issues.apache.org/jira/browse/OFBIZ-3845 > Project: OFBiz > Issue Type: Sub-task > Components: ALL COMPONENTS > Affects Versions: jQuery > Reporter: Shubham Dubey > Fix For: jQuery > > Attachments: OFBIZ-3845_MultiLookup.patch, OFBIZ-3845_MultiLookup.patch > > > Steps: > (1) Go to any page having more than one lookup. > (2) Click on any lookup and find the corrosponding information and click on tab to select the information. > > Any lookup we select, lookup for same lookupId is created, which is for the last lookup. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online. |
In reply to this post by Nicolas Malin (Jira)
[ https://issues.apache.org/jira/browse/OFBIZ-3845?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12888223#action_12888223 ] Shubham Dubey commented on OFBIZ-3845: -------------------------------------- Hi Sascha I applied the patch You provided and Yes it is working in a right manner. :) Thanks and Regards Shubham Dubey HotwaxMedia > Multiple lookups target problem > ------------------------------- > > Key: OFBIZ-3845 > URL: https://issues.apache.org/jira/browse/OFBIZ-3845 > Project: OFBiz > Issue Type: Sub-task > Components: ALL COMPONENTS > Affects Versions: jQuery > Reporter: Shubham Dubey > Fix For: jQuery > > Attachments: OFBIZ-3845_MultiLookup.patch, OFBIZ-3845_MultiLookup.patch > > > Steps: > (1) Go to any page having more than one lookup. > (2) Click on any lookup and find the corrosponding information and click on tab to select the information. > > Any lookup we select, lookup for same lookupId is created, which is for the last lookup. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online. |
In reply to this post by Nicolas Malin (Jira)
[ https://issues.apache.org/jira/browse/OFBIZ-3845?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Sascha Rodekamp updated OFBIZ-3845: ----------------------------------- Attachment: OFBIZ-3845_MultiLookup.patch here is the final patch for the lookup issue. All functionallity should work now and its much prettier than before. Only the implementation for all Themes is missing, i will provide this in a seperate patch. Cheers > Multiple lookups target problem > ------------------------------- > > Key: OFBIZ-3845 > URL: https://issues.apache.org/jira/browse/OFBIZ-3845 > Project: OFBiz > Issue Type: Sub-task > Components: ALL COMPONENTS > Affects Versions: jQuery > Reporter: Shubham Dubey > Fix For: jQuery > > Attachments: OFBIZ-3845_MultiLookup.patch, OFBIZ-3845_MultiLookup.patch, OFBIZ-3845_MultiLookup.patch > > > Steps: > (1) Go to any page having more than one lookup. > (2) Click on any lookup and find the corrosponding information and click on tab to select the information. > > Any lookup we select, lookup for same lookupId is created, which is for the last lookup. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online. |
In reply to this post by Nicolas Malin (Jira)
[ https://issues.apache.org/jira/browse/OFBIZ-3845?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Sascha Rodekamp updated OFBIZ-3845: ----------------------------------- Attachment: OFBIZ-3845_MultiLookup.patch > Multiple lookups target problem > ------------------------------- > > Key: OFBIZ-3845 > URL: https://issues.apache.org/jira/browse/OFBIZ-3845 > Project: OFBiz > Issue Type: Sub-task > Components: ALL COMPONENTS > Affects Versions: jQuery > Reporter: Shubham Dubey > Fix For: jQuery > > Attachments: OFBIZ-3845_MultiLookup.patch, OFBIZ-3845_MultiLookup.patch, OFBIZ-3845_MultiLookup.patch, OFBIZ-3845_MultiLookup.patch > > > Steps: > (1) Go to any page having more than one lookup. > (2) Click on any lookup and find the corrosponding information and click on tab to select the information. > > Any lookup we select, lookup for same lookupId is created, which is for the last lookup. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online. |
In reply to this post by Nicolas Malin (Jira)
[ https://issues.apache.org/jira/browse/OFBIZ-3845?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jacques Le Roux closed OFBIZ-3845. ---------------------------------- Assignee: Jacques Le Roux Resolution: Fixed Thanks Shubham and Sascha, Sascha your patch is in jQuery branch at r964717. BTW, It's all great to see good progress in this branch but should we not take care also on the remaining bugs in the trunk related to the similar area. I mean OFBIZ-3746 and OFBIZ-3846. Sorry to be such a pain ;) Thanks for al lthe good work! > Multiple lookups target problem > ------------------------------- > > Key: OFBIZ-3845 > URL: https://issues.apache.org/jira/browse/OFBIZ-3845 > Project: OFBiz > Issue Type: Sub-task > Components: ALL COMPONENTS > Affects Versions: jQuery > Reporter: Shubham Dubey > Assignee: Jacques Le Roux > Fix For: jQuery > > Attachments: OFBIZ-3845_MultiLookup.patch, OFBIZ-3845_MultiLookup.patch, OFBIZ-3845_MultiLookup.patch, OFBIZ-3845_MultiLookup.patch > > > Steps: > (1) Go to any page having more than one lookup. > (2) Click on any lookup and find the corrosponding information and click on tab to select the information. > > Any lookup we select, lookup for same lookupId is created, which is for the last lookup. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online. |
In reply to this post by Nicolas Malin (Jira)
[ https://issues.apache.org/jira/browse/OFBIZ-3845?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12889101#action_12889101 ] Sascha Rodekamp commented on OFBIZ-3845: ---------------------------------------- Jacque the painbringer :-D hehe no problem i'm aware of it. Next week i'll check all the related issues and try to implement a fix. Thanks for you're comitt. Have a good day Sascha > Multiple lookups target problem > ------------------------------- > > Key: OFBIZ-3845 > URL: https://issues.apache.org/jira/browse/OFBIZ-3845 > Project: OFBiz > Issue Type: Sub-task > Components: ALL COMPONENTS > Affects Versions: jQuery > Reporter: Shubham Dubey > Assignee: Jacques Le Roux > Fix For: jQuery > > Attachments: OFBIZ-3845_MultiLookup.patch, OFBIZ-3845_MultiLookup.patch, OFBIZ-3845_MultiLookup.patch, OFBIZ-3845_MultiLookup.patch > > > Steps: > (1) Go to any page having more than one lookup. > (2) Click on any lookup and find the corrosponding information and click on tab to select the information. > > Any lookup we select, lookup for same lookupId is created, which is for the last lookup. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online. |
In reply to this post by Nicolas Malin (Jira)
[ https://issues.apache.org/jira/browse/OFBIZ-3845?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Sascha Rodekamp updated OFBIZ-3845: ----------------------------------- Attachment: OFBIZ-3845_lookupFix.patch Hi Jacques, here is a little patch which should solve a problem with the "Variants" Links. I also checked the issues OFBIZ-3746 and OFBIZ-3846 and can't see a error anymore. But the Variants Screen for product WG-9944 is still empty (but not weird) :-) Have a good day Sascha > Multiple lookups target problem > ------------------------------- > > Key: OFBIZ-3845 > URL: https://issues.apache.org/jira/browse/OFBIZ-3845 > Project: OFBiz > Issue Type: Sub-task > Components: ALL COMPONENTS > Affects Versions: jQuery > Reporter: Shubham Dubey > Assignee: Jacques Le Roux > Fix For: jQuery > > Attachments: OFBIZ-3845_lookupFix.patch, OFBIZ-3845_MultiLookup.patch, OFBIZ-3845_MultiLookup.patch, OFBIZ-3845_MultiLookup.patch, OFBIZ-3845_MultiLookup.patch > > > Steps: > (1) Go to any page having more than one lookup. > (2) Click on any lookup and find the corrosponding information and click on tab to select the information. > > Any lookup we select, lookup for same lookupId is created, which is for the last lookup. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online. |
In reply to this post by Nicolas Malin (Jira)
[ https://issues.apache.org/jira/browse/OFBIZ-3845?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12891043#action_12891043 ] Jacques Le Roux commented on OFBIZ-3845: ---------------------------------------- Hi Sascha, If I understand this well this patch is for the trunk and should be rather attached to OFBIZ-3746. For the variant screen, try to open WG-9943 in the trunk demo and you will see that it takes all the screen and you get stuck then. For the product WG-9944 I will have to investigate more, right! For you, to better understand the problem, I have attached a screenshot at OFBIZ-3846. Thanks for all your valuable help! > Multiple lookups target problem > ------------------------------- > > Key: OFBIZ-3845 > URL: https://issues.apache.org/jira/browse/OFBIZ-3845 > Project: OFBiz > Issue Type: Sub-task > Components: ALL COMPONENTS > Affects Versions: jQuery > Reporter: Shubham Dubey > Assignee: Jacques Le Roux > Fix For: jQuery > > Attachments: OFBIZ-3845_lookupFix.patch, OFBIZ-3845_MultiLookup.patch, OFBIZ-3845_MultiLookup.patch, OFBIZ-3845_MultiLookup.patch, OFBIZ-3845_MultiLookup.patch > > > Steps: > (1) Go to any page having more than one lookup. > (2) Click on any lookup and find the corrosponding information and click on tab to select the information. > > Any lookup we select, lookup for same lookupId is created, which is for the last lookup. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online. |
Free forum by Nabble | Edit this page |