MisterIcy\ExcelWriter\Properties\TimeProperty

Complexity

Total Complexity

3

Size/Duplication

Total Lines	28
Duplicated Lines	0 %

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	3	1
A	renderProperty()	0	11	2

<?php
declare(strict_types=1);

namespace MisterIcy\ExcelWriter\Properties;

use MisterIcy\ExcelWriter\Exceptions\PropertyException;
use MisterIcy\ExcelWriter\Properties\Traits\DateTimeTrait;
use PhpOffice\PhpSpreadsheet\Style\NumberFormat;

/**
 * Class TimeProperty
 * @package MisterIcy\ExcelWriter\Properties
 */
final class TimeProperty extends AbstractProperty
{
    use DateTimeTrait;

    /**
     * TimeProperty constructor.
     */
    public function __construct()
    {
        $this->formatCode = NumberFormat::FORMAT_DATE_TIME3;
    }

    /**
     * @param object $object
     * @return float|mixed
     * @throws PropertyException
     */
    public function renderProperty(object $object)
    {
        $rendered = parent::renderProperty($object);
        if (!$this->isFormula) {
            return (
                $this->convertDateTimeToExcelTime(
                    $rendered

It seems like $rendered can also be of type string; however, parameter $object of MisterIcy\ExcelWriter\Properties\TimeProperty::convertDateTimeToExcelTime() does only seem to accept DateTimeInterface|null, maybe add an additional type check?

                )
            );
        }
        return $rendered;
    }
}