Author: adityasharma
Date: Mon Jul  2 05:28:22 2018
New Revision: 1834816

URL: http://svn.apache.org/viewvc?rev=1834816&view=rev
Log:
Fixed: <#if containerId?has_content> id=''</#if> is missed in renderFormOpen
(OFBIZ-10437)

With r1832307, the code that adds the id attribute to form was missing and the 
attribute autocomplete was repeated twice. 

Thanks: Wei and Jacques for discovering the issue.

Modified:
    
ofbiz/ofbiz-framework/trunk/themes/common-theme/template/macro/HtmlFormMacroLibrary.ftl

Modified: 
ofbiz/ofbiz-framework/trunk/themes/common-theme/template/macro/HtmlFormMacroLibrary.ftl
URL: 
http://svn.apache.org/viewvc/ofbiz/ofbiz-framework/trunk/themes/common-theme/template/macro/HtmlFormMacroLibrary.ftl?rev=1834816&r1=1834815&r2=1834816&view=diff
==============================================================================
--- 
ofbiz/ofbiz-framework/trunk/themes/common-theme/template/macro/HtmlFormMacroLibrary.ftl
 (original)
+++ 
ofbiz/ofbiz-framework/trunk/themes/common-theme/template/macro/HtmlFormMacroLibrary.ftl
 Mon Jul  2 05:28:22 2018
@@ -323,7 +323,7 @@ under the License.
 <#macro renderSingleFormFieldTitle></#macro>
 
 <#macro renderFormOpen linkUrl formType name viewIndexField viewSizeField 
viewIndex viewSize targetWindow="" containerId="" containerStyle="" 
autocomplete="" useRowSubmit="" focusFieldName="" hasRequiredField="">
-  <form method="post" action="${linkUrl}"<#if formType=="upload"> 
enctype="multipart/form-data"</#if><#if targetWindow?has_content> 
target="${targetWindow}"</#if> <#if focusFieldName?has_content> 
data-focus-field="${focusFieldName}"</#if> class="<#if 
containerStyle?has_content>${containerStyle}<#else>basic-form</#if><#if 
hasRequiredField?has_content> requireValidation</#if>" 
onsubmit="javascript:submitFormDisableSubmits(this)"<#if 
autocomplete?has_content> autocomplete="${autocomplete}"</#if> 
name="${name}"<#if autocomplete?has_content> 
autocomplete="${autocomplete}"</#if>><#lt/>
+  <form method="post" action="${linkUrl}"<#if formType=="upload"> 
enctype="multipart/form-data"</#if><#if targetWindow?has_content> 
target="${targetWindow}"</#if><#if containerId?has_content> 
id="${containerId}"</#if> <#if focusFieldName?has_content> 
data-focus-field="${focusFieldName}"</#if> class="<#if 
containerStyle?has_content>${containerStyle}<#else>basic-form</#if><#if 
hasRequiredField?has_content> requireValidation</#if>" 
onsubmit="javascript:submitFormDisableSubmits(this)"<#if 
autocomplete?has_content> autocomplete="${autocomplete}"</#if> 
name="${name}"><#lt/>
     <#if useRowSubmit?has_content && useRowSubmit>
       <input type="hidden" name="_useRowSubmit" value="Y"/>
       <#if linkUrl?index_of("VIEW_INDEX") &lt;= 0 && 
linkUrl?index_of(viewIndexField) &lt;= 0>


Reply via email to