Merge pull request #12 from iMi-digital/11-fix-route

#11 Fix SID appending for to/previous

README.md

@@ -30,8 +30,7 @@ To use SessionIDs in URLs add the middleware `urlsession` (if you registered the
 
 URLs generated with Laravel's URL function (for example `URL::to()`) will now have a session ID appended. 
 
-Deprecated (use route groups instead):
-
+If direct path's are or the `previous()` method is used, the SID is always added, because no route is known.
 If you would like to generate URLs without a session ID, add a `NO_ADD_SID` parameter:
 
     {{ URL::to('/', ['NO_ADD_SID' => true]) }}

src/UrlGeneratorService.php

@@ -10,14 +10,15 @@ class UrlGeneratorService extends UrlGenerator
     public function addSid($url, ?\Illuminate\Routing\Route $route = null)
     {
         // Only apply transsid to routes/routegroups with the urlsession middleware.
-        if ($route === null || !is_array($route->getAction('middleware')) || !in_array(UrlSession::class, $route->getAction('middleware'))) {
+        // or if no route is known (for example whne processing paths)
+        if ($route !== null && ( !is_array($route->getAction('middleware')) || !in_array(UrlSession::class, $route->getAction('middleware')) )) {
             return $url;
         }
 
         if (strpos($url, \Config::get('session.cookie')) !== false) {
             return $url;
         }
-        
+
         $separator = (strpos($url, '?') !== false) ? '&' : '?';
         $url .= $separator . \Config::get('session.cookie') . '=' . \Session::getId();