AlertTransformer::__construct() - Code Metrics - Inspection of "Docblock typo" - Maelstromeous/ps2alerts-api - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 7860e0...d3b4dd )

by Matthew

created 2016-01-17 04:31 UTC

AlertTransformer::__construct() B

↳ Parent: AlertTransformer

Complexity

Conditions	1
Paths	1

Size

Total Lines	25
Code Lines	23

Duplication

Lines	0
Ratio	0 %

Importance

Changes	3
Bugs	0	Features	1

Metric	Value
c	3
b	0
f	1
dl	0
loc	25
rs	8.8571
cc	1
eloc	23
nc	1
nop	11

How to fix Many Parameters

<?php

namespace Ps2alerts\Api\Transformer;

use League\Fractal\TransformerAbstract;
use Ps2alerts\Api\Repository\Metrics\ClassRepository;
use Ps2alerts\Api\Repository\Metrics\CombatHistoryRepository;
use Ps2alerts\Api\Repository\Metrics\CombatRepository;
use Ps2alerts\Api\Repository\Metrics\MapInitialRepository;
use Ps2alerts\Api\Repository\Metrics\MapRepository;
use Ps2alerts\Api\Repository\Metrics\OutfitRepository;
use Ps2alerts\Api\Repository\Metrics\PlayerRepository;
use Ps2alerts\Api\Repository\Metrics\PopulationRepository;
use Ps2alerts\Api\Repository\Metrics\VehicleRepository;
use Ps2alerts\Api\Repository\Metrics\WeaponRepository;
use Ps2alerts\Api\Repository\Metrics\XpRepository;
use Ps2alerts\Api\Transformer\AlertTransformer;
use Ps2alerts\Api\Transformer\Metrics\ClassTransformer;
use Ps2alerts\Api\Transformer\Metrics\CombatHistoryTransformer;
use Ps2alerts\Api\Transformer\Metrics\CombatTransformer;
use Ps2alerts\Api\Transformer\Metrics\MapInitialTransformer;
use Ps2alerts\Api\Transformer\Metrics\MapTransformer;
use Ps2alerts\Api\Transformer\Metrics\OutfitTransformer;
use Ps2alerts\Api\Transformer\Metrics\PlayerTransformer;
use Ps2alerts\Api\Transformer\Metrics\PopulationTransformer;
use Ps2alerts\Api\Transformer\Metrics\VehicleTransformer;
use Ps2alerts\Api\Transformer\Metrics\WeaponTransformer;
use Ps2alerts\Api\Transformer\Metrics\XpTransformer;

class AlertTransformer extends TransformerAbstract
{
    /**
     * List of available includes to this resource
     *
     * @var array
     */
    protected $availableIncludes = [
        'classes',
        'combatHistorys',
        'combats',
        'mapInitials',
        'maps',
        'outfits',
        'players',
        'populations',
        'vehicles',
        'weapons',
        'xps'
    ];

    /**
     * Repositories
     */
    protected $classRepo;
    protected $combatHistoryRepo;
    protected $combatRepo;
    protected $mapInitialRepo;
    protected $mapRepo;
    protected $outfitRepo;
    protected $playerRepo;
    protected $populationRepo;
    protected $vehicleRepo;
    protected $weaponRepo;
    protected $xpRepo;

    /**
     * Constructor
     *
     * @param ClassRepository         $classRepo
     * @param CombatHistoryRepository $combatHistoryRepo
     * @param CombatRepository        $combatRepo
     * @param MapInitialRepository    $mapInitialRepo
     * @param MapRepository           $mapRepo
     * @param OutfitRepository        $outfitRepo
     * @param PlayerRepository        $playerRepo
     * @param PopulationRepository    $populationRepo
     * @param VehicleRepository       $vehicleRepo
     * @param WeaponRepository        $weaponRepo
     * @param XpRepository            $xpRepo
     */
    public function __construct(
        ClassRepository         $classRepo,
        CombatHistoryRepository $combatHistoryRepo,
        CombatRepository        $combatRepo,
        MapInitialRepository    $mapInitialRepo,
        MapRepository           $mapRepo,
        OutfitRepository        $outfitRepo,
        PlayerRepository        $playerRepo,
        PopulationRepository    $populationRepo,
        VehicleRepository       $vehicleRepo,
        WeaponRepository        $weaponRepo,
        XpRepository            $xpRepo
    ) {
        $this->classRepo         = $classRepo;
        $this->combatHistoryRepo = $combatHistoryRepo;
        $this->combatRepo        = $combatRepo;
        $this->mapInitialRepo    = $mapInitialRepo;
        $this->mapRepo           = $mapRepo;
        $this->outfitRepo        = $outfitRepo;
        $this->playerRepo        = $playerRepo;
        $this->populationRepo    = $populationRepo;
        $this->vehicleRepo       = $vehicleRepo;
        $this->weaponRepo        = $weaponRepo;
        $this->xpRepo            = $xpRepo;
    }

    /**
     * The tranform method required by Fractal to parse the data and return proper typing and fields.
     *
     * @param  array $data Data to transform
     *
     * @return array
     */
    public function transform($data)
    {
        return [
            'id'           => (int) $data['ResultID'],
            'started'      => (int) $data['ResultStartTime'],
            'ended'        => (int) $data['ResultEndTime'],
            'server'       => (int) $data['ResultServer'],
            'zone'         => (int) $data['ResultAlertCont'],
            'winner'       => (string) $data['ResultWinner'],
            'isDraw'       => (boolean) $data['ResultDraw'],
            'isDomination' => (boolean) $data['ResultDomination'],
            'isValid'      => (boolean) $data['Valid'],
            'inProgress'   => (boolean) $data['InProgress']
        ];
    }

    /**
     * Gets the Class data and then adds it to the result
     *
     * @param  array $data
     *
     * @return \League\Fractal\Resource\Collection
     */
    public function includeClasses($data)
    {
        $data = $this->classRepo->readAllById($data['ResultID'], 'result');
        return $this->collection($data, new ClassTransformer);
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

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

    /**
     * Gets the Combat History data and then adds it to the result
     *
     * @param  array $data
     *
     * @return \League\Fractal\Resource\Collection
     */
    public function includeCombatHistorys($data)
    {
        $data = $this->combatHistoryRepo->readAllById($data['ResultID'], 'result');
        return $this->collection($data, new CombatHistoryTransformer);
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

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

    /**
     * Gets the Combat data and then adds it to the result
     *
     * @param  array $data
     *
     * @return \League\Fractal\Resource\Item
     */
    public function includeCombats($data)
    {
        $data = $this->combatRepo->readSingleById($data['ResultID'], 'result');
        return $this->item($data, new CombatTransformer);
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

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

    /**
     * Gets the Class data and then adds it to the result
     *
     * @param  array $data
     *
     * @return \League\Fractal\Resource\Collection
     */
    public function includeMapInitials($data)
    {
        $data = $this->mapInitialRepo->readAllById($data['ResultID'], 'result');
        return $this->collection($data, new MapInitialTransformer);
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

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

    /**
     * Gets the Map data and then adds it to the result
     *
     * @param  array $data
     *
     * @return \League\Fractal\Resource\Collection
     */
    public function includeMaps($data)
    {
        $data = $this->mapRepo->readAllById($data['ResultID'], 'result');
        return $this->collection($data, new MapTransformer);
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

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

    /**
     * Gets the Outfit data and then adds it to the result
     *
     * @param  array $data
     *
     * @return \League\Fractal\Resource\Collection
     */
    public function includeOutfits($data)
    {
        $data = $this->outfitRepo->readAllById($data['ResultID'], 'result');
        return $this->collection($data, new OutfitTransformer);
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

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

    /**
     * Gets the Popualtion data and then adds it to the result
     *
     * @param  array $data
     *
     * @return \League\Fractal\Resource\Collection
     */
    public function includePopulations($data)
    {
        $data = $this->populationRepo->readAllById($data['ResultID'], 'result');
        return $this->collection($data, new PopulationTransformer);
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

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

    /**
     * Gets the Player data and then adds it to the result
     *
     * @param  array $data
     *
     * @return \League\Fractal\Resource\Collection
     */
    public function includePlayers($data)
    {
        $data = $this->playerRepo->readAllById($data['ResultID'], 'result');
        return $this->collection($data, new PlayerTransformer);
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

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

    /**
     * Gets the Vehicle data and then adds it to the result
     *
     * @param  array $data
     *
     * @return \League\Fractal\Resource\Collection
     */
    public function includeVehicles($data)
    {
        $data = $this->vehicleRepo->readAllById($data['ResultID'], 'result');
        return $this->collection($data, new VehicleTransformer);
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

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

    /**
     * Gets the Weapon data and then adds it to the result
     *
     * @param  array $data
     *
     * @return \League\Fractal\Resource\Collection
     */
    public function includeWeapons($data)
    {
        $map = $this->weaponRepo->readAllById($data['ResultID'], 'result');
        return $this->collection($map, new WeaponTransformer);
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

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

    /**
     * Gets the XP data and then adds it to the result
     *
     * @param  array $data
     *
     * @return \League\Fractal\Resource\Collection
     */
    public function includeXps($data)
    {
        // NOTE TO SELF: RATE LIMIT THIS BAD BOY
        $map = $this->xpRepo->readAllById($data['ResultID'], 'result');
        return $this->collection($map, new XpTransformer);
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

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


1			<?php
2
3			namespace Ps2alerts\Api\Transformer;
4
5			use League\Fractal\TransformerAbstract;
6			use Ps2alerts\Api\Repository\Metrics\ClassRepository;
7			use Ps2alerts\Api\Repository\Metrics\CombatHistoryRepository;
8			use Ps2alerts\Api\Repository\Metrics\CombatRepository;
9			use Ps2alerts\Api\Repository\Metrics\MapInitialRepository;
10			use Ps2alerts\Api\Repository\Metrics\MapRepository;
11			use Ps2alerts\Api\Repository\Metrics\OutfitRepository;
12			use Ps2alerts\Api\Repository\Metrics\PlayerRepository;
13			use Ps2alerts\Api\Repository\Metrics\PopulationRepository;
14			use Ps2alerts\Api\Repository\Metrics\VehicleRepository;
15			use Ps2alerts\Api\Repository\Metrics\WeaponRepository;
16			use Ps2alerts\Api\Repository\Metrics\XpRepository;
17			use Ps2alerts\Api\Transformer\AlertTransformer;
18			use Ps2alerts\Api\Transformer\Metrics\ClassTransformer;
19			use Ps2alerts\Api\Transformer\Metrics\CombatHistoryTransformer;
20			use Ps2alerts\Api\Transformer\Metrics\CombatTransformer;
21			use Ps2alerts\Api\Transformer\Metrics\MapInitialTransformer;
22			use Ps2alerts\Api\Transformer\Metrics\MapTransformer;
23			use Ps2alerts\Api\Transformer\Metrics\OutfitTransformer;
24			use Ps2alerts\Api\Transformer\Metrics\PlayerTransformer;
25			use Ps2alerts\Api\Transformer\Metrics\PopulationTransformer;
26			use Ps2alerts\Api\Transformer\Metrics\VehicleTransformer;
27			use Ps2alerts\Api\Transformer\Metrics\WeaponTransformer;
28			use Ps2alerts\Api\Transformer\Metrics\XpTransformer;
29
30			class AlertTransformer extends TransformerAbstract
31			{
32			/**
33			* List of available includes to this resource
34			*
35			* @var array
36			*/
37			protected $availableIncludes = [
38			'classes',
39			'combatHistorys',
40			'combats',
41			'mapInitials',
42			'maps',
43			'outfits',
44			'players',
45			'populations',
46			'vehicles',
47			'weapons',
48			'xps'
49			];
50
51			/**
52			* Repositories
53			*/
54			protected $classRepo;
55			protected $combatHistoryRepo;
56			protected $combatRepo;
57			protected $mapInitialRepo;
58			protected $mapRepo;
59			protected $outfitRepo;
60			protected $playerRepo;
61			protected $populationRepo;
62			protected $vehicleRepo;
63			protected $weaponRepo;
64			protected $xpRepo;
65
66			/**
67			* Constructor
68			*
69			* @param ClassRepository $classRepo
70			* @param CombatHistoryRepository $combatHistoryRepo
71			* @param CombatRepository $combatRepo
72			* @param MapInitialRepository $mapInitialRepo
73			* @param MapRepository $mapRepo
74			* @param OutfitRepository $outfitRepo
75			* @param PlayerRepository $playerRepo
76			* @param PopulationRepository $populationRepo
77			* @param VehicleRepository $vehicleRepo
78			* @param WeaponRepository $weaponRepo
79			* @param XpRepository $xpRepo
80			*/
81			public function __construct(
82			ClassRepository $classRepo,
83			CombatHistoryRepository $combatHistoryRepo,
84			CombatRepository $combatRepo,
85			MapInitialRepository $mapInitialRepo,
86			MapRepository $mapRepo,
87			OutfitRepository $outfitRepo,
88			PlayerRepository $playerRepo,
89			PopulationRepository $populationRepo,
90			VehicleRepository $vehicleRepo,
91			WeaponRepository $weaponRepo,
92			XpRepository $xpRepo
93			) {
94			$this->classRepo = $classRepo;
95			$this->combatHistoryRepo = $combatHistoryRepo;
96			$this->combatRepo = $combatRepo;
97			$this->mapInitialRepo = $mapInitialRepo;
98			$this->mapRepo = $mapRepo;
99			$this->outfitRepo = $outfitRepo;
100			$this->playerRepo = $playerRepo;
101			$this->populationRepo = $populationRepo;
102			$this->vehicleRepo = $vehicleRepo;
103			$this->weaponRepo = $weaponRepo;
104			$this->xpRepo = $xpRepo;
105			}
106
107			/**
108			* The tranform method required by Fractal to parse the data and return proper typing and fields.
109			*
110			* @param array $data Data to transform
111			*
112			* @return array
113			*/
114			public function transform($data)
115			{
116			return [
117			'id' => (int) $data['ResultID'],
118			'started' => (int) $data['ResultStartTime'],
119			'ended' => (int) $data['ResultEndTime'],
120			'server' => (int) $data['ResultServer'],
121			'zone' => (int) $data['ResultAlertCont'],
122			'winner' => (string) $data['ResultWinner'],
123			'isDraw' => (boolean) $data['ResultDraw'],
124			'isDomination' => (boolean) $data['ResultDomination'],
125			'isValid' => (boolean) $data['Valid'],
126			'inProgress' => (boolean) $data['InProgress']
127			];
128			}
129
130			/**
131			* Gets the Class data and then adds it to the result
132			*
133			* @param array $data
134			*
135			* @return \League\Fractal\Resource\Collection
136			*/
137			public function includeClasses($data)
138			{
139			$data = $this->classRepo->readAllById($data['ResultID'], 'result');
140			return $this->collection($data, new ClassTransformer);
			0 ignored issues – show Documentation introduced 2016-01-17 00:22 UTC by Report Bug Copy Issue Report `new \Ps2alerts\Api\Trans...rics\ClassTransformer()` is of type `object<Ps2alerts\Api\Tra...trics\ClassTransformer>`, 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...
141			}
142
143			/**
144			* Gets the Combat History data and then adds it to the result
145			*
146			* @param array $data
147			*
148			* @return \League\Fractal\Resource\Collection
149			*/
150			public function includeCombatHistorys($data)
151			{
152			$data = $this->combatHistoryRepo->readAllById($data['ResultID'], 'result');
153			return $this->collection($data, new CombatHistoryTransformer);
			0 ignored issues – show Documentation introduced 2016-01-17 00:22 UTC by Report Bug Copy Issue Report `new \Ps2alerts\Api\Trans...batHistoryTransformer()` is of type `object<Ps2alerts\Api\Tra...mbatHistoryTransformer>`, 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...
154			}
155
156			/**
157			* Gets the Combat data and then adds it to the result
158			*
159			* @param array $data
160			*
161			* @return \League\Fractal\Resource\Item
162			*/
163			public function includeCombats($data)
164			{
165			$data = $this->combatRepo->readSingleById($data['ResultID'], 'result');
166			return $this->item($data, new CombatTransformer);
			0 ignored issues – show Documentation introduced 2016-01-17 02:39 UTC by Report Bug Copy Issue Report `new \Ps2alerts\Api\Trans...ics\CombatTransformer()` is of type `object<Ps2alerts\Api\Tra...rics\CombatTransformer>`, 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...
167			}
168
169			/**
170			* Gets the Class data and then adds it to the result
171			*
172			* @param array $data
173			*
174			* @return \League\Fractal\Resource\Collection
175			*/
176			public function includeMapInitials($data)
177			{
178			$data = $this->mapInitialRepo->readAllById($data['ResultID'], 'result');
179			return $this->collection($data, new MapInitialTransformer);
			0 ignored issues – show Documentation introduced 2016-01-17 00:22 UTC by Report Bug Copy Issue Report `new \Ps2alerts\Api\Trans...MapInitialTransformer()` is of type `object<Ps2alerts\Api\Tra...\MapInitialTransformer>`, 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...
180			}
181
182			/**
183			* Gets the Map data and then adds it to the result
184			*
185			* @param array $data
186			*
187			* @return \League\Fractal\Resource\Collection
188			*/
189			public function includeMaps($data)
190			{
191			$data = $this->mapRepo->readAllById($data['ResultID'], 'result');
192			return $this->collection($data, new MapTransformer);
			0 ignored issues – show Documentation introduced 2016-01-17 00:22 UTC by Report Bug Copy Issue Report `new \Ps2alerts\Api\Trans...etrics\MapTransformer()` is of type `object<Ps2alerts\Api\Tra...Metrics\MapTransformer>`, 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...
193			}
194
195			/**
196			* Gets the Outfit data and then adds it to the result
197			*
198			* @param array $data
199			*
200			* @return \League\Fractal\Resource\Collection
201			*/
202			public function includeOutfits($data)
203			{
204			$data = $this->outfitRepo->readAllById($data['ResultID'], 'result');
205			return $this->collection($data, new OutfitTransformer);
			0 ignored issues – show Documentation introduced 2016-01-17 00:22 UTC by Report Bug Copy Issue Report `new \Ps2alerts\Api\Trans...ics\OutfitTransformer()` is of type `object<Ps2alerts\Api\Tra...rics\OutfitTransformer>`, 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...
206			}
207
208			/**
209			* Gets the Popualtion data and then adds it to the result
210			*
211			* @param array $data
212			*
213			* @return \League\Fractal\Resource\Collection
214			*/
215			public function includePopulations($data)
216			{
217			$data = $this->populationRepo->readAllById($data['ResultID'], 'result');
218			return $this->collection($data, new PopulationTransformer);
			0 ignored issues – show Documentation introduced 2016-01-17 00:22 UTC by Report Bug Copy Issue Report `new \Ps2alerts\Api\Trans...PopulationTransformer()` is of type `object<Ps2alerts\Api\Tra...\PopulationTransformer>`, 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...
219			}
220
221			/**
222			* Gets the Player data and then adds it to the result
223			*
224			* @param array $data
225			*
226			* @return \League\Fractal\Resource\Collection
227			*/
228			public function includePlayers($data)
229			{
230			$data = $this->playerRepo->readAllById($data['ResultID'], 'result');
231			return $this->collection($data, new PlayerTransformer);
			0 ignored issues – show Documentation introduced 2016-01-17 00:22 UTC by Report Bug Copy Issue Report `new \Ps2alerts\Api\Trans...ics\PlayerTransformer()` is of type `object<Ps2alerts\Api\Tra...rics\PlayerTransformer>`, 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...
232			}
233
234			/**
235			* Gets the Vehicle data and then adds it to the result
236			*
237			* @param array $data
238			*
239			* @return \League\Fractal\Resource\Collection
240			*/
241			public function includeVehicles($data)
242			{
243			$data = $this->vehicleRepo->readAllById($data['ResultID'], 'result');
244			return $this->collection($data, new VehicleTransformer);
			0 ignored issues – show Documentation introduced 2016-01-17 00:22 UTC by Report Bug Copy Issue Report `new \Ps2alerts\Api\Trans...cs\VehicleTransformer()` is of type `object<Ps2alerts\Api\Tra...ics\VehicleTransformer>`, 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...
245			}
246
247			/**
248			* Gets the Weapon data and then adds it to the result
249			*
250			* @param array $data
251			*
252			* @return \League\Fractal\Resource\Collection
253			*/
254			public function includeWeapons($data)
255			{
256			$map = $this->weaponRepo->readAllById($data['ResultID'], 'result');
257			return $this->collection($map, new WeaponTransformer);
			0 ignored issues – show Documentation introduced 2016-01-17 00:22 UTC by Report Bug Copy Issue Report `new \Ps2alerts\Api\Trans...ics\WeaponTransformer()` is of type `object<Ps2alerts\Api\Tra...rics\WeaponTransformer>`, 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...
258			}
259
260			/**
261			* Gets the XP data and then adds it to the result
262			*
263			* @param array $data
264			*
265			* @return \League\Fractal\Resource\Collection
266			*/
267			public function includeXps($data)
268			{
269			// NOTE TO SELF: RATE LIMIT THIS BAD BOY
270			$map = $this->xpRepo->readAllById($data['ResultID'], 'result');
271			return $this->collection($map, new XpTransformer);
			0 ignored issues – show Documentation introduced 2016-01-17 00:22 UTC by Report Bug Copy Issue Report `new \Ps2alerts\Api\Trans...Metrics\XpTransformer()` is of type `object<Ps2alerts\Api\Tra...\Metrics\XpTransformer>`, 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...
272			}
273			}
274

Maelstromeous / ps2alerts-api

Push — master ( 7860e0...d3b4dd )

AlertTransformer::__construct() B

Complexity

Size

Duplication

Importance

How to fix Many Parameters

Many Parameters

Duplication Side-by-Side

Filter issues like