Skip to content
This repository was archived by the owner on Oct 26, 2024. It is now read-only.

Commit 1bdab00

Browse files
committed
Cleaning up Scrutinizer suggestions. Enhanced checks against null
1 parent f483819 commit 1bdab00

File tree

2 files changed

+2
-8
lines changed

2 files changed

+2
-8
lines changed

Data/DataCollection.php

Lines changed: 2 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -23,17 +23,12 @@ final class DataCollection extends Data implements DataInterface, \Iterator
2323
*/
2424
private $position;
2525

26-
/**
27-
* @var integer
28-
*/
29-
private $totalCount = 0;
30-
3126
/**
3227
* @param array $data
3328
*/
3429
public function __construct($data = null)
3530
{
36-
if (null != $data && is_array($data)) {
31+
if (null !== $data && is_array($data)) {
3732
$this->set('items', $data);
3833
}
3934

@@ -105,7 +100,7 @@ public function rewind()
105100
*/
106101
public function addItem($item, $position = null)
107102
{
108-
if (null == $position) {
103+
if (null === $position) {
109104
$items = $this->get('items', array());
110105
array_push($items, $item);
111106
$this->set('items', $items);

Tests/Config/ConfigEnvTest.php

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -42,7 +42,6 @@ public function testFactoryProd()
4242
}
4343

4444
/**
45-
* @covers \Asm\Config\ConfigEnv::init
4645
* @covers \Asm\Config\ConfigEnv::mergeEnvironments
4746
* @return \Asm\Config\ConfigInterface
4847
*/

0 commit comments

Comments
 (0)