From ac6c9e961d09525fdc36fc2a9de57bf619d7447b Mon Sep 17 00:00:00 2001
From: Cilibiu Bogdan <pionnegru@users.noreply.github.com>
Date: Fri, 24 Nov 2017 11:49:58 +0200
Subject: [PATCH] render condition (#80)

---
 .../favorites/favorites.component.html        |  1 +
 .../favorites/favorites.component.spec.ts     | 50 +++++++++++++++++++
 .../favorites/favorites.component.ts          |  6 ++-
 3 files changed, 56 insertions(+), 1 deletion(-)

diff --git a/src/app/components/favorites/favorites.component.html b/src/app/components/favorites/favorites.component.html
index 0a1fe1a3f..d1d736a99 100644
--- a/src/app/components/favorites/favorites.component.html
+++ b/src/app/components/favorites/favorites.component.html
@@ -23,6 +23,7 @@
 
             <button
                 mat-icon-button
+                *ngIf="showEditOption(documentList.selection)"
                 title="{{ 'APP.ACTIONS.EDIT' | translate }}"
                 [adf-edit-folder]="documentList.selection[0]?.entry">
                 <mat-icon>create</mat-icon>
diff --git a/src/app/components/favorites/favorites.component.spec.ts b/src/app/components/favorites/favorites.component.spec.ts
index 91cba5c34..5c57cfaa9 100644
--- a/src/app/components/favorites/favorites.component.spec.ts
+++ b/src/app/components/favorites/favorites.component.spec.ts
@@ -179,6 +179,56 @@ describe('Favorites Routed Component', () => {
         });
     });
 
+    describe('edit option', () => {
+        it('should return false if a file node is selected', () => {
+            const selection = [
+                {
+                    entry: {
+                        isFolder: false,
+                        isFile: true
+                    }
+                }
+            ];
+
+            const result = component.showEditOption(selection);
+            expect(result).toBe(false);
+        });
+
+        it('should return false if multiple nodes are selected', () => {
+            const selection = [
+                {
+                    entry: {
+                        isFolder: true,
+                        isFile: false
+                    }
+                },
+                {
+                    entry: {
+                        isFolder: true,
+                        isFile: false
+                    }
+                }
+            ];
+
+            const result = component.showEditOption(selection);
+            expect(result).toBe(false);
+        });
+
+        it('should return true if selected node is a folder', () => {
+            const selection = [
+                {
+                    entry: {
+                        isFolder: true,
+                        isFile: false
+                    }
+                }
+            ];
+
+            const result = component.showEditOption(selection);
+            expect(result).toBe(true);
+        });
+    });
+
     describe('refresh', () => {
         it('should call document list reload', () => {
             spyOn(component.documentList, 'reload');
diff --git a/src/app/components/favorites/favorites.component.ts b/src/app/components/favorites/favorites.component.ts
index c1bb59778..90f35f0a4 100644
--- a/src/app/components/favorites/favorites.component.ts
+++ b/src/app/components/favorites/favorites.component.ts
@@ -19,7 +19,7 @@ import { Component, ViewChild, OnInit, OnDestroy } from '@angular/core';
 import { Router } from '@angular/router';
 import { Subscription } from 'rxjs/Rx';
 
-import { MinimalNodeEntryEntity, PathElementEntity, PathInfo } from 'alfresco-js-api';
+import { MinimalNodeEntryEntity, MinimalNodeEntity, PathElementEntity, PathInfo } from 'alfresco-js-api';
 import { ContentService, NodesApiService } from '@alfresco/adf-core';
 import { DocumentListComponent } from '@alfresco/adf-content-services';
 
@@ -92,6 +92,10 @@ export class FavoritesComponent extends PageComponent implements OnInit, OnDestr
         }
     }
 
+    showEditOption(selection: MinimalNodeEntity[]) {
+        return selection && selection.length === 1 && selection[0].entry.isFolder;
+    }
+
     refresh(): void {
         if (this.documentList) {
             this.documentList.reload();