Jetpack_JSON_API_Sync_Check_Endpoint::result() - Code Metrics - Inspection of "Fix bug in tests; move wp replicastore into main s..." - Automattic/jetpack - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — add/sync-action ( ff861d...d04c20 )

unknown

created 2016-05-05 22:53 UTC

Jetpack_JSON_API_Sync_Check_Endpoint::result() A

↳ Parent: Jetpack_JSON_API_Sync_Check_Endpoint

Complexity

Conditions	3
Paths	3

Size

Total Lines	21
Code Lines	10

Duplication

Lines	0
Ratio	0 %

Importance

Changes	2
Bugs	0	Features	1

Metric	Value
cc	3
eloc	10
c	2
b	0
f	1
nc	3
nop	0
dl	0
loc	21
rs	9.3142

<?php

require_once '../../sync/class.jetpack-sync-wp-replicastore.php';

class Jetpack_JSON_API_Sync_Endpoint extends Jetpack_JSON_API_Endpoint {
	// POST /sites/%s/sync
	protected $needed_capabilities = 'manage_options';

	protected function result() {
		Jetpack::init();
		/** This action is documented in class.jetpack-sync-client.php */
		Jetpack_Sync_Actions::schedule_full_sync();

		return array( 'scheduled' => true );
	}
}

class Jetpack_JSON_API_Sync_Check_Endpoint extends Jetpack_JSON_API_Endpoint {

	// POST /sites/%s/cached-data-check

	public function callback( $path = '', $_blog_id = 0 ) {
		$blog_id = $this->api->switch_to_blog_and_validate_user( $this->api->get_blog_id( $_blog_id ) );
		if ( is_wp_error( $blog_id ) ) {
			return $blog_id;
		}

		Jetpack::init();
		/** This action is documented in class.jetpack.php */

		$sync_queue = Jetpack_Sync_Client::getInstance()->get_sync_queue();

		// lock sending from the queue while we compare checksums with the server
		$result = $sync_queue->lock( 30 ); // tries to acquire the lock for up to 30 seconds

		if ( !$result ) {
			return new WP_Error( 'unknown_error', 'Unknown error trying to lock the sync queue' );
		}

		if ( is_wp_error( $result ) ) {
			return $result;
		}

		$store = new Jetpack_Sync_WP_Replicastore();

		return $store->checksum_all();
	}
}

1			<?php
2
3			require_once '../../sync/class.jetpack-sync-wp-replicastore.php';
4
5			class Jetpack_JSON_API_Sync_Endpoint extends Jetpack_JSON_API_Endpoint {
6			// POST /sites/%s/sync
7			protected $needed_capabilities = 'manage_options';
8
9			protected function result() {
10			Jetpack::init();
11			/** This action is documented in class.jetpack-sync-client.php */
12			Jetpack_Sync_Actions::schedule_full_sync();
13
14			return array( 'scheduled' => true );
15			}
16			}
17
18			class Jetpack_JSON_API_Sync_Check_Endpoint extends Jetpack_JSON_API_Endpoint {
			0 ignored issues – show Coding Style Compatibility introduced 2016-05-04 18:05 UTC by Report Bug Copy Issue Report PSR1 recommends that each class should be in its own file to aid autoloaders. Having each class in a dedicated file usually plays nice with PSR autoloaders and is therefore a well established practice. If you use other autoloaders, you might not want to follow this rule. Loading history... Bug introduced 2016-05-05 23:02 UTC by Report Bug Copy Issue Report There is one abstract method `result` in this class; you could implement it, or declare this class as abstract. Loading history...
19			// POST /sites/%s/cached-data-check
20
21			public function callback( $path = '', $_blog_id = 0 ) {
22			$blog_id = $this->api->switch_to_blog_and_validate_user( $this->api->get_blog_id( $_blog_id ) );
23			if ( is_wp_error( $blog_id ) ) {
24			return $blog_id;
25			}
26
27			Jetpack::init();
28			/** This action is documented in class.jetpack.php */
29
30			$sync_queue = Jetpack_Sync_Client::getInstance()->get_sync_queue();
31
32			// lock sending from the queue while we compare checksums with the server
33			$result = $sync_queue->lock( 30 ); // tries to acquire the lock for up to 30 seconds
34
35			if ( !$result ) {
36			return new WP_Error( 'unknown_error', 'Unknown error trying to lock the sync queue' );
37			}
38
39			if ( is_wp_error( $result ) ) {
40			return $result;
41			}
42
43			$store = new Jetpack_Sync_WP_Replicastore();
44
45			return $store->checksum_all();
46			}
47			}

Automattic / jetpack

Push — add/sync-action ( ff861d...d04c20 )

Jetpack_JSON_API_Sync_Check_Endpoint::result() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like