mysociety / theyworkforyou

www/docs/calendar/index.php

Spacing +2 added lines, -2 removed lines

@@ -80,7 +80,7 @@ 
     foreach ($data['order'] as &$chamber) {
         if (in_array($chamber['major'] ?? 0, $majors)) {
             $URL = new \MySociety\TheyWorkForYou\Url($hansardmajors[$chamber['major']]['page_all']);
-            $URL->insert([ 'd' => $date ]);
+            $URL->insert(['d' => $date]);
             $chamber['url'] = $URL->generate();
         }
     }
@@ -110,7 +110,7 @@ 
     [$firstyear, $firstmonth, $day] = explode('-', $min_future_date);
     [$finalyear, $finalmonth, $day] = explode('-', $max_future_date);
 
-    $q =  $db->query("SELECT DISTINCT(event_date) AS event_date FROM future
+    $q = $db->query("SELECT DISTINCT(event_date) AS event_date FROM future
         WHERE event_date >= :firstdate
         AND event_date <= :finaldate
         AND deleted = 0

www/docs/user/login/index.php

Spacing +2 added lines, -2 removed lines

@@ -19,14 +19,14 @@
 if (get_http_var("submitted") == "true") {
     // Form has been submitted, so check input.
 
-    $email 		= get_http_var("email");
+    $email = get_http_var("email");
     $password 	= get_http_var("password");
     $remember 	= get_http_var("remember");
 
     // The user may have tried to do something that requires being logged in.
     // In which case we should arrive here with that page's URL in 'ret'.
     // We can then send the user there after log in.
-    $returnurl 	= get_http_var("ret");
+    $returnurl = get_http_var("ret");
 
     $errors = [];
 

scripts/alertmailer.php

Upper-Lower-Casing +1 added lines, -1 removed lines

@@ -469,7 +469,7 @@
 
     $text .= '====================';
     $sentemails++;
-    mlog("SEND $sentemails : Sending email to $current[email] ... ");
+    mlog("send $sentemails : Sending email to $current[email] ... ");
     $d = ['to' => $current['email'], 'template' => $template];
     $m = [
         'DATA' => $text,

scripts/alertgonemps.php

Braces +2 added lines, -1 removed lines

@@ -166,7 +166,8 @@
         $success = send_template_email($d, $m, true);
         mlog("sent ... ");
         # sleep if time between sending mails is less than a certain number of seconds on average
-        if (((time() - $start_time) / $sentemails) < 0.5) { # number of seconds per mail not to be quicker than
+        if (((time() - $start_time) / $sentemails) < 0.5) {
+# number of seconds per mail not to be quicker than
             mlog("pausing ... ");
             sleep(1);
         }

Upper-Lower-Casing +1 added lines, -1 removed lines

@@ -154,7 +154,7 @@
     global $globalsuccess, $sentemails, $nomail, $start_time;
 
     $sentemails++;
-    mlog("SEND $sentemails : Sending email to $current[email] ... ");
+    mlog("send $sentemails : Sending email to $current[email] ... ");
     $d = ['to' => $current['email'], 'template' => $template];
     $m = [
         'DATA' => join("\n", $data),

scripts/mprss.php

Braces +2 added lines, -1 removed lines

@@ -106,7 +106,8 @@
     // Write the text to the file...
     $filename = $rsspath . $person_id . '.rdf';
     $fh = @fopen($filename, "w");
-    if (!$fh) { # Problem writing, just carry on
+    if (!$fh) {
+# Problem writing, just carry on
         echo "Could not write to file ($filename)\n";
         continue;
     }

Spacing +1 added lines, -1 removed lines

@@ -24,7 +24,7 @@
 foreach ($q as $person) {
     $person_id = $person['person_id'];
 
-    $args =  [ 'person_id' => $person_id ];
+    $args = ['person_id' => $person_id];
     $speeches = $HANSARDLIST->display('person', $args, 'none');
 
     // Some data about this person that we'll need for the feed.

tests/PageTest.php

Spacing +11 added lines, -11 removed lines

@@ -16,20 +16,20 @@ 
     }
 
     public function testQueenie() {
-        $page = $this->fetch_page([ 'representative_type' => 'royal', 'n' => 'elizabeth_the_second' ]);
+        $page = $this->fetch_page(['representative_type' => 'royal', 'n' => 'elizabeth_the_second']);
         $this->assertStringContainsString('Elizabeth the Second', $page);
         $this->assertStringContainsString('Coronated on 2 June 1953', $page);
     }
 
     public function testSittingMP() {
-        $page = $this->fetch_page([ 'pid' => 2, 'url' => '/mp/2/test_current-mp/test_westminster_constituency' ]);
+        $page = $this->fetch_page(['pid' => 2, 'url' => '/mp/2/test_current-mp/test_westminster_constituency']);
         $this->assertStringContainsString('Test Current-MP', $page);
         $this->assertMatchesRegularExpression('#<span class="person-header__about__position__constituency">\s*Test Westminster Constituency\s*</span>#', $page);
         $this->assertMatchesRegularExpression('#<span class="person-header__about__position__role">\s*Labour\s*MP\s*</span>#', $page);
     }
 
     public function testSittingMLA() {
-        $page = $this->fetch_page([ 'pid' => 4, 'representative_type' => 'mla', 'url' => '/mp/4/test_current-mla' ]);
+        $page = $this->fetch_page(['pid' => 4, 'representative_type' => 'mla', 'url' => '/mp/4/test_current-mla']);
         $this->assertStringContainsString('Test Current-MLA', $page);
         $this->assertMatchesRegularExpression('#<span class="person-header__about__position__constituency">\s*Test Northern Ireland Constituency\s*</span>#', $page);
         $this->assertMatchesRegularExpression('#<span class="person-header__about__position__role">\s*Sinn Féin\s*MLA\s*</span>#', $page);
@@ -39,7 +39,7 @@ 
      * Ensure that the Sinn Fein message is displayed for SF MPs.
      */
     public function testSittingSinnFeinMP() {
-        $page = $this->fetch_page([ 'pid' => 15, 'url' => '/mp/15/test_current-sf-mp/test_westminster_constituency' ]);
+        $page = $this->fetch_page(['pid' => 15, 'url' => '/mp/15/test_current-sf-mp/test_westminster_constituency']);
         $this->assertStringContainsString('Sinn F&eacute;in MPs do not take their seats in Parliament.', $page);
     }
 
@@ -47,7 +47,7 @@ 
      * Ensure that the Sinn Fein message is not displayed for non-SF MPs.
      */
     public function testSittingNonSinnFeinMP() {
-        $page = $this->fetch_page([ 'pid' => 2, 'url' => '/mp/2/test_current-mp/test_westminster_constituency' ]);
+        $page = $this->fetch_page(['pid' => 2, 'url' => '/mp/2/test_current-mp/test_westminster_constituency']);
         $this->assertStringNotContainsString('Sinn F&eacute;in MPs do not take their seats in Parliament.', $page);
     }
 
@@ -55,7 +55,7 @@ 
      * Ensure that the Speaker is given the correct constituency.
      */
     public function testSpeaker() {
-        $page = $this->fetch_page([ 'pid' => 13, 'url' => '/mp/13/test_speaker/buckingham' ]);
+        $page = $this->fetch_page(['pid' => 13, 'url' => '/mp/13/test_speaker/buckingham']);
         $this->assertMatchesRegularExpression('#<span class="person-header__about__position__role">\s*Speaker\s*MP\s*</span>#', $page);
     }
 
@@ -65,7 +65,7 @@ 
         # makes sure it is empty in case there's something hanging
         # about in memcached
         $banner->set_text('', "banner");
-        $page = $this->fetch_page([ 'url' => '/' ]);
+        $page = $this->fetch_page(['url' => '/']);
         $this->assertStringNotContainsString('<div class="banner">', $page);
         $this->assertStringNotContainsString('This is a banner', $page);
 
@@ -85,20 +85,20 @@ 
         ';
 
         $banner->set_text($banner_config, "banner");
-        $page = $this->fetch_page([ 'url' => '/' ]);
+        $page = $this->fetch_page(['url' => '/']);
         $this->assertStringContainsString('This is a banner', $page);
 
         $banner->set_text('', "banner");
-        $page = $this->fetch_page([ 'url' => '/' ]);
+        $page = $this->fetch_page(['url' => '/']);
         $this->assertStringNotContainsString('<div class="banner">', $page);
         $this->assertStringNotContainsString('This is a banner', $page);
     }
 
     public function testNewMPMessage() {
-        $page = $this->fetch_page([ 'pid' => 17, 'url' => '/mp/17/recent_mp/test_westminster_constituency' ]);
+        $page = $this->fetch_page(['pid' => 17, 'url' => '/mp/17/recent_mp/test_westminster_constituency']);
         $this->assertStringNotContainsString('is a recently elected MP', $page);
         self::$db->query('UPDATE member SET entered_house = NOW() WHERE person_id = 17');
-        $page = $this->fetch_page([ 'pid' => 17, 'url' => '/mp/17/recent_mp/test_westminster_constituency' ]);
+        $page = $this->fetch_page(['pid' => 17, 'url' => '/mp/17/recent_mp/test_westminster_constituency']);
         $this->assertStringContainsString('is a recently elected MP', $page);
     }
 

tests/PartyTest.php

Spacing +3 added lines, -3 removed lines

@@ -227,7 +227,7 @@ 
     }
 
     private function getMemberFromPersonId($person_id) {
-        return new MySociety\TheyWorkForYou\Member([ "person_id" => $person_id ]);
+        return new MySociety\TheyWorkForYou\Member(["person_id" => $person_id]);
     }
 
     public function testMPPartyPolicyTextWhenDiffers() {
@@ -253,7 +253,7 @@ 
     public function testMPPartyPolicyWherePartyMissingPositions() {
         // When an MP has votes, but there is no broader party policy to compare it to
         // this goes down a funnel that shows the votes, but does not make the comparison to party.
-        $page = $this->fetch_page(['pid' => 4, 'pagetype' => 'votes','url' => '/mp/4/test_mp_d/test_westminster_constituency/votes']);
+        $page = $this->fetch_page(['pid' => 4, 'pagetype' => 'votes', 'url' => '/mp/4/test_mp_d/test_westminster_constituency/votes']);
         $this->assertStringContainsString('Test MP D', $page);
         $this->assertStringContainsString('<li class="vote-description"', $page);
         $this->assertStringNotContainsString('comparable B Party MPs voted', $page);
@@ -261,7 +261,7 @@ 
 
     public function testMPPartyPolicyTextWhenAgrees() {
         // Test when an MP mostly agrees with their party, as MP G Party 2 does with party G
-        $page = $this->fetch_page(['pagetype' => 'votes', 'pid' => 16,  'url' => '/mp/16/test_mp_g_party_2/test_westminster_constituency/votes']);
+        $page = $this->fetch_page(['pagetype' => 'votes', 'pid' => 16, 'url' => '/mp/16/test_mp_g_party_2/test_westminster_constituency/votes']);
         $this->assertStringContainsString('Test MP G Party 2', $page);
 
         $this->assertStringNotContainsString('sometimes <b>differs</b> from their party colleagues', $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() {