MichaelRubel\Formatters\Collection\DateFormatter

Complexity

Total Complexity

5

Size/Duplication

Total Lines	35
Duplicated Lines	0 %

Test Coverage

Coverage

90.91%

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	15	4
A	format()	0	5	1

<?php

declare(strict_types=1);

namespace MichaelRubel\Formatters\Collection;

use Carbon\CarbonInterface;
use Illuminate\Support\Carbon;
use MichaelRubel\Formatters\Formatter;

class DateFormatter implements Formatter
{
    /**
     * @param  string|CarbonInterface|null  $date
     * @param  string|null  $timezone
     * @param  string|null  $date_format
     */
    public function __construct(
        public string|null|CarbonInterface $date = null,
        public string|null $timezone = null,
        public string|null $date_format = 'Y-m-d',
    ) {
        if (! $this->timezone) {
            $this->timezone = config('app.timezone', 'UTC');
        }

        if (! $this->date instanceof CarbonInterface) {
            $this->date = app(Carbon::class)->parse($this->date);
        }

        if (! $this->date_format) {
            $this->date_format = 'Y-m-d';
        }
    }

    /**
     * Format the date.
     *
     * @return string
     */
    public function format(): string
    {
        return $this->date
            ->setTimezone($this->timezone)

The method setTimezone() does not exist on null.

            ->translatedFormat($this->date_format);

It seems like $this->date_format can also be of type null; however, parameter $format of Carbon\CarbonInterface::translatedFormat() does only seem to accept string, maybe add an additional type check?

    }
}