Issues in AbstractEngine.php (master) - Issues in master - o2system/parser - Measure and Improve Code Quality continuously with Scrutinizer

Issues (32)

src/Template/Abstracts/AbstractEngine.php (3 issues)

Labels

Severity

<?php
/**
 * This file is part of the O2System Framework package.
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 *
 * @author         Steeve Andrian Salim
 * @copyright      Copyright (c) Steeve Andrian Salim
 */

// ------------------------------------------------------------------------

namespace O2System\Parser\Template\Abstracts;

// ------------------------------------------------------------------------

use O2System\Spl\Traits\Collectors\FileExtensionCollectorTrait;
use O2System\Spl\Traits\Collectors\FilePathCollectorTrait;

/**
 * Class AbstractEngine
 *
 * @package O2System\Parser\Template\Abstracts
 */
abstract class AbstractEngine
{
    use FileExtensionCollectorTrait;
    use FilePathCollectorTrait;

    // ------------------------------------------------------------------------

    /**
     * AbstractEngine::parsePartials
     *
     * @param string  $filename
     * @param array   $vars
     * @param string  $optionalFilename
     *
     * @return string|null
     */
    public function parsePartials($filename, $vars = null, $optionalFilename = null)
    {
        if (empty($vars)) {
            if (isset($optionalFilename)) {
                return $this->parseFile($optionalFilename);
            }
        } else {
            return $this->parseFile($filename, $vars);
        }

        return null;
    }

    // ------------------------------------------------------------------------

    /**
     * AbstractEngine::parseFile
     *
     * @param string $filePath
     * @param array  $vars
     *
     * @return string
     */
    public function parseFile($filePath, array $vars = [])
    {
        if (class_exists('O2System\Framework', false)) {
            return view()->load($filePath, $vars, true);
            return /** @scrutinizer ignore-call */ view()->load($filePath, $vars, true);
        } else {
            $fileExtension = '.' . pathinfo($filePath, PATHINFO_EXTENSION);

            if (in_array($fileExtension, $this->fileExtensions) AND is_file($filePath)) {
                return $this->parseString(file_get_contents($filePath), $vars);
                return $this->/** @scrutinizer ignore-call */ parseString(file_get_contents($filePath), $vars);
            }

            // Try to find from filePaths
            if (count($this->filePaths)) {
                foreach ($this->filePaths as $fileDirectory) {
                    $checkFilePath = $fileDirectory . $filePath;

                    if (in_array($fileExtension, $this->fileExtensions) AND is_file($checkFilePath)) {
                        return $this->parseString(file_get_contents($checkFilePath), $vars);
                        break;
switch ($x) {
    case 1:
        return 'foo';
        break; // This break is not necessary and can be left off.
}
                    }
                }
            }
        }

        return null;
    }
}

GitHub Access Token became invalid

Issues (32)

src/Template/Abstracts/AbstractEngine.php (3 issues)

Labels

Severity

Introduced By

o2system / parser

GitHub Access Token became invalid

Issues (32)

src/Template/Abstracts/AbstractEngine.php (3 issues)

Labels

Severity

Introduced By

Duplication Side-by-Side

Filter issues like