Sync_Background_Process_Stopped_State::enter() - Code Metrics - Inspection of "1187 improve speed of sync" - insideout10/wordlift-plugin - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — develop (#1282)

by David

created 2021-02-03 11:46 UTC

Sync_Background_Process_Stopped_State::enter() A

↳ Parent: Sync_Background_Process_Stopped_State

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	3
rs	10
c	0
b	0
f	0

<?php

namespace Wordlift\Dataset\Background;

class Sync_Background_Process_Stopped_State extends Abstract_Sync_Background_Process_State {

	/**
	 * @var Sync_Background_Process
	 */
	private $context;

	function __construct( $context ) {

		parent::__construct( Sync_Background_Process::STATE_STOPPED );

		$this->context = $context;
	}

	function enter() {

		$this->context->set_state( Sync_Background_Process::STATE_STOPPED );
	}

	function leave() {

		$this->context->set_state( null );
	}

	function task( $item ) {


		$this->context->cancel_process();

		return false;
	}

}


1			<?php
2
3			namespace Wordlift\Dataset\Background;
4
5			class Sync_Background_Process_Stopped_State extends Abstract_Sync_Background_Process_State {
6
7			/**
8			* @var Sync_Background_Process
9			*/
10			private $context;
11
12			function __construct( $context ) {
			0 ignored issues – show Best Practice introduced 2021-02-03 11:38 UTC by Report Bug Copy Issue Report It is generally recommended to explicitly declare the visibility for methods. Adding explicit visibility (`private`, `protected`, or `public`) is generally recommend to communicate to other developers how, and from where this method is intended to be used. Loading history...
13			parent::__construct( Sync_Background_Process::STATE_STOPPED );
14
15			$this->context = $context;
16			}
17
18			function enter() {
			0 ignored issues – show Best Practice introduced 2021-02-03 11:38 UTC by Report Bug Copy Issue Report It is generally recommended to explicitly declare the visibility for methods. Adding explicit visibility (`private`, `protected`, or `public`) is generally recommend to communicate to other developers how, and from where this method is intended to be used. Loading history...
19			$this->context->set_state( Sync_Background_Process::STATE_STOPPED );
20			}
21
22			function leave() {
			0 ignored issues – show Best Practice introduced 2021-02-03 11:38 UTC by Report Bug Copy Issue Report It is generally recommended to explicitly declare the visibility for methods. Adding explicit visibility (`private`, `protected`, or `public`) is generally recommend to communicate to other developers how, and from where this method is intended to be used. Loading history...
23			$this->context->set_state( null );
24			}
25
26			function task( $item ) {
			0 ignored issues – show Best Practice introduced 2021-02-03 11:38 UTC by Report Bug Copy Issue Report It is generally recommended to explicitly declare the visibility for methods. Adding explicit visibility (`private`, `protected`, or `public`) is generally recommend to communicate to other developers how, and from where this method is intended to be used. Loading history...
27
28			$this->context->cancel_process();
29
30			return false;
31			}
32
33			}
34

insideout10 / wordlift-plugin

Pull Request — develop (#1282)

Sync_Background_Process_Stopped_State::enter() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like