mysociety / theyworkforyou

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/Footer.php

Spacing +1 added lines, -1 removed lines

@@ -12,7 +12,7 @@
 class Footer {
     public $data;
 
-    private $about_links =  ['help', 'about', 'linktous', 'news', 'privacy'];
+    private $about_links = ['help', 'about', 'linktous', 'news', 'privacy'];
     private $assemblies_links = ['hansard', 'sp_home', 'wales_home', 'ni_home', 'london_home'];
     private $international_links = ['australia', 'ireland', 'mzalendo'];
     private $tech_links = ['code', 'api', 'data', 'devmailinglist', 'contact'];

classes/People.php

Spacing +8 added lines, -8 removed lines

@@ -148,20 +148,20 @@ 
 
         $urls['plain'] = $URL->generate();
 
-        $URL->insert([ 'o' => 'n']);
+        $URL->insert(['o' => 'n']);
         $urls['by_name'] = $URL->generate();
 
-        $URL->insert([ 'o' => 'l']);
+        $URL->insert(['o' => 'l']);
         $urls['by_last'] = $URL->generate();
 
-        $URL->insert([ 'o' => 'f']);
+        $URL->insert(['o' => 'f']);
         $urls['by_first'] = $URL->generate();
 
-        $URL->insert([ 'o' => 'p']);
+        $URL->insert(['o' => 'p']);
         $urls['by_party'] = $URL->generate();
 
-        $URL->insert([ 'f' => 'csv']);
-        $URL->remove([ 'o']);
+        $URL->insert(['f' => 'csv']);
+        $URL->remove(['o']);
         if ($date = get_http_var('date')) {
             $URL->insert(['date' => $date]);
         }
@@ -213,7 +213,7 @@ 
                 $constituency = $row['constituency'] ? gettext($row['constituency']) : '';
                 $party = $row['party'] ? gettext($row['party']) : '';
                 $url = make_member_url($name, $constituency, $this->house, $p_id);
-                $narray =  [
+                $narray = [
                     'person_id' 	=> $p_id,
                     'given_name' => $row['given_name'],
                     'family_name' => $row['family_name'],
@@ -231,7 +231,7 @@ 
             uasort($data, 'by_peer_name');
         }
 
-        $data =  [
+        $data = [
             'info' =>  [
                 'order' => $order,
             ],

classes/SectionView/SpView.php

Spacing +1 added lines, -1 removed lines

@@ -24,7 +24,7 @@
 
     protected function getSearchSections() {
         return [
-            [ 'section' => 'sp' ],
+            ['section' => 'sp'],
         ];
     }
 }

classes/SectionView/WransView.php

Spacing +3 added lines, -3 removed lines

@@ -18,7 +18,7 @@ 
 
         $data = [];
 
-        $args = [ 'months' => 1 ];
+        $args = ['months' => 1];
         $WRANSLIST = new \WRANSLIST();
 
         $wrans = [];
@@ -50,10 +50,10 @@ 
 
     protected function getSearchSections() {
         $sections = [
-            [ 'section' => 'wrans', 'title' => 'Written Answers' ],
+            ['section' => 'wrans', 'title' => 'Written Answers'],
         ];
         if (get_http_var('type') == '') {
-            $sections[] = [ 'section' => 'wms', 'title' => 'Written Ministerial Statements' ];
+            $sections[] = ['section' => 'wms', 'title' => 'Written Ministerial Statements'];
         }
         return $sections;
     }

classes/SectionView/NiView.php

Spacing +3 added lines, -3 removed lines

@@ -32,7 +32,7 @@ 
 
     protected function getSearchSections() {
         return [
-            [ 'section' => 'ni' ],
+            ['section' => 'ni'],
         ];
     }
 
@@ -51,7 +51,7 @@ 
 
         $debates = $DEBATELIST->display('recent_debates', ['days' => 30, 'num' => 6], 'none');
         $MOREURL = new \MySociety\TheyWorkForYou\Url('nidebatesfront');
-        $MOREURL->insert([ 'more' => 1 ]);
+        $MOREURL->insert(['more' => 1]);
 
         // this makes sure that we don't repeat this debate in the list below
         $random_debate = null;
@@ -85,7 +85,7 @@ 
         }
 
         $data['featured'] = $featured;
-        $data['debates'] = [ 'recent' => $recent];
+        $data['debates'] = ['recent' => $recent];
 
         $data['regional'] = $this->getMLAList();
         $data['search_box'] = $this->getSearchBox($data);

classes/SectionView/DebatesView.php

Spacing +6 added lines, -6 removed lines

@@ -16,7 +16,7 @@ 
         $DEBATELIST = new \DEBATELIST();
         $debates = [];
         $debates['data'] = $DEBATELIST->display('biggest_debates', ['days' => 7, 'num' => 10], 'none');
-        $args = [ 'months' => 1 ];
+        $args = ['months' => 1];
         $debates['calendar'] = $DEBATELIST->display('calendar', $args, 'none');
         $debates['rssurl'] = $DATA->page_metadata($this_page, 'rss');
 
@@ -26,7 +26,7 @@ 
 
         $WHALLLIST = new \WHALLLIST();
         $whall['data'] = $WHALLLIST->display('biggest_debates', ['days' => 7, 'num' => 10], 'none');
-        $args = [ 'months' => 1 ];
+        $args = ['months' => 1];
         $whall['calendar'] = $WHALLLIST->display('calendar', $args, 'none');
         $whall['rssurl'] = $DATA->page_metadata($this_page, 'rss');
 
@@ -36,7 +36,7 @@ 
 
         $LORDSDEBATELIST = new \LORDSDEBATELIST();
         $lords['data'] = $LORDSDEBATELIST->display('biggest_debates', ['days' => 7, 'num' => 10], 'none');
-        $args = [ 'months' => 1 ];
+        $args = ['months' => 1];
         $lords['calendar'] = $LORDSDEBATELIST->display('calendar', $args, 'none');
 
         $lords['rssurl'] = $DATA->page_metadata($this_page, 'rss');
@@ -55,9 +55,9 @@ 
 
     protected function getSearchSections() {
         return [
-            [ 'section' => 'debates', 'title' => 'House of Commons' ],
-            [ 'section' => 'lords', 'title' => 'House of Lords' ],
-            [ 'section' => 'whall', 'title' => 'Westminster Hall' ],
+            ['section' => 'debates', 'title' => 'House of Commons'],
+            ['section' => 'lords', 'title' => 'House of Lords'],
+            ['section' => 'whall', 'title' => 'Westminster Hall'],
         ];
     }
 }

classes/SectionView/SpwransView.php

Spacing +9 added lines, -9 removed lines

@@ -21,7 +21,7 @@ 
         $this_page = 'spwransfront';
         $data = [];
 
-        $args = [ 'months' => 1 ];
+        $args = ['months' => 1];
         $WRANSLIST = new \SPWRANSLIST();
 
         $wrans = [];
@@ -43,7 +43,7 @@ 
 
     protected function getSearchSections() {
         return [
-            [ 'section' => 'spwrans' ],
+            ['section' => 'spwrans'],
         ];
     }
 
@@ -76,7 +76,7 @@ 
     }
 
     protected function get_question_mentions_html($row_data) {
-        if(count($row_data) == 0) {
+        if (count($row_data) == 0) {
             return '';
         }
         $result = '';
@@ -85,7 +85,7 @@ 
         // Keep the references until after the history that's in a timeline:
         $references = [];
         foreach ($row_data as $row) {
-            if(! $row["date"]) {
+            if (!$row["date"]) {
                 // If this mention isn't associated with a date, the difference won't be interesting.
                 $last_date = null;
             }
@@ -95,7 +95,7 @@ 
                 $daysdiff = (int) ((strtotime($row["date"]) - strtotime($last_date)) / 86400);
                 $daysstring = ($daysdiff == 1) ? "day" : "days";
                 $further = "";
-                if($first_difference_output) {
+                if ($first_difference_output) {
                     $first_difference_output = false;
                 } else {
                     $further = " a further";
@@ -120,7 +120,7 @@ 
                     $inner = "Mentioned in <a class=\"debate-speech__meta__link\" href=\"$url\">tabled written questions on $date</a>";
                     break;
                 case 4:
-                    if(preg_match('/^uk.org.publicwhip\/spq\/(.*)$/', $row['gid'], $m)) {
+                    if (preg_match('/^uk.org.publicwhip\/spq\/(.*)$/', $row['gid'], $m)) {
                         $URL = new \MySociety\TheyWorkForYou\Url("spwrans");
                         $URL->insert(['spid' => $m[1]]);
                         $relative_url = $URL->generate("none");
@@ -131,7 +131,7 @@ 
                     $inner = "Given a holding answer on $date";
                     break;
                 case 6:
-                    if(preg_match('/^uk.org.publicwhip\/spor\/(.*)$/', $row['mentioned_gid'], $m)) {
+                    if (preg_match('/^uk.org.publicwhip\/spor\/(.*)$/', $row['mentioned_gid'], $m)) {
                         $URL = new \MySociety\TheyWorkForYou\Url("spdebates");
                         $URL->insert(['id' => $m[1]]);
                         $relative_url = $URL->generate("none");
@@ -139,7 +139,7 @@ 
                     }
                     break;
                 case 7:
-                    if(preg_match('/^uk.org.publicwhip\/spq\/(.*)$/', $row['mentioned_gid'], $m)) {
+                    if (preg_match('/^uk.org.publicwhip\/spq\/(.*)$/', $row['mentioned_gid'], $m)) {
                         $referencing_spid = $m[1];
                         $URL = new \MySociety\TheyWorkForYou\Url("spwrans");
                         $URL->insert(['spid' => $referencing_spid]);
@@ -149,7 +149,7 @@ 
                     }
                     break;
             }
-            if($reference) {
+            if ($reference) {
                 $references[] = "\n<li>$inner.";
             } else {
                 $result .= "\n<li class=\"link-to-hansard\">$description$inner</span>";