ZsSarkany\LaravelDatabaseStickyTimezone\Connectors\Traits\TimezoneEnforcerTrait

Complexity

Total Complexity

4

Size/Duplication

Total Lines	47
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	getReadPdo()	0	9	2
A	getPdo()	0	9	2

<?php

declare(strict_types=1);

namespace ZsSarkany\LaravelDatabaseStickyTimezone\Connectors\Traits;

use ZsSarkany\LaravelDatabaseStickyTimezone\ChangeDetector;

trait TimezoneEnforcerTrait
{
    /**
     * Change detector for $pdo property
     *
     * @var ChangeDetector
     */
    protected $pdoChangeDetector;

    /**
     * Change detector for $readPdo property
     *
     * @var ChangeDetector
     */
    protected $readPdoChangeDetector;

    /**
     * Get the current PDO connection.
     *
     * @return \PDO|null
     */
    public function getPdo(): ?\PDO
    {
        if (!$this->pdoChangeDetector) {
            $this->pdoChangeDetector = new ChangeDetector;
        }

        return $this->enforcePdoTimezone(

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

            parent::getPdo(),
            $this->pdoChangeDetector
        );
    }

    /**
     * Get the current PDO connection used for reading.
     *
     * @return \PDO|null
     */
    public function getReadPdo(): ?\PDO
    {
        if (!$this->readPdoChangeDetector) {
            $this->readPdoChangeDetector = new ChangeDetector;
        }

        return $this->enforcePdoTimezone(
            parent::getReadPdo(),
            $this->readPdoChangeDetector
        );
    }
}