Source\Models\Category

Complexity

Total Complexity

11

Size/Duplication

Total Lines	77
Duplicated Lines	0 %

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	3	1
A	findByName()	0	5	1
A	bootstrap()	0	4	1
B	save()	0	39	8

<?php

namespace Source\Models;

use Source\Core\Model;

/**
 * @package Source\Models
 */
class Category extends Model
{
    /**
     * Category constructor.
     */
    public function __construct()
    {
        parent::__construct("product_type", ["id"], ["name"]);
    }

    /**
     * @param string $name
     * @param string $email
     * @param string $password
     * @param string|null $document
     * @return User
     */
    public function bootstrap(string $name): Category
    {
        $this->name = $name;

The property name does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

        return $this;
    }

    /**
     * @param string $name
     * @param string $columns
     * @return null|Category
     */
    public function findByName(string $name, string $columns = "*"): ?Category
    {
        $find = $this->find("name = :name", "name={$name}", $columns);

        return $find->fetch();
    }

    /**
     * @return bool
     */
    public function save(): bool
    {
        if (!$this->required()) {
            $this->error = "O nome da categoria é obrigatória";
            return false;
        }

        /** Category Update */
        if (!empty($this->id)) {

The property id does not exist on Source\Models\Category. Since you implemented __get, consider adding a @property annotation.

            $categoryId = $this->id;

            if ($this->find("name = :e AND id != :i", "e={$this->name}&i={$categoryId}", "id")->fetch()) {
                $this->error = "A categoria informada já está cadastrada";
                return false;
            }

            $this->update($this->safe(), "id = :id", "id={$categoryId}");
            if ($this->fail()) {
                $this->error = "Erro ao atualizar, verifique os dados";
                return false;
            }
        }

        /** Category Create */
        if (empty($this->id)) {
            if ($this->findByName($this->name, 'id')) {
                $this->error = "A categoria informada já está cadastrada";
                return false;
            }

            $categoryId = $this->create($this->safe());
            if ($this->fail()) {
                $this->error = "Erro ao cadastrar, verifique os dados";
                return false;
            }
        }

        $this->data = ($this->findById($categoryId))->data();

The variable $categoryId does not seem to be defined for all execution paths leading up to this point.

        return true;
    }
}