for testing and deploying your application
for finding and fixing issues
for empowering human code reviews
<?php
/**
* This file is part of laravel.su package.
*
* For the full copyright and license information, please view the LICENSE
* file that was distributed with this source code.
*/
declare(strict_types=1);
namespace App\Services\ContentRenderer;
use Illuminate\Support\Str;
* Class ContentHeadersRenderer.
class ContentHeadersRenderer
{
* @var string
private $body;
* @var array
private $links = [];
private $content = '';
private $tags = [
'h6' => 'h6',
'h5' => 'h6',
'h4' => 'h5',
'h3' => 'h4',
'h2' => 'h3',
'h1' => 'h2',
];
* HeadersPrerender constructor.
* @param string $body
public function __construct(string $body)
$this->body = $body;
}
* @return ContentHeadersRenderer
public function parse(): ContentHeadersRenderer
$this->links = [];
$this->content = preg_replace_callback('/<(h\d+)>(.*?)<\/h\d+>/isu', function ($data) {
list($data, $tag, $header) = $data;
$data
list($first,,$third)
This checks looks for assignemnts to variables using the list(...) function, where not all assigned variables are subsequently used.
list(...)
Consider the following code example.
<?php function returnThreeValues() { return array('a', 'b', 'c'); } list($a, $b, $c) = returnThreeValues(); print $a . " - " . $c;
Only the variables $a and $c are used. There was no need to assign $b.
$a
$c
$b
Instead, the list call could have been.
list($a,, $c) = returnThreeValues();
$id = Str::slug($header);
foreach ($this->tags as $from => $to) {
$tag = str_replace($from, $to, $tag);
$this->links[] = [
'anchor' => $id,
'title' => $header,
'level' => $tag,
$link = '<%s><a href="#%s" class="anchor" name="%s"></a>%s</%s>';
return sprintf($link, $tag, $id, $id, $header, $tag);
}, $this->body);
return $this;
* @return array
public function getLinks(): array
return $this->links;
* @return string
public function getContent(): string
return $this->content;
LaravelRUS /
laravel.ru
This checks looks for assignemnts to variables using the
list(...)function, where not all assigned variables are subsequently used.Consider the following code example.
Only the variables
$aand$care used. There was no need to assign$b.Instead, the list call could have been.