TableCellRenderer::render() - Code Metrics - thephpleague/commonmark-ext-table - Measure and Improve Code Quality continuously with Scrutinizer

TableCellRenderer::render() A
last analyzed 2020-04-04 15:58 UTC

↳ Parent: TableCellRenderer

Complexity

Conditions	4
Paths	4

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	6
CRAP Score	4.0466

Importance

Changes

Metric	Value
dl	0
loc	14
ccs	6
cts	7
cp	0.8571
rs	9.7998
c	0
b	0
f	0
cc	4
nc	4
nop	3
crap	4.0466

<?php

declare(strict_types=1);

/*
 * This is part of the league/commonmark-ext-table package.
 *
 * (c) Martin Hasoň <[email protected]>
 * (c) Webuni s.r.o. <[email protected]>
 * (c) Colin O'Dell <[email protected]>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace League\CommonMark\Ext\Table;

use League\CommonMark\Block\Element\AbstractBlock;
use League\CommonMark\Block\Renderer\BlockRendererInterface;
use League\CommonMark\ElementRendererInterface;
use League\CommonMark\HtmlElement;

final class TableCellRenderer implements BlockRendererInterface
{
    public function render(AbstractBlock $block, ElementRendererInterface $htmlRenderer, bool $inTightList = false)
    {
        if (!$block instanceof TableCell) {
            throw new \InvalidArgumentException('Incompatible block type: '.get_class($block));
        }

        $attrs = $block->getData('attributes', []);

        if ($block->align) {
''   == false // true
''   == null  // true
'ab' == false // false
'ab' == null  // false

// It is often better to use strict comparison
'' === false // false
'' === null  // false
            $attrs['style'] = (isset($attrs['style']) ? $attrs['style'].' ' : '').'text-align: '.$block->align;
        }

        return new HtmlElement($block->type, $attrs, $htmlRenderer->renderInlines($block->children()));
    }
}


1		<?php
2
3		declare(strict_types=1);
4
5		/*
6		* This is part of the league/commonmark-ext-table package.
7		*
8		* (c) Martin Hasoň <[email protected]>
9		* (c) Webuni s.r.o. <[email protected]>
10		* (c) Colin O'Dell <[email protected]>
11		*
12		* For the full copyright and license information, please view the LICENSE
13		* file that was distributed with this source code.
14		*/
15
16		namespace League\CommonMark\Ext\Table;
17
18		use League\CommonMark\Block\Element\AbstractBlock;
19		use League\CommonMark\Block\Renderer\BlockRendererInterface;
20		use League\CommonMark\ElementRendererInterface;
21		use League\CommonMark\HtmlElement;
22
23		final class TableCellRenderer implements BlockRendererInterface
24		{
25	34	public function render(AbstractBlock $block, ElementRendererInterface $htmlRenderer, bool $inTightList = false)
26		{
27	34	if (!$block instanceof TableCell) {
28		throw new \InvalidArgumentException('Incompatible block type: '.get_class($block));
29		}
30
31	34	$attrs = $block->getData('attributes', []);
32
33	34	if ($block->align) {
		0 ignored issues – show Bug Best Practice introduced 2019-07-11 12:18 UTC by Report Bug Copy Issue Report The expression `$block->align` of type `null\|string` is loosely compared to `true`; this is ambiguous if the string can be empty. You might want to explicitly use `!== null` instead. In PHP, under loose comparison (like `==`, or `!=`, or `switch` conditions), values of different types might be equal. For `string` values, the empty string `''` is a special case, in particular the following results might be unexpected: '' == false // true '' == null // true 'ab' == false // false 'ab' == null // false // It is often better to use strict comparison '' === false // false '' === null // false Loading history...
34	8	$attrs['style'] = (isset($attrs['style']) ? $attrs['style'].' ' : '').'text-align: '.$block->align;
35		}
36
37	34	return new HtmlElement($block->type, $attrs, $htmlRenderer->renderInlines($block->children()));
38		}
39		}
40

thephpleague / commonmark-ext-table

TableCellRenderer::render() A last analyzed 2020-04-04 15:58 UTC

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like

TableCellRenderer::render() A
last analyzed 2020-04-04 15:58 UTC