Oc\Country\CountryRepository

Complexity

Total Complexity

10

Size/Duplication

Total Lines	120
Duplicated Lines	26.67 %

Coupling/Cohesion

Components	1
Dependencies	3

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	update()	16	16	2
A	remove()	16	16	2
A	__construct()	0	4	1
A	fetchAll()	0	21	3
A	create()	0	13	2

<?php

namespace Oc\Country;

use Doctrine\DBAL\Connection;
use InvalidArgumentException;

/**
 * Class CountryRepository
 *
 * @package Oc\Country
 * @author Nick Lubisch <oc@nicklubisch.de>
 */
class CountryRepository
{
    /**
     * Database table name that this repository maintains.
     *
     * @var string
     */
    const TABLE = 'countries';

    /**
     * @var Connection
     */
    private $connection;

    /**
     * CountryRepository constructor.
     *
     * @param Connection $connection
     */
    public function __construct(Connection $connection)
    {
        $this->connection = $connection;
    }

    /**
     * Fetches all countries.
     *
     * @return CountryEntity[]
     */
    public function fetchAll()
    {
        $statement = $this->connection->createQueryBuilder()
            ->select('*')
            ->from(self::TABLE)
            ->execute();

        $result = $statement->fetchAll();

        if ($result === false) {
            return [];
        }

        $countries = [];

        foreach ($result as $item) {
            $countries[] = (new CountryEntity())->fromDatabaseArray($item);
        }

        return $countries;
    }

    /**
     * Creates a country in the database.
     *
     * @param CountryEntity $entity
     *
     * @return CountryEntity
     */
    public function create(CountryEntity $entity)
    {
        if (!$entity->isNew()) {
            throw new InvalidArgumentException('The entity does already exist.');
        }

        $this->connection->insert(
            self::TABLE,
            $entity->toDatabaseArray()
        );

        $entity->short = $this->connection->lastInsertId();
    }

    /**
     * Update a country in the database.
     *
     * @param CountryEntity $entity
     *
     * @return CountryEntity
     */
    public function update(CountryEntity $entity)

This method seems to be duplicated in your project.

    {
        if ($entity->isNew()) {
            throw new InvalidArgumentException('The entity does not exist.');
        }

        $this->connection->update(
            self::TABLE,
            $entity->toDatabaseArray(),
            ['short' => $entity->short]
        );

        $entity->short = $this->connection->lastInsertId();

        return $entity;
    }

    /**
     * Removes a country from the database.
     *
     * @param CountryEntity $entity
     *
     * @return CountryEntity
     */
    public function remove(CountryEntity $entity)

This method seems to be duplicated in your project.

    {
        if ($entity->isNew()) {
            throw new InvalidArgumentException('The entity does not exist.');
        }

        $this->connection->delete(
            self::TABLE,
            $entity->toDatabaseArray(),
            ['short' => $entity->short]
        );

        $entity->short = null;

        return $entity;
    }
}