Meta::get() - Code Metrics - Inspection of "normalize insert" - tarantool-php/mapper - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Branch — master (0bf65c)

by Dmitry

created 2016-03-23 18:15 UTC

Meta::get() B

↳ Parent: Meta

Complexity

Conditions	5
Paths	4

Size

Total Lines	21
Code Lines	12

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	14
CRAP Score	5.0073

Importance

Changes	2
Bugs	0	Features	0

Metric	Value
c	2
b	0
f	0
dl	0
loc	21
ccs	14
cts	15
cp	0.9333
rs	8.7624
cc	5
eloc	12
nc	4
nop	1
crap	5.0073

<?php

namespace Tarantool\Mapper\Schema;

use Tarantool\Mapper\Contracts;
use LogicException;

class Meta implements Contracts\Meta
{
    protected $manager;
    protected $types = [];

    public function __construct(Contracts\Manager $manager)
    {
        $this->manager = $manager;
    }

    /**
     * @return Type
     */
    public function get($type)
    {
        if (!array_key_exists($type, $this->types)) {
            if (!$this->manager->getSchema()->hasSpace($type)) {
                throw new LogicException("Type $type not exists");
            }

            $fields = [];
            if ($type != 'mapping') {
                $mapping = $this->manager->get('mapping')->find(['space' => $type]);
                foreach ($mapping as $row) {
                    $fields[$row->line] = $row->property;
interface SomeInterface { }
class SomeClass implements SomeInterface {
    public $a;
}

function someFunction(SomeInterface $object) {
    if ($object instanceof SomeClass) {
        $a = $object->a;
    }
}
                }
                ksort($fields);
            }


            $this->types[$type] = new Type($this->manager, $type, array_values($fields));
        }
        return $this->types[$type];
    }

    /**
     * @return Type
     */
    public function create($type, array $fields = null)
    {
        if ($this->manager->getSchema()->hasSpace($type)) {
            throw new LogicException("Type $type exists");
        }

        $this->manager->getSchema()->createSpace($type);

        $instance = new Type($this->manager, $type);

        $instance->addProperty('id');
        $instance->addIndex('id');

        if ($fields) {
            call_user_func_array([$instance, 'addProperty'], (array) $fields);
        }

        return $this->types[$type] = $instance;
    }
}


Branch — master (0bf65c)

Meta::get() B

Complexity

Size

Duplication

Code Coverage

Importance

Available Fixes

Available Fixes

1		<?php
2
3		namespace Tarantool\Mapper\Schema;
4
5		use Tarantool\Mapper\Contracts;
6		use LogicException;
7
8		class Meta implements Contracts\Meta
9		{
10		protected $manager;
11		protected $types = [];
12
13	4	public function __construct(Contracts\Manager $manager)
14		{
15	4	$this->manager = $manager;
16	4	}
17
18		/**
19		* @return Type
20		*/
21	4	public function get($type)
22		{
23	4	if (!array_key_exists($type, $this->types)) {
24	4	if (!$this->manager->getSchema()->hasSpace($type)) {
25		throw new LogicException("Type $type not exists");
26		}
27
28	4	$fields = [];
29	4	if ($type != 'mapping') {
30	4	$mapping = $this->manager->get('mapping')->find(['space' => $type]);
31	4	foreach ($mapping as $row) {
32	4	$fields[$row->line] = $row->property;
		0 ignored issues – show Bug introduced 2016-03-23 13:03 UTC by Report Bug Copy Issue Report Accessing `line` on the interface `Tarantool\Mapper\Contracts\Entity` suggest that you code against a concrete implementation. How about adding an `instanceof` check? If you access a property on an interface, you most likely code against a concrete implementation of the interface. Available Fixes Adding an additional type check: interface SomeInterface { } class SomeClass implements SomeInterface { public $a; } function someFunction(SomeInterface $object) { if ($object instanceof SomeClass) { $a = $object->a; } } Changing the type hint: interface SomeInterface { } class SomeClass implements SomeInterface { public $a; } function someFunction(SomeClass $object) { $a = $object->a; } Loading history... Bug introduced 2016-03-23 13:03 UTC by Report Bug Copy Issue Report Accessing `property` on the interface `Tarantool\Mapper\Contracts\Entity` suggest that you code against a concrete implementation. How about adding an `instanceof` check? If you access a property on an interface, you most likely code against a concrete implementation of the interface. Available Fixes Adding an additional type check: interface SomeInterface { } class SomeClass implements SomeInterface { public $a; } function someFunction(SomeInterface $object) { if ($object instanceof SomeClass) { $a = $object->a; } } Changing the type hint: interface SomeInterface { } class SomeClass implements SomeInterface { public $a; } function someFunction(SomeClass $object) { $a = $object->a; } Loading history...
33	4	}
34	4	ksort($fields);
35	4	}
36
37
38	4	$this->types[$type] = new Type($this->manager, $type, array_values($fields));
39	4	}
40	4	return $this->types[$type];
41		}
42
43		/**
44		* @return Type
45		*/
46	4	public function create($type, array $fields = null)
47		{
48	4	if ($this->manager->getSchema()->hasSpace($type)) {
49		throw new LogicException("Type $type exists");
50		}
51
52	4	$this->manager->getSchema()->createSpace($type);
53
54	4	$instance = new Type($this->manager, $type);
55
56	4	$instance->addProperty('id');
57	4	$instance->addIndex('id');
58
59	4	if ($fields) {
60	4	call_user_func_array([$instance, 'addProperty'], (array) $fields);
61	4	}
62
63	4	return $this->types[$type] = $instance;
64		}
65		}
66

tarantool-php / mapper

Branch — master (0bf65c)

Meta::get() B

Complexity

Size

Duplication

Code Coverage

Importance

Available Fixes

Available Fixes

Duplication Side-by-Side

Filter issues like