ObjectType::convertToPHPValue() - Code Metrics - Inspection of "Change type from STRING to BINARY for storing obje..." - doctrine/dbal - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Pull Request — develop (#3453)

by Evgeniy

created 2019-02-25 19:08 UTC

ObjectType::convertToPHPValue() A

↳ Parent: ObjectType

Complexity

Conditions	3
Paths	3

Size

Total Lines	16
Code Lines	9

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	8
CRAP Score	3

Importance

Changes

Metric	Value
eloc	9
dl	0
loc	16
ccs	8
cts	8
cp	1
rs	9.9666
c	0
b	0
f	0
cc	3
nc	3
nop	2
crap	3

<?php

namespace Doctrine\DBAL\Types;

use Doctrine\DBAL\ParameterType;
use Doctrine\DBAL\Platforms\AbstractPlatform;
use function is_resource;
use function restore_error_handler;
use function serialize;
use function set_error_handler;
use function stream_get_contents;
use function unserialize;

/**
 * Type that maps a PHP object to a clob SQL type.
 */
class ObjectType extends Type
{
    /**
     * {@inheritdoc}
     */
    public function getSQLDeclaration(array $fieldDeclaration, AbstractPlatform $platform)
    {
        return $platform->getBlobTypeDeclarationSQL($fieldDeclaration);
    }

    /**
     * {@inheritdoc}
     */
    public function convertToDatabaseValue($value, AbstractPlatform $platform)
    {
        return serialize($value);
    }

    /**
     * {@inheritdoc}
     */
    public function convertToPHPValue($value, AbstractPlatform $platform)
    {
        if ($value === null) {
            return null;
        }

        $value = is_resource($value) ? stream_get_contents($value) : $value;

        set_error_handler(function (int $code, string $message) : void {
            throw ConversionException::conversionFailedUnserialization($this->getName(), $message);
        });

        try {
            return unserialize($value);
        } finally {
            restore_error_handler();
        }
    }

    /**
     * {@inheritdoc}
     */
    public function getBindingType() : int
    {
        return ParameterType::BINARY;
    }

    /**
     * {@inheritdoc}
     */
    public function getName()
    {
        return Type::OBJECT;
    }

    /**
     * {@inheritdoc}
     */
    public function requiresSQLCommentHint(AbstractPlatform $platform)
    {
        return true;
    }
}


1		<?php
2
3		namespace Doctrine\DBAL\Types;
4
5		use Doctrine\DBAL\ParameterType;
6		use Doctrine\DBAL\Platforms\AbstractPlatform;
7		use function is_resource;
8		use function restore_error_handler;
9		use function serialize;
10		use function set_error_handler;
11		use function stream_get_contents;
12		use function unserialize;
13
14		/**
15		* Type that maps a PHP object to a clob SQL type.
16		*/
17		class ObjectType extends Type
18		{
19		/**
20		* {@inheritdoc}
21		*/
22	607	public function getSQLDeclaration(array $fieldDeclaration, AbstractPlatform $platform)
23		{
24	607	return $platform->getBlobTypeDeclarationSQL($fieldDeclaration);
25		}
26
27		/**
28		* {@inheritdoc}
29		*/
30	81	public function convertToDatabaseValue($value, AbstractPlatform $platform)
31		{
32	81	return serialize($value);
33		}
34
35		/**
36		* {@inheritdoc}
37		*/
38	168	public function convertToPHPValue($value, AbstractPlatform $platform)
39		{
40	168	if ($value === null) {
41	29	return null;
42		}
43
44	139	$value = is_resource($value) ? stream_get_contents($value) : $value;
45
46		set_error_handler(function (int $code, string $message) : void {
47	29	throw ConversionException::conversionFailedUnserialization($this->getName(), $message);
48	139	});
49
50		try {
51	139	return unserialize($value);
52		} finally {
53	139	restore_error_handler();
54		}
55		}
56
57		/**
58		* {@inheritdoc}
59		*/
60	28	public function getBindingType() : int
61		{
62	28	return ParameterType::BINARY;
63		}
64
65		/**
66		* {@inheritdoc}
67		*/
68	984	public function getName()
69		{
70	984	return Type::OBJECT;
71		}
72
73		/**
74		* {@inheritdoc}
75		*/
76	17768	public function requiresSQLCommentHint(AbstractPlatform $platform)
77		{
78	17768	return true;
79		}
80		}
81

doctrine / dbal

Pull Request — develop (#3453)

ObjectType::convertToPHPValue() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like