skqr/hateoas

Method __construct has 7 arguments (exceeds 4 allowed). Consider refactoring.

Open

        JsonCoder $jsonCoder,
        Raml\DocNavigator $docNavigator,
        ResourceLinksHydrant $hydrant,
        CreateBodyParser $creationBodyParser,
        UpdateBodyParser $mutationBodyParser,

The class BodyParser has a coupling between objects value of 24. Consider to reduce the number of dependencies under 13.

Open

class BodyParser
{
    const ERROR_PRIMARY_TYPE_KEY = "The resource type key is missing from the body.",
        ERROR_MISSING_SCHEMA = "A RAML schema was expected for the current action upon the resource \"%s\".",
        ERROR_MALFORMED_SCHEMA = "The RAML schema for the current action is missing the primary type key, \"%s\".",

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo {
    /**
     * @var \foo\bar\X
     */
    private $x = null;

    /**
     * @var \foo\bar\Y
     */
    private $y = null;

    /**
     * @var \foo\bar\Z
     */
    private $z = null;

    public function setFoo(\Foo $foo) {}
    public function setBar(\Bar $bar) {}
    public function setBaz(\Baz $baz) {}

    /**
     * @return \SplObjectStorage
     * @throws \OutOfRangeException
     * @throws \InvalidArgumentException
     * @throws \ErrorException
     */
    public function process(\Iterator $it) {}

    // ...
}

Avoid using static access to class '\GoIntegro\Hateoas\Util\ArrayHelper' in method 'prepareData'.

Open

            $json = Util\ArrayHelper::toObject($data);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

There must be one USE keyword per declaration

Open

use GoIntegro\Hateoas\JsonApi\JsonApiSpec,

• Exclude checks
  • Disable engine
  • Disable check

The closing parenthesis and the opening brace of a multi-line function declaration must be on the same line

Open

    {

• Exclude checks
  • Disable engine
  • Disable check

The closing parenthesis and the opening brace of a multi-line function declaration must be on the same line

Open

    {

• Exclude checks
  • Disable engine
  • Disable check

TRUE, FALSE and NULL must be lowercase; expected "null" but found "NULL"

Open

        $data = NULL;

• Exclude checks
  • Disable engine
  • Disable check

The closing parenthesis and the opening brace of a multi-line function declaration must be on the same line

Open

    {

• Exclude checks
  • Disable engine
  • Disable check

Multi-line function declarations must define one parameter per line

Open

        Params $params, $schema, array &$entityData

• Exclude checks
  • Disable engine
  • Disable check

The closing parenthesis and the opening brace of a multi-line function declaration must be on the same line

Open

    {

• Exclude checks
  • Disable engine
  • Disable check

TRUE, FALSE and NULL must be lowercase; expected "null" but found "NULL"

Open

        $schema = NULL;

• Exclude checks
  • Disable engine
  • Disable check

Multi-line function declarations must define one parameter per line

Open

        Request $request, Params $params

• Exclude checks
  • Disable engine
  • Disable check

Multi-line function declarations must define one parameter per line

Open

        Request $request, Params $params

• Exclude checks
  • Disable engine
  • Disable check

Multi-line function declarations must define one parameter per line

Open

        Params $params, $schema, array &$entityData

• Exclude checks
  • Disable engine
  • Disable check

Only one argument is allowed per line in a multi-line function call

Open

                self::ERROR_MALFORMED_SCHEMA, $params->primaryType

• Exclude checks
  • Disable engine
  • Disable check

Only one argument is allowed per line in a multi-line function call

Open

                    $request, $params, $body

• Exclude checks
  • Disable engine
  • Disable check

Only one argument is allowed per line in a multi-line function call

Open

            $method, '/' . $params->primaryType

• Exclude checks
  • Disable engine
  • Disable check

Only one argument is allowed per line in a multi-line function call

Open

                    $request, $params, $body

• Exclude checks
  • Disable engine
  • Disable check

Only one argument is allowed per line in a multi-line function call

Open

                    $request, $params, $body

• Exclude checks
  • Disable engine
  • Disable check

Only one argument is allowed per line in a multi-line function call

Open

                self::ERROR_MISSING_SCHEMA, $params->primaryType

• Exclude checks
  • Disable engine
  • Disable check

Only one argument is allowed per line in a multi-line function call

Open

                    $params, Raml\RamlSpec::HTTP_POST

• Exclude checks
  • Disable engine
  • Disable check

Only one argument is allowed per line in a multi-line function call

Open

                    $params, Raml\RamlSpec::HTTP_PUT

• Exclude checks
  • Disable engine
  • Disable check

Only one argument is allowed per line in a multi-line function call

Open

                    $request, $params, $body

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 122 characters

Open

        ERROR_UNSUPPORTED_CONTENT_TYPE = "The expected content type is \"%s\". The content type \"%s\" is not supported.";

• Exclude checks
  • Disable engine
  • Disable check

Expected 0 spaces after opening bracket; newline found

Open

        } elseif (

• Exclude checks
  • Disable engine
  • Disable check