From afc65c42656cd6691fe06359eb1ac362e35d9930 Mon Sep 17 00:00:00 2001 From: Yasa-Nataliya Date: Wed, 13 Sep 2023 12:09:46 +0530 Subject: [PATCH] [ACS-5540] modified the aspect button changes --- .../aca-content/assets/app.extensions.json | 2 +- .../aca-content/src/lib/aca-content.module.ts | 1 + .../components/details/details.component.html | 20 +--- .../components/details/details.component.scss | 34 ++++--- .../details/details.component.spec.ts | 30 +----- .../components/details/details.component.ts | 38 +++----- .../toggle-edit-offline.component.spec.ts | 97 +------------------ .../toggle-edit-offline.component.ts | 40 ++------ .../src/lib/services/node-actions.service.ts | 11 +-- projects/aca-shared/rules/src/app.rules.ts | 2 + .../aca-shared/rules/src/navigation.rules.ts | 5 + 11 files changed, 64 insertions(+), 216 deletions(-) diff --git a/projects/aca-content/assets/app.extensions.json b/projects/aca-content/assets/app.extensions.json index c65d769e9..f75b3dd51 100644 --- a/projects/aca-content/assets/app.extensions.json +++ b/projects/aca-content/assets/app.extensions.json @@ -1261,7 +1261,7 @@ "click": "EXPAND_INFO_DRAWER" }, "rules": { - "visible": "app.navigation.isNotLibraries" + "visible": "canNotShowExpand" } } ], diff --git a/projects/aca-content/src/lib/aca-content.module.ts b/projects/aca-content/src/lib/aca-content.module.ts index 47dd3bc2f..5df9187d8 100644 --- a/projects/aca-content/src/lib/aca-content.module.ts +++ b/projects/aca-content/src/lib/aca-content.module.ts @@ -192,6 +192,7 @@ export class ContentServiceExtensionModule { canToggleFavorite: rules.canToggleFavorite, isLibraryManager: rules.isLibraryManager, canEditAspects: rules.canEditAspects, + canNotShowExpand: rules.canNotShowExpand, canInfoPreview: rules.canInfoPreview, showInfoSelectionButton: rules.showInfoSelectionButton, diff --git a/projects/aca-content/src/lib/components/details/details.component.html b/projects/aca-content/src/lib/components/details/details.component.html index ff3378f3a..afcde3c44 100644 --- a/projects/aca-content/src/lib/components/details/details.component.html +++ b/projects/aca-content/src/lib/components/details/details.component.html @@ -12,21 +12,11 @@ - {{ 'APP.INFO_DRAWER.TITLE' | translate }} -
- -
diff --git a/projects/aca-content/src/lib/components/details/details.component.scss b/projects/aca-content/src/lib/components/details/details.component.scss index 778bead77..de481d464 100644 --- a/projects/aca-content/src/lib/components/details/details.component.scss +++ b/projects/aca-content/src/lib/components/details/details.component.scss @@ -1,19 +1,29 @@ app-details-manager { - .aca-close-details-button { - margin-right: 15px; - margin-top: 2px; - outline: none; - border-radius: 4px; + .acs-details-buttons { + display: flex; - &:focus { - background-color: var(--theme-selected-background-color); - outline: 2px solid var(--theme-blue-button-color); + .aca-close-details-button { + margin-right: 15px; + margin-top: 2px; + outline: none; border-radius: 4px; - } - &:focus-visible { - outline: 2px solid var(--theme-blue-button-color); - border-radius: 4px; + &:focus { + background-color: var(--theme-selected-background-color); + outline: 2px solid var(--theme-blue-button-color); + border-radius: 4px; + } + + &:focus-visible { + outline: 2px solid var(--theme-blue-button-color); + border-radius: 4px; + margin-top: 12px; + + &:focus-visible { + outline: 2px solid var(--theme-blue-button-color); + border-radius: 4px; + } + } } } } diff --git a/projects/aca-content/src/lib/components/details/details.component.spec.ts b/projects/aca-content/src/lib/components/details/details.component.spec.ts index 264445fbe..0988ca703 100644 --- a/projects/aca-content/src/lib/components/details/details.component.spec.ts +++ b/projects/aca-content/src/lib/components/details/details.component.spec.ts @@ -31,10 +31,10 @@ import { NO_ERRORS_SCHEMA } from '@angular/core'; import { Store } from '@ngrx/store'; import { ContentApiService } from '@alfresco/aca-shared'; import { STORE_INITIAL_APP_DATA, SetSelectedNodesAction } from '@alfresco/aca-shared/store'; -import { Node, NodeEntry } from '@alfresco/js-api'; +import { NodeEntry } from '@alfresco/js-api'; import { RouterTestingModule } from '@angular/router/testing'; import { AuthenticationService, PageTitleService } from '@alfresco/adf-core'; -import { NodeAspectService, SearchQueryBuilderService } from '@alfresco/adf-content-services'; +import { SearchQueryBuilderService } from '@alfresco/adf-content-services'; describe('DetailsComponent', () => { let component: DetailsComponent; @@ -42,7 +42,6 @@ describe('DetailsComponent', () => { let contentApiService: ContentApiService; let store: Store; let node: NodeEntry; - let nodeAspectService: NodeAspectService; const mockStream = new Subject(); const storeMock = { @@ -87,8 +86,6 @@ describe('DetailsComponent', () => { fixture = TestBed.createComponent(DetailsComponent); component = fixture.componentInstance; contentApiService = TestBed.inject(ContentApiService); - nodeAspectService = TestBed.inject(NodeAspectService); - component.node = { id: 'test-id' } as Node; store = TestBed.inject(Store); node = { @@ -106,7 +103,6 @@ describe('DetailsComponent', () => { } }; spyOn(contentApiService, 'getNode').and.returnValue(of(node)); - spyOn(nodeAspectService, 'updateNodeAspects'); }); afterEach(() => { @@ -132,26 +128,4 @@ describe('DetailsComponent', () => { fixture.detectChanges(); expect(store.dispatch).toHaveBeenCalledWith(new SetSelectedNodesAction([node])); }); - - it('should call updateNodeAspects when the aspect dialog is opened', () => { - component.openAspectDialog(); - fixture.detectChanges(); - expect(nodeAspectService.updateNodeAspects).toHaveBeenCalledWith('test-id'); - }); - - it('should subscribe to store and update isNodeLocked', () => { - const mockSelection = { file: { entry: { name: 'test', properties: {}, isLocked: false } } }; - spyOn(store, 'select').and.returnValue(of(mockSelection)); - fixture.detectChanges(); - expect(store.select).toHaveBeenCalled(); - expect(component.isNodeLocked).toBe(false); - }); - - it('should unsubscribe from observables on component destroy', () => { - spyOn(component.onDestroy$, 'next'); - spyOn(component.onDestroy$, 'complete'); - fixture.detectChanges(); - component.ngOnDestroy(); - expect(component.onDestroy$.complete).toHaveBeenCalled(); - }); }); diff --git a/projects/aca-content/src/lib/components/details/details.component.ts b/projects/aca-content/src/lib/components/details/details.component.ts index d454acac1..567d7c816 100644 --- a/projects/aca-content/src/lib/components/details/details.component.ts +++ b/projects/aca-content/src/lib/components/details/details.component.ts @@ -22,12 +22,12 @@ * from Hyland Software. If not, see . */ -import { Component, OnInit, ViewEncapsulation, OnDestroy, ChangeDetectorRef } from '@angular/core'; +import { Component, OnInit, ViewEncapsulation, OnDestroy } from '@angular/core'; import { ActivatedRoute } from '@angular/router'; -import { ContentApiService, PageComponent, PageLayoutComponent, ToolbarComponent, isLocked } from '@alfresco/aca-shared'; -import { NavigateToPreviousPage, SetSelectedNodesAction, getAppSelection } from '@alfresco/aca-shared/store'; +import { ContentApiService, PageComponent, PageLayoutComponent, ToolbarComponent } from '@alfresco/aca-shared'; +import { NavigateToPreviousPage, SetSelectedNodesAction } from '@alfresco/aca-shared/store'; import { Subject } from 'rxjs'; -import { BreadcrumbModule, PermissionManagerModule, NodeAspectService } from '@alfresco/adf-content-services'; +import { BreadcrumbModule, PermissionManagerModule } from '@alfresco/adf-content-services'; import { CommonModule } from '@angular/common'; import { TranslateModule } from '@ngx-translate/core'; import { MatIconModule } from '@angular/material/icon'; @@ -36,9 +36,8 @@ import { MatProgressBarModule } from '@angular/material/progress-bar'; import { MatButtonModule } from '@angular/material/button'; import { MetadataTabComponent } from '../info-drawer/metadata-tab/metadata-tab.component'; import { CommentsTabComponent } from '../info-drawer/comments-tab/comments-tab.component'; -import { NodeActionsService } from '../../services/node-actions.service'; -import { NodeEntry } from '@alfresco/js-api'; import { takeUntil } from 'rxjs/operators'; +import { ContentActionRef } from '@alfresco/adf-extensions'; @Component({ standalone: true, @@ -66,16 +65,9 @@ export class DetailsComponent extends PageComponent implements OnInit, OnDestroy isLoading: boolean; onDestroy$ = new Subject(); activeTab = 1; - selectionState: NodeEntry; - isNodeLocked = false; + actionsAspect: Array = []; - constructor( - private route: ActivatedRoute, - private contentApi: ContentApiService, - private nodeAspectService: NodeAspectService, - private nodeActionsService: NodeActionsService, - private cdr: ChangeDetectorRef - ) { + constructor(private route: ActivatedRoute, private contentApi: ContentApiService) { super(); } @@ -96,14 +88,12 @@ export class DetailsComponent extends PageComponent implements OnInit, OnDestroy this.store.dispatch(new SetSelectedNodesAction([{ entry: this.node }])); }); }); - this.store.select(getAppSelection).subscribe(({ file }) => { - this.selectionState = file; - const isNodeLockedFromStore = this.selection && isLocked(this.selectionState); - this.nodeActionsService.isNodeLocked$.pipe(takeUntil(this.onDestroy$)).subscribe((isNodeLockedFromService) => { - this.isNodeLocked = isNodeLockedFromStore || isNodeLockedFromService; - this.cdr.detectChanges(); + this.extensions + .getAllowedSidebarActions() + .pipe(takeUntil(this.onDestroy$)) + .subscribe((actionsAspect) => { + this.actionsAspect = actionsAspect; }); - }); } setActiveTab(tabName: string) { @@ -124,10 +114,6 @@ export class DetailsComponent extends PageComponent implements OnInit, OnDestroy this.store.dispatch(new NavigateToPreviousPage()); } - openAspectDialog() { - this.nodeAspectService.updateNodeAspects(this.node.id); - } - ngOnDestroy(): void { this.store.dispatch(new SetSelectedNodesAction([])); this.onDestroy$.next(); diff --git a/projects/aca-content/src/lib/components/toolbar/toggle-edit-offline/toggle-edit-offline.component.spec.ts b/projects/aca-content/src/lib/components/toolbar/toggle-edit-offline/toggle-edit-offline.component.spec.ts index 11a3c3822..e061c2a59 100644 --- a/projects/aca-content/src/lib/components/toolbar/toggle-edit-offline/toggle-edit-offline.component.spec.ts +++ b/projects/aca-content/src/lib/components/toolbar/toggle-edit-offline/toggle-edit-offline.component.spec.ts @@ -26,10 +26,9 @@ import { ToggleEditOfflineComponent } from './toggle-edit-offline.component'; import { ComponentFixture, TestBed } from '@angular/core/testing'; import { of } from 'rxjs'; import { Store } from '@ngrx/store'; -import { NodeEntry, NodesApi } from '@alfresco/js-api'; +import { NodeEntry } from '@alfresco/js-api'; import { DownloadNodesAction, EditOfflineAction, SnackbarErrorAction } from '@alfresco/aca-shared/store'; import { AppTestingModule } from '../../../testing/app-testing.module'; -import { NodeActionsService } from '../../../services/node-actions.service'; describe('ToggleEditOfflineComponent', () => { let fixture: ComponentFixture; @@ -39,49 +38,6 @@ describe('ToggleEditOfflineComponent', () => { let selectSpy: jasmine.Spy; let selection: any; - const lockedNodeEntry: NodeEntry = { - entry: { - isFile: true, - createdByUser: { - id: 'hruser', - displayName: 'hruser' - }, - modifiedAt: new Date('2023-09-08T11:54:48.325+0000'), - nodeType: 'cm:content', - content: { - mimeType: 'image/jpeg', - mimeTypeName: 'JPEG Image', - sizeInBytes: 128473, - encoding: 'UTF-8' - }, - parentId: '5a2d88ec-a29c-408a-874d-6394940c51d7', - aspectNames: ['cm:versionable', 'cm:lockable', 'cm:auditable', 'cm:taggable', 'exif:exif'], - createdAt: new Date('2023-09-07T11:10:48.788+0000'), - isFolder: false, - modifiedByUser: { - id: 'hruser', - displayName: 'hruser' - }, - name: 'e2e_favorite_file.jpg', - id: '36e5b5ad-3fa0-47e2-b256-016b868ac772', - properties: { - 'cm:lockType': 'WRITE_LOCK', - 'cm:lockOwner': { - id: 'hruser', - displayName: 'hruser' - }, - 'cm:versionType': 'MAJOR', - 'cm:versionLabel': '1.0', - 'cm:lockLifetime': 'PERSISTENT', - 'exif:pixelYDimension': 1253, - 'exif:pixelXDimension': 1024 - } - } - }; - - const nodesApiMock = jasmine.createSpyObj('nodesApi', ['lockNode', 'unlockNode']); - const nodeActionsServiceMock = jasmine.createSpyObj('nodeActionsService', ['setNodeLocked']); - beforeEach(() => { TestBed.configureTestingModule({ imports: [AppTestingModule, ToggleEditOfflineComponent], @@ -92,9 +48,7 @@ describe('ToggleEditOfflineComponent', () => { select: () => {}, dispatch: () => {} } - }, - { provide: NodesApi, useValue: nodesApiMock }, - { provide: NodeActionsService, useValue: nodeActionsServiceMock } + } ] }); @@ -179,51 +133,4 @@ describe('ToggleEditOfflineComponent', () => { }) ]); }); - - it('should call setNodeLocked with true when a node is locked', () => { - const nodeId = 'testNode1'; - nodesApiMock.lockNode.and.returnValue(Promise.resolve(lockedNodeEntry)); - - component.lockNode(nodeId).then((result) => { - expect(nodesApiMock.lockNode).toHaveBeenCalledWith(nodeId, { - type: 'ALLOW_OWNER_CHANGES', - lifetime: 'PERSISTENT' - }); - expect(nodeActionsServiceMock.setNodeLocked).toHaveBeenCalledWith(true); - expect(result).toEqual(lockedNodeEntry); - }); - }); - - it('should call setNodeLocked with false when a node is unlocked', () => { - const nodeId = 'testNode2'; - nodesApiMock.unlockNode.and.returnValue(Promise.resolve(lockedNodeEntry)); - - component.unlockNode(nodeId).then((result) => { - expect(nodesApiMock.unlockNode).toHaveBeenCalledWith(nodeId); - expect(nodeActionsServiceMock.setNodeLocked).toHaveBeenCalledWith(false); - expect(result).toEqual(lockedNodeEntry); - }); - }); - - it('should handle errors when locking a node encounters an error', () => { - const nodeId = 'testNode1'; - const error = new Error('Locking failed'); - nodesApiMock.lockNode.and.returnValue(Promise.reject(error)); - component.lockNode(nodeId).catch((err) => { - expect(nodesApiMock.lockNode).toHaveBeenCalledWith(nodeId, { type: 'ALLOW_OWNER_CHANGES', lifetime: 'PERSISTENT' }); - expect(nodeActionsServiceMock.setNodeLocked).not.toHaveBeenCalled(); - expect(err).toEqual(error); - }); - }); - - it('should handle errors when unlocking a node encounters an error', () => { - const nodeId = 'testNode1'; - const error = new Error('Unlocking failed'); - nodesApiMock.unlockNode.and.returnValue(Promise.reject(error)); - component.unlockNode(nodeId).catch((err) => { - expect(nodesApiMock.lockNode).toHaveBeenCalledWith(nodeId); - expect(nodeActionsServiceMock.setNodeLocked).not.toHaveBeenCalled(); - expect(err).toEqual(error); - }); - }); }); diff --git a/projects/aca-content/src/lib/components/toolbar/toggle-edit-offline/toggle-edit-offline.component.ts b/projects/aca-content/src/lib/components/toolbar/toggle-edit-offline/toggle-edit-offline.component.ts index a01f6e5f5..40410b94b 100644 --- a/projects/aca-content/src/lib/components/toolbar/toggle-edit-offline/toggle-edit-offline.component.ts +++ b/projects/aca-content/src/lib/components/toolbar/toggle-edit-offline/toggle-edit-offline.component.ts @@ -31,7 +31,7 @@ import { getAppSelection } from '@alfresco/aca-shared/store'; import { NodeEntry, SharedLinkEntry, Node, NodesApi } from '@alfresco/js-api'; -import { ChangeDetectorRef, Component, OnInit, ViewEncapsulation } from '@angular/core'; +import { Component, OnInit, ViewEncapsulation } from '@angular/core'; import { Store } from '@ngrx/store'; import { AppExtensionService, isLocked } from '@alfresco/aca-shared'; import { AlfrescoApiService } from '@alfresco/adf-core'; @@ -39,7 +39,6 @@ import { CommonModule } from '@angular/common'; import { TranslateModule } from '@ngx-translate/core'; import { MatMenuModule } from '@angular/material/menu'; import { MatIconModule } from '@angular/material/icon'; -import { NodeActionsService } from '../../../services/node-actions.service'; @Component({ standalone: true, @@ -60,13 +59,7 @@ export class ToggleEditOfflineComponent implements OnInit { nodeTitle = ''; isNodeLocked = false; - constructor( - private store: Store, - private alfrescoApiService: AlfrescoApiService, - private nodeActionsService: NodeActionsService, - private cdr: ChangeDetectorRef, - private extensions: AppExtensionService - ) { + constructor(private store: Store, private alfrescoApiService: AlfrescoApiService, private extensions: AppExtensionService) { this.nodesApi = new NodesApi(this.alfrescoApiService.getInstance()); } @@ -75,7 +68,6 @@ export class ToggleEditOfflineComponent implements OnInit { this.selection = file; this.isNodeLocked = this.selection && isLocked(this.selection); this.nodeTitle = this.isNodeLocked ? 'APP.ACTIONS.EDIT_OFFLINE_CANCEL' : 'APP.ACTIONS.EDIT_OFFLINE'; - this.cdr.detectChanges(); }); } @@ -127,27 +119,15 @@ export class ToggleEditOfflineComponent implements OnInit { ); } - lockNode(nodeId: string): Promise { - return this.nodesApi.lockNode(nodeId, { type: 'ALLOW_OWNER_CHANGES', lifetime: 'PERSISTENT' }).then( - (res: NodeEntry) => { - this.nodeActionsService.setNodeLocked(true); - return res; - }, - (error) => { - return error; - } - ); + lockNode(nodeId: string) { + return this.nodesApi.lockNode(nodeId, { + type: 'ALLOW_OWNER_CHANGES', + lifetime: 'PERSISTENT' + }); } - unlockNode(nodeId: string): Promise { - return this.nodesApi.unlockNode(nodeId).then( - (res: NodeEntry) => { - this.nodeActionsService.setNodeLocked(false); - return res; - }, - (error) => { - return error; - } - ); + + unlockNode(nodeId: string) { + return this.nodesApi.unlockNode(nodeId); } private update(data: Node) { diff --git a/projects/aca-content/src/lib/services/node-actions.service.ts b/projects/aca-content/src/lib/services/node-actions.service.ts index 382f5c974..248586778 100644 --- a/projects/aca-content/src/lib/services/node-actions.service.ts +++ b/projects/aca-content/src/lib/services/node-actions.service.ts @@ -24,7 +24,7 @@ import { Injectable } from '@angular/core'; import { MatDialog } from '@angular/material/dialog'; -import { Observable, Subject, of, zip, from, BehaviorSubject } from 'rxjs'; +import { Observable, Subject, of, zip, from } from 'rxjs'; import { AlfrescoApiService, TranslationService, ThumbnailService } from '@alfresco/adf-core'; import { DocumentListService, @@ -49,7 +49,6 @@ export class NodeActionsService { contentMoved: Subject = new Subject(); moveDeletedEntries: any[] = []; isSitesDestinationAvailable = false; - private isNodeLockedSubject = new BehaviorSubject(false); isNodeLocked$: Observable; _nodesApi: NodesApi; @@ -66,9 +65,7 @@ export class NodeActionsService { private apiService: AlfrescoApiService, private translation: TranslationService, private thumbnailService: ThumbnailService - ) { - this.isNodeLocked$ = this.isNodeLockedSubject.asObservable(); - } + ) {} /** * Copy node list @@ -92,10 +89,6 @@ export class NodeActionsService { return this.doBatchOperation(NodeAction.MOVE, contentEntities, permission, focusedElementOnCloseSelector); } - setNodeLocked(isLocked: boolean) { - this.isNodeLockedSubject.next(isLocked); - } - /** * General method for performing the given operation (copy|move) to multiple nodes * diff --git a/projects/aca-shared/rules/src/app.rules.ts b/projects/aca-shared/rules/src/app.rules.ts index 71333f4bc..ef23e5355 100644 --- a/projects/aca-shared/rules/src/app.rules.ts +++ b/projects/aca-shared/rules/src/app.rules.ts @@ -511,6 +511,8 @@ export const canEditAspects = (context: RuleContext): boolean => repository.isMajorVersionAvailable(context, '7') ].every(Boolean); +export const canNotShowExpand = (context: RuleContext): boolean => [!navigation.isLibraries(context), !navigation.isDetails(context)].every(Boolean); + /** * Checks if user can manage permissions for the selected node. * JSON ref: `canManagePermissions` diff --git a/projects/aca-shared/rules/src/navigation.rules.ts b/projects/aca-shared/rules/src/navigation.rules.ts index 02d6a6df9..5a3c5569c 100644 --- a/projects/aca-shared/rules/src/navigation.rules.ts +++ b/projects/aca-shared/rules/src/navigation.rules.ts @@ -110,6 +110,11 @@ export function isLibraryContent(context: RuleContext): boolean { return url && (url.endsWith('/libraries') || url.includes('/libraries/') || url.startsWith('/search-libraries')); } +export function isDetails(context: RuleContext): boolean { + const { url } = context.navigation; + return url && (url.endsWith('/details') || url.includes('/details/') || url.startsWith('/details')); +} + /** * Checks if the activated route is neither **Libraries** nor **Library Search Results**. * JSON ref: `app.navigation.isNotLibraries`