mysociety / theyworkforyou

www/docs/admin/statistics.php

Spacing +3 added lines, -3 removed lines

@@ -38,10 +38,10 @@
 <p>There are <b><?=number_format($debate_speeches)?></b> debate speeches (<?=number_format($debate_items)?> items including headers).
 <br>There are <b><?=number_format($wrans_questions)?></b> written questions (<?=number_format($wrans_items)?> items including headers and answers).
 
-<!-- Debate items / in-session day: <?=round($debate_items/$uniquedates,0)?>
-Wrans items / in-session day: <?=round($wrans_items/$uniquedates,0)?> -->
+<!-- Debate items / in-session day: <?=round($debate_items / $uniquedates, 0)?>
+Wrans items / in-session day: <?=round($wrans_items / $uniquedates, 0)?> -->
 
-<p>Per sitting day, MPs are producing <b><?=round($debate_speeches/$uniquedates,0)?></b> speeches, and <b><?=round($wrans_questions/$uniquedates,0)?></b> written answers.
+<p>Per sitting day, MPs are producing <b><?=round($debate_speeches / $uniquedates, 0)?></b> speeches, and <b><?=round($wrans_questions / $uniquedates, 0)?></b> written answers.
 </p>
 
 <?php

www/docs/search-hansard/index.php

Indentation +1 added lines, -1 removed lines

@@ -6,7 +6,7 @@
 
 $DATA->set_page_metadata($this_page, 'title', 'Hansard');
 $DATA->set_page_metadata($this_page, 'meta_description',
-             'Find the complete Hansard on TheyWorkForYou.com - transcripts from all UK parliamentary debates in easily-searchable form.');
+                'Find the complete Hansard on TheyWorkForYou.com - transcripts from all UK parliamentary debates in easily-searchable form.');
 $DATA->set_page_metadata($this_page, 'meta_keywords', "hansard, search hansard, hansard transcriptions, hansard record, parliamentary transcripts, parliament transcripts, parliament records, uk hansard, hansard uk, house of commons transcript, house of commons debates, house of commons records");
 
 $PAGE->supress_heading = true;

www/docs/email/index.php

Braces +15 added lines, -7 removed lines

@@ -14,20 +14,25 @@ 
 $sender_email = get_http_var('sender_mail');
 $sender_name = get_http_var('sender_name');
 $pid = intval(get_http_var('pid'));
-if ($pid)
+if ($pid) {
     $MEMBER = new MEMBER(array('person_id' => $pid));
+}
 
 //validate them
 $errors = array();
 
-if (!$pid)
+if (!$pid) {
     $errors[] = 'You did not get to this page from an MP or Peer page. Please go back and try again.';
-if (!preg_match('#^[^@]+@#', $recipient_email))
+}
+if (!preg_match('#^[^@]+@#', $recipient_email)) {
     $errors[] = "'$recipient_email' is not a valid recipient email address. Please have another go.";
-if (!validate_email($sender_email))
+}
+if (!validate_email($sender_email)) {
     $errors[] = "'$sender_email' is not a valid sender email address. Please have another go.";
-if (!$sender_name)
+}
+if (!$sender_name) {
     $errors[] = "If you don't give us your name, we can't tell the recipient who sent them the link. We won't store it or use for any other purpose than sending this email.";
+}
 
 if (sizeof($errors)) {
     print '<p>Please correct the following errors:</p>';
@@ -45,8 +50,11 @@ 
 <?php
 } else {
     $rep_name = $MEMBER->full_name();
-    if ($MEMBER->house_disp==1) $rep_name .= ' MP';
-    elseif ($MEMBER->house_disp==3) $rep_name .= ' MLA';
+    if ($MEMBER->house_disp==1) {
+        $rep_name .= ' MP';
+    } elseif ($MEMBER->house_disp==3) {
+        $rep_name .= ' MLA';
+    }
     $data = array (
         'template'      => 'email_a_friend',
         'to'            => $recipient_email,

Spacing +5 added lines, -5 removed lines

@@ -45,15 +45,15 @@ 
 <?php
 } else {
     $rep_name = $MEMBER->full_name();
-    if ($MEMBER->house_disp==1) $rep_name .= ' MP';
-    elseif ($MEMBER->house_disp==3) $rep_name .= ' MLA';
-    $data = array (
+    if ($MEMBER->house_disp == 1) $rep_name .= ' MP';
+    elseif ($MEMBER->house_disp == 3) $rep_name .= ' MLA';
+    $data = array(
         'template'      => 'email_a_friend',
         'to'            => $recipient_email,
         'subject'       => 'Find out all about ' . $rep_name
     );
     $url = $MEMBER->url(true);
-    $merge = array (
+    $merge = array(
         'NAME' => $sender_name,
         'EMAIL' => $sender_email,
         'REP_NAME' => $rep_name,
@@ -62,7 +62,7 @@ 
 
     $success = send_template_email($data, $merge);
     if ($success) {
-        print "<p>Your email has been sent successfully. Thank you for using TheyWorkForYou.</p> <p><a href=\"$url\">Return to ".$MEMBER->full_name()."'s page</a></p>";
+        print "<p>Your email has been sent successfully. Thank you for using TheyWorkForYou.</p> <p><a href=\"$url\">Return to " . $MEMBER->full_name() . "'s page</a></p>";
     } else {
         print "<p>Sorry, something went wrong trying to send an email. Please wait a few minutes and try again.</p>";
     }

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

www/docs/mps/index.php

Spacing +1 added lines, -1 removed lines

@@ -18,7 +18,7 @@
 $args = $people->getArgs();
 $people->setMetaData($args);
 $data = $people->getData($args);
-if ( isset($args['f']) && $args['f'] == 'csv' ) {
+if (isset($args['f']) && $args['f'] == 'csv') {
     $people->sendAsCSV($data);
 } else {
     MySociety\TheyWorkForYou\Renderer::output("people/index", $data);

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/news/index.php

Braces +21 added lines, -7 removed lines

@@ -9,10 +9,14 @@ 
 if (preg_match("#/(\d\d\d\d)/(\d\d)/(\d\d)/([a-z0-9_]+)(\.php)?$#", $uri, $matches)) {
     // Individual item
     list($all, $year, $month, $day, $ref) = $matches;
-    if ($ref == "please_build_on_") $ref = "new_full_source_";
+    if ($ref == "please_build_on_") {
+        $ref = "new_full_source_";
+    }
     foreach ($all_news as $id => $news_row) {
         list($title, $content, $date) = $news_row;
-        if (news_format_ref($title) != $ref) continue;
+        if (news_format_ref($title) != $ref) {
+            continue;
+        }
 
         $this_page = 'sitenews_individual';
         $DATA->set_page_metadata($this_page, 'title', $title);
@@ -20,7 +24,9 @@ 
         $PAGE->stripe_start();
         print news_format_body($content);
         print "<p>Posted on " . format_date(substr($date, 0, 10), LONGDATEFORMAT) . " at " . substr($date, 11);
-        if (isset($news_row[3])) print " by $news_row[3]";
+        if (isset($news_row[3])) {
+            print " by $news_row[3]";
+        }
         print " | <a href=\"" . news_individual_link($date, $title) . "\">Link to this</a>";
         break;
     }
@@ -37,12 +43,16 @@ 
     $PAGE->stripe_start();
     foreach (array_reverse($all_news) as $id => $news_row) {
         list($title, $content, $date) = $news_row;
-        if (substr($date, 0, 7) != "$year-$month") continue;
+        if (substr($date, 0, 7) != "$year-$month") {
+            continue;
+        }
         print "<h3>" . format_date(substr($date, 0, 10), LONGDATEFORMAT) . "</h3>";
         print "<h4>" . $title . "</h4>";
         print news_format_body($content);
         print "<p>Posted at " . substr($date, 11);
-        if (isset($news_row[3])) print " by $news_row[3]";
+        if (isset($news_row[3])) {
+            print " by $news_row[3]";
+        }
         print " | <a href=\"" . news_individual_link($date, $title) . "\">Link to this</a>";
     }
 } else {
@@ -52,13 +62,17 @@ 
     $PAGE->stripe_start();
     $c = 0;
     foreach ($all_news as $id => $news_row) {
-        if ($c++ == 10) break;
+        if ($c++ == 10) {
+            break;
+        }
         list($title, $content, $date) = $news_row;
         print "<h3>" . format_date(substr($date, 0, 10), LONGDATEFORMAT) . "</h3>";
         print "<h4>" . $title . "</h4>";
         print news_format_body($content);
         print "<p>Posted at " . substr($date, 11);
-        if (isset($news_row[3])) print " by $news_row[3]";
+        if (isset($news_row[3])) {
+            print " by $news_row[3]";
+        }
         print " | <a href=\"" . news_individual_link($date, $title) . "\">Link to this</a>";
     }
 }