mysociety / theyworkforyou

www/docs/admin/wikipedia.php

Spacing +1 added lines, -1 removed lines

@@ -9,7 +9,7 @@
 remove_form();
 list_ignored();
 $PAGE->stripe_end([
-    [ 'type' => 'html', 'content' => $PAGE->admin_menu() ],
+    ['type' => 'html', 'content' => $PAGE->admin_menu()],
 ]);
 $PAGE->page_end();
 

classes/Subscription.php

Spacing +2 added lines, -2 removed lines

@@ -133,7 +133,7 @@ 
     }
 
     public function update_email($email) {
-        $this->update_customer([ 'email' => $email ]);
+        $this->update_customer(['email' => $email]);
     }
 
     public function update_payment_method($payment_method) {
@@ -163,7 +163,7 @@ 
             $error = 'Sorry, we could not process your payment, please try again. ';
             $error .= 'Our payment processor returned: ' . $err['message'];
             unset($_POST['stripeToken']); # So card form is shown again
-            return [ $error ];
+            return [$error];
         }
 
         $customer = $obj->id;

classes/FacebookLogin.php

Spacing +3 added lines, -3 removed lines

@@ -43,10 +43,10 @@ 
 
         try {
             $accessToken = $helper->getAccessToken();
-        } catch(\Facebook\Exceptions\FacebookResponseException $e) {
+        } catch (\Facebook\Exceptions\FacebookResponseException $e) {
             $data['error'] = 'Graph returned an error: ' . $e->getMessage();
             return $data;
-        } catch(\Facebook\Exceptions\FacebookSDKException $e) {
+        } catch (\Facebook\Exceptions\FacebookSDKException $e) {
             $data['error'] = 'Facebook SDK returned an error: ' . $e->getMessage();
             return $data;
         }
@@ -67,7 +67,7 @@ 
             return false;
         }
 
-        if (! $accessToken->isLongLived()) {
+        if (!$accessToken->isLongLived()) {
             $oAuth2Client = $this->getFacebookObject()->getOAuth2Client();
             try {
                 $accessToken = $oAuth2Client->getLongLivedAccessToken($accessToken);

classes/Model/AnnouncementManagement.php

Spacing +3 added lines, -3 removed lines

@@ -176,7 +176,7 @@ 
         }
 
         # discard any invalid banners
-        $banners = array_filter($banners, function ($banner) {
+        $banners = array_filter($banners, function($banner) {
             return is_valid_item($banner);
         });
 
@@ -198,12 +198,12 @@ 
         }
 
         # discard any invalid announcements
-        $items = array_filter($items, function ($item) {
+        $items = array_filter($items, function($item) {
             return is_valid_item($item);
         });
 
         # limit to announcements with the correct location
-        $items = array_filter($items, function ($item) use ($location) {
+        $items = array_filter($items, function($item) use ($location) {
             return in_array($location, $item->location);
         });
 

www/includes/easyparliament/templates/html/mp/_profile_footer.php

Spacing +3 added lines, -3 removed lines

@@ -5,9 +5,9 @@ 
 
     <p><?= gettext('For an explanation of the vote descriptions please see our page about <a href="/voting-information">voting information on TheyWorkForYou</a>.') ?></p>
 
-  <?php if(isset($data['photo_attribution_text'])) { ?>
+  <?php if (isset($data['photo_attribution_text'])) { ?>
     <p>
-      <?php if(isset($data['photo_attribution_link'])) { ?>
+      <?php if (isset($data['photo_attribution_link'])) { ?>
         <?= gettext('Profile photo:') ?>
         <a href="<?= $data['photo_attribution_link'] ?>"><?= $data['photo_attribution_text'] ?></a>
       <?php } else { ?>
@@ -15,7 +15,7 @@ 
       <?php } ?>
     </p>
   <?php } ?>
-  <?php if(!$image || !$image['exists']) { ?>
+  <?php if (!$image || !$image['exists']) { ?>
     <p>
         We&rsquo;re missing a photo of <?= $full_name ?>.
         If you have a photo <em>that you can release under

www/docs/api/key.php

Spacing +2 added lines, -2 removed lines

@@ -40,9 +40,9 @@
 
             try {
                 $invoice = $sub->latest_invoice;
-                $invoice->pay([ 'expand' => [ 'payment_intent' ] ]);
+                $invoice->pay(['expand' => ['payment_intent']]);
             } catch (\Stripe\Exception\CardException $e) {
-                $invoice = \Stripe\Invoice::retrieve($sub->latest_invoice, ['expand' => [ 'payment_intent'] ]);
+                $invoice = \Stripe\Invoice::retrieve($sub->latest_invoice, ['expand' => ['payment_intent']]);
             }
         } else {
             $invoice = $sub->latest_invoice;

www/docs/api/hook.php

Spacing +2 added lines, -2 removed lines

@@ -57,10 +57,10 @@
         # Update the invoice's PaymentIntent and Charge to say it came from TWFY (for CSV export)
         # Both are shown in the Stripe admin, annoyingly
         if ($obj->payment_intent) {
-            \Stripe\PaymentIntent::update($obj->payment_intent, [ 'description' => 'TheyWorkForYou' ]);
+            \Stripe\PaymentIntent::update($obj->payment_intent, ['description' => 'TheyWorkForYou']);
         }
         if ($obj->charge) {
-            \Stripe\Charge::update($obj->charge, [ 'description' => 'TheyWorkForYou' ]);
+            \Stripe\Charge::update($obj->charge, ['description' => 'TheyWorkForYou']);
         }
     } catch (\Stripe\Exception\ApiErrorException $e) {
     }

classes/AlertView/Simple.php

Spacing +4 added lines, -4 removed lines

@@ -121,8 +121,8 @@ 
         $existing = $this->alert->fetch_by_token($confirmation);
         preg_match('/speaker:(\d+)/', $existing['criteria'], $matches);
         $old_mp_id = $matches[1];
-        $old_mp = new \MySociety\TheyWorkForYou\Member([ 'person_id' => $old_mp_id ]);
-        $new_mp = new \MySociety\TheyWorkForYou\Member([ 'constituency' => $old_mp->constituency, 'house' => 1 ]);
+        $old_mp = new \MySociety\TheyWorkForYou\Member(['person_id' => $old_mp_id]);
+        $new_mp = new \MySociety\TheyWorkForYou\Member(['constituency' => $old_mp->constituency, 'house' => 1]);
 
         $q = $this->db->query(
             "SELECT alert_id, criteria, registrationtoken FROM alerts
@@ -185,8 +185,8 @@ 
         preg_match('/speaker:(\d+)/', $existing['criteria'], $matches);
         $criteria = $matches[1];
 
-        $old_mp = new \MySociety\TheyWorkForYou\Member([ 'person_id' => $criteria ]);
-        $new_mp = new \MySociety\TheyWorkForYou\Member([ 'constituency' => $old_mp->constituency, 'house' => 1 ]);
+        $old_mp = new \MySociety\TheyWorkForYou\Member(['person_id' => $criteria]);
+        $new_mp = new \MySociety\TheyWorkForYou\Member(['constituency' => $old_mp->constituency, 'house' => 1]);
 
         if ($this->alert->fetch_by_mp($existing['email'], $new_mp->person_id)) {
             $data = [

classes/PolicyPositions.php

Spacing +1 added lines, -1 removed lines

@@ -210,7 +210,7 @@
                 $has_strong = 1;
             }
             $full_sentence = $consistency . ' ' . $policy_description;
-            $out = [ 'full_sentence' => $full_sentence, 'score' => $dmpscore, 'position' => $consistency, 'has_strong' => $has_strong ];
+            $out = ['full_sentence' => $full_sentence, 'score' => $dmpscore, 'position' => $consistency, 'has_strong' => $has_strong];
         }
 
         return $out;