[jira] [Commented] (OFBIZ-3920) HtmlFormRenderer : with attribute skip-start, lookup fields are broken

["Deepak Dixit (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/OFBIZ-3920?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14141951#comment-14141951 ] 

Deepak Dixit commented on OFBIZ-3920:
-------------------------------------

Hi Jacques,

IMO we can close this task, as call_ pattern is removed for lookup, and as per new pattern its working as expected.

> HtmlFormRenderer : with attribute skip-start, lookup fields are broken
> ----------------------------------------------------------------------
>
>                 Key: OFBIZ-3920
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-3920
>             Project: OFBiz
>          Issue Type: Bug
>          Components: framework
>    Affects Versions: Trunk
>            Reporter: Nicolas Malin
>            Assignee: Jacques Le Roux
>            Priority: Minor
>         Attachments: skipStartWithLookup.patch
>
>
> When you want to merge to form and use skip-start="true" on the second form, all contained lookup are broken. In this case, the javascript used to call Lookup screen are using the form name eg : '<a href="javascript:call_fieldlookupLayer( document.form2.lookupfield, 'Tagert', ...)', but with skip-start, the form name isn't present.
> A possible solution is to force lookup field id, but it's not managed on htmlFormMacroLibrary.ftl. I will submit a patch to make this possible. 
> I don't manage problem on descriptionField, this would need to change ModelFormRender and it's a little bit more complicated ;)
> Nicolas



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)