Bavix\SDK\FileLoader\IniLoader

Complexity

Total Complexity

5

Size/Duplication

Total Lines	45
Duplicated Lines	0 %

Test Coverage

Coverage

0%

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	asArray()	0	8	2
A	save()	0	6	1
A	writer()	0	8	2

<?php

namespace Bavix\SDK\FileLoader;

class IniLoader implements DataInterface
{

    use DataTrait;

    /**
     * @var IniWriter
     */
    protected $writer;

    /**
     * @return IniWriter
     */
    protected function writer()
    {
        if (!$this->writer)
        {
            $this->writer = new IniWriter();
        }

        return $this->writer;
    }

    /**
     * @inheritdoc
     */
    public function asArray()
    {
        if (!$this->data)

The expression $this->data of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using empty($expr) instead to make it clear that you intend to check for an array without elements.

        {
            $this->data = \parse_ini_file($this->path, true);

It seems like parse_ini_file($this->path, true) can also be of type false. However, the property $data is declared as type array. Maybe add an additional type check?

        }

        return $this->data;
    }

    /**
     * @inheritdoc
     */
    public function save($data)
    {
        $data = $this->_fromArray($data);

        return (int)$this->writer()
            ->toFile($this->path, $data);
    }

}