X-Git-Url: https://git.argeo.org/?a=blobdiff_plain;f=org.argeo.cms%2Fsrc%2Forg%2Fargeo%2Fcms%2Finternal%2Fkernel%2FHomeRepository.java;h=bfcb940c1d8336c1d74f8b7fb8c12f85609f1fed;hb=6338d85d3f970dd0eb8845693ddad90a93b99d03;hp=574470398c451e5754233996caad58d28654fada;hpb=f06ad11d9b3c77604dbf439a7047a71b47abf06f;p=lgpl%2Fargeo-commons.git

diff --git a/org.argeo.cms/src/org/argeo/cms/internal/kernel/HomeRepository.java b/org.argeo.cms/src/org/argeo/cms/internal/kernel/HomeRepository.java
index 574470398..bfcb940c1 100644
--- a/org.argeo.cms/src/org/argeo/cms/internal/kernel/HomeRepository.java
+++ b/org.argeo.cms/src/org/argeo/cms/internal/kernel/HomeRepository.java
@@ -1,6 +1,8 @@
 package org.argeo.cms.internal.kernel;
 
 import java.security.PrivilegedAction;
+import java.text.SimpleDateFormat;
+import java.util.Date;
 import java.util.HashSet;
 import java.util.Set;
 
@@ -27,12 +29,16 @@ import org.argeo.node.NodeUtils;
  * Make sure each user has a home directory available in the default workspace.
  */
 class HomeRepository extends JcrRepositoryWrapper implements KernelConstants {
+
 	/** The home base path. */
 	private String homeBasePath = KernelConstants.DEFAULT_HOME_BASE_PATH;
+	private String usersBasePath = KernelConstants.DEFAULT_USERS_BASE_PATH;
 	private String groupsBasePath = KernelConstants.DEFAULT_GROUPS_BASE_PATH;
 
 	private Set<String> checkedUsers = new HashSet<String>();
 
+	private SimpleDateFormat usersDatePath = new SimpleDateFormat("YYYY/MM");
+
 	public HomeRepository(Repository repository) {
 		super(repository);
 		putDescriptor(NodeConstants.CN, NodeConstants.HOME);
@@ -102,7 +108,7 @@ class HomeRepository extends JcrRepositoryWrapper implements KernelConstants {
 		try {
 			Node userHome = NodeUtils.getUserHome(session, username);
 			if (userHome == null) {
-				String homePath = generateUserPath(homeBasePath, username);
+				String homePath = generateUserPath(username);
 				if (session.itemExists(homePath))// duplicate user id
 					userHome = session.getNode(homePath).getParent().addNode(JcrUtils.lastPathElement(homePath));
 				else
@@ -124,7 +130,7 @@ class HomeRepository extends JcrRepositoryWrapper implements KernelConstants {
 	}
 
 	/** Generate path for a new user home */
-	private String generateUserPath(String base, String username) {
+	private String generateUserPath(String username) {
 		LdapName dn;
 		try {
 			dn = new LdapName(username);
@@ -133,15 +139,20 @@ class HomeRepository extends JcrRepositoryWrapper implements KernelConstants {
 		}
 		String userId = dn.getRdn(dn.size() - 1).getValue().toString();
 		int atIndex = userId.indexOf('@');
-		if (atIndex > 0) {
-			String domain = userId.substring(0, atIndex);
-			String name = userId.substring(atIndex + 1);
-			return base + '/' + domain + '/' + name;
-		} else if (atIndex == 0 || atIndex == (userId.length() - 1)) {
-			throw new CmsException("Unsupported username " + userId);
+		if (atIndex < 0) {
+			return homeBasePath + '/' + userId;
 		} else {
-			return base + '/' + userId;
+			return usersBasePath + '/' + usersDatePath.format(new Date()) + '/' + userId;
 		}
+		// if (atIndex > 0) {
+		// String domain = userId.substring(0, atIndex);
+		// String name = userId.substring(atIndex + 1);
+		// return base + '/' + domain + '/' + name;
+		// } else if (atIndex == 0 || atIndex == (userId.length() - 1)) {
+		// throw new CmsException("Unsupported username " + userId);
+		// } else {
+		// return base + '/' + userId;
+		// }
 	}
 
 	public void createWorkgroup(LdapName dn) {