[ofbiz-framework] branch release18.12 updated: Fixed: In place editor wrong enable on display field (OFBIZ-12603)

[nm...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

nmalin pushed a commit to branch release18.12
in repository https://gitbox.apache.org/repos/asf/ofbiz-framework.git


The following commit(s) were added to refs/heads/release18.12 by this push:
     new c88074d784 Fixed: In place editor wrong enable on display field (OFBIZ-12603)
c88074d784 is described below

commit c88074d7844e338c2fa61cd69ee80106aae24d2a
Author: Nicolas Malin <ni...@nereide.fr>
AuthorDate: Fri Apr 22 15:31:50 2022 +0200

    Fixed: In place editor wrong enable on display field (OFBIZ-12603)
    
    When, on a form, you set a widget-style on a display field, you activate the inline editor without reason.
    The issue comes from HtmlFormMacroLibrary.ftl that initialize some value in a span without control if it's necessary.
---
 themes/common-theme/template/macro/HtmlFormMacroLibrary.ftl | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/themes/common-theme/template/macro/HtmlFormMacroLibrary.ftl b/themes/common-theme/template/macro/HtmlFormMacroLibrary.ftl
index 8a6e04e844..f77d33e8df 100644
--- a/themes/common-theme/template/macro/HtmlFormMacroLibrary.ftl
+++ b/themes/common-theme/template/macro/HtmlFormMacroLibrary.ftl
@@ -28,7 +28,7 @@ under the License.
     <img src="${imageLocation}" alt=""><#lt/>
   <#else>
     <#if inPlaceEditorUrl?has_content || class?has_content || alert=="true" || title?has_content>
-      <span data-inplace-editor-url="${inPlaceEditorUrl}" data-inplace-editor-params="${inPlaceEditorParams}" <#if idName?has_content>id="cc_${idName}"</#if> <#if title?has_content>title="${title}"</#if> <@renderClass class alert />><#t/>
+      <span <#if inPlaceEditorUrl?has_content>data-inplace-editor-url="${inPlaceEditorUrl}" data-inplace-editor-params="${inPlaceEditorParams}"</#if> <#if idName?has_content>id="cc_${idName}"</#if> <#if title?has_content>title="${title}"</#if> <@renderClass class alert />><#t/>
     </#if>
 
     <#if description?has_content>