mysociety / theyworkforyou

www/includes/easyparliament/templates/html/static/linktous.php

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

@@ -24,7 +24,7 @@
     </form>
 </div>
 <!-- TheyWorkForYou box, end -->
-END;
+end;
 print $link_to_us_form;
 ?>
 

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

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

@@ -1,4 +1,4 @@
 <div class="full-page__row">
     <?php include '_business_section.php'; ?>
-    <?php $search_title = "Search $title"; include '_search.php'; ?>
+    <?php $search_title = "search $title"; include '_search.php'; ?>
 </div>

www/includes/easyparliament/staticpages/search_help.php

Spacing +1 added lines, -1 removed lines

@@ -31,7 +31,7 @@
 </li>
 
 <?php
-$user_agent = ( isset( $_SERVER['HTTP_USER_AGENT'] ) ) ? strtolower( $_SERVER['HTTP_USER_AGENT'] ) : '';
+$user_agent = (isset($_SERVER['HTTP_USER_AGENT'])) ? strtolower($_SERVER['HTTP_USER_AGENT']) : '';
 if (stristr($user_agent, 'Firefox/')) {
 ?>
      <li>You can also add TheyWorkForYou to <a href="http://mycroft.mozdev.org/download.html?name=theyworkforyou">Firefox's search box</a>.</li>

www/docs/alert/by-postcode/index.php

Spacing +1 added lines, -1 removed lines

@@ -3,7 +3,7 @@
 $new_style_template = TRUE;
 
 include_once '../../../includes/easyparliament/init.php';
-include_once INCLUDESPATH . 'easyparliament/member.php';
+include_once INCLUDESPATH.'easyparliament/member.php';
 
 $alert = new MySociety\TheyWorkForYou\AlertView\Simple($THEUSER);
 $data = $alert->display();

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

@@ -1,6 +1,6 @@
 <?php
 
-$new_style_template = TRUE;
+$new_style_template = true;
 
 include_once '../../../includes/easyparliament/init.php';
 include_once INCLUDESPATH . 'easyparliament/member.php';

www/docs/topic/index.php

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

@@ -7,7 +7,7 @@
 namespace MySociety\TheyWorkForYou;
 
 // Disable the old PAGE class.
-$new_style_template = TRUE;
+$new_style_template = true;
 global $this_page;
 $this_page = 'topics';
 

www/docs/topic/image.php

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

@@ -8,7 +8,7 @@
     $topic = $topics->getTopic($slug);
 }
 
-if (!$slug || !isset($topic) || $topic === NULL) {
+if (!$slug || !isset($topic) || $topic === null) {
     header('HTTP/1.0 404 Not Found');
     exit();
 }

www/docs/user/login/fb.php

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

@@ -9,7 +9,7 @@
     If the input is not OK, the form is displayed again with error messages.
 */
 
-$new_style_template = TRUE;
+$new_style_template = true;
 
 include_once '../../../includes/easyparliament/init.php';
 # need to include this as login code uses error_message

Indentation +3 added lines, -3 removed lines

@@ -27,11 +27,11 @@
 
 $data['fb_login_url'] = $login->getLoginURL();
 if (isset($data['token'])) {
-  $success = $login->loginUser($data['token']);
-  if (!$success) {
+    $success = $login->loginUser($data['token']);
+    if (!$success) {
     $data['error'] = 'Could not login using Facebook token';
     \MySociety\TheyWorkForYou\Renderer::output('login/facebook', $data);
-  }
+    }
 } else {
     \MySociety\TheyWorkForYou\Renderer::output('login/facebook', $data);
 }

classes/Utility/Alert.php

Spacing +9 added lines, -9 removed lines

@@ -33,15 +33,15 @@ 
                 ':email' => $email
             ));
         $out = '';
-        for ($i=0; $i<$q->rows(); ++$i) {
+        for ($i = 0; $i < $q->rows(); ++$i) {
             $row = $q->row($i);
-            $criteria = explode(' ',$row['criteria']);
+            $criteria = explode(' ', $row['criteria']);
             $ccc = array();
             $current = true;
             foreach ($criteria as $c) {
-                if (preg_match('#^speaker:(\d+)#',$c,$m)) {
+                if (preg_match('#^speaker:(\d+)#', $c, $m)) {
                     $MEMBER = new \MEMBER(array('person_id'=>$m[1]));
-                    $ccc[] = 'spoken by ' . $MEMBER->full_name();
+                    $ccc[] = 'spoken by '.$MEMBER->full_name();
                     if (!$MEMBER->current_member_anywhere()) {
                         $current = false;
                     }
@@ -49,24 +49,24 @@ 
                     $ccc[] = $c;
                 }
             }
-            $criteria = join(' ',$ccc);
-            $token = $row['alert_id'] . '-' . $row['registrationtoken'];
+            $criteria = join(' ', $ccc);
+            $token = $row['alert_id'].'-'.$row['registrationtoken'];
             $action = '<form action="/alert/" method="post"><input type="hidden" name="t" value="'.$token.'">';
             if (!$row['confirmed']) {
                 $action .= '<input type="submit" name="action" value="Confirm">';
-            } elseif ($row['deleted']==2) {
+            } elseif ($row['deleted'] == 2) {
                 $action .= '<input type="submit" name="action" value="Resume">';
             } else {
                 $action .= '<input type="submit" name="action" value="Suspend"> <input type="submit" name="action" value="Delete">';
             }
             $action .= '</form>';
-            $out .= '<tr><td>' . $criteria . '</td><td align="center">' . $action . '</td></tr>';
+            $out .= '<tr><td>'.$criteria.'</td><td align="center">'.$action.'</td></tr>';
             if (!$current) {
                 $out .= '<tr><td colspan="2"><small>&nbsp;&mdash; <em>not a current member of any body covered by TheyWorkForYou</em></small></td></tr>';
             }
         }
         if ($out) {
-            print '<table cellpadding="3" cellspacing="0"><tr><th>Criteria</th><th>Action</th></tr>' . $out . '</table>';
+            print '<table cellpadding="3" cellspacing="0"><tr><th>Criteria</th><th>Action</th></tr>'.$out.'</table>';
         } else {
             print '<p>You currently have no email alerts set up. You can create alerts <a href="/alert/">here</a>.</p>';
         }

www/docs/user/index.php

Spacing +1 added lines, -1 removed lines

@@ -32,7 +32,7 @@
         // And make sure the user is allowed to do this!
 
         $template = 'user/form';
-        if (is_numeric( get_http_var("u") ) && $THEUSER->is_able_to("edituser")) {
+        if (is_numeric(get_http_var("u")) && $THEUSER->is_able_to("edituser")) {
 
             $data = $user->getUserDetails(get_http_var('u'));
             $data['showall'] = true;

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

@@ -5,7 +5,7 @@ 
  * page so leave it like that for now.
  */
 
-$new_style_template = TRUE;
+$new_style_template = true;
 
 include_once '../../includes/easyparliament/init.php';
 # need to include this as login code uses error_message
@@ -127,7 +127,7 @@ 
                 $data['errors'] = $results['errors'];
                 $template = 'user/form';
             } else {
-                $data['edited'] = True;
+                $data['edited'] = true;
                 if (isset($results['email_changed'])) {
                     $data['email_changed'] = $results['email_changed'];
                 }

Indentation +3 added lines, -3 removed lines

@@ -105,10 +105,10 @@
 
 // if data has been submitted then handle that
 if (
-      get_http_var("submitted") == "true" && (
+        get_http_var("submitted") == "true" && (
         $this_page == 'useredit' || $this_page == 'otheruseredit' || $this_page == 'userjoin'
-      )
-   ) {
+        )
+    ) {
     // Put all the user-submitted data in an array.
     $data = $user->getUpdateDetails($this_page, $THEUSER->facebook_user);