This is an automated email from the ASF dual-hosted git repository.
lukaszlenart pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/struts.git
The following commit(s) were added to refs/heads/master by this push:
new 04bc4bd [WW-5125] use .get(...) instead of [...] for fieldErrors
new 0552815 Merge pull request #487 from yasserzamani/ww_5125
04bc4bd is described below
commit 04bc4bd97c41bd181dd45580ce12236218177aca
Author: Yasser Zamani
AuthorDate: Sun Apr 25 14:56:47 2021 +0430
[WW-5125] use .get(...) instead of [...] for fieldErrors
All [ occurrences in all ftl files reviewed. Only fieldErrors had used
[...] which replaced by .get(...) to fix WW-5125 as well as aligning with other
ftl files conduct.
---
.../template/ajaxErrorContainers/controlfooter.ftl | 4 +-
.../ajaxErrorContainers/controlheader-core.ftl | 4 +-
.../main/resources/template/css_xhtml/checkbox.ftl | 4 +-
.../resources/template/css_xhtml/controlfooter.ftl | 4 +-
.../template/css_xhtml/controlheader-core.ftl | 4 +-
core/src/main/resources/template/simple/css.ftl| 2 +-
.../main/resources/template/simple/fielderror.ftl | 4 +-
.../src/main/resources/template/xhtml/checkbox.ftl | 4 +-
.../resources/template/xhtml/controlfooter.ftl | 4 +-
.../template/xhtml/controlheader-core.ftl | 4 +-
.../apache/struts2/views/jsp/ui/TextfieldTest.java | 18 -
.../struts2/views/jsp/ui/Textfield-WW-5125.txt | 80 ++
12 files changed, 116 insertions(+), 20 deletions(-)
diff --git
a/apps/showcase/src/main/resources/template/ajaxErrorContainers/controlfooter.ftl
b/apps/showcase/src/main/resources/template/ajaxErrorContainers/controlfooter.ftl
index 588b400..ccba707 100644
---
a/apps/showcase/src/main/resources/template/ajaxErrorContainers/controlfooter.ftl
+++
b/apps/showcase/src/main/resources/template/ajaxErrorContainers/controlfooter.ftl
@@ -22,12 +22,12 @@ ${parameters.after!}<#t/>
<#lt/>
<#if (parameters.errorposition!"top") == 'bottom'>
-<#assign hasFieldErrors = parameters.name?? && fieldErrors?? &&
fieldErrors[parameters.name]??/>
+<#assign hasFieldErrors = parameters.name?? && fieldErrors?? &&
fieldErrors.get(parameters.name)??/>
<#if hasFieldErrors>
<#rt/>
<#if hasFieldErrors>
-<#list fieldErrors[parameters.name] as error>
+<#list fieldErrors.get(parameters.name) as error>
${error}<#t/>
diff --git
a/apps/showcase/src/main/resources/template/ajaxErrorContainers/controlheader-core.ftl
b/apps/showcase/src/main/resources/template/ajaxErrorContainers/controlheader-core.ftl
index 4ea2e6d..3b6ffb4 100644
---
a/apps/showcase/src/main/resources/template/ajaxErrorContainers/controlheader-core.ftl
+++
b/apps/showcase/src/main/resources/template/ajaxErrorContainers/controlheader-core.ftl
@@ -21,12 +21,12 @@
<#--
Always include elements to show errors. They may be filled later via
AJAX.
-->
-<#assign hasFieldErrors = parameters.name?? && fieldErrors?? &&
fieldErrors[parameters.name]??/>
+<#assign hasFieldErrors = parameters.name?? && fieldErrors?? &&
fieldErrors.get(parameters.name)??/>
<#if (parameters.errorposition!"top") == 'top'>
<#rt/>
<#if hasFieldErrors>
-<#list fieldErrors[parameters.name] as error>
+<#list fieldErrors.get(parameters.name) as error>
${error}<#t/>
diff --git a/core/src/main/resources/template/css_xhtml/checkbox.ftl
b/core/src/main/resources/template/css_xhtml/checkbox.ftl
index fc2b0c5..8fe7000 100644
--- a/core/src/main/resources/template/css_xhtml/checkbox.ftl
+++ b/core/src/main/resources/template/css_xhtml/checkbox.ftl
@@ -23,12 +23,12 @@ NOTE: The 'header' stuff that follows is in this one file
for checkbox due to th
that for checkboxes we do not want the label field to show up as checkboxes
handle their own
lables
-->
-<#assign hasFieldErrors = fieldErrors?? && fieldErrors[parameters.name]??/>
+<#assign hasFieldErrors = fieldErrors?? && fieldErrors.get(parameters.name)??/>
<#if parameters.id??>id="wwgrp_${parameters.id}"<#rt/>
class="wwgrp">
<#if hasFieldErrors>
<#if parameters.id??>id="wwerr_${parameters.id}"<#rt/>
class="wwerr">
-<#list fieldErrors[parameters.name] as error>
+<#list fieldErrors.get(parameters.name) as error>
<#if parameters.id??>
errorFor="${parameters.id}"<#rt/>
diff --git a/core/src/main/resources/template/css_xhtml/controlfooter.ftl
b/core/src/main/resources/template/css_xhtml/controlfooter.ftl
index d90fa7b..492a0db 100644
--- a/core/src/main/resources/template/css_xhtml/controlfooter.ftl
+++ b/core/src/main/resources/template/css_xhtml/controlfooter.ftl
@@ -31,10 +31,10 @@ ${parameters.after!}<#t/>
<#rt/>
<#if (parameters.errorposition!"top") == 'bottom'>
-<#assign hasFieldErrors = parameters.name?? && fieldErrors?? &&
fieldErrors[parameters.name
