AmericanExpress\HyperledgerFabricClient\ProtoFactory\TimestampFactory

Complexity

Total Complexity

2

Size/Duplication

Total Lines	15
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
A	fromDateTime()	0	9	2

<?php

/**
 * Copyright 2017 American Express Travel Related Services Company, Inc.
 *
 * Licensed under the Apache License, Version 2.0 (the "License");
 * you may not use this file except in compliance with the License.
 * You may obtain a copy of the License at
 *
 * http://www.apache.org/licenses/LICENSE-2.0
 *
 * Unless required by applicable law or agreed to in writing, software
 * distributed under the License is distributed on an "AS IS" BASIS,
 * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express
 * or implied. See the License for the specific language governing
 * permissions and limitations under the License.
 */

declare(strict_types=1);

namespace AmericanExpress\HyperledgerFabricClient\ProtoFactory;

use Google\Protobuf\Timestamp;

class TimestampFactory
{
    /**
     * @param \DateTime|null $dateTime
     * @return Timestamp
     */
    public static function fromDateTime(\DateTime $dateTime = null): Timestamp
    {
        $dateTime = $dateTime ?: new \DateTime('now', timezone_open('UTC'));

It seems like timezone_open('UTC') can also be of type false; however, parameter $timezone of DateTime::__construct() does only seem to accept null|DateTimeZone, maybe add an additional type check?

        $timestamp = new Timestamp();
        $timestamp->setSeconds($dateTime->format('U'));
        $timestamp->setNanos($dateTime->format('u') * 1000);

        return $timestamp;
    }
}