Skip to content

Commit c81d63d

Browse files
committed
Do not filter values in AbstractCreationalQuery
1 parent 8991871 commit c81d63d

File tree

3 files changed

+25
-6
lines changed

3 files changed

+25
-6
lines changed

src/Manipulation/AbstractCreationalQuery.php

+1-6
Original file line numberDiff line numberDiff line change
@@ -50,12 +50,7 @@ public function getValues()
5050
*/
5151
public function setValues(array $values)
5252
{
53-
$this->values = \array_filter($values, function($value) {
54-
if (is_int($value)) {
55-
return true;
56-
}
57-
return $value;
58-
});
53+
$this->values = $values;
5954

6055
return $this;
6156
}

tests/Manipulation/InsertTest.php

+12
Original file line numberDiff line numberDiff line change
@@ -62,4 +62,16 @@ public function itShouldGetColumns()
6262

6363
$this->assertInstanceOf('NilPortugues\Sql\QueryBuilder\Syntax\Column', $columns[0]);
6464
}
65+
66+
/**
67+
* @test
68+
*/
69+
public function itShouldSetNullableValues()
70+
{
71+
$values = ['user_id' => 1, 'description' => null, 'isVisible' => false];
72+
73+
$this->query->setValues($values);
74+
75+
$this->assertSame($values, $this->query->getValues());
76+
}
6577
}

tests/Manipulation/UpdateTest.php

+12
Original file line numberDiff line numberDiff line change
@@ -58,4 +58,16 @@ public function itShouldReturnValues()
5858

5959
$this->assertSame($values, $this->query->getValues());
6060
}
61+
62+
/**
63+
* @test
64+
*/
65+
public function itShouldSetNullableValues()
66+
{
67+
$values = ['user_id' => 1, 'description' => null, 'isVisible' => false];
68+
69+
$this->query->setValues($values);
70+
71+
$this->assertSame($values, $this->query->getValues());
72+
}
6173
}

0 commit comments

Comments
 (0)