App\BxConsole\Format\XlsCommand

Complexity

Total Complexity

6

Size/Duplication

Total Lines	39
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	1

3 Methods

Rating	Name	Duplication	Size	Complexity
A	getTmpXlsFile()	0	3	1
A	writeXls()	0	27	4
A	getXlsFile()	0	3	1

<?php
namespace App\BxConsole\Format;

use Psr\Log\LoggerInterface;

trait XlsCommand {

    public function getXlsFile() {

        return $this->getDocumentRoot() . static::FINAL_FOLDER . static::XLS_FILE;

The constant App\BxConsole\Format\XlsCommand::FINAL_FOLDER was not found. Maybe you did not declare it correctly or list all dependencies?

It seems like getDocumentRoot() must be provided by classes using this trait. How about adding it as abstract method to this trait?

The constant App\BxConsole\Format\XlsCommand::XLS_FILE was not found. Maybe you did not declare it correctly or list all dependencies?

    }

    protected function getTmpXlsFile() {

        return $this->getDocumentRoot() . static::TMP_FOLDER . static::XLS_FILE;

The constant App\BxConsole\Format\XlsCommand::TMP_FOLDER was not found. Maybe you did not declare it correctly or list all dependencies?

The constant App\BxConsole\Format\XlsCommand::XLS_FILE was not found. Maybe you did not declare it correctly or list all dependencies?

    }

    public function writeXls($html) {

        $htmlFile = str_replace('.xls', '.html', static::XLS_FILE);

The constant App\BxConsole\Format\XlsCommand::XLS_FILE was not found. Maybe you did not declare it correctly or list all dependencies?

        $htmlRes = file_put_contents($this->getDocumentRoot() . static::TMP_FOLDER . $htmlFile, $html);

The constant App\BxConsole\Format\XlsCommand::TMP_FOLDER was not found. Maybe you did not declare it correctly or list all dependencies?

        if($htmlRes) {

            //TODO: check size !!!
            chdir($this->getDocumentRoot() . static::TMP_FOLDER);
            $command = sprintf("libreoffice --calc --convert-to xls %s", $htmlFile);
            exec($command);
            if(file_exists($this->getTmpXlsFile())) {
                unlink($this->getDocumentRoot() . static::TMP_FOLDER . $htmlFile);
                $command = sprintf("mv %s %s", $this->getTmpXlsFile(), $this->getXlsFile());
                exec($command);
                return filesize($this->getXlsFile());
            }

        } else {
            /** @var LoggerInterface $logger */
            $logger = $this->getLogger();

It seems like getLogger() must be provided by classes using this trait. How about adding it as abstract method to this trait?

            if($logger) {

$logger is of type Psr\Log\LoggerInterface, thus it always evaluated to true.

                $logger->error($this->getName() . ' ERROR: unable write file!');

It seems like getName() must be provided by classes using this trait. How about adding it as abstract method to this trait?

            }
        }

        return 0;
    }
}