#80 extract collection callback, improve implementation

Author: aburmeis

src/main/java/com/arangodb/springframework/core/template/ArangoTemplate.java

@@ -36,6 +36,8 @@
 import com.arangodb.springframework.core.util.ArangoExceptionTranslator;
 import com.arangodb.springframework.core.util.MetadataUtils;
 import com.fasterxml.jackson.databind.node.JsonNodeFactory;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 import org.springframework.beans.BeansException;
 import org.springframework.context.ApplicationContext;
 import org.springframework.context.ApplicationContextAware;
@@ -74,6 +76,7 @@ public class ArangoTemplate implements ArangoOperations, CollectionCallback, App
     private static final String REPSERT_QUERY = "LET doc = @doc " + REPSERT_QUERY_BODY;
     private static final String REPSERT_MANY_QUERY = "FOR doc IN @docs " + REPSERT_QUERY_BODY;
 
+	private static final Logger LOGGER = LoggerFactory.getLogger(ArangoTemplate.class);
     private static final SpelExpressionParser PARSER = new SpelExpressionParser();
 
     private volatile ArangoDBVersion version;
@@ -163,9 +166,11 @@ private ArangoCollection _collection(final String name, final ArangoPersistentEn
         final ArangoCollection collection = value.getCollection();
         if (persistentEntity != null && !entities.contains(entityClass)) {
             value.addEntityClass(entityClass);
-			if (!transactional) {
-            ensureCollectionIndexes(collection(collection), persistentEntity);
-        }
+			if (transactional) {
+				LOGGER.debug("Not ensuring any indexes of collection {} for {} during transaction", collection.name(), entityClass);
+			} else {
+            	ensureCollectionIndexes(collection(collection), persistentEntity);
+        	}
 		}
         return collection;
     }

src/main/java/com/arangodb/springframework/core/template/CollectionCallback.java

@@ -0,0 +1,61 @@
+/*
+ * DISCLAIMER
+ *
+ * Copyright 2017 ArangoDB GmbH, Cologne, Germany
+ *
+ * Licensed 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.
+ *
+ * Copyright holder is ArangoDB GmbH, Cologne, Germany
+ */
+
+package com.arangodb.springframework.core.template;
+
+import com.arangodb.springframework.core.ArangoOperations;
+import com.arangodb.springframework.core.CollectionOperations;
+import org.springframework.dao.DataAccessException;
+
+/**
+ * Internal interface to handle collection operations.
+ * Typically implemented by same class as {@link com.arangodb.springframework.core.ArangoOperations}.
+ */
+public interface CollectionCallback {
+
+    /**
+     * @see com.arangodb.springframework.core.ArangoOperations#collection(Class)
+     */
+    CollectionOperations collection(Class<?> type) throws DataAccessException;
+
+    /**
+     * @see com.arangodb.springframework.core.ArangoOperations#collection(String)
+     */
+    CollectionOperations collection(String name) throws DataAccessException;
+
+
+    static CollectionCallback fromOperations(ArangoOperations operations) {
+        if (operations instanceof CollectionCallback) {
+            return (CollectionCallback) operations;
+        }
+        return new CollectionCallback() {
+            @Override
+            public CollectionOperations collection(Class<?> type) {
+                return operations.collection(type);
+            }
+
+            @Override
+            public CollectionOperations collection(String name) {
+                return operations.collection(name);
+            }
+        };
+    }
+
+}

src/main/java/com/arangodb/springframework/core/template/DefaultUserOperation.java

@@ -30,7 +30,6 @@
 import com.arangodb.entity.UserEntity;
 import com.arangodb.model.UserCreateOptions;
 import com.arangodb.model.UserUpdateOptions;
-import com.arangodb.springframework.core.CollectionOperations;
 import com.arangodb.springframework.core.UserOperations;
 
 /**
@@ -39,12 +38,6 @@
  */
 public class DefaultUserOperation implements UserOperations {
 
-	public interface CollectionCallback {
-		CollectionOperations collection(Class<?> type);
-
-		CollectionOperations collection(String name);
-	}
-
 	private final ArangoDatabase db;
 	private final String username;
 	private final PersistenceExceptionTranslator exceptionTranslator;