DoctrineDatatable\Export

Complexity

Total Complexity

6

Size/Duplication

Total Lines	83
Duplicated Lines	0 %

Importance

Changes	2
Bugs	0	Features	1

5 Methods

Rating	Name	Duplication	Size	Complexity
A	export()	0	6	1
A	setDelimiter()	0	5	1
A	data()	0	9	2
A	__construct()	0	3	1
A	setDatatable()	0	5	1

<?php

namespace DoctrineDatatable;

use Doctrine\ORM\Query;

class Export
{
    /**
     * @var Datatable
     */
    private $datatable;

    /**
     * @var resource
     */
    private $resource;

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

    public function __construct()
    {
        $this->setDelimiter("\t");
    }

    /**
     * PRIVATE METHODS.
     */

    /**
     * @author Mathieu Petrini <mathieupetrini@gmail.com>
     */
    private function data(): void
    {
        foreach ($this->datatable->createFinalQuery(array(
            'start' => 0,
            'limit' => INF, ))->getQuery()->iterate(null, Query::HYDRATE_SCALAR) as $row) {
            $line = $row[0];
            fwrite($this->resource, implode($this->delimiter, $line));
        }
        rewind($this->resource);
    }

    /**
     * PUBLIC METHODS.
     */

    /**
     * @author Mathieu Petrini <mathieupetrini@gmail.com>
     *
     * @return resource
     */
    public function export()
    {
        $this->resource = tmpfile();

It seems like tmpfile() can also be of type false. However, the property $resource is declared as type resource. Maybe add an additional type check?

        $this->data();

        return $this->resource;

The expression return $this->resource could also return false which is incompatible with the documented return type resource. Did you maybe forget to handle an error condition?

    }

    /**
     * GETTERS / SETTERS.
     */

    /**
     * @param Datatable $datatable
     *
     * @return Export
     */
    public function setDatatable(Datatable $datatable): self
    {
        $this->datatable = $datatable;

        return $this;
    }

    /**
     * @param string $delimiter
     *
     * @return Export
     */
    public function setDelimiter(string $delimiter): self
    {
        $this->delimiter = $delimiter;

        return $this;
    }
}