App\BxConsole\Format\XlsCommand::writeXls()

Complexity

Conditions	4
Paths	4

Size

Total Lines	27
Code Lines	16

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	1

<?php
    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;
    }