From c689379f92b7353bc0b7a4bb04702123e0fc3169 Mon Sep 17 00:00:00 2001
From: sauls8t <saul@documize.com>
Date: Tue, 4 Feb 2020 12:37:59 +0000
Subject: [PATCH] Restrict pin sequence changing

Only all documents filter enables the up/down pin sequence change clickers.
---
 gui/app/components/folder/documents-list.js            | 4 ++--
 gui/app/pods/folder/index/template.hbs                 | 1 +
 gui/app/templates/components/folder/documents-list.hbs | 4 ++--
 3 files changed, 5 insertions(+), 4 deletions(-)

diff --git a/gui/app/components/folder/documents-list.js b/gui/app/components/folder/documents-list.js
index 2a30038d..67f29594 100644
--- a/gui/app/components/folder/documents-list.js
+++ b/gui/app/components/folder/documents-list.js
@@ -31,8 +31,8 @@ export default Component.extend({
 	hasDocumentActions: computed('permissions.{documentDelete,documentMove}', function() {
 		return this.get('permissions.documentDelete') || this.get('permissions.documentMove');
 	}),
-    hasCategoryFilter: computed('categoryFilter', function() {
-        return !_.isEmpty(this.get('categoryFilter'));
+    showingAllDocs: computed('categoryFilter', 'numDocuments', 'documents', function() {
+        return _.isEmpty(this.get('categoryFilter')) && this.get('documents').length == this.get("numDocuments");
     }),
 
 	didReceiveAttrs() {
diff --git a/gui/app/pods/folder/index/template.hbs b/gui/app/pods/folder/index/template.hbs
index e9b317a5..41d16591 100644
--- a/gui/app/pods/folder/index/template.hbs
+++ b/gui/app/pods/folder/index/template.hbs
@@ -66,6 +66,7 @@
 			meta=model.folder.icon}}
 
 		{{folder/documents-list
+                numDocuments=model.documents.length
 			documents=filteredDocs
 			spaces=model.folders
 			space=model.folder
diff --git a/gui/app/templates/components/folder/documents-list.hbs b/gui/app/templates/components/folder/documents-list.hbs
index 4f62778f..24c4ff06 100644
--- a/gui/app/templates/components/folder/documents-list.hbs
+++ b/gui/app/templates/components/folder/documents-list.hbs
@@ -94,14 +94,14 @@
 				{{#if hasDocumentActions}}
 					<div class="sequence">
 						{{#if document.isSequenced}}
-                            {{#unless hasCategoryFilter}}
+                            {{#if showingAllDocs}}
                                 <i class="dicon {{constants.Icon.ArrowSmallUp}}" {{action "onPinSequence" document.id "u"}}>
                                     {{#attach-tooltip showDelay=250}}Move up{{/attach-tooltip}}
                                 </i>
                                 <i class="dicon {{constants.Icon.ArrowSmallDown}}" {{action "onPinSequence" document.id "d"}}>
                                     {{#attach-tooltip showDelay=250}}Move down{{/attach-tooltip}}
                                 </i>
-                            {{/unless}}
+                            {{/if}}
 							<i class="dicon {{constants.Icon.Cross}}" {{action "onUnpin" document.id}}>
 								{{#attach-tooltip showDelay=250}}Unpin{{/attach-tooltip}}
 							</i>