AbstractSerializer::replaceDashWithUppercase() - Code Metrics - Inspection of "Hardcode class name instead of using "::class"" - tobscure/json-api - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 5b5380...2e1d78 )

by Toby

created 2016-02-22 22:57 UTC

AbstractSerializer::replaceDashWithUppercase() A

↳ Parent: AbstractSerializer::getRelationshipMethodName()

Complexity

Conditions	1
Paths	1

Size

Total Lines	4
Code Lines	2

Duplication

Lines	0
Ratio	0 %

Importance

Changes	2
Bugs	0	Features	1

Metric	Value
c	2
b	0
f	1
dl	0
loc	4
rs	10
cc	1
eloc	2
nc	1
nop	1

<?php

/*
 * This file is part of JSON-API.
 *
 * (c) Toby Zerner <[email protected]>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace Tobscure\JsonApi;

use LogicException;

abstract class AbstractSerializer implements SerializerInterface
{
    /**
     * The type.
     *
     * @var string
     */
    protected $type;

    /**
     * {@inheritdoc}
     */
    public function getType($model)
    {
        return $this->type;
    }

    /**
     * {@inheritdoc}
     */
    public function getId($model)
    {
        return $model->id;
    }

    /**
     * {@inheritdoc}
     */
    public function getAttributes($model, array $fields = null)
    {
        return [];
    }

    /**
     * {@inheritdoc}
     */
    public function getLinks($model)
    {
        return [];
    }

    /**
     * {@inheritdoc}
     *
     * @throws LogicException
     */
    public function getRelationship($model, $name)
    {
        $method = $this->getRelationshipMethodName($name);

        if (method_exists($this, $method)) {
            $relationship = $this->$method($model);

            if ($relationship !== null && ! ($relationship instanceof Relationship)) {
                throw new LogicException('Relationship method must return null or an instance of Tobscure\JsonApi\Relationship');

            }

            return $relationship;
        }
    }

    /**
     * Get the serializer method name for the given relationship.
     *
     * kebab-case is converted into camelCase.
     *
     * @param string $name
     * @return string
     */
    private function getRelationshipMethodName($name)
    {
    	if (stripos($name, '-')) {
        	$name = lcfirst(implode('', array_map('ucfirst', explode('-', $name))));
    	}

    	return $name;
    }
}


1			<?php
2
3			/*
4			* This file is part of JSON-API.
5			*
6			* (c) Toby Zerner <[email protected]>
7			*
8			* For the full copyright and license information, please view the LICENSE
9			* file that was distributed with this source code.
10			*/
11
12			namespace Tobscure\JsonApi;
13
14			use LogicException;
15
16			abstract class AbstractSerializer implements SerializerInterface
17			{
18			/**
19			* The type.
20			*
21			* @var string
22			*/
23			protected $type;
24
25			/**
26			* {@inheritdoc}
27			*/
28			public function getType($model)
29			{
30			return $this->type;
31			}
32
33			/**
34			* {@inheritdoc}
35			*/
36			public function getId($model)
37			{
38			return $model->id;
39			}
40
41			/**
42			* {@inheritdoc}
43			*/
44			public function getAttributes($model, array $fields = null)
45			{
46			return [];
47			}
48
49			/**
50			* {@inheritdoc}
51			*/
52			public function getLinks($model)
53			{
54			return [];
55			}
56
57			/**
58			* {@inheritdoc}
59			*
60			* @throws LogicException
61			*/
62			public function getRelationship($model, $name)
63			{
64			$method = $this->getRelationshipMethodName($name);
65
66			if (method_exists($this, $method)) {
67			$relationship = $this->$method($model);
68
69			if ($relationship !== null && ! ($relationship instanceof Relationship)) {
70			throw new LogicException('Relationship method must return null or an instance of Tobscure\JsonApi\Relationship');
			0 ignored issues – show Coding Style introduced 2016-02-23 00:00 UTC by Report Bug Copy Issue Report This line exceeds maximum limit of 120 characters; contains 129 characters Overly long lines are hard to read on any screen. Most code styles therefor impose a maximum limit on the number of characters in a line. Loading history...
71			}
72
73			return $relationship;
74			}
75			}
76
77			/**
78			* Get the serializer method name for the given relationship.
79			*
80			* kebab-case is converted into camelCase.
81			*
82			* @param string $name
83			* @return string
84			*/
85			private function getRelationshipMethodName($name)
86			{
87			if (stripos($name, '-')) {
88			$name = lcfirst(implode('', array_map('ucfirst', explode('-', $name))));
89			}
90
91			return $name;
92			}
93			}
94

tobscure / json-api

Push — master ( 5b5380...2e1d78 )

AbstractSerializer::replaceDashWithUppercase() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like