salted-herring / silverstripe-shortlist

tests/ShortlistTest.php

Indentation +2 added lines, -2 removed lines

@@ -4,8 +4,8 @@
 {
     protected static $fixture_file = 'fixtures.yml';
 
-	public function testShortlistCreation()
-	{
+    public function testShortlistCreation()
+    {
         $shortlists = ShortList::get();
 
         $this->get('shortlist');

Spacing +3 added lines, -3 removed lines

@@ -47,7 +47,7 @@ 
         Session::start();
         $sessionID = session_id();
 
-        $response = $this->get('shortlist/add?id=' . $testpage->ID . '&type=Page&s=' . $sessionID . '&output=0');
+        $response = $this->get('shortlist/add?id='.$testpage->ID.'&type=Page&s='.$sessionID.'&output=0');
 
         $shortlist = DataObject::get_one('ShortList', array('SessionID' => $sessionID));
 
@@ -61,11 +61,11 @@ 
         Session::start();
         $sessionID = session_id();
 
-        $this->get('shortlist/add?id=' . $testpage->ID . '&type=Page&s=' . $sessionID . '&output=0');
+        $this->get('shortlist/add?id='.$testpage->ID.'&type=Page&s='.$sessionID.'&output=0');
 
         $shortlist = DataObject::get_one('ShortList', array('SessionID' => $sessionID));
 
-        $this->get('shortlist/remove?id=' . $testpage->ID . '&type=Page&s=' . $sessionID . '&output=0');
+        $this->get('shortlist/remove?id='.$testpage->ID.'&type=Page&s='.$sessionID.'&output=0');
 
         $this->assertEquals($shortlist->ShortListItems()->Count(), 0);
     }

code/extensions/ShortListPaginationExtension.php

Indentation +1 added lines, -1 removed lines

@@ -5,7 +5,7 @@
  * Provides Pagination for shortlist controller
  */
 class ShortListPaginationExtension extends DataExtension {
-	/**
+    /**
      * Get a paginated list of the shortlist items.
      *
      * @return mixed the paginated list of items, or false if the list cannot be found.

Spacing +2 added lines, -2 removed lines

@@ -37,7 +37,7 @@ 
     public function nextPage()
     {
         if ($this->owner->list->CurrentPage() < $this->owner->list->TotalPages()) {
-            return '?page=' . ($this->owner->list->CurrentPage() + 1);
+            return '?page='.($this->owner->list->CurrentPage() + 1);
         }
 
         return false;
@@ -46,7 +46,7 @@ 
     public function prevPage()
     {
         if ($this->owner->list->CurrentPage() > 1) {
-            return '?page=' . ($this->owner->list->CurrentPage() - 1);
+            return '?page='.($this->owner->list->CurrentPage() - 1);
         }
 
         return false;

code/models/ShortList.php

Spacing +1 added lines, -1 removed lines

@@ -81,7 +81,7 @@
      */
     public function Link($action = null)
     {
-        return Config::inst()->get('ShortList', 'URLSegment') . $this->URL;
+        return Config::inst()->get('ShortList', 'URLSegment').$this->URL;
     }
 
     /**

code/factories/RemoveFromshortlistAction.php

Spacing +1 added lines, -1 removed lines

@@ -22,7 +22,7 @@
             return false;
         }
 
-        $query = "ItemType = '" . $type . "' AND ItemID = " . $ID . ' AND ShortListID = ' . $shortlist->ID;
+        $query = "ItemType = '".$type."' AND ItemID = ".$ID.' AND ShortListID = '.$shortlist->ID;
 
         $item = DataObject::get_one('ShortListItem', $filter = $query);
 

code/controllers/ShortListController.php

Spacing +2 added lines, -2 removed lines

@@ -36,12 +36,12 @@
     public function index($request)
     {
         if (($shortlist = $this->getSessionShortList())) {
-            return $this->redirect(Config::inst()->get('ShortList', 'URLSegment') . $shortlist->URL);
+            return $this->redirect(Config::inst()->get('ShortList', 'URLSegment').$shortlist->URL);
         } else {
             $CrawlerDetect = new CrawlerDetect;
 
             // Check the user agent of the current 'visitor'
-            if($CrawlerDetect->isCrawler()) {
+            if ($CrawlerDetect->isCrawler()) {
                 return $this->httpError(403);
             }