plugin.php ➔ test_sync_changes() - Code Metrics - Inspection of "Adding the sync-tester plugin." - Automattic/jetpack - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — update/sync-remove-references-... ( 70b2fb...317b35 )

unknown

created 2021-04-01 20:47 UTC

plugin.php ➔ test_sync_changes() A

↳ Parent: Project

Complexity

Conditions	1
Paths	1

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	1
nc	1
nop	0
dl	0
loc	11
rs	9.9
c	0
b	0
f	0

<?php
/**
 * Jetpack Sync Tester
 *
 * Sync PR #18608 Tester.
 *
 * @link       https://jetpack.com/
 *
 * @package    Jetpack
 * @subpackage Sync
 * @since      9.6
 */

/**
 * Plugin Name: Sync PR #18608 Tester.
 */

require_once plugin_dir_path( __FILE__ ) . '/vendor/autoload_packages.php';

/**
 * Test Sync changes.
 */
function test_sync_changes() {
	// Test change to Functions::get_modules().
	error_log( 'get_modules returns: ' . print_r( Automattic\Jetpack\Sync\Functions::get_modules(), true ) );

	// Test change to Listener::get_actor(). The returned array should include the actor's ip.
	error_log( 'get_actor returns ' . print_r( Automattic\Jetpack\Sync\Listener::get_instance()->get_actor( 'jetpack_wp_login', null ), true ) );
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

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

	// Test change to Options::jetpack_sync_core_icon().
	error_log( 'calling jetpack_sync_core_icon' );
	( new Automattic\Jetpack\Sync\Modules\Options() )->jetpack_sync_core_icon();
}

add_action( 'plugins_loaded', 'test_sync_changes' );


1			<?php
2			/**
3			* Jetpack Sync Tester
4			*
5			* Sync PR #18608 Tester.
6			*
7			* @link https://jetpack.com/
8			*
9			* @package Jetpack
10			* @subpackage Sync
11			* @since 9.6
12			*/
13
14			/**
15			* Plugin Name: Sync PR #18608 Tester.
16			*/
17
18			require_once plugin_dir_path( __FILE__ ) . '/vendor/autoload_packages.php';
19
20			/**
21			* Test Sync changes.
22			*/
23			function test_sync_changes() {
24			// Test change to Functions::get_modules().
25			error_log( 'get_modules returns: ' . print_r( Automattic\Jetpack\Sync\Functions::get_modules(), true ) );
26
27			// Test change to Listener::get_actor(). The returned array should include the actor's ip.
28			error_log( 'get_actor returns ' . print_r( Automattic\Jetpack\Sync\Listener::get_instance()->get_actor( 'jetpack_wp_login', null ), true ) );
			0 ignored issues – show Documentation introduced 2021-04-01 19:59 UTC by Report Bug Copy Issue Report `null` is of type `null`, but the function expects a `object`. 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...
29
30			// Test change to Options::jetpack_sync_core_icon().
31			error_log( 'calling jetpack_sync_core_icon' );
32			( new Automattic\Jetpack\Sync\Modules\Options() )->jetpack_sync_core_icon();
33			}
34
35			add_action( 'plugins_loaded', 'test_sync_changes' );
36

Automattic / jetpack

Push — update/sync-remove-references-... ( 70b2fb...317b35 )

plugin.php ➔ test_sync_changes() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like