Ajde_View

Complexity

Total Complexity

6

Size/Duplication

Total Lines	37
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	3

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	fromController()	0	10	3
A	fromRoute()	0	13	3

<?php

class Ajde_View extends Ajde_Template
{
    /**
     * @param Ajde_Controller $controller
     *
     * @return Ajde_View
     */
    public static function fromController(Ajde_Controller $controller)
    {
        $base = MODULE_DIR.$controller->getModule().'/';
        $action = $controller->getRoute()->getController() ?
            $controller->getRoute()->getController().'/'.$controller->getAction() :
            $controller->getAction();
        $format = $controller->hasFormat() ? $controller->getFormat() : 'html';

The method hasFormat does not exist on object<Ajde_Controller>? Since you implemented __call, maybe consider adding a @method annotation.

        return new self($base, $action, $format);
    }

    /**
     * @param Ajde_Core_Route $route
     *
     * @return Ajde_View
     */
    public static function fromRoute($route)
    {
        if (!$route instanceof Ajde_Core_Route) {
            $route = new Ajde_Core_Route($route);
        }
        $base = MODULE_DIR.$route->getModule().'/';
        $action = $route->getController() ?
            $route->getController().'/'.$route->getAction() :
            $route->getAction();
        $format = $route->getFormat();

        return new self($base, $action, $format);
    }
}