fix some inconsistent return values

timse201 · Mar 29, 2024 · 053b59a · 053b59a
1 parent 87b5b03
commit 053b59a
Show file tree

Hide file tree

Showing 6 changed files with 14 additions and 13 deletions.
diff --git a/inc/class-cachify-apc.php b/inc/class-cachify-apc.php
@@ -105,7 +105,7 @@ public static function clear_cache() {
 	 * @since 2.0
 	 */
 	public static function print_cache() {
-		return;
+		// Not supported.
 	}
 
 	/**

diff --git a/inc/class-cachify-db.php b/inc/class-cachify-db.php
@@ -146,9 +146,10 @@ public static function get_stats() {
 		global $wpdb;
 
 		/* Read */
-
-		return $wpdb->get_var(
-			'SELECT SUM( CHAR_LENGTH(option_value) ) FROM `' . $wpdb->options . "` WHERE `option_name` LIKE ('\_transient%.cachify')"
+		return intval(
+			$wpdb->get_var(
+				'SELECT SUM( CHAR_LENGTH(option_value) ) FROM `' . $wpdb->options . "` WHERE `option_name` LIKE ('\_transient%.cachify')"
+			)
 		);
 	}
 
@@ -166,7 +167,7 @@ public static function get_stats() {
 	private static function _cache_signature( $detail, $meta ) {
 		/* No array? */
 		if ( ! is_array( $meta ) ) {
-			return;
+			return '';
 		}
 
 		if ( $detail ) {
@@ -218,7 +219,7 @@ private static function _page_queries() {
 	/**
 	 * Return execution time
 	 *
-	 * @return int Execution time in seconds
+	 * @return string Execution time in seconds
 	 *
 	 * @since 0.1
 	 */

diff --git a/inc/class-cachify-hdd.php b/inc/class-cachify-hdd.php
@@ -275,14 +275,14 @@ private static function _clear_dir( $dir, $recursive = false ) {
 	 *
 	 * @param string $dir Directory path.
 	 *
-	 * @return mixed Directory size
+	 * @return int|false Directory size
 	 *
 	 * @since 2.0
 	 */
 	public static function _dir_size( $dir = '.' ) {
 		/* Is directory? */
 		if ( ! is_dir( $dir ) ) {
-			return;
+			return false;
 		}
 
 		/* Read */
@@ -293,7 +293,7 @@ public static function _dir_size( $dir = '.' ) {
 
 		/* Empty? */
 		if ( empty( $objects ) ) {
-			return;
+			return false;
 		}
 
 		/* Init */

diff --git a/inc/class-cachify-memcached.php b/inc/class-cachify-memcached.php
@@ -89,7 +89,7 @@ public static function store_item( $hash, $data, $lifetime, $sig_detail ) {
 	public static function get_item( $hash ) {
 		/* Server connect */
 		if ( ! self::_connect_server() ) {
-			return;
+			return null;
 		}
 
 		/* Get item */
@@ -143,7 +143,7 @@ public static function clear_cache() {
 	 * @since 2.0.7
 	 */
 	public static function print_cache() {
-		return;
+		// Not supported.
 	}
 
 	/**

diff --git a/inc/class-cachify-redis.php b/inc/class-cachify-redis.php
@@ -77,7 +77,7 @@ public static function store_item( $hash, $data, $lifetime, $sig_detail ) {
 	public static function get_item( $hash ) {
 		/* Server connect */
 		if ( ! self::_connect_server() ) {
-			return;
+			return null;
 		}
 
 		/* Get item */

diff --git a/inc/class-cachify.php b/inc/class-cachify.php
@@ -874,7 +874,7 @@ public static function flush_cache() {
 	public static function flush_notice() {
 		/* No admin */
 		if ( ! is_admin_bar_showing() || ! apply_filters( 'cachify_user_can_flush_cache', current_user_can( 'manage_options' ) ) ) {
-			return false;
+			return;
 		}
 
 		printf(