From 1b97517ce6544f673fb76998ee6654dbd1cd550f Mon Sep 17 00:00:00 2001
From: Derek Hulley <derek.hulley@alfresco.com>
Date: Thu, 14 Jun 2007 23:58:44 +0000
Subject: [PATCH] Removed beforeUpdateParent policy calls for association
 modifications.

git-svn-id: https://svn.alfresco.com/repos/alfresco-enterprise/alfresco/HEAD/root@5968 c4b6b30b-aa2e-2d43-bbcb-ca4b014f7261
---
 .../alfresco/repo/node/db/DbNodeServiceImpl.java   | 14 --------------
 1 file changed, 14 deletions(-)

diff --git a/source/java/org/alfresco/repo/node/db/DbNodeServiceImpl.java b/source/java/org/alfresco/repo/node/db/DbNodeServiceImpl.java
index 50de3074f2..d088021b8c 100644
--- a/source/java/org/alfresco/repo/node/db/DbNodeServiceImpl.java
+++ b/source/java/org/alfresco/repo/node/db/DbNodeServiceImpl.java
@@ -297,7 +297,6 @@ public class DbNodeServiceImpl extends AbstractNodeServiceImpl
         }
         
         // Invoke policy behaviour
-        invokeBeforeUpdateNode(parentRef);
         invokeBeforeCreateNode(parentRef, assocTypeQName, assocQName, nodeTypeQName);
         invokeBeforeCreateNodeAssociation(parentRef, assocTypeQName, assocQName);
         
@@ -409,8 +408,6 @@ public class DbNodeServiceImpl extends AbstractNodeServiceImpl
         // get the primary parent assoc
         ChildAssoc oldAssoc = nodeDaoService.getPrimaryParentAssoc(nodeToMove);
         ChildAssociationRef oldAssocRef = oldAssoc.getChildAssocRef();
-        // get the old parent
-        Node oldParentNode = oldAssoc.getParent();
         
         boolean movingStore = !nodeToMoveRef.getStoreRef().equals(newParentRef.getStoreRef());
         
@@ -428,8 +425,6 @@ public class DbNodeServiceImpl extends AbstractNodeServiceImpl
         {
             invokeBeforeDeleteChildAssociation(oldAssocRef);
             invokeBeforeCreateChildAssociation(newParentRef, nodeToMoveRef, assocTypeQName, assocQName);
-            invokeBeforeUpdateNode(oldParentNode.getNodeRef());    // old parent will be updated
-            invokeBeforeUpdateNode(newParentRef);                  // new parent ditto
         }
         
         // remove the child assoc from the old parent
@@ -743,7 +738,6 @@ public class DbNodeServiceImpl extends AbstractNodeServiceImpl
     public ChildAssociationRef addChild(NodeRef parentRef, NodeRef childRef, QName assocTypeQName, QName assocQName)
     {
         // Invoke policy behaviours
-        invokeBeforeUpdateNode(parentRef);
         invokeBeforeCreateChildAssociation(parentRef, childRef, assocTypeQName, assocQName);
         
         // get the parent node and ensure that it is a container node
@@ -1262,9 +1256,6 @@ public class DbNodeServiceImpl extends AbstractNodeServiceImpl
     public AssociationRef createAssociation(NodeRef sourceRef, NodeRef targetRef, QName assocTypeQName)
             throws InvalidNodeRefException, AssociationExistsException
     {
-        // Invoke policy behaviours
-        invokeBeforeUpdateNode(sourceRef);
-        
         Node sourceNode = getNodeNotNull(sourceRef);
         Node targetNode = getNodeNotNull(targetRef);
         // see if it exists
@@ -1278,7 +1269,6 @@ public class DbNodeServiceImpl extends AbstractNodeServiceImpl
         AssociationRef assocRef = assoc.getNodeAssocRef();
 
         // Invoke policy behaviours
-        invokeOnUpdateNode(sourceRef);
         invokeOnCreateAssociation(assocRef);
         
         return assocRef;
@@ -1298,14 +1288,10 @@ public class DbNodeServiceImpl extends AbstractNodeServiceImpl
         }
         AssociationRef assocRef = assoc.getNodeAssocRef();
         
-        // Invoke policy behaviours
-        invokeBeforeUpdateNode(sourceRef);
-        
         // delete it
         nodeDaoService.deleteNodeAssoc(assoc);
         
         // Invoke policy behaviours
-        invokeOnUpdateNode(sourceRef);
         invokeOnDeleteAssociation(assocRef);
     }