Browse Source

reordered interface operations for clarity

pull/7/head
Keith Donald 14 years ago
parent
commit
ff7f3aebea
  1. 16
      org.springframework.context/src/main/java/org/springframework/format/FormatterRegistry.java
  2. 10
      org.springframework.context/src/main/java/org/springframework/format/support/FormattingConversionService.java

16
org.springframework.context/src/main/java/org/springframework/format/FormatterRegistry.java

@ -29,6 +29,14 @@ import org.springframework.core.convert.converter.ConverterRegistry; @@ -29,6 +29,14 @@ import org.springframework.core.convert.converter.ConverterRegistry;
*/
public interface FormatterRegistry extends ConverterRegistry {
/**
* Adds a Formatter to format fields of a specific type.
* The field type is implied by the parameterized Formatter instance.
* @param formatter the formatter to add
* @see #addFormatterForFieldType(Class, Formatter)
*/
void addFormatter(Formatter<?> formatter);
/**
* Adds a Formatter to format fields of the given type.
* <p>On print, if the Formatter's type T is declared and <code>fieldType</code> is not assignable to T,
@ -40,14 +48,6 @@ public interface FormatterRegistry extends ConverterRegistry { @@ -40,14 +48,6 @@ public interface FormatterRegistry extends ConverterRegistry {
*/
void addFormatterForFieldType(Class<?> fieldType, Formatter<?> formatter);
/**
* Adds a Formatter to format fields of a specific type.
* The field type is implied by the parameterized Formatter instance.
* @param formatter the formatter to add
* @see #addFormatterForFieldType(Class, Formatter)
*/
void addFormatter(Formatter<?> formatter);
/**
* Adds a Printer/Parser pair to format fields of a specific type.
* The formatter will delegate to the specified <code>printer</code> for printing

10
org.springframework.context/src/main/java/org/springframework/format/support/FormattingConversionService.java

@ -63,11 +63,6 @@ public class FormattingConversionService extends GenericConversionService @@ -63,11 +63,6 @@ public class FormattingConversionService extends GenericConversionService
}
public void addFormatterForFieldType(Class<?> fieldType, Formatter<?> formatter) {
addConverter(new PrinterConverter(fieldType, formatter, this));
addConverter(new ParserConverter(fieldType, formatter, this));
}
public void addFormatter(Formatter<?> formatter) {
final Class<?> fieldType = GenericTypeResolver.resolveTypeArgument(formatter.getClass(), Formatter.class);
if (fieldType == null) {
@ -76,6 +71,11 @@ public class FormattingConversionService extends GenericConversionService @@ -76,6 +71,11 @@ public class FormattingConversionService extends GenericConversionService
}
addFormatterForFieldType(fieldType, formatter);
}
public void addFormatterForFieldType(Class<?> fieldType, Formatter<?> formatter) {
addConverter(new PrinterConverter(fieldType, formatter, this));
addConverter(new ParserConverter(fieldType, formatter, this));
}
public void addFormatterForFieldType(Class<?> fieldType, Printer<?> printer, Parser<?> parser) {
addConverter(new PrinterConverter(fieldType, printer, this));

Loading…
Cancel
Save