mysociety / theyworkforyou

www/includes/easyparliament/templates/html/section/_business_section.php

Spacing +2 added lines, -2 removed lines

@@ -6,7 +6,7 @@ 
         </div>
 
         <?php
-            if ( isset($content['data']['data']) ) {
+            if (isset($content['data']['data'])) {
                 $data = $content['data']['data'];
                 include '_business_list.php'; ?>
                 <div class="business-section__secondary">
@@ -20,7 +20,7 @@ 
                         include '_calendar_section.php';
                         ?>
                     </div>
-                  <?php if ( isset($content['rssurl']) ) { ?>
+                  <?php if (isset($content['rssurl'])) { ?>
                     <div class="business-section__secondary__item">
                         <p class="rss-feed">
                             <a href="<?= WEBPATH . $content['rssurl'] ?>"><?= sprintf(gettext('RSS feed of %s'), $title) ?></a>

www/includes/easyparliament/templates/html/section/_section_toc.php

Spacing +2 added lines, -2 removed lines

@@ -1,7 +1,7 @@ 
 <?php
 
 $divisions_to_link = array();
-foreach($data['rows'] as $speech) {
+foreach ($data['rows'] as $speech) {
 
     # Only care about divisions...
     if ($speech['htype'] != 14) {
@@ -34,7 +34,7 @@ 
                     <li><a href="#g<?= gid_to_anchor($speech['gid']) ?>"><?= sprintf(gettext('Division number %s'), $division['number']) ?></a>
                         <?php if ($division['has_description']) { ?>
                             <br><span class="policy-vote__text">
-                                <?php include( dirname(__FILE__) . '/../divisions/_vote_description.php'); ?>
+                                <?php include(dirname(__FILE__) . '/../divisions/_vote_description.php'); ?>
                             </span>
                         <?php } ?>
 

www/includes/easyparliament/templates/html/section/_business_list_item.php

Spacing +4 added lines, -4 removed lines

@@ -1,6 +1,6 @@ 
 <a href="<?= $item['list_url'] ?>" class="business-list__title">
     <h3>
-      <?php if ( isset( $item['parent'] ) ) { ?>
+      <?php if (isset($item['parent'])) { ?>
         <?= $item['parent']['body'] ?>
         <br><?= $item['body'] ?>
       <?php } else { ?>
@@ -12,11 +12,11 @@ 
       <?=  isset($item['contentcount']) ? '&middot; ' . sprintf(ngettext('%s speech', '%s speeches', $item['contentcount']), $item['contentcount']) : '' ?>
     </span>
 </a>
-<?php if ( isset( $item['child'] ) ) { ?>
+<?php if (isset($item['child'])) { ?>
 <p class="business-list__excerpt">
-  <?php if ( isset($item['child']['speaker']) && count($item['child']['speaker']) > 0 ) { ?>
+  <?php if (isset($item['child']['speaker']) && count($item['child']['speaker']) > 0) { ?>
     <a href="<?= $item['child']['speaker']['url'] ?>"><?= $item['child']['speaker']['name'] ?></a>
   <?php } ?>
-  <?= strip_tags(trim_characters( $item['child']['body'], 0, 200 )) ?>
+  <?= strip_tags(trim_characters($item['child']['body'], 0, 200)) ?>
 </p>
 <?php } ?>

www/includes/easyparliament/templates/html/people/index.php

Indentation +1 added lines, -1 removed lines

@@ -82,7 +82,7 @@
         </div>
 
       <?php } else {
-      $pc_form = ($type == 'mlas' || $type == 'msps' || $type == 'mss');
+        $pc_form = ($type == 'mlas' || $type == 'msps' || $type == 'mss');
 ?>
 
         <form action="/<?= $pc_form ? 'postcode' : 'search' ?>/">

Spacing +15 added lines, -15 removed lines

@@ -28,7 +28,7 @@ 
                           <?= sprintf(gettext('Based on postcode <strong>%s</strong>'), $mp_data['postcode']) ?>
                           <a href="<?= $mp_data['change_url'] ?>"><?= gettext('(Change postcode)') ?></a>
                       </p>
-                    <?php if ( isset( $mp_data ) && $type != 'mlas' ) { ?>
+                    <?php if (isset($mp_data) && $type != 'mlas') { ?>
                       <h3><?php
                         if ($mp_data['former']) {
                             printf(gettext('Your former %s is'), $rep_name);
@@ -46,8 +46,8 @@ 
                         </p>
                     </a>
                     <?php }
-                    if ( isset($reps) ) {
-                        if ( isset($mp_data) && $type != 'mlas' ) { ?>
+                    if (isset($reps)) {
+                        if (isset($mp_data) && $type != 'mlas') { ?>
                     <div class="people-list__your-mp__replist-header">
                         <?php } ?>
                       <h3><?php
@@ -66,7 +66,7 @@ 
                         }
                         ?></h3>
                     </div>
-                        <?php foreach ( $reps as $rep ) { ?>
+                        <?php foreach ($reps as $rep) { ?>
                     <a href="<?= $rep['mp_url'] ?>" class="people-list__person">
                     <img class="people-list__person__image" src="<?= $rep['image'] ?>">
                         <h2 class="people-list__person__name"><?= $rep['name'] ?></h2>
@@ -89,7 +89,7 @@ 
             <div class="search-page__section search-page__section--search">
                 <div class="search-page__section__primary">
                     <p class="search-page-main-inputs">
-                    <?php if ( $type == 'peers' ) { ?>
+                    <?php if ($type == 'peers') { ?>
                         <label for="find-mp-by-name-or-postcode"><?= sprintf(gettext('Find %s by name:'), $rep_plural) ?></label>
                     <?php } elseif ($pc_form) { ?>
                         <label for="find-mp-by-name-or-postcode"><?= sprintf(gettext('Find your %s by postcode:'), $rep_name) ?></label>
@@ -137,12 +137,12 @@ 
             <div class="search-page__section__primary">
             <h2><?= sprintf(gettext('All %s'), $rep_plural) ?></h2>
 
-                <?php if ( $type != 'peers' ) { ?>
+                <?php if ($type != 'peers') { ?>
                 <ul class="search-result-display-options">
-                    <?php if ( $order == 'given_name' ) { ?>
+                    <?php if ($order == 'given_name') { ?>
                     <li><?= gettext('<strong>Sorted by</strong> First name') ?></li>
                     <li><?= gettext('Sort by') ?> <a href="<?= $urls['by_last'] ?>"><?= gettext('Last name') ?></a> / <a href="<?= $urls['by_party'] ?>"><?= gettext('Party') ?></a></li>
-                    <?php } else if ( $order == 'party' ) { ?>
+                    <?php } else if ($order == 'party') { ?>
                     <li><?= gettext('<strong>Sorted by</strong> Party') ?></li>
                     <li><?= gettext('Sort by') ?> <a href="<?= $urls['by_first'] ?>"><?= gettext('First name') ?></a> / <a href="<?= $urls['by_last'] ?>"><?= gettext('Last name') ?></a></li>
                     <?php } else { ?>
@@ -152,7 +152,7 @@ 
                 </ul>
                 <?php } else { ?>
                 <ul class="search-result-display-options">
-                    <?php if ( $order == 'party' ) { ?>
+                    <?php if ($order == 'party') { ?>
                     <li><?= gettext('<strong>Sorted by</strong> Party') ?></li>
                     <li>Sort by <a href="<?= $urls['by_name'] ?>"><?= gettext('Name') ?></a></li>
                     <?php } else { ?>
@@ -202,10 +202,10 @@ 
                     }
                 }
                 $initial_link = '';
-                foreach ( $data as $person ) {
+                foreach ($data as $person) {
                     if ($order != 'party') {
-                        $initial = substr( strtoupper($person[$a_to_z_key]), 0, 1);
-                        if ( $initial != $current_initial ) {
+                        $initial = substr(strtoupper($person[$a_to_z_key]), 0, 1);
+                        if ($initial != $current_initial) {
                             $current_initial = $initial;
                             $initial_link = "name=\"$initial\" ";
                         } else {
@@ -219,7 +219,7 @@ 
                 </noscript>
                         <h2 class="people-list__person__name"><?= ucfirst($person['name']) ?></h2>
                         <p class="people-list__person__memberships">
-                        <?php if ( $person['constituency'] ) { ?>
+                        <?php if ($person['constituency']) { ?>
                         <span class="people-list__person__constituency"><?= $person['constituency'] ?></span>
                         <?php } ?>
                         <span class="people-list__person__party <?= slugify($person['party']) ?>"><?= $person['party'] ?></span>
@@ -237,7 +237,7 @@ 
                 <a href="<?= $urls['by_csv'] ?>"><?= gettext('Download this list as a CSV') ?></a>
                     <?= gettext('suitable for Excel') ?>
                 </p>
-                <?php if ( $type == 'mps' ) { ?>
+                <?php if ($type == 'mps') { ?>
                 <style>
                 .js #past-list-dates { display: none; }
                 </style>
@@ -282,7 +282,7 @@ 
                     </p>
                 </form>
 
-                <?php } else if ( $type == 'msps' ) { ?>
+                <?php } else if ($type == 'msps') { ?>
                     <p class="past-list-dates" id="past-list-dates">
                         <a href="<?= $urls['plain'] ?>?date=2011-05-05">MSPs at 2011 election</a>
                         <a href="<?= $urls['plain'] ?>?date=2007-05-03">MSPs at 2007 election</a>

www/includes/easyparliament/templates/html/senedd/index.php

Indentation +1 added lines, -1 removed lines

@@ -41,7 +41,7 @@
                 <div class="row nested-row">
                     <div class="homepage-in-the-news homepage-content-section">
                         <?php if ( $featured ) {
-                             include dirname(__FILE__) . "/../homepage/featured.php";
+                                include dirname(__FILE__) . "/../homepage/featured.php";
                         } ?>
                     </div>
                     <div class="homepage-create-alert homepage-content-section">

Spacing +4 added lines, -4 removed lines

@@ -4,10 +4,10 @@ 
             <div class="hero__mp-search__wrap">
                 <h1><?= gettext('Do your MSs represent you?') ?></h1>
                 <div class="row collapse">
-                    <?php if ( count($data['regional']) > 0 ) { ?>
+                    <?php if (count($data['regional']) > 0) { ?>
                         <ul class="homepage-rep-list">
                             <li><?= gettext('Your MSs:') ?></li>
-                        <?php foreach ( $data['regional'] as $ms ) { ?>
+                        <?php foreach ($data['regional'] as $ms) { ?>
                             <li class="homepage-rep-list__rep"><a href="/ms/?p=<?= $ms['person_id'] ?>"><?= $ms['name'] ?></a></li>
                         <?php } ?>
                         </ul>
@@ -40,7 +40,7 @@ 
             <div class="panel panel--flushtop clearfix">
                 <div class="row nested-row">
                     <div class="homepage-in-the-news homepage-content-section">
-                        <?php if ( $featured ) {
+                        <?php if ($featured) {
                              include dirname(__FILE__) . "/../homepage/featured.php";
                         } ?>
                     </div>
@@ -85,7 +85,7 @@ 
                     <div class="homepage-recently homepage-content-section">
                         <h2><?= gettext('Recently in the Welsh Parliament') ?></h2>
                         <ul class="recently__list"><?php
-                            foreach ( $debates['recent'] as $recent ) {
+                            foreach ($debates['recent'] as $recent) {
                                 include dirname(__FILE__) . '/../homepage/recent-debates.php';
                             }
                         ?></ul>

www/includes/easyparliament/templates/html/user/view_user.php

Spacing +1 added lines, -1 removed lines

@@ -8,7 +8,7 @@
 
                 <div class="row">
                 <span class="label"><?= gettext('Website') ?></span>
-                <span class="formw"><?= $website == '' ? 'none' : '<a rel="nofollow" href="' . _htmlentities($website) . '">'. _htmlentities($website) . '</a>' ?></span>
+                <span class="formw"><?= $website == '' ? 'none' : '<a rel="nofollow" href="' . _htmlentities($website) . '">' . _htmlentities($website) . '</a>' ?></span>
                 </div>
 
                 <div class="row">

www/includes/easyparliament/templates/html/user/index.php

Spacing +1 added lines, -1 removed lines

@@ -37,7 +37,7 @@
 
           <div class="row">
             <span class="label"><?= gettext('Website') ?></span>
-            <span class="formw"><?= $website == '' ? 'none' : '<a rel="nofollow" href="' . _htmlentities($website) . '">'. _htmlentities($website) . '</a>' ?></span>
+            <span class="formw"><?= $website == '' ? 'none' : '<a rel="nofollow" href="' . _htmlentities($website) . '">' . _htmlentities($website) . '</a>' ?></span>
           </div>
 
           <?php if ($facebook_user) { ?>

www/includes/easyparliament/templates/html/mp/profile.php

Indentation +5 added lines, -5 removed lines

@@ -83,7 +83,7 @@ 
 
                             include '_vote_description.php';
 
-                          } ?>
+                            } ?>
                         </ul>
 
                         <p>We have <b>lots more</b> plain English analysis of <?= $full_name ?>&rsquo;s voting record  on issues like health, welfare, taxation and more. Visit <a href="<?= $member_url ?>/votes"><?= $full_name ?>&rsquo;s full vote analysis page</a> for more.</p>
@@ -131,7 +131,7 @@ 
 
                             include '_vote_description.php';
 
-                          } ?>
+                            } ?>
                         </ul>
 
                         <p>We have <b>lots more</b> plain English analysis of <?= $full_name ?>&rsquo;s voting record on issues like health, welfare, taxation and more. Visit <a href="<?= $member_url ?>/votes"><?= $full_name ?>&rsquo;s full vote analysis page</a> for more.</p>
@@ -171,7 +171,7 @@ 
 
                             include '_vote_description.php';
 
-                          } ?>
+                            } ?>
                         </ul>
 
                         <p class="voting-information-provenance">
@@ -413,7 +413,7 @@ 
                         printf(gettext('Voting information from <a href="%s">Public Whip</a>.'), "https://www.publicwhip.org.uk/mp.php?id=uk.org.publicwhip/member/ $member_id&amp;showall=yes"); ?></p>
 
                   <?php if($image && $image['exists']) {
-                      if(isset($data['photo_attribution_text'])) { ?>
+                        if(isset($data['photo_attribution_text'])) { ?>
                     <p>
                       <?php if(isset($data['photo_attribution_link'])) { ?>
                         <?= gettext('Profile photo:') ?>
@@ -423,7 +423,7 @@ 
                       <?php } ?>
                     </p>
                   <?php }
-                  } else { ?>
+                    } else { ?>
                     <p>
                         We&rsquo;re missing a photo of <?= $full_name ?>.
                         If you have a photo <em>that you can release under

Braces +5 added lines, -2 removed lines

@@ -217,11 +217,14 @@
                         <?= $recent_appearances['additional_links'] ?>
                         <?php endif; ?>
 
-                    <?php else: ?>
+                    <?php else {
+    : ?>
 
                         <p><?=gettext('No recent appearances to display.') ?></p>
 
-                    <?php endif; ?>
+                    <?php endif;
+}
+?>
 
                 </div>
                 <?php endif; ?>

Spacing +9 added lines, -9 removed lines

@@ -63,7 +63,7 @@ 
                 </div>
               <?php endif; ?>
 
-                <?php if ( !$current_member[HOUSE_TYPE_COMMONS] ) { ?>
+                <?php if (!$current_member[HOUSE_TYPE_COMMONS]) { ?>
                     <?php if (count($policyPositions->positions) > 0) { ?>
                     <div class="panel">
                         <a name="votes"></a>
@@ -116,7 +116,7 @@ 
                             $policy_direction = $key_vote["person_position"];
                             $policy_group = "highlighted";
                             $party_score_difference = $key_vote["score_difference"];
-                            $party_position = $key_vote['party_position'] ;
+                            $party_position = $key_vote['party_position'];
                             $comparison_party = $data["comparison_party"];
                             $current_party_comparison = $data["current_party_comparison"];
                             $party_voting_line = sprintf("%s, %s", $party, $diff['party_voting_summary']);
@@ -138,9 +138,9 @@ 
 
                         <p>We have <b>lots more</b> plain English analysis of <?= $full_name ?>&rsquo;s voting record on issues like health, welfare, taxation and more. Visit <a href="<?= $member_url ?>/votes"><?= $full_name ?>&rsquo;s full vote analysis page</a> for more.</p>
 
-                    <?php elseif (count($policyPositions->positions) > 0 ): ?>
+                    <?php elseif (count($policyPositions->positions) > 0): ?>
                         <?php if (count($party_positions) && $party_member_count > 1) { ?>
-                            <?php if ($current_party_comparison <> $comparison_party){ ?> 
+                            <?php if ($current_party_comparison <> $comparison_party) { ?> 
                                 <p>
                                     <?= $full_name ?> is a <?= $party ?> MP, but has changed parties or become independent.
                                 </p>
@@ -164,7 +164,7 @@ 
                           <?php foreach ($policyPositions->positions as $key_vote) {
                             
                             $policy_id = $key_vote['policy_id'];
-                            $covid_affected = in_array($policy_id, $covid_policy_list);                            $description = ucfirst($key_vote['desc']);
+                            $covid_affected = in_array($policy_id, $covid_policy_list); $description = ucfirst($key_vote['desc']);
                             $link = sprintf(
                                 '%s/divisions?policy=%s',
                                 $member_url,
@@ -183,7 +183,7 @@ 
                         </p>
 
                         <p>We have <b>lots more</b> plain English analysis of <?= $full_name ?>&rsquo;s voting record  on issues like health, welfare, taxation and more. Visit <a href="<?= $member_url ?>/votes"><?= $full_name ?>&rsquo;s full vote analysis page</a> for more.</p>
-                    <?php elseif (count($policyPositions->positions) == 0 ): ?>
+                    <?php elseif (count($policyPositions->positions) == 0): ?>
 
                         <p>No votes to display.</p>
 
@@ -415,10 +415,10 @@ 
                     <p><?php print gettext('This data was produced by TheyWorkForYou from a variety of sources.') . ' ';
                         printf(gettext('Voting information from <a href="%s">Public Whip</a>.'), "https://www.publicwhip.org.uk/mp.php?id=uk.org.publicwhip/member/ $member_id&amp;showall=yes"); ?></p>
 
-                  <?php if($image && $image['exists']) {
-                      if(isset($data['photo_attribution_text'])) { ?>
+                  <?php if ($image && $image['exists']) {
+                      if (isset($data['photo_attribution_text'])) { ?>
                     <p>
-                      <?php if(isset($data['photo_attribution_link'])) { ?>
+                      <?php if (isset($data['photo_attribution_link'])) { ?>
                         <?= gettext('Profile photo:') ?>
                         <a href="<?= $data['photo_attribution_link'] ?>"><?= $data['photo_attribution_text'] ?></a>
                       <?php } else { ?>

www/includes/easyparliament/templates/html/mp/header.php

Indentation +3 added lines, -3 removed lines

@@ -21,7 +21,7 @@ 
                 <?php
                     $lat = $latest_membership['party'] == 'Bishop' ? '' : $latest_membership['party'];
                     if ($latest_membership['current']) {
-                          if ($latest_membership['constituency']) {
+                            if ($latest_membership['constituency']) {
                             # First %s is party, second is type of rep (e.g. MS), third is constituency name
                             printf(gettext('<span class="person-header__about__position__role">%s %s</span> for <span class="person-header__about__position__constituency">%s</span>'),
                                 $lat, $latest_membership['rep_name'], $latest_membership['constituency']);
@@ -30,7 +30,7 @@ 
                                 $lat, $latest_membership['rep_name']);
                         }
                     } else {
-                          if ($latest_membership['constituency']) {
+                            if ($latest_membership['constituency']) {
                             printf(gettext('<span class="person-header__about__position__role">Former %s %s</span> for <span class="person-header__about__position__constituency">%s</span>'),
                                 $lat, $latest_membership['rep_name'], $latest_membership['constituency']);
                         } else {
@@ -69,7 +69,7 @@ 
                 } else if ($the_users_mp) {
                     $wtt_url = $wtt_url . "?a=WMC&amp;pc=" . _htmlentities(urlencode($user_postcode));
                 }
-              ?>
+                ?>
                 <a href="<?= $wtt_url ?>" class="button" onclick="trackLinkClick(this, 'Links', 'WriteToThem', 'Person'); return false;"><?= gettext('Send a message') ?></a>
               <?php } ?>
               <?php if ($has_email_alerts) { ?>

Spacing +5 added lines, -5 removed lines

@@ -1,9 +1,9 @@ 
 <div class="person-header <?= $current_member_anywhere ? '' : 'person-header--historical'; ?>">
     <div class="full-page__row">
         <div class="full-page__unit">
-          <?php if ( $image ) { ?>
+          <?php if ($image) { ?>
             <div class="person-header__image <?= $image['size'] == 'S' ? 'person-header__image--small' : '' ?>">
-              <?php if ( $image['size'] == 'S' ) { ?>
+              <?php if ($image['size'] == 'S') { ?>
                 <span style="background-image: url('<?= $image['url'] ?>');"></span>
               <?php } ?>
                 <img src="<?= $image['url'] ?>">
@@ -11,12 +11,12 @@ 
           <?php } ?>
             <div class="person-header__about">
                 <h1 class="person-header__about__name"><?= ucfirst($full_name) ?></h1>
-              <?php if ( $known_for ) { ?>
+              <?php if ($known_for) { ?>
                 <p class="person-header__about__known-for">
                     <?= $known_for ?>
                 </p>
               <?php } ?>
-              <?php if ( $latest_membership && $latest_membership['house'] != HOUSE_TYPE_ROYAL ) { ?>
+              <?php if ($latest_membership && $latest_membership['house'] != HOUSE_TYPE_ROYAL) { ?>
                 <p class="person-header__about__position">
                 <?php
                     $lat = $latest_membership['party'] == 'Bishop' ? '' : $latest_membership['party'];
@@ -43,7 +43,7 @@ 
               <?php } ?>
               <?php if (count($social_links) > 0) { ?>
                 <p class="person-header__about__media">
-                  <?php foreach ($social_links as $link){ ?>
+                  <?php foreach ($social_links as $link) { ?>
                       <a href="<?= $link['href'] ?>" onclick="trackLinkClick(this, 'Social-Link', '<?= $link['type'] ?>', '<?= $link['text'] ?>'); return false;"><?= $link['text'] ?></a>
                   <?php } ?>
                 </p>