Lock::remove() - Code Metrics - Inspection of "Fix incorrect function call" - Automattic/jetpack - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — add/masterbar--my-home ( 5c5e9c...8732fd )

unknown

created 2019-11-25 21:36 UTC

Lock::remove() A

↳ Parent: Lock

Complexity

Conditions	1
Paths	1

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	1
nc	1
nop	1
dl	0
loc	3
rs	10
c	0
b	0
f	0

<?php
/**
 * Lock class.
 *
 * @package automattic/jetpack-sync
 */

namespace Automattic\Jetpack\Sync;

/**
 * Lock class
 */
class Lock {
	/**
	 * Prefix of the blog lock transient.
	 *
	 * @access public
	 *
	 * @var string
	 */
	const LOCK_PREFIX = 'jp_sync_lock_';

	/**
	 * Default Lifetime of the lock.
	 *
	 * @access public
	 *
	 * @var int
	 */
	const LOCK_TRANSIENT_EXPIRY = 15; // Seconds.

	/**
	 * Attempt to lock.
	 *
	 * @access public
	 *
	 * @param string $name lock name.
	 * @param int    $expiry lock duration in seconds.
	 *
	 * @return boolean True if succeeded, false otherwise.
	 */
	public function attempt( $name, $expiry = self::LOCK_TRANSIENT_EXPIRY ) {
		$name        = self::LOCK_PREFIX . $name;
		$locked_time = get_option( $name );
		if ( $locked_time ) {
			if ( microtime( true ) < $locked_time ) {
				return false;
			}
		}
		update_option( $name, microtime( true ) + $expiry );

		return true;
	}

	/**
	 * Remove the lock.
	 *
	 * @access public
	 *
	 * @param string $name lock name.
	 */
	public function remove( $name ) {
		delete_option( self::LOCK_PREFIX . $name );
	}
}


1			<?php
2			/**
3			* Lock class.
4			*
5			* @package automattic/jetpack-sync
6			*/
7
8			namespace Automattic\Jetpack\Sync;
9
10			/**
11			* Lock class
12			*/
13			class Lock {
14			/**
15			* Prefix of the blog lock transient.
16			*
17			* @access public
18			*
19			* @var string
20			*/
21			const LOCK_PREFIX = 'jp_sync_lock_';
22
23			/**
24			* Default Lifetime of the lock.
25			*
26			* @access public
27			*
28			* @var int
29			*/
30			const LOCK_TRANSIENT_EXPIRY = 15; // Seconds.
31
32			/**
33			* Attempt to lock.
34			*
35			* @access public
36			*
37			* @param string $name lock name.
38			* @param int $expiry lock duration in seconds.
39			*
40			* @return boolean True if succeeded, false otherwise.
41			*/
42			public function attempt( $name, $expiry = self::LOCK_TRANSIENT_EXPIRY ) {
43			$name = self::LOCK_PREFIX . $name;
44			$locked_time = get_option( $name );
45			if ( $locked_time ) {
46			if ( microtime( true ) < $locked_time ) {
47			return false;
48			}
49			}
50			update_option( $name, microtime( true ) + $expiry );
51
52			return true;
53			}
54
55			/**
56			* Remove the lock.
57			*
58			* @access public
59			*
60			* @param string $name lock name.
61			*/
62			public function remove( $name ) {
63			delete_option( self::LOCK_PREFIX . $name );
64			}
65			}
66

Automattic / jetpack

Push — add/masterbar--my-home ( 5c5e9c...8732fd )

Lock::remove() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like