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

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


The following commit(s) were added to refs/heads/trunk by this push:
     new 50ee4f972a Fixed: Improvement of the Check XML tag (OFBIZ-12821)
50ee4f972a is described below

commit 50ee4f972a7067361ce1fd7e7b99e159591e6f91
Author: Jacques Le Roux <jacques.le.r...@les7arts.com>
AuthorDate: Fri Feb 16 20:52:51 2024 +0100

    Fixed: Improvement of the Check XML tag (OFBIZ-12821)
    
    Like the 2 previous commits this fixes things rather than improve them.
    
    During integration tests, the issue is in ***FormMacroLibrary.ftl files:
    When calling macro "renderCheckField", required parameter "allChecked"
    (parameter #5) was not specified
---
 themes/common-theme/template/macro/CsvFormMacroLibrary.ftl  | 2 +-
 themes/common-theme/template/macro/FoFormMacroLibrary.ftl   | 2 +-
 themes/common-theme/template/macro/TextFormMacroLibrary.ftl | 2 +-
 themes/common-theme/template/macro/XlsFormMacroLibrary.ftl  | 2 +-
 themes/common-theme/template/macro/XmlFormMacroLibrary.ftl  | 2 +-
 5 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/themes/common-theme/template/macro/CsvFormMacroLibrary.ftl 
b/themes/common-theme/template/macro/CsvFormMacroLibrary.ftl
index 406e4ac9ec..2890c4ba60 100644
--- a/themes/common-theme/template/macro/CsvFormMacroLibrary.ftl
+++ b/themes/common-theme/template/macro/CsvFormMacroLibrary.ftl
@@ -41,7 +41,7 @@ under the License.
 </#macro>
 
 <#macro renderTooltip tooltip tooltipStyle></#macro>
-<#macro renderCheckField items className alert id allChecked currentValue name 
event action conditionGroup tabindex disabled></#macro>
+<#macro renderCheckField items className alert id allChecked="" currentValue 
name event action conditionGroup tabindex disabled></#macro>
 <#macro renderRadioField items className alert currentValue 
noCurrentSelectedKey name event action conditionGroup tabindex 
disabled></#macro>
 
 <#macro renderSubmitField buttonType className alert formName action imgSrc 
ajaxUrl id title="" name="" event="" confirmation="" containerId="" tabindex="" 
disabled=false></#macro>
diff --git a/themes/common-theme/template/macro/FoFormMacroLibrary.ftl 
b/themes/common-theme/template/macro/FoFormMacroLibrary.ftl
index f011833d59..6dd20f6342 100644
--- a/themes/common-theme/template/macro/FoFormMacroLibrary.ftl
+++ b/themes/common-theme/template/macro/FoFormMacroLibrary.ftl
@@ -67,7 +67,7 @@ under the License.
 </#if>
 </#macro>
 
-<#macro renderCheckField items className alert id allChecked currentValue name 
event action conditionGroup tabindex disabled><@makeBlock "" "" /></#macro>
+<#macro renderCheckField items className alert id allChecked="" currentValue 
name event action conditionGroup tabindex disabled><@makeBlock "" "" /></#macro>
 <#macro renderRadioField items className alert currentValue 
noCurrentSelectedKey name event action conditionGroup tabindex 
disabled><@makeBlock "" "" /></#macro>
 
 <#macro renderSubmitField buttonType className alert formName action imgSrc 
ajaxUrl id title="" name="" event="" confirmation="" containerId="" tabindex="" 
disabled=false><@makeBlock "" "" /></#macro>
diff --git a/themes/common-theme/template/macro/TextFormMacroLibrary.ftl 
b/themes/common-theme/template/macro/TextFormMacroLibrary.ftl
index bb7370def5..bb5ae18f67 100644
--- a/themes/common-theme/template/macro/TextFormMacroLibrary.ftl
+++ b/themes/common-theme/template/macro/TextFormMacroLibrary.ftl
@@ -41,7 +41,7 @@ under the License.
 </#macro>
 
 <#macro renderTooltip tooltip tooltipStyle></#macro>
-<#macro renderCheckField items className alert id allChecked currentValue name 
event action conditionGroup tabindex disabled></#macro>
+<#macro renderCheckField items className alert id allChecked="" currentValue 
name event action conditionGroup tabindex disabled></#macro>
 <#macro renderRadioField items className alert currentValue 
noCurrentSelectedKey name event action conditionGroup tabindex 
disabled></#macro>
 
 <#macro renderSubmitField buttonType className alert formName action imgSrc 
ajaxUrl id title="" name="" event="" confirmation="" containerId="" tabindex="" 
disabled=false></#macro>
diff --git a/themes/common-theme/template/macro/XlsFormMacroLibrary.ftl 
b/themes/common-theme/template/macro/XlsFormMacroLibrary.ftl
index a7f307c801..4bb98a692c 100644
--- a/themes/common-theme/template/macro/XlsFormMacroLibrary.ftl
+++ b/themes/common-theme/template/macro/XlsFormMacroLibrary.ftl
@@ -42,7 +42,7 @@ under the License.
 
 <#macro renderDropDownField name className alert id multiple formName 
otherFieldName event action size firstInList currentValue explicitDescription 
allowEmpty options fieldName otherFieldName otherValue otherFieldSize 
dDFCurrent ajaxEnabled noCurrentSelectedKey ajaxOptions frequency minChars 
choices autoSelect partialSearch partialChars ignoreCase fullSearch 
conditionGroup tabindex disabled><@renderItemField explicitDescription "txf" 
className/></#macro>
 
-<#macro renderCheckField items className alert id allChecked currentValue name 
event action conditionGroup tabindex disabled><@renderItemField currentValue 
"txf" className/></#macro>
+<#macro renderCheckField items className alert id allChecked="" currentValue 
name event action conditionGroup tabindex disabled><@renderItemField 
currentValue "txf" className/></#macro>
 
 <#macro renderRadioField items className alert currentValue 
noCurrentSelectedKey name event action conditionGroup tabindex 
disabled><@renderItemField currentValue "txf" className/></#macro>
 
diff --git a/themes/common-theme/template/macro/XmlFormMacroLibrary.ftl 
b/themes/common-theme/template/macro/XmlFormMacroLibrary.ftl
index 04f45ff24e..5f189b476b 100644
--- a/themes/common-theme/template/macro/XmlFormMacroLibrary.ftl
+++ b/themes/common-theme/template/macro/XmlFormMacroLibrary.ftl
@@ -49,7 +49,7 @@ under the License.
 <#macro renderDropDownField name className alert id multiple formName 
otherFieldName event action size firstInList currentValue explicitDescription 
allowEmpty options fieldName otherFieldName otherValue otherFieldSize 
dDFCurrent ajaxEnabled noCurrentSelectedKey ajaxOptions frequency minChars 
choices autoSelect partialSearch partialChars ignoreCase fullSearch 
conditionGroup tabindex disabled>
 </#macro>
 
-<#macro renderCheckField items className alert id allChecked currentValue name 
event action conditionGroup tabindex disabled></#macro>
+<#macro renderCheckField items className alert id allChecked="" currentValue 
name event action conditionGroup tabindex disabled></#macro>
 <#macro renderRadioField items className alert currentValue 
noCurrentSelectedKey name event action conditionGroup tabindex 
disabled></#macro>
 
 <#macro renderSubmitField buttonType className alert formName action imgSrc 
ajaxUrl id title="" name="" event="" confirmation="" containerId="" tabindex="" 
disabled=false></#macro>

Reply via email to