Index: lams_common/src/java/org/lamsfoundation/lams/learningdesign/service/ExportToolContentService.java
===================================================================
diff -u -r0988ce4dfd3a003db77de92d7c158bdf548093ab -r9ad43dec7eb0960145bc92da0ae6435a2c412a58
--- lams_common/src/java/org/lamsfoundation/lams/learningdesign/service/ExportToolContentService.java	(.../ExportToolContentService.java)	(revision 0988ce4dfd3a003db77de92d7c158bdf548093ab)
+++ lams_common/src/java/org/lamsfoundation/lams/learningdesign/service/ExportToolContentService.java	(.../ExportToolContentService.java)	(revision 9ad43dec7eb0960145bc92da0ae6435a2c412a58)
@@ -541,8 +541,17 @@
 	}
 	/**
 	 * Import tool content 
+	 * TODO Remove once all tools converted to supply user id. 
 	 */
 	public Object importToolContent(String toolContentPath, IToolContentHandler toolContentHandler) throws ImportToolContentException{
+		log.error("importToolContent(String toolContentPath, IToolContentHandler toolContentHandler) to be removed - it sets the owner of files in the content repository to 1. Some tool needs to be updated.");
+		return importToolContent(toolContentPath, toolContentHandler, new Integer(1));
+	}
+
+	/**
+	 * Import tool content 
+	 */
+	public Object importToolContent(String toolContentPath, IToolContentHandler toolContentHandler, Integer userId ) throws ImportToolContentException{
 		Object toolPOJO = null;
 //		change xml to Tool POJO 
 		XStream toolXml = new XStream();
@@ -621,10 +630,10 @@
 					//upload to repository: file or pacakge
 					NodeKey key;
 					if(!isPackage)
-						key = toolContentHandler.uploadFile(is,fileName,mimeType,fileProperty);
+						key = toolContentHandler.uploadFile(is,fileName,mimeType,fileProperty, userId);
 					else{
 						String packageDirectory = ZipFileUtil.expandZip(is, realFileName);
-						key = toolContentHandler.uploadPackage(packageDirectory,initalItem);
+						key = toolContentHandler.uploadPackage(packageDirectory,initalItem, userId);
 					}
 					
 					//refresh file node Uuid and Version value to latest.
Index: lams_common/src/java/org/lamsfoundation/lams/learningdesign/service/IExportToolContentService.java
===================================================================
diff -u -r2b636f4e834cd0b45c00bb3abf6a6b5e5f3161c8 -r9ad43dec7eb0960145bc92da0ae6435a2c412a58
--- lams_common/src/java/org/lamsfoundation/lams/learningdesign/service/IExportToolContentService.java	(.../IExportToolContentService.java)	(revision 2b636f4e834cd0b45c00bb3abf6a6b5e5f3161c8)
+++ lams_common/src/java/org/lamsfoundation/lams/learningdesign/service/IExportToolContentService.java	(.../IExportToolContentService.java)	(revision 9ad43dec7eb0960145bc92da0ae6435a2c412a58)
@@ -93,7 +93,21 @@
 	 */
 	Long importLearningDesign(String learningDesignPath,User importer, Integer workspaceFolderUid, List<String> toolsErrorMsgs) throws ImportToolContentException;
 	
+	/** 
+	 * Import the tool content. This is called by tools to do the actual content import, once the tool
+	 * has set up whatever it needs to do. 
+     * TODO To be removed from the system and replaced with the call with the user id.
+	 * @Deprecated
+	 */
 	Object importToolContent(String toolContentPath, IToolContentHandler toolContentHandler)
+	throws ImportToolContentException;
+
+	/** 
+	 * Import the tool content. This is called by tools to do the actual content import, once the tool
+	 * has set up whatever it needs to do. To be removed from the system and replaced with the call with the 
+	 * user id.
+	 */
+	Object importToolContent(String toolContentPath, IToolContentHandler toolContentHandler, Integer userId)
 		throws ImportToolContentException;
 	
 	/**
Index: lams_common/src/java/org/lamsfoundation/lams/tool/BasicToolVO.java
===================================================================
diff -u -r34b959260a0f8f8285793a4481a95ca3580eabc5 -r9ad43dec7eb0960145bc92da0ae6435a2c412a58
--- lams_common/src/java/org/lamsfoundation/lams/tool/BasicToolVO.java	(.../BasicToolVO.java)	(revision 34b959260a0f8f8285793a4481a95ca3580eabc5)
+++ lams_common/src/java/org/lamsfoundation/lams/tool/BasicToolVO.java	(.../BasicToolVO.java)	(revision 9ad43dec7eb0960145bc92da0ae6435a2c412a58)
@@ -73,8 +73,11 @@
     
     /** persistent field */
     private String moderationUrl;
-    
+
     /** persistent field */
+    private String helpUrl;
+
+    /** persistent field */
     private boolean supportsRunOffline;
     
     /** persistent field */
@@ -124,6 +127,7 @@
                 String monitorUrl,
                 String contributeUrl,
                 String moderationUrl,
+                String helpUrl,
                 boolean supportsRunOffline,
                 long defaultToolContentId, 
                 String toolSignature, 
@@ -147,6 +151,7 @@
 		this.monitorUrl=monitorUrl;
 		this.contributeUrl=contributeUrl;
 		this.moderationUrl=moderationUrl;
+		this.helpUrl = helpUrl;
 		this.supportsRunOffline=supportsRunOffline;
 		this.defaultToolContentId=defaultToolContentId;
 		this.toolSignature=toolSignature;
@@ -222,6 +227,21 @@
 	}
 
 	/* (non-Javadoc)
+	 * @see org.lamsfoundation.lams.tool.IToolVO#getHelpUrl()
+	 */
+	public String getHelpUrl() {
+		return helpUrl;
+	}
+
+	/* (non-Javadoc)
+	 * @see org.lamsfoundation.lams.tool.IToolVO#setHelpUrl(java.lang.String)
+	 */
+	public void setHelpUrl(String helpUrl) {
+		this.helpUrl = helpUrl;
+	}
+
+
+	/* (non-Javadoc)
 	 * @see org.lamsfoundation.lams.tool.IToolVO#getCreateDateTime()
 	 */
 	public Date getCreateDateTime() {
Index: lams_common/src/java/org/lamsfoundation/lams/tool/IToolVO.java
===================================================================
diff -u -r27bc32d524b0e22831ba7c75b3c11b8adb23189e -r9ad43dec7eb0960145bc92da0ae6435a2c412a58
--- lams_common/src/java/org/lamsfoundation/lams/tool/IToolVO.java	(.../IToolVO.java)	(revision 27bc32d524b0e22831ba7c75b3c11b8adb23189e)
+++ lams_common/src/java/org/lamsfoundation/lams/tool/IToolVO.java	(.../IToolVO.java)	(revision 9ad43dec7eb0960145bc92da0ae6435a2c412a58)
@@ -35,6 +35,10 @@
 
 	public abstract void setContributeUrl(String contributeUrl);
 
+	public abstract String getHelpUrl();
+
+	public abstract void setHelpUrl(String helpUrl);
+
 	public abstract Date getCreateDateTime();
 
 	public abstract void setCreateDateTime(Date createDateTime);
Index: lams_common/src/java/org/lamsfoundation/lams/tool/SystemTool.java
===================================================================
diff -u -r250486f86b66ac04cece0001879c3dc39c96ed98 -r9ad43dec7eb0960145bc92da0ae6435a2c412a58
--- lams_common/src/java/org/lamsfoundation/lams/tool/SystemTool.java	(.../SystemTool.java)	(revision 250486f86b66ac04cece0001879c3dc39c96ed98)
+++ lams_common/src/java/org/lamsfoundation/lams/tool/SystemTool.java	(.../SystemTool.java)	(revision 9ad43dec7eb0960145bc92da0ae6435a2c412a58)
@@ -80,6 +80,9 @@
     private String contributeUrl;
     
     /** persistent field */
+    private String helpUrl;
+    
+    /** persistent field */
     private Date createDateTime;
     
 	/** 
@@ -100,6 +103,7 @@
                 String exportPortfolioClassUrl,
                 String monitorUrl,
                 String contributeUrl,
+                String helpUrl,
                 Date createDateTime) 
     {
         this.systemToolId = systemToolId;
@@ -114,6 +118,7 @@
         this.exportPortfolioClassUrl = exportPortfolioClassUrl;
         this.monitorUrl = monitorUrl;
         this.contributeUrl = contributeUrl;
+        this.helpUrl = helpUrl;
 
         this.createDateTime = createDateTime;
     }
@@ -290,7 +295,23 @@
         this.monitorUrl = monitorUrl;
     }
  
-    
+    /**
+     * @hibernate.property column="help_url" length="65535" not-null="false"  
+     * @return Returns the helpUrl.
+     */
+    public String getHelpUrl()
+    {
+        return helpUrl;
+    }
+    /**
+     * @param helpUrl The helpUrl to set.
+     */
+    public void setHelpUrl(String helpUrl)
+    {
+        this.helpUrl = helpUrl;
+    }
+ 
+   
     public String toString() {
         return new ToStringBuilder(this)
             .append("systemToolId", getSystemToolId())
Index: lams_common/src/java/org/lamsfoundation/lams/tool/Tool.java
===================================================================
diff -u -r34b959260a0f8f8285793a4481a95ca3580eabc5 -r9ad43dec7eb0960145bc92da0ae6435a2c412a58
--- lams_common/src/java/org/lamsfoundation/lams/tool/Tool.java	(.../Tool.java)	(revision 34b959260a0f8f8285793a4481a95ca3580eabc5)
+++ lams_common/src/java/org/lamsfoundation/lams/tool/Tool.java	(.../Tool.java)	(revision 9ad43dec7eb0960145bc92da0ae6435a2c412a58)
@@ -75,6 +75,9 @@
     private String moderationUrl;
     
     /** persistent field */
+    private String helpUrl;
+    
+    /** persistent field */
     private boolean supportsRunOffline;
     
     /** persistent field */
@@ -516,6 +519,22 @@
         this.monitorUrl = monitorUrl;
     }
  
+    /**
+     * @hibernate.property column="help_url" length="65535" not-null="false"  
+     * @return Returns the helpUrl.
+     */
+    public String getHelpUrl()
+    {
+        return helpUrl;
+    }
+    /**
+     * @param helpUrl The helpUrl to set.
+     */
+    public void setHelpUrl(String helpUrl)
+    {
+        this.helpUrl = helpUrl;
+    }
+
     
     /** 
      * @hibernate.property column="tool_identifier" length="64"       
@@ -589,6 +608,7 @@
     			this.monitorUrl,
     			this.contributeUrl,
     			this.moderationUrl,
+    			this.helpUrl,
     			this.supportsRunOffline,
     			this.defaultToolContentId, 
     			this.toolSignature, 
Index: lams_common/src/java/org/lamsfoundation/lams/usermanagement/Organisation.java
===================================================================
diff -u -rc21c3386cec2ee819ba267d84d532676d2f4990d -r9ad43dec7eb0960145bc92da0ae6435a2c412a58
--- lams_common/src/java/org/lamsfoundation/lams/usermanagement/Organisation.java	(.../Organisation.java)	(revision c21c3386cec2ee819ba267d84d532676d2f4990d)
+++ lams_common/src/java/org/lamsfoundation/lams/usermanagement/Organisation.java	(.../Organisation.java)	(revision 9ad43dec7eb0960145bc92da0ae6435a2c412a58)
@@ -60,6 +60,9 @@
     private Date createDate;
 
     /** persistent field */
+    private User createdBy;
+
+    /** persistent field */
     private Workspace workspace;
 
     /** persistent field */
@@ -91,14 +94,16 @@
     private Boolean courseAdminCanCreateGuestAccounts;
 
     /** full constructor */
-    public Organisation(String name, String description, Organisation parentOrganisation, Date createDate, Workspace workspace, 
+    public Organisation(String name, String description, Organisation parentOrganisation, 
+    			Date createDate, User createdBy, Workspace workspace, 
     			OrganisationType organisationType, Set userOrganisations, Set childOrganisations, Set lessons,
     			Boolean courseAdminCanAddNewUsers, Boolean courseAdminCanBrowseAllUsers, Boolean courseAdminCanChangeStatusOfCourse, 
     			Boolean courseAdminCanCreateGuestAccounts) {
         this.name = name;
         this.description = description;
         this.parentOrganisation = parentOrganisation;
         this.createDate = createDate;
+        this.createdBy = createdBy;
         this.workspace = workspace;
         this.organisationType = organisationType;
         this.userOrganisations = userOrganisations;
@@ -120,8 +125,9 @@
     }
 
     /** minimal constructor */
-    public Organisation(Date createDate, Workspace workspace, OrganisationType organisationType, Set userOrganisations, Set lessons) {
+    public Organisation(Date createDate, User createdBy, Workspace workspace, OrganisationType organisationType, Set userOrganisations, Set lessons) {
         this.createDate = createDate;
+        this.createdBy = createdBy;
         this.workspace = workspace;
         this.organisationType = organisationType;
         this.userOrganisations = userOrganisations;
@@ -135,12 +141,14 @@
     }
     public Organisation(String name, 
     					String description,						
-						Date createDate,						 
+						Date createDate,
+						User createdBy,
 						OrganisationType organisationType) {
 		super();
 		this.name = name;
 		this.description = description;		
 		this.createDate = createDate;		
+		this.createdBy = createdBy;
 		this.organisationType = organisationType;
 	}
 
@@ -229,6 +237,20 @@
         this.createDate = createDate;
     }
 
+    /**
+     * 		@hibernate.many-to-one not-null="true"
+     *      @hibernate.column name="created_by"    
+     *         
+     */
+    public User getCreatedBy() {
+        return this.createdBy;
+    }
+
+    public void setCreatedBy(User createdBy) {
+        this.createdBy = createdBy;
+    }
+
+
     /** 
      *            @hibernate.many-to-one
      *             cascade="all"
Index: lams_common/src/java/org/lamsfoundation/lams/usermanagement/service/UserManagementService.java
===================================================================
diff -u -r8511338e13292dc22334f9b15640687858919592 -r9ad43dec7eb0960145bc92da0ae6435a2c412a58
--- lams_common/src/java/org/lamsfoundation/lams/usermanagement/service/UserManagementService.java	(.../UserManagementService.java)	(revision 8511338e13292dc22334f9b15640687858919592)
+++ lams_common/src/java/org/lamsfoundation/lams/usermanagement/service/UserManagementService.java	(.../UserManagementService.java)	(revision 9ad43dec7eb0960145bc92da0ae6435a2c412a58)
@@ -439,9 +439,12 @@
 	public Organisation saveOrganisation( Organisation organisation, Integer userID ) 	 
     { 	 
 	 
+    		User creator = (User)findById(User.class,userID);
+    			
             if ( organisation.getOrganisationId() == null ) { 	 
                     Date createDateTime = new Date(); 	 
-                    organisation.setCreateDate(createDateTime); 	 
+                    organisation.setCreateDate(createDateTime);
+                    organisation.setCreatedBy(creator);
 	 
                     if(organisation.getOrganisationType().getOrganisationTypeId().equals(OrganisationType.COURSE_TYPE)){ 	 
                             Workspace workspace = createWorkspaceForOrganisation(organisation.getName(), userID, createDateTime); 	 
Index: lams_contentrepository/.classpath
===================================================================
diff -u -rb2f56ba87d93e9daa8cb2c38fa906da531d6f132 -r9ad43dec7eb0960145bc92da0ae6435a2c412a58
--- lams_contentrepository/.classpath	(.../.classpath)	(revision b2f56ba87d93e9daa8cb2c38fa906da531d6f132)
+++ lams_contentrepository/.classpath	(.../.classpath)	(revision 9ad43dec7eb0960145bc92da0ae6435a2c412a58)
@@ -4,8 +4,9 @@
 	<classpathentry kind="src" path="conf/hibernate/mappings"/>
 	<classpathentry kind="src" path="test/java"/>
 	<classpathentry kind="con" path="org.eclipse.jdt.launching.JRE_CONTAINER"/>
-	<classpathentry kind="con" path="com.genuitec.eclipse.j2eedt.core.J2EE14_CONTAINER"/>
 	<classpathentry kind="con" path="org.eclipse.jdt.USER_LIBRARY/3rdParty"/>
 	<classpathentry kind="lib" path="/lams_build/lib/lams/lams.jar"/>
+	<classpathentry kind="lib" path="D:/jboss-4.0.2/server/default/lib/jboss-j2ee.jar"/>
+	<classpathentry kind="lib" path="D:/jboss-4.0.2/server/default/lib/javax.servlet.jar"/>
 	<classpathentry kind="output" path="build"/>
 </classpath>
Index: lams_contentrepository/conf/hibernate/mappings/org/lamsfoundation/lams/contentrepository/CrNodeVersion.hbm.xml
===================================================================
diff -u -r6e5832d17263bc6c6f7683733e91e22f97eab569 -r9ad43dec7eb0960145bc92da0ae6435a2c412a58
--- lams_contentrepository/conf/hibernate/mappings/org/lamsfoundation/lams/contentrepository/CrNodeVersion.hbm.xml	(.../CrNodeVersion.hbm.xml)	(revision 6e5832d17263bc6c6f7683733e91e22f97eab569)
+++ lams_contentrepository/conf/hibernate/mappings/org/lamsfoundation/lams/contentrepository/CrNodeVersion.hbm.xml	(.../CrNodeVersion.hbm.xml)	(revision 9ad43dec7eb0960145bc92da0ae6435a2c412a58)
@@ -46,6 +46,15 @@
             length="14"
         />
 
+        <property
+            name="userId"
+            type="java.lang.Integer"
+            update="true"
+            insert="true"
+            column="user_id"
+            not-null="true"
+        />
+
         <many-to-one
             name="node"
             class="org.lamsfoundation.lams.contentrepository.CrNode"
Index: lams_contentrepository/src/java/org/lamsfoundation/lams/contentrepository/CrNode.java
===================================================================
diff -u -r41de4a8b5ff40c43e8eaadc76a33ebce112d6801 -r9ad43dec7eb0960145bc92da0ae6435a2c412a58
--- lams_contentrepository/src/java/org/lamsfoundation/lams/contentrepository/CrNode.java	(.../CrNode.java)	(revision 41de4a8b5ff40c43e8eaadc76a33ebce112d6801)
+++ lams_contentrepository/src/java/org/lamsfoundation/lams/contentrepository/CrNode.java	(.../CrNode.java)	(revision 9ad43dec7eb0960145bc92da0ae6435a2c412a58)
@@ -78,7 +78,7 @@
 
 	/** Create the CrNode and its initial CrNodeVersion.  
 	 */
-	public CrNode(String relPath, String nodeTypeName, Date createdDateTime, CrWorkspace workspace, 
+	public CrNode(String relPath, String nodeTypeName, Date createdDateTime, Integer userId, CrWorkspace workspace, 
 			CrNodeVersion parentNodeVersion, String versionDescription)	{ 
 		
         this.path = relPath;
@@ -90,7 +90,7 @@
         
         this.crWorkspace = workspace;
         this.parentNodeVersion = parentNodeVersion;
-		this.addCrNodeVersion(new CrNodeVersion(this, createdDateTime, incrementNextVersionId(), versionDescription));
+		this.addCrNodeVersion(new CrNodeVersion(this, createdDateTime, incrementNextVersionId(), versionDescription, userId));
 	}
 
     /** full constructor */
Index: lams_contentrepository/src/java/org/lamsfoundation/lams/contentrepository/CrNodeVersion.java
===================================================================
diff -u -r41de4a8b5ff40c43e8eaadc76a33ebce112d6801 -r9ad43dec7eb0960145bc92da0ae6435a2c412a58
--- lams_contentrepository/src/java/org/lamsfoundation/lams/contentrepository/CrNodeVersion.java	(.../CrNodeVersion.java)	(revision 41de4a8b5ff40c43e8eaadc76a33ebce112d6801)
+++ lams_contentrepository/src/java/org/lamsfoundation/lams/contentrepository/CrNodeVersion.java	(.../CrNodeVersion.java)	(revision 9ad43dec7eb0960145bc92da0ae6435a2c412a58)
@@ -54,6 +54,9 @@
     private Date createdDateTime;
 
     /** persistent field */
+    private Integer userId;
+
+    /** persistent field */
     private org.lamsfoundation.lams.contentrepository.CrNode node;
 
     /* child Nodes. persistent field 
@@ -78,10 +81,11 @@
 
 	/** Normal constructor 
 	 */
-	public CrNodeVersion (CrNode node, Date createdDate, Long versionId, String versionDescription) {
+	public CrNodeVersion (CrNode node, Date createdDate, Long versionId, String versionDescription, Integer userId) {
 		this.createdDateTime = createdDate;
 		this.node = node;
 		this.versionId = versionId;
+		this.userId = userId;
 		setVersionDescription(versionDescription);
 	}
 
@@ -142,6 +146,21 @@
         this.createdDateTime = createdDateTime;
     }
 
+	/** 
+     *            @hibernate.property
+     *		       type="java.lang.Integer"
+     *		       column="user_id"
+     *			   not-null="true"
+     */
+	public Integer getUserId() {
+		return userId;
+	}
+
+	public void setUserId(Integer userId) {
+		this.userId = userId;
+	}
+
+
     /** 
      * bi-directional many-to-one association to CrNode
      * 
Index: lams_contentrepository/src/java/org/lamsfoundation/lams/contentrepository/IVersionedNode.java
===================================================================
diff -u -r41de4a8b5ff40c43e8eaadc76a33ebce112d6801 -r9ad43dec7eb0960145bc92da0ae6435a2c412a58
--- lams_contentrepository/src/java/org/lamsfoundation/lams/contentrepository/IVersionedNode.java	(.../IVersionedNode.java)	(revision 41de4a8b5ff40c43e8eaadc76a33ebce112d6801)
+++ lams_contentrepository/src/java/org/lamsfoundation/lams/contentrepository/IVersionedNode.java	(.../IVersionedNode.java)	(revision 9ad43dec7eb0960145bc92da0ae6435a2c412a58)
@@ -200,5 +200,12 @@
 	 */
 	public InputStream getFile() throws FileException;
 
+   /**
+     * Get the user_id of the user who created this node.
+     *  
+     * @return userId
+     */
+    public Integer getUserId();
 
+
 }
\ No newline at end of file
Index: lams_contentrepository/src/java/org/lamsfoundation/lams/contentrepository/client/IToolContentHandler.java
===================================================================
diff -u -r86bb830745541b9a9966b58fb4132cf55b56d5c3 -r9ad43dec7eb0960145bc92da0ae6435a2c412a58
--- lams_contentrepository/src/java/org/lamsfoundation/lams/contentrepository/client/IToolContentHandler.java	(.../IToolContentHandler.java)	(revision 86bb830745541b9a9966b58fb4132cf55b56d5c3)
+++ lams_contentrepository/src/java/org/lamsfoundation/lams/contentrepository/client/IToolContentHandler.java	(.../IToolContentHandler.java)	(revision 9ad43dec7eb0960145bc92da0ae6435a2c412a58)
@@ -87,6 +87,9 @@
     /**
      * Save a file in the content repository.
      * 
+     * TODO To be removed from the system and replaced with the call with the user id.
+	 * @Deprecated
+     * 
      * @param stream Input filestream. Mandatory.
      * @param fileName Input filename. Mandatory.
      * @param mimeType Mimetype of file. Optional.
@@ -102,8 +105,35 @@
             RepositoryCheckedException;
 
     /**
+     * Save a file in the content repository.
+     * 
+     * @param stream Input filestream. Mandatory.
+     * @param fileName Input filename. Mandatory.
+     * @param mimeType Mimetype of file. Optional.
+     * @param fileProperty is this for online or offline instructions? Should be TYPE_ONLINE or TYPE_OFFLINE. Mandatory.
+     * @return key to the new content repository node
+     * @throws InvalidParameterException One of the mandatory parameters is missing.
+     * @throws FileException An error occured writing the input stream to disk.
+     * @throws RepositoryCheckedException Some other error occured.
+     */
+    public abstract NodeKey uploadFile(InputStream stream, String fileName,
+            String mimeType, String fileProperty, Integer userId)
+            throws RepositoryCheckedException, InvalidParameterException,
+            RepositoryCheckedException;
+
+    /**
      * Save a directory of files in the content repository.
      * 
+     * TODO To be removed from the system and replaced with the call with the user id.
+	 * @Deprecated
+	 */
+    public abstract NodeKey uploadPackage(String dirPath, String startFile) 
+    throws RepositoryCheckedException, InvalidParameterException, 
+    RepositoryCheckedException;
+
+    /**
+     * Save a directory of files in the content repository.
+     * 
 	 * @param ticket ticket issued on login. Identifies tool and workspace - mandatory 
 	 * @param dirPath directory path containing files - mandatory
 	 * @param startFile relative path of initial file - optional
@@ -112,7 +142,7 @@
      * @throws FileException An error occured writing the files to disk.
      * @throws RepositoryCheckedException Some other error occured.
      */
-    public abstract NodeKey uploadPackage(String dirPath, String startFile) 
+    public abstract NodeKey uploadPackage(String dirPath, String startFile, Integer userId) 
     		throws RepositoryCheckedException, InvalidParameterException, 
     		RepositoryCheckedException;
  
@@ -127,14 +157,26 @@
 
     /**
      * Copy an entry in the content repository.
-     * 
+     * TODO To be removed from the system and replaced with the call with the user id.
+	 * @Deprecated
+	 *
      * @param uuid id of the file node. Mandatory
      * @throws ItemNotFoundException Node to copy cannot be found 
      * @throws RepositoryCheckedException Some other error occured.
      */
     public abstract NodeKey copyFile(Long uuid) 
     		throws ItemNotFoundException, RepositoryCheckedException;
 
+    /**
+     * Copy an entry in the content repository.
+     * 
+     * @param uuid id of the file node. Mandatory
+     * @throws ItemNotFoundException Node to copy cannot be found 
+     * @throws RepositoryCheckedException Some other error occured.
+     */
+    public abstract NodeKey copyFile(Long uuid, Integer userId) 
+    		throws ItemNotFoundException, RepositoryCheckedException;
+
     /** Get a file node. 
      * @param uuid id of the file node. Mandatory
      * @throws FileException An error occured writing the input stream to disk.
Index: lams_contentrepository/src/java/org/lamsfoundation/lams/contentrepository/client/ToolContentHandler.java
===================================================================
diff -u -r7aa77bcecb501c019e15d8dfe06d7d3542c37460 -r9ad43dec7eb0960145bc92da0ae6435a2c412a58
--- lams_contentrepository/src/java/org/lamsfoundation/lams/contentrepository/client/ToolContentHandler.java	(.../ToolContentHandler.java)	(revision 7aa77bcecb501c019e15d8dfe06d7d3542c37460)
+++ lams_contentrepository/src/java/org/lamsfoundation/lams/contentrepository/client/ToolContentHandler.java	(.../ToolContentHandler.java)	(revision 9ad43dec7eb0960145bc92da0ae6435a2c412a58)
@@ -188,6 +188,14 @@
 	}
 
     /**
+     * TODO To be removed from the system and replaced with the call with the user id.
+     */
+    public NodeKey uploadFile(InputStream stream, String fileName, String mimeType, String fileProperty) throws RepositoryCheckedException, InvalidParameterException, RepositoryCheckedException {
+		log.error("uploadFile(InputStream stream, String fileName, String mimeType, String fileProperty) to be removed - it sets the owner of files in the content repository to 1. Some tool needs to be updated.");
+		return uploadFile(stream, fileName, mimeType, fileProperty, new Integer(1));
+    }
+
+    /**
      * Save a file in the content repository.
      * 
      * @param stream Input filestream. Mandatory.
@@ -199,18 +207,18 @@
      * @throws FileException An error occured writing the input stream to disk.
      * @throws RepositoryCheckedException Some other error occured.
      */
-    public NodeKey uploadFile(InputStream stream, String fileName, String mimeType, String fileProperty) throws RepositoryCheckedException, InvalidParameterException, RepositoryCheckedException {
+    public NodeKey uploadFile(InputStream stream, String fileName, String mimeType, String fileProperty, Integer userId) throws RepositoryCheckedException, InvalidParameterException, RepositoryCheckedException {
         if ( fileProperty == null )
             throw new InvalidParameterException("uploadFile: fileProperty parameter empty. Should be either TYPE_ONLINE or TYPE_OFFLINE");
 	    
         NodeKey nodeKey = null;
         try {
 		    try {
-		        nodeKey = getRepositoryService().addFileItem(getTicket(false), stream, fileName, mimeType, null);
+		        nodeKey = getRepositoryService().addFileItem(getTicket(false), stream, fileName, mimeType, null, userId);
 		    } catch (AccessDeniedException e) {
 		        log.warn("Unable to access repository to add file "+fileName
 					+"AccessDeniedException: "+e.getMessage()+" Retrying login.");
-	            nodeKey = getRepositoryService().addFileItem(getTicket(true), stream, fileName, mimeType, null);
+	            nodeKey = getRepositoryService().addFileItem(getTicket(true), stream, fileName, mimeType, null, userId);
 		    }
 	        
 		    try {
@@ -231,6 +239,14 @@
     }
 
     /**
+     * TODO To be removed from the system and replaced with the call with the user id.
+     */
+    public NodeKey uploadPackage(String dirPath, String startFile) throws RepositoryCheckedException, InvalidParameterException, RepositoryCheckedException {
+		log.error("uploadPackage(String dirPath, String startFile)  to be removed - it sets the owner of files in the content repository to 1. Some tool needs to be updated.");
+		return uploadPackage(dirPath, startFile, new Integer(1));
+    }
+
+    /**
      * Save a directory of files in the content repository.
      * 
 	 * @param ticket ticket issued on login. Identifies tool and workspace - mandatory 
@@ -241,18 +257,18 @@
      * @throws FileException An error occured writing the files to disk.
      * @throws RepositoryCheckedException Some other error occured.
      */
-    public NodeKey uploadPackage(String dirPath, String startFile) throws RepositoryCheckedException, InvalidParameterException, RepositoryCheckedException {
+    public NodeKey uploadPackage(String dirPath, String startFile, Integer userId) throws RepositoryCheckedException, InvalidParameterException, RepositoryCheckedException {
         if ( dirPath == null )
             throw new InvalidParameterException("uploadFile: dirPath parameter empty. Directory containing files must be supplied");
          	    
         NodeKey nodeKey = null;
         try {
 		    try {
-		        nodeKey = getRepositoryService().addPackageItem(getTicket(false), dirPath, startFile, null);
+		        nodeKey = getRepositoryService().addPackageItem(getTicket(false), dirPath, startFile, null, userId);
 		    } catch (AccessDeniedException e) {
 		        log.warn("Unable to access repository to add directory of files. AccessDeniedException: "
 		        		+e.getMessage()+" Retrying login.");
-		        nodeKey = getRepositoryService().addPackageItem(getTicket(true), dirPath, startFile, null);
+		        nodeKey = getRepositoryService().addPackageItem(getTicket(true), dirPath, startFile, null, userId);
 		    }
 	        
         } catch (RepositoryCheckedException e2) {
@@ -296,21 +312,29 @@
 		} 
     }
     /**
+     * TODO To be removed from the system and replaced with the call with the user id.
+     */
+    public NodeKey copyFile(Long uuid) throws ItemNotFoundException, RepositoryCheckedException {
+		log.error("copyFile(Long uuid) to be removed - it sets the owner of files in the content repository to 1. Some tool needs to be updated.");
+    	return copyFile(uuid, new Integer(1));
+    }
+
+    /**
      * Copy an entry in the content repository.
      * 
      * @param uuid id of the file node. Mandatory
      * @throws ItemNotFoundException Node to copy cannot be found 
      * @throws RepositoryCheckedException Some other error occured.
      */
-    public NodeKey copyFile(Long uuid) throws ItemNotFoundException, RepositoryCheckedException {
+    public NodeKey copyFile(Long uuid, Integer userId) throws ItemNotFoundException, RepositoryCheckedException {
         NodeKey nodeKey = null;
         try {
 	        try {
-				nodeKey = getRepositoryService().copyNodeVersion(getTicket(false), uuid, null);
+				nodeKey = getRepositoryService().copyNodeVersion(getTicket(false), uuid, null, userId);
 		    } catch (AccessDeniedException e) {
 		        log.warn("Unable to access repository to add copy node "+uuid
 					+"AccessDeniedException: "+e.getMessage()+" Retrying login.");
-	            nodeKey = getRepositoryService().copyNodeVersion(getTicket(true), uuid, null);
+	            nodeKey = getRepositoryService().copyNodeVersion(getTicket(true), uuid, null, userId);
 		    }
 	    } catch (ItemNotFoundException e) {
 	        log.warn("Unable to to copy node "+uuid
Index: lams_contentrepository/src/java/org/lamsfoundation/lams/contentrepository/package.html
===================================================================
diff -u -r3b0119f1c5ef6fd0fb497a0219c2dd2f7cdf6d52 -r9ad43dec7eb0960145bc92da0ae6435a2c412a58
--- lams_contentrepository/src/java/org/lamsfoundation/lams/contentrepository/package.html	(.../package.html)	(revision 3b0119f1c5ef6fd0fb497a0219c2dd2f7cdf6d52)
+++ lams_contentrepository/src/java/org/lamsfoundation/lams/contentrepository/package.html	(.../package.html)	(revision 9ad43dec7eb0960145bc92da0ae6435a2c412a58)
@@ -193,15 +193,17 @@
 							fileInputStream, 
 							filename, 
 							fileMimeType, 
-							"This is the initial version");
+							"This is the initial version",
+							user.getUserId());
 
 	User uploads a new version of the file:
 	NodeKey nodeKey2 = repositoryService.updateFileItem(repositoryTicket, 
 							nodeKey1.getUuid(), 
 							filename,
 							fileInputStream, 
 							fileMimeType, 
-							"This is the updated version);
+							"This is the updated version,
+							user.getUserId());
 
 	Get the second version of the file:
 	IVersionedNode node = repositoryService.getFileItem(repositoryTicket, 
@@ -210,12 +212,14 @@
 	Copy the initial version of the file node. CopyNodeVersion copies the specified version or last version (if not specified), duplicates the actual files.
 	NodeKey nodeKey3 = repositoryService.copyNodeVersion(repositoryTicket, 
 							nodeKey1.getUuId(),
-							nodeKey1.getVersion());
+							nodeKey1.getVersion(),
+							user.getUserId());
 
 	Copy the latest version of the file node. 
 	NodeKey nodeKey3 = repositoryService.copyNodeVersion(repositoryTicket, 
 							nodeKey1.getUuId(),
-							null);
+							null,
+							user.getUserId());
 
 	Delete the second version of the file:
 	String[] problemFiles = repositoryService.deleteVersion(repositoryTicket,nodeKey2.getUuid(), nodeKey2.getVersion());
Index: lams_contentrepository/src/java/org/lamsfoundation/lams/contentrepository/service/INodeFactory.java
===================================================================
diff -u -r41de4a8b5ff40c43e8eaadc76a33ebce112d6801 -r9ad43dec7eb0960145bc92da0ae6435a2c412a58
--- lams_contentrepository/src/java/org/lamsfoundation/lams/contentrepository/service/INodeFactory.java	(.../INodeFactory.java)	(revision 41de4a8b5ff40c43e8eaadc76a33ebce112d6801)
+++ lams_contentrepository/src/java/org/lamsfoundation/lams/contentrepository/service/INodeFactory.java	(.../INodeFactory.java)	(revision 9ad43dec7eb0960145bc92da0ae6435a2c412a58)
@@ -52,7 +52,7 @@
 	public abstract SimpleVersionedNode createFileNode(CrWorkspace workspace,
 			SimpleVersionedNode parentNode, String relPath,
 			InputStream istream, String filename, String mimeType,
-			String versionDescription)
+			String versionDescription, Integer userId)
 			throws org.lamsfoundation.lams.contentrepository.InvalidParameterException;
 
 	/**
@@ -64,7 +64,7 @@
 	 * @throws RepositoryRuntimeException if an internal error occurs.
 	 */
 	public abstract SimpleVersionedNode createPackageNode(
-			CrWorkspace workspace, String initialPath, String versionDescription)
+			CrWorkspace workspace, String initialPath, String versionDescription, Integer userId)
 			throws org.lamsfoundation.lams.contentrepository.InvalidParameterException;
 
 	/**
@@ -75,7 +75,7 @@
 	 * @throws RepositoryRuntimeException if an internal error occurs.
 	 */
 	public abstract SimpleVersionedNode createDataNode(CrWorkspace workspace,
-			SimpleVersionedNode parentNode, String versionDescription)
+			SimpleVersionedNode parentNode, String versionDescription, Integer userId)
 			throws org.lamsfoundation.lams.contentrepository.InvalidParameterException;
 
 	/**
@@ -117,7 +117,8 @@
 	 * @throws RepositoryRuntimeException if an internal error occurs.
 	 * @see org.lamsfoundation.lams.contentrepository.IVersionedNode#createNewVersion(java.lang.String, java.lang.String)
 	 */
-	public abstract SimpleVersionedNode getNodeNewVersion(Long workspaceId, Long uuid, Long versionId, String versionDescription) throws ItemNotFoundException;
+	public abstract SimpleVersionedNode getNodeNewVersion(Long workspaceId, Long uuid, Long versionId, 
+			String versionDescription, Integer userId) throws ItemNotFoundException;
 
 
 	/** Copy the supplied node/version to a new node. Does not copy the history
@@ -131,7 +132,7 @@
 	 *   file, filename or mimetype properties are invalid.
 	 * @throws ValueFormatException will only occur if there is an internal bug as it will only happen if the filename or mimetype properties are not strings. 
 	 */
-	public abstract SimpleVersionedNode copy(SimpleVersionedNode originalNode)
+	public abstract SimpleVersionedNode copy(SimpleVersionedNode originalNode, Integer userId)
 			throws FileException, ValueFormatException,
 			InvalidParameterException;
 
Index: lams_contentrepository/src/java/org/lamsfoundation/lams/contentrepository/service/IRepositoryService.java
===================================================================
diff -u -r1f93624009089d9d6559361b0061709ecf9f49b6 -r9ad43dec7eb0960145bc92da0ae6435a2c412a58
--- lams_contentrepository/src/java/org/lamsfoundation/lams/contentrepository/service/IRepositoryService.java	(.../IRepositoryService.java)	(revision 1f93624009089d9d6559361b0061709ecf9f49b6)
+++ lams_contentrepository/src/java/org/lamsfoundation/lams/contentrepository/service/IRepositoryService.java	(.../IRepositoryService.java)	(revision 9ad43dec7eb0960145bc92da0ae6435a2c412a58)
@@ -131,6 +131,15 @@
 	public void updateCredentials(ICredentials oldCredential, ICredentials newCredential)
 		throws AccessDeniedException, RepositoryCheckedException;
 
+	/**
+    * TODO To be removed from the system and replaced with the call with the user id.
+	 * @Deprecated
+	*/
+	public abstract NodeKey addFileItem(ITicket ticket, InputStream istream, String filename,
+			String mimeType, String versionDescription) 
+		throws FileException, AccessDeniedException, InvalidParameterException;
+	
+
 	/** 
 	 * Add a new file to the repository. This will create 
 	 * a completely new entry (node) in the repository, starting
@@ -139,14 +148,15 @@
 	 * @param istream new file, as an input stream - mandatory
 	 * @param mimeType mime type of file - optional
 	 * @param versionDescription human readable comment about the version - optional
+	 * @param userId user who is creating this node - mandatory
 	 * @return nodeKey (uuid and version)
      * @throws AccessDeniedException if ticket doesn't allow this action
      * @throws FileException if unable to save node due to file error
      * @throws InvalidParameterException if a required parameter is missing
      * @throws RepositoryRuntimeException if any internal errors have occured 
 	 */
 	public abstract NodeKey addFileItem(ITicket ticket, InputStream istream, String filename,
-					String mimeType, String versionDescription) 
+					String mimeType, String versionDescription, Integer userId) 
 			throws FileException, AccessDeniedException, InvalidParameterException;
 
 	/** 
@@ -161,14 +171,15 @@
 	 * @param dirPath directory path containing files - mandatory
 	 * @param startFile relative path of initial file - optional
 	 * @param versionDescription human readable comment about the version - optional
+	 * @param userId user who is creating this node - mandatory
 	 * @return nodeKey (uuid and version)
      * @throws AccessDeniedException if ticket doesn't allow this action
      * @throws FileException if unable to save node due to file error
      * @throws InvalidParameterException if a required parameter is missing
      * @throws RepositoryRuntimeException if any internal errors have occured 
 	 */
 	public abstract NodeKey addPackageItem(ITicket ticket, String dirPath, String startFile,
-				String versionDescription) 
+				String versionDescription, Integer userId) 
 		throws AccessDeniedException, InvalidParameterException, FileException;
 
     /** 
@@ -179,6 +190,7 @@
      * @param istream new file, as an input stream - mandatory
      * @param versionDescription human readable comment about the version - optional
      * @param mimeType mime type of file - optional
+	 * @param userId user who is creating this node - mandatory
      * @throws AccessDeniedException if ticket doesn't allow this action
      * @throws ItemNotFoundException if node with uuid cannot be found
      * @throws FileException if unable to save node due to file error
@@ -187,7 +199,7 @@
      */
 	public NodeKey updateFileItem(ITicket ticket, Long uuid, String filename, 
 	   				InputStream istream, String mimeType, 
-					String versionDescription) 
+					String versionDescription, Integer userId) 
 	   		throws AccessDeniedException, ItemNotFoundException, 
 					FileException, InvalidParameterException;
 
@@ -205,14 +217,15 @@
 	 * @param startFile relative path of initial file - optional
 	 * @param versionDescription human readable comment about the version - optional
 	 * @return nodeKey (uuid and version)
+	 * @param userId user who is creating this node - mandatory
      * @throws AccessDeniedException if ticket doesn't allow this action
      * @throws ItemNotFoundException if node with uuid cannot be found
      * @throws FileException if unable to save node due to file error
      * @throws InvalidParameterException if a required parameter is missing
      * @throws RepositoryRuntimeException if any internal errors have occured 
 	 */
 	public abstract NodeKey updatePackageItem(ITicket ticket, Long uuid, String dirPath,
-			String startFile, String versionDescription)
+			String startFile, String versionDescription, Integer userId)
 			throws AccessDeniedException, ItemNotFoundException, 
 			FileException, InvalidParameterException ;
 
@@ -246,11 +259,12 @@
 	 * @param ticket ticket issued on login. Identifies tool and workspace - mandatory 
 	 * @param uuid id of the file/package - mandatory
 	 * @param version desired version - if null gets latest version
+	 * @param userId user who is creating this node - mandatory
 	 * @return nodeKey (uuid and version)
 	 * @throws AccessDeniedException if the ticket is invalid
 	 * @throws ItemNotFoundException if the node cannot be found
      */
-    public NodeKey copyNodeVersion(ITicket ticket, Long uuid, Long versionId) throws AccessDeniedException, ItemNotFoundException;
+    public NodeKey copyNodeVersion(ITicket ticket, Long uuid, Long versionId, Integer userId) throws AccessDeniedException, ItemNotFoundException;
 
 	/** 
 	 * Get an item from the repository based on the UUID. This
Index: lams_contentrepository/src/java/org/lamsfoundation/lams/contentrepository/service/NodeFactory.java
===================================================================
diff -u -r41de4a8b5ff40c43e8eaadc76a33ebce112d6801 -r9ad43dec7eb0960145bc92da0ae6435a2c412a58
--- lams_contentrepository/src/java/org/lamsfoundation/lams/contentrepository/service/NodeFactory.java	(.../NodeFactory.java)	(revision 41de4a8b5ff40c43e8eaadc76a33ebce112d6801)
+++ lams_contentrepository/src/java/org/lamsfoundation/lams/contentrepository/service/NodeFactory.java	(.../NodeFactory.java)	(revision 9ad43dec7eb0960145bc92da0ae6435a2c412a58)
@@ -70,11 +70,12 @@
 	 * @see org.lamsfoundation.lams.contentrepository.service.INodeFactory#createFileNode(org.lamsfoundation.lams.contentrepository.CrWorkspace, org.lamsfoundation.lams.contentrepository.service.SimpleVersionedNode, java.lang.String, java.io.InputStream, java.lang.String, java.lang.String, java.lang.String)
 	 */
 	public SimpleVersionedNode createFileNode(CrWorkspace workspace, SimpleVersionedNode parentNode, 
-			String relPath, InputStream istream, String filename, String mimeType, String versionDescription) 
+			String relPath, InputStream istream, String filename, String mimeType, 
+			String versionDescription, Integer userId) 
 					throws org.lamsfoundation.lams.contentrepository.InvalidParameterException {
 
 		SimpleVersionedNode initialNodeVersion = createBasicNode(NodeType.FILENODE,
-				workspace, parentNode, relPath, versionDescription);
+				workspace, parentNode, relPath, versionDescription, userId);
 		initialNodeVersion.setFile(istream, filename, mimeType);
 
 		return initialNodeVersion;
@@ -84,11 +85,11 @@
 	 * @see org.lamsfoundation.lams.contentrepository.service.INodeFactory#createPackageNode(org.lamsfoundation.lams.contentrepository.CrWorkspace, java.lang.String, java.lang.String)
 	 */
 	public SimpleVersionedNode createPackageNode(CrWorkspace workspace, 
-			String initialPath, String versionDescription) 
+			String initialPath, String versionDescription, Integer userId) 
 					throws org.lamsfoundation.lams.contentrepository.InvalidParameterException {
 
 		SimpleVersionedNode initialNodeVersion = createBasicNode(NodeType.PACKAGENODE, 
-				workspace, null, null, versionDescription);
+				workspace, null, null, versionDescription, userId);
 		initialNodeVersion.setProperty(PropertyName.INITIALPATH, initialPath);
 
 		return initialNodeVersion;
@@ -98,24 +99,24 @@
 	 * @see org.lamsfoundation.lams.contentrepository.service.INodeFactory#createDataNode(org.lamsfoundation.lams.contentrepository.CrWorkspace, org.lamsfoundation.lams.contentrepository.service.SimpleVersionedNode, java.lang.String)
 	 */
 	public SimpleVersionedNode createDataNode(CrWorkspace workspace, SimpleVersionedNode parentNode, 
-			String versionDescription) 
+			String versionDescription, Integer userId) 
 					throws org.lamsfoundation.lams.contentrepository.InvalidParameterException {
 
 		SimpleVersionedNode initialNodeVersion = createBasicNode(NodeType.DATANODE, 
-				workspace, parentNode, null, versionDescription);
+				workspace, parentNode, null, versionDescription, userId);
 			
 		return initialNodeVersion;
 	}
 	
 	/** Create the core part of a node */
 	private SimpleVersionedNode createBasicNode(String nodeType, CrWorkspace workspace, SimpleVersionedNode parentNode, 
-			String relPath, String versionDescription) {
+			String relPath, String versionDescription, Integer userId) {
 		
 		SimpleVersionedNode initialNodeVersion = (SimpleVersionedNode) beanFactory.getBean("node", SimpleVersionedNode.class);
 
 		Date createdDate = new Date(System.currentTimeMillis());
 		CrNodeVersion parentNodeVersion = parentNode != null ? parentNode.getNodeVersion() : null; 
-		CrNode node = new CrNode(relPath, nodeType, createdDate, workspace, parentNodeVersion, versionDescription);
+		CrNode node = new CrNode(relPath, nodeType, createdDate, userId, workspace, parentNodeVersion, versionDescription);
 		CrNodeVersion nodeVersion = node.getNodeVersion(null);
 
 		initialNodeVersion.setNode(node);
@@ -172,7 +173,7 @@
 	/* (non-Javadoc)
 	 * @see org.lamsfoundation.lams.contentrepository.service.INodeFactory#getNode(java.lang.Long, java.lang.Long, java.lang.Long, java.lang.String)
 	 */
-	public SimpleVersionedNode getNodeNewVersion(Long workspaceId, Long uuid, Long versionId, String versionDescription) throws ItemNotFoundException {
+	public SimpleVersionedNode getNodeNewVersion(Long workspaceId, Long uuid, Long versionId, String versionDescription, Integer userId) throws ItemNotFoundException {
 
 		SimpleVersionedNode existingNode = getNode(workspaceId, uuid, versionId);
 		CrNode existingCrNode = existingNode.getNode();
@@ -182,7 +183,7 @@
 		
 		SimpleVersionedNode newNode = (SimpleVersionedNode) beanFactory.getBean("node", SimpleVersionedNode.class);
 		newNode.setNode(existingCrNode);
-		CrNodeVersion newVersion = new CrNodeVersion(existingCrNode, new Date(System.currentTimeMillis()), nextVersionId, versionDescription); 
+		CrNodeVersion newVersion = new CrNodeVersion(existingCrNode, new Date(System.currentTimeMillis()), nextVersionId, versionDescription, userId); 
 		newNode.setNodeVersion(newVersion);
 		existingCrNode.addCrNodeVersion(newVersion);
 		
@@ -192,9 +193,9 @@
 	/* (non-Javadoc)
 	 * @see org.lamsfoundation.lams.contentrepository.service.INodeFactory#copy(org.lamsfoundation.lams.contentrepository.service.SimpleVersionedNode)
 	 */
-	public SimpleVersionedNode copy( SimpleVersionedNode originalNode ) throws FileException, ValueFormatException, InvalidParameterException {
+	public SimpleVersionedNode copy( SimpleVersionedNode originalNode, Integer userId ) throws FileException, ValueFormatException, InvalidParameterException {
 		
-		return copy(originalNode, null);
+		return copy(originalNode, null, userId);
 	}
 	
 	/** Private method to handle the recursive copy. The parent node is needed to set up the
@@ -207,14 +208,14 @@
 	 * @throws ValueFormatException
 	 * @throws InvalidParameterException
 	 */
-	private SimpleVersionedNode copy( SimpleVersionedNode originalNode, CrNodeVersion parentNodeVersion ) throws FileException, ValueFormatException, InvalidParameterException {
+	private SimpleVersionedNode copy( SimpleVersionedNode originalNode, CrNodeVersion parentNodeVersion, Integer userId ) throws FileException, ValueFormatException, InvalidParameterException {
 		
 		SimpleVersionedNode newNode = (SimpleVersionedNode) beanFactory.getBean("node", SimpleVersionedNode.class);
 
 		// copy the basic CrNode/CrNodeVersion fields. Set a new timestamp and new ids
 		Date createdDate = new Date(System.currentTimeMillis());
 		
-		CrNode newCrNode = new CrNode(originalNode.getPath(), originalNode.getNodeType(), createdDate, 
+		CrNode newCrNode = new CrNode(originalNode.getPath(), originalNode.getNodeType(), createdDate, userId,
 				originalNode.getNode().getCrWorkspace(), parentNodeVersion, originalNode.getVersionDescription());
 		newNode.setNode(newCrNode);
 		
@@ -247,7 +248,7 @@
 		Iterator iter = originalNode.getChildNodes().iterator();
 		while ( iter.hasNext() ) {
 			SimpleVersionedNode childNode = (SimpleVersionedNode) iter.next();
-			SimpleVersionedNode newChildNode = this.copy(childNode, newCrNodeVersion);
+			SimpleVersionedNode newChildNode = this.copy(childNode, newCrNodeVersion, userId);
 			newNode.addChildNode(newChildNode);
 		}
 		
Index: lams_contentrepository/src/java/org/lamsfoundation/lams/contentrepository/service/SimpleRepository.java
===================================================================
diff -u -r02736c0c2d48d544c38aeecb76d71992fecffad7 -r9ad43dec7eb0960145bc92da0ae6435a2c412a58
--- lams_contentrepository/src/java/org/lamsfoundation/lams/contentrepository/service/SimpleRepository.java	(.../SimpleRepository.java)	(revision 02736c0c2d48d544c38aeecb76d71992fecffad7)
+++ lams_contentrepository/src/java/org/lamsfoundation/lams/contentrepository/service/SimpleRepository.java	(.../SimpleRepository.java)	(revision 9ad43dec7eb0960145bc92da0ae6435a2c412a58)
@@ -24,7 +24,6 @@
 /* $$Id$$ */	
 package org.lamsfoundation.lams.contentrepository.service;
 
-import java.io.File;
 import java.io.FileNotFoundException;
 import java.io.FileOutputStream;
 import java.io.IOException;
@@ -39,7 +38,6 @@
 import java.util.SortedSet;
 import java.util.TreeMap;
 
-import org.apache.commons.lang.StringUtils;
 import org.apache.log4j.Logger;
 import org.lamsfoundation.lams.contentrepository.AccessDeniedException;
 import org.lamsfoundation.lams.contentrepository.CrCredential;
@@ -64,8 +62,10 @@
 import org.lamsfoundation.lams.contentrepository.ValidationException;
 import org.lamsfoundation.lams.contentrepository.ValueFormatException;
 import org.lamsfoundation.lams.contentrepository.WorkspaceNotFoundException;
+import org.lamsfoundation.lams.contentrepository.client.IToolContentHandler;
 import org.lamsfoundation.lams.contentrepository.dao.ICredentialDAO;
 import org.lamsfoundation.lams.contentrepository.dao.IWorkspaceDAO;
+import org.lamsfoundation.lams.learningdesign.service.ImportToolContentException;
 import org.lamsfoundation.lams.util.FileUtil;
 import org.lamsfoundation.lams.util.FileUtilException;
 import org.lamsfoundation.lams.util.zipfile.ZipFileUtil;
@@ -361,18 +361,31 @@
 		return ( ticket != null &&  ticketIdSet.contains(ticket.getTicketId()) );
     }
 
+	/**
+	 * Import tool content 
+	 * TODO Remove once all tools converted to supply user id. 
+	 */
+	public NodeKey addFileItem(ITicket ticket, InputStream istream,
+			String filename, String mimeType, String versionDescription)
+			throws FileException, AccessDeniedException,
+			InvalidParameterException {
+		log.error("addFileItem() to be removed - it sets the owner of files in the content repository to 1. Some tool needs to be updated.");
+		return addFileItem(ticket, istream,
+				filename, mimeType, versionDescription, new Integer(1));
+	}
+
 	/* (non-Javadoc)
 	 * @see org.lamsfoundation.lams.contentrepository.IRepository#addFileItem(org.lamsfoundation.lams.contentrepository.ITicket, java.io.InputStream, java.lang.String, java.lang.String, java.lang.String)
 	 */
 	public NodeKey addFileItem(ITicket ticket, InputStream istream,
-			String filename, String mimeType, String versionDescription)
+			String filename, String mimeType, String versionDescription, Integer userId)
 			throws FileException, AccessDeniedException,
 			InvalidParameterException {
 		
     	try { 
     		CrWorkspace workspace = getWorkspace(ticket.getWorkspaceId());
     		SimpleVersionedNode initialNodeVersion = nodeFactory.createFileNode(workspace, 
-    				 null, null, istream, filename, mimeType, versionDescription); 
+    				 null, null, istream, filename, mimeType, versionDescription, userId); 
        		initialNodeVersion.save();
     		return initialNodeVersion.getNodeKey();
 		} catch ( ValidationException e) {
@@ -391,7 +404,7 @@
 	 * @see org.lamsfoundation.lams.contentrepository.IRepository#addPackageItem(org.lamsfoundation.lams.contentrepository.ITicket, java.lang.String, java.lang.String, java.lang.String)
 	 */
 	public NodeKey addPackageItem(ITicket ticket, String dirPath,
-			String startFile, String versionDescription)
+			String startFile, String versionDescription, Integer userId)
 			throws AccessDeniedException, InvalidParameterException,
 			FileException {
 		
@@ -406,10 +419,10 @@
 		}
 
     	SimpleVersionedNode packageNode = null;
-   		packageNode = nodeFactory.createPackageNode(workspace, startFile, versionDescription); 
+   		packageNode = nodeFactory.createPackageNode(workspace, startFile, versionDescription, userId); 
     	
 		try { 
-			packageNode.addPackageFiles(workspace, dirPath, versionDescription); 
+			packageNode.addPackageFiles(workspace, dirPath, versionDescription, userId); 
 			packageNode.save();
     	} catch ( ValidationException e) {
     		// if this is thrown, then it is bug - nothing external should cause it.
@@ -512,12 +525,12 @@
 	 * @see org.lamsfoundation.lams.contentrepository.IRepository#updateFileItem(org.lamsfoundation.lams.contentrepository.ITicket, java.lang.Long, java.lang.String, java.io.InputStream, java.lang.String, java.lang.String)
 	 */
 	public NodeKey updateFileItem(ITicket ticket, Long uuid, String filename,
-			InputStream istream, String mimeType, String versionDescription)
+			InputStream istream, String mimeType, String versionDescription, Integer userId)
 			throws AccessDeniedException, ItemNotFoundException, FileException,
 			InvalidParameterException {
 	   	
 		SimpleVersionedNode newNodeVersion = nodeFactory.getNodeNewVersion(ticket.getWorkspaceId(), 
-				uuid, null, versionDescription);
+				uuid, null, versionDescription, userId);
 
 	   	if ( ! newNodeVersion.isNodeType( NodeType.FILENODE) )
 	   		throw new InvalidParameterException("Node is not a file node - it is a "+newNodeVersion.getNodeType()
@@ -537,12 +550,12 @@
 	 * @see org.lamsfoundation.lams.contentrepository.IRepository#updatePackageItem(org.lamsfoundation.lams.contentrepository.ITicket, java.lang.Long, java.lang.String, java.lang.String, java.lang.String)
 	 */
 	public NodeKey updatePackageItem(ITicket ticket, Long uuid, String dirPath,
-			String startFile, String versionDescription)
+			String startFile, String versionDescription, Integer userId)
 			throws AccessDeniedException, ItemNotFoundException, FileException,
 			InvalidParameterException {
 		
 		SimpleVersionedNode newNodeVersion = nodeFactory.getNodeNewVersion(ticket.getWorkspaceId(), 
-				uuid, null, versionDescription);
+				uuid, null, versionDescription, userId);
 
 	   	if ( ! newNodeVersion.isNodeType( NodeType.PACKAGENODE) )
 	   		throw new InvalidParameterException("Node is not a package node - it is a "+newNodeVersion.getNodeType()
@@ -553,7 +566,7 @@
 		try { 
 			CrWorkspace workspace = getWorkspace(ticket.getWorkspaceId());
 			
-			newNodeVersion.addPackageFiles(workspace, dirPath, versionDescription); 
+			newNodeVersion.addPackageFiles(workspace, dirPath, versionDescription, userId); 
 			newNodeVersion.save();
 
     	} catch ( ValidationException e) {
@@ -591,12 +604,12 @@
     /* (non-Javadoc)
 	 * @see org.lamsfoundation.lams.contentrepository.IRepository#copyNodeVersion(org.lamsfoundation.lams.contentrepository.ITicket, java.lang.Long, java.lang.Long)
 	 */
-    public NodeKey copyNodeVersion(ITicket ticket, Long uuid, Long versionId) throws AccessDeniedException, ItemNotFoundException {
+    public NodeKey copyNodeVersion(ITicket ticket, Long uuid, Long versionId, Integer userId) throws AccessDeniedException, ItemNotFoundException {
 
         
     	try { 
             SimpleVersionedNode originalNode = nodeFactory.getNode(ticket.getWorkspaceId(),uuid,versionId);
-           	SimpleVersionedNode newNode = nodeFactory.copy(originalNode);
+           	SimpleVersionedNode newNode = nodeFactory.copy(originalNode, userId);
            	newNode.save();
     		return newNode.getNodeKey();
     		 
Index: lams_contentrepository/src/java/org/lamsfoundation/lams/contentrepository/service/SimpleVersionedNode.java
===================================================================
diff -u -r41de4a8b5ff40c43e8eaadc76a33ebce112d6801 -r9ad43dec7eb0960145bc92da0ae6435a2c412a58
--- lams_contentrepository/src/java/org/lamsfoundation/lams/contentrepository/service/SimpleVersionedNode.java	(.../SimpleVersionedNode.java)	(revision 41de4a8b5ff40c43e8eaadc76a33ebce112d6801)
+++ lams_contentrepository/src/java/org/lamsfoundation/lams/contentrepository/service/SimpleVersionedNode.java	(.../SimpleVersionedNode.java)	(revision 9ad43dec7eb0960145bc92da0ae6435a2c412a58)
@@ -309,6 +309,16 @@
 		
 	}
 	
+	/**
+	 * @see org.lamsfoundation.lams.contentrepository.IVersionedNode#getUserId()
+	 */
+	public Integer getUserId() {
+		nodeObjectInitilised("Unable to get user id.");
+		return nodeVersion.getUserId();
+		
+	}
+	
+
 	/** Get the file, as an inputstream. It is the responsibility 
 	 * of the caller to close the stream. Note: this should only be 
 	 * called once the node is saved - do not call it directly after
@@ -803,7 +813,7 @@
 	 * @throws InvalidParameterException
 	 * @throws FileException
 	 */
-	protected void addPackageFiles(CrWorkspace workspace, String dirPath, String versionDescription) 
+	protected void addPackageFiles(CrWorkspace workspace, String dirPath, String versionDescription, Integer userId) 
 				throws InvalidParameterException, FileException, ValidationException {
 		
     	File directory = new File(dirPath);
@@ -817,7 +827,7 @@
     	if ( removePathToMakeRelPath.charAt(removePathToMakeRelPath.length()-1) != File.separatorChar )
     		removePathToMakeRelPath += File.separatorChar;
 		
-    	processDirectory(workspace, removePathToMakeRelPath, directory, versionDescription);
+    	processDirectory(workspace, removePathToMakeRelPath, directory, versionDescription, userId);
 	}
 
 	/** 
@@ -838,7 +848,7 @@
 	 * not be null.
 	 * @throws FileException
 	 */
-	private void processDirectory(CrWorkspace workspace, String removePathToMakeRelPath, File dirFile, String versionDescription) 
+	private void processDirectory(CrWorkspace workspace, String removePathToMakeRelPath, File dirFile, String versionDescription, Integer userId) 
 			throws InvalidParameterException, FileException, ValidationException {
 
 		if ( ! dirFile.exists() || ! dirFile.isDirectory() || ! dirFile.canRead() ) {
@@ -858,7 +868,7 @@
 				if ( file.isDirectory() ) {
 					
 					// recurse to get files in this directory
-					processDirectory(workspace, removePathToMakeRelPath, file, versionDescription);
+					processDirectory(workspace, removePathToMakeRelPath, file, versionDescription, userId);
 					
 				} else {
 					
@@ -881,7 +891,7 @@
 					// the file node. Mime type is unknown.
 					// no need to the new node as a childe node, as createFileNode will do it.
 					FileInputStream istream = new FileInputStream(file);
-					nodeFactory.createFileNode(workspace, this, relPath, istream, filename, null, versionDescription);
+					nodeFactory.createFileNode(workspace, this, relPath, istream, filename, null, versionDescription, userId);
 				}
 			}
 			
@@ -915,9 +925,9 @@
 	 *   file, filename or mimetype properties are invalid.
 	 * @throws ValueFormatException will only occur if there is an internal bug as it will only happen if the filename or mimetype properties are not strings. 
 	 */
-	public SimpleVersionedNode copy( ) throws FileException, ValueFormatException, InvalidParameterException  {
+	public SimpleVersionedNode copy( Integer userId) throws FileException, ValueFormatException, InvalidParameterException  {
 		
-		return nodeFactory.copy(this);
+		return nodeFactory.copy(this, userId);
 
 	}
 
Index: lams_contentrepository/src/java/org/lamsfoundation/lams/contentrepository/struts/action/AddFileContentAction.java
===================================================================
diff -u -r41de4a8b5ff40c43e8eaadc76a33ebce112d6801 -r9ad43dec7eb0960145bc92da0ae6435a2c412a58
--- lams_contentrepository/src/java/org/lamsfoundation/lams/contentrepository/struts/action/AddFileContentAction.java	(.../AddFileContentAction.java)	(revision 41de4a8b5ff40c43e8eaadc76a33ebce112d6801)
+++ lams_contentrepository/src/java/org/lamsfoundation/lams/contentrepository/struts/action/AddFileContentAction.java	(.../AddFileContentAction.java)	(revision 9ad43dec7eb0960145bc92da0ae6435a2c412a58)
@@ -58,6 +58,7 @@
  */
 public class AddFileContentAction extends RepositoryDispatchAction {
 
+	public static final Integer dummyUserId = new Integer(1);
 	// --------------------------------------------------------- Instance Variables
 
 	// --------------------------------------------------------- Methods
@@ -120,11 +121,11 @@
 			if ( uuid != null ) {
 				// create a new version of this node
 				nodeKey = getRepository().updateFileItem(ticket, uuid, filename,
-						is, contentType, versionDescription);
+						is, contentType, versionDescription, dummyUserId);
 			} else {
 				// create a new node
 				nodeKey = getRepository().addFileItem(ticket, is, filename, 
-						contentType, versionDescription);
+						contentType, versionDescription, dummyUserId);
 			}
 
 			// normally the application would store the node key and
@@ -198,10 +199,10 @@
 			if ( uuid != null ) {
 				// create a new version of this node
 				nodeKey = getRepository().updatePackageItem(ticket, uuid, dirName,
-						entryString, versionDescription);
+						entryString, versionDescription, dummyUserId);
 			} else {
 				// create a new node
-				nodeKey = getRepository().addPackageItem(ticket,dirName, entryString, versionDescription);
+				nodeKey = getRepository().addPackageItem(ticket,dirName, entryString, versionDescription, dummyUserId);
 			}
 			
 			// normally the application would store the node key and
Index: lams_contentrepository/src/java/org/lamsfoundation/lams/contentrepository/struts/action/NodeSelectionAction.java
===================================================================
diff -u -r41de4a8b5ff40c43e8eaadc76a33ebce112d6801 -r9ad43dec7eb0960145bc92da0ae6435a2c412a58
--- lams_contentrepository/src/java/org/lamsfoundation/lams/contentrepository/struts/action/NodeSelectionAction.java	(.../NodeSelectionAction.java)	(revision 41de4a8b5ff40c43e8eaadc76a33ebce112d6801)
+++ lams_contentrepository/src/java/org/lamsfoundation/lams/contentrepository/struts/action/NodeSelectionAction.java	(.../NodeSelectionAction.java)	(revision 9ad43dec7eb0960145bc92da0ae6435a2c412a58)
@@ -305,7 +305,7 @@
 		log.debug("Copy node "+uuid+" version "+version);
 
 		try {
-			NodeKey key = getRepository().copyNodeVersion(ticket,uuid,version);
+			NodeKey key = getRepository().copyNodeVersion(ticket,uuid,version, AddFileContentAction.dummyUserId);
 			log.info("Copy node, new ids "+key);
 			return getList(mapping,form,request,response);
 
Index: lams_contentrepository/test/java/org/lamsfoundation/lams/contentrepository/client/TestToolContentHandlerImpl.java
===================================================================
diff -u -r41de4a8b5ff40c43e8eaadc76a33ebce112d6801 -r9ad43dec7eb0960145bc92da0ae6435a2c412a58
--- lams_contentrepository/test/java/org/lamsfoundation/lams/contentrepository/client/TestToolContentHandlerImpl.java	(.../TestToolContentHandlerImpl.java)	(revision 41de4a8b5ff40c43e8eaadc76a33ebce112d6801)
+++ lams_contentrepository/test/java/org/lamsfoundation/lams/contentrepository/client/TestToolContentHandlerImpl.java	(.../TestToolContentHandlerImpl.java)	(revision 9ad43dec7eb0960145bc92da0ae6435a2c412a58)
@@ -104,9 +104,9 @@
     public void testUploadFile() {
         try {
             NodeKey offlineNodeKey = handler.uploadFile(CRResources.getSingleFile(), CRResources.singleFileName, 
-                    CRResources.singleFileMimeType, ToolContentHandlerImpl.TYPE_OFFLINE);
+                    CRResources.singleFileMimeType, ToolContentHandlerImpl.TYPE_OFFLINE, mmmUser);
             NodeKey onlineNodeKey = handler.uploadFile(CRResources.getSingleFile(), CRResources.singleFileName, 
-                    CRResources.singleFileMimeType, ToolContentHandlerImpl.TYPE_ONLINE);
+                    CRResources.singleFileMimeType, ToolContentHandlerImpl.TYPE_ONLINE, mmmUser);
 
             Long uuid = offlineNodeKey.getUuid();
             IVersionedNode node = handler.getFileNode(uuid);
@@ -137,7 +137,7 @@
         try {
 		    // unpack the zip file so we have a directory to play with 
     	    String tempDir = ZipFileUtil.expandZip(CRResources.getZipFile(), CRResources.zipFileName);
-		    NodeKey nodeKey = handler.uploadPackage(tempDir, "index.html"); 
+		    NodeKey nodeKey = handler.uploadPackage(tempDir, "index.html", mmmUser); 
 
 		    Long uuid = nodeKey.getUuid();
             IVersionedNode node = handler.getFileNode(uuid);
Index: lams_contentrepository/test/java/org/lamsfoundation/lams/contentrepository/service/BaseTestCase.java
===================================================================
diff -u -r41de4a8b5ff40c43e8eaadc76a33ebce112d6801 -r9ad43dec7eb0960145bc92da0ae6435a2c412a58
--- lams_contentrepository/test/java/org/lamsfoundation/lams/contentrepository/service/BaseTestCase.java	(.../BaseTestCase.java)	(revision 41de4a8b5ff40c43e8eaadc76a33ebce112d6801)
+++ lams_contentrepository/test/java/org/lamsfoundation/lams/contentrepository/service/BaseTestCase.java	(.../BaseTestCase.java)	(revision 9ad43dec7eb0960145bc92da0ae6435a2c412a58)
@@ -57,6 +57,7 @@
 	protected static final Long INITIAL_WORKSPACE_ID = new Long(1);
 	protected static final String INITIAL_WORKSPACE_USER = "atool";
 	protected static final char[] INITIAL_WORKSPACE_PASSWORD = {'a','t','o','o','l'};
+	protected static final Integer mmmUser = new Integer(4); 
 	
 	protected static final String SECONDARY_WORKSPACE = "atoolWorkspace2";
 
@@ -92,8 +93,9 @@
 
 	protected String[] getContextConfigLocation() {
 		return new String[] {
-				IRepositoryService.CORE_LOCAL_CONTEXT_PATH,
-		        IRepositoryService.REPOSITORY_CONTEXT_PATH,
+				"/org/lamsfoundation/lams/contentrepository/applicationContext.xml",
+				"/org/lamsfoundation/lams/localApplicationContext.xml",
+				"/org/lamsfoundation/lams/toolApplicationContext.xml",
 		        "/org/lamsfoundation/lams/contentrepository/testApplicationContext.xml"
 		        };
 	}
Index: lams_contentrepository/test/java/org/lamsfoundation/lams/contentrepository/service/TestSimpleRepository.java
===================================================================
diff -u -r41de4a8b5ff40c43e8eaadc76a33ebce112d6801 -r9ad43dec7eb0960145bc92da0ae6435a2c412a58
--- lams_contentrepository/test/java/org/lamsfoundation/lams/contentrepository/service/TestSimpleRepository.java	(.../TestSimpleRepository.java)	(revision 41de4a8b5ff40c43e8eaadc76a33ebce112d6801)
+++ lams_contentrepository/test/java/org/lamsfoundation/lams/contentrepository/service/TestSimpleRepository.java	(.../TestSimpleRepository.java)	(revision 9ad43dec7eb0960145bc92da0ae6435a2c412a58)
@@ -436,15 +436,15 @@
 		try {
 			if ( uuid == null ) {
 				// new file
-				keys = repository.addFileItem(ticket, file, filename, null, v1Description);
+				keys = repository.addFileItem(ticket, file, filename, null, v1Description, mmmUser);
 				assertTrue("File save returns uuid",keys != null && keys.getUuid() != null);
 				assertTrue("File save got expected version "+expectedVersion,
 					keys != null && keys.getVersion() != null 
 					&& keys.getVersion().equals(expectedVersion));
 			} else {
 				// update existing node
 				keys = repository.updateFileItem(ticket, uuid, filename, 
-				        file, null, v2Description);
+				        file, null, v2Description, mmmUser);
 				assertTrue("File save returns same uuid",keys != null && keys.getUuid().equals(uuid));
 				assertTrue("File save got expected version "+expectedVersion,
 					keys != null && keys.getVersion() != null 
@@ -484,8 +484,8 @@
 
 	   // copy each version of the file node
 		try {
-			NodeKey copy1 = repository.copyNodeVersion(ticket, keys.getUuid(), one);
-			NodeKey copylatest = repository.copyNodeVersion(ticket, keys.getUuid(), null);
+			NodeKey copy1 = repository.copyNodeVersion(ticket, keys.getUuid(), one, mmmUser);
+			NodeKey copylatest = repository.copyNodeVersion(ticket, keys.getUuid(), null, mmmUser);
 
 			assertNotSame("Copy 1 is a different uuid to the original.", copy1.getUuid(), keys.getUuid());
 			checkFileNodeExist(fileDAO, copy1.getUuid(), one, 1, v1Description);
@@ -562,13 +562,13 @@
 			directory.list();
 
 			if ( uuid == null ) {
-				keys = repository.addPackageItem(ticket,  tempDir, "index.html", v1Description);
+				keys = repository.addPackageItem(ticket,  tempDir, "index.html", v1Description, mmmUser);
 				assertTrue("Package save returns uuid",keys != null && keys.getUuid() != null);
 				assertTrue("Package save got version 1",
 						keys != null && keys.getVersion() != null 
 						&& keys.getVersion().longValue() == 1);
 			} else {
-				keys = repository.updatePackageItem(ticket, uuid, tempDir, "index.html", v2Description);
+				keys = repository.updatePackageItem(ticket, uuid, tempDir, "index.html", v2Description, mmmUser);
 				assertTrue("Package save returns uuid",keys != null && keys.getUuid() != null);
 				assertTrue("Package save got version >1",
 						keys != null && keys.getVersion() != null 
@@ -606,7 +606,7 @@
 
 	    // copy the package node
 		try {
-			NodeKey copy = repository.copyNodeVersion(ticket, keys.getUuid(), null);
+			NodeKey copy = repository.copyNodeVersion(ticket, keys.getUuid(), null, mmmUser);
 
 			assertNotSame("Copy 1 is a different uuid to the original.", copy.getUuid(), keys.getUuid());
 			checkPackage(copy);
Index: lams_contentrepository/test/java/org/lamsfoundation/lams/contentrepository/service/TestSimpleVersionedNode.java
===================================================================
diff -u -r41de4a8b5ff40c43e8eaadc76a33ebce112d6801 -r9ad43dec7eb0960145bc92da0ae6435a2c412a58
--- lams_contentrepository/test/java/org/lamsfoundation/lams/contentrepository/service/TestSimpleVersionedNode.java	(.../TestSimpleVersionedNode.java)	(revision 41de4a8b5ff40c43e8eaadc76a33ebce112d6801)
+++ lams_contentrepository/test/java/org/lamsfoundation/lams/contentrepository/service/TestSimpleVersionedNode.java	(.../TestSimpleVersionedNode.java)	(revision 9ad43dec7eb0960145bc92da0ae6435a2c412a58)
@@ -409,9 +409,9 @@
 	public void testAddNode() {
 		try { 
 			// file nodes are tested in another test
-			SimpleVersionedNode dataNode = nodeFactory.createDataNode(getWorkspace(INITIAL_WORKSPACE_ID), null,"Initial Version");
+			SimpleVersionedNode dataNode = nodeFactory.createDataNode(getWorkspace(INITIAL_WORKSPACE_ID), null,"Initial Version", mmmUser);
 
-			SimpleVersionedNode packageNode = nodeFactory.createPackageNode(getWorkspace(INITIAL_WORKSPACE_ID), "index.html", "Initial Version");
+			SimpleVersionedNode packageNode = nodeFactory.createPackageNode(getWorkspace(INITIAL_WORKSPACE_ID), "index.html", "Initial Version", mmmUser);
 			
 			assertTrue("2 nodes created, but not saved into database ", 
 					dataNode != null && dataNode.getUUID() == null
@@ -474,7 +474,7 @@
 				
 			try {
 				if ( newUuid  == null ) {
-					fileNode = nodeFactory.createFileNode(getWorkspace(INITIAL_WORKSPACE_ID), null, null, null, filename, null, null);
+					fileNode = nodeFactory.createFileNode(getWorkspace(INITIAL_WORKSPACE_ID), null, null, null, filename, null, null, mmmUser);
 				} else {
 					fileNode = (SimpleVersionedNode) getNode(INITIAL_WORKSPACE_ID, newUuid);
 					fileNode.setFile(null, filename, null);
@@ -486,7 +486,7 @@
 
 			try {
 				if ( newUuid  == null ) {
-					fileNode = nodeFactory.createFileNode(getWorkspace(INITIAL_WORKSPACE_ID), null, null, is, null, null, null);
+					fileNode = nodeFactory.createFileNode(getWorkspace(INITIAL_WORKSPACE_ID), null, null, is, null, null, null, mmmUser);
 				} else {
 					fileNode = (SimpleVersionedNode) getNode(INITIAL_WORKSPACE_ID, newUuid);
 					fileNode.setFile(is, null, null);
@@ -497,7 +497,7 @@
 			}
 
 			
-			fileNode = nodeFactory.createFileNode(getWorkspace(INITIAL_WORKSPACE_ID), null, null, is, filename, null, null);
+			fileNode = nodeFactory.createFileNode(getWorkspace(INITIAL_WORKSPACE_ID), null, null, is, filename, null, null, mmmUser);
 			fileNode.setFile(is, filename, null);
 			saveCheckNode("File", NodeType.FILENODE, fileNode);
 			newUuid = fileNode.getUUID();
@@ -562,7 +562,7 @@
 	public void testAddNodeBadType() {
 		try {
 	
-			SimpleVersionedNode newNode = nodeFactory.createDataNode(getWorkspace(INITIAL_WORKSPACE_ID), null,"Initial Version");
+			SimpleVersionedNode newNode = nodeFactory.createDataNode(getWorkspace(INITIAL_WORKSPACE_ID), null,"Initial Version", mmmUser);
 			newNode.getNode().setType("XYZ");
 			newNode.save();
 			fail("NoSuchNodeTypeException should have been thrown");