src/dbo/templates.php ➔ templates_import()

Complexity

Conditions	29
Paths	26

Size

Total Lines	206
Code Lines	112

Duplication

Lines	59
Ratio	28.64 %

Importance

Changes

0

<?php
function templates_import ($project_id, $xml, &$error)
{
    debug_write_log(DEBUG_TRACE, '[templates_import]');
    debug_write_log(DEBUG_DUMP,  '[templates_import] $project_id = ' . $project_id);

    // Allocation of XML code to permissions.
    $permissions = array
    (
        'create'    => PERMIT_CREATE_RECORD,
        'modify'    => PERMIT_MODIFY_RECORD,
        'postpone'  => PERMIT_POSTPONE_RECORD,
        'resume'    => PERMIT_RESUME_RECORD,
        'reassign'  => PERMIT_REASSIGN_RECORD,
        'reopen'    => PERMIT_REOPEN_RECORD,
        'comment'   => PERMIT_ADD_COMMENTS,
        'attach'    => PERMIT_ATTACH_FILES,
        'remove'    => PERMIT_REMOVE_FILES,
        'secret'    => PERMIT_CONFIDENTIAL_COMMENTS,
        'remind'    => PERMIT_SEND_REMINDERS,
        'delete'    => PERMIT_DELETE_RECORD,
        'addsubrec' => PERMIT_ADD_SUBRECORDS,
        'remsubrec' => PERMIT_REMOVE_SUBRECORDS,
        'view'      => PERMIT_VIEW_RECORD,
    );

    // Enumerate templates.
    $templates = $xml->xpath('/project/template');

The method xpath cannot be called on $xml (of type string).

    if ($templates !== FALSE)
    {
        foreach ($templates as $template)
        {
            $rs = dal_query('templates/count.sql');

            if (MAX_TEMPLATES_NUMBER != 0 && $rs->fetch(0) >= MAX_TEMPLATES_NUMBER)

This code seems to be duplicated across your project.

            {
                debug_write_log(DEBUG_NOTICE, 'Maximum amount of templates is already reached.');
                return TRUE;
            }

            $template['name']         = ustrcut($template['name'],         MAX_TEMPLATE_NAME);
            $template['prefix']       = ustrcut($template['prefix'],       MAX_TEMPLATE_PREFIX);
            $template['critical_age'] = ustrcut($template['critical_age'], ustrlen(MAX_TEMPLATE_DAYS_COUNT));
            $template['frozen_time']  = ustrcut($template['frozen_time'],  ustrlen(MAX_TEMPLATE_DAYS_COUNT));
            $template['description']  = ustrcut($template['description'],  MAX_TEMPLATE_DESCRIPTION);

            $guest_access = ($template['guest_access'] == 'yes');

            // Validate template.
            switch (template_validate($template['name'], $template['prefix'], $template['critical_age'], $template['frozen_time']))
            {
                case NO_ERROR:
                    break;  // nop
                case ERROR_INCOMPLETE_FORM:
                    $error = get_html_resource(RES_ALERT_REQUIRED_ARE_EMPTY_ID);
                    return FALSE;
                case ERROR_INVALID_INTEGER_VALUE:
                    $error = get_html_resource(RES_ALERT_INVALID_INTEGER_VALUE_ID);
                    return FALSE;
                case ERROR_INTEGER_VALUE_OUT_OF_RANGE:
                    $error = ustrprocess(get_html_resource(RES_ALERT_INTEGER_VALUE_OUT_OF_RANGE_ID), MIN_TEMPLATE_DAYS_COUNT, MAX_TEMPLATE_DAYS_COUNT);
                    return FALSE;
                default:
                    debug_write_log(DEBUG_WARNING, '[templates_import] Template validation failure.');
                    $error = get_html_resource(RES_ALERT_UNKNOWN_ERROR_ID);
                    return FALSE;
            }

            // Create template.
            switch (template_create($project_id,
                                    $template['name'],
                                    $template['prefix'],
                                    $template['critical_age'],
                                    $template['frozen_time'],
                                    $template['description'],
                                    $guest_access))
            {
                case NO_ERROR:
                    break;
                case ERROR_ALREADY_EXISTS:
                    $error = get_html_resource(RES_ALERT_TEMPLATE_ALREADY_EXISTS_ID);
                    return FALSE;
                default:
                    debug_write_log(DEBUG_WARNING, '[templates_import] Template creation failure.');
                    $error = get_html_resource(RES_ALERT_UNKNOWN_ERROR_ID);
                    return FALSE;
            }

            $rs = dal_query('templates/fndk2.sql', $project_id, ustrtolower($template['name']));

            if ($rs->rows == 0)

The property $rows is declared protected in CRecordset. Since you implemented __get(), maybe consider adding a @property or @property-read annotation. This makes it easier for IDEs to provide auto-completion.

This code seems to be duplicated across your project.

            {
                debug_write_log(DEBUG_WARNING, '[templates_import] Created template not found.');
                $error = get_html_resource(RES_ALERT_UNKNOWN_ERROR_ID);
                return FALSE;
            }

            $template_id = $rs->fetch('template_id');

            // Set author permissions.
            $permits = $template->xpath('./permissions/author/permit');

The method xpath cannot be called on $template (of type array<string,string,{"description":"string"}>).

            if ($permits !== FALSE)

This code seems to be duplicated across your project.

            {
                $perms = 0;

                foreach ($permits as $permit)
                {
                    $permit = strval($permit);

                    if (array_key_exists($permit, $permissions))
                    {
                        $perms |= $permissions[$permit];
                    }
                }

                template_author_perm_set($template_id, $perms);
            }

            // Set responsible permissions.
            $permits = $template->xpath('./permissions/responsible/permit');

The method xpath cannot be called on $template (of type array<string,string,{"description":"string"}>).

            if ($permits !== FALSE)

This code seems to be duplicated across your project.

            {
                $perms = 0;

                foreach ($permits as $permit)
                {
                    $permit = strval($permit);

                    if (array_key_exists($permit, $permissions))
                    {
                        $perms |= $permissions[$permit];
                    }
                }

                template_responsible_perm_set($template_id, $perms);
            }

            // Set registered permissions.
            $permits = $template->xpath('./permissions/registered/permit');

The method xpath cannot be called on $template (of type array<string,string,{"description":"string"}>).

            if ($permits !== FALSE)

This code seems to be duplicated across your project.

            {
                $perms = 0;

                foreach ($permits as $permit)
                {
                    $permit = strval($permit);

                    if (array_key_exists($permit, $permissions))
                    {
                        $perms |= $permissions[$permit];
                    }
                }

                template_registered_perm_set($template_id, $perms);
            }

            // Enumerate groups permissions.
            $groups = $template->xpath('./permissions/group');

The method xpath cannot be called on $template (of type array<string,string,{"description":"string"}>).

            if ($groups !== FALSE)
            {
                foreach ($groups as $group)
                {
                    if (isset($group->permit))
                    {
                        // Find the group.
                        $rs = dal_query('groups/fndk.sql',
                                        $group['type'] == 'global' ? 'is null' : '=' . $project_id,
                                        ustrtolower(ustrcut($group['name'], MAX_GROUP_NAME)));

                        if ($rs->rows != 0)

The property $rows is declared protected in CRecordset. Since you implemented __get(), maybe consider adding a @property or @property-read annotation. This makes it easier for IDEs to provide auto-completion.

                        {
                            $group_id = $rs->fetch('group_id');

                            // Set group permissions.
                            $perms = 0;

                            foreach ($group->permit as $permit)
                            {
                                $permit = strval($permit);

                                if (array_key_exists($permit, $permissions))
                                {
                                    $perms |= $permissions[$permit];
                                }
                            }

                            group_set_permissions($group_id, $template_id, $perms);
                        }
                    }
                }
            }

            // Import states.
            if (!states_import($template_id, $template, $error))

$template is of type array<string,string,{"description":"string"}>, but the function expects a string.

            {
                return FALSE;
            }
        }
    }

    return TRUE;
}