midgard_admin_asgard_handler

Complexity

Total Complexity

3

Size/Duplication

Total Lines	15
Duplicated Lines	0 %

Test Coverage

Coverage

85.71%

Importance

Changes	1
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
A	get_response()	0	10	3

<?php
/**
 * @package midgard.admin.asgard
 * @author CONTENT CONTROL http://www.contentcontrol-berlin.de/
 * @copyright CONTENT CONTROL http://www.contentcontrol-berlin.de/
 * @license http://www.gnu.org/licenses/gpl.html GNU General Public License
 */

/**
 * Style helper
 *
 * @package midgard.admin.asgard
 */
trait midgard_admin_asgard_handler
{
    /**
     * @return midcom_response_styled
     */
    public function get_response($element = null) : midcom_response_styled
    {
        if (isset($_GET['ajax'])) {
            midcom::get()->skip_page_style = true;
        }
        $this->populate_breadcrumb_line();

It seems like populate_breadcrumb_line() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        if ($element) {
            return $this->show($element, 'ASGARD_ROOT');

It seems like show() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        }
        return new midcom_response_styled(midcom_core_context::get(), 'ASGARD_ROOT');
    }
}