Re: svn commit: r993508 - /ofbiz/trunk/framework/images/webapp/images/selectall.js

[Jacques Le Roux]

Of course when I began to work on this I tried to use description-field-name 
but was unable to access it and discovered it was
useless in the autocomplete context. BTW using jQuery for this should be far 
easier as the
http://bassistance.de/jquery-plugins/jquery-plugin-autocomplete/ is now part of 
jQuery.ui...

Jacques

From: <jler...@apache.org>

Author: jleroux
Date: Tue Sep  7 19:55:49 2010
New Revision: 993508

URL: http://svn.apache.org/viewvc?rev=993508&view=rev
Log:
Introduces a very simple mean to set a dependent Id field when using a Name 
field as a lookup (the fields must have the same
prefix, eg: partyName, partyId).
It uses the description (Id) shown with the Name. Hence the Lookup screen must 
be set in order to show a description in the
autocomplete part.
It seems this is not always easy notably when you need to show at least 2 parts 
for the Name (eg Person).
At least it easy to set and it works well for simples case for now (eg 
PartyGroup)

I did not find another way since Ajax.Autocompleter can't update another field.
The alternative would be navigation to the next hidden field, at least it would 
avoid the mandatory Id/Name pair.
But it's more difficult to demonstrate in Example component

Modified:
   ofbiz/trunk/framework/images/webapp/images/selectall.js

Modified: ofbiz/trunk/framework/images/webapp/images/selectall.js
URL: 
http://svn.apache.org/viewvc/ofbiz/trunk/framework/images/webapp/images/selectall.js?rev=993508&r1=993507&r2=993508&view=diff
==============================================================================
--- ofbiz/trunk/framework/images/webapp/images/selectall.js (original)
+++ ofbiz/trunk/framework/images/webapp/images/selectall.js Tue Sep  7 19:55:49 
2010
@@ -232,6 +232,7 @@ function confirmActionFormLink(msg, form
  * @param target The URL to call to update the HTML container
  * @param targetParams The URL parameters
*/
+
function ajaxUpdateArea(areaId, target, targetParams) {
    waitSpinnerShow();
    new Ajax.Updater(areaId, target, {parameters: targetParams, evalScripts: 
true,
@@ -377,7 +378,23 @@ function setLookDescription(textFieldId,
    if (description) {
        var start = description.lastIndexOf(' [');
        if (start != -1) {
+            // To allow to set a dependent Id field when using a Name field as 
a lookup (the fields must have the same prefix,
eg: partyName, partyId)
+            // It uses the description (Id) shown with the Name. Hence the 
Lookup screen must be set in order to show a
description in the autocomplete part.
+            // It seems this is not always easy notably when you need to show 
at least 2 parts for the Name (eg Person).
+            // At least it easy to set and it works well for simples case for 
now (eg PartyGroup)
+            var dependentId = textFieldId.replace(/Name/, "Id"); // Raw but ok 
for now, needs safe navigation...
+            // I did not find another way since Ajax.Autocompleter can't 
update another field
+            // The alternative would be navigation to the next hidden field, 
at least it would avoid the mandatory Id/Name pair
+            // But it's more difficult to demonstrate in Example component
+            // dependentId = 
(textFieldId.next('div').down('input[type=hidden]');
+            $(dependentId).clear();
+            var dependentIdValue = (description.substring(start + 1, 
description.length).replace(/\[/g, "")).replace(/\]/g, "");
+            if ($(dependentId)) {
+                $(dependentId).value = dependentIdValue;
+            }
+
            description = description.substring(0, start);
+            $(dependentId).value = description;
        }
    }
    var lookupWrapperEl = $(textFieldId).up('.field-lookup');