ker0x\Push\Adapter\Fcm\Message\Data

Complexity

Total Complexity

6

Size/Duplication

Total Lines	54
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	2

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	8	2
A	build()	0	4	1
A	fromArray()	0	13	3

<?php
namespace ker0x\Push\Adapter\Fcm\Message;

use ker0x\Push\Adapter\Fcm\Message\Exception\InvalidDataException;

class Data implements BuilderInterface
{

    /**
     * @var array
     */
    protected $data = [];

    /**
     * Data constructor.
     *
     * @param array|\ker0x\Push\Adapter\Fcm\Message\DataBuilder $dataBuilder The data we want to send.
     * @throws \ker0x\Push\Adapter\Fcm\Message\Exception\InvalidDataException
     */
    public function __construct($dataBuilder)
    {
        if (is_array($dataBuilder)) {
            $dataBuilder = $this->fromArray($dataBuilder);
        }

        $this->data = $dataBuilder->getAllData();

It seems like $dataBuilder->getAllData() can be null. However, the property $data is declared as array. Maybe change the type of the property to array|null or add a type check?

    }

    /**
     * Return data as an array.
     *
     * @return array
     */
    public function build()
    {
        return $this->data;
    }

    /**
     * Build data from an array.
     *
     * @param array $dataArray Array of data for the notification.
     * @return \ker0x\Push\Adapter\Fcm\Message\DataBuilder
     * @throws \ker0x\Push\Adapter\Fcm\Message\Exception\InvalidDataException
     */
    private function fromArray(array $dataArray)
    {
        if (empty($dataArray)) {
            throw InvalidDataException::arrayEmpty();
        }

        $dataBuilder = new DataBuilder();
        foreach ($dataArray as $key => $value) {
            $dataBuilder->addData($key, $value);
        }

        return $dataBuilder;
    }
}