Spring/spring-data-rest
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Fixed a bug introduced by a change in how the validators were looked up. Not sure why I made the original change to begin with but the functionality has been been reverted to its previous behavior.

Browse Source
This commit is contained in:
Jon Brisbin
2012-10-02 10:06:53 -05:00
committed by Jon Brisbin
parent ce466d00ab
commit 1100693c53
2 changed files with 2 additions and 21 deletions
Download Patch File Download Diff File Expand all files Collapse all files

21
spring-data-rest-repository/src/main/java/org/springframework/data/rest/repository/context/ValidatingRepositoryEventListener.java
View File

@@ -121,26 +121,7 @@ public class ValidatingRepositoryEventListener
o,
repositoryMetadataFor(domainType).entityMetadata());
String eventName = null;
if("beforeSave".equals(event)) {
eventName = "before" + domainType.getSimpleName() + "Save";
} else if("afterSave".equals(event)) {
eventName = "after" + domainType.getSimpleName() + "Save";
} else if("beforeLinkSave".equals(event)) {
eventName = "before" + domainType.getSimpleName() + "LinkSave";
} else if("afterLinkSave".equals(event)) {
eventName = "after" + domainType.getSimpleName() + "LinkSave";
} else if("beforeDelete".equals(event)) {
eventName = "before" + domainType.getSimpleName() + "Delete";
} else if("afterDelete".equals(event)) {
eventName = "after" + domainType.getSimpleName() + "Delete";
}
if(null == eventName) {
return errors;
}
Collection<Validator> validators = this.validators.get(eventName);
Collection<Validator> validators = this.validators.get(event);
if(null != validators) {
for(Validator v : validators) {
if(v.supports(o.getClass())) {

2
spring-data-rest-webmvc/src/test/java/org/springframework/data/rest/test/ApplicationRestConfig.java
View File

@@ -88,7 +88,7 @@ public class ApplicationRestConfig {
*
* @return
*/
@Bean public PersonValidator beforePersonSaveValidator() {
@Bean public PersonValidator beforeSavePersonValidator() {
return new PersonValidator();
}