RackTransformer - Code Metrics - nikhil-pandey/guitar-shop - Measure and Improve Code Quality continuously with Scrutinizer

RackTransformer A
last analyzed 2016-04-24 04:31 UTC

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	56
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	5

Test Coverage

Coverage

Importance

Changes	8
Bugs	3	Features	2

Metric	Value
wmc	3
c	8
b	3
f	2
lcom	0
cbo	5
dl	0
loc	56
rs	10
ccs	0
cts	23
cp	0

3 Methods

Rating	Name	Size	Complexity
A	includeWarehouse()	4	1
A	includeGuitars()	4	1
A	transform()	15	1

<?php

namespace App\Transformers;

use App\Rack;
use League\Fractal\TransformerAbstract;

class RackTransformer extends TransformerAbstract
{
    /**
     * List of resources possible to include.
     *
     * @var array
     */
    protected $availableIncludes = ['warehouse', 'guitars'];

    /**
     * Turn this item object into a generic array.
     *
     * @param Rack $rack
     *
     * @return array
     */
    public function transform(Rack $rack)
    {
        return [
            'id'       => (int) $rack->id,
            'name'     => (string) $rack->name,
            'capacity' => (int) $rack->capacity,
            'used'     => (int) $rack->used,
            'usage'    => (float) $rack->usage,
            'make_id'  => (int) $rack->make_id,
            'make'     => [
                'id'   => (int) $rack->make->id,
                'name' => (string) $rack->make->name,
            ],
        ];
    }

    /**
     * Include warehouse.
     *
     * @param Rack $rack
     *
     * @return \League\Fractal\Resource\Collection
     */
    public function includeWarehouse(Rack $rack)
    {
        return $this->item($rack->warehouse, new WarehouseTransformer);
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
    }

    /**
     * Include guitars.
     *
     * @param Rack $rack
     *
     * @return \League\Fractal\Resource\Collection
     */
    public function includeGuitars(Rack $rack)
    {
        return $this->collection($rack->guitars, new GuitarTransformer);
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
    }
}


1			<?php
2
3			namespace App\Transformers;
4
5			use App\Rack;
6			use League\Fractal\TransformerAbstract;
7
8			class RackTransformer extends TransformerAbstract
9			{
10			/**
11			* List of resources possible to include.
12			*
13			* @var array
14			*/
15			protected $availableIncludes = ['warehouse', 'guitars'];
16
17			/**
18			* Turn this item object into a generic array.
19			*
20			* @param Rack $rack
21			*
22			* @return array
23			*/
24			public function transform(Rack $rack)
25			{
26			return [
27			'id' => (int) $rack->id,
28			'name' => (string) $rack->name,
29			'capacity' => (int) $rack->capacity,
30			'used' => (int) $rack->used,
31			'usage' => (float) $rack->usage,
32			'make_id' => (int) $rack->make_id,
33			'make' => [
34			'id' => (int) $rack->make->id,
35			'name' => (string) $rack->make->name,
36			],
37			];
38			}
39
40			/**
41			* Include warehouse.
42			*
43			* @param Rack $rack
44			*
45			* @return \League\Fractal\Resource\Collection
46			*/
47			public function includeWarehouse(Rack $rack)
48			{
49			return $this->item($rack->warehouse, new WarehouseTransformer);
			0 ignored issues – show Documentation introduced 2016-03-20 02:32 UTC by Report Bug Copy Issue Report `new \App\Transformers\WarehouseTransformer()` is of type `object<App\Transformers\WarehouseTransformer>`, but the function expects a `callable`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
50			}
51
52			/**
53			* Include guitars.
54			*
55			* @param Rack $rack
56			*
57			* @return \League\Fractal\Resource\Collection
58			*/
59			public function includeGuitars(Rack $rack)
60			{
61			return $this->collection($rack->guitars, new GuitarTransformer);
			0 ignored issues – show Documentation introduced 2016-03-20 02:32 UTC by Report Bug Copy Issue Report `new \App\Transformers\GuitarTransformer()` is of type `object<App\Transformers\GuitarTransformer>`, but the function expects a `callable`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
62			}
63			}
64

nikhil-pandey / guitar-shop

GitHub Access Token became invalid

RackTransformer A last analyzed 2016-04-24 04:31 UTC

Complexity

Size/Duplication

Coupling/Cohesion

Test Coverage

Importance

3 Methods

Duplication Side-by-Side

Filter issues like

RackTransformer A
last analyzed 2016-04-24 04:31 UTC