From 88ffbaaa22e9cb150dd39b18866b41a35981b533 Mon Sep 17 00:00:00 2001
From: Coleman Watts <coleman@civicrm.org>
Date: Wed, 14 Oct 2015 21:13:29 -0400
Subject: [PATCH] CRM-17176 - Improve report-menu-cleanup button

---
 templates/CRM/Admin/Page/Navigation.tpl | 19 ++++++++++++++-----
 1 file changed, 14 insertions(+), 5 deletions(-)

diff --git a/templates/CRM/Admin/Page/Navigation.tpl b/templates/CRM/Admin/Page/Navigation.tpl
index 0ba4b3ba9c..ade8db52ee 100644
--- a/templates/CRM/Admin/Page/Navigation.tpl
+++ b/templates/CRM/Admin/Page/Navigation.tpl
@@ -42,7 +42,8 @@
     <div id="navigation-tree" class="navigation-tree" style="height:auto; border-collapse:separate; background-color:#FFFFFF;"></div>
     <div class="spacer"></div>
     <div>
-      <a href="{crmURL p="#"}" id='nav-reset'><span>{ts}Click here{/ts}</span></a> {ts}to reset the reports menu structure to the default.{/ts}
+      {* TODO: fa-broom would be better, but not implemented yet. https://github.com/FortAwesome/Font-Awesome/issues/239 *}
+      <a href="#" class="nav-reset crm-hover-button"><i class="crm-i fa-undo"></i> {ts}Cleanup reports menu{/ts}</a>
     </div>
     <div class="spacer"></div>
   </div>
@@ -57,10 +58,6 @@
   </style>
   <script type="text/javascript">
     CRM.$(function($) {
-      $('#nav-reset').on('click', function(e) {
-        e.preventDefault();
-        CRM.api3('Navigation', 'reset', {'for' : 'report'}, ts('Report menu reset. Changes will be visible when you refresh'))
-      });
       $("#navigation-tree").jstree({
         plugins: [ "themes", "json_data", "dnd","ui", "crrm","contextmenu" ],
         json_data: {
@@ -170,6 +167,18 @@
           $("#navigation-tree").jstree('refresh');
           $("#reset-menu").show();
         });
+
+      $('a.nav-reset').on('click', function(e) {
+        e.preventDefault();
+        CRM.confirm({message: '{/literal}{ts escape='js'}This will rebuild the "Reports" menu to include all currently active reports.{/ts}{literal}'})
+          .on('crmConfirm:yes', function () {
+            CRM.api3('Navigation', 'reset', {'for': 'report'}, true)
+              .done(function() {
+                $("#navigation-tree").jstree('refresh');
+                $("#reset-menu").show();
+              })
+          });
+      });
     });
 </script>
 {/literal}
-- 
2.25.1