Scrawler\Arca\Traits\Model\Setter

Complexity

Total Complexity

7

Size/Duplication

Total Lines	55
Duplicated Lines	0 %

Importance

Changes	2
Bugs	0	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	setProperties()	0	7	2
A	setLoadedProperties()	0	10	2
A	setLoadedAllProperties()	0	5	2
A	setLoaded()	0	6	1

<?php

/*
 * This file is part of the Scrawler package.
 *
 * (c) Pranjal Pandey <its.pranjalpandey@gmail.com>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace Scrawler\Arca\Traits\Model;

use Doctrine\DBAL\Types\Type;
use Scrawler\Arca\Model;

/**
 * Setter trait to provide setter methods to the model.
 */
trait Setter
{
    /**
     * Set all properties of model via array.
     *
     * @param array<mixed> $properties
     */
    public function setProperties(array $properties): Model
    {
        foreach ($properties as $key => $value) {
            $this->set($key, $value);

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

        }

        return $this;

The expression return $this returns the type Scrawler\Arca\Traits\Model\Setter which includes types incompatible with the type-hinted return Scrawler\Arca\Model.

    }

    /**
     * Set all properties of model loaded via database.
     *
     * @param array<mixed> $properties
     */
    public function setLoadedProperties(array $properties): Model
    {
        $this->__properties['self'] = $properties;

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

        $this->setLoadedAllProperties($properties);
        foreach (array_keys($properties) as $key) {
            $this->__properties['type'][$key] = $this->tableManager->getTable($this->table)->getColumn($key)->getComment();
        }
        $this->__meta['id'] = $properties['id'];

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

        return $this;

The expression return $this returns the type Scrawler\Arca\Traits\Model\Setter which includes types incompatible with the type-hinted return Scrawler\Arca\Model.

    }

    /**
     * get php value from database value.
     *
     * @param array<mixed> $properties
     */
    private function setLoadedAllProperties(array $properties): void
    {
        foreach ($properties as $key => $value) {
            $type = Type::getType($this->tableManager->getTable($this->table)->getColumn($key)->getComment());
            $this->__properties['all'][$key] = $type->convertToPHPValue($value, $this->connection->getDatabasePlatform());

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

        }
    }

    /**
     * call when model is loaded from database.
     */
    public function setLoaded(): Model
    {
        $this->__meta['is_loaded'] = true;

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

        $this->__meta['id_error'] = false;

        return $this;

The expression return $this returns the type Scrawler\Arca\Traits\Model\Setter which includes types incompatible with the type-hinted return Scrawler\Arca\Model.

    }
}