mysociety / theyworkforyou

www/docs/departments/index.php

Braces +3 added lines, -2 removed lines

@@ -33,8 +33,9 @@
     if (isset($arr[3])) {
         print '<a href="'.$link.'/questions">Written Questions</a>';
     }
-    if (count($arr)==2)
-        print ' | ';
+    if (count($arr)==2) {
+            print ' | ';
+    }
     if (isset($arr[4])) {
         print '<a href="'.$link.'/statements">Written Ministerial Statements</a>';
     }

Spacing +5 added lines, -5 removed lines

@@ -16,7 +16,7 @@ 
     group by body, major
     order by body');
 $data = array();
-for ($i=0; $i<$q->rows(); $i++) {
+for ($i = 0; $i < $q->rows(); $i++) {
     $body = $q->field($i, 'body');
     $major = $q->field($i, 'major');
     $data[$body][$major] = true;
@@ -26,17 +26,17 @@ 
 
 print '<ul>';
 foreach ($data as $body => $arr) {
-    $link = strtolower(str_replace(' ','_',$body));
+    $link = strtolower(str_replace(' ', '_', $body));
     print '<li>';
     print $body;
     print ' &mdash; ';
     if (isset($arr[3])) {
-        print '<a href="'.$link.'/questions">Written Questions</a>';
+        print '<a href="' . $link . '/questions">Written Questions</a>';
     }
-    if (count($arr)==2)
+    if (count($arr) == 2)
         print ' | ';
     if (isset($arr[4])) {
-        print '<a href="'.$link.'/statements">Written Ministerial Statements</a>';
+        print '<a href="' . $link . '/statements">Written Ministerial Statements</a>';
     }
     print '</li>';
 }

www/docs/video/stamps.php

Braces +9 added lines, -4 removed lines

@@ -17,7 +17,9 @@ 
 $adate = $q->field(0, 'adate');
 $atime = $q->field(0, 'atime');
 $video = \MySociety\TheyWorkForYou\Utility\Video::fromTimestamp($videodb, $adate, $atime);
-if (!$video) exit;
+if (!$video) {
+    exit;
+}
 
 $start = date('H:i:s', strtotime($video['broadcast_start']. ' GMT'));
 $end = date('H:i:s', strtotime($video['broadcast_end'] . ' GMT'));
@@ -35,7 +37,9 @@ 
 print '<stamps>';
 for ($i=0; $i<$q->rows(); $i++) {
     $gid = str_replace('uk.org.publicwhip/', '', $q->field($i, 'gid'));
-    if (isset($gids[$gid])) continue;
+    if (isset($gids[$gid])) {
+        continue;
+    }
     $timetoend = $q->field($i, 'timetoend') - $file_offset;
     if ($timetoend>0) {
         $video = \MySociety\TheyWorkForYou\Utility\Video::fromTimestamp($videodb, $q->field($i, 'adate'), $q->field($i, 'atime'));
@@ -45,8 +49,9 @@ 
         $end = date('H:i:s', strtotime($video['broadcast_end'] . ' GMT'));
     }
     $timediff = $q->field($i, 'timediff') - $file_offset;
-    if ($timediff>=0)
-        print "<stamp><gid>$gid</gid><file>$video[id]</file><time>$timediff</time></stamp>\n";
+    if ($timediff>=0) {
+            print "<stamp><gid>$gid</gid><file>$video[id]</file><time>$timediff</time></stamp>\n";
+    }
     $gids[$gid] = true;
 }
 print '</stamps>';

Spacing +7 added lines, -7 removed lines

@@ -19,7 +19,7 @@ 
 $video = \MySociety\TheyWorkForYou\Utility\Video::fromTimestamp($videodb, $adate, $atime);
 if (!$video) exit;
 
-$start = date('H:i:s', strtotime($video['broadcast_start']. ' GMT'));
+$start = date('H:i:s', strtotime($video['broadcast_start'] . ' GMT'));
 $end = date('H:i:s', strtotime($video['broadcast_end'] . ' GMT'));
 
 $q = $db->query("select video_timestamps.gid, adate, atime, time_to_sec(timediff(atime, '$start')) as timediff,
@@ -33,26 +33,26 @@ 
 $gids = array();
 $file_offset = 0;
 print '<stamps>';
-for ($i=0; $i<$q->rows(); $i++) {
+for ($i = 0; $i < $q->rows(); $i++) {
     $gid = str_replace('uk.org.publicwhip/', '', $q->field($i, 'gid'));
     if (isset($gids[$gid])) continue;
     $timetoend = $q->field($i, 'timetoend') - $file_offset;
-    if ($timetoend>0) {
+    if ($timetoend > 0) {
         $video = \MySociety\TheyWorkForYou\Utility\Video::fromTimestamp($videodb, $q->field($i, 'adate'), $q->field($i, 'atime'));
-        $new_start = date('H:i:s', strtotime($video['broadcast_start']. ' GMT'));
+        $new_start = date('H:i:s', strtotime($video['broadcast_start'] . ' GMT'));
         $file_offset += timediff($new_start, $start);
         $start = $new_start;
         $end = date('H:i:s', strtotime($video['broadcast_end'] . ' GMT'));
     }
     $timediff = $q->field($i, 'timediff') - $file_offset;
-    if ($timediff>=0)
+    if ($timediff >= 0)
         print "<stamp><gid>$gid</gid><file>$video[id]</file><time>$timediff</time></stamp>\n";
     $gids[$gid] = true;
 }
 print '</stamps>';
 
 function timediff($a, $b) {
-    return substr($a, 0, 2)*3600 + substr($a, 3, 2)*60 + substr($a, 6, 2)
-        - substr($b, 0, 2)*3600 - substr($b, 3, 2)*60 - substr($b, 6, 2);
+    return substr($a, 0, 2) * 3600 + substr($a, 3, 2) * 60 + substr($a, 6, 2)
+        - substr($b, 0, 2) * 3600 - substr($b, 3, 2) * 60 - substr($b, 6, 2);
 
 }

www/docs/video/delete.php

Braces +5 added lines, -2 removed lines

@@ -5,7 +5,9 @@ 
 $gid = get_http_var('gid');
 $oops = get_http_var('oops');
 
-if (!$oops || !$gid) exit;
+if (!$oops || !$gid) {
+    exit;
+}
 
 $gid = "uk.org.publicwhip/$gid";
 
@@ -16,7 +18,8 @@ 
 
 $db = new ParlDB;
 $q = $db->query("update video_timestamps set deleted=1 where id = :oops and gid = :gid and current_timestamp<whenstamped+interval 30 second", $params);
-if ($q->affected_rows())
+if ($q->affected_rows()) {
     $db->query("update hansard set video_status = video_status & 11 where gid = :gid", $params);
+}
 
 print "<status>OK</status>";

www/docs/video/submit.php

Braces +8 added lines, -3 removed lines

@@ -18,11 +18,16 @@
 $htime = $q->field(0, 'htime');
 $atime = $q->field(0, 'atime');
 $adate = $q->field(0, 'adate');
-if ($atime) $htime = $atime;
-if ($adate) $hdate = $adate;
+if ($atime) {
+    $htime = $atime;
+}
+if ($adate) {
+    $hdate = $adate;
+}
 
-if (!$hdate || !$htime || !$time)
+if (!$hdate || !$htime || !$time) {
     exit;
+}
 
 $videodb = \MySociety\TheyWorkForYou\Utility\Video::dbConnect();
 if (!$file) {

www/docs/video/next.php

Braces +3 added lines, -1 removed lines

@@ -5,7 +5,9 @@
 $action = get_http_var('action');
 $pid = intval(get_http_var('pid'));
 $major = intval(get_http_var('major'));
-if (!$major) $major = 1;
+if (!$major) {
+    $major = 1;
+}
 
 if ($action == 'next' || $action=='nextneeded') {
     $gid = get_http_var('gid');

Spacing +3 added lines, -3 removed lines

@@ -7,7 +7,7 @@ 
 $major = intval(get_http_var('major'));
 if (!$major) $major = 1;
 
-if ($action == 'next' || $action=='nextneeded') {
+if ($action == 'next' || $action == 'nextneeded') {
     $gid = get_http_var('gid');
     $file = intval(get_http_var('file'));
     $time = intval(get_http_var('time'));
@@ -27,7 +27,7 @@ 
     $q = $db->query("select gid, hpos from hansard
         where hpos>$hpos and hdate='$hdate' and major=$major
         and (htype=12 or htype=13) "
-        . ($action=='nextneeded'?'and video_status in (1,3)':'') . "
+        . ($action == 'nextneeded' ? 'and video_status in (1,3)' : '') . "
         ORDER BY hpos LIMIT 1");
     if (!$q->rows()) {
         $PAGE->page_start();
@@ -41,7 +41,7 @@ 
     } else {
         $new_gid = $q->field(0, 'gid');
         $new_hpos = $q->field(0, 'hpos');
-        if ($action=='nextneeded') {
+        if ($action == 'nextneeded') {
             $q = $db->query("select adate, atime from hansard, video_timestamps
                 where hansard.gid = video_timestamps.gid and deleted=0
                     and hpos<$new_hpos and hdate='$hdate' and major=$major

www/docs/user/logout/index.php

Spacing +1 added lines, -1 removed lines

@@ -14,7 +14,7 @@
     // So we can send the user back to where they came from.
     $URL->insert(array("ret"=>get_http_var("ret")));
 }
-$THEUSER->logout( $URL->generate() );
+$THEUSER->logout($URL->generate());
 
 $message['title'] = 'You are now logged out';
 

www/docs/admin/edittopic.php

Indentation +19 added lines, -19 removed lines

@@ -11,34 +11,34 @@ 
 
 $slug = get_http_var('id');
 if ($slug) {
-  $topic = $topics->getTopic($slug);
+    $topic = $topics->getTopic($slug);
 } else {
-  $topic = new \MySociety\TheyWorkForYou\Topic();
+    $topic = new \MySociety\TheyWorkForYou\Topic();
 }
 
 $action = get_http_var('action');
 switch ($action) {
     case 'add':
       $success = add_topic($topic);
-      break;
+        break;
     case 'update':
       $success = update_topic($topic);
-      break;
+        break;
     case 'setimage':
       $success = add_image($topic);
-      break;
+        break;
     case 'addcontent':
       $success = add_content($topic);
-      break;
+        break;
     case 'deletecontent':
       $success = delete_content($topic);
-      break;
+        break;
     case 'addpolicysets':
       $success = add_policy_sets($topic);
-      break;
+        break;
     case 'addpolicies':
       $success = add_policies($topic);
-      break;
+        break;
     default:
       $success = NULL;
 }
@@ -125,10 +125,10 @@ 
             <select name="sets[]" multiple>
               <option value="">None</option>
             <?php
-              $policies = new \MySociety\TheyWorkForYou\Policies;
-              $set_descriptions = $policies->getSetDescriptions();
-              $related_sets = $topic->getPolicySets();
-              foreach ($set_descriptions as $set => $description) { ?>
+                $policies = new \MySociety\TheyWorkForYou\Policies;
+                $set_descriptions = $policies->getSetDescriptions();
+                $related_sets = $topic->getPolicySets();
+                foreach ($set_descriptions as $set => $description) { ?>
               <option value="<?= $set ?>" <?= in_array($set, $related_sets) ? 'selected' : '' ?>><?= $description ?></option>
             <?php } ?>
             <input type="submit" value="Update">
@@ -143,10 +143,10 @@ 
             <select name="policies[]" multiple>
               <option value="">None</option>
             <?php
-              $policies = new \MySociety\TheyWorkForYou\Policies;
-              $all_policies = $policies->getPolicies();
-              $related_policies = $topic->getPolicies();
-              foreach ($all_policies as $number => $description) { ?>
+                $policies = new \MySociety\TheyWorkForYou\Policies;
+                $all_policies = $policies->getPolicies();
+                $related_policies = $topic->getPolicies();
+                foreach ($all_policies as $number => $description) { ?>
               <option value="<?= $number ?>" <?= in_array($number, $related_policies) ? 'selected' : '' ?>><?= $description ?></option>
             <?php } ?>
 
@@ -214,9 +214,9 @@ 
             $file_info['tmp_name'],
             $topic->image_path()
         );
-     } catch (ErrorException $e) {
+        } catch (ErrorException $e) {
         return false;
-     }
+        }
 
     if ($image_saved) {
         return $topic->save();

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

@@ -40,7 +40,7 @@
       $success = add_policies($topic);
       break;
     default:
-      $success = NULL;
+      $success = null;
 }
 
 if (!is_null($success)) {

www/docs/admin/alert_stats.php

Spacing +18 added lines, -18 removed lines

@@ -7,7 +7,7 @@ 
 
 $PAGE->page_start();
 $PAGE->stripe_start();
-$PAGE->block_start(array ('id'=>'alerts', 'title'=>'Alert Statistics'));
+$PAGE->block_start(array('id'=>'alerts', 'title'=>'Alert Statistics'));
 $db = new ParlDB;
 
 $q_confirmed = $db->query('select count(*) as c from alerts where confirmed and not deleted');
@@ -18,13 +18,13 @@ 
 
 print '<h3>Headline stats</h3> <table>';
 $data = array(
-    'header' => array( 'Alert Type', 'Count' ),
+    'header' => array('Alert Type', 'Count'),
     'rows' => array(
-        array( 'Confirmed', $q_confirmed->field('0', 'c') ),
-        array( 'Not Confirmed', $q_not_confirmed->field('0', 'c') ),
-        array( 'Deleted', $q_deleted->field('0', 'c') ),
-        array( 'For a Speaker<sup>*</sup>', $q_speaker->field('0', 'c') ),
-        array( 'For a Keyword', $q_keyword->field('0', 'c') ),
+        array('Confirmed', $q_confirmed->field('0', 'c')),
+        array('Not Confirmed', $q_not_confirmed->field('0', 'c')),
+        array('Deleted', $q_deleted->field('0', 'c')),
+        array('For a Speaker<sup>*</sup>', $q_speaker->field('0', 'c')),
+        array('For a Keyword', $q_keyword->field('0', 'c')),
     )
 );
 
@@ -35,17 +35,17 @@ 
 $q = $db->query('select alert_id, criteria from alerts where criteria not like "%speaker:%" and criteria like "%,%" and confirmed and not deleted');
 print '<h3>People who probably wanted separate signups</h3>';
 $rows = array();
-for ($i=0; $i<$q->rows(); $i++) {
+for ($i = 0; $i < $q->rows(); $i++) {
     $id = $q->field($i, 'alert_id');
     $criteria = $q->field($i, 'criteria');
     $rows[] = array($id, $critera);
 }
-$data = array( 'rows' => $rows );
+$data = array('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');
 $tots = array(); $name = array();
-for ($i=0; $i<$q->rows(); $i++) {
+for ($i = 0; $i < $q->rows(); $i++) {
     $c = $q->field($i, 'c');
     $criteria = $q->field($i, 'criteria');
     if (!preg_match('#^speaker:(\d+)#', $criteria, $m)) continue;
@@ -59,7 +59,7 @@ 
 }
 $q = $db->query('select count(*) as c, criteria from alerts where criteria like "speaker:%" and not confirmed group by criteria order by c desc');
 $unconfirmed = array();
-for ($i=0; $i<$q->rows(); $i++) {
+for ($i = 0; $i < $q->rows(); $i++) {
     $c = $q->field($i, 'c');
     $criteria = $q->field($i, 'criteria');
     if (!preg_match('#^speaker:(\d+)#', $criteria, $m)) continue;
@@ -70,15 +70,15 @@ 
     }
 }
 
-$people_header = array( 'Name', 'Confirmed', 'Unconfirmed');
+$people_header = array('Name', 'Confirmed', 'Unconfirmed');
 print '<h3>Alert signups by MP/Peer</h3>';
 $rows = array();
 foreach ($tots as $person_id => $c) {
     $u = isset($unconfirmed[$person_id]) ? $unconfirmed[$person_id] : 0;
-    $rows[] = array( $name[$person_id], $c, $u );
+    $rows[] = array($name[$person_id], $c, $u);
 }
 $data = array(
-    'header' => array( 'Name', 'Confirmed', 'Unconfirmed'),
+    'header' => array('Name', 'Confirmed', 'Unconfirmed'),
     'rows' => $rows
 );
 $PAGE->display_table($data);
@@ -87,14 +87,14 @@ 
 $confirmed = array();
 $total = array();
 $q = $db->query("select count(*) as c, criteria from alerts where criteria not like '%speaker:%' and confirmed and not deleted group by criteria having c>1 order by c desc");
-for ($i=0; $i<$q->rows(); $i++) {
+for ($i = 0; $i < $q->rows(); $i++) {
     $c = $q->field($i, 'c');
     $criteria = $q->field($i, 'criteria');
     $confirmed[$criteria] = $c;
     $total[$criteria] = 1;
 }
 $q = $db->query("select count(*) as c, criteria from alerts where criteria not like '%speaker:%' and not confirmed group by criteria having c>1 order by c desc");
-for ($i=0; $i<$q->rows(); $i++) {
+for ($i = 0; $i < $q->rows(); $i++) {
     $c = $q->field($i, 'c');
     $criteria = $q->field($i, 'criteria');
     $unconfirmed[$criteria] = $c;
@@ -105,10 +105,10 @@ 
 foreach ($total as $criteria => $tot) {
     $c = isset($confirmed[$criteria]) ? $confirmed[$criteria] : 0;
     $u = isset($unconfirmed[$criteria]) ? $unconfirmed[$criteria] : 0;
-    $rows[] = array( $criteria, $c, $u );
+    $rows[] = array($criteria, $c, $u);
 }
 $data = array(
-    'header' => array( 'Criteria', 'Confirmed', 'Unconfirmed'),
+    'header' => array('Criteria', 'Confirmed', 'Unconfirmed'),
     'rows' => $rows
 );
 $PAGE->display_table($data);

Braces +12 added lines, -4 removed lines

@@ -48,11 +48,15 @@ 
 for ($i=0; $i<$q->rows(); $i++) {
     $c = $q->field($i, 'c');
     $criteria = $q->field($i, 'criteria');
-    if (!preg_match('#^speaker:(\d+)#', $criteria, $m)) continue;
+    if (!preg_match('#^speaker:(\d+)#', $criteria, $m)) {
+        continue;
+    }
     $person_id = $m[1];
     $MEMBER = new MEMBER(array('person_id'=>$person_id));
     if ($MEMBER->valid) {
-        if (!array_key_exists($person_id, $tots)) $tots[$person_id] = 0;
+        if (!array_key_exists($person_id, $tots)) {
+            $tots[$person_id] = 0;
+        }
         $tots[$person_id] += $c;
         $name[$person_id] = $MEMBER->full_name();
     }
@@ -62,10 +66,14 @@ 
 for ($i=0; $i<$q->rows(); $i++) {
     $c = $q->field($i, 'c');
     $criteria = $q->field($i, 'criteria');
-    if (!preg_match('#^speaker:(\d+)#', $criteria, $m)) continue;
+    if (!preg_match('#^speaker:(\d+)#', $criteria, $m)) {
+        continue;
+    }
     $person_id = $m[1];
     if (isset($name[$person_id])) {
-        if (!array_key_exists($person_id, $unconfirmed)) $unconfirmed[$person_id] = 0;
+        if (!array_key_exists($person_id, $unconfirmed)) {
+            $unconfirmed[$person_id] = 0;
+        }
         $unconfirmed[$person_id] += $c;
     }
 }

www/docs/admin/reporting_stats.php

Spacing +2 added lines, -2 removed lines

@@ -7,7 +7,7 @@ 
 
 $PAGE->page_start();
 $PAGE->stripe_start();
-$PAGE->block_start(array ('id'=>'alerts', 'title'=>'Reporting Statistics'));
+$PAGE->block_start(array('id'=>'alerts', 'title'=>'Reporting Statistics'));
 $db = new ParlDB;
 $q = $db->query('select year(created) as the_year, month(created) as the_month, count(*) as c from alerts where confirmed and not deleted group by year(created) desc, month(created) desc');
 print '<h3>Alert signups per month</h3> <table>';
@@ -16,7 +16,7 @@ 
 print '<thead><tr><th>Year</th><th style="padding-right: 15px;">Month</th><th>Sign Ups</th><tr></thead>';
 
 $row_count = $q->rows();
-for ($i=0; $i<$row_count; $i++) {
+for ($i = 0; $i < $row_count; $i++) {
     $year = $q->field($i, 'the_year');
     $month = $q->field($i, 'the_month');
     $count = $q->field($i, 'c');