You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@continuum.apache.org by ca...@apache.org on 2006/09/18 04:51:22 UTC

svn commit: r447202 - /maven/continuum/branches/continuum-acegi/continuum-security/continuum-security-acegi/src/main/java/org/apache/maven/continuum/security/acegi/AcegiContinuum.java

Author: carlos
Date: Sun Sep 17 19:51:22 2006
New Revision: 447202

URL: http://svn.apache.org/viewvc?view=rev&rev=447202
Log:
Return added/updated object for notifiers and build definitions

Modified:
    maven/continuum/branches/continuum-acegi/continuum-security/continuum-security-acegi/src/main/java/org/apache/maven/continuum/security/acegi/AcegiContinuum.java

Modified: maven/continuum/branches/continuum-acegi/continuum-security/continuum-security-acegi/src/main/java/org/apache/maven/continuum/security/acegi/AcegiContinuum.java
URL: http://svn.apache.org/viewvc/maven/continuum/branches/continuum-acegi/continuum-security/continuum-security-acegi/src/main/java/org/apache/maven/continuum/security/acegi/AcegiContinuum.java?view=diff&rev=447202&r1=447201&r2=447202
==============================================================================
--- maven/continuum/branches/continuum-acegi/continuum-security/continuum-security-acegi/src/main/java/org/apache/maven/continuum/security/acegi/AcegiContinuum.java (original)
+++ maven/continuum/branches/continuum-acegi/continuum-security/continuum-security-acegi/src/main/java/org/apache/maven/continuum/security/acegi/AcegiContinuum.java Sun Sep 17 19:51:22 2006
@@ -83,16 +83,20 @@
         return aclEventHandler;
     }
 
-    public void addBuildDefinitionToProject( int projectId, BuildDefinition buildDefinition )
+    public BuildDefinition addBuildDefinitionToProject( int projectId, BuildDefinition buildDefinition )
         throws ContinuumException
     {
-        getContinuum().addBuildDefinitionToProject( projectId, buildDefinition );
+        BuildDefinition definition = getContinuum().addBuildDefinitionToProject( projectId, buildDefinition );
+        //getAclEventHandler().afterAddProjectBuildDefinition( definition, projectId );
+        return definition;
     }
 
-    public void addBuildDefinitionToProjectGroup( int projectGroupId, BuildDefinition buildDefinition )
+    public BuildDefinition addBuildDefinitionToProjectGroup( int projectGroupId, BuildDefinition buildDefinition )
         throws ContinuumException
     {
-        getContinuum().addBuildDefinitionToProjectGroup( projectGroupId, buildDefinition );
+        BuildDefinition definition = getContinuum().addBuildDefinitionToProjectGroup( projectGroupId, buildDefinition );
+        //getAclEventHandler().afterAddProjectGroupBuildDefinition( definition, projectGroupId );
+        return definition;
     }
 
     public ContinuumProjectBuildingResult addMavenOneProject( String metadataUrl )
@@ -111,16 +115,20 @@
         return result;
     }
 
-    public void addNotifier( int projectId, ProjectNotifier notifier )
+    public ProjectNotifier addNotifier( int projectId, ProjectNotifier notifier )
         throws ContinuumException
     {
-        getContinuum().addNotifier( projectId, notifier );
+        ProjectNotifier projectNotifier = getContinuum().addNotifier( projectId, notifier );
+        //getAclEventHandler().afterAddProjectNotifier( projectNotifier, projectId );
+        return projectNotifier;
     }
 
-    public void addNotifier( int projectId, String notifierType, Map configuration )
+    public ProjectNotifier addNotifier( int projectId, String notifierType, Map configuration )
         throws ContinuumException
     {
-        getContinuum().addNotifier( projectId, notifierType, configuration );
+        ProjectNotifier projectNotifier = getContinuum().addNotifier( projectId, notifierType, configuration );
+        //getAclEventHandler().afterAddProjectNotifier( projectNotifier, projectId );
+        return projectNotifier;
     }
 
     public int addProject( Project project, String executorId )
@@ -205,7 +213,9 @@
 
     public Collection getAllProjectGroupsWithProjects()
     {
-        return getContinuum().getAllProjectGroupsWithProjects();
+        Collection groups = getContinuum().getAllProjectGroupsWithProjects();
+        //getAclEventHandler().afterReturningProjectGroup( groups );
+        return groups;
     }
 
     public Collection getAllProjects( int start, int end )
@@ -502,16 +512,16 @@
         getContinuum().removeUserGroup( userGroupId );
     }
 
-    public void updateBuildDefinitionForProject( int projectId, BuildDefinition buildDefinition )
+    public BuildDefinition updateBuildDefinitionForProject( int projectId, BuildDefinition buildDefinition )
         throws ContinuumException
     {
-        getContinuum().updateBuildDefinitionForProject( projectId, buildDefinition );
+        return getContinuum().updateBuildDefinitionForProject( projectId, buildDefinition );
     }
 
-    public void updateBuildDefinitionForProjectGroup( int projectGroupId, BuildDefinition buildDefinition )
+    public BuildDefinition updateBuildDefinitionForProjectGroup( int projectGroupId, BuildDefinition buildDefinition )
         throws ContinuumException
     {
-        getContinuum().updateBuildDefinitionForProjectGroup( projectGroupId, buildDefinition );
+        return getContinuum().updateBuildDefinitionForProjectGroup( projectGroupId, buildDefinition );
     }
 
     public void updateConfiguration( Map parameters )
@@ -520,16 +530,16 @@
         getContinuum().updateConfiguration( parameters );
     }
 
-    public void updateNotifier( int projectId, int notifierId, Map configuration )
+    public ProjectNotifier updateNotifier( int projectId, int notifierId, Map configuration )
         throws ContinuumException
     {
-        getContinuum().updateNotifier( projectId, notifierId, configuration );
+        return getContinuum().updateNotifier( projectId, notifierId, configuration );
     }
 
-    public void updateNotifier( int projectId, ProjectNotifier notifier )
+    public ProjectNotifier updateNotifier( int projectId, ProjectNotifier notifier )
         throws ContinuumException
     {
-        getContinuum().updateNotifier( projectId, notifier );
+        return getContinuum().updateNotifier( projectId, notifier );
     }
 
     public void updateProject( Project project )