SearchKit - Sync menu & navigation permissions to use 'administer Civ…

…iCRM data' Fixes dev/core#2619 https://lab.civicrm.org/dev/core/-/issues/2619
civicrm · May 20, 2021 · 919be70 · 919be70
1 parent fd8f3fc
commit 919be70
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 1 deletion.
diff --git a/ext/search_kit/CRM/Search/Upgrader.php b/ext/search_kit/CRM/Search/Upgrader.php
@@ -18,6 +18,7 @@ public function enable() {
       ->addValue('icon', 'crm-i fa-search-plus')
       ->addValue('has_separator', 2)
       ->addValue('weight', 99)
+      ->addValue('permission', 'administer CiviCRM data')
       ->execute();
   }
 
@@ -133,4 +134,14 @@ public function upgrade_1003() {
     return TRUE;
   }
 
+  /**
+   * Upgrade 1004 - fix menu permission.
+   * @return bool
+   */
+  public function upgrade_1004() {
+    $this->ctx->log->info('Applying update 1000 - fix menu permission.');
+    CRM_Core_DAO::executeQuery("UPDATE civicrm_navigation SET permission = 'administer CiviCRM data' WHERE url = 'civicrm/admin/search'");
+    return TRUE;
+  }
+
 }
diff --git a/ext/search_kit/xml/Menu/search_kit.xml b/ext/search_kit/xml/Menu/search_kit.xml
@@ -8,6 +8,6 @@
   <item>
     <path>civicrm/admin/search</path>
     <page_callback>CRM_Search_Page_Admin</page_callback>
-    <access_arguments>administer CiviCRM</access_arguments>
+    <access_arguments>administer CiviCRM data</access_arguments>
   </item>
 </menu>