WebStream\Test\Controller\Test1Controller

Complexity

Total Complexity

9

Size/Duplication

Total Lines	61
Duplicated Lines	0 %

Importance

Changes

0

9 Methods

Rating	Name	Duplication	Size	Complexity
A	test3()	0	2	1
A	test9()	0	2	1
A	test2()	0	3	1
A	test4Error()	0	3	1
A	before()	0	3	1
A	test10()	0	2	1
A	after()	0	3	1
A	test4()	0	3	1
A	test1()	0	2	1

<?php
namespace WebStream\Test\Controller;

use WebStream\Core\CoreController;
use WebStream\Annotation\Attributes\ExceptionHandler;
use WebStream\Annotation\Attributes\Filter;
use WebStream\Annotation\Attributes\Header;
use WebStream\Annotation\Attributes\Template;
use WebStream\Annotation\Attributes\Custom\CustomControllerAnnotation;
use WebStream\Exception\Extend\ForbiddenAccessException;

class Test1Controller extends CoreController
{
    /**
     * @Filter(type="before")
     */
    public function before()
    {
        echo "b1";
    }

    /**
     * @Filter(type="after")
     */
    public function after()
    {
        echo "a1";
    }

    /**
     * @Template("test1.tmpl")
     */
    public function test1()
    {
    }

    public function test2()
    {
        $this->Test1->test1();

The method test1() does not exist on null.

The property Test1 does not exist on WebStream\Test\Controller\Test1Controller. Since you implemented __get, consider adding a @property annotation.

    }

    /**
     * @Header(allowMethod="post")
     */
    public function test3()
    {
    }

    public function test4()
    {
        throw new ForbiddenAccessException("error");
    }

    /**
     * @ExceptionHandler("WebStream\Exception\Extend\ForbiddenAccessException")
     */
    public function test4Error($params)
    {
        var_dump($params);

var_dump($params) looks like debug code. Are you sure you do not want to remove it?

    }


    public function test9()
    {

    }

    /**
     * @CustomControllerAnnotation(type="custom")
     */
    public function test10()
    {
    }
}