You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@struts.apache.org by lu...@apache.org on 2024/01/16 19:40:49 UTC
(struts) branch fix/WW-5387-remove created (now e9738698a)
This is an automated email from the ASF dual-hosted git repository.
lukaszlenart pushed a change to branch fix/WW-5387-remove
in repository https://gitbox.apache.org/repos/asf/struts.git
at e9738698a WW-5387 Fixes remove() signature
This branch includes the following new commits:
new e9738698a WW-5387 Fixes remove() signature
The 1 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails. The revisions
listed as "add" were already present in the repository and have only
been added to this reference.
(struts) 01/01: WW-5387 Fixes remove() signature
Posted by lu...@apache.org.
This is an automated email from the ASF dual-hosted git repository.
lukaszlenart pushed a commit to branch fix/WW-5387-remove
in repository https://gitbox.apache.org/repos/asf/struts.git
commit e9738698a465aebb69d83f89e80792bf4bb349e0
Author: Lukasz Lenart <lu...@apache.org>
AuthorDate: Tue Jan 16 20:36:00 2024 +0100
WW-5387 Fixes remove() signature
---
.../main/java/org/apache/struts2/dispatcher/ApplicationMap.java | 9 +++++++--
core/src/main/java/org/apache/struts2/dispatcher/RequestMap.java | 9 +++++++--
.../java/org/apache/struts2/portlet/PortletApplicationMap.java | 9 +++++++--
3 files changed, 21 insertions(+), 6 deletions(-)
diff --git a/core/src/main/java/org/apache/struts2/dispatcher/ApplicationMap.java b/core/src/main/java/org/apache/struts2/dispatcher/ApplicationMap.java
index f0c5b4bf2..6b9053338 100644
--- a/core/src/main/java/org/apache/struts2/dispatcher/ApplicationMap.java
+++ b/core/src/main/java/org/apache/struts2/dispatcher/ApplicationMap.java
@@ -151,11 +151,16 @@ public class ApplicationMap extends AbstractMap<String, Object> implements Seria
* @param key the attribute to remove.
* @return the entry that was just removed.
*/
- public Object remove(final String key) {
+ @Override
+ public Object remove(Object key) {
+ if (key == null) {
+ return null;
+ }
+
entries = null;
Object value = get(key);
- context.removeAttribute(key);
+ context.removeAttribute(key.toString());
return value;
}
diff --git a/core/src/main/java/org/apache/struts2/dispatcher/RequestMap.java b/core/src/main/java/org/apache/struts2/dispatcher/RequestMap.java
index b8e5b8e67..2af730d94 100644
--- a/core/src/main/java/org/apache/struts2/dispatcher/RequestMap.java
+++ b/core/src/main/java/org/apache/struts2/dispatcher/RequestMap.java
@@ -125,11 +125,16 @@ public class RequestMap extends AbstractMap<String, Object> implements Serializa
* @param key the name of the attribute to remove.
* @return the value that was removed or <tt>null</tt> if the value was not found (and hence, not removed).
*/
- public Object remove(final String key) {
+ @Override
+ public Object remove(final Object key) {
+ if (key == null) {
+ return null;
+ }
+
entries = null;
Object value = get(key);
- request.removeAttribute(key);
+ request.removeAttribute(key.toString());
return value;
}
diff --git a/plugins/portlet/src/main/java/org/apache/struts2/portlet/PortletApplicationMap.java b/plugins/portlet/src/main/java/org/apache/struts2/portlet/PortletApplicationMap.java
index 701deb004..236e0aa67 100644
--- a/plugins/portlet/src/main/java/org/apache/struts2/portlet/PortletApplicationMap.java
+++ b/plugins/portlet/src/main/java/org/apache/struts2/portlet/PortletApplicationMap.java
@@ -197,11 +197,16 @@ public class PortletApplicationMap extends AbstractMap<String, Object> implement
* the attribute to remove.
* @return the entry that was just removed.
*/
- public Object remove(String key) {
+ @Override
+ public Object remove(Object key) {
+ if (key == null) {
+ return null;
+ }
+
entries = null;
Object value = get(key);
- context.removeAttribute(key);
+ context.removeAttribute(key.toString());
return value;
}