Passed
Pull Request — master (#31)
by Josh
03:43
created

TableCell::getRow()   A

Complexity

Conditions 1
Paths 1

Size

Total Lines 4
Code Lines 2

Duplication

Lines 0
Ratio 0 %

Code Coverage

Tests 0
CRAP Score 2

Importance

Changes 2
Bugs 0 Features 0
Metric Value
c 2
b 0
f 0
dl 0
loc 4
ccs 0
cts 2
cp 0
rs 10
cc 1
eloc 2
nc 1
nop 0
crap 2
1
<?php
2
3
namespace Caxy\HtmlDiff\Table;
4
5
class TableCell extends AbstractTableElement
6
{
7
    /**
8
     * @var TableRow
9
     */
10
    protected $row;
11
12
    public function getRow()
13
    {
14
        return $this->row;
15
    }
16
17
    public function setRow(TableRow $row = null)
18
    {
19
        $this->row = $row;
20
21
        if (!in_array($this, $row->getCells())) {
22
            $row->addCell($this);
0 ignored issues
show
Bug introduced by
It seems like $row is not always an object, but can also be of type null. Maybe add an additional type check?

If a variable is not always an object, we recommend to add an additional type check to ensure your method call is safe:

function someFunction(A $objectMaybe = null)
{
    if ($objectMaybe instanceof A) {
        $objectMaybe->doSomething();
    }
}
Loading history...
23
        }
24
25
        return $this;
26
    }
27
28
    public function getColspan()
29
    {
30
        return $this->getAttribute('colspan') ?: 1;
31
    }
32
33
    public function getRowspan()
34
    {
35
        return $this->getAttribute('rowspan') ?: 1;
36
    }
37
}
38