mysociety / theyworkforyou

www/docs/addcomment/index.php

Spacing +1 added lines, -1 removed lines

@@ -66,7 +66,7 @@
             'posted' => date('Y-m-d H:i:s', time()),
             'modflagged' => null,
             'visible' => 1,
-            'preview' => true,	// Extra tag so we know this is just a preview.
+            'preview' => true, // Extra tag so we know this is just a preview.
         ];
 
         $COMMENTLIST = new COMMENTLIST();

www/includes/easyparliament/templates/html/register/_register_field.php

Braces +5 added lines, -2 removed lines

@@ -12,11 +12,14 @@
         <?php $detail = $upper_detail; ?>
     </ul>
     </li>
-<?php else : ?>
+<?php else {
+    : ?>
     <li class="interest-detail">
     <?php if ($detail->has_value()): ?>
         <span class="interest-detail-name"><?= $detail->display_as ?>: </span>
         <span class="interest-detail-value"><?= htmlspecialchars($detail->value) ?></span>
-    <?php endif; ?>
+    <?php endif;
+}
+?>
     </li>
 <?php endif; ?>
\ No newline at end of file

classes/GlossaryView/AtoZView.php

Spacing +1 added lines, -1 removed lines

@@ -45,7 +45,7 @@
         $url->insert(['gl' => $glossary->next_term['glossary_id']]);
         $next_link = $url->generate('url');
 
-        $nextprev =  [
+        $nextprev = [
             'next'	=>  [
                 'url'	=> $next_link,
                 'title'	=> 'Next term',

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

Braces +4 added lines, -1 removed lines

@@ -92,7 +92,10 @@
                                                             E-mail correspondence with APPG
                                                         <?php } ?>
                                                     </li>
-                                                    <li><span class="appg-property-label">APPG Website:</span> <?php if ($membership->appg->website): ?><a href="<?= $membership->appg->website ?>"><?= $membership->appg->website ?></a><?php else: ?>N/A<?php endif; ?></li>
+                                                    <li><span class="appg-property-label">APPG Website:</span> <?php if ($membership->appg->website): ?><a href="<?= $membership->appg->website ?>"><?= $membership->appg->website ?></a><?php else {
+    : ?>N/A<?php endif;
+}
+?></li>
                                                     <li><span class="appg-property-label">APPG register:</span> <a href="<?= $membership->appg->source_url ?>">Parliament website</a></li>
                                                 </ul>
                                             </div>

www/includes/easyparliament/templates/html/glossary/addterm_matches.php

Indentation +6 added lines, -6 removed lines

@@ -21,12 +21,12 @@
             <p>It seems we already have <?= ngettext('a definition', 'some definitions', $glossary->num_search_matches) ?> for that. Would you care to see <?= ngettext('it', 'them', $glossary->num_search_matches) ?>?</p>
             <ul class="glossary">
             <?php
-              foreach ($glossary->search_matches as $match) {
-                  $URL = new \MySociety\TheyWorkForYou\Url('glossary');
-                  $URL->insert(['gl' => $match['glossary_id']]);
-                  $URL->remove(['g']);
-                  $term_link = $URL->generate('url');
-                  ?>
+                foreach ($glossary->search_matches as $match) {
+                    $URL = new \MySociety\TheyWorkForYou\Url('glossary');
+                    $URL->insert(['gl' => $match['glossary_id']]);
+                    $URL->remove(['g']);
+                    $term_link = $URL->generate('url');
+                    ?>
                   <li>
                     <a href="<?= $term_link ?>"><?= $match['title']?></a>
                   </li>

classes/GlossaryView/TermView.php

Spacing +1 added lines, -1 removed lines

@@ -45,7 +45,7 @@
         $url->insert(['gl' => $glossary->next_term['glossary_id']]);
         $next_link = $url->generate('url');
 
-        $nextprev =  [
+        $nextprev = [
             'next'	=>  [
                 'url'	=> $next_link,
                 'title'	=> 'Next term',

www/includes/easyparliament/page.php

Spacing +6 added lines, -6 removed lines

@@ -51,7 +51,7 @@ 
                 $THISPAGE = new \MySociety\TheyWorkForYou\Url($this_page);
 
                 $LOGINURL = new \MySociety\TheyWorkForYou\Url('userlogin');
-                $LOGINURL->insert(['ret' => $THISPAGE->generate('none') ]);
+                $LOGINURL->insert(['ret' => $THISPAGE->generate('none')]);
 
                 $text = "<a href=\"" . $LOGINURL->generate() . '">' . gettext('You’d better sign in!') . '</a>';
             } else {
@@ -391,7 +391,7 @@ 
         if (is_string($message)) {
             // Sometimes we're just sending a single line to this function
             // rather like the bigger array...
-            $message =  [
+            $message = [
                 'text' => $message,
             ];
         }
@@ -405,7 +405,7 @@ 
             if (!isset($page_errors)) {
                 $page_errors = [];
             }
-            $page_errors[]  = $message;
+            $page_errors[] = $message;
         }
 
         if ($fatal) {
@@ -1022,7 +1022,7 @@ 
 
                 $body = trim_characters($report['body'], 0, 40);
 
-                $tabledata['rows'][] =  [
+                $tabledata['rows'][] = [
                     _htmlentities($report['firstname'] . ' ' . $report['lastname']),
                     _htmlentities($body),
                     $report['reported'],
@@ -1104,11 +1104,11 @@ 
         // Returns HTML suitable for putting in the sidebar on Admin pages.
         global $this_page, $DATA;
 
-        $pages =  ['admin_home',
+        $pages = ['admin_home',
             'admin_comments', 'admin_searchlogs', 'admin_popularsearches', 'admin_failedsearches',
             'alert_stats', 'admin_statistics', 'admin_reportstats',
             'admin_commentreports', 'admin_glossary', 'admin_glossary_pending', 'admin_badusers',
-            'admin_profile_message', 'admin_photos', 'admin_mpurls', 'admin_policies', 'admin_banner', 'admin_announcement','admin_featured', 'admin_topics',
+            'admin_profile_message', 'admin_photos', 'admin_mpurls', 'admin_policies', 'admin_banner', 'admin_announcement', 'admin_featured', 'admin_topics',
             'admin_wikipedia',
         ];
 

tests/CommentTest.php

Spacing +4 added lines, -4 removed lines

@@ -132,7 +132,7 @@ 
 
         $commentId = $comment->create($data);
 
-        $page = $this->fetch_page([ 'type' => 'debates', 'id' => '2014-01-01b.1.2' ]);
+        $page = $this->fetch_page(['type' => 'debates', 'id' => '2014-01-01b.1.2']);
         $this->assertStringContainsString('This is a...', $page);
 
         $THEUSER->_update([
@@ -146,7 +146,7 @@ 
             'organisation' => '',
         ]);
 
-        $page = $this->fetch_page([ 'type' => 'debates', 'id' => '2014-01-01b.1.2' ]);
+        $page = $this->fetch_page(['type' => 'debates', 'id' => '2014-01-01b.1.2']);
         $this->assertStringNotContainsString('This is a...', $page);
     }
 
@@ -169,12 +169,12 @@ 
 
         self::$db->query("UPDATE comments SET user_id = 2 WHERE comment_id = $commentId");
 
-        $page = $this->fetch_page([ 'type' => 'debates', 'id' => '2014-01-01b.1.2' ]);
+        $page = $this->fetch_page(['type' => 'debates', 'id' => '2014-01-01b.1.2']);
         $this->assertStringNotContainsString('This is a...', $page);
 
         self::$db->query("UPDATE comments SET posted = '2024-10-09 12:42:11' WHERE comment_id = $commentId");
 
-        $page = $this->fetch_page([ 'type' => 'debates', 'id' => '2014-01-01b.1.2' ]);
+        $page = $this->fetch_page(['type' => 'debates', 'id' => '2014-01-01b.1.2']);
         $this->assertStringContainsString('This is a...', $page);
     }
 

www/includes/easyparliament/templates/html/alert/_mp_alert_form.php

Indentation +7 added lines, -7 removed lines

@@ -20,8 +20,8 @@ 
   <h3><?= sprintf(gettext('Sign up for alerts when people matching <i>%s</i> speaks'), _htmlspecialchars($search_term)) ?></h3>
   <ul>
     <?php
-      foreach ($members as $row) {
-          ?>
+        foreach ($members as $row) {
+            ?>
       <li>
           <form action="<?= $actionurl ?>" method="post">
               <input type="hidden" name="t" value="<?= _htmlspecialchars($token) ?>">
@@ -30,11 +30,11 @@ 
               <input type="hidden" name="ignore_speaker_votes" value="<?= $ignore_speaker_votes ?>">
               <?php
                     $name = member_full_name($row['house'], $row['title'], $row['given_name'], $row['family_name'], $row['lordofname']);
-          if ($row['constituency']) {
-              $name .= ' (' . gettext($row['constituency']) . ')';
-          }
-          printf(gettext('When %s speaks.'), $name);
-          ?>
+            if ($row['constituency']) {
+                $name .= ' (' . gettext($row['constituency']) . ')';
+            }
+            printf(gettext('When %s speaks.'), $name);
+            ?>
               <input type="submit" class="button small" value="<?= gettext('Subscribe') ?>"></form>
           </form>
       </li>