DATAMONGO-1564 - Polishing.

Fix JavaDoc references and minor a import formatting issue.

Original pull request: #429.

Author: mp911de

spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/aggregation/AbstractAggregationExpression.java

@@ -44,6 +44,7 @@ public Document toDocument(AggregationOperationContext context) {
 		return toDocument(this.value, context);
 	}
 
+	@SuppressWarnings("unchecked")
 	public Document toDocument(Object value, AggregationOperationContext context) {
 
 		Object valueToUse;
@@ -79,6 +80,7 @@ protected static List<Field> asFields(String... fieldRefs) {
 		return Fields.fields(fieldRefs).asList();
 	}
 
+	@SuppressWarnings("unchecked")
 	private Object unpack(Object value, AggregationOperationContext context) {
 
 		if (value instanceof AggregationExpression) {
@@ -122,13 +124,13 @@ protected List<Object> append(Object value) {
 		return Arrays.asList(this.value, value);
 	}
 
+	@SuppressWarnings("unchecked")
 	protected java.util.Map<String, Object> append(String key, Object value) {
 
 		if (!(this.value instanceof java.util.Map)) {
 			throw new IllegalArgumentException("o_O");
 		}
-		java.util.Map<String, Object> clone = new LinkedHashMap<String, Object>(
-				(java.util.Map<String, Object>) this.value);
+		java.util.Map<String, Object> clone = new LinkedHashMap<String, Object>((java.util.Map<String, Object>) this.value);
 		clone.put(key, value);
 		return clone;
 
@@ -142,7 +144,7 @@ protected List<Object> values() {
 		if (value instanceof java.util.Map) {
 			return new ArrayList<Object>(((java.util.Map) value).values());
 		}
-		return new ArrayList<Object>(Arrays.asList(value));
+		return new ArrayList<Object>(Collections.singletonList(value));
 	}
 
 	protected abstract String getMongoMethod();

spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/aggregation/AccumulatorOperators.java

@@ -43,7 +43,7 @@ public static AccumulatorOperatorFactory valueOf(String fieldReference) {
 	/**
 	 * Take the numeric value referenced resulting from given {@link AggregationExpression}.
 	 *
-	 * @param fieldReference must not be {@literal null}.
+	 * @param expression must not be {@literal null}.
 	 * @return
 	 */
 	public static AccumulatorOperatorFactory valueOf(AggregationExpression expression) {
@@ -71,7 +71,7 @@ public AccumulatorOperatorFactory(String fieldReference) {
 		}
 
 		/**
-		 * Creates new {@link ArrayOperatorFactory} for given {@link AggregationExpression}.
+		 * Creates new {@link AccumulatorOperatorFactory} for given {@link AggregationExpression}.
 		 *
 		 * @param expression must not be {@literal null}.
 		 */
@@ -83,7 +83,7 @@ public AccumulatorOperatorFactory(AggregationExpression expression) {
 		}
 
 		/**
-		 * Creates new {@link AggregationExpressions} that takes the associated numeric value expression and calculates and
+		 * Creates new {@link AggregationExpression} that takes the associated numeric value expression and calculates and
 		 * returns the sum.
 		 *
 		 * @return
@@ -93,7 +93,7 @@ public Sum sum() {
 		}
 
 		/**
-		 * Creates new {@link AggregationExpressions} that takes the associated numeric value expression and returns the
+		 * Creates new {@link AggregationExpression} that takes the associated numeric value expression and returns the
 		 * average value.
 		 *
 		 * @return
@@ -103,7 +103,7 @@ public Avg avg() {
 		}
 
 		/**
-		 * Creates new {@link AggregationExpressions} that takes the associated numeric value expression and returns the
+		 * Creates new {@link AggregationExpression} that takes the associated numeric value expression and returns the
 		 * maximum value.
 		 *
 		 * @return
@@ -113,7 +113,7 @@ public Max max() {
 		}
 
 		/**
-		 * Creates new {@link AggregationExpressions} that takes the associated numeric value expression and returns the
+		 * Creates new {@link AggregationExpression} that takes the associated numeric value expression and returns the
 		 * minimum value.
 		 *
 		 * @return
@@ -123,7 +123,7 @@ public Min min() {
 		}
 
 		/**
-		 * Creates new {@link AggregationExpressions} that takes the associated numeric value expression and calculates the
+		 * Creates new {@link AggregationExpression} that takes the associated numeric value expression and calculates the
 		 * population standard deviation of the input values.
 		 *
 		 * @return
@@ -133,7 +133,7 @@ public StdDevPop stdDevPop() {
 		}
 
 		/**
-		 * Creates new {@link AggregationExpressions} that takes the associated numeric value expression and calculates the
+		 * Creates new {@link AggregationExpression} that takes the associated numeric value expression and calculates the
 		 * sample standard deviation of the input values.
 		 *
 		 * @return
@@ -214,9 +214,10 @@ public Sum and(AggregationExpression expression) {
 		}
 
 		/* (non-Javadoc)
-		 * @see org.springframework.data.mongodb.core.aggregation.AggregationExpressions.AbstractAggregationExpression#toDocument(java.lang.Object, org.springframework.data.mongodb.core.aggregation.AggregationOperationContext)
+		 * @see org.springframework.data.mongodb.core.aggregation.AccumulatorOperators.AbstractAggregationExpression#toDocument(java.lang.Object, org.springframework.data.mongodb.core.aggregation.AggregationOperationContext)
 		 */
 		@Override
+		@SuppressWarnings("unchecked")
 		public Document toDocument(Object value, AggregationOperationContext context) {
 
 			if (value instanceof List) {
@@ -296,9 +297,10 @@ public Avg and(AggregationExpression expression) {
 		}
 
 		/* (non-Javadoc)
-		 * @see org.springframework.data.mongodb.core.aggregation.AggregationExpressions.AbstractAggregationExpression#toDocument(java.lang.Object, org.springframework.data.mongodb.core.aggregation.AggregationOperationContext)
+		 * @see org.springframework.data.mongodb.core.aggregation.AccumulatorOperators.AbstractAggregationExpression#toDocument(java.lang.Object, org.springframework.data.mongodb.core.aggregation.AggregationOperationContext)
 		 */
 		@Override
+		@SuppressWarnings("unchecked")
 		public Document toDocument(Object value, AggregationOperationContext context) {
 
 			if (value instanceof List) {
@@ -378,9 +380,10 @@ public Max and(AggregationExpression expression) {
 		}
 
 		/* (non-Javadoc)
-		 * @see org.springframework.data.mongodb.core.aggregation.AggregationExpressions.AbstractAggregationExpression#toDocument(java.lang.Object, org.springframework.data.mongodb.core.aggregation.AggregationOperationContext)
+		 * @see org.springframework.data.mongodb.core.aggregation.AccumulatorOperators.AbstractAggregationExpression#toDocument(java.lang.Object, org.springframework.data.mongodb.core.aggregation.AggregationOperationContext)
 		 */
 		@Override
+		@SuppressWarnings("unchecked")
 		public Document toDocument(Object value, AggregationOperationContext context) {
 
 			if (value instanceof List) {
@@ -460,9 +463,10 @@ public Min and(AggregationExpression expression) {
 		}
 
 		/* (non-Javadoc)
-		 * @see org.springframework.data.mongodb.core.aggregation.AggregationExpressions.AbstractAggregationExpression#toDocument(java.lang.Object, org.springframework.data.mongodb.core.aggregation.AggregationOperationContext)
+		 * @see org.springframework.data.mongodb.core.aggregation.AccumulatorOperators.AbstractAggregationExpression#toDocument(java.lang.Object, org.springframework.data.mongodb.core.aggregation.AggregationOperationContext)
 		 */
 		@Override
+		@SuppressWarnings("unchecked")
 		public Document toDocument(Object value, AggregationOperationContext context) {
 
 			if (value instanceof List) {
@@ -542,9 +546,10 @@ public StdDevPop and(AggregationExpression expression) {
 		}
 
 		/* (non-Javadoc)
-		 * @see org.springframework.data.mongodb.core.aggregation.AggregationExpressions.AbstractAggregationExpression#toDocument(java.lang.Object, org.springframework.data.mongodb.core.aggregation.AggregationOperationContext)
+		 * @see org.springframework.data.mongodb.core.aggregation.AccumulatorOperators.AbstractAggregationExpression#toDocument(java.lang.Object, org.springframework.data.mongodb.core.aggregation.AggregationOperationContext)
 		 */
 		@Override
+		@SuppressWarnings("unchecked")
 		public Document toDocument(Object value, AggregationOperationContext context) {
 
 			if (value instanceof List) {
@@ -624,9 +629,10 @@ public StdDevSamp and(AggregationExpression expression) {
 		}
 
 		/* (non-Javadoc)
-		 * @see org.springframework.data.mongodb.core.aggregation.AggregationExpressions.AbstractAggregationExpression#toDocument(java.lang.Object, org.springframework.data.mongodb.core.aggregation.AggregationOperationContext)
+		 * @see org.springframework.data.mongodb.core.aggregation.AccumulatorOperators.AbstractAggregationExpression#toDocument(java.lang.Object, org.springframework.data.mongodb.core.aggregation.AggregationOperationContext)
 		 */
 		@Override
+		@SuppressWarnings("unchecked")
 		public Document toDocument(Object value, AggregationOperationContext context) {
 
 			if (value instanceof List) {

spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/aggregation/AggregationFunctionExpressions.java

@@ -30,7 +30,7 @@
  * @author Christoph Strobl
  * @author Mark Paluch
  * @since 1.7
- * @deprecated since 1.10. Please use {@link AggregationExpressions} instead.
+ * @deprecated since 1.10. Please use {@link ArithmeticOperators} and {@link ComparisonOperators} instead.
  */
 @Deprecated
 public enum AggregationFunctionExpressions {