Commit f79dd1ba authored by Taylor Otwell's avatar Taylor Otwell

finished refactoring of the redirect generator.

parent 9be3d1a5
...@@ -12,8 +12,6 @@ return array( ...@@ -12,8 +12,6 @@ return array(
{ {
$config = $container->resolve('laravel.config'); $config = $container->resolve('laravel.config');
$connections = $config->get('database.connections');
return new Database\Manager($config->get('database.connections'), $config->get('database.default')); return new Database\Manager($config->get('database.connections'), $config->get('database.default'));
}), }),
...@@ -24,28 +22,6 @@ return array( ...@@ -24,28 +22,6 @@ return array(
}), }),
'laravel.form' => array('singleton' => true, 'resolver' => function($container)
{
$request = $container->resolve('laravel.request');
$html = $container->resolve('laravel.html');
$url = $container->resolve('laravel.url');
$token = ($container->registered('laravel.session.driver')) ? $container->resolve('laravel.session.driver')->get('csrf_token') : null;
return new Form($request, $html, $url, $token);
}),
'laravel.html' => array('singleton' => true, 'resolver' => function($container)
{
$encoding = $container->resolve('laravel.config')->get('application.encoding');
return new HTML($container->resolve('laravel.url'), $encoding);
}),
'laravel.input' => array('singleton' => true, 'resolver' => function($container) 'laravel.input' => array('singleton' => true, 'resolver' => function($container)
{ {
$application = $container->resolve('laravel.application'); $application = $container->resolve('laravel.application');
...@@ -75,11 +51,9 @@ return array( ...@@ -75,11 +51,9 @@ return array(
}), }),
'laravel.responder' => array('singleton' => true, 'resolver' => function($container) 'laravel.redirect' => array('singleton' => true, 'resolver' => function($container)
{ {
require_once SYS_PATH.'response'.PHP; return new Redirect($container->resolve('laravel.url'));
return new Response_Factory($container->resolve('laravel.view'), $container->resolve('laravel.file'));
}), }),
...@@ -91,13 +65,11 @@ return array( ...@@ -91,13 +65,11 @@ return array(
'laravel.url' => array('singleton' => true, 'resolver' => function($container) 'laravel.url' => array('singleton' => true, 'resolver' => function($container)
{ {
$config = $container->resolve('laravel.config');
$request = $container->resolve('laravel.request'); $request = $container->resolve('laravel.request');
$base = $config->get('application.url'); $base = $container->resolve('laravel.config')->get('application.url');
$index = $config->get('application.index'); $index = $container->resolve('laravel.config')->get('application.index');
return new URL($container->resolve('laravel.router'), $base, $index, $request->secure); return new URL($container->resolve('laravel.router'), $base, $index, $request->secure);
}), }),
......
...@@ -3,18 +3,18 @@ ...@@ -3,18 +3,18 @@
class Redirect extends Response { class Redirect extends Response {
/** /**
* Create a redirect response. * Create a new redirect generator instance.
*
* <code>
* // Create a redirect for the "user/profile" URI
* return Redirect::to('user/profile');
* *
* // Create a redirect using the 301 status code * @param URL $url
* return Redirect::to('user/profile', 301); * @return void
* */
* // Create a redirect using the "refresh" method public function __construct(URL $url)
* return Redirect::to('user/profile', 302, 'refresh'); {
* </code> $this->url = $url;
}
/**
* Create a redirect response.
* *
* @param string $url * @param string $url
* @param int $status * @param int $status
...@@ -22,36 +22,33 @@ class Redirect extends Response { ...@@ -22,36 +22,33 @@ class Redirect extends Response {
* @param bool $https * @param bool $https
* @return Redirect * @return Redirect
*/ */
public static function to($url, $status = 302, $method = 'location', $https = false) public function to($url, $status = 302, $method = 'location', $https = false)
{ {
$url = URL::to($url, $https); $url = $this->url->to($url, $https);
parent::__construct('', $status);
if ($method == 'location') if ($method == 'location')
{ {
return parent::__construct('', $status)->header('Refresh', '0;url='.$url); return $this->header('Refresh', '0;url='.$url);
} }
else else
{ {
return parent::__construct('', $status)->header('Location', $url); return $this->header('Location', $url);
} }
} }
/** /**
* Create a redirect response to a HTTPS URL. * Create a redirect response to a HTTPS URL.
* *
* <code>
* // Create a HTTPS redirect to the "user/profile" URI
* return Redirect::to_secure('user/profile');
* </code>
*
* @param string $url * @param string $url
* @param int $status * @param int $status
* @param string $method * @param string $method
* @return Response * @return Response
*/ */
public static function to_secure($url, $status = 302, $method = 'location') public function to_secure($url, $status = 302, $method = 'location')
{ {
return static::to($url, $status, $method, true); return $this->to($url, $status, $method, true);
} }
/** /**
...@@ -59,11 +56,6 @@ class Redirect extends Response { ...@@ -59,11 +56,6 @@ class Redirect extends Response {
* *
* This is useful for passing status messages or other temporary data to the next request. * This is useful for passing status messages or other temporary data to the next request.
* *
* <code>
* // Flash a status message to the session on a redirect
* return Redirect::to('user/profile')->with('status', 'Welcome Back!');
* </code>
*
* @param string $key * @param string $key
* @param mixed $value * @param mixed $value
* @return Response * @return Response
...@@ -76,28 +68,20 @@ class Redirect extends Response { ...@@ -76,28 +68,20 @@ class Redirect extends Response {
} }
/** /**
* Magic Method to handle redirecting to named routes. * Magic Method to handle creation of redirects to named routes.
*
* <code>
* // Create a redirect to the "profile" route
* return Redirect::to_profile();
*
* // Create a redirect to the "profile" route using HTTPS
* return Redirect::to_secure_profile();
* </code>
*/ */
public static function __callStatic($method, $parameters) public function __call($method, $parameters)
{ {
$parameters = (isset($parameters[0])) ? $parameters[0] : array(); $parameters = (isset($parameters[0])) ? $parameters[0] : array();
if (strpos($method, 'to_secure_') === 0) if (strpos($method, 'to_secure_') === 0)
{ {
return static::to(URL::to_route(substr($method, 10), $parameters, true)); return $this->to($this->url->to_route(substr($method, 10), $parameters, true));
} }
if (strpos($method, 'to_') === 0) if (strpos($method, 'to_') === 0)
{ {
return static::to(URL::to_route(substr($method, 3), $parameters)); return $this->to($this->url->to_route(substr($method, 3), $parameters));
} }
throw new \Exception("Method [$method] is not defined on the Redirect class."); throw new \Exception("Method [$method] is not defined on the Redirect class.");
......
...@@ -178,7 +178,8 @@ class Response { ...@@ -178,7 +178,8 @@ class Response {
* *
* @param mixed $content * @param mixed $content
* @param int $status * @param int $status
*/ * @return void
*/
public function __construct($content, $status = 200) public function __construct($content, $status = 200)
{ {
$this->content = $content; $this->content = $content;
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment