bugs? lookup descriptionFieldName

[stamilo]

hi,
   i use lookup field to set two field from popup,

  when i set the description-field-name field ,

  it can't work, i change the code, see blow,

Index: HtmlFormRenderer.java
===================================================================
--- HtmlFormRenderer.java    (revision 673262)
+++ HtmlFormRenderer.java    (working copy)
@@ -2003,7 +2003,7 @@

writer.append(modelFormField.getModelForm().getCurrentFormName(context));
             writer.append('.');
             writer.append(modelFormField.getParameterName(context));
-            writer.append(",'");
+            writer.append(",");
             writer.append(descriptionFieldName);
             writer.append(",'");
         } else {


  it can work now ,


  is it a bug ?

 :)

[Jacques Le Roux]

This is really not clear, could you be more specific ? For instance, version you are using, what you intend to do (from a
functionnal POV), steps to reproduce, etc.

Jacques

From: "stamilo" <[hidden email]>

> hi,
>   i use lookup field to set two field from popup,
>
>  when i set the description-field-name field ,
>
>  it can't work, i change the code, see blow,
>
> Index: HtmlFormRenderer.java
> ===================================================================
> --- HtmlFormRenderer.java    (revision 673262)
> +++ HtmlFormRenderer.java    (working copy)
> @@ -2003,7 +2003,7 @@
>
> writer.append(modelFormField.getModelForm().getCurrentFormName(context));
>             writer.append('.');
>             writer.append(modelFormField.getParameterName(context));
> -            writer.append(",'");
> +            writer.append(",");
>             writer.append(descriptionFieldName);
>             writer.append(",'");
>         } else {
>
>
>  it can work now ,
>
>
>  is it a bug ?
>
> :)
>

[stamilo]

:0
i use ofbiz trunk, svn revision number is 673262;
i create a test applicate , want to use lookup field,
when set one field , that can run, all right.

when i want to set two field , i set description-field-name field,
but can't work, the popup windows can't show,
there is a error in java script , so i edit the source code,
delete a char "'",  :) , now it can run

On Sun, Jul 6, 2008 at 5:12 AM, Jacques Le Roux <
[hidden email]> wrote:

> This is really not clear, could you be more specific ? For instance,
> version you are using, what you intend to do (from a functionnal POV), steps
> to reproduce, etc.
>
> Jacques
>
> From: "stamilo" <[hidden email]>
>
>  hi,
>>  i use lookup field to set two field from popup,
>>
>>  when i set the description-field-name field ,
>>
>>  it can't work, i change the code, see blow,
>>
>> Index: HtmlFormRenderer.java
>> ===================================================================
>> --- HtmlFormRenderer.java    (revision 673262)
>> +++ HtmlFormRenderer.java    (working copy)
>> @@ -2003,7 +2003,7 @@
>>
>> writer.append(modelFormField.getModelForm().getCurrentFormName(context));
>>            writer.append('.');
>>            writer.append(modelFormField.getParameterName(context));
>> -            writer.append(",'");
>> +            writer.append(",");
>>            writer.append(descriptionFieldName);
>>            writer.append(",'");
>>        } else {
>>
>>
>>  it can work now ,
>>
>>
>>  is it a bug ?
>>
>> :)
>>
>>
>

[stamilo]

i can't find the usage for lookup field set 2 field ,
this my code:

main form:
<field name="fSyclmc"    title="${uiLabelMap.fSyclmc}">
   <lookup target-form-name="lookupsyclmc" description-field-name =
"document.formname.fCldesc"></lookup>

lookup form:
        <field name="fKey" title="ID" widget-style="buttontext">
            <hyperlink also-hidden="false" target-type="plain"
description="${fKey}"
target="javascript:set_values('${fType}','${fCldesc}')"/>
        </field>

and the page have follow link:
javascript:call_fieldlookup3(document.formname.fSyclmc,'
document.formname.fSyclgg,'lookupsyclmc');

after i modify the java souce code, the link change to :
javascript:call_fieldlookup3(document.formname.fSyclmc,
document.formname.fSyclgg,'lookupsyclmc');


On Mon, Jul 7, 2008 at 8:18 AM, stamilo <[hidden email]> wrote:

> :0
> i use ofbiz trunk, svn revision number is 673262;
> i create a test applicate , want to use lookup field,
> when set one field , that can run, all right.
>
> when i want to set two field , i set description-field-name field,
> but can't work, the popup windows can't show,
> there is a error in java script , so i edit the source code,
> delete a char "'",  :) , now it can run
>
>
> On Sun, Jul 6, 2008 at 5:12 AM, Jacques Le Roux <
> [hidden email]> wrote:
>
>> This is really not clear, could you be more specific ? For instance,
>> version you are using, what you intend to do (from a functionnal POV), steps
>> to reproduce, etc.
>>
>> Jacques
>>
>> From: "stamilo" <[hidden email]>
>>
>>  hi,
>>>  i use lookup field to set two field from popup,
>>>
>>>  when i set the description-field-name field ,
>>>
>>>  it can't work, i change the code, see blow,
>>>
>>> Index: HtmlFormRenderer.java
>>> ===================================================================
>>> --- HtmlFormRenderer.java    (revision 673262)
>>> +++ HtmlFormRenderer.java    (working copy)
>>> @@ -2003,7 +2003,7 @@
>>>
>>> writer.append(modelFormField.getModelForm().getCurrentFormName(context));
>>>            writer.append('.');
>>>            writer.append(modelFormField.getParameterName(context));
>>> -            writer.append(",'");
>>> +            writer.append(",");
>>>            writer.append(descriptionFieldName);
>>>            writer.append(",'");
>>>        } else {
>>>
>>>
>>>  it can work now ,
>>>
>>>
>>>  is it a bug ?
>>>
>>> :)
>>>
>>>
>>
>