mysociety / theyworkforyou

www/docs/news/home_page_include.php

Braces +3 added lines, -1 removed lines

@@ -7,7 +7,9 @@
 
 $c = 0;
 foreach ($all_news as $id => $news_row) {
-    if ($c++ == 2) break;
+    if ($c++ == 2) {
+        break;
+    }
     list($title, $content, $date) = $news_row;
     $url = news_individual_link($date, $title);
     print "<h5><a href=\"$url\">" . $title . "</a></h5>";

www/docs/news/rdf.php

Braces +6 added lines, -2 removed lines

@@ -25,7 +25,9 @@ 
 <?php
     $c = 0;
     foreach ($all_news as $id => $news_row) {
-        if ($c++ == 10) break;
+        if ($c++ == 10) {
+            break;
+        }
         list($title, $content, $date) = $news_row;
         $url = "https://www.theyworkforyou.com".news_individual_link($date, $title);
         print "<rdf:li rdf:resource=\"$url\" />\n";
@@ -39,7 +41,9 @@ 
 <?php
     $c = 0;
     foreach ($all_news as $id => $news_row) {
-        if ($c++ == 10) break;
+        if ($c++ == 10) {
+            break;
+        }
         list($title, $content, $date) = $news_row;
         $url = "https://www.theyworkforyou.com".news_individual_link($date, $title);
         $excerpt = trim_characters(news_format_body($content), 0, 250);

Spacing +2 added lines, -2 removed lines

@@ -27,7 +27,7 @@ 
     foreach ($all_news as $id => $news_row) {
         if ($c++ == 10) break;
         list($title, $content, $date) = $news_row;
-        $url = "https://www.theyworkforyou.com".news_individual_link($date, $title);
+        $url = "https://www.theyworkforyou.com" . news_individual_link($date, $title);
         print "<rdf:li rdf:resource=\"$url\" />\n";
     }
 ?>
@@ -41,7 +41,7 @@ 
     foreach ($all_news as $id => $news_row) {
         if ($c++ == 10) break;
         list($title, $content, $date) = $news_row;
-        $url = "https://www.theyworkforyou.com".news_individual_link($date, $title);
+        $url = "https://www.theyworkforyou.com" . news_individual_link($date, $title);
         $excerpt = trim_characters(news_format_body($content), 0, 250);
         $date = str_replace(" ", "T", $date) . "+00:00";
 ?>

www/docs/api/api_getAlerts.php

Indentation +4 added lines, -4 removed lines

@@ -1,10 +1,10 @@
 <?php
 
 function api_getAlerts_start_date($start_date) {
-  $args = array ('start_date' => $start_date, 'end_date' => get_http_var('end_date'));
-  $alert = new ALERT();
-  $data = $alert->fetch_between($confirmed=1, $deleted=0, $args['start_date'], $args['end_date']);
-  api_output($data);
+    $args = array ('start_date' => $start_date, 'end_date' => get_http_var('end_date'));
+    $alert = new ALERT();
+    $data = $alert->fetch_between($confirmed=1, $deleted=0, $args['start_date'], $args['end_date']);
+    api_output($data);
 }
 
 ?>

Spacing +2 added lines, -2 removed lines

@@ -1,9 +1,9 @@
 <?php
 
 function api_getAlerts_start_date($start_date) {
-  $args = array ('start_date' => $start_date, 'end_date' => get_http_var('end_date'));
+  $args = array('start_date' => $start_date, 'end_date' => get_http_var('end_date'));
   $alert = new ALERT();
-  $data = $alert->fetch_between($confirmed=1, $deleted=0, $args['start_date'], $args['end_date']);
+  $data = $alert->fetch_between($confirmed = 1, $deleted = 0, $args['start_date'], $args['end_date']);
   api_output($data);
 }
 

www/docs/api/api_getComments.php

Spacing +2 added lines, -2 removed lines

@@ -24,7 +24,7 @@ 
 }
 
 function api_getComments_start_date($start_date) {
-        $args = array (
+        $args = array(
             'start_date' => $start_date,
             'end_date' => get_http_var('end_date')
         );
@@ -33,7 +33,7 @@ 
 }
 
 function api_getComments_search($s) {
-        $args = array (
+        $args = array(
             's' => $s,
             'p' => get_http_var('page'),
             'num' => get_http_var('num'),

www/docs/api/index.php

Indentation +8 added lines, -8 removed lines

@@ -26,17 +26,17 @@ 
     $match = 0;
     foreach ($methods as $method => $data) {
         if (strtolower($q_method) == strtolower($method)) {
-      if (isset($data['superuser']) && $data['superuser']) {
+        if (isset($data['superuser']) && $data['superuser']) {
         $super_check = api_is_superuser_key($key);
         if (!$super_check) {
                 if (get_http_var('docs')) {
-                  api_front_page();
+                    api_front_page();
                 } else {
-                  api_error('Invalid API key.');
-                  exit;
-              }
+                    api_error('Invalid API key.');
+                    exit;
+                }
             }
-      }
+        }
 
             api_log_call($key);
             $match++;
@@ -73,9 +73,9 @@ 
             '". Possible functions are: ' .
             join(', ', array_keys($methods));
         if (get_http_var('output')) {
-          api_error($msg);
+            api_error($msg);
         } else {
-          api_front_page($msg);
+            api_front_page($msg);
         }
     } else {
         if (get_http_var('docs')) {

Spacing +10 added lines, -10 removed lines

@@ -47,7 +47,7 @@ 
             foreach ($data['parameters'] as $parameter) {
                 if ($q_param = trim(get_http_var($parameter))) {
                     $match++;
-                    include_once 'api_'. $method . '.php';
+                    include_once 'api_' . $method . '.php';
                     api_call_user_func_or_error('api_' . $method . '_' . $parameter, array($q_param), 'API call not yet functional', 'api');
                     break;
                 }
@@ -57,9 +57,9 @@ 
                     api_error('No parameter provided to function "' .
                     _htmlspecialchars($q_method) .
                         '". Possible choices are: ' .
-                        join(', ', $data['parameters']) );
+                        join(', ', $data['parameters']));
                 } else {
-                    include_once 'api_'. $method . '.php';
+                    include_once 'api_' . $method . '.php';
                     api_call_user_func_or_error('api_' . $method, array(), 'API call not yet functional', 'api');
                     break;
                 }
@@ -93,7 +93,7 @@ 
     $DATA->set_page_metadata($this_page, 'title', "$method function");
     $PAGE->page_start();
     $PAGE->stripe_start();
-    include_once 'api_'. $method . '.php';
+    include_once 'api_' . $method . '.php';
     print '<p align="center"><strong>https://www.theyworkforyou.com/api/' . $method . '</strong></p>';
     api_call_user_func_or_error('api_' . $method . '_front', array(), 'No documentation yet', 'html');
 ?>
@@ -102,20 +102,20 @@ 
 <form method="get" action="?#output">
 <p>
 <?php foreach ($methods[$method]['parameters'] as $parameter) {
-    print $parameter . ': <input type="text" name="'.$parameter.'" value="';
+    print $parameter . ': <input type="text" name="' . $parameter . '" value="';
     if ($val = get_http_var($parameter))
         print _htmlspecialchars($val);
     print '" size="30"><br>';
 }
 ?>
 Output:
-<input id="output_js" type="radio" name="output" value="js"<?php if (get_http_var('output')=='js' || !get_http_var('output')) print ' checked'?>>
+<input id="output_js" type="radio" name="output" value="js"<?php if (get_http_var('output') == 'js' || !get_http_var('output')) print ' checked'?>>
 <label for="output_js">JS</label>
-<input id="output_xml" type="radio" name="output" value="xml"<?php if (get_http_var('output')=='xml') print ' checked'?>>
+<input id="output_xml" type="radio" name="output" value="xml"<?php if (get_http_var('output') == 'xml') print ' checked'?>>
 <label for="output_xml">XML</label>
-<input id="output_php" type="radio" name="output" value="php"<?php if (get_http_var('output')=='php') print ' checked'?>>
+<input id="output_php" type="radio" name="output" value="php"<?php if (get_http_var('output') == 'php') print ' checked'?>>
 <label for="output_php">Serialised PHP</label>
-<input id="output_rabx" type="radio" name="output" value="rabx"<?php if (get_http_var('output')=='rabx') print ' checked'?>>
+<input id="output_rabx" type="radio" name="output" value="rabx"<?php if (get_http_var('output') == 'rabx') print ' checked'?>>
 <label for="output_rabx">RABX</label>
 
 <input type="submit" value="Go">
@@ -130,7 +130,7 @@ 
         }
         print '<h4><a name="output"></a>Output</h4>';
         print '<p>URL for this: <strong>https://www.theyworkforyou.com/api/';
-        print $method . '?' . join('&amp;', $qs) . '&amp;output='._htmlspecialchars(get_http_var('output')).'</strong></p>';
+        print $method . '?' . join('&amp;', $qs) . '&amp;output=' . _htmlspecialchars(get_http_var('output')) . '</strong></p>';
         print '<pre>' . _htmlspecialchars($explorer) . '</pre>';
     }
     $sidebar = api_sidebar();

Braces +23 added lines, -12 removed lines

@@ -8,7 +8,8 @@ 
 if ($q_method = get_http_var('method')) {
     if (get_http_var('docs')) {
         $key = 'DOCS';
-    } else {
+    }
+    else {
         $key = get_http_var('key');
         if (!$key) {
             api_error('No API key provided. Please see https://www.theyworkforyou.com/api/key for more information.');
@@ -18,7 +19,8 @@ 
         if (!$check) {
             api_error('Invalid API key.');
             exit;
-        } elseif ($check === 'disabled') {
+        }
+        elseif ($check === 'disabled') {
             api_error('Your API key has been disabled.');
             exit;
         }
@@ -31,7 +33,8 @@ 
         if (!$super_check) {
                 if (get_http_var('docs')) {
                   api_front_page();
-                } else {
+                }
+                else {
                   api_error('Invalid API key.');
                   exit;
               }
@@ -58,7 +61,8 @@ 
                     _htmlspecialchars($q_method) .
                         '". Possible choices are: ' .
                         join(', ', $data['parameters']) );
-                } else {
+                }
+                else {
                     include_once 'api_'. $method . '.php';
                     api_call_user_func_or_error('api_' . $method, array(), 'API call not yet functional', 'api');
                     break;
@@ -74,16 +78,19 @@ 
             join(', ', array_keys($methods));
         if (get_http_var('output')) {
           api_error($msg);
-        } else {
+        }
+        else {
           api_front_page($msg);
         }
-    } else {
+    }
+    else {
         if (get_http_var('docs')) {
             $explorer = ob_get_clean();
             api_documentation_front($method, $explorer);
         }
     }
-} else {
+}
+else {
     api_front_page();
 }
 
@@ -103,13 +110,15 @@ 
 <p>
 <?php foreach ($methods[$method]['parameters'] as $parameter) {
     print $parameter . ': <input type="text" name="'.$parameter.'" value="';
-    if ($val = get_http_var($parameter))
-        print _htmlspecialchars($val);
+    if ($val = get_http_var($parameter)) {
+            print _htmlspecialchars($val);
+    }
     print '" size="30"><br>';
 }
 ?>
 Output:
-<input id="output_js" type="radio" name="output" value="js"<?php if (get_http_var('output')=='js' || !get_http_var('output')) print ' checked'?>>
+<input id="output_js" type="radio" name="output" value="js"<?php if (get_http_var('output')=='js' || !get_http_var('output')) {
+    print ' checked'?>>
 <label for="output_js">JS</label>
 <input id="output_xml" type="radio" name="output" value="xml"<?php if (get_http_var('output')=='xml') print ' checked'?>>
 <label for="output_xml">XML</label>
@@ -124,9 +133,11 @@ 
 <?php
     if ($explorer) {
         $qs = array();
+}
         foreach ($methods[$method]['parameters'] as $parameter) {
-            if (get_http_var($parameter))
-                $qs[] = _htmlspecialchars(rawurlencode($parameter) . '=' . urlencode(get_http_var($parameter)));
+            if (get_http_var($parameter)) {
+                            $qs[] = _htmlspecialchars(rawurlencode($parameter) . '=' . urlencode(get_http_var($parameter)));
+            }
         }
         print '<h4><a name="output"></a>Output</h4>';
         print '<p>URL for this: <strong>https://www.theyworkforyou.com/api/';

www/docs/bills/2005-06/identity-cards/index.php

Spacing +5 added lines, -5 removed lines

@@ -2,12 +2,12 @@ 
 
 $this_page = 'bill_index';
 include_once '../../../../includes/easyparliament/init.php';
-$DATA->set_page_metadata($this_page, 'heading','Identity Cards Act 2006');
+$DATA->set_page_metadata($this_page, 'heading', 'Identity Cards Act 2006');
 
 $PAGE->page_start();
 $PAGE->stripe_start();
 
-$PAGE->block_start(array ('title'=>'House of Commons - Normal Run'));
+$PAGE->block_start(array('title'=>'House of Commons - Normal Run'));
 ?>
 <ul> <li>Back on 25th May 2005, a bill was introduced: the <a
 href="http://www.publications.parliament.uk/pa/cm200506/cmbills/009/2006009.htm">Identity
@@ -32,7 +32,7 @@ 
 
 <?php
 $PAGE->block_end();
-$PAGE->block_start(array ('title'=>'House of Lords - Normal Run'));
+$PAGE->block_start(array('title'=>'House of Lords - Normal Run'));
 ?>
 
 <ul>
@@ -49,7 +49,7 @@ 
 
 <?php
 $PAGE->block_end();
-$PAGE->block_start(array ('title'=>'Ping Pong'));
+$PAGE->block_start(array('title'=>'Ping Pong'));
 ?>
 
 <ul>
@@ -94,7 +94,7 @@ 
 $PAGE->block_end();
 
     $includes = array(
-        array (
+        array(
             'type' => 'include',
             'content' => 'bills_intro'
         ),

www/docs/divisions/division.php

Indentation +5 added lines, -5 removed lines

@@ -43,11 +43,11 @@
             $data['mp_vote']['with_majority'] = true;
         }
     } else {
-      if ($data['division']['date'] < $MEMBER->entered_house(1)['date']) {
-          $data['before_mp'] = true;
-      } else if ($data['division']['date'] > $MEMBER->left_house(1)['date']) {
-          $data['after_mp'] = true;
-      }
+        if ($data['division']['date'] < $MEMBER->entered_house(1)['date']) {
+            $data['before_mp'] = true;
+        } else if ($data['division']['date'] > $MEMBER->left_house(1)['date']) {
+            $data['after_mp'] = true;
+        }
     }
 }
 

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

@@ -51,7 +51,7 @@
     }
 }
 
-$data['debate_time_human'] = False;
+$data['debate_time_human'] = false;
 $data['debate_day_human'] = format_date($data['division']['date'], LONGDATEFORMAT);
 $data['col_country'] = 'UK';
 $data['location'] = 'in the House of Commons';

Braces +6 added lines, -3 removed lines

@@ -16,7 +16,8 @@ 
 if ($mp = get_http_var('p')) {
     $MEMBER = new MySociety\TheyWorkForYou\Member(array('person_id' => $mp, 'house' => HOUSE_TYPE_COMMONS));
     $main_vote_mp = true;
-} else if ($THEUSER->postcode_is_set()) {
+}
+else if ($THEUSER->postcode_is_set()) {
     $MEMBER = new MySociety\TheyWorkForYou\Member(array('postcode' => $THEUSER->postcode(), 'house' => HOUSE_TYPE_COMMONS));
 }
 
@@ -42,10 +43,12 @@ 
         if ($data['mp_vote']['vote'] == $data['division']['vote']) {
             $data['mp_vote']['with_majority'] = true;
         }
-    } else {
+    }
+    else {
       if ($data['division']['date'] < $MEMBER->entered_house(1)['date']) {
           $data['before_mp'] = true;
-      } else if ($data['division']['date'] > $MEMBER->left_house(1)['date']) {
+      }
+      else if ($data['division']['date'] > $MEMBER->left_house(1)['date']) {
           $data['after_mp'] = true;
       }
     }

www/docs/divisions/index.php

Indentation +5 added lines, -5 removed lines

@@ -10,12 +10,12 @@
 }
 
 if (isset($MEMBER)) {
-  $divisions = new MySociety\TheyWorkForYou\Divisions($MEMBER);
-  $data = array('divisions' => $divisions->getRecentMemberDivisions(30, 'Parliament'));
-  $data['mp_name'] = ucfirst($MEMBER->full_name());
+    $divisions = new MySociety\TheyWorkForYou\Divisions($MEMBER);
+    $data = array('divisions' => $divisions->getRecentMemberDivisions(30, 'Parliament'));
+    $data['mp_name'] = ucfirst($MEMBER->full_name());
 } else {
-  $divisions = new MySociety\TheyWorkForYou\Divisions();
-  $data = $divisions->getRecentDivisions(30);
+    $divisions = new MySociety\TheyWorkForYou\Divisions();
+    $data = $divisions->getRecentDivisions(30);
 }
 
 $data['last_updated'] = MySociety\TheyWorkForYou\Divisions::getMostRecentDivisionDate()['latest'];

Braces +2 added lines, -1 removed lines

@@ -13,7 +13,8 @@
   $divisions = new MySociety\TheyWorkForYou\Divisions($MEMBER);
   $data = array('divisions' => $divisions->getRecentMemberDivisions(30, 'Parliament'));
   $data['mp_name'] = ucfirst($MEMBER->full_name());
-} else {
+}
+else {
   $divisions = new MySociety\TheyWorkForYou\Divisions();
   $data = $divisions->getRecentDivisions(30);
 }

scripts/wikipedia-cron.php

Indentation +33 added lines, -33 removed lines

@@ -16,39 +16,39 @@ 
 contributions('212.35.252.2'); # LibDem
 
 function contributions($ip) {
-	global $dir;
+    global $dir;
 
-	$file = fetch("/w/index.php?title=Special:Contributions&limit=100&target=$ip");
-	if (!$file) return;
-	$fp = fopen($dir . $ip, 'w');
-	fwrite($fp, $file['body']);
-	fclose($fp);
-	preg_match_all('#<li>(.*?) \(<a[^>]*>hist</a>\) \(<a href="(.*?title=(.*?)&.*?oldid=(.*?))"[^>]*>diff</a>\)  <a[^>]*>(.*?)</a> .*?</li>#', $file['body'], $m, PREG_SET_ORDER);
-	foreach ($m as $row) {
-		# print "$row[3] / $row[4]";
-		$filename = html_entity_decode("$row[3].$row[4]");
-		$path = $dir;
-		if (strstr($filename, '/')) {
-			$bits = explode('/', $filename);
-			array_pop($bits);
-			foreach ($bits as $bit) {
-				@mkdir($path . $bit);
-				$path .= "$bit/";
-			}
-		}
-		$cache = $dir . $filename;
-		if (!is_file($cache)) {
-			# print " - fetching";
-			$file = fetch(html_entity_decode($row[2]));
-			if (!$file) continue;
-			$fp = fopen($cache, 'w');
-			if ($fp) {
-				fwrite($fp, $file['body']);
-				fclose($fp);
-			}
-		}
-		# print "\n";
-	}
+    $file = fetch("/w/index.php?title=Special:Contributions&limit=100&target=$ip");
+    if (!$file) return;
+    $fp = fopen($dir . $ip, 'w');
+    fwrite($fp, $file['body']);
+    fclose($fp);
+    preg_match_all('#<li>(.*?) \(<a[^>]*>hist</a>\) \(<a href="(.*?title=(.*?)&.*?oldid=(.*?))"[^>]*>diff</a>\)  <a[^>]*>(.*?)</a> .*?</li>#', $file['body'], $m, PREG_SET_ORDER);
+    foreach ($m as $row) {
+        # print "$row[3] / $row[4]";
+        $filename = html_entity_decode("$row[3].$row[4]");
+        $path = $dir;
+        if (strstr($filename, '/')) {
+            $bits = explode('/', $filename);
+            array_pop($bits);
+            foreach ($bits as $bit) {
+                @mkdir($path . $bit);
+                $path .= "$bit/";
+            }
+        }
+        $cache = $dir . $filename;
+        if (!is_file($cache)) {
+            # print " - fetching";
+            $file = fetch(html_entity_decode($row[2]));
+            if (!$file) continue;
+            $fp = fopen($cache, 'w');
+            if ($fp) {
+                fwrite($fp, $file['body']);
+                fclose($fp);
+            }
+        }
+        # print "\n";
+    }
 }
 
 function fetch($url) {
@@ -56,7 +56,7 @@ 
     $host = 'en.wikipedia.org';
     $fp = @fsockopen($host, 80, $errno, $errstr, 10);
     if (!$fp) {
-	    #print "$errstr ($errno)\n";
+        #print "$errstr ($errno)\n";
             return '';
         }
     $out = "GET $url HTTP/1.1\r\n";

Braces +6 added lines, -2 removed lines

@@ -19,7 +19,9 @@ 
 	global $dir;
 
 	$file = fetch("/w/index.php?title=Special:Contributions&limit=100&target=$ip");
-	if (!$file) return;
+	if (!$file) {
+	    return;
+	}
 	$fp = fopen($dir . $ip, 'w');
 	fwrite($fp, $file['body']);
 	fclose($fp);
@@ -40,7 +42,9 @@ 
 		if (!is_file($cache)) {
 			# print " - fetching";
 			$file = fetch(html_entity_decode($row[2]));
-			if (!$file) continue;
+			if (!$file) {
+			    continue;
+			}
 			$fp = fopen($cache, 'w');
 			if ($fp) {
 				fwrite($fp, $file['body']);

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

@@ -59,7 +59,7 @@
 	    #print "$errstr ($errno)\n";
             return '';
         }
-    $out = "GET $url HTTP/1.1\r\n";
+    $out = "get $url HTTP/1.1\r\n";
     $out .= "Host: $host\r\n";
     $out .= "User-Agent: $ua\r\n";
     $out .= "Accept: text/xml,application/xml,application/xhtml+xml,text/html;q=0.9;text/plain;q=0.8,image/png,*/*;q=0.5\r\n";