Issues in test.php - All Issues - Inspection of "Added yii-specific FileStorage with yii-aliases su..." - hiqdev/php-confirmator - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 7b3ac4...466150 )

by Andrii

created 2018-08-27 15:33 UTC

All Issues
New 0

tests/test.php (1 issue)

Labels

Documentation 1

Severity

Minor 1

Upgrade to new PHP Analysis Engine

These results are based on our legacy PHP analysis, consider migrating to our new PHP analysis engine instead. Learn more

#!/usr/bin/env php
<?php

use hiqdev\php\confirmator\Token;

require '_bootstrap.php';

$token = new Token([
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
    'action' => 'test',
]);

var_dump($token);


1			#!/usr/bin/env php
2			<?php
3
4			use hiqdev\php\confirmator\Token;
5
6			require '_bootstrap.php';
7
8			$token = new Token([
			0 ignored issues – show Documentation introduced 2016-10-27 13:28 UTC by Report Bug Copy Issue Report Show Similar Issues like this `array('action' => 'test')` is of type `array<string,string,{"action":"string"}>`, but the function expects a `object<hiqdev\php\confirmator\ServiceInterface>`. 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...
9			'action' => 'test',
10			]);
11
12			var_dump($token);
13

hiqdev / php-confirmator

Push — master ( 7b3ac4...466150 )

tests/test.php (1 issue)

Labels

Severity

Introduced By

Upgrade to new PHP Analysis Engine

Duplication Side-by-Side

Filter issues like