From 45a02efafca973a9dec4964b55f30cba32e8edf9 Mon Sep 17 00:00:00 2001
From: Stephen Kraffmiller <skraffmiller@hmdc.harvard.edu>
Date: Fri, 8 Jan 2016 14:29:35 -0500
Subject: [PATCH] #2797 add message when no files selected for editing tags

---
 src/main/webapp/dataset.xhtml       | 20 ++++++++++++++++++++
 src/main/webapp/filesFragment.xhtml | 10 ++++------
 2 files changed, 24 insertions(+), 6 deletions(-)

diff --git a/src/main/webapp/dataset.xhtml b/src/main/webapp/dataset.xhtml
index 48a7ad5b752..040907767f4 100755
--- a/src/main/webapp/dataset.xhtml
+++ b/src/main/webapp/dataset.xhtml
@@ -806,6 +806,17 @@
                             <p:commandButton value="#{bundle.close}" onclick="PF('selectFilesForEditMetadata').hide();PF('blockDatasetForm').hide();"/>
                         </div>
                     </p:dialog>
+                    
+                    <p:dialog id="selectFilesForEditTags" styleClass="smallPopUp"
+                              header="#{bundle['dataset.noSelectedFiles.header']}" widgetVar="selectFilesForEditTags" modal ="true">
+                        <p class="text-danger"><span class="glyphicon glyphicon-warning-sign"/> #{bundle['dataset.noSelectedFilesForMetadataEdit']}</p>
+                        <div class="button-block">
+                            <p:commandButton value="#{bundle.close}" onclick="PF('selectFilesForEditTags').hide();PF('blockDatasetForm').hide();"/>
+                        </div>
+                    </p:dialog>
+
+                    <p:remoteCommand  name="refreshTagsCommand" action="#{DatasetPage.refreshTagsPopUp()}" update=":datasetForm:fileTagsPopup"
+                                      oncomplete="PF('fileTagsPopup').show();"/>
 
                     <!-- We no longer need this remoteCommand; see my comments in the javascript method testFilesSelectedForEditMetadata() -->
                     <!-- for more info. - L.A. 4.2.1 -->
@@ -1277,6 +1288,15 @@
                         
                     }
                     
+                    function testFilesSelectedForTags(){
+                        var count = PF('filesTable').getSelectedRowsCount();
+                        if (count === 0) {
+                            PF('selectFilesForEditTags').show();
+                        } else {
+                           refreshTagsCommand();
+                        }                       
+                    }
+                    
                     function testFilesSelectedForDelete(){
                         var count = PF('filesTable').getSelectedRowsCount();
                         if (count === 0) {
diff --git a/src/main/webapp/filesFragment.xhtml b/src/main/webapp/filesFragment.xhtml
index 95a8fb9c8b4..51d4e40ea11 100644
--- a/src/main/webapp/filesFragment.xhtml
+++ b/src/main/webapp/filesFragment.xhtml
@@ -81,13 +81,11 @@
                                     </p:commandLink>
                                 </li>
                                 <li class="#{DatasetPage.locked  ? 'disabled' : ''}">
-                                    <p:commandLink id="fileCategoriesLink"                                               
-                                                   actionListener="#{DatasetPage.refreshTagsPopUp()}"
-                                                   update=":datasetForm:fileTagsPopup"
-                                                   oncomplete="PF('fileTagsPopup').show();bind_bsui_components();">
-                                        #{bundle['file.tags']}
+                                    <p:commandLink id="fileCategoriesLinkTwo"    
+                                                   onclick="testFilesSelectedForTags()">
+                                        #{bundle['file.tags']} 
                                     </p:commandLink>    
-                                </li>
+                                </li>                                                               
                             </ul>
                             <p:remoteCommand  name="openEditFilesPageCommand2" partialSubmit="true" async="true" update="" action="#{DatasetPage.editFileMetadata()}"  />
                         </div>