Pink-Crab / WP_PSR16_Cache

src/Cache_Item.php

Spacing +1 added lines, -1 removed lines

@@ -55,7 +55,7 @@
 	 * @param mixed $data
 	 * @param int $expiry
 	 */
-	public function __construct( string $key, $data, int $expiry ) {
+	public function __construct(string $key, $data, int $expiry) {
 		$this->key    = $key;
 		$this->data   = $data;
 		$this->expiry = $expiry;

src/CacheInterface_Trait.php

Spacing +19 added lines, -19 removed lines

@@ -33,11 +33,11 @@ 
 	 * @return bool
 	 * @throws InvalidArgumentException
 	 */
-	protected function is_valid_key_value( $key ) : bool {
-		if ( ! \is_string( $key ) || empty( $key ) ) {
-			throw new \InvalidArgumentException( 'Key must be a valid string' );
+	protected function is_valid_key_value($key) : bool {
+		if ( ! \is_string($key) || empty($key)) {
+			throw new \InvalidArgumentException('Key must be a valid string');
 		}
-		return ! (bool) \preg_match( '|[\{\}\(\)/\\\@\:]|', $key );
+		return ! (bool) \preg_match('|[\{\}\(\)/\\\@\:]|', $key);
 	}
 
 	/**
@@ -46,20 +46,20 @@ 
 	 * @param mixed $ttl
 	 * @return int
 	 */
-	public function ttl_to_seconds( $ttl ):int {
-		switch ( true ) {
-			case is_a( $ttl, DateInterval::class ):
-				$days  = (int) $ttl->format( '%a' );
-				$hours = (int) $ttl->format( '%h' );
-				$mins  = (int) $ttl->format( '%i' );
-				$secs  = (int) $ttl->format( '%s' );
+	public function ttl_to_seconds($ttl):int {
+		switch (true) {
+			case is_a($ttl, DateInterval::class):
+				$days  = (int) $ttl->format('%a');
+				$hours = (int) $ttl->format('%h');
+				$mins  = (int) $ttl->format('%i');
+				$secs  = (int) $ttl->format('%s');
 
-				return ( $days * 24 * 60 * 60 )
-					+ ( $hours * 60 * 60 )
-					+ ( $mins * 60 )
+				return ($days * 24 * 60 * 60)
+					+ ($hours * 60 * 60)
+					+ ($mins * 60)
 					+ $secs;
 
-			case is_numeric( $ttl ):
+			case is_numeric($ttl):
 				return (int) $ttl;
 
 			default:
@@ -73,13 +73,13 @@ 
 	 * @param array<mixed> $array
 	 * @return bool
 	 */
-	protected function all_true( array $array ): bool {
-		foreach ( $array as $value ) {
-			if ( ! is_bool( $value ) ) {
+	protected function all_true(array $array): bool {
+		foreach ($array as $value) {
+			if ( ! is_bool($value)) {
 				return false;
 			}
 
-			if ( $value === false ) {
+			if ($value === false) {
 				return false;
 			}
 		}

src/File_Cache.php

Spacing +41 added lines, -41 removed lines

@@ -68,8 +68,8 @@ 
 	 * @param string $filepath
 	 * @param string $extension
 	 */
-	public function __construct( string $filepath, string $extension = '.do' ) {
-		$this->filepath  = rtrim( $filepath, '\\/' );
+	public function __construct(string $filepath, string $extension = '.do') {
+		$this->filepath  = rtrim($filepath, '\\/');
 		$this->extension = $extension;
 
 		$this->set_wp_file_system();
@@ -94,8 +94,8 @@ 
 	 * @return void
 	 */
 	protected function maybe_create_cache_dir(): void {
-		if ( ! $this->wp_filesystem->exists( $this->filepath ) ) {
-			$this->wp_filesystem->mkdir( $this->filepath );
+		if ( ! $this->wp_filesystem->exists($this->filepath)) {
+			$this->wp_filesystem->mkdir($this->filepath);
 		}
 	}
 
@@ -106,11 +106,11 @@ 
 	 * @return bool
 	 * @throws InvalidArgumentException
 	 */
-	public function has( $key ) {
-		if ( ! $this->is_valid_key_value( $key ) ) {
+	public function has($key) {
+		if ( ! $this->is_valid_key_value($key)) {
 			return false;
 		}
-		return ! is_null( $this->get( $key ) );
+		return ! is_null($this->get($key));
 	}
 
 	/**
@@ -123,14 +123,14 @@ 
 	 * @return bool
 	 * @throws InvalidArgumentException
 	 */
-	public function set( $key, $value, $ttl = null ) {
-		if ( ! $this->is_valid_key_value( $key ) ) {
+	public function set($key, $value, $ttl = null) {
+		if ( ! $this->is_valid_key_value($key)) {
 			return false;
 		}
 
 		return $this->wp_filesystem->put_contents(
-			$this->compile_file_path( $key ),
-			serialize( $this->compile_cache_item( $key, $value, $ttl ) ) // phpcs:disable WordPress.PHP.DiscouragedPHPFunctions.serialize_serialize
+			$this->compile_file_path($key),
+			serialize($this->compile_cache_item($key, $value, $ttl)) // phpcs:disable WordPress.PHP.DiscouragedPHPFunctions.serialize_serialize
 		);
 	}
 
@@ -142,11 +142,11 @@ 
 	 * @return mixed
 	 * @throws InvalidArgumentException
 	 */
-	public function get( $key, $default = null ) {
-		if ( ! $this->is_valid_key_value( $key ) ) {
+	public function get($key, $default = null) {
+		if ( ! $this->is_valid_key_value($key)) {
 			return $default;
 		}
-		$contents = $this->get_contents( $key );
+		$contents = $this->get_contents($key);
 		return $contents ? $contents->data : $default;
 	}
 
@@ -157,12 +157,12 @@ 
 	 * @return bool
 	 * @throws InvalidArgumentException
 	 */
-	public function delete( $key ) {
-		if ( ! $this->is_valid_key_value( $key ) ) {
+	public function delete($key) {
+		if ( ! $this->is_valid_key_value($key)) {
 			return false;
 		}
 		return $this->wp_filesystem->delete(
-			$this->compile_file_path( $key )
+			$this->compile_file_path($key)
 		);
 	}
 
@@ -172,7 +172,7 @@ 
 	 * @return bool
 	 */
 	public function clear() {
-		return $this->wp_filesystem->delete( $this->filepath, true );
+		return $this->wp_filesystem->delete($this->filepath, true);
 	}
 
 	/**
@@ -182,11 +182,11 @@ 
 	 * @param string|float|int|array<mixed>|object|resource|bool $default
 	 * @return array<string, mixed>
 	 */
-	public function getMultiple( $keys, $default = null ) {
+	public function getMultiple($keys, $default = null) {
 		return array_reduce(
 			$keys,
-			function( $carry, $key ) use ( $default ) {
-				$carry[ $key ] = $this->get( $key, $default );
+			function($carry, $key) use ($default) {
+				$carry[ $key ] = $this->get($key, $default);
 				return $carry;
 			},
 			array()
@@ -200,12 +200,12 @@ 
 	 * @param int|null $ttl
 	 * @return bool
 	 */
-	public function setMultiple( $values, $ttl = null ) {
+	public function setMultiple($values, $ttl = null) {
 		return $this->all_true(
 			array_reduce(
-				array_keys( $values ),
-				function( $carry, $key ) use ( $values, $ttl ) {
-					$carry[ $key ] = $this->set( $key, $values[ $key ], $ttl );
+				array_keys($values),
+				function($carry, $key) use ($values, $ttl) {
+					$carry[ $key ] = $this->set($key, $values[ $key ], $ttl);
 					return $carry;
 				},
 				array()
@@ -219,11 +219,11 @@ 
 	 * @param array<int, string> $keys
 	 * @return bool
 	 */
-	public function deleteMultiple( $keys ) {
+	public function deleteMultiple($keys) {
 		return $this->all_true(
 			array_map(
-				function( $key ) {
-					return $this->delete( $key );
+				function($key) {
+					return $this->delete($key);
 				},
 				$keys
 			)
@@ -237,7 +237,7 @@ 
 	 * @param string $filename;
 	 * @return string
 	 */
-	protected function compile_file_path( string $filename ): string {
+	protected function compile_file_path(string $filename): string {
 		return \wp_normalize_path(
 			sprintf(
 				'%s/%s%s',
@@ -256,11 +256,11 @@ 
 	 * @param null|int|\DateInterval $ttl
 	 * @return Cache_Item
 	 */
-	protected function compile_cache_item( string $key, $data, $ttl ): Cache_Item {
+	protected function compile_cache_item(string $key, $data, $ttl): Cache_Item {
 		return new Cache_Item(
 			$key,
 			$data,
-			$this->compile_expiry( $this->ttl_to_seconds( $ttl ) )
+			$this->compile_expiry($this->ttl_to_seconds($ttl))
 		);
 	}
 
@@ -270,7 +270,7 @@ 
 	 * @param int $expiry
 	 * @return int
 	 */
-	protected function compile_expiry( int $expiry ): int {
+	protected function compile_expiry(int $expiry): int {
 		return $expiry !== 0 ? $expiry + time() : 0;
 	}
 
@@ -283,22 +283,22 @@ 
 	 * @param Cache_Item $data
 	 * @return bool
 	 */
-	protected function validate_contents( string $key, Cache_Item $data ): bool {
-		switch ( true ) {
+	protected function validate_contents(string $key, Cache_Item $data): bool {
+		switch (true) {
 			// Key passes doesnt match cache.
 			case $data->key !== $key:
 				return false;
 
 			// Expiry isnt numeric
-			case ! is_numeric( $data->expiry ):
+			case ! is_numeric($data->expiry):
 				return false;
 
 			// If expiry is 0 (Never expires), return true.
-			case intval( $data->expiry ) === 0:
+			case intval($data->expiry) === 0:
 				return true;
 
 			// If a timestamp, but expired.
-			case intval( $data->expiry ) < time():
+			case intval($data->expiry) < time():
 				return false;
 
 			default:
@@ -312,13 +312,13 @@ 
 	 * @param string $key
 	 * @return Cache_Item|null If we have valid data (not expired), the data else null
 	 */
-	protected function get_contents( string $key ): ?Cache_Item {
+	protected function get_contents(string $key): ?Cache_Item {
 
-		$file_contents = $this->wp_filesystem->get_contents( $this->compile_file_path( $key ) ) ?: '';
+		$file_contents = $this->wp_filesystem->get_contents($this->compile_file_path($key)) ?: '';
 
-		$file_contents = \maybe_unserialize( $file_contents );
+		$file_contents = \maybe_unserialize($file_contents);
 
-		return is_a( $file_contents, Cache_Item::class ) && $this->validate_contents( $key, $file_contents )
+		return is_a($file_contents, Cache_Item::class) && $this->validate_contents($key, $file_contents)
 			? $file_contents
 			: null;
 	}

src/Transient_Cache.php

Spacing +28 added lines, -28 removed lines

@@ -57,7 +57,7 @@ 
 	 *
 	 * @param string|null $group
 	 */
-	public function __construct( ?string $group = null ) {
+	public function __construct(?string $group = null) {
 		$this->group = $group;
 	}
 
@@ -71,14 +71,14 @@ 
 	 * @return bool
 	 * @throws InvalidArgumentException
 	 */
-	public function set( $key, $value, $ttl = null ) {
-		if ( ! $this->is_valid_key_value( $key ) ) {
+	public function set($key, $value, $ttl = null) {
+		if ( ! $this->is_valid_key_value($key)) {
 			return false;
 		}
 		return \set_transient(
-			$this->parse_key( $key ),
+			$this->parse_key($key),
 			$value,
-			$this->ttl_to_seconds( $ttl )
+			$this->ttl_to_seconds($ttl)
 		);
 	}
 
@@ -90,11 +90,11 @@ 
 	 * @return mixed
 	 * @throws InvalidArgumentException
 	 */
-	public function get( $key, $default = null ) {
-		if ( ! $this->is_valid_key_value( $key ) ) {
+	public function get($key, $default = null) {
+		if ( ! $this->is_valid_key_value($key)) {
 			return $default;
 		}
-		return \get_transient( $this->parse_key( $key ) ) ?: $default;
+		return \get_transient($this->parse_key($key)) ?: $default;
 	}
 
 	/**
@@ -104,11 +104,11 @@ 
 	 * @return bool
 	 * @throws InvalidArgumentException
 	 */
-	public function delete( $key ) {
-		if ( ! $this->is_valid_key_value( $key ) ) {
+	public function delete($key) {
+		if ( ! $this->is_valid_key_value($key)) {
 			return false;
 		}
-		return \delete_transient( $this->parse_key( $key ) );
+		return \delete_transient($this->parse_key($key));
 	}
 
 	/**
@@ -118,12 +118,12 @@ 
 	 */
 	public function clear() {
 		$results = array_map(
-			function( $e ) {
-				return \delete_transient( $this->parse_key( $e ) );
+			function($e) {
+				return \delete_transient($this->parse_key($e));
 			},
 			$this->get_group_keys()
 		);
-		return ! \in_array( false, $results, true );
+		return ! \in_array(false, $results, true);
 	}
 
 	/**
@@ -133,11 +133,11 @@ 
 	 * @param mixed $default
 	 * @return array<string, mixed>
 	 */
-	public function getMultiple( $keys, $default = null ) {
+	public function getMultiple($keys, $default = null) {
 		return array_reduce(
 			$keys,
-			function( $carry, $key ) use ( $default ) {
-				$carry[ $key ] = $this->get( $key, $default );
+			function($carry, $key) use ($default) {
+				$carry[ $key ] = $this->get($key, $default);
 				return $carry;
 			},
 			array()
@@ -151,12 +151,12 @@ 
 	 * @param DateInterval|int|null $ttl
 	 * @return bool
 	 */
-	public function setMultiple( $values, $ttl = null ) {
+	public function setMultiple($values, $ttl = null) {
 		return $this->all_true(
 			array_reduce(
-				array_keys( $values ),
-				function( $carry, $key ) use ( $values, $ttl ) {
-					$carry[ $key ] = $this->set( $key, $values[ $key ], $ttl );
+				array_keys($values),
+				function($carry, $key) use ($values, $ttl) {
+					$carry[ $key ] = $this->set($key, $values[ $key ], $ttl);
 					return $carry;
 				},
 				array()
@@ -170,12 +170,12 @@ 
 	 * @param array<int, string> $keys
 	 * @return bool
 	 */
-	public function deleteMultiple( $keys ) {
+	public function deleteMultiple($keys) {
 		return $this->all_true(
 			array_reduce(
 				$keys,
-				function( $carry, $key ) {
-					$carry[ $key ] = $this->delete( $key );
+				function($carry, $key) {
+					$carry[ $key ] = $this->delete($key);
 					return $carry;
 				},
 				array()
@@ -189,8 +189,8 @@ 
 	 * @param string $key
 	 * @return bool
 	 */
-	public function has( $key ) {
-		return ! is_null( $this->get( $key ) );
+	public function has($key) {
+		return ! is_null($this->get($key));
 	}
 
 	/**
@@ -199,7 +199,7 @@ 
 	 * @param string $key
 	 * @return string
 	 */
-	protected function parse_key( string $key ): string {
+	protected function parse_key(string $key): string {
 		return \sprintf(
 			'%s%s',
 			$this->group_key_prefix(),
@@ -226,7 +226,7 @@ 
 	protected function get_group_keys(): array {
 		// Extract the base cache keys (excluding pre/postfixes)
 		return array_map(
-			function( $key ) {
+			function($key) {
 				return \str_replace(
 					'_transient_' . $this->group_key_prefix(),
 					'',