From d1ff001040faedc0ae270749be22b7fa14d5a9c9 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Grzegorz=20Ja=C5=9Bkowski?=
 <138671284+g-jaskowski@users.noreply.github.com>
Date: Tue, 19 Sep 2023 11:14:58 +0200
Subject: [PATCH] [ACS-5686] add reselect to edit offline, shorten template
 (#3433)

* ACS-5686 add reselect to edit offline, shorten template

* ACS-5686 merge icon condition, use optional chain expression
---
 .../toggle-edit-offline.component.ts          | 24 ++++++++++---------
 1 file changed, 13 insertions(+), 11 deletions(-)

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 a3a59af237..2862777322 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
@@ -22,7 +22,14 @@
  * from Hyland Software. If not, see <http://www.gnu.org/licenses/>.
  */
 
-import { AppStore, DownloadNodesAction, EditOfflineAction, SnackbarErrorAction, getAppSelection } from '@alfresco/aca-shared/store';
+import {
+  AppStore,
+  DownloadNodesAction,
+  EditOfflineAction,
+  SetSelectedNodesAction,
+  SnackbarErrorAction,
+  getAppSelection
+} from '@alfresco/aca-shared/store';
 import { NodeEntry, SharedLinkEntry, Node, NodesApi } from '@alfresco/js-api';
 import { Component, OnInit, ViewEncapsulation } from '@angular/core';
 import { Store } from '@ngrx/store';
@@ -39,15 +46,8 @@ import { MatIconModule } from '@angular/material/icon';
   selector: 'app-toggle-edit-offline',
   template: `
     <button mat-menu-item [attr.title]="nodeTitle | translate" (click)="onClick()">
-      <ng-container *ngIf="isNodeLocked">
-        <mat-icon>cancel</mat-icon>
-        <span>{{ 'APP.ACTIONS.EDIT_OFFLINE_CANCEL' | translate }}</span>
-      </ng-container>
-
-      <ng-container *ngIf="!isNodeLocked">
-        <mat-icon>edit</mat-icon>
-        <span>{{ 'APP.ACTIONS.EDIT_OFFLINE' | translate }}</span>
-      </ng-container>
+      <mat-icon>{{ isNodeLocked ? 'cancel' : 'edit' }}</mat-icon>
+      <span>{{ (isNodeLocked ? 'APP.ACTIONS.EDIT_OFFLINE_CANCEL' : 'APP.ACTIONS.EDIT_OFFLINE') | translate }}</span>
     </button>
   `,
   encapsulation: ViewEncapsulation.None,
@@ -84,6 +84,7 @@ export class ToggleEditOfflineComponent implements OnInit {
 
         this.update(response?.entry);
         this.store.dispatch(new EditOfflineAction(this.selection));
+        this.store.dispatch(new SetSelectedNodesAction([this.selection]));
       } catch {
         this.onUnlockError();
       }
@@ -94,6 +95,7 @@ export class ToggleEditOfflineComponent implements OnInit {
         this.update(response?.entry);
         this.store.dispatch(new DownloadNodesAction([this.selection]));
         this.store.dispatch(new EditOfflineAction(this.selection));
+        this.store.dispatch(new SetSelectedNodesAction([this.selection]));
       } catch {
         this.onLockError();
       }
@@ -128,7 +130,7 @@ export class ToggleEditOfflineComponent implements OnInit {
   }
 
   private update(data: Node) {
-    if (data && data.properties) {
+    if (data?.properties) {
       const properties = this.selection.entry.properties || {};
 
       properties['cm:lockLifetime'] = data.properties['cm:lockLifetime'];