Rougin\Wildfire\ResultTrait

Complexity

Total Complexity

16

Size/Duplication

Total Lines	118
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Test Coverage

Coverage

100%

Importance

Changes	1
Bugs	0	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	asDropdown()	0	13	2
A	result()	0	18	3
D	createObject()	0	43	9
A	getQueryResult()	0	10	2

<?php

namespace Rougin\Wildfire;

/**
 * Result Trait
 *
 * Contains functions for retrieving data.
 * 
 * @package Wildfire
 * @author  Rougin Royce Gutib <rougingutib@gmail.com>
 */
trait ResultTrait
{
    /**
     * @var array
     */
    protected $tables = [];

    /**
     * Lists all data in dropdown format.
     *
     * @param  string $description
     * @return array
     */
    public function asDropdown($description = 'description')
    {
        $data = [];
        $id = $this->describe->getPrimaryKey($this->table);

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

The property table does not seem to exist. Did you mean tables?

        $result = $this->query->result();

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

        foreach ($result as $row) {
            $data[$row->$id] = ucwords($row->$description);
        }

        return $data;
    }

    /**
     * Return the result
     * 
     * @return object
     */
    public function result()
    {
        $data = $this->getQueryResult();
        $result = [];

        if (empty($this->table)) {

The property table does not seem to exist. Did you mean tables?

            $this->get();

It seems like get() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        }

        foreach ($data as $row)
        {
            $object = $this->createObject($this->table, $row);

The property table does not seem to exist. Did you mean tables?

            array_push($result, $object);
        }

        return $result;
    }

    /**
     * Create an object from the specified data
     *
     * @param  string $table
     * @param  object $row
     * @return array
     */
    protected function createObject($table, $row)
    {
        $newTable = ucfirst(Inflector::singular($table));
        $model = new $newTable;

        if ( ! array_key_exists($table, $this->tables)) {
            $tableInfo = $this->describe->getTable($table);

            $this->tables[$table] = $tableInfo;
        } else {
            $tableInfo = $this->tables[$table];
        }

        foreach ($row as $key => $value) {
            foreach ($tableInfo as $column) {
                if ($column->getField() != $key) {
                    continue;
                }

                $model->$key = $value;
            }
        }

        foreach ($row as $key => $value) {
            foreach ($tableInfo as $column) {
                if ($column->getField() != $key || ! $column->isForeignKey()) {
                    continue;
                }

                $foreignColumn = $column->getReferencedField();
                $foreignTable = $column->getReferencedTable();

                $delimiters = [ $foreignColumn => $value ];
                $foreignData = $this->find($foreignTable, $delimiters);

It seems like find() must be provided by classes using this trait. How about adding it as abstract method to this trait?

                $newColumn = Inflector::singular($foreignTable);

                $model->$newColumn = $foreignData;
            }
        }

        return $model;
    }

    /**
     * Gets the data result from the specified query.
     * 
     * @return array|object
     */
    protected function getQueryResult()
    {
        $result = $this->query;

        if (method_exists($this->query, 'result')) {
            $result = $this->query->result();
        }

        return $result;
    }
}