Completed
Push — master ( e5585d...67bac6 )
by Colin
02:44
created

TableCellRenderer   A

Complexity

Total Complexity 5

Size/Duplication

Total Lines 20
Duplicated Lines 0 %

Coupling/Cohesion

Components 1
Dependencies 1

Test Coverage

Coverage 77.78%

Importance

Changes 0
Metric Value
wmc 5
lcom 1
cbo 1
dl 0
loc 20
ccs 7
cts 9
cp 0.7778
rs 10
c 0
b 0
f 0

1 Method

Rating   Name   Duplication   Size   Complexity  
A render() 0 17 5
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
use League\CommonMark\Util\Xml;
23
24
class TableCellRenderer implements BlockRendererInterface
25
{
26 34
    public function render(AbstractBlock $block, ElementRendererInterface $htmlRenderer, bool $inTightList = false)
27
    {
28 34
        if (!$block instanceof TableCell) {
29
            throw new \InvalidArgumentException('Incompatible block type: '.get_class($block));
30
        }
31
32 34
        $attrs = [];
33 34
        foreach ($block->getData('attributes', []) as $key => $value) {
34
            $attrs[$key] = Xml::escape($value);
35
        }
36
37 34
        if ($block->align) {
0 ignored issues
show
Bug Best Practice introduced by
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...
38 8
            $attrs['style'] = (isset($attrs['style']) ? $attrs['style'].' ' : '').'text-align: '.$block->align;
39
        }
40
41 34
        return new HtmlElement($block->type, $attrs, $htmlRenderer->renderInlines($block->children()));
42
    }
43
}
44