mysociety / theyworkforyou

classes/MiniSurvey.php

Spacing +11 added lines, -11 removed lines

@@ -2,8 +2,8 @@ 
 
 namespace MySociety\TheyWorkForYou;
 
-require_once INCLUDESPATH . "../../commonlib/phplib/random.php";
-require_once INCLUDESPATH . "../../commonlib/phplib/auth.php";
+require_once INCLUDESPATH."../../commonlib/phplib/random.php";
+require_once INCLUDESPATH."../../commonlib/phplib/auth.php";
 
 class MiniSurvey {
 
@@ -25,10 +25,10 @@ 
         if ($hide_question) {
             $always_ask = 0;
             $show_survey_qn = $current_question;
-            setcookie('survey', $current_question, time()+60*60*24*365, '/');
+            setcookie('survey', $current_question, time() + 60 * 60 * 24 * 365, '/');
         } elseif ($has_answered_question == $current_question && !$always_ask) {
             $show_survey_qn = $current_question;
-            setcookie('survey', $current_question, time()+60*60*24*365, '/');
+            setcookie('survey', $current_question, time() + 60 * 60 * 24 * 365, '/');
         } elseif (isset($_COOKIE['survey'])) {
             $show_survey_qn = $_COOKIE['survey'];
         }
@@ -38,18 +38,18 @@ 
 
             $page_url = '';
             $hide_url = '';
-            if ( in_array( $this_page, array('mp', 'peer', 'msp', 'mla', 'royal') ) ) {
+            if (in_array($this_page, array('mp', 'peer', 'msp', 'mla', 'royal'))) {
                 global $MEMBER;
-                if ( $MEMBER ) {
-                    $page_url = $MEMBER->url(true) . "?answered_survey=$current_question";
-                    $hide_url = $MEMBER->url() . "?hide_survey=$current_question";
+                if ($MEMBER) {
+                    $page_url = $MEMBER->url(true)."?answered_survey=$current_question";
+                    $hide_url = $MEMBER->url()."?hide_survey=$current_question";
                 }
             } else {
                 $URL = new \URL($this_page);
-                $URL->insert(array('answered_survey' => $current_question ));
-                $page_url = 'https://' . DOMAIN . $URL->generate();
+                $URL->insert(array('answered_survey' => $current_question));
+                $page_url = 'https://'.DOMAIN.$URL->generate();
                 $URL = new \URL($this_page);
-                $URL->insert(array('hide_survey' => $current_question ));
+                $URL->insert(array('hide_survey' => $current_question));
                 $hide_url = $URL->generate();
             }
 

classes/Search/ParseArgs.php

Spacing +7 added lines, -7 removed lines

@@ -13,11 +13,11 @@ 
         $searchstring = '';
 
         if ($advphrase = get_http_var('phrase')) {
-            $searchstring .= ' "' . $advphrase . '"';
+            $searchstring .= ' "'.$advphrase.'"';
         }
 
         if ($advexclude = get_http_var('exclude')) {
-            $searchstring .= ' -' . join(' -', preg_split('/\s+/', $advexclude));
+            $searchstring .= ' -'.join(' -', preg_split('/\s+/', $advexclude));
         }
 
         return $searchstring;
@@ -81,12 +81,12 @@ 
             $q = \MySociety\TheyWorkForYou\Utility\Search::searchMemberDbLookup($searchspeaker);
             $pids = array();
             $row_count = $q->rows();
-            for ($i=0; $i<$row_count; $i++) {
+            for ($i = 0; $i < $row_count; $i++) {
                 $pids[$q->field($i, 'person_id')] = true;
             }
             $pids = array_keys($pids);
             if (count($pids) > 0) {
-                $searchstring .= ' speaker:' . join(' speaker:', $pids);
+                $searchstring .= ' speaker:'.join(' speaker:', $pids);
             }
         }
 
@@ -97,18 +97,18 @@ 
         $searchstring = '';
 
         if ($advdept = get_http_var('department')) {
-            $searchstring .= ' department:' . preg_replace('#[^a-z]#i', '', $advdept);
+            $searchstring .= ' department:'.preg_replace('#[^a-z]#i', '', $advdept);
         }
 
         if ($advparty = get_http_var('party')) {
-            $searchstring .= ' party:' . join(' party:', explode(',', $advparty));
+            $searchstring .= ' party:'.join(' party:', explode(',', $advparty));
         }
 
         $advsection = get_http_var('section');
         if (!$advsection)
             $advsection = get_http_var('maj'); # Old URLs had this
         if (is_array($advsection)) {
-            $searchstring .= ' section:' . join(' section:', $advsection);
+            $searchstring .= ' section:'.join(' section:', $advsection);
         } elseif ($advsection) {
             $searchstring .= " section:$advsection";
         }

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 = '1935-10-01';
+            if ($from) {
+                $from = $from['iso'];
+            } else {
+                $from = '1935-10-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) {

classes/Topic.php

Indentation +29 added lines, -29 removed lines

@@ -33,7 +33,7 @@ 
         $this->db = new \ParlDB;
 
         if (is_null($data)) {
-          return;
+            return;
         }
 
         $this->id = $data['id'];
@@ -124,7 +124,7 @@ 
 
     private function _getContentIDs() {
         $q = $this->db->query(
-          "SELECT body, gid, ep.epobject_id FROM epobject ep
+            "SELECT body, gid, ep.epobject_id FROM epobject ep
            JOIN hansard h on ep.epobject_id = h.epobject_id
            JOIN topic_epobjects te on te.epobject_id = ep.epobject_id
            WHERE topic_key = :topic_key",
@@ -180,24 +180,24 @@ 
 
     function addContent($gid) {
         $q = $this->db->query(
-          "SELECT epobject_id FROM hansard WHERE gid = :gid",
-          array(
+            "SELECT epobject_id FROM hansard WHERE gid = :gid",
+            array(
             ":gid" => $gid
-          )
+            )
         );
 
         if (!$q->success() || $q->rows == 0) {
-          return false;
+            return false;
         }
 
         $epobject_id = $q->field(0, 'epobject_id');
 
         $q = $this->db->query(
-          "INSERT INTO topic_epobjects (topic_key, epobject_id) VALUES (:topic, :ep_id)",
-          array(
+            "INSERT INTO topic_epobjects (topic_key, epobject_id) VALUES (:topic, :ep_id)",
+            array(
             ":topic" => $this->id,
             ":ep_id" => $epobject_id
-          )
+            )
         );
 
         return $q->success();
@@ -205,31 +205,31 @@ 
 
     function deleteContent($id) {
         $q = $this->db->query(
-          "DELETE FROM topic_epobjects WHERE topic_key = :topic AND epobject_id = :ep_id",
-          array(
+            "DELETE FROM topic_epobjects WHERE topic_key = :topic AND epobject_id = :ep_id",
+            array(
             ":topic" => $this->id,
             ":ep_id" => $id
-          )
+            )
         );
 
         return $q->success();
     }
 
     function getPolicySets() {
-      $q = $this->db->query(
+        $q = $this->db->query(
         "SELECT policyset FROM topic_policysets WHERE topic_key = :key",
         array(
-          ':key' => $this->id
+            ':key' => $this->id
         )
-      );
+        );
 
-      $sets = array();
-      $count = $q->rows;
-      for ($i = 0; $i < $count; $i++) {
+        $sets = array();
+        $count = $q->rows;
+        for ($i = 0; $i < $count; $i++) {
         $sets[] = $q->field($i, 'policyset');
-      }
+        }
 
-      return $sets;
+        return $sets;
     }
 
     function addPolicySets($sets) {
@@ -259,20 +259,20 @@ 
     }
 
     function getPolicies() {
-      $q = $this->db->query(
+        $q = $this->db->query(
         'SELECT policy_id FROM topic_policies WHERE topic_key = :key',
         array(
-          ':key' => $this->id
+            ':key' => $this->id
         )
-      );
+        );
 
-      $policies = array();
-      $count = $q->rows;
-      for ($i = 0; $i < $count; $i++) {
+        $policies = array();
+        $count = $q->rows;
+        for ($i = 0; $i < $count; $i++) {
         $policies[] = $q->field($i, 'policy_id');
-      }
+        }
 
-      return $policies;
+        return $policies;
     }
 
     function addPolicies($policies) {
@@ -316,7 +316,7 @@ 
 
     function save() {
         $q = $this->db->query(
-          "REPLACE INTO topics
+            "REPLACE INTO topics
           (id, title, slug, description, search_string, front_page, image)
           VALUES
           (:id, :title, :slug, :description, :search_string, :front_page, :image)",

Spacing +5 added lines, -5 removed lines

@@ -53,7 +53,7 @@ 
 
     function sctitle() {
         $title = $this->title;
-        if (strpos($title, 'The ') === 0 ) {
+        if (strpos($title, 'The ') === 0) {
             $title = lcfirst($title);
         }
 
@@ -74,7 +74,7 @@ 
 
     function url() {
         $url = new \URL('topic');
-        return $url->generate() . $this->slug;
+        return $url->generate().$this->slug;
     }
 
     function image() {
@@ -82,7 +82,7 @@ 
     }
 
     function image_url() {
-        return "/topic/image.php?id=" . $this->slug();
+        return "/topic/image.php?id=".$this->slug();
     }
 
     function image_path() {
@@ -242,7 +242,7 @@ 
             );
         } else {
             foreach ($sets as $set) {
-                if ($set == '' ) {
+                if ($set == '') {
                     continue;
                 }
                 $q = $this->db->query(
@@ -285,7 +285,7 @@ 
             );
         } else {
             foreach ($policies as $policy) {
-                if ($policy == '' ) {
+                if ($policy == '') {
                     continue;
                 }
                 $q = $this->db->query(

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

@@ -28,7 +28,7 @@
      *
      */
 
-    public function __construct($data = NULL)
+    public function __construct($data = null)
     {
         $this->db = new \ParlDB;
 

classes/SectionView/SpwransView.php

Spacing +25 added lines, -25 removed lines

@@ -21,7 +21,7 @@ 
         $this_page = 'spwransfront';
         $data = array();
 
-        $args = array( 'months' => 1 );
+        $args = array('months' => 1);
         $WRANSLIST = new \SPWRANSLIST;
 
         $wrans = array();
@@ -43,7 +43,7 @@ 
 
     protected function getSearchSections() {
         return array(
-            array( 'section' => 'spwrans' )
+            array('section' => 'spwrans')
         );
     }
 
@@ -54,29 +54,29 @@ 
         $SPWRANSLIST = new \SPWRANSLIST;
         $gid = $SPWRANSLIST->get_gid_from_spid($spid);
         if ($gid) {
-            if (preg_match('/uk\.org\.publicwhip\/spwa\/(\d{4}-\d\d-\d\d\.(.*))/',$gid,$m)) {
+            if (preg_match('/uk\.org\.publicwhip\/spwa\/(\d{4}-\d\d-\d\d\.(.*))/', $gid, $m)) {
                 $URL = new \URL('spwrans');
                 $URL->reset();
-                $URL->insert( array('id' => $m[1]) );
-                $fragment_identifier = '#g' . $m[2];
-                header('Location: ' . $URL->generate('none') . $fragment_identifier, true, 303);
+                $URL->insert(array('id' => $m[1]));
+                $fragment_identifier = '#g'.$m[2];
+                header('Location: '.$URL->generate('none').$fragment_identifier, true, 303);
                 exit;
-            } elseif (preg_match('/uk\.org\.publicwhip\/spor\/(\d{4}-\d\d-\d\d\.(.*))/',$gid,$m)) {
+            } elseif (preg_match('/uk\.org\.publicwhip\/spor\/(\d{4}-\d\d-\d\d\.(.*))/', $gid, $m)) {
                 $URL = new \URL('spdebates');
                 $URL->reset();
-                $URL->insert( array('id' => $m[1]) );
-                $fragment_identifier = '#g' . $m[2];
-                header('Location: ' . $URL->generate('none') . $fragment_identifier, true, 303);
+                $URL->insert(array('id' => $m[1]));
+                $fragment_identifier = '#g'.$m[2];
+                header('Location: '.$URL->generate('none').$fragment_identifier, true, 303);
                 exit;
             } else {
-                $PAGE->error_message ("Strange GID ($gid) for that Scottish Parliament ID.");
+                $PAGE->error_message("Strange GID ($gid) for that Scottish Parliament ID.");
             }
         }
-        $PAGE->error_message ("Couldn't match that Scottish Parliament ID to a GID.");
+        $PAGE->error_message("Couldn't match that Scottish Parliament ID to a GID.");
     }
 
     protected function get_question_mentions_html($row_data) {
-        if( count($row_data) == 0 ) {
+        if (count($row_data) == 0) {
             return '';
         }
         $result = '';
@@ -85,19 +85,19 @@ 
         $first_difference_output = TRUE;
         // Keep the references until after the history that's in a timeline:
         $references = array();
-        for( $i = 0; $i < $nrows; $i++ ) {
+        for ($i = 0; $i < $nrows; $i++) {
             $row = $row_data[$i];
-            if( ! $row["date"] ) {
+            if (!$row["date"]) {
                 // If this mention isn't associated with a date, the difference won't be interesting.
                 $last_date = NULL;
             }
             $description = '';
             if ($last_date && ($last_date != $row["date"])) {
                 // Calculate how long the gap was in days:
-                $daysdiff = (integer)((strtotime($row["date"]) - strtotime($last_date)) / 86400);
+                $daysdiff = (integer) ((strtotime($row["date"]) - strtotime($last_date)) / 86400);
                 $daysstring = ($daysdiff == 1) ? "day" : "days";
                 $further = "";
-                if( $first_difference_output ) {
+                if ($first_difference_output) {
                     $first_difference_output = FALSE;
                 } else {
                     $further = " a further";
@@ -108,7 +108,7 @@ 
             $inner = "BUG: Unknown mention type $row[type]";
             $date = format_date($row['date'], SHORTDATEFORMAT);
             $url = $row['url'];
-            if ( strpos($url, 'business/businessBulletin') !== FALSE ) {
+            if (strpos($url, 'business/businessBulletin') !== FALSE) {
                 $url = str_replace('www.scottish', 'archive.scottish', $url);
             }
             switch ($row["type"]) {
@@ -122,9 +122,9 @@ 
                     $inner = "Mentioned in <a class=\"debate-speech__meta__link\" href=\"$url\">tabled written questions on $date</a>";
                     break;
                 case 4:
-                    if( preg_match('/^uk.org.publicwhip\/spq\/(.*)$/',$row['gid'],$m) ) {
+                    if (preg_match('/^uk.org.publicwhip\/spq\/(.*)$/', $row['gid'], $m)) {
                         $URL = new \URL("spwrans");
-                        $URL->insert( array('spid' => $m[1]) );
+                        $URL->insert(array('spid' => $m[1]));
                         $relative_url = $URL->generate("none");
                         $inner = "Given a <a class=\"debate-speech__meta__link\" href=\"$relative_url\">written answer on $date</a>";
                     }
@@ -133,25 +133,25 @@ 
                     $inner = "Given a holding answer on $date";
                     break;
                 case 6:
-                    if( preg_match('/^uk.org.publicwhip\/spor\/(.*)$/',$row['mentioned_gid'],$m) ) {
+                    if (preg_match('/^uk.org.publicwhip\/spor\/(.*)$/', $row['mentioned_gid'], $m)) {
                         $URL = new \URL("spdebates");
-                        $URL->insert( array('id' => $m[1]) );
+                        $URL->insert(array('id' => $m[1]));
                         $relative_url = $URL->generate("none");
                         $inner = "<a href=\"$relative_url\">Asked in parliament on $date</a>";
                     }
                     break;
                 case 7:
-                    if( preg_match('/^uk.org.publicwhip\/spq\/(.*)$/',$row['mentioned_gid'],$m) ) {
+                    if (preg_match('/^uk.org.publicwhip\/spq\/(.*)$/', $row['mentioned_gid'], $m)) {
                         $referencing_spid = $m[1];
                         $URL = new \URL("spwrans");
-                        $URL->insert( array('spid' => $referencing_spid) );
+                        $URL->insert(array('spid' => $referencing_spid));
                         $relative_url = $URL->generate("none");
                         $inner = "Referenced in <a href=\"$relative_url\">question $referencing_spid</a>";
                         $reference = TRUE;
                     }
                     break;
             }
-            if( $reference ) {
+            if ($reference) {
                 $references[] = "\n<li>$inner.";
             } else {
                 $result .= "\n<li class=\"link-to-hansard\">$description$inner</span>";

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

@@ -81,15 +81,15 @@ 
         }
         $result = '';
         $nrows = count($row_data);
-        $last_date = NULL;
-        $first_difference_output = TRUE;
+        $last_date = null;
+        $first_difference_output = true;
         // Keep the references until after the history that's in a timeline:
         $references = array();
         for( $i = 0; $i < $nrows; $i++ ) {
             $row = $row_data[$i];
             if( ! $row["date"] ) {
                 // If this mention isn't associated with a date, the difference won't be interesting.
-                $last_date = NULL;
+                $last_date = null;
             }
             $description = '';
             if ($last_date && ($last_date != $row["date"])) {
@@ -98,17 +98,17 @@ 
                 $daysstring = ($daysdiff == 1) ? "day" : "days";
                 $further = "";
                 if( $first_difference_output ) {
-                    $first_difference_output = FALSE;
+                    $first_difference_output = false;
                 } else {
                     $further = " a further";
                 }
                 $description = "\n<span class=\"question-mention-gap\">After$further $daysdiff $daysstring,</span> ";
             }
-            $reference = FALSE;
+            $reference = false;
             $inner = "BUG: Unknown mention type $row[type]";
             $date = format_date($row['date'], SHORTDATEFORMAT);
             $url = $row['url'];
-            if ( strpos($url, 'business/businessBulletin') !== FALSE ) {
+            if ( strpos($url, 'business/businessBulletin') !== false ) {
                 $url = str_replace('www.scottish', 'archive.scottish', $url);
             }
             switch ($row["type"]) {
@@ -147,7 +147,7 @@ 
                         $URL->insert( array('spid' => $referencing_spid) );
                         $relative_url = $URL->generate("none");
                         $inner = "Referenced in <a href=\"$relative_url\">question $referencing_spid</a>";
-                        $reference = TRUE;
+                        $reference = true;
                     }
                     break;
             }

classes/SectionView/SpView.php

Spacing +1 added lines, -1 removed lines

@@ -24,7 +24,7 @@
 
     protected function getSearchSections() {
         return array(
-            array( 'section' => 'sp' )
+            array('section' => 'sp')
         );
     }
 }

classes/SectionView/WransView.php

Spacing +6 added lines, -6 removed lines

@@ -12,13 +12,13 @@ 
 
     protected function display_front() {
         global $DATA, $this_page;
-        if ( get_http_var('type') == 'wrans') {
+        if (get_http_var('type') == 'wrans') {
             return parent::display_front();
         }
 
         $data = array();
 
-        $args = array( 'months' => 1 );
+        $args = array('months' => 1);
         $WRANSLIST = new \WRANSLIST;
 
         $wrans = array();
@@ -50,10 +50,10 @@ 
 
     protected function getSearchSections() {
         $sections = array(
-            array( 'section' => 'wrans', 'title' => 'Written Answers' ),
+            array('section' => 'wrans', 'title' => 'Written Answers'),
         );
-        if ( get_http_var('type') == '') {
-            $sections[] = array( 'section' => 'wms', 'title' => 'Written Ministerial Statements' );
+        if (get_http_var('type') == '') {
+            $sections[] = array('section' => 'wms', 'title' => 'Written Ministerial Statements');
         }
         return $sections;
     }
@@ -62,7 +62,7 @@ 
     # speech bar the first (assuming that's the question)
     private $votelinks_so_far = 0;
 
-    protected function generate_votes ($votes, $id, $gid) {
+    protected function generate_votes($votes, $id, $gid) {
         /*
         Returns HTML for the 'Does this answer the question?' links (wrans) in the sidebar.
         $votes = => array (

classes/SectionView/PbcView.php

Spacing +6 added lines, -6 removed lines

@@ -42,12 +42,12 @@ 
 
         if ($bill_id && $id) {
             return $this->display_section_or_speech(array(
-                'gid' => $standingprefix . $id,
+                'gid' => $standingprefix.$id,
             ));
         } elseif ($bill_id) {
             # Display the page for a particular bill
             $this_page = 'pbc_bill';
-            $args = array (
+            $args = array(
                 'id' => $bill_id,
                 'title' => $this->bill,
                 'session' => $this->session,
@@ -60,13 +60,13 @@ 
         } elseif ($this->session && $this->bill) {
             # Illegal bill title, redirect to session page
             $URL = new \URL('pbc_session');
-            header('Location: ' . $URL->generate() . urlencode($this->session));
+            header('Location: '.$URL->generate().urlencode($this->session));
             exit;
         } elseif ($this->session) {
             # Display the bills for a particular session
             $this_page = 'pbc_session';
             $DATA->set_page_metadata($this_page, 'title', "Session $this->session");
-            $args = array (
+            $args = array(
                 'session' => $this->session,
             );
             $data = array();
@@ -88,12 +88,12 @@ 
 
     protected function getSearchSections() {
         return array(
-            array( 'section' => 'pbc' )
+            array('section' => 'pbc')
         );
     }
 
     protected function front_content() {
-        return $this->list->display( 'recent_pbc_debates', array( 'num' => 50 ), 'none' );
+        return $this->list->display('recent_pbc_debates', array('num' => 50), 'none');
     }
 
     protected function display_front() {

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

@@ -65,7 +65,7 @@
         } elseif ($this->session) {
             # Display the bills for a particular session
             $this_page = 'pbc_session';
-            $DATA->set_page_metadata($this_page, 'title', "Session $this->session");
+            $DATA->set_page_metadata($this_page, 'title', "session $this->session");
             $args = array (
                 'session' => $this->session,
             );

classes/SectionView/DebatesView.php

Spacing +11 added lines, -11 removed lines

@@ -16,7 +16,7 @@ 
         $DEBATELIST = new \DEBATELIST;
         $debates = array();
         $debates['data'] = $DEBATELIST->display('biggest_debates', array('days'=>7, 'num'=>10), 'none');
-        $args = array( 'months' => 1 );
+        $args = array('months' => 1);
         $debates['calendar'] = $DEBATELIST->display('calendar', $args, 'none');
         $debates['rssurl'] = $DATA->page_metadata($this_page, 'rss');
 
@@ -26,7 +26,7 @@ 
 
         $WHALLLIST = new \WHALLLIST;
         $whall['data'] = $WHALLLIST->display('biggest_debates', array('days'=>7, 'num'=>10), 'none');
-        $args = array( 'months' => 1 );
+        $args = array('months' => 1);
         $whall['calendar'] = $WHALLLIST->display('calendar', $args, 'none');
         $whall['rssurl'] = $DATA->page_metadata($this_page, 'rss');
 
@@ -36,7 +36,7 @@ 
 
         $LORDSDEBATELIST = new \LORDSDEBATELIST;
         $lords['data'] = $LORDSDEBATELIST->display('biggest_debates', array('days'=>7, 'num'=>10), 'none');
-        $args = array( 'months' => 1 );
+        $args = array('months' => 1);
         $lords['calendar'] = $LORDSDEBATELIST->display('calendar', $args, 'none');
 
         $lords['rssurl'] = $DATA->page_metadata($this_page, 'rss');
@@ -55,9 +55,9 @@ 
 
     protected function getSearchSections() {
         return array(
-            array( 'section' => 'debates', 'title' => 'House of Commons' ),
-            array( 'section' => 'lords', 'title' => 'House of Lords' ),
-            array( 'section' => 'whall', 'title' => 'Westminster Hall' )
+            array('section' => 'debates', 'title' => 'House of Commons'),
+            array('section' => 'lords', 'title' => 'House of Lords'),
+            array('section' => 'whall', 'title' => 'Westminster Hall')
         );
     }
 
@@ -69,11 +69,11 @@ 
         if (!$this->first_gid) $this->first_gid = $row['gid'];
 
         $video_content = '';
-        if (!$this->first_video_displayed && $row['video_status']&4 && !($row['video_status']&8)) {
+        if (!$this->first_video_displayed && $row['video_status'] & 4 && !($row['video_status'] & 8)) {
             $video_content = $this->video_sidebar($row, $heading_hpos, $speeches);
             $this->first_video_displayed = true;
         }
-        if (!$video_content && !$this->first_speech_displayed && $row['video_status']&1 && !($row['video_status']&12)) {
+        if (!$video_content && !$this->first_speech_displayed && $row['video_status'] & 1 && !($row['video_status'] & 12)) {
             $video_content = $this->video_advert($row);
             $this->first_speech_displayed = true;
         }
@@ -104,11 +104,11 @@ 
         if ($count > 1) {
             $out .= '<div class="debate__video" id="video_wrap"><div>';
             if ($row['gid'] != $this->first_gid) {
-                $out .= '<p class="video-instructions">This video starts around ' . ($row['hpos']-$heading_hpos) . ' speeches in (<a href="#g' . gid_to_anchor($row['gid']) . '">move there in text</a>)</p>';
+                $out .= '<p class="video-instructions">This video starts around '.($row['hpos'] - $heading_hpos).' speeches in (<a href="#g'.gid_to_anchor($row['gid']).'">move there in text</a>)</p>';
             }
         }
         $out .= \MySociety\TheyWorkForYou\Utility\Video::object($video['id'], $start, "$gid_type/$row[gid]");
-        $flashvars = 'gid=' . "$gid_type/$row[gid]" . '&amp;file=' . $video['id'] . '&amp;start=' . $start;
+        $flashvars = 'gid='."$gid_type/$row[gid]".'&amp;file='.$video['id'].'&amp;start='.$start;
         $out .= "<strong>Embed this video</strong><p class='video-instructions'>Copy and paste this code on your website</p><input readonly onclick='this.focus();this.select();' type='text' name='embed' size='40' value=\"<embed src='https://www.theyworkforyou.com/video/parlvid.swf' width='320' height='230' allowfullscreen='true' allowscriptaccess='always' flashvars='$flashvars'></embed>\">";
         if ($count > 1) {
             $out .= '<p class="hide-video"><a href="" onclick="return showVideo();">Hide</a></p>';
@@ -125,7 +125,7 @@ 
     <div style="border:solid 1px #9999ff; background-color: #ccccff; padding: 4px; text-align: center;
     background-image: url(\'/images/video-x-generic.png\'); background-repeat: no-repeat; padding-left: 40px;
     background-position: 0 2px; margin-bottom: 1em;">
-    Help us <a href="/video/?from=debate&amp;gid=' . $gid_type . '/' . $row['gid'] . '">match the video for this speech</a>
+    Help us <a href="/video/?from=debate&amp;gid=' . $gid_type.'/'.$row['gid'].'">match the video for this speech</a>
     to get the right video playing here
     </div>
     ';

Braces +6 added lines, -2 removed lines

@@ -66,7 +66,9 @@ 
     private $first_gid = '';
 
     protected function get_video_html($row, $heading_hpos, $speeches) {
-        if (!$this->first_gid) $this->first_gid = $row['gid'];
+        if (!$this->first_gid) {
+            $this->first_gid = $row['gid'];
+        }
 
         $video_content = '';
         if (!$this->first_video_displayed && $row['video_status']&4 && !($row['video_status']&8)) {
@@ -97,7 +99,9 @@ 
         $adate = $vq->field(0, 'adate');
         $time = $vq->field(0, 'atime');
         $videodb = \MySociety\TheyWorkForYou\Utility\Video::dbConnect();
-        if (!$videodb) return '';
+        if (!$videodb) {
+            return '';
+        }
         $video = \MySociety\TheyWorkForYou\Utility\Video::fromTimestamp($videodb, $adate, $time);
         $start = $video['offset'];
         $out = '';

classes/SectionView/NiView.php

Spacing +8 added lines, -8 removed lines

@@ -32,7 +32,7 @@ 
 
     protected function getSearchSections() {
         return array(
-            array( 'section' => 'ni' )
+            array('section' => 'ni')
         );
     }
 
@@ -52,23 +52,23 @@ 
 
         $debates = $DEBATELIST->display('recent_debates', array('days' => 30, 'num' => 6), 'none');
         $MOREURL = new \URL('nidebatesfront');
-        $MOREURL->insert( array( 'more' => 1 ) );
+        $MOREURL->insert(array('more' => 1));
 
         // this makes sure that we don't repeat this debate in the list below
         $random_debate = NULL;
-        if ( isset($debates['data']) && count($debates['data']) ) {
+        if (isset($debates['data']) && count($debates['data'])) {
             $random_debate = $debates['data'][0];
         }
 
         $recent = array();
-        if ( isset($debates['data']) && count($debates['data']) ) {
+        if (isset($debates['data']) && count($debates['data'])) {
             // at the start of a session there may be less than 6
             // debates
             $max = 6;
-            if ( count($debates['data']) < 6 ) {
+            if (count($debates['data']) < 6) {
                 $max = count($debates['data']);
             }
-            for ( $i = 1; $i < $max; $i++ ) {
+            for ($i = 1; $i < $max; $i++) {
                 $debate = $debates['data'][$i];
                 $debate['desc'] = "Northern Ireland Assembly debates";
                 $debate['more_url'] = $MOREURL->generate();
@@ -77,7 +77,7 @@ 
         }
 
         $featured = array();
-        if ( $random_debate ) {
+        if ($random_debate) {
             $featured = $random_debate;
             $featured['more_url'] = $MOREURL->generate();
             $featured['desc'] = 'Northern Ireland Assembly debate';
@@ -86,7 +86,7 @@ 
         }
 
         $data['featured'] = $featured;
-        $data['debates'] = array( 'recent' => $recent);
+        $data['debates'] = array('recent' => $recent);
 
         $data['regional'] = $this->getMLAList();
         $data['template'] = 'ni/index';

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

@@ -43,7 +43,7 @@ 
         $data = array();
         $urls = array();
 
-        $data['popular_searches'] = NULL;
+        $data['popular_searches'] = null;
 
 
         $data['urls'] = $this->getURLs($data);
@@ -55,7 +55,7 @@ 
         $MOREURL->insert( array( 'more' => 1 ) );
 
         // this makes sure that we don't repeat this debate in the list below
-        $random_debate = NULL;
+        $random_debate = null;
         if ( isset($debates['data']) && count($debates['data']) ) {
             $random_debate = $debates['data'][0];
         }