org_openpsa_projects_role_dba

Complexity

Total Complexity

6

Size/Duplication

Total Lines	50
Duplicated Lines	0 %

Test Coverage

Coverage

87.5%

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	_on_updating()	0	3	1
A	check_duplicates()	0	12	2
A	_on_creating()	0	3	1
A	add()	0	16	2

<?php
/**
 * @package org.openpsa.projects
 * @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
 */

/**
 * @property integer $project
 * @property integer $role
 * @property string $description
 * @property integer $person
 * @property integer $status
 * @package org.openpsa.projects
 */
class org_openpsa_projects_role_dba extends midcom_core_dbaobject
{
    public $__midcom_class_name__ = __CLASS__;
    public $__mgdschema_class_name__ = 'org_openpsa_role';

    public $_use_rcs = false;

    public static function add(int $project, int $person, int $role) : bool
    {
        $mc = self::new_collector('project', $project);
        $mc->add_constraint('role', '=', $role);
        $mc->add_constraint('person', '=', $person);
        $mc->execute();
        if ($mc->count() > 0) {
            //Resource is already present, aborting silently
            return true;
        }

        $new_role = new self();
        $new_role->person = $person;
        $new_role->role = $role;
        $new_role->project = $project;
        return $new_role->create();
    }

    /**
     * Returns true for NO existing duplicates
     */
    public function check_duplicates() : bool
    {
        $qb = new midgard_query_builder('org_openpsa_role');
        $qb->add_constraint('person', '=', $this->person);
        $qb->add_constraint('objectGuid', '=', $this->objectGuid);

The property objectGuid does not exist on org_openpsa_projects_role_dba. Since you implemented __get, consider adding a @property annotation.

        $qb->add_constraint('role', '=', $this->role);

        if ($this->id) {
            $qb->add_constraint('id', '<>', $this->id);
        }

        return $qb->count() == 0;
    }

    public function _on_creating() : bool
    {
        return $this->check_duplicates();
    }

    public function _on_updating() : bool
    {
        return $this->check_duplicates();
    }
}