You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ofbiz.apache.org by pg...@apache.org on 2019/01/04 15:02:45 UTC
svn commit: r1850385 - in
/ofbiz/ofbiz-framework/trunk/framework/entity/src/main/java/org/apache/ofbiz/entity/condition:
EntityConditionList.java EntityConditionListBase.java
Author: pgil
Date: Fri Jan 4 15:02:45 2019
New Revision: 1850385
URL: http://svn.apache.org/viewvc?rev=1850385&view=rev
Log:
Improved: Refactoring ‘EntityCondition’ - Rewrite EntityConditionListBase class
(OFBIZ-10691)
Unecessary ‘this’ has been removed and the javadoc has been
expanded. The visibility of the class has been reduced to package
only. The ‘conditionList’ field has been renamed to ‘conditions’ and
the subclasses has been adapted.
Thanks Mathieu for the contribution
Modified:
ofbiz/ofbiz-framework/trunk/framework/entity/src/main/java/org/apache/ofbiz/entity/condition/EntityConditionList.java
ofbiz/ofbiz-framework/trunk/framework/entity/src/main/java/org/apache/ofbiz/entity/condition/EntityConditionListBase.java
Modified: ofbiz/ofbiz-framework/trunk/framework/entity/src/main/java/org/apache/ofbiz/entity/condition/EntityConditionList.java
URL: http://svn.apache.org/viewvc/ofbiz/ofbiz-framework/trunk/framework/entity/src/main/java/org/apache/ofbiz/entity/condition/EntityConditionList.java?rev=1850385&r1=1850384&r2=1850385&view=diff
==============================================================================
--- ofbiz/ofbiz-framework/trunk/framework/entity/src/main/java/org/apache/ofbiz/entity/condition/EntityConditionList.java (original)
+++ ofbiz/ofbiz-framework/trunk/framework/entity/src/main/java/org/apache/ofbiz/entity/condition/EntityConditionList.java Fri Jan 4 15:02:45 2019
@@ -47,7 +47,7 @@ public final class EntityConditionList<T
* @return the size of the internal list of condition expressions
*/
public int getConditionListSize() {
- return conditionList.size();
+ return conditions.size();
}
/**
@@ -57,7 +57,7 @@ public final class EntityConditionList<T
*/
@SuppressWarnings("unchecked")
public Iterator<T> getConditionIterator() {
- return (Iterator<T>)conditionList.iterator();
+ return (Iterator<T>)conditions.iterator();
}
@Override
Modified: ofbiz/ofbiz-framework/trunk/framework/entity/src/main/java/org/apache/ofbiz/entity/condition/EntityConditionListBase.java
URL: http://svn.apache.org/viewvc/ofbiz/ofbiz-framework/trunk/framework/entity/src/main/java/org/apache/ofbiz/entity/condition/EntityConditionListBase.java?rev=1850385&r1=1850384&r2=1850385&view=diff
==============================================================================
--- ofbiz/ofbiz-framework/trunk/framework/entity/src/main/java/org/apache/ofbiz/entity/condition/EntityConditionListBase.java (original)
+++ ofbiz/ofbiz-framework/trunk/framework/entity/src/main/java/org/apache/ofbiz/entity/condition/EntityConditionListBase.java Fri Jan 4 15:02:45 2019
@@ -28,54 +28,71 @@ import org.apache.ofbiz.entity.config.mo
import org.apache.ofbiz.entity.model.ModelEntity;
/**
- * Encapsulates a list of EntityConditions to be used as a single EntityCondition combined as specified
- *
+ * Represents a combination of multiple condition expressions.
*/
@SuppressWarnings("serial")
-public abstract class EntityConditionListBase<T extends EntityCondition> implements EntityCondition {
+abstract class EntityConditionListBase<T extends EntityCondition> implements EntityCondition {
public static final String module = EntityConditionListBase.class.getName();
-
- protected final List<? extends T> conditionList;
+ /** The list of condition expressions to combine. */
+ protected final List<? extends T> conditions;
+ /** The infix operator used to combine every elements in the list of conditions. */
protected final EntityJoinOperator operator;
- protected EntityConditionListBase(List<? extends T> conditionList, EntityJoinOperator operator) {
- this.conditionList = conditionList;
+ /**
+ * Constructs a combination of multiple condition expressions.
+ *
+ * @param conditions the list of condition expressions to combine
+ * @param operator the infix operator used to combine every elements in the list of conditions
+ */
+ protected EntityConditionListBase(List<? extends T> conditions, EntityJoinOperator operator) {
+ this.conditions = conditions;
this.operator = operator;
}
+ /**
+ * Gets the infix operator used to combine every elements in the list of conditions.
+ *
+ * @return the infix operator used to combine every elements in the list of conditions.
+ */
public EntityJoinOperator getOperator() {
- return this.operator;
+ return operator;
}
+ /**
+ * Gets the condition expression stored at a particular of the internal list of conditions.
+ *
+ * @param index the index of the condition expression to find
+ * @return the corresponding condition expression
+ */
public T getCondition(int index) {
- return this.conditionList.get(index);
+ return conditions.get(index);
}
@Override
public boolean isEmpty() {
- return operator.isEmpty(conditionList);
+ return operator.isEmpty(conditions);
}
@Override
public String makeWhereString(ModelEntity modelEntity, List<EntityConditionParam> entityConditionParams, Datasource datasourceInfo) {
StringBuilder sql = new StringBuilder();
- operator.addSqlValue(sql, modelEntity, entityConditionParams, conditionList, datasourceInfo);
+ operator.addSqlValue(sql, modelEntity, entityConditionParams, conditions, datasourceInfo);
return sql.toString();
}
@Override
public void checkCondition(ModelEntity modelEntity) throws GenericModelException {
- operator.validateSql(modelEntity, conditionList);
+ operator.validateSql(modelEntity, conditions);
}
@Override
public boolean mapMatches(Delegator delegator, Map<String, ? extends Object> map) {
- return operator.mapMatches(delegator, map, conditionList);
+ return operator.mapMatches(delegator, map, conditions);
}
@Override
public EntityCondition freeze() {
- return operator.freeze(conditionList);
+ return operator.freeze(conditions);
}
@Override
@@ -85,12 +102,12 @@ public abstract class EntityConditionLis
}
EntityConditionListBase<?> other = UtilGenerics.cast(obj);
- return conditionList.equals(other.conditionList) && operator.equals(other.operator);
+ return conditions.equals(other.conditions) && operator.equals(other.operator);
}
@Override
public int hashCode() {
- return conditionList.hashCode() + operator.hashCode();
+ return conditions.hashCode() + operator.hashCode();
}
@Override