[jboss-svn-commits] JBL Code SVN: r30255 - in labs/jbossrules/trunk: drools-repository/src/main/java/org/drools/repository and 1 other directory.

[jboss-svn-commits at lists.jboss.org]

Author: jervisliu
Date: 2009-11-20 03:45:21 -0500 (Fri, 20 Nov 2009)
New Revision: 30255

Modified:
   labs/jbossrules/trunk/drools-guvnor/src/main/java/org/drools/guvnor/server/ServiceImplementation.java
   labs/jbossrules/trunk/drools-repository/src/main/java/org/drools/repository/PackageItem.java
Log:
GUVNOR-267: BRMS logs package creation, but not package archiving or deletion

Modified: labs/jbossrules/trunk/drools-guvnor/src/main/java/org/drools/guvnor/server/ServiceImplementation.java
===================================================================
--- labs/jbossrules/trunk/drools-guvnor/src/main/java/org/drools/guvnor/server/ServiceImplementation.java	2009-11-19 20:52:13 UTC (rev 30254)
+++ labs/jbossrules/trunk/drools-guvnor/src/main/java/org/drools/guvnor/server/ServiceImplementation.java	2009-11-20 08:45:21 UTC (rev 30255)
@@ -1673,6 +1673,8 @@
         }
 
         try {
+            log.info( "USER:" + getCurrentUserName() + " COPYING package [" + sourcePackageName + "] to  package [" + destPackageName + "]");
+      	
             repository.copyPackage( sourcePackageName,
                                     destPackageName );
         } catch ( RulesRepositoryException e ) {
@@ -1765,8 +1767,10 @@
             Identity.instance().checkPermission( new PackageUUIDType( uuid ),
                                                  RoleTypes.PACKAGE_ADMIN );
         }
+
         try {
             PackageItem item = repository.loadPackageByUUID( uuid );
+            log.info( "USER:" + getCurrentUserName() + " REMOVEING package [" + item.getName() + "]" );
             item.remove();
             repository.save();
         } catch ( RulesRepositoryException e ) {
@@ -1783,6 +1787,7 @@
             Identity.instance().checkPermission( new PackageUUIDType( uuid ),
                                                  RoleTypes.PACKAGE_ADMIN );
         }
+        log.info( "USER:" + getCurrentUserName() + " RENAMING package [UUID: " + uuid + "] to package [" + newName +"]");
 
         return repository.renamePackage( uuid,
                                          newName );

Modified: labs/jbossrules/trunk/drools-repository/src/main/java/org/drools/repository/PackageItem.java
===================================================================
--- labs/jbossrules/trunk/drools-repository/src/main/java/org/drools/repository/PackageItem.java	2009-11-19 20:52:13 UTC (rev 30254)
+++ labs/jbossrules/trunk/drools-repository/src/main/java/org/drools/repository/PackageItem.java	2009-11-20 08:45:21 UTC (rev 30255)
@@ -287,6 +287,7 @@
     public void remove() {
     	checkIsUpdateable();
     	try {
+            log.info( "USER:" + getCurrentUserName() + " REMOVEING package [" + getName() + "]" );
 			this.node.remove();
 		} catch (RepositoryException e) {
 			throw new RulesRepositoryException("Was not able to delete package.", e);
@@ -874,6 +875,8 @@
 	public PackageItem createPackage(String subPackageName) throws RepositoryException {
 
 		node.checkout();
+        log.info( "USER:" + getCurrentUserName() + " CREATEING package [" + subPackageName + "]" );
+		
         Node rulePackageNode = node.addNode( subPackageName, PackageItem.RULE_PACKAGE_TYPE_NAME );
 
         rulePackageNode.addNode( PackageItem.ASSET_FOLDER_NAME, "drools:versionableAssetFolder" );
@@ -890,7 +893,9 @@
 		return new PackageItem(this.rulesRepository, rulePackageNode);
 	}
 
+    private String getCurrentUserName() {
+        return this.rulesRepository.getSession().getUserID();
+    }
 
 
-
 }
\ No newline at end of file