mysociety / theyworkforyou

tests/DivisionsTest.php

Spacing +5 added lines, -5 removed lines

@@ -17,17 +17,17 @@ 
     }
 
     private function fetch_division_page() {
-        return $this->fetch_page( array( 'pagetype' => 'divisions', 'pid' => 2, 'policy' => 363, 'url' => '/mp/2/test_current-mp/test_westminster_constituency/divisions' ) );
+        return $this->fetch_page(array('pagetype' => 'divisions', 'pid' => 2, 'policy' => 363, 'url' => '/mp/2/test_current-mp/test_westminster_constituency/divisions'));
     }
 
     private function fetch_mp_recent_page()
     {
-        $vars = array( 'pagetype' => 'recent', 'pid' => 2, 'url' => '/mp/2/test_current-mp/test_westminster_constituency/recent' );
+        $vars = array('pagetype' => 'recent', 'pid' => 2, 'url' => '/mp/2/test_current-mp/test_westminster_constituency/recent');
         return $this->base_fetch_page($vars, 'mp', 'index.php', '/mp/recent.php');
     }
 
     private function fetch_recent_page() {
-        return $this->base_fetch_page( array('url' => '/divisions' ), 'divisions', 'index.php', '/divisions/index.php' );
+        return $this->base_fetch_page(array('url' => '/divisions'), 'divisions', 'index.php', '/divisions/index.php');
     }
 
 
@@ -113,7 +113,7 @@ 
     }
 
     public function testNotEnoughInfoStatement() {
-        return $this->fetch_page( array( 'pagetype' => 'divisions', 'pid' => 2, 'policy' => 810, 'url' => '/mp/2/test_current-mp/test_westminster_constituency/divisions' ) );
+        return $this->fetch_page(array('pagetype' => 'divisions', 'pid' => 2, 'policy' => 810, 'url' => '/mp/2/test_current-mp/test_westminster_constituency/divisions'));
         $this->assertContains('we don’t have enough information to calculate Test Current-MP’s position', $page);
     }
 
@@ -124,7 +124,7 @@ 
     }
 
     public function testSingleDivision() {
-        $page = $this->base_fetch_page( array('url' => '/divisions/division.php', 'vote' => 'pw-3012-01-01-1-commons' ), 'divisions', 'division.php', '/divisions/division.php' );
+        $page = $this->base_fetch_page(array('url' => '/divisions/division.php', 'vote' => 'pw-3012-01-01-1-commons'), 'divisions', 'division.php', '/divisions/division.php');
         $this->assertContains('A majority of MPs  <b>voted in favour</b> of a thing', $page);
         $this->assertContains('198 for', $page);
         $this->assertContains('98 against', $page);

tests/PeopleTest.php

Spacing +1 added lines, -1 removed lines

@@ -11,7 +11,7 @@
      */
     public function getDataSet()
     {
-        return $this->createMySQLXMLDataSet(dirname(__FILE__).'/_fixtures/people.xml');
+        return $this->createMySQLXMLDataSet(dirname(__FILE__) . '/_fixtures/people.xml');
     }
 
     /**

tests/HansardTest.php

Spacing +1 added lines, -1 removed lines

@@ -11,7 +11,7 @@
      */
     public function getDataSet()
     {
-        return $this->createMySQLXMLDataSet(dirname(__FILE__).'/_fixtures/hansard.xml');
+        return $this->createMySQLXMLDataSet(dirname(__FILE__) . '/_fixtures/hansard.xml');
     }
 
     /**

tests/FetchPageTestCase.php

Spacing +1 added lines, -1 removed lines

@@ -9,7 +9,7 @@
     protected function base_fetch_page($vars, $dir, $page = 'index.php', $req_uri = '')
     {
         foreach ($vars as $k => $v) {
-            $vars[$k] =  $k . '=' . urlencode($v);
+            $vars[$k] = $k . '=' . urlencode($v);
         }
 
         if (!$req_uri) {

tests/VotesTest.php

Spacing +1 added lines, -1 removed lines

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

tests/AcceptBasicTest.php

Spacing +1 added lines, -1 removed lines

@@ -11,7 +11,7 @@
      */
     public function getDataSet()
     {
-        return $this->createMySQLXMLDataSet(dirname(__FILE__).'/_fixtures/acceptance.xml');
+        return $this->createMySQLXMLDataSet(dirname(__FILE__) . '/_fixtures/acceptance.xml');
     }
 
     private function fetch_page($path, $file = 'index.php', $vars = array())

tests/AlertsTest.php

Spacing +6 added lines, -6 removed lines

@@ -11,7 +11,7 @@ 
      */
     public function getDataSet()
     {
-        return $this->createMySQLXMLDataSet(dirname(__FILE__).'/_fixtures/alerts.xml');
+        return $this->createMySQLXMLDataSet(dirname(__FILE__) . '/_fixtures/alerts.xml');
     }
 
     /**
@@ -31,7 +31,7 @@ 
     {
         $ALERT = new ALERT();
 
-        $response = $ALERT->fetch(0,0);
+        $response = $ALERT->fetch(0, 0);
 
         // Make sure we only get one response
         $this->assertEquals(1, count($response['data']));
@@ -48,7 +48,7 @@ 
     {
         $ALERT = new ALERT();
 
-        $response = $ALERT->fetch(1,0);
+        $response = $ALERT->fetch(1, 0);
 
         // Make sure we only get two responses
         $this->assertEquals(2, count($response['data']));
@@ -65,7 +65,7 @@ 
     {
         $ALERT = new ALERT();
 
-        $response = $ALERT->fetch(0,1);
+        $response = $ALERT->fetch(0, 1);
 
         // Make sure we only get one response
         $this->assertEquals(1, count($response['data']));
@@ -82,7 +82,7 @@ 
     {
         $ALERT = new ALERT();
 
-        $response = $ALERT->fetch(1,1);
+        $response = $ALERT->fetch(1, 1);
 
         // Make sure we only get one response
         $this->assertEquals(1, count($response['data']));
@@ -99,7 +99,7 @@ 
     {
         $ALERT = new ALERT();
 
-        $response = $ALERT->fetch_between(1,1, '2014-02-01', '2014-02-02');
+        $response = $ALERT->fetch_between(1, 1, '2014-02-01', '2014-02-02');
 
         // Make sure we only get one response
         $this->assertEquals(1, count($response['alerts']));

scripts/generate_party_positions.php

Spacing +4 added lines, -4 removed lines

@@ -6,17 +6,17 @@
 $policies = new MySociety\TheyWorkForYou\Policies;
 
 $party_count = 0;
-foreach ( $parties as $party ) {
+foreach ($parties as $party) {
     $party = new MySociety\TheyWorkForYou\Party($party);
 
     $positions = $party->calculateAllPolicyPositions($policies);
 
-    if ( count( $positions ) ) {
+    if (count($positions)) {
         $party_count++;
     }
 
-    foreach ( $positions as $position ) {
-        $party->cache_position( $position );
+    foreach ($positions as $position) {
+        $party->cache_position($position);
     }
 }
 

scripts/video-availability.php

Braces +2 added lines, -1 removed lines

@@ -22,7 +22,8 @@
     if ($start_date == $end_date) {
         $qq = $db->query("update hansard set video_status = video_status | 1
 	    where hdate='$start_date' and htime>='$start_time' and htime<'$end_time' and major=1");
-    } else {
+    }
+    else {
         $qq = $db->query("update hansard set video_status = video_status | 1
 	    where ((hdate='$start_date' and htime>='$start_time')
 	       or (hdate='$end_date' and htime<'$end_time')) and major=1");