midcom_helper_filesync_exporter_structure

Complexity

Total Complexity

13

Size/Duplication

Total Lines	105
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	8

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
B	read_node()	0	34	3
A	read_structure()	0	12	1
A	export()	0	10	1
C	draw_array()	0	43	8

<?php
/**
 * @package midcom.helper.filesync
 * @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
 */

/**
 * Export a site structure to the format used by Site Wizard
 *
 * @package midcom.helper.filesync
 */
class midcom_helper_filesync_exporter_structure extends midcom_helper_filesync_exporter

PSR1 recommends that each class must be in a namespace of at least one level to avoid collisions.

{
    private function read_node(midcom_db_topic $node)
    {
        $node_array = array();
        $node_array['name'] = $node->name;
        $node_array['title'] = $node->extra;
        $node_array['component'] = $node->component;
        $node_array['style'] = $node->style;
        $node_array['style_inherit'] = $node->styleInherit;

        // Per-component specialties
        switch ($node->component)
        {
            case 'net.nehmer.static':
                $node_array['create_index'] = true;
        }

        // Get parameters
        $node_array['parameters'] = $node->list_parameters();

        // TODO: Implement ACL exporting
        $node_array['acl'] = array();

        // Recurse subnodes
        $node_array['nodes'] = array();
        $qb = midcom_db_topic::new_query_builder();
        $qb->add_constraint('up', '=', $node->id);
        $children = $qb->execute();
        foreach ($children as $child)

The expression $children of type array|false is not guaranteed to be traversable. How about adding an additional type check?

        {
            $node_array['nodes'][$child->name] = $this->read_node($child);
        }

        return $node_array;
    }

    public function read_structure(midcom_db_topic $root_node, $structure_name)
    {
        // Prepare structure
        $structure = array();
        $structure[$structure_name] = array();
        $structure[$structure_name]['name'] = $structure_name;
        $structure[$structure_name]['title'] = midcom::get()->config->get('midcom_site_title');
        // Read the topic data
        $structure[$structure_name]['root'] = $this->read_node($root_node);

        file_put_contents("{$this->root_dir}{$structure_name}.inc", $this->draw_array($structure));
    }

    private function draw_array($array, $prefix = '')
    {
        $data = '';
        foreach ($array as $key => $val)
        {
            $data .= $prefix;
            if (!is_numeric($key))
            {
                $data .= "'{$key}' => ";
            }

            switch (gettype($val))
            {
                case 'boolean':
                    $data .= ($val)?'true':'false';
                    break;
                case 'array':
                    if (empty($val))
                    {
                        $data .= 'array()';
                    }
                    else
                    {
                        $data .= "array\n{$prefix}(\n" . $this->draw_array($val, "{$prefix}    ") . "{$prefix})";
                    }
                    break;

                default:
                    if (is_numeric($val))
                    {
                        $data .= $val;
                    }
                    else
                    {
                        $data .= "'" . str_replace("'", "\'", $val) . "'";
                    }
            }

            $data .= ",\n";

        }
        return $data;
    }

    public function export()
    {
        // Generate a safe name for the structure
        $generator = midcom::get()->serviceloader->load('midcom_core_service_urlgenerator');
        $structure_name = $generator->from_string(midcom::get()->get_page_prefix());

        $root_topic = midcom_core_context::get()->get_key(MIDCOM_CONTEXT_ROOTTOPIC);

        $this->read_structure($root_topic, $structure_name);
    }
}