salted-herring / silverstripe-shortlist

code/extensions/ShortListExtension.php

Indentation +34 added lines, -34 removed lines

@@ -12,15 +12,15 @@ 
      * */
     public function getShortlistURL($full = false)
     {
-	    if ($full) {
-		    $shortlist = $this->getShortList();
+        if ($full) {
+            $shortlist = $this->getShortList();
 
-	        if ($shortlist && $shortlist->exists()) {
-	            return $shortlist->Link();
-	        }
-	    } else {
-		    return Config::inst()->get('ShortList', 'URLSegment');
-	    }
+            if ($shortlist && $shortlist->exists()) {
+                return $shortlist->Link();
+            }
+        } else {
+            return Config::inst()->get('ShortList', 'URLSegment');
+        }
 
 
         return false;
@@ -31,39 +31,39 @@ 
      * */
     public function getSessionID()
     {
-		Session::start();
-		return session_id();
-	}
+        Session::start();
+        return session_id();
+    }
 
-	public function getShortList()
-	{
-		return DataObject::get_one('ShortList', $filter = array('SessionID' => session_id()));
-	}
+    public function getShortList()
+    {
+        return DataObject::get_one('ShortList', $filter = array('SessionID' => session_id()));
+    }
 
-	public function getShortListCount()
-	{
-		$shortlist = $this->getShortList();
+    public function getShortListCount()
+    {
+        $shortlist = $this->getShortList();
 
-		if ($shortlist && $shortlist->exists()) {
-			return $shortlist->ShortListItems()->Count();
-		}
+        if ($shortlist && $shortlist->exists()) {
+            return $shortlist->ShortListItems()->Count();
+        }
 
-		return 0;
-	}
+        return 0;
+    }
 
-	public function AddedToShortList($ID = null, $type = null) {
-		if (is_null($ID) || is_null($type)) {
-			return false;
-		}
+    public function AddedToShortList($ID = null, $type = null) {
+        if (is_null($ID) || is_null($type)) {
+            return false;
+        }
 
-		$shortlist = $this->getShortList();
+        $shortlist = $this->getShortList();
 
-		if ($shortlist) {
-			$items = $shortlist->ShortListItems()->filter(array('ItemID' => $ID, 'ItemType' => $type));
+        if ($shortlist) {
+            $items = $shortlist->ShortListItems()->filter(array('ItemID' => $ID, 'ItemType' => $type));
 
-			return $items->Count() > 0;
-		}
+            return $items->Count() > 0;
+        }
 
-		return 0;
-	}
+        return 0;
+    }
 }

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/controllers/ShortListController.php

Spacing +1 added lines, -1 removed lines

@@ -35,7 +35,7 @@
     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 {
             /*
 if (!ShortList::isBrowser()) {

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);