AlertTransformer::includeCombatHistorys() - Code Metrics - Inspection of "Linking it all together. Big ass file." - Maelstromeous/ps2alerts-api - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — staging ( 5f9089...8e7d49 )

by Matthew

created 2016-01-17 00:18 UTC

AlertTransformer::includeCombatHistorys() A

↳ Parent: AlertTransformer

Complexity

Conditions	1
Paths	1

Size

Total Lines	5
Code Lines	3

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
c	1
b	0
f	0
dl	0
loc	5
rs	9.4285
cc	1
eloc	3
nc	1
nop	1

<?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\FactionRepository;
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\FactionTransformer;
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',
        'factions',
        'mapInitials',
        'maps',
        'outfits',
        'players',
        'populations',
        'vehicles',
        'weapons',
        'xps'
    ];

    /**
     * Repositories
     */
    protected $classRepo;
    protected $combatHistoryRepo;
    protected $factionRepo;
    protected $mapInitialRepo;
    protected $mapRepo;
    protected $outfitRepo;
    protected $playerRepo;
    protected $popRepo;
    protected $vehicleRepo;
    protected $weaponRepo;
    protected $xpRepo;

    /**
     * Constructor
     *
     * @param ClassRepository         $classRepo
     * @param CombatHistoryRepository $combatHistoryRepo
     * @param FactionRepository       $factionRepo
     * @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,
        FactionRepository       $factionRepo,
        MapInitialRepository    $mapInitialRepo,
        MapRepository           $mapRepo,
        OutfitRepository        $outfitRepo,
        PlayerRepository        $playerRepo,
        PopulationRepository    $populationRepo,
        VehicleRepository       $vehicleRepo,
        WeaponRepository        $weaponRepo,
        XpRepository            $xpRepo
    ) {
        $this->classRepo         = $classRepo;
        $this->combatHistoryRepo = $combatHistoryRepo;
        $this->factionRepo       = $factionRepo;
        $this->mapInitialRepo    = $mapInitialRepo;
        $this->mapRepo           = $mapRepo;
        $this->outfitRepo        = $outfitRepo;
        $this->playerRepo        = $playerRepo;
        $this->populationRepo    = $populationRepo;
class MyClass { }

$x = new MyClass();
$x->foo = true;
        $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->readAll($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->readAll($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 Faction data and then adds it to the result
     *
     * @param  array $data
     *
     * @return League\Fractal\Resource\Collection
     */
    public function includeFactions($data)
    {
        $data = $this->factionRepo->readAll($data['ResultID'], 'result');
        return $this->item($data, new FactionTransformer);
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->readAll($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->readAll($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->readAll($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->readAll($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->readAll($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->readAll($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->readAll($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->readAll($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\FactionRepository;
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\FactionTransformer;
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			'factions',
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 $factionRepo;
57			protected $mapInitialRepo;
58			protected $mapRepo;
59			protected $outfitRepo;
60			protected $playerRepo;
61			protected $popRepo;
62			protected $vehicleRepo;
63			protected $weaponRepo;
64			protected $xpRepo;
65
66			/**
67			* Constructor
68			*
69			* @param ClassRepository $classRepo
70			* @param CombatHistoryRepository $combatHistoryRepo
71			* @param FactionRepository $factionRepo
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			FactionRepository $factionRepo,
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->factionRepo = $factionRepo;
97			$this->mapInitialRepo = $mapInitialRepo;
98			$this->mapRepo = $mapRepo;
99			$this->outfitRepo = $outfitRepo;
100			$this->playerRepo = $playerRepo;
101			$this->populationRepo = $populationRepo;
			0 ignored issues – show Bug introduced 2016-01-17 00:22 UTC by Report Bug Copy Issue Report The property `populationRepo` does not exist. Did you maybe forget to declare it? In PHP it is possible to write to properties without declaring them. For example, the following is perfectly valid PHP code: class MyClass { } $x = new MyClass(); $x->foo = true; Generally, it is a good practice to explictly declare properties to avoid accidental typos and provide IDE auto-completion: class MyClass { public $foo; } $x = new MyClass(); $x->foo = true; Loading history...
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->readAll($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->readAll($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 Faction data and then adds it to the result
158			*
159			* @param array $data
160			*
161			* @return League\Fractal\Resource\Collection
162			*/
163			public function includeFactions($data)
164			{
165			$data = $this->factionRepo->readAll($data['ResultID'], 'result');
166			return $this->item($data, new FactionTransformer);
			0 ignored issues – show Documentation introduced 2016-01-17 00:22 UTC by Report Bug Copy Issue Report `new \Ps2alerts\Api\Trans...cs\FactionTransformer()` is of type `object<Ps2alerts\Api\Tra...ics\FactionTransformer>`, 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->readAll($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->readAll($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->readAll($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->readAll($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->readAll($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->readAll($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->readAll($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->readAll($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 — staging ( 5f9089...8e7d49 )

AlertTransformer::includeCombatHistorys() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like