Anax\MVC\CDatabaseModel

Complexity

Total Complexity

17

Size/Duplication

Total Lines	226
Duplicated Lines	15.93 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes	2
Bugs	0	Features	1

14 Methods

Rating	Name	Duplication	Size	Complexity
A	getSource()	0	4	1
A	findAll()	9	9	1
A	findRecent()	11	11	1
A	getProperties()	0	8	1
A	setProperties()	0	9	3
A	create()	0	16	1
A	find()	9	9	1
A	save()	0	11	2
A	update()	0	17	1
A	delete()	0	9	1
A	query()	0	7	1
A	where()	0	6	1
A	andWhere()	0	6	1
A	execute()	7	7	1

<?php

namespace Anax\MVC;

/**
 * Model for Users.
 *
 */
class CDatabaseModel implements \Anax\DI\IInjectionAware
{
    use \Anax\DI\TInjectable;

    /**
     * Get the table name.
     *
     * @return string with the table name.
     */
    public function getSource()
    {
        return strtolower(implode('', array_slice(explode('\\', get_class($this)), -1)));
    }

    /**
     * Find and return all.
     *
     * @return array
     */
    public function findAll()

This method seems to be duplicated in your project.

    {
        $this->db->select()

The property db does not exist on object<Anax\MVC\CDatabaseModel>. Since you implemented __get, maybe consider adding a @property annotation.

                 ->from($this->getSource());

        $this->db->execute();

The property db does not exist on object<Anax\MVC\CDatabaseModel>. Since you implemented __get, maybe consider adding a @property annotation.

        $this->db->setFetchModeClass(__CLASS__);

The property db does not exist on object<Anax\MVC\CDatabaseModel>. Since you implemented __get, maybe consider adding a @property annotation.

        return $this->db->fetchAll();

The property db does not exist on object<Anax\MVC\CDatabaseModel>. Since you implemented __get, maybe consider adding a @property annotation.

    }

    /**
     * Find and return recent items.
     * query requires column "created" in table.
     *
     * @return array
     */
    public function findRecent($count)

This method seems to be duplicated in your project.

    {
        $this->db->select()

The property db does not exist on object<Anax\MVC\CDatabaseModel>. Since you implemented __get, maybe consider adding a @property annotation.

                 ->from($this->getSource())
                 ->orderby('created DESC')
                 ->limit($count);

        $this->db->execute();

The property db does not exist on object<Anax\MVC\CDatabaseModel>. Since you implemented __get, maybe consider adding a @property annotation.

        $this->db->setFetchModeClass(__CLASS__);

The property db does not exist on object<Anax\MVC\CDatabaseModel>. Since you implemented __get, maybe consider adding a @property annotation.

        return $this->db->fetchAll();

The property db does not exist on object<Anax\MVC\CDatabaseModel>. Since you implemented __get, maybe consider adding a @property annotation.

    }

    /**
     * Get object properties.
     *
     * @return array with object properties.
     */
    public function getProperties()
    {
        $properties = get_object_vars($this);
        unset($properties['di']);
        unset($properties['db']);

        return $properties;
    }
    /**
     * Set object properties.
     *
     * @param array $properties with properties to set.
     *
     * @return void
     */
    public function setProperties($properties)
    {
        // Update object with incoming values, if any
        if (!empty($properties)) {
            foreach ($properties as $key => $val) {
                $this->$key = $val;
            }
        }
    }
    /**
     * Create new row.
     *
     * @param array $values key/values to save.
     *
     * @return boolean true or false if saving went okey.
     */
    public function create($values)
    {
        $keys   = array_keys($values);
        $values = array_values($values);

        $this->db->insert(

The property db does not exist on object<Anax\MVC\CDatabaseModel>. Since you implemented __get, maybe consider adding a @property annotation.

            $this->getSource(),
            $keys
        );

        $res = $this->db->execute($values);

The property db does not exist on object<Anax\MVC\CDatabaseModel>. Since you implemented __get, maybe consider adding a @property annotation.

        $this->id = $this->db->lastInsertId();

The property id does not exist. Did you maybe forget to declare it?

The property db does not exist on object<Anax\MVC\CDatabaseModel>. Since you implemented __get, maybe consider adding a @property annotation.

        return $res;
    }
    /**
     * Find and return specific.
     *
     * @return this
     */
    public function find($id)

This method seems to be duplicated in your project.

    {
        $this->db->select()

The property db does not exist on object<Anax\MVC\CDatabaseModel>. Since you implemented __get, maybe consider adding a @property annotation.

                 ->from($this->getSource())
                 ->where("id = ?");

        $this->db->execute([$id]);

The property db does not exist on object<Anax\MVC\CDatabaseModel>. Since you implemented __get, maybe consider adding a @property annotation.

        return $this->db->fetchInto($this);

The property db does not exist on object<Anax\MVC\CDatabaseModel>. Since you implemented __get, maybe consider adding a @property annotation.

    }
    /**
     * Save current object/row.
     *
     * @param array $values key/values to save or empty to use object properties.
     *
     * @return boolean true or false if saving went okey.
     */
    public function save($values = [])
    {
        $this->setProperties($values);
        $values = $this->getProperties();

        if (isset($values['id'])) {
            return $this->update($values);
        } else {
            return $this->create($values);
        }
    }
    /**
     * Update row.
     *
     * @param array $values key/values to save.
     *
     * @return boolean true or false if saving went okey.
     */
    public function update($values)
    {
        $keys   = array_keys($values);
        $values = array_values($values);

        // Its update, remove id and use as where-clause
        unset($keys['id']);
        $values[] = $this->id;

        $this->db->update(

The property db does not exist on object<Anax\MVC\CDatabaseModel>. Since you implemented __get, maybe consider adding a @property annotation.

            $this->getSource(),
            $keys,
            "id = ?"
        );

        return $this->db->execute($values);

The property db does not exist on object<Anax\MVC\CDatabaseModel>. Since you implemented __get, maybe consider adding a @property annotation.

    }
    /**
     * Delete row.
     *
     * @param integer $id to delete.
     *
     * @return boolean true or false if deleting went okey.
     */
    public function delete($id)
    {
        $this->db->delete(

The property db does not exist on object<Anax\MVC\CDatabaseModel>. Since you implemented __get, maybe consider adding a @property annotation.

            $this->getSource(),
            'id = ?'
        );

        return $this->db->execute([$id]);

The property db does not exist on object<Anax\MVC\CDatabaseModel>. Since you implemented __get, maybe consider adding a @property annotation.

    }
    /**
     * Build a select-query.
     *
     * @param string $columns which columns to select.
     *
     * @return $this
     */
    public function query($columns = '*')
    {
        $this->db->select($columns)

The property db does not exist on object<Anax\MVC\CDatabaseModel>. Since you implemented __get, maybe consider adding a @property annotation.

                 ->from($this->getSource());

        return $this;
    }
    /**
     * Build the where part.
     *
     * @param string $condition for building the where part of the query.
     *
     * @return $this
     */
    public function where($condition)
    {
        $this->db->where($condition);

The property db does not exist on object<Anax\MVC\CDatabaseModel>. Since you implemented __get, maybe consider adding a @property annotation.

        return $this;
    }
    /**
     * Build the where part.
     *
     * @param string $condition for building the where part of the query.
     *
     * @return $this
     */
    public function andWhere($condition)
    {
        $this->db->andWhere($condition);

The property db does not exist on object<Anax\MVC\CDatabaseModel>. Since you implemented __get, maybe consider adding a @property annotation.

        return $this;
    }
    /**
     * Execute the query built.
     *
     * @param string $query custom query.

There is no parameter named $query. Was it maybe removed?

     *
     * @return $this
     */
    public function execute($params = [])

This method seems to be duplicated in your project.

    {
        $this->db->execute($this->db->getSQL(), $params);

The property db does not exist on object<Anax\MVC\CDatabaseModel>. Since you implemented __get, maybe consider adding a @property annotation.

        $this->db->setFetchModeClass(__CLASS__);

The property db does not exist on object<Anax\MVC\CDatabaseModel>. Since you implemented __get, maybe consider adding a @property annotation.

        return $this->db->fetchAll();

The property db does not exist on object<Anax\MVC\CDatabaseModel>. Since you implemented __get, maybe consider adding a @property annotation.

    }
}