midgard_admin_asgard_copytree

Complexity

Total Complexity

14

Size/Duplication

Total Lines	142
Duplicated Lines	0 %

Test Coverage

Coverage

86.96%

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	_is_collapsed()	0	6	2
B	_draw_element()	0	50	7
A	draw()	0	16	4

<?php
/**
 * @package midgard.admin.asgard
 * @author The Midgard Project, http://www.midgard-project.org
 * @copyright The Midgard Project, http://www.midgard-project.org
 * @license http://www.gnu.org/licenses/lgpl.html GNU Lesser General Public License
 */

/**
 * Copy/delete tree branch viewer
 *
 * @package midgard.admin.asgard
 */
class midgard_admin_asgard_copytree extends midgard_admin_asgard_navigation
{
    /**
     * Switch to determine if the whole tree should be copied
     *
     * @var boolean
     */
    public $copy_tree = false;

    /**
     * Switch to determine the visibility of inputs
     *
     * @var boolean
     */
    public $inputs = true;

    /**
     * Choose the target type
     *
     * @var String
     */
    public $input_type;

    /**
     * Choose the target name for the form
     *
     * @var String
     */
    public $input_name;

    /**
     * Show the link to view the object
     *
     * @var boolean
     */
    public $view_link = false;

    /**
     * Show the link to view the object
     *
     * @var boolean
     */
    public $edit_link = false;

    /**
     * Page prefix
     *
     * @var String
     */
    public $page_prefix = '';

    /**
     * Constructor, connect to the parent class constructor.
     *
     * @param mixed $object
     * @param array $request_data
     */
    public function __construct($object, &$request_data)
    {
        parent::__construct($object, $request_data);
        $this->page_prefix = midcom_core_context::get()->get_key(MIDCOM_CONTEXT_ANCHORPREFIX);

It seems like midcom_core_context::get()->get_key(MIDCOM_CONTEXT_ANCHORPREFIX) can also be of type false. However, the property $page_prefix is declared as type string. Maybe add an additional type check?

    }

    protected function _draw_element($object, string $label, int $level, bool $autoexpand = false)
    {
        $ref = midcom_helper_reflector_tree::get($object);
        $span_class = '';
        $css_class = $this->get_css_classes($object, $ref->mgdschema_class);

It seems like $ref->mgdschema_class can also be of type null; however, parameter $mgdschema_class of midgard_admin_asgard_navigation::get_css_classes() does only seem to accept string, maybe add an additional type check?

        $this->shown_objects[$object->guid] = true;

        echo "<li class=\"{$css_class}\">\n";

        $label = htmlspecialchars($ref->get_object_label($object));
        $icon = $ref->get_object_icon($object);
        if (empty($label)) {
            $label = "#{$object->id}";
        }

        $checked = ($this->copy_tree) ? ' checked="checked"' : '';

        if ($this->inputs) {
            // This value is used for compiling the exclusion list: if the object is found from this list, but not from the selection list,
            // it means that the selection did not include the object GUID
            echo "<input type=\"hidden\" name=\"all_objects[]\" value=\"{$object->guid}\" />\n";

            echo "<label for=\"item_{$object->guid}\">\n";
            echo "<input id=\"item_{$object->guid}\" type=\"{$this->input_type}\" name=\"{$this->input_name}\" value=\"{$object->guid}\"{$checked} />\n";
        }

        echo "<span class=\"title{$span_class}\" title=\"" . sprintf($this->_l10n->get('%s (%s)'), $label, $ref->get_class_label()) . "\">{$icon}{$label}</span>\n";

        // Show the link to the object
        if ($this->view_link) {
            echo "<a href=\"{$this->page_prefix}__mfa/asgard/object/view/{$object->guid}/\" class=\"thickbox\" target=\"_blank\" title=\"" . $this->_l10n->get('view object') . "\">\n";
            echo "<i class=\"fa fa-eye\"></i>\n";
            echo "</a>\n";
        }

        // Show the link to the object
        if ($this->edit_link) {
            echo "<a href=\"{$this->page_prefix}__mfa/asgard/object/edit/{$object->guid}/\" target='_blank' title=\"" . $this->_l10n->get('edit object') . "\">\n";
            echo "<i class=\"fa fa-pencil\"> </i>\n";
            echo "</a>\n";
        }

        if ($this->inputs) {
            echo "</label>\n";
        }

        // List the child elements
        $this->_list_child_elements($object, $level + 1);

        echo "</li>\n";
    }

    protected function _is_collapsed($type, $total) : bool
    {
        if ($this->inputs) {
            return false;
        }
        return parent::_is_collapsed($type, $total);
    }

    /**
     * Draw the tree selector
     */
    public function draw()
    {
        if (!$this->input_type) {
            $this->input_type = 'checkbox';
        }

        if (!$this->input_name) {
            if ($this->input_type === 'checkbox') {
                $this->input_name = 'selected[]';
            } else {
                $this->input_name = 'target';
            }
        }

        $root_object = $this->_object;
        $this->_list_child_elements($root_object);
    }
}