mysociety / theyworkforyou

tests/AlertsPageTest.php

Spacing +5 added lines, -5 removed lines

@@ -21,31 +21,31 @@
     }
 
     public function testKeywordOnly() {
-        $page = $this->fetch_page([ 'alertsearch' => 'elephant']);
+        $page = $this->fetch_page(['alertsearch' => 'elephant']);
         $this->assertStringContainsString('Receive alerts when [elephant] is mentioned', $page);
     }
 
     public function testPostCodeOnly() {
-        $page = $this->fetch_page([ 'alertsearch' => 'SE17 3HE']);
+        $page = $this->fetch_page(['alertsearch' => 'SE17 3HE']);
         $this->assertStringContainsString('when 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);
     }

tests/UserPageTest.php

Spacing +1 added lines, -1 removed lines

@@ -40,7 +40,7 @@
     }
 
     public function testEditUserInfo() {
-        $page = $this->fetch_user_page(['pg' => 'edit' ]);
+        $page = $this->fetch_user_page(['pg' => 'edit']);
         $this->assertStringContainsString('Edit your details', $page);
         $this->assertStringContainsString('name="pg" value="edit"', $page);
         $this->assertStringContainsString('value="Test"', $page);

tests/VotesTest.php

Spacing +1 added lines, -1 removed lines

@@ -13,7 +13,7 @@
     }
 
     private function fetch_votes_page() {
-        return $this->fetch_page([ 'pagetype' => 'votes', 'pid' => 2, 'url' => '/mp/2/test_current-mp/test_westminster_constituency/votes' ]);
+        return $this->fetch_page(['pagetype' => 'votes', 'pid' => 2, 'url' => '/mp/2/test_current-mp/test_westminster_constituency/votes']);
     }
 
     public function testVoteSummary() {

classes/Utility/LibFilter.php

Spacing +4 added lines, -4 removed lines

@@ -153,7 +153,7 @@ 
 
     public function escape_comments($data) {
 
-        $data = preg_replace_callback("/<!--(.*?)-->/s", function ($matches) {
+        $data = preg_replace_callback("/<!--(.*?)-->/s", function($matches) {
             return '<!--' . HtmlSpecialChars($this->StripSingle($matches[1])) . '-->';
         }, $data);
 
@@ -203,7 +203,7 @@ 
 
     public function check_tags($data) {
 
-        $data = preg_replace_callback("/<(.*?)>/s", function ($matches) {
+        $data = preg_replace_callback("/<(.*?)>/s", function($matches) {
             return $this->process_tag($this->StripSingle($matches[1]));
         }, $data);
 
@@ -391,7 +391,7 @@ 
 
         $data = preg_replace_callback(
             '/(^|[^\w\s\';,\\-])(\s*)([a-z])/',
-            function ($m) {
+            function($m) {
                 return $m[1] . $m[2] . strtoupper($m[3]);
             },
             $data
@@ -432,7 +432,7 @@ 
 
         $data = preg_replace_callback(
             '!&([^&;]*)(?=(;|&|$))!',
-            function ($matches) {
+            function($matches) {
                 return $this->check_entity($this->StripSingle($matches[1]), $this->StripSingle($matches[2]));
             },
             $data

classes/Utility/Wikipedia.php

Spacing +7 added lines, -7 removed lines

@@ -106,7 +106,7 @@ 
         }
 
         # Sort into order, largest first
-        usort($phrases, function ($a, $b) {
+        usort($phrases, function($a, $b) {
             return strlen($a) < strlen($b);
         });
 
@@ -156,31 +156,31 @@ 
 
     public static function antiTagInTag($content = '', $format = 'htmlhead') {
         $tagend = -1;
-        for($tagstart = strpos($content, '<', $tagend + 1) ; $tagstart !== false && $tagstart < strlen($content); $tagstart = strpos($content, '<', $tagend)) {
+        for ($tagstart = strpos($content, '<', $tagend + 1); $tagstart !== false && $tagstart < strlen($content); $tagstart = strpos($content, '<', $tagend)) {
             // got the start of a tag.  Now find the proper end!
             $walker = $tagstart + 1;
             $open = 1;
-            while($open != 0 && $walker < strlen($content)) {
+            while ($open != 0 && $walker < strlen($content)) {
                 $nextopen = strpos($content, '<', $walker);
                 $nextclose = strpos($content, '>', $walker);
-                if($nextclose === false) {
+                if ($nextclose === false) {
                     // ERROR! Open waka without close waka!
                     // echo '<code>Error in antiTagInTag - malformed tag!</code> ';
                     return $content;
                 }
-                if($nextopen === false || $nextopen > $nextclose) {
+                if ($nextopen === false || $nextopen > $nextclose) {
                     // No more opens, but there was a close; or, a close happens before the next open.
                     // walker goes to the close+1, and open decrements
                     $open--;
                     $walker = $nextclose + 1;
-                } elseif($nextopen < $nextclose) {
+                } elseif ($nextopen < $nextclose) {
                     // an open before the next close
                     $open++;
                     $walker = $nextopen + 1;
                 }
             }
             $tagend = $walker;
-            if($tagend > strlen($content)) {
+            if ($tagend > strlen($content)) {
                 $tagend = strlen($content);
             } else {
                 $tagend--;

classes/Search/Normal.php

Spacing +5 added lines, -5 removed lines

@@ -12,7 +12,7 @@ 
         }
 
         $o = get_http_var('o');
-        $args =  [
+        $args = [
             's' => $this->searchstring,
             'p' => $pagenum,
             'num' => get_http_var('num'),
@@ -246,22 +246,22 @@ 
 
             if ($page != 1) {
                 $prev_page = $page - 1;
-                $URL->insert([ 'p' => $prev_page ]);
+                $URL->insert(['p' => $prev_page]);
                 $pagelinks['prev'] = [
                     'url' => $URL->generate(),
                 ];
-                $URL->insert([ 'p' => 1 ]);
+                $URL->insert(['p' => 1]);
                 $pagelinks['firstpage'] = [
                     'url' => $URL->generate(),
                 ];
             }
             if ($page != $numpages) {
                 $next_page = $page + 1;
-                $URL->insert([ 'p' => $next_page ]);
+                $URL->insert(['p' => $next_page]);
                 $pagelinks['next'] = [
                     'url' => $URL->generate(),
                 ];
-                $URL->insert([ 'p' => $numpages ]);
+                $URL->insert(['p' => $numpages]);
                 $pagelinks['lastpage'] = [
                     'url' => $URL->generate(),
                 ];

classes/Divisions.php

Spacing +5 added lines, -5 removed lines

@@ -68,7 +68,7 @@ 
         $where = [];
         if ($houses) {
             if (is_string($houses)) {
-                $houses = [ $houses ];
+                $houses = [$houses];
             }
             $where[] = 'house IN ("' . implode('", "', $houses) . '")';
         }
@@ -410,7 +410,7 @@ 
                     FROM person_names AS pn
                     WHERE person_id = :person_id
                     AND start_date <= :division_date AND end_date >= :division_date",
-                    [ ':person_id' => $vote['proxy'], ':division_date' => $row['division_date'] ]
+                    [':person_id' => $vote['proxy'], ':division_date' => $row['division_date']]
                 )->first();
                 $detail['proxy'] = ucfirst(member_full_name(
                     HOUSE_TYPE_COMMONS,
@@ -438,7 +438,7 @@ 
 
         foreach ($votes as $vote => $count) { // array('yes_votes', 'no_votes', 'absent_votes', 'both_votes') as $vote) {
             $votes[$vote . '_by_party'] = $votes[$vote];
-            usort($votes[$vote . '_by_party'], function ($a, $b) {
+            usort($votes[$vote . '_by_party'], function($a, $b) {
                 return $a['party'] > $b['party'];
             });
         }
@@ -561,7 +561,7 @@ 
     }
 
     private function constructYesNoVoteDescription($direction, $title, $short_text) {
-        $text = ' ' ;
+        $text = ' ';
         if ($short_text) {
             $text .= sprintf(gettext('voted %s'), $short_text);
         } else {
@@ -711,7 +711,7 @@ 
 
         $gid = get_canonical_gid($gid);
 
-        $q = $this->db->query("SELECT gid, major FROM hansard WHERE epobject_id = ( SELECT subsection_id FROM hansard WHERE gid = :gid )", [ ':gid' => $gid ])->first();
+        $q = $this->db->query("SELECT gid, major FROM hansard WHERE epobject_id = ( SELECT subsection_id FROM hansard WHERE gid = :gid )", [':gid' => $gid])->first();
         if (!$q) {
             return '';
         }

classes/Url.php

Spacing +3 added lines, -3 removed lines

@@ -41,7 +41,7 @@ 
 
         // These stores an associative array of key/value pairs that
         // we'll want passed on to other pages.
-        $this->session_vars =  [];
+        $this->session_vars = [];
 
         // Prevent things using $DATA running if it hasn't been set, ie in testing
         if (isset($DATA)) {
@@ -92,7 +92,7 @@ 
 
     public function reset() {
         // Call this to remove all the session_vars.
-        $this->session_vars =  [];
+        $this->session_vars = [];
     }
 
     /**
@@ -165,7 +165,7 @@ 
 
         global $DATA;
 
-        $url_args =  [];
+        $url_args = [];
 
         foreach (array_merge($this->session_vars, $overrideVars) as $key => $var) {
             if (is_array($var)) {

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', 'wranswmsfront', 'pbc_front', 'divisions_recent_commons', 'divisions_recent_lords',  'calendar_summary'],
+            ['hansard', 'mps', 'peers', 'alldebatesfront', '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'],