for testing and deploying your application
for finding and fixing issues
for empowering human code reviews
<?php
/**
* This file is part of the Cecil/Cecil package.
*
* Copyright (c) Arnaud Ligny <[email protected]>
* For the full copyright and license information, please view the LICENSE
* file that was distributed with this source code.
*/
namespace Cecil\Assets;
use Cecil\Exception\Exception;
use Cecil\Util;
class Asset extends AbstractAsset
{
protected $asset = [];
protected $fileLoaded = false;
* Loads a file.
* @param string $path
* @return self
public function getFile(string $path): self
$filePath = Util::joinFile($this->config->getStaticPath(), $path);
if (!Util::getFS()->exists($filePath)) {
throw new Exception(sprintf('Asset file "%s" doesn\'t exist.', $path));
}
$this->fileLoaded = true;
$fileInfo = new \SplFileInfo($filePath);
$this->asset['path'] = $path;
$this->asset['ext'] = $fileInfo->getExtension();
return $this;
* @return string
public function __toString(): string
return $this->asset['path'];
public function getHtml(): string
if (!$this->fileLoaded) {
throw new Exception(\sprintf('%s() error: you must load a file first.', __FUNCTION__));
switch ($this->asset['ext']) {
case 'css':
return \sprintf('<link rel="stylesheet" href="%s">', $this->asset['path']);
break;
break
The break statement is not necessary if it is preceded for example by a return statement:
return
switch ($x) { case 1: return 'foo'; break; // This break is not necessary and can be left off. }
If you would like to keep this construct to be consistent with other case statements, you can safely mark this issue as a false-positive.
case
default:
return 'POUET';
Cecilapp /
Cecil
The
breakstatement is not necessary if it is preceded for example by areturnstatement:If you would like to keep this construct to be consistent with other
casestatements, you can safely mark this issue as a false-positive.