You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@ofbiz.apache.org by "Jacques Le Roux (JIRA)" <ji...@apache.org> on 2017/03/31 19:25:41 UTC
[jira] [Assigned] (OFBIZ-9287) Refactor the runAction method in
AbstractModelAction, ModelFormAction and ModelTreeAction classes
[ https://issues.apache.org/jira/browse/OFBIZ-9287?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Jacques Le Roux reassigned OFBIZ-9287:
--------------------------------------
Assignee: Jacques Le Roux
> Refactor the runAction method in AbstractModelAction, ModelFormAction and ModelTreeAction classes
> -------------------------------------------------------------------------------------------------
>
> Key: OFBIZ-9287
> URL: https://issues.apache.org/jira/browse/OFBIZ-9287
> Project: OFBiz
> Issue Type: Improvement
> Reporter: Jacques Le Roux
> Assignee: Jacques Le Roux
> Priority: Minor
> Labels: refactoring
>
> This 3 methods share a pattern which should be refactored in a common protected method. The pattern is
> {code}
> if (!this.resultMapNameAcsr.isEmpty()) {
> this.resultMapNameAcsr.put(context, result);
> String queryString = (String) result.get("queryString");
> context.put("queryString", queryString);
> context.put("queryStringMap", result.get("queryStringMap"));
> if (UtilValidate.isNotEmpty(queryString)) {
> try {
> String queryStringEncoded = queryString.replaceAll("&", "%26");
> context.put("queryStringEncoded", queryStringEncoded);
> } catch (PatternSyntaxException e) {
> // obviously a PatternSyntaxException should not occur here
> }
> }
> } else {
> context.putAll(result);
> }
> {code}
> Enough for a refactor I'd say.
--
This message was sent by Atlassian JIRA
(v6.3.15#6346)