You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ofbiz.apache.org by mb...@apache.org on 2017/10/22 13:58:51 UTC
svn commit: r1812918 [1/2] -
/ofbiz/ofbiz-framework/trunk/framework/widget/src/main/java/org/apache/ofbiz/widget/model/
Author: mbrohl
Date: Sun Oct 22 13:58:51 2017
New Revision: 1812918
URL: http://svn.apache.org/viewvc?rev=1812918&view=rev
Log:
Improved: Fixing defects reported by FindBugs, package
org.apache.ofbiz.widget.model.
(OFBIZ-9710)
Thanks Dennis Balkir for reporting and providing the patch.
Modified:
ofbiz/ofbiz-framework/trunk/framework/widget/src/main/java/org/apache/ofbiz/widget/model/AbstractModelAction.java
ofbiz/ofbiz-framework/trunk/framework/widget/src/main/java/org/apache/ofbiz/widget/model/AbstractModelCondition.java
ofbiz/ofbiz-framework/trunk/framework/widget/src/main/java/org/apache/ofbiz/widget/model/CommonWidgetModels.java
ofbiz/ofbiz-framework/trunk/framework/widget/src/main/java/org/apache/ofbiz/widget/model/HtmlWidget.java
ofbiz/ofbiz-framework/trunk/framework/widget/src/main/java/org/apache/ofbiz/widget/model/IterateSectionWidget.java
ofbiz/ofbiz-framework/trunk/framework/widget/src/main/java/org/apache/ofbiz/widget/model/ModelForm.java
ofbiz/ofbiz-framework/trunk/framework/widget/src/main/java/org/apache/ofbiz/widget/model/ModelFormAction.java
ofbiz/ofbiz-framework/trunk/framework/widget/src/main/java/org/apache/ofbiz/widget/model/ModelFormField.java
ofbiz/ofbiz-framework/trunk/framework/widget/src/main/java/org/apache/ofbiz/widget/model/ModelFormFieldBuilder.java
ofbiz/ofbiz-framework/trunk/framework/widget/src/main/java/org/apache/ofbiz/widget/model/ModelMenu.java
ofbiz/ofbiz-framework/trunk/framework/widget/src/main/java/org/apache/ofbiz/widget/model/ModelMenuAction.java
ofbiz/ofbiz-framework/trunk/framework/widget/src/main/java/org/apache/ofbiz/widget/model/ModelMenuItem.java
ofbiz/ofbiz-framework/trunk/framework/widget/src/main/java/org/apache/ofbiz/widget/model/ModelScreen.java
ofbiz/ofbiz-framework/trunk/framework/widget/src/main/java/org/apache/ofbiz/widget/model/ModelScreenCondition.java
ofbiz/ofbiz-framework/trunk/framework/widget/src/main/java/org/apache/ofbiz/widget/model/ModelScreenWidget.java
ofbiz/ofbiz-framework/trunk/framework/widget/src/main/java/org/apache/ofbiz/widget/model/ModelTree.java
ofbiz/ofbiz-framework/trunk/framework/widget/src/main/java/org/apache/ofbiz/widget/model/ModelTreeAction.java
ofbiz/ofbiz-framework/trunk/framework/widget/src/main/java/org/apache/ofbiz/widget/model/ModelWidgetCondition.java
Modified: ofbiz/ofbiz-framework/trunk/framework/widget/src/main/java/org/apache/ofbiz/widget/model/AbstractModelAction.java
URL: http://svn.apache.org/viewvc/ofbiz/ofbiz-framework/trunk/framework/widget/src/main/java/org/apache/ofbiz/widget/model/AbstractModelAction.java?rev=1812918&r1=1812917&r2=1812918&view=diff
==============================================================================
--- ofbiz/ofbiz-framework/trunk/framework/widget/src/main/java/org/apache/ofbiz/widget/model/AbstractModelAction.java (original)
+++ ofbiz/ofbiz-framework/trunk/framework/widget/src/main/java/org/apache/ofbiz/widget/model/AbstractModelAction.java Sun Oct 22 13:58:51 2017
@@ -118,7 +118,7 @@ public abstract class AbstractModelActio
public static List<ModelAction> readSubActions(ModelWidget modelWidget, Element parentElement) {
List<? extends Element> actionElementList = UtilXml.childElementList(parentElement);
- List<ModelAction> actions = new ArrayList<ModelAction>(actionElementList.size());
+ List<ModelAction> actions = new ArrayList<>(actionElementList.size());
for (Element actionElement : actionElementList) {
actions.add(newInstance(modelWidget, actionElement));
}
@@ -618,7 +618,7 @@ public abstract class AbstractModelActio
@Override
public void runAction(Map<String, Object> context) throws GeneralException {
if (location.endsWith(".xml")) {
- Map<String, Object> localContext = new HashMap<String, Object>();
+ Map<String, Object> localContext = new HashMap<>();
localContext.putAll(context);
DispatchContext ctx = WidgetWorker.getDispatcher(context).getDispatchContext();
MethodContext methodContext = new MethodContext(ctx, localContext, null);
@@ -682,7 +682,7 @@ public abstract class AbstractModelActio
if ("true".equals(autoFieldMapString)) {
DispatchContext dc = WidgetWorker.getDispatcher(context).getDispatchContext();
// try a map called "parameters", try it first so values from here are overriden by values in the main context
- Map<String, Object> combinedMap = new HashMap<String, Object>();
+ Map<String, Object> combinedMap = new HashMap<>();
Map<String, Object> parametersObj = UtilGenerics.toMap(context.get("parameters"));
if (parametersObj != null) {
combinedMap.putAll(parametersObj);
@@ -698,7 +698,7 @@ public abstract class AbstractModelActio
}
}
if (serviceContext == null) {
- serviceContext = new HashMap<String, Object>();
+ serviceContext = new HashMap<>();
}
if (this.fieldMap != null) {
EntityFinderUtil.expandFieldMapToContext(this.fieldMap, context, serviceContext);
@@ -769,7 +769,7 @@ public abstract class AbstractModelActio
Object newValue = null;
String originalName = this.fromField.getOriginalName();
List<String> currentWidgetTrail = UtilGenerics.toList(context.get("_WIDGETTRAIL_"));
- List<String> trailList = new ArrayList<String>();
+ List<String> trailList = new ArrayList<>();
if (currentWidgetTrail != null) {
trailList.addAll(currentWidgetTrail);
}
Modified: ofbiz/ofbiz-framework/trunk/framework/widget/src/main/java/org/apache/ofbiz/widget/model/AbstractModelCondition.java
URL: http://svn.apache.org/viewvc/ofbiz/ofbiz-framework/trunk/framework/widget/src/main/java/org/apache/ofbiz/widget/model/AbstractModelCondition.java?rev=1812918&r1=1812917&r2=1812918&view=diff
==============================================================================
--- ofbiz/ofbiz-framework/trunk/framework/widget/src/main/java/org/apache/ofbiz/widget/model/AbstractModelCondition.java (original)
+++ ofbiz/ofbiz-framework/trunk/framework/widget/src/main/java/org/apache/ofbiz/widget/model/AbstractModelCondition.java Sun Oct 22 13:58:51 2017
@@ -78,7 +78,7 @@ public abstract class AbstractModelCondi
public static List<ModelCondition> readSubConditions(ModelConditionFactory factory, ModelWidget modelWidget,
Element conditionElement) {
List<? extends Element> subElementList = UtilXml.childElementList(conditionElement);
- List<ModelCondition> condList = new ArrayList<ModelCondition>(subElementList.size());
+ List<ModelCondition> condList = new ArrayList<>(subElementList.size());
for (Element subElement : subElementList) {
condList.add(factory.newInstance(modelWidget, subElement));
}
@@ -248,7 +248,7 @@ public abstract class AbstractModelCondi
if (fieldVal == null) {
fieldVal = "";
}
- List<Object> messages = new LinkedList<Object>();
+ List<Object> messages = new LinkedList<>();
Boolean resultBool = BaseCompare.doRealCompare(fieldVal, value, operator, type, format, messages, null, null, true);
if (messages.size() > 0) {
messages.add(0, "Error with comparison in if-compare between field [" + fieldAcsr.toString() + "] with value ["
@@ -326,7 +326,7 @@ public abstract class AbstractModelCondi
if (fieldVal == null) {
fieldVal = "";
}
- List<Object> messages = new LinkedList<Object>();
+ List<Object> messages = new LinkedList<>();
Boolean resultBool = BaseCompare.doRealCompare(fieldVal, toFieldVal, operator, type, format, messages, null, null,
false);
if (messages.size() > 0) {
@@ -595,29 +595,28 @@ public abstract class AbstractModelCondi
Debug.logError(e, module);
return false;
}
- if (permService != null) {
- // build the context
- Map<String, Object> svcCtx = permService.makeValid(serviceContext, ModelService.IN_PARAM);
- svcCtx.put("resourceDescription", resource);
- if (UtilValidate.isNotEmpty(mainAction)) {
- svcCtx.put("mainAction", mainAction);
- }
- // invoke the service
- Map<String, Object> resp;
- try {
- resp = dispatcher.runSync(permService.name, svcCtx, 300, true);
- } catch (GenericServiceException e) {
- Debug.logError(e, module);
- return false;
- }
- if (ServiceUtil.isError(resp) || ServiceUtil.isFailure(resp)) {
- Debug.logError(ServiceUtil.getErrorMessage(resp), module);
- return false;
- }
- Boolean hasPermission = (Boolean) resp.get("hasPermission");
- if (hasPermission != null) {
- return hasPermission.booleanValue();
- }
+ // build the context
+ Map<String, Object> svcCtx = permService.makeValid(serviceContext, ModelService.IN_PARAM);
+ svcCtx.put("resourceDescription", resource);
+ if (UtilValidate.isNotEmpty(mainAction)) {
+ svcCtx.put("mainAction", mainAction);
+ }
+ // invoke the service
+ Map<String, Object> resp;
+ try {
+ resp = dispatcher.runSync(permService.name, svcCtx, 300, true);
+ }
+ catch (GenericServiceException e) {
+ Debug.logError(e, module);
+ return false;
+ }
+ if (ServiceUtil.isError(resp) || ServiceUtil.isFailure(resp)) {
+ Debug.logError(ServiceUtil.getErrorMessage(resp), module);
+ return false;
+ }
+ Boolean hasPermission = (Boolean) resp.get("hasPermission");
+ if (hasPermission != null) {
+ return hasPermission.booleanValue();
}
}
return false;
@@ -812,9 +811,8 @@ public abstract class AbstractModelCondi
if (foundOneTrue) {
// now found two true, so return false
return false;
- } else {
- foundOneTrue = true;
}
+ foundOneTrue = true;
}
}
return foundOneTrue;
Modified: ofbiz/ofbiz-framework/trunk/framework/widget/src/main/java/org/apache/ofbiz/widget/model/CommonWidgetModels.java
URL: http://svn.apache.org/viewvc/ofbiz/ofbiz-framework/trunk/framework/widget/src/main/java/org/apache/ofbiz/widget/model/CommonWidgetModels.java?rev=1812918&r1=1812917&r2=1812918&view=diff
==============================================================================
--- ofbiz/ofbiz-framework/trunk/framework/widget/src/main/java/org/apache/ofbiz/widget/model/CommonWidgetModels.java (original)
+++ ofbiz/ofbiz-framework/trunk/framework/widget/src/main/java/org/apache/ofbiz/widget/model/CommonWidgetModels.java Sun Oct 22 13:58:51 2017
@@ -59,7 +59,7 @@ public final class CommonWidgetModels {
public static class AutoEntityParameters {
private String entityName;
- List<String> excludeList = new ArrayList<String>();
+ List<String> excludeList = new ArrayList<>();
boolean includeNonPk;
boolean includePk;
private String includeType;
@@ -83,7 +83,7 @@ public final class CommonWidgetModels {
@SuppressWarnings("unchecked")
public Map<String, String> getParametersMap(Map<String, Object> context, String defaultEntityName) {
- Map<String, String> autEntityParams = new HashMap<String, String>();
+ Map<String, String> autEntityParams = new HashMap<>();
Delegator delegator = (Delegator) context.get("delegator");
if (delegator == null) {
Debug.logError(
@@ -102,21 +102,19 @@ public final class CommonWidgetModels {
}
Iterator<ModelField> fieldsIter = entity.getFieldsIterator();
- if (fieldsIter != null) {
- while (fieldsIter.hasNext()) {
- ModelField field = fieldsIter.next();
- String fieldName = field.getName();
- FlexibleMapAccessor<Object> fma = FlexibleMapAccessor.getInstance(fieldName);
- boolean shouldExclude = excludeList.contains(fieldName);
- if ((!shouldExclude) && (!field.getIsAutoCreatedInternal())
- && ((field.getIsPk() && includePk) || (!field.getIsPk() && includeNonPk))) {
- Object flexibleValue = fma.get(context);
- if (UtilValidate.isEmpty(flexibleValue) && context.containsKey("parameters")) {
- flexibleValue = fma.get((Map<String, Object>) context.get("parameters"));
- }
- if (UtilValidate.isNotEmpty(flexibleValue) || sendIfEmpty) {
- autEntityParams.put(fieldName, String.valueOf(flexibleValue));
- }
+ while (fieldsIter.hasNext()) {
+ ModelField field = fieldsIter.next();
+ String fieldName = field.getName();
+ FlexibleMapAccessor<Object> fma = FlexibleMapAccessor.getInstance(fieldName);
+ boolean shouldExclude = excludeList.contains(fieldName);
+ if ((!shouldExclude) && (!field.getIsAutoCreatedInternal())
+ && ((field.getIsPk() && includePk) || (!field.getIsPk() && includeNonPk))) {
+ Object flexibleValue = fma.get(context);
+ if (UtilValidate.isEmpty(flexibleValue) && context.containsKey("parameters")) {
+ flexibleValue = fma.get((Map<String, Object>) context.get("parameters"));
+ }
+ if (UtilValidate.isNotEmpty(flexibleValue) || sendIfEmpty) {
+ autEntityParams.put(fieldName, String.valueOf(flexibleValue));
}
}
}
@@ -125,9 +123,7 @@ public final class CommonWidgetModels {
}
public static class AutoServiceParameters {
- List<String> excludeList = new ArrayList<String>();
- boolean includeNonPk;
- boolean includePk;
+ List<String> excludeList = new ArrayList<>();
boolean sendIfEmpty;
private String serviceName;
@@ -146,7 +142,7 @@ public final class CommonWidgetModels {
@SuppressWarnings("unchecked")
public Map<String, String> getParametersMap(Map<String, Object> context, String defaultServiceName) {
- Map<String, String> autServiceParams = new HashMap<String, String>();
+ Map<String, String> autServiceParams = new HashMap<>();
LocalDispatcher dispatcher = (LocalDispatcher) context.get("dispatcher");
if (dispatcher == null) {
Debug.logError(
@@ -365,7 +361,7 @@ public final class CommonWidgetModels {
if (parameterElementList.isEmpty()) {
this.parameterList = Collections.emptyList();
} else {
- List<Parameter> parameterList = new ArrayList<Parameter>(
+ List<Parameter> parameterList = new ArrayList<>(
parameterElementList.size());
for (Element parameterElement : parameterElementList) {
parameterList.add(new Parameter(parameterElement));
@@ -482,7 +478,7 @@ public final class CommonWidgetModels {
}
public Map<String, String> getParameterMap(Map<String, Object> context, String defaultEntityName, String defaultServiceName) {
- Map<String, String> fullParameterMap = new HashMap<String, String>();
+ Map<String, String> fullParameterMap = new HashMap<>();
for (Parameter parameter : this.parameterList) {
fullParameterMap.put(parameter.getName(), parameter.getValue(context));
}
@@ -496,7 +492,7 @@ public final class CommonWidgetModels {
}
public Map<String, String> getParameterMap(Map<String, Object> context) {
- Map<String, String> fullParameterMap = new HashMap<String, String>();
+ Map<String, String> fullParameterMap = new HashMap<>();
for (Parameter parameter : this.parameterList) {
fullParameterMap.put(parameter.getName(), parameter.getValue(context));
}
@@ -655,9 +651,8 @@ public final class CommonWidgetModels {
returnValue = retVal.toString();
}
return returnValue;
- } else {
- return null;
}
+ return null;
}
}
}
Modified: ofbiz/ofbiz-framework/trunk/framework/widget/src/main/java/org/apache/ofbiz/widget/model/HtmlWidget.java
URL: http://svn.apache.org/viewvc/ofbiz/ofbiz-framework/trunk/framework/widget/src/main/java/org/apache/ofbiz/widget/model/HtmlWidget.java?rev=1812918&r1=1812917&r2=1812918&view=diff
==============================================================================
--- ofbiz/ofbiz-framework/trunk/framework/widget/src/main/java/org/apache/ofbiz/widget/model/HtmlWidget.java (original)
+++ ofbiz/ofbiz-framework/trunk/framework/widget/src/main/java/org/apache/ofbiz/widget/model/HtmlWidget.java Sun Oct 22 13:58:51 2017
@@ -60,7 +60,7 @@ public class HtmlWidget extends ModelScr
public static final String module = HtmlWidget.class.getName();
private static final UtilCache<String, Template> specialTemplateCache = UtilCache.createUtilCache("widget.screen.template.ftl.general", 0, 0, false);
- protected static Configuration specialConfig = FreeMarkerWorker.makeConfiguration(new ExtendedWrapper(FreeMarkerWorker.version));
+ protected static final Configuration specialConfig = FreeMarkerWorker.makeConfiguration(new ExtendedWrapper(FreeMarkerWorker.version));
// not sure if this is the best way to get FTL to use my fancy MapModel derivative, but should work at least...
public static class ExtendedWrapper extends BeansWrapper {
@@ -115,7 +115,7 @@ public class HtmlWidget extends ModelScr
if (childElementList.isEmpty()) {
this.subWidgets = Collections.emptyList();
} else {
- List<ModelScreenWidget> subWidgets = new ArrayList<ModelScreenWidget>(childElementList.size());
+ List<ModelScreenWidget> subWidgets = new ArrayList<>(childElementList.size());
for (Element childElement : childElementList) {
if ("html-template".equals(childElement.getNodeName())) {
subWidgets.add(new HtmlTemplate(modelScreen, childElement));
@@ -227,7 +227,7 @@ public class HtmlWidget extends ModelScr
public static class HtmlTemplateDecorator extends ModelScreenWidget {
protected FlexibleStringExpander locationExdr;
- protected Map<String, ModelScreenWidget> sectionMap = new HashMap<String, ModelScreenWidget>();
+ protected Map<String, ModelScreenWidget> sectionMap = new HashMap<>();
public HtmlTemplateDecorator(ModelScreen modelScreen, Element htmlTemplateDecoratorElement) {
super(modelScreen, htmlTemplateDecoratorElement);
Modified: ofbiz/ofbiz-framework/trunk/framework/widget/src/main/java/org/apache/ofbiz/widget/model/IterateSectionWidget.java
URL: http://svn.apache.org/viewvc/ofbiz/ofbiz-framework/trunk/framework/widget/src/main/java/org/apache/ofbiz/widget/model/IterateSectionWidget.java?rev=1812918&r1=1812917&r2=1812918&view=diff
==============================================================================
--- ofbiz/ofbiz-framework/trunk/framework/widget/src/main/java/org/apache/ofbiz/widget/model/IterateSectionWidget.java (original)
+++ ofbiz/ofbiz-framework/trunk/framework/widget/src/main/java/org/apache/ofbiz/widget/model/IterateSectionWidget.java Sun Oct 22 13:58:51 2017
@@ -56,8 +56,8 @@ import org.w3c.dom.Element;
public class IterateSectionWidget extends ModelScreenWidget {
public static final String module = IterateSectionWidget.class.getName();
- public static int DEFAULT_PAGE_SIZE = 5;
- public static int MAX_PAGE_SIZE = 10000;
+ public static final int DEFAULT_PAGE_SIZE = 5;
+ public static final int MAX_PAGE_SIZE = 10000;
private final List<ModelScreenWidget.Section> sectionList;
private final FlexibleMapAccessor<Object> listNameExdr;
@@ -96,7 +96,7 @@ public class IterateSectionWidget extend
if (childElementList.isEmpty()) {
this.sectionList = Collections.emptyList();
} else {
- List<ModelScreenWidget.Section> sectionList = new ArrayList<ModelScreenWidget.Section>(childElementList.size());
+ List<ModelScreenWidget.Section> sectionList = new ArrayList<>(childElementList.size());
for (Element sectionElement: childElementList) {
ModelScreenWidget.Section section = new ModelScreenWidget.Section(modelScreen, sectionElement, false);
sectionList.add(section);
@@ -112,7 +112,7 @@ public class IterateSectionWidget extend
@Override
public void renderWidgetString(Appendable writer, Map<String, Object> context, ScreenStringRenderer screenStringRenderer) throws GeneralException, IOException {
int viewIndex = 0;
- int viewSize = this.viewSize;
+ int locViewSize = this.viewSize;
int lowIndex = -1;
int highIndex = -1;
int listSize = 0;
@@ -151,7 +151,9 @@ public class IterateSectionWidget extend
String viewIndexString = params.get("VIEW_INDEX" + "_" + WidgetWorker.getPaginatorNumber(context));
String viewSizeString = params.get("VIEW_SIZE" + "_" + WidgetWorker.getPaginatorNumber(context));
viewIndex = Integer.parseInt(viewIndexString);
- viewSize = Integer.parseInt(viewSizeString);
+ locViewSize = Integer.parseInt(viewSizeString);
+ } catch (RuntimeException e) {
+ throw e;
} catch (Exception e) {
try {
viewIndex = ((Integer) context.get("viewIndex")).intValue();
@@ -160,11 +162,11 @@ public class IterateSectionWidget extend
}
}
context.put("viewIndex", Integer.valueOf(viewIndex));
- lowIndex = viewIndex * viewSize;
- highIndex = (viewIndex + 1) * viewSize;
+ lowIndex = viewIndex * locViewSize;
+ highIndex = (viewIndex + 1) * locViewSize;
} else {
viewIndex = 0;
- viewSize = MAX_PAGE_SIZE;
+ locViewSize = MAX_PAGE_SIZE;
lowIndex = 0;
highIndex = MAX_PAGE_SIZE;
}
@@ -231,9 +233,8 @@ public class IterateSectionWidget extend
public boolean getPaginate(Map<String, Object> context) {
if (!this.paginate.isEmpty() && UtilValidate.isNotEmpty(this.paginate.expandString(context))) {
return Boolean.valueOf(this.paginate.expandString(context)).booleanValue();
- } else {
- return true;
}
+ return true;
}
public int getViewSize() {
Modified: ofbiz/ofbiz-framework/trunk/framework/widget/src/main/java/org/apache/ofbiz/widget/model/ModelForm.java
URL: http://svn.apache.org/viewvc/ofbiz/ofbiz-framework/trunk/framework/widget/src/main/java/org/apache/ofbiz/widget/model/ModelForm.java?rev=1812918&r1=1812917&r2=1812918&view=diff
==============================================================================
--- ofbiz/ofbiz-framework/trunk/framework/widget/src/main/java/org/apache/ofbiz/widget/model/ModelForm.java (original)
+++ ofbiz/ofbiz-framework/trunk/framework/widget/src/main/java/org/apache/ofbiz/widget/model/ModelForm.java Sun Oct 22 13:58:51 2017
@@ -83,19 +83,19 @@ public abstract class ModelForm extends
public static final String module = ModelForm.class.getName();
public static final String DEFAULT_FORM_RESULT_LIST_NAME = "defaultFormResultList";
/** Pagination settings and defaults. */
- public static int DEFAULT_PAGE_SIZE = 10;
- public static int MAX_PAGE_SIZE = 10000;
- public static String DEFAULT_PAG_INDEX_FIELD = "viewIndex";
- public static String DEFAULT_PAG_SIZE_FIELD = "viewSize";
- public static String DEFAULT_PAG_STYLE = "nav-pager";
- public static String DEFAULT_PAG_FIRST_STYLE = "nav-first";
- public static String DEFAULT_PAG_PREV_STYLE = "nav-previous";
- public static String DEFAULT_PAG_NEXT_STYLE = "nav-next";
- public static String DEFAULT_PAG_LAST_STYLE = "nav-last";
+ public static final int DEFAULT_PAGE_SIZE = 10;
+ public static final int MAX_PAGE_SIZE = 10000;
+ public static final String DEFAULT_PAG_INDEX_FIELD = "viewIndex";
+ public static final String DEFAULT_PAG_SIZE_FIELD = "viewSize";
+ public static final String DEFAULT_PAG_STYLE = "nav-pager";
+ public static final String DEFAULT_PAG_FIRST_STYLE = "nav-first";
+ public static final String DEFAULT_PAG_PREV_STYLE = "nav-previous";
+ public static final String DEFAULT_PAG_NEXT_STYLE = "nav-next";
+ public static final String DEFAULT_PAG_LAST_STYLE = "nav-last";
/** Sort field default styles. */
- public static String DEFAULT_SORT_FIELD_STYLE = "sort-order";
- public static String DEFAULT_SORT_FIELD_ASC_STYLE = "sort-order-asc";
- public static String DEFAULT_SORT_FIELD_DESC_STYLE = "sort-order-desc";
+ public static final String DEFAULT_SORT_FIELD_STYLE = "sort-order";
+ public static final String DEFAULT_SORT_FIELD_ASC_STYLE = "sort-order-asc";
+ public static final String DEFAULT_SORT_FIELD_DESC_STYLE = "sort-order-desc";
private final List<ModelAction> actions;
private final List<AltRowStyle> altRowStyles;
private final List<AltTarget> altTargets;
@@ -126,11 +126,11 @@ public abstract class ModelForm extends
/** This is a list of FieldGroups in the order they were created.
* Can also include Banner objects.
*/
- private final List<FieldGroupBase> fieldGroupList;
+ protected final List<FieldGroupBase> fieldGroupList;
/** This Map is keyed with the field name and has a FieldGroup for the value.
* Can also include Banner objects.
*/
- private final Map<String, FieldGroupBase> fieldGroupMap;
+ protected final Map<String, FieldGroupBase> fieldGroupMap;
/** This List will contain one copy of each field for each field name in the order
* they were encountered in the service, entity, or form definition; field definitions
* with constraints will also be in this list but may appear multiple times for the same
@@ -140,7 +140,7 @@ public abstract class ModelForm extends
* necessary to use the Map. The Map is used when loading the form definition to keep the
* list clean and implement the override features for field definitions.
*/
- private final List<ModelFormField> fieldList;
+ protected final List<ModelFormField> fieldList;
private final String focusFieldName;
private final String formLocation;
private final String formTitleAreaStyle;
@@ -207,7 +207,7 @@ public abstract class ModelForm extends
}
} else {
try {
- defaultViewSizeInt = Integer.valueOf(viewSize);
+ defaultViewSizeInt = Integer.parseInt(viewSize);
} catch (NumberFormatException e) {
}
}
@@ -375,7 +375,7 @@ public abstract class ModelForm extends
paginateTarget = parentModel.paginateTarget;
}
this.paginateTarget = paginateTarget;
- ArrayList<AltTarget> altTargets = new ArrayList<AltTarget>();
+ ArrayList<AltTarget> altTargets = new ArrayList<>();
for (Element altTargetElement : UtilXml.childElementList(formElement, "alt-target")) {
altTargets.add(new AltTarget(altTargetElement));
}
@@ -384,7 +384,7 @@ public abstract class ModelForm extends
}
altTargets.trimToSize();
this.altTargets = Collections.unmodifiableList(altTargets);
- ArrayList<ModelAction> actions = new ArrayList<ModelAction>();
+ ArrayList<ModelAction> actions = new ArrayList<>();
if (parentModel != null) {
actions.addAll(parentModel.actions);
}
@@ -394,7 +394,7 @@ public abstract class ModelForm extends
}
actions.trimToSize();
this.actions = Collections.unmodifiableList(actions);
- ArrayList<ModelAction> rowActions = new ArrayList<ModelAction>();
+ ArrayList<ModelAction> rowActions = new ArrayList<>();
if (parentModel != null) {
rowActions.addAll(parentModel.rowActions);
}
@@ -404,9 +404,9 @@ public abstract class ModelForm extends
}
rowActions.trimToSize();
this.rowActions = Collections.unmodifiableList(rowActions);
- ArrayList<UpdateArea> onPaginateUpdateAreas = new ArrayList<UpdateArea>();
- ArrayList<UpdateArea> onSubmitUpdateAreas = new ArrayList<UpdateArea>();
- ArrayList<UpdateArea> onSortColumnUpdateAreas = new ArrayList<UpdateArea>();
+ ArrayList<UpdateArea> onPaginateUpdateAreas = new ArrayList<>();
+ ArrayList<UpdateArea> onSubmitUpdateAreas = new ArrayList<>();
+ ArrayList<UpdateArea> onSortColumnUpdateAreas = new ArrayList<>();
if (parentModel != null) {
onPaginateUpdateAreas.addAll(parentModel.onPaginateUpdateAreas);
onSubmitUpdateAreas.addAll(parentModel.onSubmitUpdateAreas);
@@ -453,7 +453,7 @@ public abstract class ModelForm extends
this.onSubmitUpdateAreas = Collections.unmodifiableList(onSubmitUpdateAreas);
onSortColumnUpdateAreas.trimToSize();
this.onSortColumnUpdateAreas = Collections.unmodifiableList(onSortColumnUpdateAreas);
- ArrayList<AltRowStyle> altRowStyles = new ArrayList<AltRowStyle>();
+ ArrayList<AltRowStyle> altRowStyles = new ArrayList<>();
if (parentModel != null) {
altRowStyles.addAll(parentModel.altRowStyles);
}
@@ -463,12 +463,12 @@ public abstract class ModelForm extends
}
altRowStyles.trimToSize();
this.altRowStyles = Collections.unmodifiableList(altRowStyles);
- Set<String> useWhenFields = new HashSet<String>();
+ Set<String> useWhenFields = new HashSet<>();
if (parentModel != null) {
useWhenFields.addAll(parentModel.useWhenFields);
}
- List<ModelFormFieldBuilder> fieldBuilderList = new ArrayList<ModelFormFieldBuilder>();
- Map<String, ModelFormFieldBuilder> fieldBuilderMap = new HashMap<String, ModelFormFieldBuilder>();
+ List<ModelFormFieldBuilder> fieldBuilderList = new ArrayList<>();
+ Map<String, ModelFormFieldBuilder> fieldBuilderMap = new HashMap<>();
if (parentModel != null) {
// Create this fieldList/Map from clones of parentModel's
for (ModelFormField parentChildField : parentModel.fieldList) {
@@ -478,15 +478,15 @@ public abstract class ModelForm extends
fieldBuilderMap.put(builder.getName(), builder);
}
}
- Map<String, FieldGroupBase> fieldGroupMap = new HashMap<String, FieldGroupBase>();
+ Map<String, FieldGroupBase> fieldGroupMap = new HashMap<>();
if (parentModel != null) {
fieldGroupMap.putAll(parentModel.fieldGroupMap);
}
- ArrayList<FieldGroupBase> fieldGroupList = new ArrayList<FieldGroupBase>();
+ ArrayList<FieldGroupBase> fieldGroupList = new ArrayList<>();
if (parentModel != null) {
fieldGroupList.addAll(parentModel.fieldGroupList);
}
- ArrayList<String> lastOrderFields = new ArrayList<String>();
+ ArrayList<String> lastOrderFields = new ArrayList<>();
if (parentModel != null) {
lastOrderFields.addAll(parentModel.lastOrderFields);
}
@@ -605,15 +605,11 @@ public abstract class ModelForm extends
} else {
this.useRowSubmit = "true".equals(useRowSubmit);
}
- FlexibleStringExpander rowCountExdr = FlexibleStringExpander.getInstance(formElement.getAttribute("row-count"));
- if (rowCountExdr.isEmpty() && parentModel != null) {
- rowCountExdr = parentModel.rowCountExdr;
- }
this.rowCountExdr = paginate;
- List<ModelFormFieldBuilder> multiSubmitBuilders = new ArrayList<ModelFormFieldBuilder>();
- ArrayList<AutoFieldsService> autoFieldsServices = new ArrayList<AutoFieldsService>();
- ArrayList<AutoFieldsEntity> autoFieldsEntities = new ArrayList<AutoFieldsEntity>();
- ArrayList<SortField> sortOrderFields = new ArrayList<SortField>();
+ List<ModelFormFieldBuilder> multiSubmitBuilders = new ArrayList<>();
+ ArrayList<AutoFieldsService> autoFieldsServices = new ArrayList<>();
+ ArrayList<AutoFieldsEntity> autoFieldsEntities = new ArrayList<>();
+ ArrayList<SortField> sortOrderFields = new ArrayList<>();
this.defaultFieldGroup = new FieldGroup(null, this, sortOrderFields, fieldGroupMap);
for (Element autoFieldsServiceElement : UtilXml.childElementList(formElement, "auto-fields-service")) {
AutoFieldsService autoFieldsService = new AutoFieldsService(autoFieldsServiceElement);
@@ -666,7 +662,7 @@ public abstract class ModelForm extends
}
}
if (sortOrderFields.size() > 0) {
- List<ModelFormFieldBuilder> sortedFields = new ArrayList<ModelFormFieldBuilder>();
+ List<ModelFormFieldBuilder> sortedFields = new ArrayList<>();
for (SortField sortField : sortOrderFields) {
String fieldName = sortField.getFieldName();
if (UtilValidate.isEmpty(fieldName)) {
@@ -692,7 +688,7 @@ public abstract class ModelForm extends
fieldBuilderList = sortedFields;
}
if (UtilValidate.isNotEmpty(lastOrderFields)) {
- List<ModelFormFieldBuilder> lastedFields = new LinkedList<ModelFormFieldBuilder>();
+ List<ModelFormFieldBuilder> lastedFields = new LinkedList<>();
for (String fieldName : lastOrderFields) {
if (UtilValidate.isEmpty(fieldName)) {
continue;
@@ -711,12 +707,12 @@ public abstract class ModelForm extends
//now put all lastedFields at the field list end
fieldBuilderList.addAll(lastedFields);
}
- List<ModelFormField> fieldList = new ArrayList<ModelFormField>(fieldBuilderList.size());
+ List<ModelFormField> fieldList = new ArrayList<>(fieldBuilderList.size());
for (ModelFormFieldBuilder builder : fieldBuilderList) {
fieldList.add(builder.build());
}
this.fieldList = Collections.unmodifiableList(fieldList);
- List<ModelFormField> multiSubmitFields = new ArrayList<ModelFormField>(multiSubmitBuilders.size());
+ List<ModelFormField> multiSubmitFields = new ArrayList<>(multiSubmitBuilders.size());
for (ModelFormFieldBuilder builder : multiSubmitBuilders) {
multiSubmitFields.add(builder.build());
}
@@ -797,24 +793,22 @@ public abstract class ModelForm extends
ModelEntity modelEntity;
try {
modelEntity = entityModelReader.getModelEntity(modelParam.entityName);
- if (modelEntity != null) {
- ModelField modelField = modelEntity.getField(modelParam.fieldName);
- if (modelField != null) {
- // okay, populate using the entity field info...
- ModelFormFieldBuilder builder = new ModelFormFieldBuilder();
- builder.setModelForm(this);
- builder.setName(modelField.getName());
- builder.setEntityName(modelEntity.getEntityName());
- builder.setFieldName(modelField.getName());
- builder.induceFieldInfoFromEntityField(modelEntity, modelField, autoFieldsService.defaultFieldType);
- if (UtilValidate.isNotEmpty(autoFieldsService.mapName)) {
- builder.setMapName(autoFieldsService.mapName);
- }
- builder.setRequiredField(!modelParam.optional);
- addUpdateField(builder, useWhenFields, fieldBuilderList, fieldBuilderMap);
- // continue to skip creating based on service param
- continue;
+ ModelField modelField = modelEntity.getField(modelParam.fieldName);
+ if (modelField != null) {
+ // okay, populate using the entity field info...
+ ModelFormFieldBuilder builder = new ModelFormFieldBuilder();
+ builder.setModelForm(this);
+ builder.setName(modelField.getName());
+ builder.setEntityName(modelEntity.getEntityName());
+ builder.setFieldName(modelField.getName());
+ builder.induceFieldInfoFromEntityField(modelEntity, modelField, autoFieldsService.defaultFieldType);
+ if (UtilValidate.isNotEmpty(autoFieldsService.mapName)) {
+ builder.setMapName(autoFieldsService.mapName);
}
+ builder.setRequiredField(!modelParam.optional);
+ addUpdateField(builder, useWhenFields, fieldBuilderList, fieldBuilderMap);
+ // continue to skip creating based on service param
+ continue;
}
} catch (GenericEntityException e) {
Debug.logError(e, module);
@@ -856,17 +850,16 @@ public abstract class ModelForm extends
fieldBuilderList.add(builder);
}
return;
+ }
+ // not a conditional field, see if a named field exists in Map
+ ModelFormFieldBuilder existingField = fieldBuilderMap.get(builder.getName());
+ if (existingField != null) {
+ // does exist, update the field by doing a merge/override
+ existingField.mergeOverrideModelFormField(builder);
} else {
- // not a conditional field, see if a named field exists in Map
- ModelFormFieldBuilder existingField = fieldBuilderMap.get(builder.getName());
- if (existingField != null) {
- // does exist, update the field by doing a merge/override
- existingField.mergeOverrideModelFormField(builder);
- } else {
- // does not exist, add to List and Map
- fieldBuilderList.add(builder);
- fieldBuilderMap.put(builder.getName(), builder);
- }
+ // does not exist, add to List and Map
+ fieldBuilderList.add(builder);
+ fieldBuilderMap.put(builder.getName(), builder);
}
}
@@ -903,9 +896,8 @@ public abstract class ModelForm extends
// use the name if there is no id
if (UtilValidate.isNotEmpty(this.containerId)) {
return this.containerId;
- } else {
- return this.getName();
}
+ return this.getName();
}
public String getContainerStyle() {
@@ -1019,9 +1011,8 @@ public abstract class ModelForm extends
public String getItemIndexSeparator() {
if (UtilValidate.isNotEmpty(this.itemIndexSeparator)) {
return this.itemIndexSeparator;
- } else {
- return "_o_";
}
+ return "_o_";
}
public List<String> getLastOrderFields() {
@@ -1106,9 +1097,8 @@ public abstract class ModelForm extends
String paginate = this.paginate.expandString(context);
if (!paginate.isEmpty()) {
return Boolean.valueOf(paginate).booleanValue();
- } else {
- return true;
}
+ return true;
}
public String getPaginateFirstLabel() {
@@ -1455,7 +1445,7 @@ public abstract class ModelForm extends
int position = 1;
try {
if (UtilValidate.isNotEmpty(positionStr)) {
- position = Integer.valueOf(positionStr);
+ position = Integer.parseInt(positionStr);
}
} catch (Exception e) {
Debug.logError(e, "Could not convert position attribute of the field element to an integer: [" + positionStr
@@ -1479,7 +1469,7 @@ public abstract class ModelForm extends
int position = 1;
try {
if (UtilValidate.isNotEmpty(positionStr)) {
- position = Integer.valueOf(positionStr);
+ position = Integer.parseInt(positionStr);
}
} catch (Exception e) {
Debug.logError(e, "Could not convert position attribute of the field element to an integer: [" + positionStr
@@ -1664,7 +1654,8 @@ public abstract class ModelForm extends
Integer posParam = null;
try {
posParam = Integer.valueOf(position);
- } catch (Exception e) {/* just ignore the exception*/
+ } catch (Exception e) {
+ Debug.logInfo("The class SortField caused an exception", module);
}
this.position = posParam;
} else {
@@ -1712,7 +1703,7 @@ public abstract class ModelForm extends
if (parameterElementList.isEmpty()) {
this.parameterList = Collections.emptyList();
} else {
- List<CommonWidgetModels.Parameter> parameterList = new ArrayList<CommonWidgetModels.Parameter>(parameterElementList.size());
+ List<CommonWidgetModels.Parameter> parameterList = new ArrayList<>(parameterElementList.size());
for (Element parameterElement : parameterElementList) {
parameterList.add(new CommonWidgetModels.Parameter(parameterElement));
}
@@ -1765,7 +1756,7 @@ public abstract class ModelForm extends
}
public Map<String, String> getParameterMap(Map<String, Object> context) {
- Map<String, String> fullParameterMap = new HashMap<String, String>();
+ Map<String, String> fullParameterMap = new HashMap<>();
if (autoServiceParameters != null) {
fullParameterMap.putAll(autoServiceParameters.getParametersMap(context, defaultServiceName));
}
Modified: ofbiz/ofbiz-framework/trunk/framework/widget/src/main/java/org/apache/ofbiz/widget/model/ModelFormAction.java
URL: http://svn.apache.org/viewvc/ofbiz/ofbiz-framework/trunk/framework/widget/src/main/java/org/apache/ofbiz/widget/model/ModelFormAction.java?rev=1812918&r1=1812917&r2=1812918&view=diff
==============================================================================
--- ofbiz/ofbiz-framework/trunk/framework/widget/src/main/java/org/apache/ofbiz/widget/model/ModelFormAction.java (original)
+++ ofbiz/ofbiz-framework/trunk/framework/widget/src/main/java/org/apache/ofbiz/widget/model/ModelFormAction.java Sun Oct 22 13:58:51 2017
@@ -46,7 +46,7 @@ public abstract class ModelFormAction {
public static List<ModelAction> readSubActions(ModelForm modelForm, Element parentElement) {
List<? extends Element> actionElementList = UtilXml.childElementList(parentElement);
- List<ModelAction> actions = new ArrayList<ModelAction>(actionElementList.size());
+ List<ModelAction> actions = new ArrayList<>(actionElementList.size());
for (Element actionElement : UtilXml.childElementList(parentElement)) {
if ("service".equals(actionElement.getNodeName())) {
actions.add(new Service(modelForm, actionElement));
@@ -138,7 +138,7 @@ public abstract class ModelFormAction {
this.serviceNameExdr = FlexibleStringExpander.getInstance(serviceElement.getAttribute("service-name"));
this.resultMapNameAcsr = FlexibleMapAccessor.getInstance(serviceElement.getAttribute("result-map"));
this.autoFieldMapExdr = FlexibleStringExpander.getInstance(serviceElement.getAttribute("auto-field-map"));
- FlexibleStringExpander resultMapListNameExdr = FlexibleStringExpander.getInstance("");
+ FlexibleStringExpander resultMapListNameExdr;
if (UtilValidate.isEmpty(serviceElement.getAttribute("result-map-list"))
&& UtilValidate.isEmpty(serviceElement.getAttribute("result-map-list-name"))) {
if (UtilValidate.isEmpty(serviceElement.getAttribute("result-map-list-iterator"))
@@ -196,7 +196,7 @@ public abstract class ModelFormAction {
.makeValidContext(serviceNameExpanded, ModelService.IN_PARAM, context);
}
} else {
- serviceContext = new HashMap<String, Object>();
+ serviceContext = new HashMap<>();
}
if (this.fieldMap != null) {
EntityFinderUtil.expandFieldMapToContext(this.fieldMap, context, serviceContext);
Modified: ofbiz/ofbiz-framework/trunk/framework/widget/src/main/java/org/apache/ofbiz/widget/model/ModelFormField.java
URL: http://svn.apache.org/viewvc/ofbiz/ofbiz-framework/trunk/framework/widget/src/main/java/org/apache/ofbiz/widget/model/ModelFormField.java?rev=1812918&r1=1812917&r2=1812918&view=diff
==============================================================================
--- ofbiz/ofbiz-framework/trunk/framework/widget/src/main/java/org/apache/ofbiz/widget/model/ModelFormField.java (original)
+++ ofbiz/ofbiz-framework/trunk/framework/widget/src/main/java/org/apache/ofbiz/widget/model/ModelFormField.java Sun Oct 22 13:58:51 2017
@@ -112,16 +112,16 @@ public class ModelFormField {
private final FlexibleMapAccessor<Object> entryAcsr;
private final String event;
private final FieldInfo fieldInfo;
- private final String fieldName;
+ protected final String fieldName;
private final String headerLink;
private final String headerLinkStyle;
private final String idName;
private final FlexibleMapAccessor<Map<String, ? extends Object>> mapAcsr;
- private final ModelForm modelForm;
- private final String name;
+ protected final ModelForm modelForm;
+ protected final String name;
private final List<UpdateArea> onChangeUpdateAreas;
private final List<UpdateArea> onClickUpdateAreas;
- private final String parameterName;
+ protected final String parameterName;
private final Integer position;
private final String redWhen;
private final Boolean requiredField;
@@ -167,12 +167,12 @@ public class ModelFormField {
if (builder.getOnChangeUpdateAreas().isEmpty()) {
this.onChangeUpdateAreas = Collections.emptyList();
} else {
- this.onChangeUpdateAreas = Collections.unmodifiableList(new ArrayList<UpdateArea>(builder.getOnChangeUpdateAreas()));
+ this.onChangeUpdateAreas = Collections.unmodifiableList(new ArrayList<>(builder.getOnChangeUpdateAreas()));
}
if (builder.getOnClickUpdateAreas().isEmpty()) {
this.onClickUpdateAreas = Collections.emptyList();
} else {
- this.onClickUpdateAreas = Collections.unmodifiableList(new ArrayList<UpdateArea>(builder.getOnClickUpdateAreas()));
+ this.onClickUpdateAreas = Collections.unmodifiableList(new ArrayList<>(builder.getOnClickUpdateAreas()));
}
this.parameterName = builder.getParameterName();
this.position = builder.getPosition();
@@ -355,7 +355,7 @@ public class ModelFormField {
} else if (retVal instanceof Collection) {
Collection<Object> col = UtilGenerics.checkCollection(retVal);
Iterator<Object> iter = col.iterator();
- List<Object> newCol = new ArrayList<Object>(col.size());
+ List<Object> newCol = new ArrayList<>(col.size());
while (iter.hasNext()) {
Object item = iter.next();
if (item == null) {
@@ -433,9 +433,8 @@ public class ModelFormField {
String parentFormName = this.getParentFormName();
if (UtilValidate.isNotEmpty(parentFormName)) {
return parentFormName + "_" + this.getFieldName();
- } else {
- return this.modelForm.getName() + "_" + this.getFieldName();
}
+ return this.modelForm.getName() + "_" + this.getFieldName();
}
public String getTabindex() {
@@ -513,9 +512,8 @@ public class ModelFormField {
Integer itemIndex = (Integer) context.get("itemIndex");
if (itemIndex != null && "multi".equals(this.modelForm.getType())) {
return baseName + this.modelForm.getItemIndexSeparator() + itemIndex.intValue();
- } else {
- return baseName;
}
+ return baseName;
}
public int getPosition() {
@@ -974,8 +972,7 @@ public class ModelFormField {
String allCheckedStr = this.allChecked.expandString(context);
if (!allCheckedStr.isEmpty())
return Boolean.valueOf("true".equals(allCheckedStr));
- else
- return null;
+ return null;
}
@Override
@@ -1211,9 +1208,8 @@ public class ModelFormField {
public String getDefaultValue(Map<String, Object> context) {
if (this.defaultValue != null) {
return this.defaultValue.expandString(context);
- } else {
- return "";
}
+ return "";
}
public String getInputMethod() {
@@ -1481,9 +1477,8 @@ public class ModelFormField {
public String getDefaultValue(Map<String, Object> context) {
if (this.defaultValue != null) {
return this.defaultValue.expandString(context);
- } else {
- return "";
}
+ return "";
}
public FlexibleStringExpander getDescription() {
@@ -1784,9 +1779,8 @@ public class ModelFormField {
Integer itemIndex = (Integer) context.get("itemIndex");
if (itemIndex != null && "multi".equals(getModelFormField().modelForm.getType())) {
return baseName + getModelFormField().modelForm.getItemIndexSeparator() + itemIndex.intValue();
- } else {
- return baseName;
}
+ return baseName;
}
public String getSize() {
@@ -1835,7 +1829,7 @@ public class ModelFormField {
this.cache = !"false".equals(entityOptionsElement.getAttribute("cache"));
List<? extends Element> constraintElements = UtilXml.childElementList(entityOptionsElement, "entity-constraint");
if (!constraintElements.isEmpty()) {
- List<EntityFinderUtil.ConditionExpr> constraintList = new ArrayList<EntityFinderUtil.ConditionExpr>(
+ List<EntityFinderUtil.ConditionExpr> constraintList = new ArrayList<>(
constraintElements.size());
for (Element constraintElement : constraintElements) {
constraintList.add(new EntityFinderUtil.ConditionExpr(constraintElement));
@@ -1850,7 +1844,7 @@ public class ModelFormField {
this.keyFieldName = entityOptionsElement.getAttribute("key-field-name");
List<? extends Element> orderByElements = UtilXml.childElementList(entityOptionsElement, "entity-order-by");
if (!orderByElements.isEmpty()) {
- List<String> orderByList = new ArrayList<String>(orderByElements.size());
+ List<String> orderByList = new ArrayList<>(orderByElements.size());
for (Element orderByElement : orderByElements) {
orderByList.add(orderByElement.getAttribute("field-name"));
}
@@ -1887,7 +1881,7 @@ public class ModelFormField {
// first expand any conditions that need expanding based on the current context
EntityCondition findCondition = null;
if (UtilValidate.isNotEmpty(this.constraintList)) {
- List<EntityCondition> expandedConditionList = new LinkedList<EntityCondition>();
+ List<EntityCondition> expandedConditionList = new LinkedList<>();
for (EntityFinderUtil.Condition condition : constraintList) {
ModelEntity modelEntity = delegator.getModelEntity(this.entityName);
if (modelEntity == null) {
@@ -2016,7 +2010,7 @@ public class ModelFormField {
super(element, modelFormField);
this.noCurrentSelectedKey = FlexibleStringExpander.getInstance(element.getAttribute("no-current-selected-key"));
// read all option and entity-options sub-elements, maintaining order
- ArrayList<OptionSource> optionSources = new ArrayList<OptionSource>();
+ ArrayList<OptionSource> optionSources = new ArrayList<>();
List<? extends Element> childElements = UtilXml.childElementList(element);
if (childElements.size() > 0) {
for (Element childElement : childElements) {
@@ -2043,7 +2037,7 @@ public class ModelFormField {
if (original.optionSources.isEmpty()) {
this.optionSources = original.optionSources;
} else {
- List<OptionSource> optionSources = new ArrayList<OptionSource>(original.optionSources.size());
+ List<OptionSource> optionSources = new ArrayList<>(original.optionSources.size());
for (OptionSource source : original.optionSources) {
optionSources.add(source.copy(modelFormField));
}
@@ -2054,7 +2048,7 @@ public class ModelFormField {
protected FieldInfoWithOptions(int fieldSource, int fieldType, List<OptionSource> optionSources) {
super(fieldSource, fieldType, null);
this.noCurrentSelectedKey = FlexibleStringExpander.getInstance("");
- this.optionSources = Collections.unmodifiableList(new ArrayList<OptionSource>(optionSources));
+ this.optionSources = Collections.unmodifiableList(new ArrayList<>(optionSources));
}
public FieldInfoWithOptions(int fieldSource, int fieldType, ModelFormField modelFormField) {
@@ -2064,7 +2058,7 @@ public class ModelFormField {
}
public List<OptionValue> getAllOptionValues(Map<String, Object> context, Delegator delegator) {
- List<OptionValue> optionValues = new LinkedList<OptionValue>();
+ List<OptionValue> optionValues = new LinkedList<>();
for (OptionSource optionSource : this.optionSources) {
optionSource.addOptionValues(optionValues, context, delegator);
}
@@ -2190,6 +2184,8 @@ public class ModelFormField {
org.apache.ofbiz.entity.model.ModelReader entityModelReader = ((org.apache.ofbiz.entity.Delegator)context.get("delegator")).getModelReader();
org.apache.ofbiz.service.DispatchContext dispatchContext = ((org.apache.ofbiz.service.LocalDispatcher)context.get("dispatcher")).getDispatchContext();
modelForm = FormFactory.getFormFromLocation(location, name, entityModelReader, dispatchContext);
+ } catch (RuntimeException e) {
+ throw e;
} catch (Exception e) {
String errMsg = "Error rendering form named [" + name + "] at location [" + location + "]: ";
Debug.logError(e, errMsg, module);
@@ -2269,6 +2265,8 @@ public class ModelFormField {
org.apache.ofbiz.entity.model.ModelReader entityModelReader = ((org.apache.ofbiz.entity.Delegator)context.get("delegator")).getModelReader();
org.apache.ofbiz.service.DispatchContext dispatchContext = ((org.apache.ofbiz.service.LocalDispatcher)context.get("dispatcher")).getDispatchContext();
modelForm = GridFactory.getGridFromLocation(location, name, entityModelReader, dispatchContext);
+ } catch (RuntimeException e) {
+ throw e;
} catch (Exception e) {
String errMsg = "Error rendering grid named [" + name + "] at location [" + location + "]: ";
Debug.logError(e, errMsg, module);
@@ -2328,9 +2326,8 @@ public class ModelFormField {
valueEnc = simpleEncoder.encode(valueEnc);
}
return valueEnc;
- } else {
- return getModelFormField().getEntry(context);
}
+ return getModelFormField().getEntry(context);
}
@Override
@@ -2684,9 +2681,8 @@ public class ModelFormField {
public String getDefaultValue(Map<String, Object> context) {
if (this.defaultValue != null) {
return this.defaultValue.expandString(context);
- } else {
- return "";
}
+ return "";
}
public FlexibleStringExpander getDescription() {
@@ -2817,7 +2813,7 @@ public class ModelFormField {
}
public Map<String, Object> getFieldMap(Map<String, Object> context) {
- Map<String, Object> inPlaceEditorContext = new HashMap<String, Object>();
+ Map<String, Object> inPlaceEditorContext = new HashMap<>();
EntityFinderUtil.expandFieldMapToContext(this.fieldMap, context, inPlaceEditorContext);
return inPlaceEditorContext;
}
@@ -2905,9 +2901,8 @@ public class ModelFormField {
public String getUrl(Map<String, Object> context) {
if (this.url != null) {
return this.url.expandString(context);
- } else {
- return "";
}
+ return "";
}
}
@@ -2952,7 +2947,7 @@ public class ModelFormField {
List<? extends Object> dataList = UtilGenerics.checkList(this.listAcsr.get(context));
if (dataList != null && dataList.size() != 0) {
for (Object data : dataList) {
- Map<String, Object> localContext = new HashMap<String, Object>();
+ Map<String, Object> localContext = new HashMap<>();
localContext.putAll(context);
if (UtilValidate.isNotEmpty(this.listEntryName)) {
localContext.put(this.listEntryName, data);
@@ -3115,7 +3110,7 @@ public class ModelFormField {
}
public List<String> getTargetParameterList() {
- List<String> paramList = new LinkedList<String>();
+ List<String> paramList = new LinkedList<>();
if (UtilValidate.isNotEmpty(this.targetParameter)) {
StringTokenizer stk = new StringTokenizer(this.targetParameter, ", ");
while (stk.hasMoreTokens()) {
@@ -3469,6 +3464,8 @@ public class ModelFormField {
ScreenRenderer subRenderer = new ScreenRenderer(writer, mapStack, renderer.getScreenStringRenderer());
writer.append(subRenderer.render(location, name));
}
+ } catch (RuntimeException e) {
+ throw e;
} catch (Exception e) {
String errMsg = "Error rendering included screen named [" + name + "] at location [" + location + "]: " + e.toString();
Debug.logError(e, errMsg, module);
@@ -3948,9 +3945,8 @@ public class ModelFormField {
public String getDefaultValue(Map<String, Object> context) {
if (this.defaultValue != null) {
return this.defaultValue.expandString(context);
- } else {
- return "";
}
+ return "";
}
public int getRows() { return rows; }
@@ -4121,9 +4117,8 @@ public class ModelFormField {
public String getDefaultValue(Map<String, Object> context) {
if (this.defaultValue != null) {
return this.defaultValue.expandString(context);
- } else {
- return "";
}
+ return "";
}
public boolean getDisabled() {
@@ -4260,7 +4255,7 @@ public class ModelFormField {
if (UtilValidate.isNotEmpty(parameters)) {
String fieldName = this.getModelFormField().getName();
if (parameters.containsKey(fieldName)) {
- ignoreCase = "Y".equals((String) parameters.get(fieldName.concat("_ic")));
+ ignoreCase = "Y".equals(parameters.get(fieldName.concat("_ic")));
}
}
return ignoreCase;
Modified: ofbiz/ofbiz-framework/trunk/framework/widget/src/main/java/org/apache/ofbiz/widget/model/ModelFormFieldBuilder.java
URL: http://svn.apache.org/viewvc/ofbiz/ofbiz-framework/trunk/framework/widget/src/main/java/org/apache/ofbiz/widget/model/ModelFormFieldBuilder.java?rev=1812918&r1=1812917&r2=1812918&view=diff
==============================================================================
--- ofbiz/ofbiz-framework/trunk/framework/widget/src/main/java/org/apache/ofbiz/widget/model/ModelFormFieldBuilder.java (original)
+++ ofbiz/ofbiz-framework/trunk/framework/widget/src/main/java/org/apache/ofbiz/widget/model/ModelFormFieldBuilder.java Sun Oct 22 13:58:51 2017
@@ -86,8 +86,8 @@ public class ModelFormFieldBuilder {
private FlexibleMapAccessor<Map<String, ? extends Object>> mapAcsr = null;
private ModelForm modelForm = null;
private String name = "";
- private List<UpdateArea> onChangeUpdateAreas = new ArrayList<UpdateArea>();
- private List<UpdateArea> onClickUpdateAreas = new ArrayList<UpdateArea>();
+ private List<UpdateArea> onChangeUpdateAreas = new ArrayList<>();
+ private List<UpdateArea> onClickUpdateAreas = new ArrayList<>();
private String parameterName = "";
private Integer position = null;
private String redWhen = "";
@@ -568,7 +568,7 @@ public class ModelFormFieldBuilder {
Integer.valueOf(250), null);
this.setFieldInfo(textField);
} else if ("indicator".equals(modelField.getType())) {
- List<OptionSource> optionSources = new ArrayList<OptionSource>();
+ List<OptionSource> optionSources = new ArrayList<>();
optionSources.add(new ModelFormField.SingleOption("Y", null, null));
optionSources.add(new ModelFormField.SingleOption("N", null, null));
ModelFormField.DropDownField dropDownField = new ModelFormField.DropDownField(FieldInfo.SOURCE_AUTO_ENTITY,
@@ -616,13 +616,11 @@ public class ModelFormFieldBuilder {
return false;
try {
ModelEntity modelEntity = entityModelReader.getModelEntity(this.getEntityName());
- if (modelEntity != null) {
- ModelField modelField = modelEntity.getField(this.getFieldName());
- if (modelField != null) {
- // okay, populate using the entity field info...
- this.induceFieldInfoFromEntityField(modelEntity, modelField, defaultFieldType);
- return true;
- }
+ ModelField modelField = modelEntity.getField(this.getFieldName());
+ if (modelField != null) {
+ // okay, populate using the entity field info...
+ this.induceFieldInfoFromEntityField(modelEntity, modelField, defaultFieldType);
+ return true;
}
} catch (GenericEntityException e) {
Debug.logError(e, module);
@@ -692,20 +690,18 @@ public class ModelFormFieldBuilder {
return false;
try {
ModelService modelService = dispatchContext.getModelService(this.getServiceName());
- if (modelService != null) {
- ModelParam modelParam = modelService.getParam(this.getAttributeName());
- if (modelParam != null) {
- if (UtilValidate.isNotEmpty(modelParam.entityName) && UtilValidate.isNotEmpty(modelParam.fieldName)) {
- this.entityName = modelParam.entityName;
- this.fieldName = modelParam.fieldName;
- if (this.induceFieldInfoFromEntityField(defaultFieldType, entityModelReader)) {
- return true;
- }
+ ModelParam modelParam = modelService.getParam(this.getAttributeName());
+ if (modelParam != null) {
+ if (UtilValidate.isNotEmpty(modelParam.entityName) && UtilValidate.isNotEmpty(modelParam.fieldName)) {
+ this.entityName = modelParam.entityName;
+ this.fieldName = modelParam.fieldName;
+ if (this.induceFieldInfoFromEntityField(defaultFieldType, entityModelReader)) {
+ return true;
}
-
- this.induceFieldInfoFromServiceParam(modelService, modelParam, defaultFieldType);
- return true;
}
+
+ this.induceFieldInfoFromServiceParam(modelService, modelParam, defaultFieldType);
+ return true;
}
} catch (GenericServiceException e) {
Debug.logError(e,
Modified: ofbiz/ofbiz-framework/trunk/framework/widget/src/main/java/org/apache/ofbiz/widget/model/ModelMenu.java
URL: http://svn.apache.org/viewvc/ofbiz/ofbiz-framework/trunk/framework/widget/src/main/java/org/apache/ofbiz/widget/model/ModelMenu.java?rev=1812918&r1=1812917&r2=1812918&view=diff
==============================================================================
--- ofbiz/ofbiz-framework/trunk/framework/widget/src/main/java/org/apache/ofbiz/widget/model/ModelMenu.java (original)
+++ ofbiz/ofbiz-framework/trunk/framework/widget/src/main/java/org/apache/ofbiz/widget/model/ModelMenu.java Sun Oct 22 13:58:51 2017
@@ -104,7 +104,7 @@ public class ModelMenu extends ModelWidg
/** XML Constructor */
public ModelMenu(Element menuElement, String menuLocation) {
super(menuElement);
- ArrayList<ModelAction> actions = new ArrayList<ModelAction>();
+ ArrayList<ModelAction> actions = new ArrayList<>();
String defaultAlign = "";
String defaultAlignStyle = "";
FlexibleStringExpander defaultAssociatedContentId = FlexibleStringExpander.getInstance("");
@@ -123,8 +123,8 @@ public class ModelMenu extends ModelWidg
String fillStyle = "";
String id = "";
FlexibleStringExpander menuContainerStyleExdr = FlexibleStringExpander.getInstance("");
- ArrayList<ModelMenuItem> menuItemList = new ArrayList<ModelMenuItem>();
- Map<String, ModelMenuItem> menuItemMap = new HashMap<String, ModelMenuItem>();
+ ArrayList<ModelMenuItem> menuItemList = new ArrayList<>();
+ Map<String, ModelMenuItem> menuItemMap = new HashMap<>();
String menuWidth = "";
String orientation = "horizontal";
FlexibleMapAccessor<String> selectedMenuItemContextFieldName = FlexibleMapAccessor.getInstance("");
Modified: ofbiz/ofbiz-framework/trunk/framework/widget/src/main/java/org/apache/ofbiz/widget/model/ModelMenuAction.java
URL: http://svn.apache.org/viewvc/ofbiz/ofbiz-framework/trunk/framework/widget/src/main/java/org/apache/ofbiz/widget/model/ModelMenuAction.java?rev=1812918&r1=1812917&r2=1812918&view=diff
==============================================================================
--- ofbiz/ofbiz-framework/trunk/framework/widget/src/main/java/org/apache/ofbiz/widget/model/ModelMenuAction.java (original)
+++ ofbiz/ofbiz-framework/trunk/framework/widget/src/main/java/org/apache/ofbiz/widget/model/ModelMenuAction.java Sun Oct 22 13:58:51 2017
@@ -49,7 +49,7 @@ public abstract class ModelMenuAction {
public static List<ModelAction> readSubActions(ModelMenu modelMenu, Element parentElement) {
List<? extends Element> actionElementList = UtilXml.childElementList(parentElement);
- List<ModelAction> actions = new ArrayList<ModelAction>(actionElementList.size());
+ List<ModelAction> actions = new ArrayList<>(actionElementList.size());
for (Element actionElement : actionElementList) {
if ("set".equals(actionElement.getNodeName())) {
actions.add(new SetField(modelMenu, actionElement));
Modified: ofbiz/ofbiz-framework/trunk/framework/widget/src/main/java/org/apache/ofbiz/widget/model/ModelMenuItem.java
URL: http://svn.apache.org/viewvc/ofbiz/ofbiz-framework/trunk/framework/widget/src/main/java/org/apache/ofbiz/widget/model/ModelMenuItem.java?rev=1812918&r1=1812917&r2=1812918&view=diff
==============================================================================
--- ofbiz/ofbiz-framework/trunk/framework/widget/src/main/java/org/apache/ofbiz/widget/model/ModelMenuItem.java (original)
+++ ofbiz/ofbiz-framework/trunk/framework/widget/src/main/java/org/apache/ofbiz/widget/model/ModelMenuItem.java Sun Oct 22 13:58:51 2017
@@ -74,7 +74,7 @@ public class ModelMenuItem extends Model
private final String disableIfEmpty;
private final String entityName;
private final Boolean hideIfSelected;
- private final MenuLink link;
+ protected final MenuLink link;
private final List<ModelMenuItem> menuItemList;
private final ModelMenu modelMenu;
private final String overrideName;
@@ -143,8 +143,8 @@ public class ModelMenuItem extends Model
// read in add item defs, add/override one by one using the menuItemList and menuItemMap
List<? extends Element> itemElements = UtilXml.childElementList(menuItemElement, "menu-item");
if (!itemElements.isEmpty()) {
- ArrayList<ModelMenuItem> menuItemList = new ArrayList<ModelMenuItem>();
- Map<String, ModelMenuItem> menuItemMap = new HashMap<String, ModelMenuItem>();
+ ArrayList<ModelMenuItem> menuItemList = new ArrayList<>();
+ Map<String, ModelMenuItem> menuItemMap = new HashMap<>();
for (Element itemElement : itemElements) {
ModelMenuItem modelMenuItem = new ModelMenuItem(itemElement, modelMenu, this);
addUpdateMenuItem(modelMenuItem, menuItemList, menuItemMap);
@@ -320,7 +320,7 @@ public class ModelMenuItem extends Model
if (this.associatedContentId != null) {
retStr = associatedContentId.expandString(context);
}
- if (retStr.isEmpty()) {
+ if (retStr == null || retStr.isEmpty()) {
retStr = this.modelMenu.getDefaultAssociatedContentId(context);
}
return retStr;
@@ -329,9 +329,8 @@ public class ModelMenuItem extends Model
public String getCellWidth() {
if (!this.cellWidth.isEmpty()) {
return this.cellWidth;
- } else {
- return this.modelMenu.getDefaultCellWidth();
}
+ return this.modelMenu.getDefaultCellWidth();
}
public ModelMenuCondition getCondition() {
@@ -365,9 +364,8 @@ public class ModelMenuItem extends Model
public Boolean getHideIfSelected() {
if (hideIfSelected != null) {
return this.hideIfSelected;
- } else {
- return this.modelMenu.getDefaultHideIfSelected();
}
+ return this.modelMenu.getDefaultHideIfSelected();
}
public MenuLink getLink() {
@@ -409,9 +407,8 @@ public class ModelMenuItem extends Model
public int getPosition() {
if (this.position == null) {
return 1;
- } else {
- return position.intValue();
}
+ return position.intValue();
}
public String getSelectedStyle() {
@@ -453,9 +450,8 @@ public class ModelMenuItem extends Model
public String getTooltip(Map<String, Object> context) {
if (UtilValidate.isNotEmpty(tooltip)) {
return tooltip.expandString(context);
- } else {
- return "";
}
+ return "";
}
public String getTooltipStyle() {
@@ -529,7 +525,7 @@ public class ModelMenuItem extends Model
public MenuLink(GenericValue portalPage, ModelMenuItem parentMenuItem, Locale locale) {
this.linkMenuItem = parentMenuItem;
- List<Parameter> parameterList = new ArrayList<Parameter>();
+ List<Parameter> parameterList = new ArrayList<>();
if (parentMenuItem.link != null) {
parameterList.addAll(parentMenuItem.link.getParameterList());
}
Modified: ofbiz/ofbiz-framework/trunk/framework/widget/src/main/java/org/apache/ofbiz/widget/model/ModelScreen.java
URL: http://svn.apache.org/viewvc/ofbiz/ofbiz-framework/trunk/framework/widget/src/main/java/org/apache/ofbiz/widget/model/ModelScreen.java?rev=1812918&r1=1812917&r2=1812918&view=diff
==============================================================================
--- ofbiz/ofbiz-framework/trunk/framework/widget/src/main/java/org/apache/ofbiz/widget/model/ModelScreen.java (original)
+++ ofbiz/ofbiz-framework/trunk/framework/widget/src/main/java/org/apache/ofbiz/widget/model/ModelScreen.java Sun Oct 22 13:58:51 2017
@@ -163,6 +163,8 @@ public class ModelScreen extends ModelWi
// render the screen, starting with the top-level section
this.section.renderWidgetString(writer, context, screenStringRenderer);
TransactionUtil.commit(beganTransaction);
+ } catch (RuntimeException e) {
+ throw e;
} catch (Exception e) {
String errMsg = "Error rendering screen [" + this.sourceLocation + "#" + getName() + "]: " + e.toString();
Debug.logError(errMsg + ". Rolling back transaction.", module);
Modified: ofbiz/ofbiz-framework/trunk/framework/widget/src/main/java/org/apache/ofbiz/widget/model/ModelScreenCondition.java
URL: http://svn.apache.org/viewvc/ofbiz/ofbiz-framework/trunk/framework/widget/src/main/java/org/apache/ofbiz/widget/model/ModelScreenCondition.java?rev=1812918&r1=1812917&r2=1812918&view=diff
==============================================================================
--- ofbiz/ofbiz-framework/trunk/framework/widget/src/main/java/org/apache/ofbiz/widget/model/ModelScreenCondition.java (original)
+++ ofbiz/ofbiz-framework/trunk/framework/widget/src/main/java/org/apache/ofbiz/widget/model/ModelScreenCondition.java Sun Oct 22 13:58:51 2017
@@ -83,9 +83,8 @@ public final class ModelScreenCondition
}
if ("if-empty-section".equals(conditionElement.getNodeName())) {
return new IfEmptySection(this, modelWidget, conditionElement);
- } else {
- return super.newInstance(this, modelWidget,conditionElement);
}
+ return super.newInstance(this, modelWidget,conditionElement);
}
}
}
Modified: ofbiz/ofbiz-framework/trunk/framework/widget/src/main/java/org/apache/ofbiz/widget/model/ModelScreenWidget.java
URL: http://svn.apache.org/viewvc/ofbiz/ofbiz-framework/trunk/framework/widget/src/main/java/org/apache/ofbiz/widget/model/ModelScreenWidget.java?rev=1812918&r1=1812917&r2=1812918&view=diff
==============================================================================
--- ofbiz/ofbiz-framework/trunk/framework/widget/src/main/java/org/apache/ofbiz/widget/model/ModelScreenWidget.java (original)
+++ ofbiz/ofbiz-framework/trunk/framework/widget/src/main/java/org/apache/ofbiz/widget/model/ModelScreenWidget.java Sun Oct 22 13:58:51 2017
@@ -80,7 +80,7 @@ public abstract class ModelScreenWidget
if (subElementList.isEmpty()) {
return Collections.emptyList();
}
- List<ModelScreenWidget> subWidgets = new ArrayList<ModelScreenWidget>(subElementList.size());
+ List<ModelScreenWidget> subWidgets = new ArrayList<>(subElementList.size());
for (Element subElement: subElementList) {
subWidgets.add(WidgetFactory.getModelScreenWidget(modelScreen, subElement));
}
@@ -111,7 +111,7 @@ public abstract class ModelScreenWidget
public SectionsRenderer(Map<String, ModelScreenWidget> sectionMap, Map<String, Object> context, Appendable writer,
ScreenStringRenderer screenStringRenderer) {
- Map<String, ModelScreenWidget> localMap = new HashMap<String, ModelScreenWidget>();
+ Map<String, ModelScreenWidget> localMap = new HashMap<>();
localMap.putAll(sectionMap);
this.sectionMap = Collections.unmodifiableMap(localMap);
this.context = context;
@@ -307,9 +307,8 @@ public abstract class ModelScreenWidget
public String getBoundaryCommentName() {
if (isMainSection) {
return getModelScreen().getSourceLocation() + "#" + getModelScreen().getName();
- } else {
- return getName();
}
+ return getName();
}
public List<ModelAction> getActions() {
@@ -344,7 +343,7 @@ public abstract class ModelScreenWidget
this.idExdr = FlexibleStringExpander.getInstance(containerElement.getAttribute("id"));
this.styleExdr = FlexibleStringExpander.getInstance(containerElement.getAttribute("style"));
List<? extends Element> subElementList = UtilXml.childElementList(containerElement, "column");
- List<Column> columns = new ArrayList<Column>(subElementList.size());
+ List<Column> columns = new ArrayList<>(subElementList.size());
for (Element element : subElementList) {
columns.add(new Column(modelScreen, element));
}
@@ -540,7 +539,7 @@ public abstract class ModelScreenWidget
this.titleExdr = FlexibleStringExpander.getInstance(screenletElement.getAttribute("title"));
List<? extends Element> subElementList = UtilXml.childElementList(screenletElement);
// Make a copy of the unmodifiable List so we can modify it.
- List<ModelScreenWidget> subWidgets = new ArrayList<ModelScreenWidget>(ModelScreenWidget.readSubWidgets(getModelScreen(), subElementList));
+ List<ModelScreenWidget> subWidgets = new ArrayList<>(ModelScreenWidget.readSubWidgets(getModelScreen(), subElementList));
Menu navigationMenu = null;
String navMenuName = screenletElement.getAttribute("navigation-menu-name");
if (!navMenuName.isEmpty()) {
@@ -760,7 +759,7 @@ public abstract class ModelScreenWidget
// build the widgetpath
List<String> widgetTrail = UtilGenerics.toList(context.get("_WIDGETTRAIL_"));
if (widgetTrail == null) {
- widgetTrail = new LinkedList<String>();
+ widgetTrail = new LinkedList<>();
}
String thisName = nameExdr.expandString(context);
@@ -826,7 +825,7 @@ public abstract class ModelScreenWidget
super(modelScreen, decoratorScreenElement);
this.nameExdr = FlexibleStringExpander.getInstance(decoratorScreenElement.getAttribute("name"));
this.locationExdr = FlexibleStringExpander.getInstance(decoratorScreenElement.getAttribute("location"));
- Map<String, ModelScreenWidget> sectionMap = new HashMap<String, ModelScreenWidget>();
+ Map<String, ModelScreenWidget> sectionMap = new HashMap<>();
List<? extends Element> decoratorSectionElementList = UtilXml.childElementList(decoratorScreenElement, "decorator-section");
for (Element decoratorSectionElement: decoratorSectionElementList) {
DecoratorSection decoratorSection = new DecoratorSection(modelScreen, decoratorSectionElement);
@@ -1153,6 +1152,8 @@ public abstract class ModelScreenWidget
String location = this.getLocation(context);
try {
modelForm = GridFactory.getGridFromLocation(location, name, getModelScreen().getDelegator(context).getModelReader(), getModelScreen().getDispatcher(context).getDispatchContext());
+ } catch (RuntimeException e) {
+ throw e;
} catch (Exception e) {
String errMsg = "Error rendering included form named [" + name + "] at location [" + location + "]: ";
Debug.logError(e, errMsg, module);
@@ -1284,7 +1285,7 @@ public abstract class ModelScreenWidget
public PlatformSpecific(ModelScreen modelScreen, Element platformSpecificElement) {
super(modelScreen, platformSpecificElement);
- Map<String, ModelScreenWidget> subWidgets = new HashMap<String, ModelScreenWidget>();
+ Map<String, ModelScreenWidget> subWidgets = new HashMap<>();
List<? extends Element> childElements = UtilXml.childElementList(platformSpecificElement);
if (childElements != null) {
for (Element childElement: childElements) {