[ACS-5540] modified the aspect button changes

projects/aca-content/assets/app.extensions.json

@@ -1261,7 +1261,7 @@
             "click": "EXPAND_INFO_DRAWER"
           },
           "rules": {
-            "visible": "app.navigation.isNotLibraries"
+            "visible": "canNotShowExpand"
           }
         }
       ],

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,
 

projects/aca-content/src/lib/components/details/details.component.html

@@ -12,21 +12,11 @@
           -
           <span class="acs-details-breadcrumb-item">{{ 'APP.INFO_DRAWER.TITLE' | translate }}</span>
         </div>
-        <div>
-          <button *ngIf="!isNodeLocked"
-                  mat-icon-button
-                  (click)="openAspectDialog()"
-                  [attr.title]="'CORE.METADATA.ACTIONS.EDIT_ASPECTS' | translate"
-                  [attr.aria-label]="'CORE.METADATA.ACCESSIBILITY.EDIT_ASPECTS' | translate"
-                  data-automation-id="meta-data-edit-aspect">
-                  <mat-icon>playlist_add</mat-icon>
-          </button>
-          <button class="aca-close-details-button"
-                  mat-icon-button
-                  data-automation-id="close-library"
-                  title="{{ 'APP.INFO_DRAWER.CLOSE' | translate }}"
-                  (click)="goBack()">
-                  <mat-icon>close</mat-icon>
+        <div class="acs-details-buttons">
+          <aca-toolbar [items]="actionsAspect" info-drawer-buttons></aca-toolbar>
+          <button class="aca-close-details-button" mat-icon-button data-automation-id="close-library"
+            title="{{ 'APP.INFO_DRAWER.CLOSE' | translate }}" (click)="goBack()">
+            <mat-icon>close</mat-icon>
           </button>
         </div>
       </div>

projects/aca-content/src/lib/components/details/details.component.scss

@@ -1,13 +1,17 @@
 app-details-manager {
-  .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);
+  .acs-details-buttons {
+    display: flex;
+
+    .aca-close-details-button {
+      margin-right: 15px;
+      outline: none;
       border-radius: 4px;
+      margin-top: 12px;
+
+      &:focus-visible {
+        outline: 2px solid var(--theme-blue-button-color);
+        border-radius: 4px;
+      }
     }
   }
 }

projects/aca-content/src/lib/components/details/details.component.spec.ts

@@ -31,18 +31,17 @@ 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;
   let fixture: ComponentFixture<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();
-  });
 });

projects/aca-content/src/lib/components/details/details.component.ts

@@ -22,12 +22,12 @@
  * from Hyland Software. If not, see <http://www.gnu.org/licenses/>.
  */
 
-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<boolean>();
   activeTab = 1;
-  selectionState: NodeEntry;
-  isNodeLocked = false;
+  actionsAspect: Array<ContentActionRef> = [];
 
-  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();

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<ToggleEditOfflineComponent>;
@@ -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);
-    });
-  });
 });