diff --git a/spring-data-commons-aspects/src/main/java/org/springframework/persistence/support/ChangeSetConstructorEntityInstantiator.java b/spring-data-commons-aspects/src/main/java/org/springframework/persistence/support/ChangeSetConstructorEntityInstantiator.java index 47fb10322..83f11f12d 100644 --- a/spring-data-commons-aspects/src/main/java/org/springframework/persistence/support/ChangeSetConstructorEntityInstantiator.java +++ b/spring-data-commons-aspects/src/main/java/org/springframework/persistence/support/ChangeSetConstructorEntityInstantiator.java @@ -9,7 +9,6 @@ package org.springframework.persistence.support; */ public class ChangeSetConstructorEntityInstantiator extends AbstractConstructorEntityInstantiator{ - @Override protected void setState(ChangeSetBacked entity, ChangeSet cs) { entity.setChangeSet(cs); } diff --git a/spring-data-commons-core/src/main/java/org/springframework/data/mapping/BasicMappingConfigurationBuilder.java b/spring-data-commons-core/src/main/java/org/springframework/data/mapping/BasicMappingConfigurationBuilder.java index 6fe9dbe64..368f29383 100644 --- a/spring-data-commons-core/src/main/java/org/springframework/data/mapping/BasicMappingConfigurationBuilder.java +++ b/spring-data-commons-core/src/main/java/org/springframework/data/mapping/BasicMappingConfigurationBuilder.java @@ -45,7 +45,6 @@ public class BasicMappingConfigurationBuilder implements MappingConfigurationBui protected static ConcurrentMap, BeanInfo> beanInfo = new ConcurrentHashMap, BeanInfo>(); protected Logger log = LoggerFactory.getLogger(getClass()); - @Override public boolean isPersistentEntity(Class type) { if (type.isAnnotationPresent(Persistent.class)) { return true; @@ -64,12 +63,10 @@ public class BasicMappingConfigurationBuilder implements MappingConfigurationBui return false; } - @Override public PersistentEntity createPersistentEntity(TypeInformation typeInformation, MappingContext mappingContext) throws MappingConfigurationException { return new BasicPersistentEntity(mappingContext, typeInformation); } - @Override public boolean isPersistentProperty(Field field, PropertyDescriptor descriptor) throws MappingConfigurationException { if (UNMAPPED_FIELDS.contains(field.getName()) || isTransient(field)) { return false; @@ -77,13 +74,11 @@ public class BasicMappingConfigurationBuilder implements MappingConfigurationBui return true; } - @Override public PersistentProperty createPersistentProperty(Field field, PropertyDescriptor descriptor, TypeInformation information) throws MappingConfigurationException { return new BasicPersistentProperty(field, descriptor, information); } @SuppressWarnings({"unchecked"}) - @Override public PreferredConstructor getPreferredConstructor(Class type) throws MappingConfigurationException { // Find the right constructor PreferredConstructor preferredConstructor = null; @@ -137,7 +132,6 @@ public class BasicMappingConfigurationBuilder implements MappingConfigurationBui return preferredConstructor; } - @Override public boolean isAssociation(Field field, PropertyDescriptor descriptor) throws MappingConfigurationException { if (!isTransient(field)) { if (field.isAnnotationPresent(Reference.class)) { @@ -152,7 +146,6 @@ public class BasicMappingConfigurationBuilder implements MappingConfigurationBui return false; } - @Override public Association createAssociation(PersistentProperty property) { // Only support uni-directional associations in the Basic configuration Association association = new Association(property, null); diff --git a/spring-data-commons-core/src/main/java/org/springframework/data/mapping/BasicMappingContext.java b/spring-data-commons-core/src/main/java/org/springframework/data/mapping/BasicMappingContext.java index 0829ec6a9..95f6d5ac8 100644 --- a/spring-data-commons-core/src/main/java/org/springframework/data/mapping/BasicMappingContext.java +++ b/spring-data-commons-core/src/main/java/org/springframework/data/mapping/BasicMappingContext.java @@ -89,12 +89,10 @@ public class BasicMappingContext implements MappingContext, InitializingBean, Ap this.customSimpleTypes = customSimpleTypes; } - @Override public void setApplicationContext(ApplicationContext applicationContext) throws BeansException { this.applicationContext = applicationContext; } - @Override public Collection> getPersistentEntities() { return persistentEntities.values(); } @@ -102,13 +100,11 @@ public class BasicMappingContext implements MappingContext, InitializingBean, Ap /* (non-Javadoc) * @see org.springframework.data.mapping.model.MappingContext#getPersistentEntity(java.lang.Class) */ - @Override public PersistentEntity getPersistentEntity(Class type) { return getPersistentEntity(new ClassTypeInformation(type)); } @SuppressWarnings({"unchecked"}) - @Override public PersistentEntity getPersistentEntity(TypeInformation type) { return (PersistentEntity) persistentEntities.get(type); } @@ -135,7 +131,6 @@ public class BasicMappingContext implements MappingContext, InitializingBean, Ap ReflectionUtils.doWithFields(type, new FieldCallback() { - @Override public void doWith(Field field) throws IllegalArgumentException, IllegalAccessException { try { PropertyDescriptor descriptor = descriptors.get(field.getName()); @@ -222,12 +217,10 @@ public class BasicMappingContext implements MappingContext, InitializingBean, Ap return information == null || MappingBeanHelper.isSimpleType(information.getType()) ? null : information; } - @Override public PersistentEntity addPersistentEntity(Class type) { return addPersistentEntity(new ClassTypeInformation(type)); } - @Override public void addEntityValidator(PersistentEntity entity, Validator validator) { List v = validators.get(entity); if (null == v) { @@ -237,37 +230,30 @@ public class BasicMappingContext implements MappingContext, InitializingBean, Ap v.add(validator); } - @Override public void addTypeConverter(Converter converter) { conversionService.addConverter(converter); } - @Override public ConversionService getConversionService() { return conversionService; } - @Override public ConverterRegistry getConverterRegistry() { return conversionService; } - @Override public List getEntityValidators(PersistentEntity entity) { return validators.get(entity); } - @Override public MappingConfigurationBuilder getMappingConfigurationBuilder() { return builder; } - @Override public void setMappingConfigurationBuilder(MappingConfigurationBuilder builder) { this.builder = builder; } - @Override public boolean isPersistentEntity(Object value) { if (null != value) { Class clazz; @@ -281,7 +267,6 @@ public class BasicMappingContext implements MappingContext, InitializingBean, Ap return false; } - @Override public void afterPropertiesSet() throws Exception { Assert.notNull(builder, "No mapping configuration provider configured."); } diff --git a/spring-data-commons-core/src/main/java/org/springframework/data/mapping/BasicPersistentEntity.java b/spring-data-commons-core/src/main/java/org/springframework/data/mapping/BasicPersistentEntity.java index aa13cb156..04ebd1aab 100644 --- a/spring-data-commons-core/src/main/java/org/springframework/data/mapping/BasicPersistentEntity.java +++ b/spring-data-commons-core/src/main/java/org/springframework/data/mapping/BasicPersistentEntity.java @@ -49,47 +49,38 @@ public class BasicPersistentEntity implements PersistentEntity { this.information = information; } - @Override public PreferredConstructor getPreferredConstructor() { return preferredConstructor; } - @Override public void setPreferredConstructor(PreferredConstructor constructor) { this.preferredConstructor = constructor; } - @Override public String getName() { return type.getName(); } - @Override public PersistentProperty getIdProperty() { return idProperty; } - @Override public void setIdProperty(PersistentProperty property) { idProperty = property; } - @Override public Collection getPersistentProperties() { return persistentProperties.values(); } - @Override public void addPersistentProperty(PersistentProperty property) { persistentProperties.put(property.getName(), property); } - @Override public Collection getAssociations() { return associations.values(); } - @Override public void addAssociation(Association association) { associations.put(association.getInverse().getName(), association); } @@ -98,32 +89,26 @@ public class BasicPersistentEntity implements PersistentEntity { return persistentProperties.get(name); } - @Override public Class getType() { return type; } - @Override public TypeInformation getPropertyInformation() { return information; } - @Override public Collection getPersistentPropertyNames() { return persistentProperties.keySet(); } - @Override public MappingContext getMappingContext() { return mappingContext; } - @Override public void afterPropertiesSet() throws Exception { } - @Override public void doWithProperties(PropertyHandler handler) { for (PersistentProperty property : persistentProperties.values()) { if (!property.isTransient() && !property.isAssociation() && !property.isIdProperty()) { @@ -132,7 +117,6 @@ public class BasicPersistentEntity implements PersistentEntity { } } - @Override public void doWithAssociations(AssociationHandler handler) { for (Association association : associations.values()) { handler.doWithAssociation(association); diff --git a/spring-data-commons-core/src/main/java/org/springframework/data/mapping/BasicPersistentProperty.java b/spring-data-commons-core/src/main/java/org/springframework/data/mapping/BasicPersistentProperty.java index 7f973ef83..e488ae8a3 100644 --- a/spring-data-commons-core/src/main/java/org/springframework/data/mapping/BasicPersistentProperty.java +++ b/spring-data-commons-core/src/main/java/org/springframework/data/mapping/BasicPersistentProperty.java @@ -64,75 +64,60 @@ public class BasicPersistentProperty implements PersistentProperty { } } - @Override public Object getOwner() { return owner; } - @Override public void setOwner(Object owner) { if (null != owner && owner.getClass().isAssignableFrom(PersistentEntity.class)) { this.owner = (PersistentEntity) owner; } } - @Override public String getName() { return name; } - @Override public Class getType() { return information.getType(); } - @Override public TypeInformation getTypeInformation() { return information; } - @Override public PropertyDescriptor getPropertyDescriptor() { return propertyDescriptor; } - @Override public Field getField() { return field; } - @Override public Value getValueAnnotation() { return value; } - @Override public boolean isTransient() { return isTransient; } - @Override public boolean isAssociation() { return null != association; } - @Override public Association getAssociation() { return association; } - @Override public void setAssociation(Association association) { this.association = association; } - @Override public boolean isCollection() { return Collection.class.isAssignableFrom(getType()) || isArray(); } - - @Override public boolean isMap() { return Map.class.isAssignableFrom(getType()); } @@ -140,12 +125,10 @@ public class BasicPersistentProperty implements PersistentProperty { /* (non-Javadoc) * @see org.springframework.data.mapping.model.PersistentProperty#isArray() */ - @Override public boolean isArray() { return getType().isArray(); } - @Override public boolean isComplexType() { if (isCollection() || isArray()) { return !MappingBeanHelper.isSimpleType(getComponentType()); @@ -154,12 +137,10 @@ public class BasicPersistentProperty implements PersistentProperty { } } - @Override public boolean isEntity() { return isComplexType() && !isTransient() && !isCollection() && !isMap(); } - @Override public Class getComponentType() { return isMap() || isCollection() ? information.getComponentType().getType() : null; } @@ -167,12 +148,10 @@ public class BasicPersistentProperty implements PersistentProperty { /* (non-Javadoc) * @see org.springframework.data.mapping.model.PersistentProperty#getMapValueType() */ - @Override public Class getMapValueType() { return isMap() ? information.getMapValueType().getType() : null; } - @Override public boolean isIdProperty() { return field.isAnnotationPresent(Id.class); } diff --git a/spring-data-commons-core/src/main/java/org/springframework/data/repository/support/RepositoryFactoryBeanSupport.java b/spring-data-commons-core/src/main/java/org/springframework/data/repository/support/RepositoryFactoryBeanSupport.java index 792bb9a72..ce574b292 100644 --- a/spring-data-commons-core/src/main/java/org/springframework/data/repository/support/RepositoryFactoryBeanSupport.java +++ b/spring-data-commons-core/src/main/java/org/springframework/data/repository/support/RepositoryFactoryBeanSupport.java @@ -92,7 +92,6 @@ public abstract class RepositoryFactoryBeanSupport, /* (non-Javadoc) * @see org.springframework.data.repository.support.RepositoryFactoryInformation#getRepositoryInterface() */ - @Override public Class getRepositoryInterface() { return repositoryInterface; diff --git a/spring-data-commons-core/src/main/java/org/springframework/data/util/TypeDiscoverer.java b/spring-data-commons-core/src/main/java/org/springframework/data/util/TypeDiscoverer.java index cbaaf35f4..8ecd493ee 100644 --- a/spring-data-commons-core/src/main/java/org/springframework/data/util/TypeDiscoverer.java +++ b/spring-data-commons-core/src/main/java/org/springframework/data/util/TypeDiscoverer.java @@ -147,7 +147,6 @@ class TypeDiscoverer implements TypeInformation { /* (non-Javadoc) * @see org.springframework.data.util.TypeInformation#getMapValueType() */ - @Override public TypeInformation getMapValueType() { if (!Map.class.isAssignableFrom(getType())) { @@ -161,7 +160,6 @@ class TypeDiscoverer implements TypeInformation { /* (non-Javadoc) * @see org.springframework.data.util.TypeInformation#getComponentType() */ - @Override public TypeInformation getComponentType() { if (!(Map.class.isAssignableFrom(getType()) || Collection.class.isAssignableFrom(getType()))) { diff --git a/spring-data-commons-core/src/test/java/org/springframework/data/repository/support/AbstractEntityInformationUnitTests.java b/spring-data-commons-core/src/test/java/org/springframework/data/repository/support/AbstractEntityInformationUnitTests.java index aaac8c846..56f04a315 100644 --- a/spring-data-commons-core/src/test/java/org/springframework/data/repository/support/AbstractEntityInformationUnitTests.java +++ b/spring-data-commons-core/src/test/java/org/springframework/data/repository/support/AbstractEntityInformationUnitTests.java @@ -70,7 +70,6 @@ public class AbstractEntityInformationUnitTests { /* (non-Javadoc) * @see org.springframework.data.repository.support.EntityInformation#getIdType() */ - @Override public Class getIdType() { return Serializable.class;