Author: aadamchik
Date: Mon Nov 27 19:07:00 2006
New Revision: 479867
URL: http://svn.apache.org/viewvc?view=rev&rev=479867
Log:
CAY-712: Allow modifying objects in validate* methods
(fix for 2.0 branch)
Added:
incubator/cayenne/main/branches/STABLE-2.0/cayenne/cayenne-java/src/tests/java/org/apache/cayenne/unit/util/ValidationDelegate.java
Modified:
incubator/cayenne/main/branches/STABLE-2.0/cayenne/cayenne-java/src/cayenne/java/org/apache/cayenne/access/ObjectStoreGraphDiff.java
incubator/cayenne/main/branches/STABLE-2.0/cayenne/cayenne-java/src/tests/java/org/apache/art/Artist.java
incubator/cayenne/main/branches/STABLE-2.0/cayenne/cayenne-java/src/tests/java/org/apache/cayenne/access/DataContextValidationTst.java
Modified:
incubator/cayenne/main/branches/STABLE-2.0/cayenne/cayenne-java/src/cayenne/java/org/apache/cayenne/access/ObjectStoreGraphDiff.java
URL:
http://svn.apache.org/viewvc/incubator/cayenne/main/branches/STABLE-2.0/cayenne/cayenne-java/src/cayenne/java/org/apache/cayenne/access/ObjectStoreGraphDiff.java?view=diff&rev=479867&r1=479866&r2=479867
==============================================================================
---
incubator/cayenne/main/branches/STABLE-2.0/cayenne/cayenne-java/src/cayenne/java/org/apache/cayenne/access/ObjectStoreGraphDiff.java
(original)
+++
incubator/cayenne/main/branches/STABLE-2.0/cayenne/cayenne-java/src/cayenne/java/org/apache/cayenne/access/ObjectStoreGraphDiff.java
Mon Nov 27 19:07:00 2006
@@ -20,6 +20,7 @@
package org.apache.cayenne.access;
import java.util.ArrayList;
+import java.util.Collection;
import java.util.Collections;
import java.util.Iterator;
import java.util.List;
@@ -67,6 +68,11 @@
ValidationResult result = new ValidationResult();
boolean noop = true;
+ // build a new collection for validation as validation methods may
result in
+ // ObjectStore modifications
+
+ Collection objectsToValidate = null;
+
Iterator it = getChangesByObjectId().entrySet().iterator();
while (it.hasNext()) {
@@ -74,10 +80,21 @@
if (!((ObjectDiff) entry.getValue()).isNoop()) {
- noop = false;
+ if (noop) {
+ noop = false;
+ objectsToValidate = new ArrayList();
+ }
// accessing objectMap directly to avoid unneeded
synchronization.
- DataObject object = (DataObject)
objectStore.getNodeNoSync(entry.getKey());
+
objectsToValidate.add(objectStore.getNodeNoSync(entry.getKey()));
+ }
+ }
+
+ if (objectsToValidate != null) {
+ Iterator validationIt = objectsToValidate.iterator();
+ while (validationIt.hasNext()) {
+ DataObject object = (DataObject) validationIt.next();
+
switch (object.getPersistenceState()) {
case PersistenceState.NEW:
object.validateForInsert(result);
@@ -90,10 +107,10 @@
break;
}
}
- }
-
- if (result.hasFailures()) {
- throw new ValidationException(result);
+
+ if (result.hasFailures()) {
+ throw new ValidationException(result);
+ }
}
return noop;
Modified:
incubator/cayenne/main/branches/STABLE-2.0/cayenne/cayenne-java/src/tests/java/org/apache/art/Artist.java
URL:
http://svn.apache.org/viewvc/incubator/cayenne/main/branches/STABLE-2.0/cayenne/cayenne-java/src/tests/java/org/apache/art/Artist.java?view=diff&rev=479867&r1=479866&r2=479867
==============================================================================
---
incubator/cayenne/main/branches/STABLE-2.0/cayenne/cayenne-java/src/tests/java/org/apache/art/Artist.java
(original)
+++
incubator/cayenne/main/branches/STABLE-2.0/cayenne/cayenne-java/src/tests/java/org/apache/art/Artist.java
Mon Nov 27 19:07:00 2006
@@ -1,10 +1,12 @@
package org.apache.art;
import org.apache.art.auto._Artist;
+import org.apache.cayenne.unit.util.ValidationDelegate;
import org.apache.cayenne.validation.ValidationResult;
public class Artist extends _Artist {
+ protected transient ValidationDelegate validationDelegate;
protected boolean validateForSaveCalled;
public boolean isValidateForSaveCalled() {
@@ -15,8 +17,15 @@
validateForSaveCalled = false;
}
+ public void setValidationDelegate(ValidationDelegate validationDelegate) {
+ this.validationDelegate = validationDelegate;
+ }
+
public void validateForSave(ValidationResult validationResult) {
validateForSaveCalled = true;
+ if (validationDelegate != null) {
+ validationDelegate.validateForSave(this, validationResult);
+ }
super.validateForSave(validationResult);
}
}
Modified:
incubator/cayenne/main/branches/STABLE-2.0/cayenne/cayenne-java/src/tests/java/org/apache/cayenne/access/DataContextValidationTst.java
URL:
http://svn.apache.org/viewvc/incubator/cayenne/main/branches/STABLE-2.0/cayenne/cayenne-java/src/tests/java/org/apache/cayenne/access/DataContextValidationTst.java?view=diff&rev=479867&r1=479866&r2=479867
==============================================================================
---
incubator/cayenne/main/branches/STABLE-2.0/cayenne/cayenne-java/src/tests/java/org/apache/cayenne/access/DataContextValidationTst.java
(original)
+++
incubator/cayenne/main/branches/STABLE-2.0/cayenne/cayenne-java/src/tests/java/org/apache/cayenne/access/DataContextValidationTst.java
Mon Nov 27 19:07:00 2006
@@ -17,11 +17,14 @@
* under the License.
****************************************************************/
-
package org.apache.cayenne.access;
import org.apache.art.Artist;
+import org.apache.art.Painting;
+import org.apache.cayenne.query.SelectQuery;
import org.apache.cayenne.unit.CayenneTestCase;
+import org.apache.cayenne.unit.util.ValidationDelegate;
+import org.apache.cayenne.validation.ValidationResult;
/**
* @author Andrus Adamchik
@@ -54,5 +57,36 @@
a2.setArtistName("a2");
context.commitChanges();
assertFalse(a2.isValidateForSaveCalled());
+ }
+
+ public void testValidationModifyingContext() throws Exception {
+ deleteTestData();
+
+ ValidationDelegate delegate = new ValidationDelegate() {
+
+ public void validateForSave(Object object, ValidationResult
validationResult) {
+
+ Artist a = (Artist) object;
+ Painting p = (Painting)
a.getObjectContext().newObject(Painting.class);
+ p.setPaintingTitle("XXX");
+ p.setToArtist(a);
+ }
+ };
+
+ DataContext context = createDataContext();
+
+ context.setValidatingObjectsOnCommit(true);
+ Artist a1 = (Artist) context.newObject(Artist.class);
+ a1.setValidationDelegate(delegate);
+ a1.setArtistName("a1");
+
+ // add another artist to ensure that modifying context works when more
than one
+ // object is committed
+ Artist a2 = (Artist) context.newObject(Artist.class);
+ a2.setValidationDelegate(delegate);
+ a2.setArtistName("a2");
+ context.commitChanges();
+
+ assertEquals(2, context.performQuery(new
SelectQuery(Painting.class)).size());
}
}
Added:
incubator/cayenne/main/branches/STABLE-2.0/cayenne/cayenne-java/src/tests/java/org/apache/cayenne/unit/util/ValidationDelegate.java
URL:
http://svn.apache.org/viewvc/incubator/cayenne/main/branches/STABLE-2.0/cayenne/cayenne-java/src/tests/java/org/apache/cayenne/unit/util/ValidationDelegate.java?view=auto&rev=479867
==============================================================================
---
incubator/cayenne/main/branches/STABLE-2.0/cayenne/cayenne-java/src/tests/java/org/apache/cayenne/unit/util/ValidationDelegate.java
(added)
+++
incubator/cayenne/main/branches/STABLE-2.0/cayenne/cayenne-java/src/tests/java/org/apache/cayenne/unit/util/ValidationDelegate.java
Mon Nov 27 19:07:00 2006
@@ -0,0 +1,31 @@
+/*****************************************************************
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements. See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership. The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License. You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied. See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
+ ****************************************************************/
+package org.apache.cayenne.unit.util;
+
+import org.apache.cayenne.validation.ValidationResult;
+
+/**
+ * A callback interface used by unit tests to implement custom validation
logic.
+ *
+ * @author Andrus Adamchik
+ */
+public interface ValidationDelegate {
+
+ void validateForSave(Object object, ValidationResult validationResult);
+}
