Skip to content
Projects
Groups
Snippets
Help
Loading...
Help
Submit feedback
Contribute to GitLab
Sign in / Register
Toggle navigation
D
doctrine-dbal
Project
Project
Details
Activity
Releases
Cycle Analytics
Repository
Repository
Files
Commits
Branches
Tags
Contributors
Graph
Compare
Charts
Issues
0
Issues
0
List
Boards
Labels
Milestones
Merge Requests
0
Merge Requests
0
CI / CD
CI / CD
Pipelines
Jobs
Schedules
Charts
Wiki
Wiki
Snippets
Snippets
Members
Members
Collapse sidebar
Close sidebar
Activity
Graph
Charts
Create a new issue
Jobs
Commits
Issue Boards
Open sidebar
Tomáš Trávníček
doctrine-dbal
Commits
16e2d112
Commit
16e2d112
authored
Dec 12, 2010
by
Benjamin Eberlei
Browse files
Options
Browse Files
Download
Email Patches
Plain Diff
DBAL-73 - Allow false to be handled correctly by Array and Object types.
parent
8cbb70c6
Changes
4
Hide whitespace changes
Inline
Side-by-side
Showing
4 changed files
with
35 additions
and
4 deletions
+35
-4
ArrayType.php
lib/Doctrine/DBAL/Types/ArrayType.php
+1
-3
ObjectType.php
lib/Doctrine/DBAL/Types/ObjectType.php
+18
-1
ArrayTest.php
tests/Doctrine/Tests/DBAL/Types/ArrayTest.php
+8
-0
ObjectTest.php
tests/Doctrine/Tests/DBAL/Types/ObjectTest.php
+8
-0
No files found.
lib/Doctrine/DBAL/Types/ArrayType.php
View file @
16e2d112
<?php
/*
* $Id$
*
* THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
* "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
* LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
...
...
@@ -46,7 +44,7 @@ class ArrayType extends Type
$value
=
(
is_resource
(
$value
))
?
stream_get_contents
(
$value
)
:
$value
;
$val
=
unserialize
(
$value
);
if
(
$val
===
false
)
{
if
(
$val
===
false
&&
$value
!=
'b:0;'
)
{
throw
ConversionException
::
conversionFailed
(
$value
,
$this
->
getName
());
}
return
$val
;
...
...
lib/Doctrine/DBAL/Types/ObjectType.php
View file @
16e2d112
<?php
/*
* THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
* "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
* LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
* A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
* OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
* SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
* LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
* DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
* THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
* (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
* OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
*
* This software consists of voluntary contributions made by many individuals
* and is licensed under the LGPL. For more information, see
* <http://www.doctrine-project.org>.
*/
namespace
Doctrine\DBAL\Types
;
...
...
@@ -27,7 +44,7 @@ class ObjectType extends Type
$value
=
(
is_resource
(
$value
))
?
stream_get_contents
(
$value
)
:
$value
;
$val
=
unserialize
(
$value
);
if
(
$val
===
false
)
{
if
(
$val
===
false
&&
$value
!==
'b:0;'
)
{
throw
ConversionException
::
conversionFailed
(
$value
,
$this
->
getName
());
}
return
$val
;
...
...
tests/Doctrine/Tests/DBAL/Types/ArrayTest.php
View file @
16e2d112
...
...
@@ -50,4 +50,12 @@ class ArrayTest extends \Doctrine\Tests\DbalTestCase
{
$this
->
assertNull
(
$this
->
_type
->
convertToPHPValue
(
null
,
$this
->
_platform
));
}
/**
* @group DBAL-73
*/
public
function
testFalseConversion
()
{
$this
->
assertFalse
(
$this
->
_type
->
convertToPHPValue
(
serialize
(
false
),
$this
->
_platform
));
}
}
\ No newline at end of file
tests/Doctrine/Tests/DBAL/Types/ObjectTest.php
View file @
16e2d112
...
...
@@ -45,4 +45,12 @@ class ObjectTest extends \Doctrine\Tests\DbalTestCase
{
$this
->
assertNull
(
$this
->
_type
->
convertToPHPValue
(
null
,
$this
->
_platform
));
}
/**
* @group DBAL-73
*/
public
function
testFalseConversion
()
{
$this
->
assertFalse
(
$this
->
_type
->
convertToPHPValue
(
serialize
(
false
),
$this
->
_platform
));
}
}
\ No newline at end of file
Write
Preview
Markdown
is supported
0%
Try again
or
attach a new file
Attach a file
Cancel
You are about to add
0
people
to the discussion. Proceed with caution.
Finish editing this message first!
Cancel
Please
register
or
sign in
to comment