mysociety / theyworkforyou

www/docs/admin/profile-message.php

Upper-Lower-Casing +1 added lines, -1 removed lines

@@ -91,6 +91,6 @@
     <span class="formw"><input type="submit" name="submit" value="Update"></span>
 </div>
 </form>
-EOF;
+eof;
     return $out;
 }

Spacing +1 added lines, -1 removed lines

@@ -47,7 +47,7 @@
         if ($row['constituency']) {
             $desc .= ', ' . $row['constituency'];
         }
-        $out .= '<option value="'.$p_id.'">'.$desc.'</option>' . "\n";
+        $out .= '<option value="' . $p_id . '">' . $desc . '</option>' . "\n";
     }
 
     $out .= ' </select></span> </div> ';

Braces +3 added lines, -1 removed lines

@@ -43,7 +43,9 @@
         $house = $row['house'];
         $desc = member_full_name($house, $row['title'], $row['given_name'], $row['family_name'], $row['lordofname']) .
                 " " . $houses[$house];
-        if ($row['party']) $desc .= ' (' . $row['party'] . ')';
+        if ($row['party']) {
+            $desc .= ' (' . $row['party'] . ')';
+        }
         if ($row['constituency']) {
             $desc .= ', ' . $row['constituency'];
         }

tests/AlertsTest.php

Spacing +4 added lines, -4 removed lines

@@ -11,7 +11,7 @@ 
      */
     public function getDataSet()
     {
-        return $this->createMySQLXMLDataSet(dirname(__FILE__).'/_fixtures/alerts.xml');
+        return $this->createMySQLXMLDataSet(dirname(__FILE__) . '/_fixtures/alerts.xml');
     }
 
     /**
@@ -31,7 +31,7 @@ 
     {
         $ALERT = new ALERT();
 
-        $response = $ALERT->fetch(0,0);
+        $response = $ALERT->fetch(0, 0);
 
         // Make sure we only get one response
         $this->assertEquals(1, count($response['data']));
@@ -48,7 +48,7 @@ 
     {
         $ALERT = new ALERT();
 
-        $response = $ALERT->fetch(1,0);
+        $response = $ALERT->fetch(1, 0);
 
         // Make sure we only get two responses
         $this->assertEquals(2, count($response['data']));
@@ -65,7 +65,7 @@ 
     {
         $ALERT = new ALERT();
 
-        $response = $ALERT->fetch_between(1,0, '2014-02-03', '2014-02-04');
+        $response = $ALERT->fetch_between(1, 0, '2014-02-03', '2014-02-04');
 
         // Make sure we only get one response
         $this->assertEquals(1, count($response['alerts']));

www/includes/easyparliament/templates/html/search/by-person.php

Braces +2 added lines, -1 removed lines

@@ -111,7 +111,8 @@
                                 <?php if ( $house !=  HOUSE_TYPE_LORDS) { ?>
                                   <?= isset($speaker['office']) ? ' - ' . join('; ', $speaker['office']) : '' ?>
                                 <?php } ?>
-                              <?php } else { // no $pid ?>
+                              <?php } else {
+// no $pid ?>
                                 <?= $speaker['name'] ?>
                               <?php } ?>
                             </td>

Spacing +22 added lines, -22 removed lines

@@ -9,17 +9,17 @@ 
             <div class="search-page__section__primary">
                 <h2>Who says <em class="current-search-term"><?= _htmlentities($searchstring) ?></em> the most?</h2>
 
-              <?php if ( isset($error) ) { ?>
-                <?php if ( $error == 'No results' && isset( $house ) && $house != HOUSE_TYPE_ROYAL ) { ?>
+              <?php if (isset($error)) { ?>
+                <?php if ($error == 'No results' && isset($house) && $house != HOUSE_TYPE_ROYAL) { ?>
                   <ul class="search-result-display-options">
                       <li>
-                        <?php if ( $house ==  HOUSE_TYPE_COMMONS) { ?>
+                        <?php if ($house == HOUSE_TYPE_COMMONS) { ?>
                           No results for MPs only
-                        <?php } else if ( $house ==  HOUSE_TYPE_LORDS) { ?>
+                        <?php } else if ($house == HOUSE_TYPE_LORDS) { ?>
                           No results for Peers only
-                        <?php } else if ( $house == HOUSE_TYPE_SCOTLAND) { ?>
+                        <?php } else if ($house == HOUSE_TYPE_SCOTLAND) { ?>
                           No results for MSPs only
-                        <?php } else if ( $house ==  HOUSE_TYPE_NI) { ?>
+                        <?php } else if ($house == HOUSE_TYPE_NI) { ?>
                           No results for MLAs only
                         <?php } ?>
                           |
@@ -31,7 +31,7 @@ 
                 <?php } ?>
               <?php } ?>
 
-              <?php if ( $wtt ) { ?>
+              <?php if ($wtt) { ?>
                 <p><strong>Now, try reading what a couple of these Lords are saying,
                 to help you find someone appropriate. When you've found someone,
                 follow the "I want to write to (name of lord)" link on their results page
@@ -39,37 +39,37 @@ 
                 </strong></p>
               <?php } ?>
 
-              <?php if ( isset($speakers) && count($speakers) ) { ?>
+              <?php if (isset($speakers) && count($speakers)) { ?>
 
-                  <?php if ( !$wtt ) { ?>
+                  <?php if (!$wtt) { ?>
                     <ul class="search-result-display-options">
                         <li>Results grouped by person</li>
                         <li>
-                          <?php if ( $house ==  HOUSE_TYPE_ROYAL) { ?>
+                          <?php if ($house == HOUSE_TYPE_ROYAL) { ?>
                             Show All
                           <?php } else { ?>
                             <a href="<?= $this_url->generate('html') ?>">Show All</a>
                           <?php } ?>
                             |
-                          <?php if ( $house ==  HOUSE_TYPE_COMMONS) { ?>
+                          <?php if ($house == HOUSE_TYPE_COMMONS) { ?>
                             MPs only
                           <?php } else { ?>
                             <a href="<?= $this_url->generate('html', array('house'=>1)) ?>">MPs only</a>
                           <?php } ?>
                             |
-                          <?php if ( $house ==  HOUSE_TYPE_LORDS) { ?>
+                          <?php if ($house == HOUSE_TYPE_LORDS) { ?>
                             Peers only
                           <?php } else { ?>
                             <a href="<?= $this_url->generate('html', array('house'=>2)) ?>">Lords only</a>
                           <?php } ?>
                             |
-                          <?php if ( $house ==  HOUSE_TYPE_SCOTLAND) { ?>
+                          <?php if ($house == HOUSE_TYPE_SCOTLAND) { ?>
                             MSPs only
                           <?php } else { ?>
                             <a href="<?= $this_url->generate('html', array('house'=>4)) ?>">MSPs only</a>
                           <?php } ?>
                             |
-                          <?php if ( $house ==  HOUSE_TYPE_NI) { ?>
+                          <?php if ($house == HOUSE_TYPE_NI) { ?>
                             MLAs only
                           <?php } else { ?>
                             <a href="<?= $this_url->generate('html', array('house'=>HOUSE_TYPE_NI)) ?>">MLAs only</a>
@@ -90,25 +90,25 @@ 
                         </tr>
                     </thead>
                     <tbody>
-                      <?php foreach ( $speakers as $pid => $speaker ) { ?>
+                      <?php foreach ($speakers as $pid => $speaker) { ?>
 
-                        <?php if ( $wtt && $pid == 0 ) { continue; } // skip heading count for WTT lords list ?>
+                        <?php if ($wtt && $pid == 0) { continue; } // skip heading count for WTT lords list ?>
 
                         <tr>
                             <td><?= $speaker['count'] ?></td>
                             <td>
-                              <?php if ( $pid ) { ?>
-                                <?php if ( !$wtt || $speaker['left'] == '9999-12-31' ) { ?>
+                              <?php if ($pid) { ?>
+                                <?php if (!$wtt || $speaker['left'] == '9999-12-31') { ?>
                                   <a href="/search/?q=<?= _htmlentities($searchstring) ?>&amp;pid=<?= $pid ?><?= isset($wtt) && $speaker['left'] == '9999-12-31' ? '&amp;wtt=2' : '' ?>">
                                 <?php } ?>
                                 <?= isset($speaker['name']) ? $speaker['name'] : 'N/A' ?>
-                                <?php if ( !$wtt || $speaker['left'] == '9999-12-31' ) { ?>
+                                <?php if (!$wtt || $speaker['left'] == '9999-12-31') { ?>
                                   </a>
                                 <?php } ?>
-                                <?php if ( isset($speaker['party']) ) { ?>
+                                <?php if (isset($speaker['party'])) { ?>
                                   <span class="search-results-grouped__speaker-party">(<?= $speaker['party'] ?>)</span>
                                 <?php } ?>
-                                <?php if ( $house !=  HOUSE_TYPE_LORDS) { ?>
+                                <?php if ($house != HOUSE_TYPE_LORDS) { ?>
                                   <?= isset($speaker['office']) ? ' - ' . join('; ', $speaker['office']) : '' ?>
                                 <?php } ?>
                               <?php } else { // no $pid ?>
@@ -116,7 +116,7 @@ 
                               <?php } ?>
                             </td>
                             <td>
-                              <?php if ( format_date($speaker['pmindate'], 'M Y') == format_date($speaker['pmaxdate'], 'M Y') ) { ?>
+                              <?php if (format_date($speaker['pmindate'], 'M Y') == format_date($speaker['pmaxdate'], 'M Y')) { ?>
                                 <?= format_date($speaker['pmindate'], 'M Y') ?>
                               <?php } else { ?>
                                 <?= format_date($speaker['pmindate'], 'M Y') ?>&nbsp;&ndash;&nbsp;<?= format_date($speaker['pmaxdate'], 'M Y') ?>

classes/EditQueue.php

Indentation +3 added lines, -3 removed lines

@@ -160,9 +160,9 @@
                                     VALUES
                                     (:id, :type, :enabled);',
                                     array(
-                                      ':id' => $q->insert_id(),
-                                      ':type' => EPTYPE_GLOSSARY,
-                                      ':enabled' => 1,
+                                        ':id' => $q->insert_id(),
+                                        ':type' => EPTYPE_GLOSSARY,
+                                        ':enabled' => 1,
                                     ));
                     // Again, no point carrying on if this fails,
                     // so remove the previous entry

Braces +3 added lines, -6 removed lines

@@ -194,8 +194,7 @@ 
                             ));
             if (!$q->success()) {
                 break;
-            }
-            else {
+            } else {
                 // Now send them an email telling them they've been approved
 
 
@@ -236,8 +235,7 @@ 
                             WHERE edit_id=" . $decline_id . ";");
             if (!$q->success()) {
                 break;
-            }
-            else {
+            } else {
                 // Scrub that one from the list of pending items
                 unset ($this->pending[$decline_id]);
             }
@@ -275,8 +273,7 @@ 
             $this->update_pending_count();
 
             return true;
-        }
-        else {
+        } else {
             return false;
         }
     }

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

Indentation +1 added lines, -1 removed lines

@@ -55,7 +55,7 @@
                 } else if ($the_users_mp) {
                     $wtt_url = $wtt_url . "?a=WMC&pc=" . _htmlentities(urlencode($user_postcode));
                 }
-              ?>
+                ?>
                 <a href="<?= $wtt_url ?>" class="button" onclick="trackLinkClick(this, 'Links', 'WriteToThem', 'Person'); return false;">Send a message</a>
               <?php } ?>
               <?php if ($has_email_alerts) { ?>

Spacing +6 added lines, -6 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,19 +11,19 @@ 
           <?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">
                     <span class="person-header__about__position__role">
                         <?= $latest_membership['current'] ? '' : 'Former' ?>
                         <?= $latest_membership['party'] == 'Bishop' ? '' : $latest_membership['party'] ?>
                         <?= $latest_membership['rep_name'] ?>
                     </span>
-                  <?php if ( $latest_membership['constituency'] ) { ?>
+                  <?php if ($latest_membership['constituency']) { ?>
                     for
                     <span class="person-header__about__position__constituency">
                         <?= $latest_membership['constituency'] ?>
@@ -33,7 +33,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>

classes/Search/ParseArgs.php

Braces +14 added lines, -6 removed lines

@@ -28,11 +28,17 @@ 
 
         if (get_http_var('from') || get_http_var('to')) {
             $from = parse_date(get_http_var('from'));
-            if ($from) $from = $from['iso'];
-            else $from = '1919-01-01';
+            if ($from) {
+                $from = $from['iso'];
+            } else {
+                $from = '1919-01-01';
+            }
             $to = parse_date(get_http_var('to'));
-            if ($to) $to = $to['iso'];
-            else $to = date('Y-m-d');
+            if ($to) {
+                $to = $to['iso'];
+            } else {
+                $to = date('Y-m-d');
+            }
             $searchstring .= " $from..$to";
         }
 
@@ -105,8 +111,10 @@ 
         }
 
         $advsection = get_http_var('section');
-        if (!$advsection)
-            $advsection = get_http_var('maj'); # Old URLs had this
+        if (!$advsection) {
+                    $advsection = get_http_var('maj');
+        }
+        # Old URLs had this
         if (is_array($advsection)) {
             $searchstring .= ' section:' . join(' section:', $advsection);
         } elseif ($advsection) {

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">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/mp/_vote_footer.php

Spacing +2 added lines, -2 removed lines

@@ -6,9 +6,9 @@
     <a href="/help/#vote-descriptions">vote descriptions</a> and
     <a href="/help/#votingrecord">how the voting record is decided</a></p>
 
-  <?php if(isset($data['photo_attribution_text'])) { ?>
+  <?php if (isset($data['photo_attribution_text'])) { ?>
     <p>
-      <?php if(isset($data['photo_attribution_link'])) { ?>
+      <?php if (isset($data['photo_attribution_link'])) { ?>
         Profile photo:
         <a href="<?= $data['photo_attribution_link'] ?>"><?= $data['photo_attribution_text'] ?></a>
       <?php } else { ?>

www/docs/admin/users.php

Upper-Lower-Casing +1 added lines, -1 removed lines

@@ -20,7 +20,7 @@
     <input type="hidden" name="scrub" value="$email">
     <input type="submit" value="Delete user">
 </form>
-EOF;
+eof;
 }
 
 $user_data = array(