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

RackTransformer::transform() A
last analyzed 2016-04-24 04:31 UTC

↳ Parent: RackTransformer

Complexity

Conditions	1
Paths	1

Size

Total Lines	15
Code Lines	11

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	2

Importance

Changes	8
Bugs	3	Features	2

Metric	Value
c	8
b	3
f	2
dl	0
loc	15
rs	9.4285
ccs	0
cts	15
cp	0
cc	1
eloc	11
nc	1
nop	1
crap	2

<?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::transform() A last analyzed 2016-04-24 04:31 UTC

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like

RackTransformer::transform() A
last analyzed 2016-04-24 04:31 UTC