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/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");

www/docs/trackback/index.php

Spacing +6 added lines, -6 removed lines

@@ -5,12 +5,12 @@ 
 $this_page = 'trackback';
 
 // The gid is the gid of the thing being trackedback to.
-$epobject_id= get_http_var('e');			// eg, '3424'
+$epobject_id = get_http_var('e'); // eg, '3424'
 
-$url 		= get_http_var('url');
+$url = get_http_var('url');
 $blog_name 	= get_http_var('blog_name');
-$title 		= get_http_var('title');
-$excerpt 	= get_http_var('excerpt');
+$title = get_http_var('title');
+$excerpt = get_http_var('excerpt');
 
 if ($title == '' && $url == '' && $blog_name == '') {
     // If it doesn't look like a trackback at all...
@@ -23,11 +23,11 @@ 
     exit;
 }
 
-if ((strlen(''.$epobject_id)) && (empty($HTTP_GET_VARS['__mode'])) && (strlen(''.$url))) {
+if ((strlen('' . $epobject_id)) && (empty($HTTP_GET_VARS['__mode'])) && (strlen('' . $url))) {
 
     header('Content-Type: text/xml');
 
-    $trackbackdata = array (
+    $trackbackdata = array(
         'epobject_id' 	=> $epobject_id,
         'url' 			=> $url,
         'blog_name'		=> $blog_name,