From cddec89ec218b9faf2c3dbd0d8e111218c878976 Mon Sep 17 00:00:00 2001
From: Boubaker Khanfir <boubaker.khanfir@gmail.com>
Date: Thu, 16 Jan 2025 15:43:17 +0100
Subject: [PATCH] feat: Change default path computing algorithm - MEED-8176 -
 Meeds-io/MIPs#173 Prior to this change, when the user attempts to access
 anonymously to the portal using root website url (/), the request is
 forwarded to the first anonymously accessible site which isn't marked as 'non
 default site' (like spaces public sites). This change will adopt this new
 strategy: - When the user accesses to the Website and the public site is
 anonymously accessible, the forward request to the public site - When the
 user accesses to the Website and the public site isn't anonymously
 accessible, the forward request to the login page (even if other sites are
 publically accessible) - When the user is authenticated and has a favorite
 path, then forward the request to the user favorite home path - When the user
 is authenticated and doesn't have a favorite path, then forward the user to
 the first accessible path switch display order as established in Sidebar Menu
 which changes the display Order automatically.

---
 .../config/UserPortalConfigService.java       | 42 ++++++++++++-------
 1 file changed, 28 insertions(+), 14 deletions(-)

diff --git a/component/portal/src/main/java/org/exoplatform/portal/config/UserPortalConfigService.java b/component/portal/src/main/java/org/exoplatform/portal/config/UserPortalConfigService.java
index bd80b74531..56cdf626eb 100644
--- a/component/portal/src/main/java/org/exoplatform/portal/config/UserPortalConfigService.java
+++ b/component/portal/src/main/java/org/exoplatform/portal/config/UserPortalConfigService.java
@@ -310,17 +310,8 @@ public String getDefaultPath(String username) {
     if (StringUtils.isNotBlank(userHomePage)) {
       return userHomePage;
     } else {
-      List<PortalConfig> portalConfigList = getAccessiblePortalSites(username);
-      if (CollectionUtils.isEmpty(portalConfigList)) {
-        return null;
-      }
-      return portalConfigList.stream()
-                             .filter(PortalConfig::isDefaultSite)
-                             .filter(site -> PortalConfig.PORTAL_TYPE.equalsIgnoreCase(site.getType()))
-                             .map(portalConfig -> getDefaultSitePath(portalConfig.getName(), username))
-                             .filter(Objects::nonNull)
-                             .findFirst()
-                             .orElse(null);
+      PortalConfig portalConfig = getDefaultSite(username);
+      return portalConfig == null ? null : getDefaultSitePath(portalConfig.getName(), username);
     }
   }
 
@@ -358,7 +349,7 @@ public UserNode getSiteRootNode(String siteType, String siteName, String usernam
     if (StringUtils.equalsIgnoreCase(siteType, PortalConfig.PORTAL_TYPE)) {
       userPortalConfig = getUserPortalConfig(siteName, username);
     } else {
-      PortalConfig defaultPortalConfig = getAccessiblePortalSites(username).stream().findFirst().orElse(null);
+      PortalConfig defaultPortalConfig = getDefaultSite(username);
       if (defaultPortalConfig != null) {
         userPortalConfig = getUserPortalConfig(defaultPortalConfig.getName(), username);
       }
@@ -380,6 +371,29 @@ public UserNode getSiteRootNode(String siteType, String siteName, String usernam
     return userPortal.getNode(navigation, org.exoplatform.portal.mop.navigation.Scope.ALL, builder.build(), null);
   }
 
+  public PortalConfig getDefaultSite(String username) {
+    if (StringUtils.isBlank(username)) {
+      PortalConfig publicSitePortalConfig = layoutService.getPortalConfig(SiteKey.portal(PUBLIC_SITE_NAME));
+      if (publicSitePortalConfig == null || !userAcl.hasAccessPermission(publicSitePortalConfig, null)) {
+        return null;
+      } else {
+        return publicSitePortalConfig;
+      }
+    } else {
+      List<PortalConfig> portalConfigList = getAccessiblePortalSites(username);
+      if (CollectionUtils.isEmpty(portalConfigList)) {
+        return null;
+      } else {
+        return portalConfigList.stream()
+                               .filter(PortalConfig::isDefaultSite)
+                               .filter(p -> PortalConfig.PORTAL_TYPE.equalsIgnoreCase(p.getType()))
+                               .filter(Objects::nonNull)
+                               .findFirst()
+                               .orElse(null);
+      }
+    }
+  }
+
   public UserNode getSiteNodeOrGlobalNode(String portalType, // NOSONAR
                                           String portalName,
                                           String nodePath,
@@ -517,8 +531,8 @@ public void setUserHomePage(String username, String path) {
    * Returns the default portal template to be used when creating a site
    *
    * @return the default portal template name
-   * @deprecated Site Templates has been changed to be stored in database to make
-   *             it dynamically managed by UI rather than statis pages and
+   * @deprecated Site Templates has been changed to be stored in database to
+   *             make it dynamically managed by UI rather than statis pages and
    *             navigation from source files
    */
   @Deprecated(forRemoval = true, since = "7.0")