Balloon\Reader\GaufretteAdapter

Complexity

Total Complexity

6

Size/Duplication

Total Lines	50
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes	1
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	read()	0	7	2
A	write()	0	10	3

<?php
namespace Balloon\Reader;

use Gaufrette\Filesystem;

/**
 * Class GaufretteAdapter
 * @package Balloon\Reader
 * @author Raphaël Lefebvre <raphael@raphaellefebvre.be>
 */
class GaufretteAdapter implements IFileReader
{

    /**
     * @var Filesystem
     */
    private $gaufrette;

    /**
     * @var string
     */
    private $filePath;

    /**
     * @param Filesystem $gaufrette
     * @param string $filePath
     */
    public function __construct(Filesystem $gaufrette, $filePath)
    {
        $this->gaufrette = $gaufrette;
        $this->filePath = $filePath;
    }

    /**
     * @return mixed

Consider making the return type a bit more specific; maybe use string|boolean.

     */
    public function read()
    {
        if(!$this->gaufrette->has($this->filePath)){
            return '';
        }
        return $this->gaufrette->read($this->filePath);
    }

    /**
     * @param mixed $data
     * @param int $mode
     * @return int

Should the return type not be integer|boolean?

     */
    public function write($data, $mode = 0)
    {
        if($mode & FILE_APPEND){
            $data = $this->read() . $data;
        }
        if(!$this->gaufrette->has($this->filePath)){
            $this->gaufrette->createFile($this->filePath);
        }
        return $this->gaufrette->write($this->filePath, $data, true);
    }
}