salted-herring / silverstripe-shortlist

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

tests/ShortlistTest.php

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