mysociety / theyworkforyou

www/includes/easyparliament/templates/html/divisions/_your_mp.php

Indentation +16 added lines, -16 removed lines

@@ -11,29 +11,29 @@
     <a href="<?= $data['mp_data']['mp_url'] ?>" class="your-mp__content">
       <?php if (isset($mp_vote)) { ?>
         <span class="your-mp__vote your-mp__vote--<?= $mp_vote['vote'] ?>"><?php
-          switch ($mp_vote['vote']) {
-              case 'aye':
+            switch ($mp_vote['vote']) {
+                case 'aye':
                   echo 'Aye';
-                  break;
-              case 'no':
+                    break;
+                case 'no':
                   echo 'No';
-                  break;
-              case 'absent':
+                    break;
+                case 'absent':
                   echo 'Absent';
-                  break;
-              case 'both':
+                    break;
+                case 'both':
                   echo 'Abstain';
-                  break;
-              case 'tellaye':
+                    break;
+                case 'tellaye':
                   echo 'Aye (Teller)';
-                  break;
-              case 'tellno':
+                    break;
+                case 'tellno':
                   echo 'No (Teller)';
-                  break;
-              default:
+                    break;
+                default:
                   echo 'N/A';
-          }
-          ?></span>
+            }
+            ?></span>
       <?php } elseif (isset($before_mp) || isset($after_mp)) { ?>
         <span class="your-mp__vote">N/A</span>
       <?php } ?>

Spacing +1 added lines, -1 removed lines

@@ -1,5 +1,5 @@
 <div class="debate-speech__division__your-mp">
-  <?php if (!isset($main_vote_mp) || ! $main_vote_mp) { /* $main_vote_mp is true if an MP has been requested via the URL */ ?>
+  <?php if (!isset($main_vote_mp) || !$main_vote_mp) { /* $main_vote_mp is true if an MP has been requested via the URL */ ?>
     <div class="your-mp__header">
       <h3>How your <?= $division['members']['singular'] ?> voted</h3>
       <p>

Braces +2 added lines, -1 removed lines

@@ -1,5 +1,6 @@
 <div class="debate-speech__division__your-mp">
-  <?php if (!isset($main_vote_mp) || ! $main_vote_mp) { /* $main_vote_mp is true if an MP has been requested via the URL */ ?>
+  <?php if (!isset($main_vote_mp) || ! $main_vote_mp) {
+/* $main_vote_mp is true if an MP has been requested via the URL */ ?>
     <div class="your-mp__header">
       <h3>How your <?= $division['members']['singular'] ?> voted</h3>
       <p>

www/includes/easyparliament/templates/html/topic/topic.php

Braces +3 added lines, -1 removed lines

@@ -58,10 +58,12 @@
                     to <a href="<?= $member_url ?>/votes">see all their votes</a>.
                 </p>
 
-              <?php else: ?>
+              <?php else {
+    : ?>
                 <ul class="vote-descriptions">
                   <?php
                   $policy_ids = [];
+}
 
                   foreach ($positions as $position) {
                       if (!in_array($position['policy_id'], $policy_ids)) {

Indentation +17 added lines, -17 removed lines

@@ -61,23 +61,23 @@
               <?php else: ?>
                 <ul class="vote-descriptions">
                   <?php
-                  $policy_ids = [];
-
-                  foreach ($positions as $position) {
-                      if (!in_array($position['policy_id'], $policy_ids)) {
-                          $description = ucfirst($position['desc']);
-                          $link = sprintf(
-                              '%s/divisions?policy=%s',
-                              $member_url,
-                              $position['policy_id']
-                          );
-                          $link_text = $position['position'] != 'has never voted on' ? 'Show votes' : 'Details';
-                          $key_vote = $position;
-                          include(dirname(__DIR__) . '/mp/_vote_description.php');
-
-                          $policy_ids[] = $position['policy_id'];
-                      }
-                  } ?>
+                    $policy_ids = [];
+
+                    foreach ($positions as $position) {
+                        if (!in_array($position['policy_id'], $policy_ids)) {
+                            $description = ucfirst($position['desc']);
+                            $link = sprintf(
+                                '%s/divisions?policy=%s',
+                                $member_url,
+                                $position['policy_id']
+                            );
+                            $link_text = $position['position'] != 'has never voted on' ? 'Show votes' : 'Details';
+                            $key_vote = $position;
+                            include(dirname(__DIR__) . '/mp/_vote_description.php');
+
+                            $policy_ids[] = $position['policy_id'];
+                        }
+                    } ?>
                 </ul>
 
               <?php endif; ?>

www/includes/easyparliament/templates/api/comments.php

Spacing +1 added lines, -1 removed lines

@@ -82,7 +82,7 @@
         // COMMENT REPORTING LINK.
 
         if (($this_page != 'commentreport' &&
-            $this_page != 'addcomment'  &&
+            $this_page != 'addcomment' &&
             $this_page != 'admin_commentreport')
             && $THEUSER->is_able_to('reportcomment')
             && $THEUSER->user_id() != $comment['user_id']

www/includes/utility.php

Braces +6 added lines, -3 removed lines

@@ -319,7 +319,8 @@ 
 
     if (preg_match("/^(\d\d\d\d)-(\d\d?)-(\d\d?)$/", $date, $matches)) {
         [$string, $year, $month, $day] = $matches;
-        if ($year < 1902) { # gmdate fns only go back to Dec. 1901
+        if ($year < 1902) {
+# gmdate fns only go back to Dec. 1901
             if ($format == SHORTDATEFORMAT) {
                 return ($day + 0) . ' ' . $format_date_months_short[$month + 0] . " $year";
             } else {
@@ -1046,7 +1047,8 @@ 
         'Lords in Waiting (HM Household)' => 'Government Whip',
         'Baronesses in Waiting, HM Household' => 'Government Whip',
     ];
-    if ($pos) { # Government post, or Chairman of Select Committee
+    if ($pos) {
+# Government post, or Chairman of Select Committee
         $pretty = $pos;
         if ($dept && $dept != 'No Department') {
             $pretty .= ", $dept";
@@ -1054,7 +1056,8 @@ 
         if (array_key_exists($pretty, $lookup)) {
             $pretty = $lookup[$pretty];
         }
-    } else { # Member of Select Committee
+    } else {
+# Member of Select Committee
         $pretty = "Member, $dept";
     }
     return $pretty;

Spacing +15 added lines, -15 removed lines

@@ -29,7 +29,7 @@ 
     if ($debug_level != '') {
 
         // Set which level shows which types of debug info.
-        $levels =  [
+        $levels = [
             1 =>  ['THEUSER', 'TIME', 'SQLERROR', 'PAGE', 'TEMPLATE', 'SEARCH', 'ALERTS', 'MP'],
             2 =>  ['SQL', 'EMAIL', 'WIKIPEDIA', 'hansardlist', 'debatelist', 'wranslist', 'whalllist'],
             3 =>  ['SQLRESULT'],
@@ -78,7 +78,7 @@ 
     // consider are E_WARNING, E_NOTICE, E_USER_ERROR,
     // E_USER_WARNING and E_USER_NOTICE
     # Commented out are ones that a user function cannot handle.
-    $errortype =  [
+    $errortype = [
         #E_ERROR            => "Error",
         E_WARNING           => "Warning",
         #E_PARSE            => "Parsing Error",
@@ -355,14 +355,14 @@ 
     $in_seconds = strtotime($datetime);
     $now = time();
 
-    $diff   =  $now - $in_seconds;
-    $months =  floor($diff / 2419200);
+    $diff   = $now - $in_seconds;
+    $months = floor($diff / 2419200);
     $diff   -= $months * 2419200;
-    $weeks  =  floor($diff / 604800);
+    $weeks  = floor($diff / 604800);
     $diff   -= $weeks * 604800;
-    $days   =  floor($diff / 86400);
+    $days   = floor($diff / 86400);
     $diff   -= $days * 86400;
-    $hours  =  floor($diff / 3600);
+    $hours  = floor($diff / 3600);
     $diff   -= $hours * 3600;
     $minutes = floor($diff / 60);
     $diff   -= $minutes * 60;
@@ -497,11 +497,11 @@ 
 
     if ($filter_type == 'strict') {
         // No tags allowed at all!
-        $filter->allowed =  [];
+        $filter->allowed = [];
     } else {
         // Comment.
         // Only allowing <a href>, <b>, <strong>, <i> and <em>
-        $filter->allowed =  [
+        $filter->allowed = [
             'a' => ['href'],
             'strong' => [],
             'em' => [],
@@ -528,7 +528,7 @@ 
     $text = preg_replace('(href="[^"]*")', '$0 rel="nofollow"', $text);
     $text = preg_replace_callback(
         '/(rel="nofollow">)(http[^<]{60,})</',
-        function ($matches) {
+        function($matches) {
             return $matches[1] . substr($matches[2], 0, 60) . '...<';
         },
         $text
@@ -891,7 +891,7 @@ 
 function entities_to_numbers($string) {
     $string = str_replace(
         ['&Ouml;', '&acirc;', '&uacute;', '&aacute;', '&iacute;', '&ocirc;', '&eacute;'],
-        ['&#214;', '&#226;',  '&#250;',   '&#225;',   '&#237;',   '&#244;',  '&#233;'  ],
+        ['&#214;', '&#226;', '&#250;', '&#225;', '&#237;', '&#244;', '&#233;'],
         $string
     );
     return $string;
@@ -908,9 +908,9 @@ 
         }
     }
 
-    $s   = [' ', '&amp;', '&ocirc;',  '&Ouml;',  '&ouml;',   '&acirc;',  '&iacute;', '&aacute;', '&uacute;', '&eacute;', '&oacute;', '&Oacute;'];
-    $s2  = [" ", "&",     "\xc3\xb4", "\xc3\96", "\xc3\xb6", "\xc3\xa5", "\xc3\xad", "\xc3\xa1", "\xc3\xba", "\xc3\xa9", "\xc3\xb3", "\xc3\x93"];
-    $r   = ['_', 'and',   'o',        'o',       'o',        'a',        'i',        'a',        'u',        'e',        'o',        'o'];
+    $s   = [' ', '&amp;', '&ocirc;', '&Ouml;', '&ouml;', '&acirc;', '&iacute;', '&aacute;', '&uacute;', '&eacute;', '&oacute;', '&Oacute;'];
+    $s2  = [" ", "&", "\xc3\xb4", "\xc3\96", "\xc3\xb6", "\xc3\xa5", "\xc3\xad", "\xc3\xa1", "\xc3\xba", "\xc3\xa9", "\xc3\xb3", "\xc3\x93"];
+    $r   = ['_', 'and', 'o', 'o', 'o', 'a', 'i', 'a', 'u', 'e', 'o', 'o'];
     $name = preg_replace('#^the #', '', strtolower($name));
 
     $out = '';
@@ -1089,7 +1089,7 @@ 
                 }
                 $out .= '<li>' . $body . '<ul>';
             } else {
-                $LISTURL->insert([ 'id' => $gid ]);
+                $LISTURL->insert(['id' => $gid]);
                 $out .= '<li><a href="' . $LISTURL->generate() . '">';
                 $out .= $body . '</a>';
             }

www/docs/admin/featured.php

Spacing +2 added lines, -2 removed lines

@@ -30,7 +30,7 @@ 
     global $hansardmajors;
     global $db;
 
-    $q = $db->query("SELECT major FROM hansard WHERE gid = :gid", [ ':gid' => $gid ])->first();
+    $q = $db->query("SELECT major FROM hansard WHERE gid = :gid", [':gid' => $gid])->first();
     $url_gid = fix_gid_from_db($gid);
     $url = new \MySociety\TheyWorkForYou\Url($hansardmajors[$q['major']]['page']);
     $url->insert(['id' => $url_gid]);
@@ -118,7 +118,7 @@ 
     print "<h2>Preview Content</h2>";
     if ($gid !== null) {
         $h = new MySociety\TheyWorkForYou\Homepage();
-        $featured = $h->getFeaturedDebate($gid, $title, $context, [ $related_gid1, $related_gid2, $related_gid3 ]);
+        $featured = $h->getFeaturedDebate($gid, $title, $context, [$related_gid1, $related_gid2, $related_gid3]);
 
         include INCLUDESPATH . 'easyparliament/templates/html/homepage/featured.php';
     } else {

www/docs/admin/policies.php

Spacing +2 added lines, -2 removed lines

@@ -128,9 +128,9 @@
 
     $rettext .= '<div id="subnav_websites">';
     foreach ($subnav as $label => $path) {
-        $rettext .=  '<a href="' . $path . '">' . $label . '</a>';
+        $rettext .= '<a href="' . $path . '">' . $label . '</a>';
     }
-    $rettext .=  '</div>';
+    $rettext .= '</div>';
 
     return $rettext;
 }

www/docs/admin/alert_stats.php

Spacing +12 added lines, -12 removed lines

@@ -18,13 +18,13 @@ 
 
 print '<h3>Headline stats</h3> <table>';
 $data = [
-    'header' => [ 'Alert Type', 'Count' ],
+    'header' => ['Alert Type', 'Count'],
     'rows' => [
-        [ 'Confirmed', $q_confirmed ],
-        [ 'Not Confirmed', $q_not_confirmed ],
-        [ 'Deleted', $q_deleted ],
-        [ 'For a Speaker<sup>*</sup>', $q_speaker ],
-        [ 'For a Keyword', $q_keyword ],
+        ['Confirmed', $q_confirmed],
+        ['Not Confirmed', $q_not_confirmed],
+        ['Deleted', $q_deleted],
+        ['For a Speaker<sup>*</sup>', $q_speaker],
+        ['For a Keyword', $q_keyword],
     ],
 ];
 
@@ -40,7 +40,7 @@ 
     $criteria = $row['criteria'];
     $rows[] = [$id, $critera];
 }
-$data = [ 'rows' => $rows ];
+$data = ['rows' => $rows];
 $PAGE->display_table($data);
 
 $q = $db->query('select count(*) as c, criteria from alerts where criteria like "speaker:%" and confirmed and not deleted group by criteria order by c desc');
@@ -79,15 +79,15 @@ 
     }
 }
 
-$people_header = [ 'Name', 'Confirmed', 'Unconfirmed'];
+$people_header = ['Name', 'Confirmed', 'Unconfirmed'];
 print '<h3>Alert signups by MP/Peer</h3>';
 $rows = [];
 foreach ($tots as $person_id => $c) {
     $u = $unconfirmed[$person_id] ?? 0;
-    $rows[] = [ $name[$person_id], $c, $u ];
+    $rows[] = [$name[$person_id], $c, $u];
 }
 $data = [
-    'header' => [ 'Name', 'Confirmed', 'Unconfirmed'],
+    'header' => ['Name', 'Confirmed', 'Unconfirmed'],
     'rows' => $rows,
 ];
 $PAGE->display_table($data);
@@ -114,10 +114,10 @@ 
 foreach ($total as $criteria => $tot) {
     $c = $confirmed[$criteria] ?? 0;
     $u = $unconfirmed[$criteria] ?? 0;
-    $rows[] = [ $criteria, $c, $u ];
+    $rows[] = [$criteria, $c, $u];
 }
 $data = [
-    'header' => [ 'Criteria', 'Confirmed', 'Unconfirmed'],
+    'header' => ['Criteria', 'Confirmed', 'Unconfirmed'],
     'rows' => $rows,
 ];
 $PAGE->display_table($data);

www/docs/admin/edittopic.php

Indentation +1 added lines, -1 removed lines

@@ -125,7 +125,7 @@
             <select name="sets[]" multiple>
               <option value="">None</option>
             <?php
-              $policies = new \MySociety\TheyWorkForYou\Policies();
+                $policies = new \MySociety\TheyWorkForYou\Policies();
 $set_descriptions = $policies->getSetDescriptions();
 $related_sets = $topic->getPolicySets();
 foreach ($set_descriptions as $set => $description) { ?>

www/docs/admin/websites.php

Spacing +3 added lines, -3 removed lines

@@ -106,7 +106,7 @@ 
     $sysretval = 0;
     $personid = get_http_var('editperson');
 
-    $q  = $db->query("DELETE FROM personinfo WHERE data_key = 'mp_website' AND personinfo.person_id = :person_id", [
+    $q = $db->query("DELETE FROM personinfo WHERE data_key = 'mp_website' AND personinfo.person_id = :person_id", [
         ':person_id' => $personid,
     ]);
 
@@ -142,9 +142,9 @@ 
 
     $rettext .= '<div id="subnav_websites">';
     foreach ($subnav as $label => $path) {
-        $rettext .=  '<a href="' . $path . '">' . $label . '</a>';
+        $rettext .= '<a href="' . $path . '">' . $label . '</a>';
     }
-    $rettext .=  '</div>';
+    $rettext .= '</div>';
 
     return $rettext;
 }