Gewaer\CustomFields\AbstractCustomFieldsModel::getAllCustomFields()

Complexity

Conditions	4
Paths	5

Size

Total Lines	49
Code Lines	17

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	12
CRAP Score	4.5923

Importance

Changes

0

<?php
    public function getAllCustomFields(array $fields = [], int $company_id = 0)

The parameter $company_id is not used and could be removed.

    {
        //We does it only find names in plural? We need to fix this or make a workaroun
        if (!$models = Modules::findFirstByName($this->getSource())) {

The method findFirstByName() does not exist on Gewaer\Models\Modules. Since you implemented __callStatic, consider adding a @method annotation.

            return;
        }

        $conditions = [];

The assignment to $conditions is dead and can be removed.

        $fieldsIn = null;

        if (!empty($fields)) {
            $fieldsIn = " and name in ('" . implode("','", $fields) . ')';
        }

        $conditions = 'modules_id = ? ' . $fieldsIn;

        $bind = [$this->getId(),$this->di->getApp()->getId(), $models->getId(), $this->di->getUserData()->default_company];

The method getApp() does not exist on Phalcon\Mvc\Model\Resultset.

The method getUserData() does not exist on Phalcon\Mvc\Model\Resultset.

The property di does not exist on Gewaer\CustomFields\AbstractCustomFieldsModel. Since you implemented __get, consider adding a @property annotation.

        // $customFieldsValueTable = $this->getSource() . '_custom_fields';
        $customFieldsValueTable = $this->getSource() . '_custom_fields';

        //We are to make a new query to replace old gewaer implementation.
        $result = $this->getReadConnection()->prepare("SELECT l.{$this->getSource()}_id,

The method prepare() does not exist on Phalcon\Db\AdapterInterface. It seems like you code against a sub-type of said class. However, the method does not exist in Phalcon\Db\Adapter. Are you sure you never get one of those?

                                               c.id as field_id,
                                               c.name,
                                               l.value ,
                                               c.users_id,
                                               l.created_at,
                                               l.updated_at
                                        FROM {$customFieldsValueTable} l,
                                             custom_fields c
                                        WHERE c.id = l.custom_fields_id
                                          AND l.{$this->getSource()}_id = ?
                                          AND c.apps_id = ?
                                          AND c.modules_id = ?
                                          AND c.companies_id = ? 
                                          AND l.companies_id = c.companies_id");

        $result->execute($bind);

        // $listOfCustomFields = $result->fetchAll();
        $listOfCustomFields = [];

        while ($row = $result->fetch(\PDO::FETCH_OBJ)) {
            $listOfCustomFields[$row->name] = $row->value;
        }

        return $listOfCustomFields;

The expression return $listOfCustomFields returns the type array which is incompatible with the documented return type Phalcon\Mvc\Model.

    }