LPS-206531 Using RepositoryId instead of themeDisplay in order to dis…

…play the tags/categories assignable to the content.
marcellustavares · Jan 30, 2024 · 2704782 · 2704782
1 parent 8b892ac
commit 2704782
Show file tree

Hide file tree

Showing 4 changed files with 23 additions and 1 deletion.
diff --git a/...java/com/liferay/document/library/web/internal/display/context/DLAdminDisplayContext.java b/...java/com/liferay/document/library/web/internal/display/context/DLAdminDisplayContext.java
@@ -309,6 +309,23 @@ public String getRememberCheckBoxStateURLRegex() {
 			"\\?");
 	}
 
+	public long getRepositoryGroupId(long scopeGroupId, long repositoryId) {
+		Repository repository = RepositoryLocalServiceUtil.fetchRepository(
+			repositoryId);
+
+		if (repository != null) {
+			return repository.getGroupId();
+		}
+
+		Group group = GroupLocalServiceUtil.fetchGroup(repositoryId);
+
+		if (group != null) {
+			return group.getGroupId();
+		}
+
+		return scopeGroupId;
+	}
+
 	public long getRepositoryId() {
 		if (_repositoryId != 0) {
 			return _repositoryId;

diff --git a/...document/library/web/internal/display/context/DLAdminManagementToolbarDisplayContext.java b/...document/library/web/internal/display/context/DLAdminManagementToolbarDisplayContext.java
@@ -908,7 +908,9 @@ private long[] _getGroupIds() {
 			_groupIds =
 				_siteConnectedGroupGroupProvider.
 					getCurrentAndAncestorSiteAndDepotGroupIds(
-						_themeDisplay.getScopeGroupId());
+						_dlAdminDisplayContext.getRepositoryGroupId(
+							_themeDisplay.getScopeGroupId(),
+							_dlAdminDisplayContext.getRepositoryId()));
 		}
 		catch (PortalException portalException) {
 			if (_log.isDebugEnabled()) {

diff --git a/...nt-library-web/src/main/resources/META-INF/resources/document_library/edit_file_entry.jsp b/...nt-library-web/src/main/resources/META-INF/resources/document_library/edit_file_entry.jsp
@@ -543,12 +543,14 @@ renderResponse.setTitle(headerTitle);
 								className="<%= DLFileEntry.class.getName() %>"
 								classPK="<%= assetClassPK %>"
 								classTypePK="<%= (fileEntryTypeId < 0) ? DLFileEntryTypeConstants.FILE_ENTRY_TYPE_ID_BASIC_DOCUMENT : fileEntryTypeId %>"
+								groupIds="<%= SiteConnectedGroupGroupProviderUtil.getCurrentAndAncestorSiteAndDepotGroupIds(dlAdminDisplayContext.getRepositoryGroupId(scopeGroupId, repositoryId)) %>"
 								visibilityTypes="<%= AssetVocabularyConstants.VISIBILITY_TYPES %>"
 							/>
 
 							<liferay-asset:asset-tags-selector
 								className="<%= DLFileEntry.class.getName() %>"
 								classPK="<%= assetClassPK %>"
+								groupIds="<%= SiteConnectedGroupGroupProviderUtil.getCurrentAndAncestorSiteAndDepotGroupIds(dlAdminDisplayContext.getRepositoryGroupId(scopeGroupId, repositoryId)) %>"
 							/>
 
 							<c:if test="<%= (fileEntry != null) && dlAdminDisplayContext.isAutoTaggingEnabled() %>">

diff --git a/...rary/document-library-web/src/main/resources/META-INF/resources/document_library/init.jsp b/...rary/document-library-web/src/main/resources/META-INF/resources/document_library/init.jsp
@@ -12,6 +12,7 @@ taglib uri="http://liferay.com/tld/react" prefix="react" %><%@
 taglib uri="http://liferay.com/tld/user" prefix="liferay-user" %>
 
 <%@ page import="com.liferay.bulk.selection.BulkSelectionRunner" %><%@
+page import="com.liferay.depot.util.SiteConnectedGroupGroupProviderUtil" %><%@
 page import="com.liferay.digital.signature.constants.DigitalSignaturePortletKeys" %><%@
 page import="com.liferay.document.library.configuration.DLConfiguration" %><%@
 page import="com.liferay.document.library.exception.DLFileEntryConfigurationException" %><%@