Vectorface\SnappyRouter\Exception\MethodNotAllowedException

Complexity

Total Complexity

4

Size/Duplication

Total Lines	32
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	0

Test Coverage

Coverage

100%

Importance

Changes	1
Bugs	0	Features	1

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	getAssociatedStatusCode()	0	7	3

<?php

namespace Vectorface\SnappyRouter\Exception;

use \Exception;
use Vectorface\SnappyRouter\Response\AbstractResponse;

/**
 * An exception indicating an invalid HTTP action was specified.
 * @copyright Copyright (c) 2014, VectorFace, Inc.
 * @author Dan Bruce <dbruce@vectorface.com>
 */
class MethodNotAllowedException extends Exception implements RouterExceptionInterface
{
    // as per RFC2616 (http://www.w3.org/Protocols/rfc2616/rfc2616-sec10.html#sec10.4.6)
    // we must specify a comma separated list of allowed methods if we return
    // a 405 response
    private $allowedMethods;

    /**
     * Constructor for the method.
     * @param string $message The error message string.
     * @param array $allowedMethods The array of methods that are allowed.
     */
    public function __construct($message, $allowedMethods)
    {
        parent::__construct($message);
        $this->allowedMethods = $allowedMethods;
    }

    /**
     * Returns the associated status code with the exception. By default, most exceptions correspond
     * to a server error (HTTP 500). Override this method if you want your exception to generate a
     * different status code.
     * @return The associated status code.
     */
    public function getAssociatedStatusCode()
    {
        if (!empty($this->allowedMethods) && is_array($this->allowedMethods)) {
            @header(sprintf('Allow: %s', implode(',', $this->allowedMethods)));

It seems like you do not handle an error condition here. This can introduce security issues, and is generally not recommended.

        }
        return AbstractResponse::RESPONSE_METHOD_NOT_ALLOWED;
    }
}