Rougin\Wildfire\Wildfire

Complexity

Total Complexity

14

Size/Duplication

Total Lines	123
Duplicated Lines	10.57 %

Coupling/Cohesion

Components	1
Dependencies	8

Test Coverage

Coverage

87.76%

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	7	1
A	delete()	0	12	3
B	find()	0	24	4
B	get()	0	25	4
A	__call()	13	13	2

<?php

namespace Rougin\Wildfire;

use Rougin\Wildfire\Traits\ObjectTrait;
use Rougin\Wildfire\Traits\ResultTrait;
use Rougin\Wildfire\Traits\DatabaseTrait;
use Rougin\Wildfire\Traits\DescribeTrait;

/**
 * Wildfire
 *
 * Yet another wrapper for CodeIgniter's Query Builder Class.
 *
 * @package Wildfire
 * @author  Rougin Royce Gutib <rougingutib@gmail.com>
 */
class Wildfire
{
    use DatabaseTrait, DescribeTrait, ObjectTrait, ResultTrait;

    /**
     * @param \CI_DB|null        $database
     * @param \CI_DB_result|null $query
     */
    public function __construct($database = null, $query = null)
    {
        $this->setDatabase($database);

        $this->describe = $this->getDescribe($this->db);
        $this->query = $query;
    }

    /**
     * Deletes the data from the specified table by the given delimiters.
     *
     * @param  string|object $table
     * @param  integer|array $delimiters
     * @return mixed
     */
    public function delete($table, $delimiters)
    {
        $tableName = ($table instanceof CodeigniterModel) ? $table->getTableName() : $table;

        if (is_integer($delimiters)) {
            $delimiters = [ $this->describe->getPrimaryKey($tableName) => $delimiters ];

It seems like $tableName defined by $table instanceof \Rougin\Wildfire\CodeigniterModel ? $table->getTableName() : $table on line 43 can also be of type object; however, Rougin\Describe\Describe::getPrimaryKey() does only seem to accept string, maybe add an additional type check?

        }

        $this->db->where($delimiters);

        return $this->db->delete($tableName);

It seems like $tableName defined by $table instanceof \Rougin\Wildfire\CodeigniterModel ? $table->getTableName() : $table on line 43 can also be of type object; however, CI_DB_query_builder::delete() does only seem to accept string, maybe add an additional type check?

    }

    /**
     * Finds the row from the specified ID or with the list of delimiters from
     * the specified table.
     *
     * @param  object|string  $table
     * @param  array|integer  $delimiters
     * @param  boolean        $isForeignKey
     * @return object|boolean
     */
    public function find($table, $delimiters = [], $isForeignKey = false)
    {
        list($model, $tableName) = $this->getModel($table, $isForeignKey);

        if (! is_array($delimiters)) {
            $primaryKey = $this->describe->getPrimaryKey($tableName);

            $delimiters = [ $primaryKey => $delimiters ];
        }

        $this->db->where($delimiters);

        $query = $this->db->get($tableName);

        if ($query->num_rows() > 0) {
            if ($table == $model) {
                $tableName = $model;
            }

            return $this->createObject($tableName, $query->row(), $isForeignKey);
        }

        return false;
    }

    /**
     * Returns all rows from the specified table.
     *
     * @param  object|string $table
     * @return self
     */
    public function get($table = '')
    {
        list($model, $tableName) = $this->getModel($table);

        if ($this->query == null) {
            $this->query = $this->db->get($tableName);
        }

        $this->table = $tableName;

        if ($table == $model) {
            $this->table = $model;
        }

        // Guess the specified table from the query
        if (empty($table)) {
            $query = $this->db->last_query();

            preg_match('/\bfrom\b\s*(\w+)/i', $query, $matches);

            $this->table = $matches[1];
        }

        return $this;
    }

    /**
     * Calls methods from this class in underscore case.
     *
     * @param  string $method
     * @param  mixed  $parameters
     * @return mixed
     */
    public function __call($method, $parameters)

This method seems to be duplicated in your project.

    {
        $method = camelize($method);
        $result = $this;

        if (method_exists($this, $method)) {
            $class = [$this, $method];
            
            $result = call_user_func_array($class, $parameters);
        }

        return $result;
    }
}