Issues in class.jetpack-data.php - Fixed Issues - Inspection of "Merge branch 'update/package-sync-wp-super-cache'..." - Automattic/jetpack - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — update/package-sync-wp-super-c... ( 646a03...89a5a9 )

unknown

created 2019-06-18 10:25 UTC

class.jetpack-data.php (2 issues)

Labels

Documentation 2

Severity

Minor 2

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

<?php

use \Automattic\Jetpack\Connection\Manager as Connection_Manager;

class Jetpack_Data {
	/**
	 * @deprecated 7.5 Use Connection_Manager instead.
	 */
	public static function get_access_token( $user_id = false, $token_key = false ) {
		$connection = new Connection_Manager();
		return $connection->get_access_token( $user_id, $token_key );
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

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


1			<?php
2
3			use \Automattic\Jetpack\Connection\Manager as Connection_Manager;
4
5			class Jetpack_Data {
6			/**
7			* @deprecated 7.5 Use Connection_Manager instead.
8			*/
9			public static function get_access_token( $user_id = false, $token_key = false ) {
10			$connection = new Connection_Manager();
11			return $connection->get_access_token( $user_id, $token_key );
			0 ignored issues – show Documentation introduced 2019-06-06 00:27 UTC by Report Bug Copy Issue Report Show Similar Issues like this `$user_id` is of type `boolean`, but the function expects a `false\|integer`. 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... Documentation introduced 2019-06-06 00:27 UTC by Report Bug Copy Issue Report Show Similar Issues like this `$token_key` is of type `boolean`, but the function expects a `false\|string`. 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...
12			}
13			}
14

Automattic / jetpack

Push — update/package-sync-wp-super-c... ( 646a03...89a5a9 )

class.jetpack-data.php (2 issues)

Labels

Severity

Introduced By

Upgrade to new PHP Analysis Engine

Duplication Side-by-Side

Filter issues like