mysociety / theyworkforyou

www/includes/easyparliament/templates/html/homepage/devolved-list.php

Spacing +2 added lines, -2 removed lines

@@ -1,7 +1,7 @@
 <?php
-$nav_items =  [
+$nav_items = [
     ['hansard', 'alldebatesfront', 'mps', 'peers', 'interests_home', 'wranswmsfront', 'divisions_recent_commons', 'divisions_recent_lords'],
-    ['sp_home', 'sp_home', 'spdebatesfront', 'msps',  'spwransfront', 'divisions_recent_sp'],
+    ['sp_home', 'sp_home', 'spdebatesfront', 'msps', 'spwransfront', 'divisions_recent_sp'],
     ['wales_home', 'wales_home', 'wales_debates', 'mss', 'welshlanguage', 'divisions_recent_wales'],
     ['ni_home', 'ni_home', 'nioverview', 'mlas'],
 ];

www/includes/easyparliament/templates/html/interests/_register_field.php

Braces +5 added lines, -2 removed lines

@@ -13,12 +13,15 @@
             <?php $detail = $upper_detail; ?>
         </ul>
         </li>
-    <?php else : ?>
+    <?php else {
+    : ?>
         <?php if ($detail->has_value()): ?>
         <li class="interest-detail">
             <span class="interest-detail-name"><?= $detail->display_as ?>: </span>
             <span class="interest-detail-value"><?= htmlspecialchars($detail->value) ?></span>
         </li>
-        <?php endif; ?>
+        <?php endif;
+}
+?>
 
     <?php endif; ?>

www/includes/easyparliament/templates/html/interests/highlighted_2024.php

Spacing +1 added lines, -1 removed lines

@@ -53,7 +53,7 @@
                                     <th>MP's comment</th>
                                 </tr>
                     <?php foreach ($register->persons as $person) { ?>
-                        <?php foreach($person->categories as $category) { ?>
+                        <?php foreach ($person->categories as $category) { ?>
                             <?php if ($category->category_id != $category_id) { ?>
                                 <?php continue; ?>
                             <?php }; ?>

www/includes/easyparliament/templates/html/interests/_register_entry.php

Braces +5 added lines, -2 removed lines

@@ -28,9 +28,12 @@
     <?php // This is a more mininal style of entry, don't use the header structure, just print the xml content?>
     <?php if ($entry->content_format == "xml") : ?>
         <?= $entry->content ?>
-    <?php else: ?>
+    <?php else {
+    : ?>
         <p class="interest-content"><?= htmlspecialchars($entry->content) ?></p>
-    <?php endif; ?>
+    <?php endif;
+}
+?>
 <?php endif; ?>
 
     </ul>

classes/Renderer/Header.php

Spacing +3 added lines, -3 removed lines

@@ -169,7 +169,7 @@ 
 
         if ($nextprev) {
             // Four different kinds of back/forth links we might build.
-            $links =  ["first", "prev", "up", "next", "last"];
+            $links = ["first", "prev", "up", "next", "last"];
 
             foreach ($links as $type) {
                 if ($link = $this->generate_next_prev_link($nextprev, $type)) {
@@ -307,9 +307,9 @@ 
         // Page names mapping to those in metadata.php.
         // Links in the top menu, and the sublinks we see if
         // we're within that section.
-        $nav_items =  [
+        $nav_items = [
             ['home'],
-            ['hansard', 'mps', 'peers', 'alldebatesfront', 'interests_home', 'wranswmsfront', 'pbc_front', 'divisions_recent_commons', 'divisions_recent_lords',  'calendar_summary'],
+            ['hansard', 'mps', 'peers', 'alldebatesfront', 'interests_home', 'wranswmsfront', 'pbc_front', 'divisions_recent_commons', 'divisions_recent_lords', 'calendar_summary'],
             ['sp_home', 'spoverview', 'msps', 'spdebatesfront', 'divisions_recent_sp'], #'spwransfront'
             ['ni_home', 'nioverview', 'mlas'],
             ['wales_home', 'seneddoverview', 'mss', 'wales_debates', 'divisions_recent_wales'],

www/includes/easyparliament/templates/html/alert/_alert_form.php

Indentation +2 added lines, -2 removed lines

@@ -137,10 +137,10 @@
                   <h3><?= gettext('Current keywords in this alert:') ?></h3>
                     <ul class="keyword-list">
                       <?php foreach ($keywords as $word) {
-                          if (!in_array($word, $skip_keyword_terms)) { ?>
+                            if (!in_array($word, $skip_keyword_terms)) { ?>
                           <li class="keyword-list__tag keyword-list__tag--included"><?= _htmlspecialchars($word) ?>
                       <?php }
-                          } ?>
+                            } ?>
                     </ul>
                   </div>
 

www/includes/easyparliament/alert.php

Spacing +7 added lines, -7 removed lines

@@ -40,7 +40,7 @@ 
     public $token_checked = null;
     private $alert_id = "";
     public $email = "";
-    public $criteria = "";		// Sets the terms that are used to produce the search results.
+    public $criteria = ""; // Sets the terms that are used to produce the search results.
     public $ignore_speaker_votes = 0;
 
     private $db;
@@ -72,7 +72,7 @@ 
             $contents = ['criteria' => $row['criteria'], 'count' => $row['cnt']];
             $data[] = $contents;
         }
-        $data =  ['alerts' => $data];
+        $data = ['alerts' => $data];
         return $data;
     }
 
@@ -101,7 +101,7 @@ 
 
         $data = $q->fetchAll();
         $info = "Alert";
-        $data =  ['info' => $info, 'data' => $data];
+        $data = ['info' => $info, 'data' => $data];
 
         return $data;
     }
@@ -300,12 +300,12 @@ 
         }
 
         // Arrays we need to send a templated email.
-        $data =  [
+        $data = [
             'to' 		=> $details['email'],
             'template' 	=> 'alert_confirmation',
         ];
 
-        $merge =  [
+        $merge = [
             'CONFIRMURL'	=> $confirmurl,
             'CRITERIA'	=> $this->criteria_pretty(),
         ];
@@ -319,7 +319,7 @@ 
     }
 
     public function send_already_signedup_email($details) {
-        $data =  [
+        $data = [
             'to' 		=> $details['email'],
             'template' 	=> 'alert_already_signedup',
         ];
@@ -327,7 +327,7 @@ 
         $criteria = \MySociety\TheyWorkForYou\Utility\Alert::detailsToCriteria($details);
         $this->criteria = $criteria;
 
-        $merge =  [
+        $merge = [
             'CRITERIA'	=> $this->criteria_pretty(),
         ];
 

tests/AlertsPageTest.php

Spacing +15 added lines, -15 removed lines

@@ -25,66 +25,66 @@ 
     }
 
     public function testKeywordOnly() {
-        $page = $this->fetch_page([ 'alertsearch' => 'elephant']);
+        $page = $this->fetch_page(['alertsearch' => 'elephant']);
         $this->assertStringContainsString('What word or phrase would you like to receive alerts about', $page);
         $this->assertStringContainsString('<input type="text" id="words0" name="words[]" aria-required="true" value="elephant"', $page);
     }
 
     public function testSpeakerId() {
-        $page = $this->fetch_page([ 'alertsearch' => 'speaker:2']);
+        $page = $this->fetch_page(['alertsearch' => 'speaker:2']);
         $this->assertStringContainsString('Mrs Test Current-MP', $page);
     }
 
     public function testPostCodeOnly() {
-        $page = $this->fetch_page([ 'alertsearch' => 'SE17 3HE']);
+        $page = $this->fetch_page(['alertsearch' => 'SE17 3HE']);
         $this->assertStringContainsString('Mrs Test Current-MP', $page);
     }
 
     public function testPostCodeWithKeyWord() {
-        $page = $this->fetch_page([ 'alertsearch' => 'SE17 3HE elephant']);
+        $page = $this->fetch_page(['alertsearch' => 'SE17 3HE elephant']);
         $this->assertStringContainsString('You have used a postcode and something else', $page);
         $this->assertStringContainsString('Mentions of [elephant] by your MP, Mrs Test Current-MP', $page);
         $this->assertStringNotContainsString('by your MSP', $page);
     }
 
     public function testScottishPostcodeWithKeyword() {
-        $page = $this->fetch_page([ 'alertsearch' => 'PH6 2DB elephant']);
+        $page = $this->fetch_page(['alertsearch' => 'PH6 2DB elephant']);
         $this->assertStringContainsString('You have used a postcode and something else', $page);
         $this->assertStringContainsString('Mentions of [elephant] by your MP, Mr Test2 Current-MP', $page);
         $this->assertStringContainsString('Mentions of [elephant] by your MSP, Mrs Test Current-MSP', $page);
     }
 
     public function testPostcodeAndKeywordWithNoSittingMP() {
-        $page = $this->fetch_page([ 'alertsearch' => 'OX1 4LF elephant']);
+        $page = $this->fetch_page(['alertsearch' => 'OX1 4LF elephant']);
         $this->assertStringContainsString('You have used a postcode and something else', $page);
         $this->assertStringNotContainsString('Did you mean to get alerts for when your MP', $page);
     }
 
     public function testBasicKeyWordAlertsCreation() {
-        $page = $this->fetch_page([ 'step' => 'define']);
+        $page = $this->fetch_page(['step' => 'define']);
         $this->assertStringContainsString('What word or phrase would you like to receive alerts about', $page);
         $this->assertStringContainsString('<input type="text" id="words0" name="words[]" aria-required="true" value=""', $page);
 
-        $page = $this->fetch_page([ 'step' => 'review', 'email' => 'test@example.org', 'words[]' => 'fish']);
+        $page = $this->fetch_page(['step' => 'review', 'email' => 'test@example.org', 'words[]' => 'fish']);
         $this->assertStringContainsString('Review Your Alert', $page);
         $this->assertStringContainsString('<input type="hidden" name="words[]" value="fish"', $page);
 
-        $page = $this->fetch_page([ 'step' => 'confirm', 'email' => 'test@example.org', 'words[]' => 'fish']);
+        $page = $this->fetch_page(['step' => 'confirm', 'email' => 'test@example.org', 'words[]' => 'fish']);
         $this->assertStringContainsString('We’re nearly done', $page);
         $this->assertStringContainsString('You should receive an email shortly', $page);
     }
 
     public function testMultipleKeyWordAlertsCreation() {
-        $page = $this->fetch_page([ 'step' => 'define']);
+        $page = $this->fetch_page(['step' => 'define']);
         $this->assertStringContainsString('What word or phrase would you like to receive alerts about', $page);
         $this->assertStringContainsString('<input type="text" id="words0" name="words[]" aria-required="true" value=""', $page);
 
-        $page = $this->fetch_page([ 'step' => 'review', 'email' => 'test@example.org', 'words[]' => ['fish', 'salmon']]);
+        $page = $this->fetch_page(['step' => 'review', 'email' => 'test@example.org', 'words[]' => ['fish', 'salmon']]);
         $this->assertStringContainsString('Review Your Alert', $page);
         $this->assertStringContainsString('<input type="hidden" name="words[]" value="fish"', $page);
         $this->assertStringContainsString('<input type="hidden" name="words[]" value="salmon"', $page);
 
-        $page = $this->fetch_page([ 'step' => 'confirm', 'email' => 'test@example.org', 'words[]' => ['fish', 'salmon']]);
+        $page = $this->fetch_page(['step' => 'confirm', 'email' => 'test@example.org', 'words[]' => ['fish', 'salmon']]);
         $this->assertStringContainsString('You should receive an email shortly', $page);
     }
 
@@ -93,12 +93,12 @@ 
         $this->assertStringContainsString('What word or phrase would you like to receive alerts about', $page);
         $this->assertStringContainsString('<input type="text" id="words0" name="words[]" aria-required="true" value=""', $page);
 
-        $page = $this->get_page([ 'step' => 'review', 'words[]' => ['fish', 'salmon']]);
+        $page = $this->get_page(['step' => 'review', 'words[]' => ['fish', 'salmon']]);
         $this->assertStringContainsString('Review Your Alert', $page);
         $this->assertStringContainsString('<input type="hidden" name="words[]" value="fish"', $page);
         $this->assertStringContainsString('<input type="hidden" name="words[]" value="salmon"', $page);
 
-        $page = $this->get_page([ 'step' => 'confirm', 'words[]' => ['fish', 'salmon']]);
+        $page = $this->get_page(['step' => 'confirm', 'words[]' => ['fish', 'salmon']]);
         $this->assertStringContainsString('You will now receive email alerts on any day when [fish salmon] is mentioned in parliament', $page);
     }
 
@@ -125,7 +125,7 @@ 
         $this->assertStringContainsString('<input type="hidden" name="words[]" value="fish"', $page);
         $this->assertStringContainsString('<input type="hidden" name="representative" value="Mrs Test Current-MP"', $page);
 
-        $page = $this->get_page([ 'step' => 'confirm', 'words[]' => 'fish', 'representative' => 'Mrs Test Current-MP']);
+        $page = $this->get_page(['step' => 'confirm', 'words[]' => 'fish', 'representative' => 'Mrs Test Current-MP']);
         $this->assertStringContainsString('You will now receive email alerts on any day when Mrs Test Current-MP mentions [fish] in parliament', $page);
     }
 }

www/includes/easyparliament/templates/html/alert/index.php

Indentation +15 added lines, -15 removed lines

@@ -129,22 +129,22 @@ 
           </div>
       <?php } elseif ($step !== '') { ?>
         <?php include '_alert_form.php';
-      } elseif ($this_step == '') {
-          if(
-              !$results && (
-                  $members ||
+        } elseif ($this_step == '') {
+            if(
+                !$results && (
+                    $members ||
                 (isset($constituencies) && count($constituencies) > 0) ||
                 ($alertsearch)
-              )
-          ) {
-              /* We need to disambiguate the user's instructions */
-              $member_options = false;
-              ?>
+                )
+            ) {
+                /* We need to disambiguate the user's instructions */
+                $member_options = false;
+                ?>
         <div class="alert-section alert-section--disambiguation">
             <div class="alert-section__primary">
 
               <?php if ($members) {
-                  $member_options = true; ?>
+                    $member_options = true; ?>
                 <h3><?= sprintf(gettext('Sign up for alerts when people matching <i>%s</i> speaks'), _htmlspecialchars($search_term)) ?></h3>
                 <ul>
                   <?php
@@ -157,7 +157,7 @@ 
                             <input type="hidden" name="pid" value="<?= $row['person_id'] ?>">
                             <input type="hidden" name="ignore_speaker_votes" value="<?= $ignore_speaker_votes ?>">
                             <?php
-                                  $name = member_full_name($row['house'], $row['title'], $row['given_name'], $row['family_name'], $row['lordofname']);
+                                    $name = member_full_name($row['house'], $row['title'], $row['given_name'], $row['family_name'], $row['lordofname']);
                         if ($row['constituency']) {
                             $name .= ' (' . gettext($row['constituency']) . ')';
                         }
@@ -171,7 +171,7 @@ 
               <?php } ?>
 
               <?php if (isset($constituencies) && count($constituencies) > 0) {
-                  $member_options = true; ?>
+                    $member_options = true; ?>
                 <h3><?= sprintf(gettext('Sign up for alerts when MPs for constituencies matching <i>%s</i> speaks'), _htmlspecialchars($search_term)) ?></h3>
                 <ul>
                 <?php foreach ($constituencies as $constituency => $member) { ?>
@@ -189,7 +189,7 @@ 
               <?php } ?>
 
               <?php if ($alertsearch) {
-                  if (!$member_options) { ?>
+                    if (!$member_options) { ?>
                 <h3><?= gettext('That doesn’t match a person, postcode or constituency. Search again to refine your email alert.') ?></h3>
                 <ul>
                     <?php if (isset($mistakes['postcode_and'])) { ?>
@@ -200,8 +200,8 @@ 
                             want? You will only get an alert if all of these
                             are mentioned in the same debate.') ?>
                           <?php if (isset($member_alertsearch)) {
-                              printf(gettext('Did you mean to get alerts for when your representative mentions something instead? If so maybe you want to subscribe to…'));
-                          } ?>
+                                printf(gettext('Did you mean to get alerts for when your representative mentions something instead? If so maybe you want to subscribe to…'));
+                            } ?>
                         </em>
                     </li>
                     <?php } ?>

Spacing +1 added lines, -1 removed lines

@@ -130,7 +130,7 @@
       <?php } elseif ($step !== '') { ?>
         <?php include '_alert_form.php';
       } elseif ($this_step == '') {
-          if(
+          if (
               !$results && (
                   $members ||
                 (isset($constituencies) && count($constituencies) > 0) ||