mysociety / theyworkforyou

www/docs/mps/multiple.php

Braces +3 added lines, -1 removed lines

@@ -28,7 +28,9 @@
 
 function out($h1, $h2, $title) {
     global $out;
-    if (!isset($out[$h1][$h2])) return;
+    if (!isset($out[$h1][$h2])) {
+        return;
+    }
     print "<h3>$title</h3>\n<ul>\n";
     foreach ($out[$h1][$h2] as $m) {
         print '<li><a href="' . $m->url() . '">' . $m->full_name() . "</a></li>\n";

Spacing +1 added lines, -1 removed lines

@@ -12,7 +12,7 @@
     from member as m1, member as m2
     where m1.person_id = m2.person_id and m1.house != m2.house and m1.house < m2.house
         and m1.left_house='9999-12-31' and m2.left_house='9999-12-31'");
-for ($i=0; $i<$q->rows(); $i++) {
+for ($i = 0; $i < $q->rows(); $i++) {
     $pid = $q->field($i, 'person_id');
     $member = new MEMBER(array('person_id' => $pid));
     $h1 = $q->field($i, 'house1');

www/docs/parliament/index.php

Indentation +1 added lines, -1 removed lines

@@ -6,7 +6,7 @@
 
 $DATA->set_page_metadata($this_page, 'title', 'Parliament');
 $DATA->set_page_metadata($this_page, 'meta_description',
-             'What goes on in Parliament? Up-to-the-minute records on all MPs, debates, speeches and statements from the House of Commons, House of Lords, Scottish Parliament, and Northern Ireland Assembly.');
+                'What goes on in Parliament? Up-to-the-minute records on all MPs, debates, speeches and statements from the House of Commons, House of Lords, Scottish Parliament, and Northern Ireland Assembly.');
 $DATA->set_page_metadata($this_page, 'meta_keywords', 'parliament, uk parliament, parliamentary, house of commons, house commons, house of lords, house lords, house of parliament, parliment, houses parliament, parliament uk, member of parliament, welsh assembly, scottish parliament, the parliament, house of parliment, houses of parliment, parliment uk, uk parliment, houses of parliament, parliament houses, parliament of uk, parliament in uk, the house of parliament, scottish parliment, members of parliament, parliament members, scotish parliament, parliament scottish, the house of commons, british parliament, what is parliament, the house of lords, the scottish parliament, london parliament, parliament london, the houses of parliament, english parliament, northern ireland parliament, the british parliament, northern ireland assembly, history of parliament, parliament history, parliament of england, england parliament');
 
 $PAGE->supress_heading = true;

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;
       }
     }