Comodojo\Dispatcher\Output\Redirect

Complexity

Total Complexity

7

Size/Duplication

Total Lines	41
Duplicated Lines	0 %

Test Coverage

Coverage

0%

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
B	compose()	0	39	7

<?php namespace Comodojo\Dispatcher\Output;

use \Comodojo\Dispatcher\Request\Model as Request;
use \Comodojo\Dispatcher\Response\Model as Response;
use \Comodojo\Dispatcher\Router\Route;

/**
 * @package     Comodojo Dispatcher
 * @author      Marco Giovinazzi <marco.giovinazzi@comodojo.org>
 * @author      Marco Castiello <marco.castiello@gmail.com>
 * @license     MIT
 *
 * LICENSE:
 *
 * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
 * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
 * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
 * AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
 * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
 * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
 * THE SOFTWARE.
 */

 class Redirect {

     public static function compose(
         Request $request,
         Response $response,
         Route $route
     ) {

         $status = $response->getStatus();
         $content = $response->getContent();
         $headers = $response->getHeaders();

         $code = $route->getRedirectCode();

         $location = $route->getRedirectLocation();
         $uri = empty($location) ? (string) $request->getUri() : $location;

         $message = $route->getRedirectMessage();

         if ( $route->getRedirectType() == Route::REDIRECT_REFRESH ) {

Operator == prohibited; use === instead

             $output = empty($message) ?
                 "Please follow <a href=\"$location\">this link</a>"

As per coding-style, please use concatenation or sprintf for the variable $location instead of interpolation.

                 : $message;

             $status->set(200);
             $headers->set("Refresh", "0;url=$location");

The string literal Refresh does not require double quotes, as per coding-style, please use single quotes.

As per coding-style, please use concatenation or sprintf for the variable $location instead of interpolation.

             $content->set($output);

         } else {

             if ( !empty($code) ) {
                 $status->set($code);
             } else if ( $request->getVersion() === 'HTTP/1.1' ) {

The condition $request->getVersion() === 'HTTP/1.1' is always false.

                 $status->set( (string) $request->getMethod() !== 'GET' ? 303 : 307);
             } else {
                 $status->set(302);
             }

             $content->set($message);
             $response->getLocation()->set($uri);

         }

     }

 }