Polish Javadoc and imports in datetime package

Issue: SPR-7121

spring-context/src/main/java/org/springframework/format/datetime/DateFormatterRegistrar.java

@@ -23,19 +23,18 @@ import org.springframework.core.convert.converter.Converter;
 import org.springframework.core.convert.converter.ConverterRegistry;
 import org.springframework.format.FormatterRegistrar;
 import org.springframework.format.FormatterRegistry;
-import org.springframework.format.datetime.joda.JodaTimeFormatterRegistrar;
 import org.springframework.util.Assert;
 
 /**
  * Configures Date formatting for use with Spring.
  * <p>
  * Designed for direct instantiation but also exposes the static
- * {@link #addDateConverters(ConverterRegistry)} utility method for ad hoc use against any
+ * {@link #addDateConverters(ConverterRegistry)} utility method for ad hoc use
- * {@code ConverterRegistry} instance.
+ * against any {@code ConverterRegistry} instance.
  *
  * @author Phillip Webb
  * @since 3.2
- * @see JodaTimeFormatterRegistrar
+ * @see org.springframework.format.datetime.joda.JodaTimeFormatterRegistrar
  * @see FormatterRegistrar#registerFormatters
  */
 public class DateFormatterRegistrar implements FormatterRegistrar {
@@ -52,7 +51,7 @@ public class DateFormatterRegistrar implements FormatterRegistrar {
 	}
 
 	/**
-	 * Set the date formatter to register. If not specified default {@link DateFormatter}
+	 * Set the date formatter to register. If not specified the default {@link DateFormatter}
 	 * will be used. This method can be used if additional formatter configuration is
 	 * required.
 	 * @param dateFormatter the date formatter

spring-context/src/main/java/org/springframework/format/datetime/DateTimeFormatAnnotationFormatterFactory.java

@@ -28,15 +28,15 @@ import org.springframework.format.Formatter;
 import org.springframework.format.Parser;
 import org.springframework.format.Printer;
 import org.springframework.format.annotation.DateTimeFormat;
-import org.springframework.format.datetime.joda.JodaDateTimeFormatAnnotationFormatterFactory;
 import org.springframework.util.StringValueResolver;
 
 /**
- * Formats fields annotated with the {@link DateTimeFormat} annotation.
+ * Formats fields annotated with the {@link DateTimeFormat} annotation using
+ * a {@link DateFormatter}.
  *
  * @author Phillip Webb
- * @see JodaDateTimeFormatAnnotationFormatterFactory
  * @since 3.2
+ * @see org.springframework.format.datetime.joda.JodaDateTimeFormatAnnotationFormatterFactory
  */
 public class DateTimeFormatAnnotationFormatterFactory implements
 		AnnotationFormatterFactory<DateTimeFormat>, EmbeddedValueResolverAware {

spring-context/src/main/java/org/springframework/format/datetime/joda/DateTimeFormatterFactory.java

@@ -22,13 +22,14 @@ import org.joda.time.DateTimeZone;
 import org.joda.time.format.DateTimeFormat;
 import org.joda.time.format.DateTimeFormatter;
 import org.joda.time.format.ISODateTimeFormat;
+
 import org.springframework.beans.factory.FactoryBean;
 import org.springframework.format.annotation.DateTimeFormat.ISO;
 import org.springframework.util.StringUtils;
 
 /**
  * {@link FactoryBean} that creates a Joda {@link DateTimeFormatter}. Formatters will be
- * created using the defined {@link #setPattern(String) pattern}, {@link #setIso(ISO) ISO}
+ * created using the defined {@link #setPattern(String) pattern}, {@link #setIso(ISO) ISO},
  * or {@link #setStyle(String) style} (considered in that order).
  *
  * @author Phillip Webb
@@ -48,13 +49,13 @@ public class DateTimeFormatterFactory implements FactoryBean<DateTimeFormatter>
 
 
 	/**
-	 * Create a new {@link DateTimeFormatterFactory} instance.
+	 * Create a new {@code DateTimeFormatterFactory} instance.
 	 */
 	public DateTimeFormatterFactory() {
 	}
 
 	/**
-	 * Create a new {@link DateTimeFormatterFactory} instance.
+	 * Create a new {@code DateTimeFormatterFactory} instance.
 	 * @param pattern the pattern to use to format date values
 	 */
 	public DateTimeFormatterFactory(String pattern) {
@@ -75,8 +76,8 @@ public class DateTimeFormatterFactory implements FactoryBean<DateTimeFormatter>
 	}
 
 	/**
-	 * Get a new DateTimeFormatter using this factory. If no specific
+	 * Get a new {@code DateTimeFormatter} using this factory. If no specific
-	 * {@link #setStyle(String) style} {@link #setIso(ISO) ISO} or
+	 * {@link #setStyle(String) style}, {@link #setIso(ISO) ISO}, or
 	 * {@link #setPattern(String) pattern} have been defined the
 	 * {@link DateTimeFormat#mediumDateTime() medium date time format} will be used.
 	 * @return a new date time formatter
@@ -88,9 +89,9 @@ public class DateTimeFormatterFactory implements FactoryBean<DateTimeFormatter>
 	}
 
 	/**
-	 * Get a new DateTimeFormatter using this factory. If no specific
+	 * Get a new {@code DateTimeFormatter} using this factory. If no specific
-	 * {@link #setStyle(String) style} {@link #setIso(ISO) ISO} or
+	 * {@link #setStyle(String) style}, {@link #setIso(ISO) ISO}, or
-	 * {@link #setPattern(String) pattern} have been defined the specific
+	 * {@link #setPattern(String) pattern} have been defined the supplied
 	 * {@code fallbackFormatter} will be used.
 	 * @param fallbackFormatter the fall-back formatter to use when no specific factory
 	 *        properties have been set (can be {@code null}).
@@ -125,7 +126,7 @@ public class DateTimeFormatterFactory implements FactoryBean<DateTimeFormatter>
 
 
 	/**
-	 * Set the TimeZone to normalize the date values into, if any.
+	 * Set the {@code TimeZone} to normalize the date values into, if any.
 	 * @param timeZone the time zone
 	 */
 	public void setTimeZone(TimeZone timeZone) {
@@ -133,16 +134,16 @@ public class DateTimeFormatterFactory implements FactoryBean<DateTimeFormatter>
 	}
 
 	/**
-	 * Set the two character to use to format date values. The first character used for
+	 * Set the two characters to use to format date values. The first character is used for
-	 * the date style, the second is for the time style. Supported characters are
+	 * the date style; the second is for the time style. Supported characters are:
 	 * <ul>
 	 * <li>'S' = Small</li>
 	 * <li>'M' = Medium</li>
 	 * <li>'L' = Long</li>
 	 * <li>'F' = Full</li>
 	 * <li>'-' = Omitted</li>
-	 * <ul>
+	 * </ul>
-	 * This method mimics the styles supported by Joda Time.
+	 * <p>This method mimics the styles supported by Joda Time.
 	 * @param style two characters from the set {"S", "M", "L", "F", "-"}
 	 */
 	public void setStyle(String style) {
@@ -150,8 +151,8 @@ public class DateTimeFormatterFactory implements FactoryBean<DateTimeFormatter>
 	}
 
 	/**
-	 * Set the ISO format used for this date.
+	 * Set the ISO format used to format date values.
-	 * @param iso the iso format
+	 * @param iso the ISO format
 	 */
 	public void setIso(ISO iso) {
 		this.iso = iso;
@@ -164,4 +165,5 @@ public class DateTimeFormatterFactory implements FactoryBean<DateTimeFormatter>
 	public void setPattern(String pattern) {
 		this.pattern = pattern;
 	}
+
 }

spring-context/src/main/java/org/springframework/format/datetime/joda/JodaDateTimeFormatAnnotationFormatterFactory.java

@@ -29,6 +29,7 @@ import org.joda.time.LocalTime;
 import org.joda.time.ReadableInstant;
 import org.joda.time.ReadablePartial;
 import org.joda.time.format.DateTimeFormatter;
+
 import org.springframework.context.EmbeddedValueResolverAware;
 import org.springframework.format.AnnotationFormatterFactory;
 import org.springframework.format.Parser;

spring-context/src/main/java/org/springframework/format/datetime/joda/JodaTimeFormatterRegistrar.java

@@ -27,15 +27,15 @@ import org.joda.time.LocalTime;
 import org.joda.time.ReadableInstant;
 import org.joda.time.format.DateTimeFormat;
 import org.joda.time.format.DateTimeFormatter;
+
 import org.springframework.format.FormatterRegistrar;
 import org.springframework.format.FormatterRegistry;
 import org.springframework.format.Parser;
 import org.springframework.format.Printer;
 import org.springframework.format.annotation.DateTimeFormat.ISO;
-import org.springframework.format.datetime.DateFormatterRegistrar;
 
 /**
- * Configures Joda Time's Formatting system for use with Spring.
+ * Configures Joda Time's formatting system for use with Spring.
  *
  * @author Keith Donald
  * @author Juergen Hoeller
@@ -46,7 +46,7 @@ import org.springframework.format.datetime.DateFormatterRegistrar;
  * @see #setDateTimeStyle
  * @see #setUseIsoFormat
  * @see FormatterRegistrar#registerFormatters
- * @see DateFormatterRegistrar
+ * @see org.springframework.format.datetime.DateFormatterRegistrar
  */
 public class JodaTimeFormatterRegistrar implements FormatterRegistrar {
 
@@ -107,8 +107,8 @@ public class JodaTimeFormatterRegistrar implements FormatterRegistrar {
 
 	/**
 	 * Set the formatter that will be used for objects representing date values.
-	 * This formatter will be used for the {@link LocalDate} type. When specified
+	 * <p>This formatter will be used for the {@link LocalDate} type. When specified
-	 * {@link #setDateStyle(String) dateStyle} and
+	 * the {@link #setDateStyle(String) dateStyle} and
 	 * {@link #setUseIsoFormat(boolean) useIsoFormat} properties will be ignored.
 	 * @param formatter the formatter to use
 	 * @see #setTimeFormatter(DateTimeFormatter)
@@ -120,9 +120,9 @@ public class JodaTimeFormatterRegistrar implements FormatterRegistrar {
 	}
 
 	/**
-	 * Set the formatter that will be used for objects representing date values.
+	 * Set the formatter that will be used for objects representing time values.
-	 * This formatter will be used for the {@link LocalTime} type. When specified
+	 * <p>This formatter will be used for the {@link LocalTime} type. When specified
-	 * {@link #setTimeStyle(String) timeStyle} and
+	 * the {@link #setTimeStyle(String) timeStyle} and
 	 * {@link #setUseIsoFormat(boolean) useIsoFormat} properties will be ignored.
 	 * @param formatter the formatter to use
 	 * @see #setDateFormatter(DateTimeFormatter)
@@ -135,9 +135,9 @@ public class JodaTimeFormatterRegistrar implements FormatterRegistrar {
 
 	/**
 	 * Set the formatter that will be used for objects representing date and time values.
-	 * This formatter will be used for {@link LocalDateTime}, {@link ReadableInstant},
+	 * <p>This formatter will be used for {@link LocalDateTime}, {@link ReadableInstant},
 	 * {@link Date} and {@link Calendar} types. When specified
-	 * {@link #setDateTimeStyle(String) dateTimeStyle} and
+	 * the {@link #setDateTimeStyle(String) dateTimeStyle} and
 	 * {@link #setUseIsoFormat(boolean) useIsoFormat} properties will be ignored.
 	 * @param formatter the formatter to use
 	 * @see #setDateFormatter(DateTimeFormatter)