From c0fdbd77cfc15f1460b7d7acfd121ac587f3d507 Mon Sep 17 00:00:00 2001
From: eileen <emcnaughton@wikimedia.org>
Date: Thu, 10 Sep 2020 13:18:00 +1200
Subject: [PATCH] Style fixes

I removed the commented out functions rather than figuring out how to style them right
---
 ext/afform/mock/afform_mock.php               | 28 -------------
 ext/afform/mock/ang/mockBespoke.ang.php       | 40 +++++++++----------
 .../phpunit/api/v4/AfformPaletteTest.php      |  3 --
 3 files changed, 19 insertions(+), 52 deletions(-)

diff --git a/ext/afform/mock/afform_mock.php b/ext/afform/mock/afform_mock.php
index b101e0d2d1..578aed00f9 100644
--- a/ext/afform/mock/afform_mock.php
+++ b/ext/afform/mock/afform_mock.php
@@ -140,31 +140,3 @@ function afform_mock_civicrm_entityTypes(&$entityTypes) {
 function afform_mock_civicrm_themes(&$themes) {
   _afform_mock_civix_civicrm_themes($themes);
 }
-
-// --- Functions below this ship commented out. Uncomment as required. ---
-
-/**
- * Implements hook_civicrm_preProcess().
- *
- * @link http://wiki.civicrm.org/confluence/display/CRMDOC/hook_civicrm_preProcess
- *
-function afform_mock_civicrm_preProcess($formName, &$form) {
-
-} // */
-
-/**
- * Implements hook_civicrm_navigationMenu().
- *
- * @link http://wiki.civicrm.org/confluence/display/CRMDOC/hook_civicrm_navigationMenu
- *
-function afform_mock_civicrm_navigationMenu(&$menu) {
-  _afform_mock_civix_insert_navigation_menu($menu, 'Mailings', array(
-    'label' => E::ts('New subliminal message'),
-    'name' => 'mailing_subliminal_message',
-    'url' => 'civicrm/mailing/subliminal',
-    'permission' => 'access CiviMail',
-    'operator' => 'OR',
-    'separator' => 0,
-  ));
-  _afform_mock_civix_navigationMenu($menu);
-} // */
diff --git a/ext/afform/mock/ang/mockBespoke.ang.php b/ext/afform/mock/ang/mockBespoke.ang.php
index ecbfb40aa9..467b5894df 100644
--- a/ext/afform/mock/ang/mockBespoke.ang.php
+++ b/ext/afform/mock/ang/mockBespoke.ang.php
@@ -2,28 +2,26 @@
 // This file declares an Angular module which can be autoloaded
 // in CiviCRM. See also:
 // \https://docs.civicrm.org/dev/en/latest/hooks/hook_civicrm_angularModules/n
-return array (
+return [
   'js' =>
-  array (
-    0 => 'ang/mockBespoke.js',
-    1 => 'ang/mockBespoke/*.js',
-    2 => 'ang/mockBespoke/*/*.js',
-  ),
+    [
+      0 => 'ang/mockBespoke.js',
+      1 => 'ang/mockBespoke/*.js',
+      2 => 'ang/mockBespoke/*/*.js',
+    ],
   'css' =>
-  array (
-    0 => 'ang/mockBespoke.css',
-  ),
+    [
+      0 => 'ang/mockBespoke.css',
+    ],
   'partials' =>
-  array (
-    0 => 'ang/mockBespoke',
-  ),
+    [
+      0 => 'ang/mockBespoke',
+    ],
   'requires' =>
-  array (
-    0 => 'crmUi',
-    1 => 'crmUtil',
-    2 => 'ngRoute',
-  ),
-  'settings' =>
-  array (
-  ),
-);
+    [
+      0 => 'crmUi',
+      1 => 'crmUtil',
+      2 => 'ngRoute',
+    ],
+  'settings' => [],
+];
diff --git a/ext/afform/mock/tests/phpunit/api/v4/AfformPaletteTest.php b/ext/afform/mock/tests/phpunit/api/v4/AfformPaletteTest.php
index c181150bbd..7f2ec85877 100644
--- a/ext/afform/mock/tests/phpunit/api/v4/AfformPaletteTest.php
+++ b/ext/afform/mock/tests/phpunit/api/v4/AfformPaletteTest.php
@@ -1,8 +1,5 @@
 <?php
 
-use Civi\Test\HeadlessInterface;
-use Civi\Test\TransactionalInterface;
-
 /**
  * AfformPalette API Test
  * @group headless
-- 
2.25.1