[
https://issues.apache.org/jira/browse/WW-5440?focusedWorklogId=925833&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-925833
]
ASF GitHub Bot logged work on WW-5440:
--------------------------------------
Author: ASF GitHub Bot
Created on: 14/Jul/24 13:31
Start Date: 14/Jul/24 13:31
Worklog Time Spent: 10m
Work Description: lukaszlenart commented on code in PR #986:
URL: https://github.com/apache/struts/pull/986#discussion_r1677136348
##########
plugins/config-browser/src/main/java/org/apache/struts2/config_browser/ShowConfigAction.java:
##########
@@ -98,16 +104,17 @@ public PropertyDescriptor[] getProperties() {
return properties;
}
+ @Override
public String execute() throws Exception {
super.execute();
config = configHelper.getActionConfig(namespace, actionName);
- actionNames = new
TreeSet<String>(configHelper.getActionNames(namespace));
+ actionNames = new TreeSet<>(configHelper.getActionNames(namespace));
try {
Object action = objectFactory.buildAction(actionName, namespace,
config, null);
properties = reflectionProvider.getPropertyDescriptors(action);
} catch (Exception e) {
- LOG.error("Unable to get properties for action " + actionName, e);
- addActionError("Unable to retrieve action properties: " +
e.toString());
+ LOG.error("Unable to get properties for action {}", actionName, e);
Review Comment:
Ah, nice! I were always using `ParameterizedMessage` in such case, good to
know!
Issue Time Tracking
-------------------
Worklog Id: (was: 925833)
Time Spent: 1h 50m (was: 1h 40m)
> Fix OGNL allowlist compatibility with Convention plugin
> -------------------------------------------------------
>
> Key: WW-5440
> URL: https://issues.apache.org/jira/browse/WW-5440
> Project: Struts 2
> Issue Type: Bug
> Components: Plugin - Convention
> Affects Versions: 6.4.0
> Reporter: Kusal Kithul-Godage
> Priority: Minor
> Fix For: 6.6.0, 7.0.0
>
> Time Spent: 1h 50m
> Remaining Estimate: 0h
>
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
