MaxBeckers\AmazonAlexa\Request\Request\AlexaSkillEvent\AlexaSkillEventRequest

Complexity

Total Complexity

3

Size/Duplication

Total Lines	38
Duplicated Lines	0 %

Test Coverage

Coverage

80%

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	setTime()	0	7	2
A	setRequestData()	0	9	1

<?php

namespace MaxBeckers\AmazonAlexa\Request\Request\AlexaSkillEvent;

use MaxBeckers\AmazonAlexa\Request\Request\AbstractRequest;

/**
 * @author Maximilian Beckers <beckers.maximilian@gmail.com>
 */
abstract class AlexaSkillEventRequest extends AbstractRequest
{
    /**
     * @var string
     */
    public $eventCreationTime;

    /**
     * @var string
     */
    public $eventPublishingTime;

    /**
     * @var string
     */
    public $requestId;

    /**
     * @param array $amazonRequest
     */
    protected function setRequestData(array $amazonRequest)
    {
        $this->requestId = $amazonRequest['requestId'];

        $this->setTime('timestamp', $amazonRequest['timeStamp']);
        $this->setTime('eventCreationTime', $amazonRequest['eventCreationTime']);
        $this->setTime('eventPublishingTime', $amazonRequest['eventPublishingTime']);

        $this->locale = $amazonRequest['locale'];

The property locale does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

    }

    private function setTime($attribute, $value)
    {
        //Workaround for amazon developer console sending unix timestamp
        try {
            $this->{$attribute} = new \DateTime($value);
        } catch (\Exception $e) {
            $this->{$attribute} = (new \DateTime())->setTimestamp(intval($value / 1000));
        }
    }
}