Permalink
Browse files

~ JSUTIL-17: Removed comparison types Date & Date_without_time from V…

…alueFieldCondition, because this condition does not support date types anyway.
  • Loading branch information...
1 parent e0982bf commit af361415a9bbe5e60cb0058119b4f0225d2ed4f4 Stefan committed Jul 11, 2011
@@ -90,7 +90,12 @@ public ConditionChecker getChecker(ComparisonType type, ConditionType condition)
* @return
*/
public List<ComparisonType> getComparisonTypes() {
- return new ArrayList<ComparisonType>(COMPARISONS_CACHE.values());
+
+ List<ComparisonType> comparisonTypes = new ArrayList<ComparisonType>(COMPARISONS_CACHE.values());
+ // Date fields are removed, because date comparison is not implemented yet. - See also FieldCollectionsUtils.
+ comparisonTypes.remove(DATE);
+ comparisonTypes.remove(DATE_WITHOUT_TIME);
+ return comparisonTypes;
}
/**
@@ -421,7 +421,7 @@ public boolean isFieldRequired(Issue issue, Field field) {
List<Field> allFields = getAllFields();
allFields.removeAll(getNonValueFieldConditionFields());
- // Date fields are removed, because date comparison is not implemented yet.
+ // Date fields are removed, because date comparison is not implemented yet. - See also ConditionCheckerFactory.
allFields.removeAll(getAllDateFields());
return allFields;

0 comments on commit af36141

Please sign in to comment.