insideout10 / wordlift-plugin

src/wordlift/http/class-cacheable-http-client.php

Indentation +103 added lines, -103 removed lines

@@ -15,108 +15,108 @@
 // @@todo: add a hook to clear the cached files now and then.
 class Cacheable_Http_Client extends Simple_Http_Client {
 
-	/**
-	 * The TTL of cached responses in seconds.
-	 *
-	 * @var int $ttl The TTL in seconds.
-	 * @access private
-	 * @since 1.0.0
-	 */
-	private $ttl;
-
-	/**
-	 * The cache dir where the cached responses are written.
-	 *
-	 * @since 1.0.0
-	 * @access private
-	 * @var string $cache_dir The cache dir where the cached responses are written.
-	 */
-	private $cache_dir;
-
-	/**
-	 * A {@link Wordlift_Log_Service} instance.
-	 *
-	 * @var \Wordlift_Log_Service $log A {@link Wordlift_Log_Service} instance.
-	 * @access private
-	 * @since 1.0.0
-	 */
-	private $log;
-
-	/**
-	 * Create a {@link Cacheable_Http_Client} with the specified TTL, default 900 secs.
-	 *
-	 * @param int $ttl The cache TTL, default 900 secs.
-	 *
-	 * @since 1.0.0
-	 */
-	public function __construct( $ttl = 900 ) {
-
-		$this->log = \Wordlift_Log_Service::get_logger( get_class() );
-
-		$this->ttl = $ttl;
-
-		// Get the temp dir and add the directory separator if missing.
-		$temp_dir = get_temp_dir();
-		if ( DIRECTORY_SEPARATOR !== substr( $temp_dir, - strlen( DIRECTORY_SEPARATOR ) ) ) {
-			$temp_dir .= DIRECTORY_SEPARATOR;
-		}
-		$this->cache_dir = $temp_dir . 'wlfb-http-cache';
-
-		$this->log->trace( "Creating the cache folder {$this->cache_dir}..." );
-		wp_mkdir_p( $this->cache_dir );
-
-	}
-
-	/**
-	 * @inheritDoc
-	 */
-	public function request( $url, $options = array() ) {
-
-		// Create a hash and a path to the cache file.
-		$hash     = md5( $url ) . '-' . md5( serialize( $options ) );
-		$filename = $this->get_path( $hash );
-
-		// If the cache file exists and it's not too old, then return it.
-		if ( file_exists( $filename ) && $this->ttl >= time() - filemtime( $filename ) ) {
-			$this->log->trace( "Cache HIT.\n" );
-
-			return json_decode( file_get_contents( $filename ), true );
-		}
-
-		$this->log->trace( "Cache MISS for URL $url, hash $hash.\n" );
-
-		// Get a fresh response and return it.
-		$response = parent::request( $url, $options );
-
-		// Return immediately, do not cache.
-		if ( is_wp_error( $response ) ) {
-			return $response;
-		}
-
-		// Do not cache response with invalid status codes or status code different from 2xx.
-		$code = wp_remote_retrieve_response_code( $response );
-		if ( ! is_numeric( $code ) || 2 !== intval( $code ) / 100 ) {
-			return $response;
-		}
-
-		// Cache.
-		@unlink( $filename );
-		@file_put_contents( $filename, json_encode( $response ) );
-
-		return $response;
-	}
-
-	/**
-	 * Get the full path for the given `$hash`. The file is not checked for its existence.
-	 *
-	 * @param string $hash A file hash.
-	 *
-	 * @return string The full path to the file.
-	 * @since 1.0.0
-	 */
-	private function get_path( $hash ) {
-
-		return $this->cache_dir . DIRECTORY_SEPARATOR . $hash;
-	}
+    /**
+     * The TTL of cached responses in seconds.
+     *
+     * @var int $ttl The TTL in seconds.
+     * @access private
+     * @since 1.0.0
+     */
+    private $ttl;
+
+    /**
+     * The cache dir where the cached responses are written.
+     *
+     * @since 1.0.0
+     * @access private
+     * @var string $cache_dir The cache dir where the cached responses are written.
+     */
+    private $cache_dir;
+
+    /**
+     * A {@link Wordlift_Log_Service} instance.
+     *
+     * @var \Wordlift_Log_Service $log A {@link Wordlift_Log_Service} instance.
+     * @access private
+     * @since 1.0.0
+     */
+    private $log;
+
+    /**
+     * Create a {@link Cacheable_Http_Client} with the specified TTL, default 900 secs.
+     *
+     * @param int $ttl The cache TTL, default 900 secs.
+     *
+     * @since 1.0.0
+     */
+    public function __construct( $ttl = 900 ) {
+
+        $this->log = \Wordlift_Log_Service::get_logger( get_class() );
+
+        $this->ttl = $ttl;
+
+        // Get the temp dir and add the directory separator if missing.
+        $temp_dir = get_temp_dir();
+        if ( DIRECTORY_SEPARATOR !== substr( $temp_dir, - strlen( DIRECTORY_SEPARATOR ) ) ) {
+            $temp_dir .= DIRECTORY_SEPARATOR;
+        }
+        $this->cache_dir = $temp_dir . 'wlfb-http-cache';
+
+        $this->log->trace( "Creating the cache folder {$this->cache_dir}..." );
+        wp_mkdir_p( $this->cache_dir );
+
+    }
+
+    /**
+     * @inheritDoc
+     */
+    public function request( $url, $options = array() ) {
+
+        // Create a hash and a path to the cache file.
+        $hash     = md5( $url ) . '-' . md5( serialize( $options ) );
+        $filename = $this->get_path( $hash );
+
+        // If the cache file exists and it's not too old, then return it.
+        if ( file_exists( $filename ) && $this->ttl >= time() - filemtime( $filename ) ) {
+            $this->log->trace( "Cache HIT.\n" );
+
+            return json_decode( file_get_contents( $filename ), true );
+        }
+
+        $this->log->trace( "Cache MISS for URL $url, hash $hash.\n" );
+
+        // Get a fresh response and return it.
+        $response = parent::request( $url, $options );
+
+        // Return immediately, do not cache.
+        if ( is_wp_error( $response ) ) {
+            return $response;
+        }
+
+        // Do not cache response with invalid status codes or status code different from 2xx.
+        $code = wp_remote_retrieve_response_code( $response );
+        if ( ! is_numeric( $code ) || 2 !== intval( $code ) / 100 ) {
+            return $response;
+        }
+
+        // Cache.
+        @unlink( $filename );
+        @file_put_contents( $filename, json_encode( $response ) );
+
+        return $response;
+    }
+
+    /**
+     * Get the full path for the given `$hash`. The file is not checked for its existence.
+     *
+     * @param string $hash A file hash.
+     *
+     * @return string The full path to the file.
+     * @since 1.0.0
+     */
+    private function get_path( $hash ) {
+
+        return $this->cache_dir . DIRECTORY_SEPARATOR . $hash;
+    }
 
 }

Spacing +21 added lines, -21 removed lines

@@ -49,59 +49,59 @@ 
 	 *
 	 * @since 1.0.0
 	 */
-	public function __construct( $ttl = 900 ) {
+	public function __construct($ttl = 900) {
 
-		$this->log = \Wordlift_Log_Service::get_logger( get_class() );
+		$this->log = \Wordlift_Log_Service::get_logger(get_class());
 
 		$this->ttl = $ttl;
 
 		// Get the temp dir and add the directory separator if missing.
 		$temp_dir = get_temp_dir();
-		if ( DIRECTORY_SEPARATOR !== substr( $temp_dir, - strlen( DIRECTORY_SEPARATOR ) ) ) {
+		if (DIRECTORY_SEPARATOR !== substr($temp_dir, - strlen(DIRECTORY_SEPARATOR))) {
 			$temp_dir .= DIRECTORY_SEPARATOR;
 		}
-		$this->cache_dir = $temp_dir . 'wlfb-http-cache';
+		$this->cache_dir = $temp_dir.'wlfb-http-cache';
 
-		$this->log->trace( "Creating the cache folder {$this->cache_dir}..." );
-		wp_mkdir_p( $this->cache_dir );
+		$this->log->trace("Creating the cache folder {$this->cache_dir}...");
+		wp_mkdir_p($this->cache_dir);
 
 	}
 
 	/**
 	 * @inheritDoc
 	 */
-	public function request( $url, $options = array() ) {
+	public function request($url, $options = array()) {
 
 		// Create a hash and a path to the cache file.
-		$hash     = md5( $url ) . '-' . md5( serialize( $options ) );
-		$filename = $this->get_path( $hash );
+		$hash     = md5($url).'-'.md5(serialize($options));
+		$filename = $this->get_path($hash);
 
 		// If the cache file exists and it's not too old, then return it.
-		if ( file_exists( $filename ) && $this->ttl >= time() - filemtime( $filename ) ) {
-			$this->log->trace( "Cache HIT.\n" );
+		if (file_exists($filename) && $this->ttl >= time() - filemtime($filename)) {
+			$this->log->trace("Cache HIT.\n");
 
-			return json_decode( file_get_contents( $filename ), true );
+			return json_decode(file_get_contents($filename), true);
 		}
 
-		$this->log->trace( "Cache MISS for URL $url, hash $hash.\n" );
+		$this->log->trace("Cache MISS for URL $url, hash $hash.\n");
 
 		// Get a fresh response and return it.
-		$response = parent::request( $url, $options );
+		$response = parent::request($url, $options);
 
 		// Return immediately, do not cache.
-		if ( is_wp_error( $response ) ) {
+		if (is_wp_error($response)) {
 			return $response;
 		}
 
 		// Do not cache response with invalid status codes or status code different from 2xx.
-		$code = wp_remote_retrieve_response_code( $response );
-		if ( ! is_numeric( $code ) || 2 !== intval( $code ) / 100 ) {
+		$code = wp_remote_retrieve_response_code($response);
+		if ( ! is_numeric($code) || 2 !== intval($code) / 100) {
 			return $response;
 		}
 
 		// Cache.
-		@unlink( $filename );
-		@file_put_contents( $filename, json_encode( $response ) );
+		@unlink($filename);
+		@file_put_contents($filename, json_encode($response));
 
 		return $response;
 	}
@@ -114,9 +114,9 @@ 
 	 * @return string The full path to the file.
 	 * @since 1.0.0
 	 */
-	private function get_path( $hash ) {
+	private function get_path($hash) {
 
-		return $this->cache_dir . DIRECTORY_SEPARATOR . $hash;
+		return $this->cache_dir.DIRECTORY_SEPARATOR.$hash;
 	}
 
 }

src/wordlift/http/class-simple-http-client.php

Indentation +13 added lines, -13 removed lines

@@ -10,22 +10,22 @@
  */
 class Simple_Http_Client implements Http_Client {
 
-	/**
-	 * @inheritDoc
-	 */
-	public function get( $url, $options = array() ) {
+    /**
+     * @inheritDoc
+     */
+    public function get( $url, $options = array() ) {
 
-		$options['method'] = 'GET';
+        $options['method'] = 'GET';
 
-		return $this->request( $url, $options );
-	}
+        return $this->request( $url, $options );
+    }
 
-	/**
-	 * @inheritDoc
-	 */
-	public function request( $url, $options = array() ) {
+    /**
+     * @inheritDoc
+     */
+    public function request( $url, $options = array() ) {
 
-		return wp_remote_request( $url, $options );
-	}
+        return wp_remote_request( $url, $options );
+    }
 
 }

Spacing +4 added lines, -4 removed lines

@@ -13,19 +13,19 @@
 	/**
 	 * @inheritDoc
 	 */
-	public function get( $url, $options = array() ) {
+	public function get($url, $options = array()) {
 
 		$options['method'] = 'GET';
 
-		return $this->request( $url, $options );
+		return $this->request($url, $options);
 	}
 
 	/**
 	 * @inheritDoc
 	 */
-	public function request( $url, $options = array() ) {
+	public function request($url, $options = array()) {
 
-		return wp_remote_request( $url, $options );
+		return wp_remote_request($url, $options);
 	}
 
 }

src/install/class-wordlift-install-3-23-4.php

Indentation +21 added lines, -21 removed lines

@@ -14,32 +14,32 @@
  */
 class Wordlift_Install_3_23_4 extends Wordlift_Install {
 
-	/**
-	 * {@inheritdoc}
-	 */
-	protected static $version = '3.23.4';
+    /**
+     * {@inheritdoc}
+     */
+    protected static $version = '3.23.4';
 
-	public function install() {
+    public function install() {
 
-		$existing_term = get_term_by( 'slug', 'web-page', Wordlift_Entity_Type_Taxonomy_Service::TAXONOMY_NAME );
+        $existing_term = get_term_by( 'slug', 'web-page', Wordlift_Entity_Type_Taxonomy_Service::TAXONOMY_NAME );
 
-		// Bail out if term exists.
-		if ( false !== $existing_term ) {
-			return;
-		}
+        // Bail out if term exists.
+        if ( false !== $existing_term ) {
+            return;
+        }
 
-		$term = wp_insert_term(
-			'WebPage',
-			Wordlift_Entity_Type_Taxonomy_Service::TAXONOMY_NAME,
-			array(
-				'slug'        => 'web-page',
-				'description' => 'A Web Page.',
-			)
-		);
+        $term = wp_insert_term(
+            'WebPage',
+            Wordlift_Entity_Type_Taxonomy_Service::TAXONOMY_NAME,
+            array(
+                'slug'        => 'web-page',
+                'description' => 'A Web Page.',
+            )
+        );
 
-		update_term_meta( $term['term_id'], '_wl_name', 'WebPage' );
-		update_term_meta( $term['term_id'], '_wl_uri', "http://schema.org/WebPage" );
+        update_term_meta( $term['term_id'], '_wl_name', 'WebPage' );
+        update_term_meta( $term['term_id'], '_wl_uri', "http://schema.org/WebPage" );
 
-	}
+    }
 
 }

Spacing +4 added lines, -4 removed lines

@@ -21,10 +21,10 @@ 
 
 	public function install() {
 
-		$existing_term = get_term_by( 'slug', 'web-page', Wordlift_Entity_Type_Taxonomy_Service::TAXONOMY_NAME );
+		$existing_term = get_term_by('slug', 'web-page', Wordlift_Entity_Type_Taxonomy_Service::TAXONOMY_NAME);
 
 		// Bail out if term exists.
-		if ( false !== $existing_term ) {
+		if (false !== $existing_term) {
 			return;
 		}
 
@@ -37,8 +37,8 @@ 
 			)
 		);
 
-		update_term_meta( $term['term_id'], '_wl_name', 'WebPage' );
-		update_term_meta( $term['term_id'], '_wl_uri', "http://schema.org/WebPage" );
+		update_term_meta($term['term_id'], '_wl_name', 'WebPage');
+		update_term_meta($term['term_id'], '_wl_uri', "http://schema.org/WebPage");
 
 	}
 

src/includes/cache/class-wordlift-file-cache-service.php

Indentation +211 added lines, -211 removed lines

@@ -16,231 +16,231 @@
  */
 class Wordlift_File_Cache_Service implements Wordlift_Cache_Service {
 
-	/**
-	 * The cache directory.
-	 *
-	 * @since  3.16.0
-	 * @access private
-	 * @var string $cache_dir The root cache directory (ending with a trailing slash).
-	 */
-	private $cache_dir;
-
-	/**
-	 * The file extension for cache files (e.g. `.wlcache`).
-	 *
-	 * @since  3.16.0
-	 * @access private
-	 * @var string $file_extension The file extension for cache files (e.g. `.wlcache`).
-	 */
-	private $file_extension;
-
-	/**
-	 * A {@link Wordlift_Log_Service} instance.
-	 *
-	 * @since  3.16.0
-	 * @access private
-	 * @var \Wordlift_Log_Service $log A {@link Wordlift_Log_Service} instance.
-	 */
-	private $log;
-
-	/**
-	 * The {@link Wordlift_File_Cache_Service} registered instances.
-	 *
-	 * Each {@link Wordlift_File_Cache_Service} adds itself to the registered
-	 * instances.
-	 *
-	 * @since  3.16.3
-	 * @access private
-	 * @var array $instances An array of {@link Wordlift_File_Cache_Service} instances.
-	 */
-	private static $instances = array();
-
-	private static $instance;
-
-	/**
-	 * Create a {@link Wordlift_File_Cache_Service} instance.
-	 *
-	 * The File Cache Service requires a base cache directory (to which a unique
-	 * id for the current site will be appended) and a file extension for cache
-	 * files (by default `.wlcache`) is used.
-	 *
-	 * @param string $cache_dir The base cache directory.
-	 * @param string $file_extension The file extension, by default `.wlcache`.
-	 *
-	 * @since 3.16.0
-	 *
-	 */
-	public function __construct( $cache_dir, $file_extension = '.wlcache' ) {
-
-		$this->log = Wordlift_Log_Service::get_logger( get_class() );
-
-		// Set the cache directory using the base directory provided by the caller
-		// and appending a hash for the unique site id.
-		$this->cache_dir      = trailingslashit( $cache_dir ) . md5( get_site_url() ) . '/';
-		$this->file_extension = $file_extension;
-
-		// Create the cache dir.
-		if ( ! file_exists( $this->cache_dir ) ) {
-			@mkdir( $this->cache_dir, 0755, true );
-		}
-
-		// Add ourselves to the list of instances.
-		self::$instances[] = $this;
-
-		// Initialize the singleton and the ajax method.
-		if ( ! isset( self::$instance ) ) {
-			self::$instance = $this;
-
-			add_action( 'wp_ajax_wl_file_cache__flush_all', array( 'Wordlift_File_Cache_Service', 'flush_all' ) );
-		}
-
-		$this->log->debug( "File Cache service initialized on $this->cache_dir." );
-
-	}
-
-	/**
-	 * Get the cached response for the specified `id`.
-	 *
-	 * @param int $id The cache `id`.
-	 *
-	 * @return mixed|false The cached contents or false if the cache isn't found.
-	 * @since 3.16.0
-	 *
-	 */
-	function get_cache( $id ) {
-
-		// Bail out if we don't have the cache.
-		if ( ! $this->has_cache( $id ) ) {
-			return false;
-		}
-
-		// Get the filename.
-		$filename = $this->get_filename( $id );
-
-		$this->log->trace( "Trying to get cache contents for $id from $filename..." );
-
-		// Try to decode the contents.
-		$contents = json_decode( file_get_contents( $filename ), true );
-
-		// Return false if decoding failed, otherwise the decoded contents.
-		return $contents ?: false;
-	}
-
-	/**
-	 * Set the cache contents for the specified `id`.
-	 *
-	 * @param int $id The cache id.
-	 *
-	 * @return bool True if the `id` has a cache.
-	 * @since 3.16.0
-	 *
-	 */
-	function has_cache( $id ) {
-
-		// Get the filename.
-		$filename = $this->get_filename( $id );
-
-		// Bail out if the file doesn't exist.
-		return file_exists( $filename );
-	}
-
-	/**
-	 * @inheritdoc
-	 */
-	function set_cache( $id, $contents ) {
-
-		$filename = $this->get_filename( $id );
-
-		$this->log->trace( "Writing cache contents for $id to $filename..." );
-
-		@file_put_contents( $filename, wp_json_encode( $contents ) );
-
-	}
-
-	/**
-	 * Delete the cache for the specified `id`.
-	 *
-	 * @param int $id The cache `id`.
-	 *
-	 * @since 3.16.0
-	 *
-	 */
-	function delete_cache( $id ) {
-
-		$filename = $this->get_filename( $id );
+    /**
+     * The cache directory.
+     *
+     * @since  3.16.0
+     * @access private
+     * @var string $cache_dir The root cache directory (ending with a trailing slash).
+     */
+    private $cache_dir;
+
+    /**
+     * The file extension for cache files (e.g. `.wlcache`).
+     *
+     * @since  3.16.0
+     * @access private
+     * @var string $file_extension The file extension for cache files (e.g. `.wlcache`).
+     */
+    private $file_extension;
+
+    /**
+     * A {@link Wordlift_Log_Service} instance.
+     *
+     * @since  3.16.0
+     * @access private
+     * @var \Wordlift_Log_Service $log A {@link Wordlift_Log_Service} instance.
+     */
+    private $log;
+
+    /**
+     * The {@link Wordlift_File_Cache_Service} registered instances.
+     *
+     * Each {@link Wordlift_File_Cache_Service} adds itself to the registered
+     * instances.
+     *
+     * @since  3.16.3
+     * @access private
+     * @var array $instances An array of {@link Wordlift_File_Cache_Service} instances.
+     */
+    private static $instances = array();
+
+    private static $instance;
+
+    /**
+     * Create a {@link Wordlift_File_Cache_Service} instance.
+     *
+     * The File Cache Service requires a base cache directory (to which a unique
+     * id for the current site will be appended) and a file extension for cache
+     * files (by default `.wlcache`) is used.
+     *
+     * @param string $cache_dir The base cache directory.
+     * @param string $file_extension The file extension, by default `.wlcache`.
+     *
+     * @since 3.16.0
+     *
+     */
+    public function __construct( $cache_dir, $file_extension = '.wlcache' ) {
+
+        $this->log = Wordlift_Log_Service::get_logger( get_class() );
+
+        // Set the cache directory using the base directory provided by the caller
+        // and appending a hash for the unique site id.
+        $this->cache_dir      = trailingslashit( $cache_dir ) . md5( get_site_url() ) . '/';
+        $this->file_extension = $file_extension;
+
+        // Create the cache dir.
+        if ( ! file_exists( $this->cache_dir ) ) {
+            @mkdir( $this->cache_dir, 0755, true );
+        }
+
+        // Add ourselves to the list of instances.
+        self::$instances[] = $this;
+
+        // Initialize the singleton and the ajax method.
+        if ( ! isset( self::$instance ) ) {
+            self::$instance = $this;
+
+            add_action( 'wp_ajax_wl_file_cache__flush_all', array( 'Wordlift_File_Cache_Service', 'flush_all' ) );
+        }
+
+        $this->log->debug( "File Cache service initialized on $this->cache_dir." );
+
+    }
+
+    /**
+     * Get the cached response for the specified `id`.
+     *
+     * @param int $id The cache `id`.
+     *
+     * @return mixed|false The cached contents or false if the cache isn't found.
+     * @since 3.16.0
+     *
+     */
+    function get_cache( $id ) {
+
+        // Bail out if we don't have the cache.
+        if ( ! $this->has_cache( $id ) ) {
+            return false;
+        }
+
+        // Get the filename.
+        $filename = $this->get_filename( $id );
+
+        $this->log->trace( "Trying to get cache contents for $id from $filename..." );
+
+        // Try to decode the contents.
+        $contents = json_decode( file_get_contents( $filename ), true );
+
+        // Return false if decoding failed, otherwise the decoded contents.
+        return $contents ?: false;
+    }
+
+    /**
+     * Set the cache contents for the specified `id`.
+     *
+     * @param int $id The cache id.
+     *
+     * @return bool True if the `id` has a cache.
+     * @since 3.16.0
+     *
+     */
+    function has_cache( $id ) {
+
+        // Get the filename.
+        $filename = $this->get_filename( $id );
+
+        // Bail out if the file doesn't exist.
+        return file_exists( $filename );
+    }
+
+    /**
+     * @inheritdoc
+     */
+    function set_cache( $id, $contents ) {
+
+        $filename = $this->get_filename( $id );
+
+        $this->log->trace( "Writing cache contents for $id to $filename..." );
+
+        @file_put_contents( $filename, wp_json_encode( $contents ) );
+
+    }
+
+    /**
+     * Delete the cache for the specified `id`.
+     *
+     * @param int $id The cache `id`.
+     *
+     * @since 3.16.0
+     *
+     */
+    function delete_cache( $id ) {
+
+        $filename = $this->get_filename( $id );
 
-		$this->log->trace( "Deleting cache contents for $id, file $filename..." );
+        $this->log->trace( "Deleting cache contents for $id, file $filename..." );
 
-		if ( file_exists( $filename ) ) {
-			@unlink( $filename );
-		}
-
-	}
+        if ( file_exists( $filename ) ) {
+            @unlink( $filename );
+        }
+
+    }
 
-	/**
-	 * Flush the whole cache.
-	 *
-	 * @since 3.16.0
-	 */
-	function flush() {
+    /**
+     * Flush the whole cache.
+     *
+     * @since 3.16.0
+     */
+    function flush() {
 
-		// Bail out if the cache dir isn't set.
-		if ( empty( $this->cache_dir ) || '/' === $this->cache_dir ) {
-			return;
-		}
+        // Bail out if the cache dir isn't set.
+        if ( empty( $this->cache_dir ) || '/' === $this->cache_dir ) {
+            return;
+        }
 
-		$this->log->trace( "Flushing cache contents from $this->cache_dir..." );
+        $this->log->trace( "Flushing cache contents from $this->cache_dir..." );
 
-		$handle = @opendir( $this->cache_dir );
+        $handle = @opendir( $this->cache_dir );
 
-		// Bail out if the directory can't be opened.
-		if ( false === $handle ) {
-			return;
-		}
-
-		// Calculate the file extension length for matching file names.
-		$file_extension_length = strlen( $this->file_extension );
-
-		// Loop into the directory to delete files.
-		while ( false !== ( $entry = readdir( $handle ) ) ) {
-			if ( substr( $entry, - $file_extension_length ) === $this->file_extension
-			     && file_exists( $this->cache_dir . $entry ) ) {
-				$this->log->trace( "Deleting file {$this->cache_dir}{$entry}..." );
+        // Bail out if the directory can't be opened.
+        if ( false === $handle ) {
+            return;
+        }
+
+        // Calculate the file extension length for matching file names.
+        $file_extension_length = strlen( $this->file_extension );
+
+        // Loop into the directory to delete files.
+        while ( false !== ( $entry = readdir( $handle ) ) ) {
+            if ( substr( $entry, - $file_extension_length ) === $this->file_extension
+                 && file_exists( $this->cache_dir . $entry ) ) {
+                $this->log->trace( "Deleting file {$this->cache_dir}{$entry}..." );
 
-				@unlink( $this->cache_dir . $entry );
-			}
-		}
+                @unlink( $this->cache_dir . $entry );
+            }
+        }
 
-		// Finally closed the directory.
-		closedir( $handle );
+        // Finally closed the directory.
+        closedir( $handle );
 
-	}
+    }
 
-	public static function flush_all() {
+    public static function flush_all() {
 
-		foreach ( self::$instances as $instance ) {
-			$instance->flush();
-		}
+        foreach ( self::$instances as $instance ) {
+            $instance->flush();
+        }
 
-		if ( defined( 'DOING_AJAX' ) && DOING_AJAX
-		     && isset( $_REQUEST['action'] ) && 'wl_file_cache__flush_all' === $_REQUEST['action'] ) {
-			wp_send_json_success();
-		}
-
-	}
+        if ( defined( 'DOING_AJAX' ) && DOING_AJAX
+             && isset( $_REQUEST['action'] ) && 'wl_file_cache__flush_all' === $_REQUEST['action'] ) {
+            wp_send_json_success();
+        }
+
+    }
 
-	/**
-	 * Get the filename holding the cache contents for the specified `id`.
-	 *
-	 * @param int $id The cache `id`.
-	 *
-	 * @return string The filename.
-	 * @since 3.16.0
-	 *
-	 */
-	private function get_filename( $id ) {
+    /**
+     * Get the filename holding the cache contents for the specified `id`.
+     *
+     * @param int $id The cache `id`.
+     *
+     * @return string The filename.
+     * @since 3.16.0
+     *
+     */
+    private function get_filename( $id ) {
 
-		return $this->cache_dir . md5( $id ) . $this->file_extension;
-	}
+        return $this->cache_dir . md5( $id ) . $this->file_extension;
+    }
 
 }

Spacing +41 added lines, -41 removed lines

@@ -70,31 +70,31 @@ 
 	 * @since 3.16.0
 	 *
 	 */
-	public function __construct( $cache_dir, $file_extension = '.wlcache' ) {
+	public function __construct($cache_dir, $file_extension = '.wlcache') {
 
-		$this->log = Wordlift_Log_Service::get_logger( get_class() );
+		$this->log = Wordlift_Log_Service::get_logger(get_class());
 
 		// Set the cache directory using the base directory provided by the caller
 		// and appending a hash for the unique site id.
-		$this->cache_dir      = trailingslashit( $cache_dir ) . md5( get_site_url() ) . '/';
+		$this->cache_dir      = trailingslashit($cache_dir).md5(get_site_url()).'/';
 		$this->file_extension = $file_extension;
 
 		// Create the cache dir.
-		if ( ! file_exists( $this->cache_dir ) ) {
-			@mkdir( $this->cache_dir, 0755, true );
+		if ( ! file_exists($this->cache_dir)) {
+			@mkdir($this->cache_dir, 0755, true);
 		}
 
 		// Add ourselves to the list of instances.
 		self::$instances[] = $this;
 
 		// Initialize the singleton and the ajax method.
-		if ( ! isset( self::$instance ) ) {
+		if ( ! isset(self::$instance)) {
 			self::$instance = $this;
 
-			add_action( 'wp_ajax_wl_file_cache__flush_all', array( 'Wordlift_File_Cache_Service', 'flush_all' ) );
+			add_action('wp_ajax_wl_file_cache__flush_all', array('Wordlift_File_Cache_Service', 'flush_all'));
 		}
 
-		$this->log->debug( "File Cache service initialized on $this->cache_dir." );
+		$this->log->debug("File Cache service initialized on $this->cache_dir.");
 
 	}
 
@@ -107,20 +107,20 @@ 
 	 * @since 3.16.0
 	 *
 	 */
-	function get_cache( $id ) {
+	function get_cache($id) {
 
 		// Bail out if we don't have the cache.
-		if ( ! $this->has_cache( $id ) ) {
+		if ( ! $this->has_cache($id)) {
 			return false;
 		}
 
 		// Get the filename.
-		$filename = $this->get_filename( $id );
+		$filename = $this->get_filename($id);
 
-		$this->log->trace( "Trying to get cache contents for $id from $filename..." );
+		$this->log->trace("Trying to get cache contents for $id from $filename...");
 
 		// Try to decode the contents.
-		$contents = json_decode( file_get_contents( $filename ), true );
+		$contents = json_decode(file_get_contents($filename), true);
 
 		// Return false if decoding failed, otherwise the decoded contents.
 		return $contents ?: false;
@@ -135,25 +135,25 @@ 
 	 * @since 3.16.0
 	 *
 	 */
-	function has_cache( $id ) {
+	function has_cache($id) {
 
 		// Get the filename.
-		$filename = $this->get_filename( $id );
+		$filename = $this->get_filename($id);
 
 		// Bail out if the file doesn't exist.
-		return file_exists( $filename );
+		return file_exists($filename);
 	}
 
 	/**
 	 * @inheritdoc
 	 */
-	function set_cache( $id, $contents ) {
+	function set_cache($id, $contents) {
 
-		$filename = $this->get_filename( $id );
+		$filename = $this->get_filename($id);
 
-		$this->log->trace( "Writing cache contents for $id to $filename..." );
+		$this->log->trace("Writing cache contents for $id to $filename...");
 
-		@file_put_contents( $filename, wp_json_encode( $contents ) );
+		@file_put_contents($filename, wp_json_encode($contents));
 
 	}
 
@@ -165,14 +165,14 @@ 
 	 * @since 3.16.0
 	 *
 	 */
-	function delete_cache( $id ) {
+	function delete_cache($id) {
 
-		$filename = $this->get_filename( $id );
+		$filename = $this->get_filename($id);
 
-		$this->log->trace( "Deleting cache contents for $id, file $filename..." );
+		$this->log->trace("Deleting cache contents for $id, file $filename...");
 
-		if ( file_exists( $filename ) ) {
-			@unlink( $filename );
+		if (file_exists($filename)) {
+			@unlink($filename);
 		}
 
 	}
@@ -185,45 +185,45 @@ 
 	function flush() {
 
 		// Bail out if the cache dir isn't set.
-		if ( empty( $this->cache_dir ) || '/' === $this->cache_dir ) {
+		if (empty($this->cache_dir) || '/' === $this->cache_dir) {
 			return;
 		}
 
-		$this->log->trace( "Flushing cache contents from $this->cache_dir..." );
+		$this->log->trace("Flushing cache contents from $this->cache_dir...");
 
-		$handle = @opendir( $this->cache_dir );
+		$handle = @opendir($this->cache_dir);
 
 		// Bail out if the directory can't be opened.
-		if ( false === $handle ) {
+		if (false === $handle) {
 			return;
 		}
 
 		// Calculate the file extension length for matching file names.
-		$file_extension_length = strlen( $this->file_extension );
+		$file_extension_length = strlen($this->file_extension);
 
 		// Loop into the directory to delete files.
-		while ( false !== ( $entry = readdir( $handle ) ) ) {
-			if ( substr( $entry, - $file_extension_length ) === $this->file_extension
-			     && file_exists( $this->cache_dir . $entry ) ) {
-				$this->log->trace( "Deleting file {$this->cache_dir}{$entry}..." );
+		while (false !== ($entry = readdir($handle))) {
+			if (substr($entry, - $file_extension_length) === $this->file_extension
+			     && file_exists($this->cache_dir.$entry)) {
+				$this->log->trace("Deleting file {$this->cache_dir}{$entry}...");
 
-				@unlink( $this->cache_dir . $entry );
+				@unlink($this->cache_dir.$entry);
 			}
 		}
 
 		// Finally closed the directory.
-		closedir( $handle );
+		closedir($handle);
 
 	}
 
 	public static function flush_all() {
 
-		foreach ( self::$instances as $instance ) {
+		foreach (self::$instances as $instance) {
 			$instance->flush();
 		}
 
-		if ( defined( 'DOING_AJAX' ) && DOING_AJAX
-		     && isset( $_REQUEST['action'] ) && 'wl_file_cache__flush_all' === $_REQUEST['action'] ) {
+		if (defined('DOING_AJAX') && DOING_AJAX
+		     && isset($_REQUEST['action']) && 'wl_file_cache__flush_all' === $_REQUEST['action']) {
 			wp_send_json_success();
 		}
 
@@ -238,9 +238,9 @@ 
 	 * @since 3.16.0
 	 *
 	 */
-	private function get_filename( $id ) {
+	private function get_filename($id) {
 
-		return $this->cache_dir . md5( $id ) . $this->file_extension;
+		return $this->cache_dir.md5($id).$this->file_extension;
 	}
 
 }

src/includes/class-wordlift-post-excerpt-helper.php

Indentation +39 added lines, -39 removed lines

@@ -23,55 +23,55 @@
  */
 class Wordlift_Post_Excerpt_Helper {
 
-	/**
-	 * Get the text excerpt for the provided {@link WP_Post}.
-	 *
-	 * Since anyone can hook on the excerpt generation filters, and
-	 * amend it with non textual content, we play it self and generate
-	 * the excerpt ourselves, mimicking the way wordpress core does it.
-	 *
-	 * @since 3.10.0
-	 *
-	 * @param WP_Post $post The {@link WP_Post}.
-	 * @param int     $length The desired excerpt length.
-	 * @param string  $more The desired more string.
-	 *
-	 * @return string The excerpt.
-	 */
-	public static function get_text_excerpt( $post, $length = 55, $more = '...' ) {
+    /**
+     * Get the text excerpt for the provided {@link WP_Post}.
+     *
+     * Since anyone can hook on the excerpt generation filters, and
+     * amend it with non textual content, we play it self and generate
+     * the excerpt ourselves, mimicking the way wordpress core does it.
+     *
+     * @since 3.10.0
+     *
+     * @param WP_Post $post The {@link WP_Post}.
+     * @param int     $length The desired excerpt length.
+     * @param string  $more The desired more string.
+     *
+     * @return string The excerpt.
+     */
+    public static function get_text_excerpt( $post, $length = 55, $more = '...' ) {
 
-		/*
+        /*
 		 * Apply the `wl_post_content` filter, in case 3rd parties want to change the post content, e.g.
 		 * because the content is written elsewhere.
 		 *
 		 * @since 3.20.0
 		 */
-		$post_content = apply_filters( 'wl_post_content', $post->post_content, $post );
+        $post_content = apply_filters( 'wl_post_content', $post->post_content, $post );
 
-		// Filter shortcode content in post_content, before using it for trimming
-		$post_content = do_shortcode( $post_content );
+        // Filter shortcode content in post_content, before using it for trimming
+        $post_content = do_shortcode( $post_content );
 
-		// Get the excerpt and trim it. Use the `post_excerpt` if available.
-		$excerpt = wp_trim_words( ! empty( $post->post_excerpt ) ? $post->post_excerpt : $post_content, $length, $more );
+        // Get the excerpt and trim it. Use the `post_excerpt` if available.
+        $excerpt = wp_trim_words( ! empty( $post->post_excerpt ) ? $post->post_excerpt : $post_content, $length, $more );
 
-		// Remove shortcodes and decode html entities.
-		return html_entity_decode( self::strip_all_shortcodes( $excerpt ) );
-	}
+        // Remove shortcodes and decode html entities.
+        return html_entity_decode( self::strip_all_shortcodes( $excerpt ) );
+    }
 
-	/**
-	 * Remove all the shortcodes from the content. We're using our own function
-	 * because WordPress' own `strip_shortcodes` only takes into consideration
-	 * shortcodes for installed plugins/themes.
-	 *
-	 * @since 3.12.0
-	 *
-	 * @param string $content The content with shortcodes.
-	 *
-	 * @return string The content without shortcodes.
-	 */
-	private static function strip_all_shortcodes( $content ) {
+    /**
+     * Remove all the shortcodes from the content. We're using our own function
+     * because WordPress' own `strip_shortcodes` only takes into consideration
+     * shortcodes for installed plugins/themes.
+     *
+     * @since 3.12.0
+     *
+     * @param string $content The content with shortcodes.
+     *
+     * @return string The content without shortcodes.
+     */
+    private static function strip_all_shortcodes( $content ) {
 
-		return preg_replace( '/\[[^]]+\]/', '', $content );
-	}
+        return preg_replace( '/\[[^]]+\]/', '', $content );
+    }
 
 }

Spacing +7 added lines, -7 removed lines

@@ -38,7 +38,7 @@ 
 	 *
 	 * @return string The excerpt.
 	 */
-	public static function get_text_excerpt( $post, $length = 55, $more = '...' ) {
+	public static function get_text_excerpt($post, $length = 55, $more = '...') {
 
 		/*
 		 * Apply the `wl_post_content` filter, in case 3rd parties want to change the post content, e.g.
@@ -46,16 +46,16 @@ 
 		 *
 		 * @since 3.20.0
 		 */
-		$post_content = apply_filters( 'wl_post_content', $post->post_content, $post );
+		$post_content = apply_filters('wl_post_content', $post->post_content, $post);
 
 		// Filter shortcode content in post_content, before using it for trimming
-		$post_content = do_shortcode( $post_content );
+		$post_content = do_shortcode($post_content);
 
 		// Get the excerpt and trim it. Use the `post_excerpt` if available.
-		$excerpt = wp_trim_words( ! empty( $post->post_excerpt ) ? $post->post_excerpt : $post_content, $length, $more );
+		$excerpt = wp_trim_words( ! empty($post->post_excerpt) ? $post->post_excerpt : $post_content, $length, $more);
 
 		// Remove shortcodes and decode html entities.
-		return html_entity_decode( self::strip_all_shortcodes( $excerpt ) );
+		return html_entity_decode(self::strip_all_shortcodes($excerpt));
 	}
 
 	/**
@@ -69,9 +69,9 @@ 
 	 *
 	 * @return string The content without shortcodes.
 	 */
-	private static function strip_all_shortcodes( $content ) {
+	private static function strip_all_shortcodes($content) {
 
-		return preg_replace( '/\[[^]]+\]/', '', $content );
+		return preg_replace('/\[[^]]+\]/', '', $content);
 	}
 
 }

src/wordlift/autocomplete/class-autocomplete-service.php

Indentation +10 added lines, -10 removed lines

@@ -11,15 +11,15 @@
 
 interface Autocomplete_Service {
 
-	/**
-	 * Query the service for the specified data.
-	 *
-	 * @param string $query The query.
-	 * @param string $scope The scope.
-	 * @param string|string[] $excludes URLs to exclude.
-	 *
-	 * @return array An array of results.
-	 */
-	public function query( $query, $scope, $excludes );
+    /**
+     * Query the service for the specified data.
+     *
+     * @param string $query The query.
+     * @param string $scope The scope.
+     * @param string|string[] $excludes URLs to exclude.
+     *
+     * @return array An array of results.
+     */
+    public function query( $query, $scope, $excludes );
 
 }

Spacing +1 added lines, -1 removed lines

@@ -20,6 +20,6 @@
 	 *
 	 * @return array An array of results.
 	 */
-	public function query( $query, $scope, $excludes );
+	public function query($query, $scope, $excludes);
 
 }

src/admin/class-wordlift-admin-mappings-page.php

Indentation +24 added lines, -24 removed lines

@@ -16,37 +16,37 @@
  */
 class Wordlift_Admin_Mappings_Page extends Wordlift_Admin_Page {
 
-	/**
-	 * {@inheritdoc}
-	 */
-	function get_page_title() {
+    /**
+     * {@inheritdoc}
+     */
+    function get_page_title() {
 
-		return __( 'Schema.org Types', 'wordlift' );
-	}
+        return __( 'Schema.org Types', 'wordlift' );
+    }
 
-	/**
-	 * {@inheritdoc}
-	 */
-	function get_menu_title() {
+    /**
+     * {@inheritdoc}
+     */
+    function get_menu_title() {
 
-		return __( 'Schema.org Types', 'wordlift' );
-	}
+        return __( 'Schema.org Types', 'wordlift' );
+    }
 
-	/**
-	 * {@inheritdoc}
-	 */
-	function get_menu_slug() {
+    /**
+     * {@inheritdoc}
+     */
+    function get_menu_slug() {
 
-		return 'wl_mappings';
-	}
+        return 'wl_mappings';
+    }
 
-	/**
-	 * {@inheritdoc}
-	 */
+    /**
+     * {@inheritdoc}
+     */
 
-	function get_partial_name() {
+    function get_partial_name() {
 
-		return 'wordlift-admin-mappings-page.php';
-	}
+        return 'wordlift-admin-mappings-page.php';
+    }
 
 }

Spacing +2 added lines, -2 removed lines

@@ -21,7 +21,7 @@ 
 	 */
 	function get_page_title() {
 
-		return __( 'Schema.org Types', 'wordlift' );
+		return __('Schema.org Types', 'wordlift');
 	}
 
 	/**
@@ -29,7 +29,7 @@ 
 	 */
 	function get_menu_title() {
 
-		return __( 'Schema.org Types', 'wordlift' );
+		return __('Schema.org Types', 'wordlift');
 	}
 
 	/**

src/wordlift/mappings/class-mappings-dbo.php

Indentation +331 added lines, -331 removed lines

@@ -12,336 +12,336 @@
  */
 final class Mappings_DBO {
 
-	/**
-	 * The {@link wpdb} instance.
-	 *
-	 * @since  3.25.0
-	 * @access private
-	 * @var \wpdb $wpdb The {@link wpdb} instance.
-	 */
-	private $wpdb = null;
-
-	/**
-	 * Construct DBO object.
-	 */
-	public function __construct() {
-		global $wpdb;
-		$this->wpdb = $wpdb;
-	}
-
-	/**
-	 * Returns an array of mappings.
-	 *
-	 * Each mapping contains one or more rule groups which in turn contain one or more rules.
-	 *
-	 * @return array An array of mappings.
-	 */
-	public function get_mappings() {
-		$mapping_table_name = $this->wpdb->prefix . WL_MAPPING_TABLE_NAME;
-
-		return $this->wpdb->get_results(
-			"SELECT * FROM $mapping_table_name",
-			ARRAY_A
-		);
-	}
-
-	/**
-	 * Returns an array of active mappings.
-	 *
-	 * Each mapping contains one or more rule groups which in turn contain one or more rules.
-	 *
-	 * @return array An array of mappings.
-	 */
-	public function get_active_mappings() {
-		$mapping_table_name = $this->wpdb->prefix . WL_MAPPING_TABLE_NAME;
-
-		return $this->wpdb->get_results(
-			"SELECT * FROM $mapping_table_name WHERE mapping_status = 'active'",
-			ARRAY_A
-		);
-	}
-
-	/**
-	 * Returns a list of property rows
-	 *
-	 * @param int $mapping_id Primary key of mapping table.
-	 *
-	 * @return array List of property items belong to $mapping_id.
-	 */
-	public function get_properties( $mapping_id ) {
-		$property_table_name = $this->wpdb->prefix . WL_PROPERTY_TABLE_NAME;
-		$property_rows       = $this->wpdb->get_results(
-			$this->wpdb->prepare( "SELECT * FROM $property_table_name WHERE mapping_id=%d", $mapping_id ),
-			ARRAY_A
-		);
-
-		return $property_rows;
-	}
-
-	/**
-	 * Check if the row exists in the table
-	 *
-	 * @param string $table_name The table name you want to query, completely escaped value.
-	 * @param string $primary_key_name The primary key you want to query, should be escaped before passing.
-	 * @param int $primary_key_value The primary key value, no need to escape.
-	 *
-	 * @return bool Returns true if the row exists, false if it does not
-	 */
-	private function check_if_row_exists( $table_name, $primary_key_name, $primary_key_value ) {
-		$primary_key_value = (int) $primary_key_value;
-		$count             = (int) $this->wpdb->get_var(
-			$this->wpdb->prepare(
-				"SELECT COUNT($primary_key_name) from $table_name where $primary_key_name = %d",
-				$primary_key_value
-			)
-		);
-
-		return $count > 0;
-	}
-
-	/**
-	 * Insert new mapping item with title
-	 *
-	 * @param string $title Title of the mapping item.
-	 *
-	 * @return int Id of the inserted mapping item.
-	 */
-	public function insert_mapping_item( $title ) {
-		$mapping_table_name = $this->wpdb->prefix . WL_MAPPING_TABLE_NAME;
-		$this->wpdb->insert(
-			$mapping_table_name,
-			array( 'mapping_title' => $title )
-		);
-
-		return $this->wpdb->insert_id;
-	}
-
-	/**
-	 * Update mapping item with new title
-	 *
-	 * @param array $mapping_data Array of the mapping data.
-	 *
-	 * @return int Id of the inserted mapping item
-	 */
-	public function insert_or_update_mapping_item( $mapping_data ) {
-		$mapping_table_name = $this->wpdb->prefix . WL_MAPPING_TABLE_NAME;
-		$mapping_id         = array_key_exists( 'mapping_id', $mapping_data ) ? (int) $mapping_data['mapping_id'] : null;
-		if ( $this->check_if_row_exists( $mapping_table_name, 'mapping_id', $mapping_data['mapping_id'] ) ) {
-			$this->wpdb->update(
-				$mapping_table_name,
-				$mapping_data,
-				array( 'mapping_id' => $mapping_id )
-			);
-		} else {
-			$this->wpdb->insert(
-				$mapping_table_name,
-				$mapping_data
-			);
-			$mapping_id = (int) $this->wpdb->insert_id;
-		}
-
-		return $mapping_id;
-	}
-
-	/**
-	 * Updates rule item.
-	 *
-	 * @param array $rule_item_data The rule_item_data, should contain rule_id.
-	 *
-	 * @return int $rule_id The inserted rule id.
-	 */
-	public function insert_or_update_rule_item( $rule_item_data ) {
-		$rule_table_name = $this->wpdb->prefix . WL_RULE_TABLE_NAME;
-		$rule_id         = array_key_exists( 'rule_id', $rule_item_data ) ? $rule_item_data['rule_id'] : null;
-		if ( $this->check_if_row_exists( $rule_table_name, 'rule_id', $rule_id ) ) {
-			$this->wpdb->update( $rule_table_name, $rule_item_data, array( 'rule_id' => $rule_id ) );
-		} else {
-			$this->wpdb->insert( $rule_table_name, $rule_item_data );
-			$rule_id = $this->wpdb->insert_id;
-		}
-
-		return $rule_id;
-	}
-
-	/**
-	 * If a rule group exists doesn't do anything, but if rule group
-	 * didn't exist then it inserts a new entry.
-	 *
-	 * @param int $mapping_id Primary key for mapping table.
-	 *
-	 * @return int The inserted rule group id.
-	 */
-	public function insert_rule_group( $mapping_id ) {
-		$rule_group_table_name = $this->wpdb->prefix . WL_RULE_GROUP_TABLE_NAME;
-		$this->wpdb->insert(
-			$rule_group_table_name,
-			array(
-				'mapping_id' => $mapping_id,
-			)
-		);
-
-		return $this->wpdb->insert_id;
-	}
-
-	/**
-	 * Deletes a rule item by rule_id from rule and rule group table.
-	 *
-	 * @param int $rule_id Primary key for rule table.
-	 *
-	 * @return void
-	 */
-	public function delete_rule_item( $rule_id ) {
-		$rule_table_name = $this->wpdb->prefix . WL_RULE_TABLE_NAME;
-		$this->wpdb->delete( $rule_table_name, array( 'rule_id' => $rule_id ) );
-	}
-
-	/**
-	 * Deletes a rule group item by rule_group_id from rule group table.
-	 *
-	 * @param int $rule_group_id Primary key for rule table.
-	 *
-	 * @return void
-	 */
-	public function delete_rule_group_item( $rule_group_id ) {
-		$rule_group_table_name = $this->wpdb->prefix . WL_RULE_GROUP_TABLE_NAME;
-		$this->wpdb->delete( $rule_group_table_name, array( 'rule_group_id' => $rule_group_id ) );
-	}
-
-	/**
-	 * Deletes a mapping item by mapping_id
-	 *
-	 * @param int $mapping_id Primary key for mapping table.
-	 *
-	 * @return void
-	 */
-	public function delete_mapping_item( $mapping_id ) {
-		$mapping_table_name = $this->wpdb->prefix . WL_MAPPING_TABLE_NAME;
-		$this->wpdb->delete( $mapping_table_name, array( 'mapping_id' => $mapping_id ) );
-	}
-
-	/**
-	 * Gets a list of rule group items.
-	 *
-	 * @param int $mapping_id Primary key for mapping table.
-	 *
-	 * @return array Get list of rule group items.
-	 */
-	public function get_rule_group_list( $mapping_id ) {
-		$rule_group_table_name = $this->wpdb->prefix . WL_RULE_GROUP_TABLE_NAME;
-
-		return $this->wpdb->get_results(
-			$this->wpdb->prepare(
-				"SELECT rule_group_id FROM $rule_group_table_name WHERE mapping_id=%d",
-				$mapping_id
-			),
-			ARRAY_A
-		);
-	}
-
-	/**
-	 * Gets a list of rule group items.
-	 *
-	 * @param int $mapping_id Primary key for mapping table.
-	 *
-	 * @return array Get list of rule group items.
-	 */
-	public function get_rule_groups_by_mapping( $mapping_id ) {
-
-		$rule_group_table_name = $this->wpdb->prefix . WL_RULE_GROUP_TABLE_NAME;
-		$rule_group_rows       = $this->wpdb->get_results(
-			$this->wpdb->prepare(
-				"SELECT rule_group_id FROM $rule_group_table_name WHERE mapping_id=%d",
-				$mapping_id
-			),
-			ARRAY_A
-		);
-
-		// List of all rule group items.
-		$rule_groups = array();
-		foreach ( $rule_group_rows as $rule_group_row ) {
-			$rule_groups[] = array(
-				'rule_group_id' => $rule_group_row['rule_group_id'],
-				'rules'         => $this->get_rules_by_rule_group( $rule_group_row['rule_group_id'] ),
-			);
-		}
-
-		return $rule_groups;
-	}
-
-	/**
-	 * Gets a list of rule items belong to rule_group_id.
-	 *
-	 * @param int $rule_group_id Indicates which group the item belongs to.
-	 *
-	 * @return array Get list of rule items.
-	 */
-	public function get_rules_by_rule_group( $rule_group_id ) {
-		$rule_table_name = $this->wpdb->prefix . WL_RULE_TABLE_NAME;
-
-		return $this->wpdb->get_results(
-			$this->wpdb->prepare(
-				"SELECT * FROM $rule_table_name WHERE rule_group_id=%d",
-				$rule_group_id
-			),
-			ARRAY_A
-		);
-	}
-
-	/**
-	 * Insert/Update property item.
-	 *
-	 * @param array $property_data Property row from table/ui.
-	 *
-	 * @return int Inserted Property Id.
-	 */
-	public function insert_or_update_property( $property_data ) {
-		$property_table_name = $this->wpdb->prefix . WL_PROPERTY_TABLE_NAME;
-		$property_id         = array_key_exists( 'property_id', $property_data ) ? $property_data['property_id'] : null;
-
-		if ( $this->check_if_row_exists( $property_table_name, 'property_id', $property_id ) ) {
-			$this->wpdb->update(
-				$property_table_name,
-				$property_data,
-				array( 'property_id' => $property_id )
-			);
-		} else {
-			$this->wpdb->insert( $property_table_name, $property_data );
-			$property_id = $this->wpdb->insert_id;
-		}
-
-		return $property_id;
-	}
-
-	/**
-	 * Gets a single mapping item row.
-	 *
-	 * @param int $mapping_id Primary key of mapping table.
-	 *
-	 * @return array Returns single mapping table row..
-	 */
-	public function get_mapping_item_data( $mapping_id ) {
-		$mapping_table_name = $this->wpdb->prefix . WL_MAPPING_TABLE_NAME;
-
-		return $this->wpdb->get_row(
-			$this->wpdb->prepare(
-				"SELECT * FROM $mapping_table_name WHERE mapping_id=%d",
-				$mapping_id
-			),
-			ARRAY_A
-		);
-	}
-
-	/**
-	 * Delete property item.
-	 *
-	 * @param int $property_id Primary key for property table.
-	 *
-	 * @return int|false The number of rows updated, or false on error.
-	 */
-	public function delete_property( $property_id ) {
-		$property_table_name = $this->wpdb->prefix . WL_PROPERTY_TABLE_NAME;
-
-		return $this->wpdb->delete( $property_table_name, array( 'property_id' => $property_id ) );
-	}
+    /**
+     * The {@link wpdb} instance.
+     *
+     * @since  3.25.0
+     * @access private
+     * @var \wpdb $wpdb The {@link wpdb} instance.
+     */
+    private $wpdb = null;
+
+    /**
+     * Construct DBO object.
+     */
+    public function __construct() {
+        global $wpdb;
+        $this->wpdb = $wpdb;
+    }
+
+    /**
+     * Returns an array of mappings.
+     *
+     * Each mapping contains one or more rule groups which in turn contain one or more rules.
+     *
+     * @return array An array of mappings.
+     */
+    public function get_mappings() {
+        $mapping_table_name = $this->wpdb->prefix . WL_MAPPING_TABLE_NAME;
+
+        return $this->wpdb->get_results(
+            "SELECT * FROM $mapping_table_name",
+            ARRAY_A
+        );
+    }
+
+    /**
+     * Returns an array of active mappings.
+     *
+     * Each mapping contains one or more rule groups which in turn contain one or more rules.
+     *
+     * @return array An array of mappings.
+     */
+    public function get_active_mappings() {
+        $mapping_table_name = $this->wpdb->prefix . WL_MAPPING_TABLE_NAME;
+
+        return $this->wpdb->get_results(
+            "SELECT * FROM $mapping_table_name WHERE mapping_status = 'active'",
+            ARRAY_A
+        );
+    }
+
+    /**
+     * Returns a list of property rows
+     *
+     * @param int $mapping_id Primary key of mapping table.
+     *
+     * @return array List of property items belong to $mapping_id.
+     */
+    public function get_properties( $mapping_id ) {
+        $property_table_name = $this->wpdb->prefix . WL_PROPERTY_TABLE_NAME;
+        $property_rows       = $this->wpdb->get_results(
+            $this->wpdb->prepare( "SELECT * FROM $property_table_name WHERE mapping_id=%d", $mapping_id ),
+            ARRAY_A
+        );
+
+        return $property_rows;
+    }
+
+    /**
+     * Check if the row exists in the table
+     *
+     * @param string $table_name The table name you want to query, completely escaped value.
+     * @param string $primary_key_name The primary key you want to query, should be escaped before passing.
+     * @param int $primary_key_value The primary key value, no need to escape.
+     *
+     * @return bool Returns true if the row exists, false if it does not
+     */
+    private function check_if_row_exists( $table_name, $primary_key_name, $primary_key_value ) {
+        $primary_key_value = (int) $primary_key_value;
+        $count             = (int) $this->wpdb->get_var(
+            $this->wpdb->prepare(
+                "SELECT COUNT($primary_key_name) from $table_name where $primary_key_name = %d",
+                $primary_key_value
+            )
+        );
+
+        return $count > 0;
+    }
+
+    /**
+     * Insert new mapping item with title
+     *
+     * @param string $title Title of the mapping item.
+     *
+     * @return int Id of the inserted mapping item.
+     */
+    public function insert_mapping_item( $title ) {
+        $mapping_table_name = $this->wpdb->prefix . WL_MAPPING_TABLE_NAME;
+        $this->wpdb->insert(
+            $mapping_table_name,
+            array( 'mapping_title' => $title )
+        );
+
+        return $this->wpdb->insert_id;
+    }
+
+    /**
+     * Update mapping item with new title
+     *
+     * @param array $mapping_data Array of the mapping data.
+     *
+     * @return int Id of the inserted mapping item
+     */
+    public function insert_or_update_mapping_item( $mapping_data ) {
+        $mapping_table_name = $this->wpdb->prefix . WL_MAPPING_TABLE_NAME;
+        $mapping_id         = array_key_exists( 'mapping_id', $mapping_data ) ? (int) $mapping_data['mapping_id'] : null;
+        if ( $this->check_if_row_exists( $mapping_table_name, 'mapping_id', $mapping_data['mapping_id'] ) ) {
+            $this->wpdb->update(
+                $mapping_table_name,
+                $mapping_data,
+                array( 'mapping_id' => $mapping_id )
+            );
+        } else {
+            $this->wpdb->insert(
+                $mapping_table_name,
+                $mapping_data
+            );
+            $mapping_id = (int) $this->wpdb->insert_id;
+        }
+
+        return $mapping_id;
+    }
+
+    /**
+     * Updates rule item.
+     *
+     * @param array $rule_item_data The rule_item_data, should contain rule_id.
+     *
+     * @return int $rule_id The inserted rule id.
+     */
+    public function insert_or_update_rule_item( $rule_item_data ) {
+        $rule_table_name = $this->wpdb->prefix . WL_RULE_TABLE_NAME;
+        $rule_id         = array_key_exists( 'rule_id', $rule_item_data ) ? $rule_item_data['rule_id'] : null;
+        if ( $this->check_if_row_exists( $rule_table_name, 'rule_id', $rule_id ) ) {
+            $this->wpdb->update( $rule_table_name, $rule_item_data, array( 'rule_id' => $rule_id ) );
+        } else {
+            $this->wpdb->insert( $rule_table_name, $rule_item_data );
+            $rule_id = $this->wpdb->insert_id;
+        }
+
+        return $rule_id;
+    }
+
+    /**
+     * If a rule group exists doesn't do anything, but if rule group
+     * didn't exist then it inserts a new entry.
+     *
+     * @param int $mapping_id Primary key for mapping table.
+     *
+     * @return int The inserted rule group id.
+     */
+    public function insert_rule_group( $mapping_id ) {
+        $rule_group_table_name = $this->wpdb->prefix . WL_RULE_GROUP_TABLE_NAME;
+        $this->wpdb->insert(
+            $rule_group_table_name,
+            array(
+                'mapping_id' => $mapping_id,
+            )
+        );
+
+        return $this->wpdb->insert_id;
+    }
+
+    /**
+     * Deletes a rule item by rule_id from rule and rule group table.
+     *
+     * @param int $rule_id Primary key for rule table.
+     *
+     * @return void
+     */
+    public function delete_rule_item( $rule_id ) {
+        $rule_table_name = $this->wpdb->prefix . WL_RULE_TABLE_NAME;
+        $this->wpdb->delete( $rule_table_name, array( 'rule_id' => $rule_id ) );
+    }
+
+    /**
+     * Deletes a rule group item by rule_group_id from rule group table.
+     *
+     * @param int $rule_group_id Primary key for rule table.
+     *
+     * @return void
+     */
+    public function delete_rule_group_item( $rule_group_id ) {
+        $rule_group_table_name = $this->wpdb->prefix . WL_RULE_GROUP_TABLE_NAME;
+        $this->wpdb->delete( $rule_group_table_name, array( 'rule_group_id' => $rule_group_id ) );
+    }
+
+    /**
+     * Deletes a mapping item by mapping_id
+     *
+     * @param int $mapping_id Primary key for mapping table.
+     *
+     * @return void
+     */
+    public function delete_mapping_item( $mapping_id ) {
+        $mapping_table_name = $this->wpdb->prefix . WL_MAPPING_TABLE_NAME;
+        $this->wpdb->delete( $mapping_table_name, array( 'mapping_id' => $mapping_id ) );
+    }
+
+    /**
+     * Gets a list of rule group items.
+     *
+     * @param int $mapping_id Primary key for mapping table.
+     *
+     * @return array Get list of rule group items.
+     */
+    public function get_rule_group_list( $mapping_id ) {
+        $rule_group_table_name = $this->wpdb->prefix . WL_RULE_GROUP_TABLE_NAME;
+
+        return $this->wpdb->get_results(
+            $this->wpdb->prepare(
+                "SELECT rule_group_id FROM $rule_group_table_name WHERE mapping_id=%d",
+                $mapping_id
+            ),
+            ARRAY_A
+        );
+    }
+
+    /**
+     * Gets a list of rule group items.
+     *
+     * @param int $mapping_id Primary key for mapping table.
+     *
+     * @return array Get list of rule group items.
+     */
+    public function get_rule_groups_by_mapping( $mapping_id ) {
+
+        $rule_group_table_name = $this->wpdb->prefix . WL_RULE_GROUP_TABLE_NAME;
+        $rule_group_rows       = $this->wpdb->get_results(
+            $this->wpdb->prepare(
+                "SELECT rule_group_id FROM $rule_group_table_name WHERE mapping_id=%d",
+                $mapping_id
+            ),
+            ARRAY_A
+        );
+
+        // List of all rule group items.
+        $rule_groups = array();
+        foreach ( $rule_group_rows as $rule_group_row ) {
+            $rule_groups[] = array(
+                'rule_group_id' => $rule_group_row['rule_group_id'],
+                'rules'         => $this->get_rules_by_rule_group( $rule_group_row['rule_group_id'] ),
+            );
+        }
+
+        return $rule_groups;
+    }
+
+    /**
+     * Gets a list of rule items belong to rule_group_id.
+     *
+     * @param int $rule_group_id Indicates which group the item belongs to.
+     *
+     * @return array Get list of rule items.
+     */
+    public function get_rules_by_rule_group( $rule_group_id ) {
+        $rule_table_name = $this->wpdb->prefix . WL_RULE_TABLE_NAME;
+
+        return $this->wpdb->get_results(
+            $this->wpdb->prepare(
+                "SELECT * FROM $rule_table_name WHERE rule_group_id=%d",
+                $rule_group_id
+            ),
+            ARRAY_A
+        );
+    }
+
+    /**
+     * Insert/Update property item.
+     *
+     * @param array $property_data Property row from table/ui.
+     *
+     * @return int Inserted Property Id.
+     */
+    public function insert_or_update_property( $property_data ) {
+        $property_table_name = $this->wpdb->prefix . WL_PROPERTY_TABLE_NAME;
+        $property_id         = array_key_exists( 'property_id', $property_data ) ? $property_data['property_id'] : null;
+
+        if ( $this->check_if_row_exists( $property_table_name, 'property_id', $property_id ) ) {
+            $this->wpdb->update(
+                $property_table_name,
+                $property_data,
+                array( 'property_id' => $property_id )
+            );
+        } else {
+            $this->wpdb->insert( $property_table_name, $property_data );
+            $property_id = $this->wpdb->insert_id;
+        }
+
+        return $property_id;
+    }
+
+    /**
+     * Gets a single mapping item row.
+     *
+     * @param int $mapping_id Primary key of mapping table.
+     *
+     * @return array Returns single mapping table row..
+     */
+    public function get_mapping_item_data( $mapping_id ) {
+        $mapping_table_name = $this->wpdb->prefix . WL_MAPPING_TABLE_NAME;
+
+        return $this->wpdb->get_row(
+            $this->wpdb->prepare(
+                "SELECT * FROM $mapping_table_name WHERE mapping_id=%d",
+                $mapping_id
+            ),
+            ARRAY_A
+        );
+    }
+
+    /**
+     * Delete property item.
+     *
+     * @param int $property_id Primary key for property table.
+     *
+     * @return int|false The number of rows updated, or false on error.
+     */
+    public function delete_property( $property_id ) {
+        $property_table_name = $this->wpdb->prefix . WL_PROPERTY_TABLE_NAME;
+
+        return $this->wpdb->delete( $property_table_name, array( 'property_id' => $property_id ) );
+    }
 
 }

Spacing +50 added lines, -50 removed lines

@@ -37,7 +37,7 @@ 
 	 * @return array An array of mappings.
 	 */
 	public function get_mappings() {
-		$mapping_table_name = $this->wpdb->prefix . WL_MAPPING_TABLE_NAME;
+		$mapping_table_name = $this->wpdb->prefix.WL_MAPPING_TABLE_NAME;
 
 		return $this->wpdb->get_results(
 			"SELECT * FROM $mapping_table_name",
@@ -53,7 +53,7 @@ 
 	 * @return array An array of mappings.
 	 */
 	public function get_active_mappings() {
-		$mapping_table_name = $this->wpdb->prefix . WL_MAPPING_TABLE_NAME;
+		$mapping_table_name = $this->wpdb->prefix.WL_MAPPING_TABLE_NAME;
 
 		return $this->wpdb->get_results(
 			"SELECT * FROM $mapping_table_name WHERE mapping_status = 'active'",
@@ -68,10 +68,10 @@ 
 	 *
 	 * @return array List of property items belong to $mapping_id.
 	 */
-	public function get_properties( $mapping_id ) {
-		$property_table_name = $this->wpdb->prefix . WL_PROPERTY_TABLE_NAME;
+	public function get_properties($mapping_id) {
+		$property_table_name = $this->wpdb->prefix.WL_PROPERTY_TABLE_NAME;
 		$property_rows       = $this->wpdb->get_results(
-			$this->wpdb->prepare( "SELECT * FROM $property_table_name WHERE mapping_id=%d", $mapping_id ),
+			$this->wpdb->prepare("SELECT * FROM $property_table_name WHERE mapping_id=%d", $mapping_id),
 			ARRAY_A
 		);
 
@@ -87,7 +87,7 @@ 
 	 *
 	 * @return bool Returns true if the row exists, false if it does not
 	 */
-	private function check_if_row_exists( $table_name, $primary_key_name, $primary_key_value ) {
+	private function check_if_row_exists($table_name, $primary_key_name, $primary_key_value) {
 		$primary_key_value = (int) $primary_key_value;
 		$count             = (int) $this->wpdb->get_var(
 			$this->wpdb->prepare(
@@ -106,11 +106,11 @@ 
 	 *
 	 * @return int Id of the inserted mapping item.
 	 */
-	public function insert_mapping_item( $title ) {
-		$mapping_table_name = $this->wpdb->prefix . WL_MAPPING_TABLE_NAME;
+	public function insert_mapping_item($title) {
+		$mapping_table_name = $this->wpdb->prefix.WL_MAPPING_TABLE_NAME;
 		$this->wpdb->insert(
 			$mapping_table_name,
-			array( 'mapping_title' => $title )
+			array('mapping_title' => $title)
 		);
 
 		return $this->wpdb->insert_id;
@@ -123,14 +123,14 @@ 
 	 *
 	 * @return int Id of the inserted mapping item
 	 */
-	public function insert_or_update_mapping_item( $mapping_data ) {
-		$mapping_table_name = $this->wpdb->prefix . WL_MAPPING_TABLE_NAME;
-		$mapping_id         = array_key_exists( 'mapping_id', $mapping_data ) ? (int) $mapping_data['mapping_id'] : null;
-		if ( $this->check_if_row_exists( $mapping_table_name, 'mapping_id', $mapping_data['mapping_id'] ) ) {
+	public function insert_or_update_mapping_item($mapping_data) {
+		$mapping_table_name = $this->wpdb->prefix.WL_MAPPING_TABLE_NAME;
+		$mapping_id         = array_key_exists('mapping_id', $mapping_data) ? (int) $mapping_data['mapping_id'] : null;
+		if ($this->check_if_row_exists($mapping_table_name, 'mapping_id', $mapping_data['mapping_id'])) {
 			$this->wpdb->update(
 				$mapping_table_name,
 				$mapping_data,
-				array( 'mapping_id' => $mapping_id )
+				array('mapping_id' => $mapping_id)
 			);
 		} else {
 			$this->wpdb->insert(
@@ -150,13 +150,13 @@ 
 	 *
 	 * @return int $rule_id The inserted rule id.
 	 */
-	public function insert_or_update_rule_item( $rule_item_data ) {
-		$rule_table_name = $this->wpdb->prefix . WL_RULE_TABLE_NAME;
-		$rule_id         = array_key_exists( 'rule_id', $rule_item_data ) ? $rule_item_data['rule_id'] : null;
-		if ( $this->check_if_row_exists( $rule_table_name, 'rule_id', $rule_id ) ) {
-			$this->wpdb->update( $rule_table_name, $rule_item_data, array( 'rule_id' => $rule_id ) );
+	public function insert_or_update_rule_item($rule_item_data) {
+		$rule_table_name = $this->wpdb->prefix.WL_RULE_TABLE_NAME;
+		$rule_id         = array_key_exists('rule_id', $rule_item_data) ? $rule_item_data['rule_id'] : null;
+		if ($this->check_if_row_exists($rule_table_name, 'rule_id', $rule_id)) {
+			$this->wpdb->update($rule_table_name, $rule_item_data, array('rule_id' => $rule_id));
 		} else {
-			$this->wpdb->insert( $rule_table_name, $rule_item_data );
+			$this->wpdb->insert($rule_table_name, $rule_item_data);
 			$rule_id = $this->wpdb->insert_id;
 		}
 
@@ -171,8 +171,8 @@ 
 	 *
 	 * @return int The inserted rule group id.
 	 */
-	public function insert_rule_group( $mapping_id ) {
-		$rule_group_table_name = $this->wpdb->prefix . WL_RULE_GROUP_TABLE_NAME;
+	public function insert_rule_group($mapping_id) {
+		$rule_group_table_name = $this->wpdb->prefix.WL_RULE_GROUP_TABLE_NAME;
 		$this->wpdb->insert(
 			$rule_group_table_name,
 			array(
@@ -190,9 +190,9 @@ 
 	 *
 	 * @return void
 	 */
-	public function delete_rule_item( $rule_id ) {
-		$rule_table_name = $this->wpdb->prefix . WL_RULE_TABLE_NAME;
-		$this->wpdb->delete( $rule_table_name, array( 'rule_id' => $rule_id ) );
+	public function delete_rule_item($rule_id) {
+		$rule_table_name = $this->wpdb->prefix.WL_RULE_TABLE_NAME;
+		$this->wpdb->delete($rule_table_name, array('rule_id' => $rule_id));
 	}
 
 	/**
@@ -202,9 +202,9 @@ 
 	 *
 	 * @return void
 	 */
-	public function delete_rule_group_item( $rule_group_id ) {
-		$rule_group_table_name = $this->wpdb->prefix . WL_RULE_GROUP_TABLE_NAME;
-		$this->wpdb->delete( $rule_group_table_name, array( 'rule_group_id' => $rule_group_id ) );
+	public function delete_rule_group_item($rule_group_id) {
+		$rule_group_table_name = $this->wpdb->prefix.WL_RULE_GROUP_TABLE_NAME;
+		$this->wpdb->delete($rule_group_table_name, array('rule_group_id' => $rule_group_id));
 	}
 
 	/**
@@ -214,9 +214,9 @@ 
 	 *
 	 * @return void
 	 */
-	public function delete_mapping_item( $mapping_id ) {
-		$mapping_table_name = $this->wpdb->prefix . WL_MAPPING_TABLE_NAME;
-		$this->wpdb->delete( $mapping_table_name, array( 'mapping_id' => $mapping_id ) );
+	public function delete_mapping_item($mapping_id) {
+		$mapping_table_name = $this->wpdb->prefix.WL_MAPPING_TABLE_NAME;
+		$this->wpdb->delete($mapping_table_name, array('mapping_id' => $mapping_id));
 	}
 
 	/**
@@ -226,8 +226,8 @@ 
 	 *
 	 * @return array Get list of rule group items.
 	 */
-	public function get_rule_group_list( $mapping_id ) {
-		$rule_group_table_name = $this->wpdb->prefix . WL_RULE_GROUP_TABLE_NAME;
+	public function get_rule_group_list($mapping_id) {
+		$rule_group_table_name = $this->wpdb->prefix.WL_RULE_GROUP_TABLE_NAME;
 
 		return $this->wpdb->get_results(
 			$this->wpdb->prepare(
@@ -245,9 +245,9 @@ 
 	 *
 	 * @return array Get list of rule group items.
 	 */
-	public function get_rule_groups_by_mapping( $mapping_id ) {
+	public function get_rule_groups_by_mapping($mapping_id) {
 
-		$rule_group_table_name = $this->wpdb->prefix . WL_RULE_GROUP_TABLE_NAME;
+		$rule_group_table_name = $this->wpdb->prefix.WL_RULE_GROUP_TABLE_NAME;
 		$rule_group_rows       = $this->wpdb->get_results(
 			$this->wpdb->prepare(
 				"SELECT rule_group_id FROM $rule_group_table_name WHERE mapping_id=%d",
@@ -258,10 +258,10 @@ 
 
 		// List of all rule group items.
 		$rule_groups = array();
-		foreach ( $rule_group_rows as $rule_group_row ) {
+		foreach ($rule_group_rows as $rule_group_row) {
 			$rule_groups[] = array(
 				'rule_group_id' => $rule_group_row['rule_group_id'],
-				'rules'         => $this->get_rules_by_rule_group( $rule_group_row['rule_group_id'] ),
+				'rules'         => $this->get_rules_by_rule_group($rule_group_row['rule_group_id']),
 			);
 		}
 
@@ -275,8 +275,8 @@ 
 	 *
 	 * @return array Get list of rule items.
 	 */
-	public function get_rules_by_rule_group( $rule_group_id ) {
-		$rule_table_name = $this->wpdb->prefix . WL_RULE_TABLE_NAME;
+	public function get_rules_by_rule_group($rule_group_id) {
+		$rule_table_name = $this->wpdb->prefix.WL_RULE_TABLE_NAME;
 
 		return $this->wpdb->get_results(
 			$this->wpdb->prepare(
@@ -294,18 +294,18 @@ 
 	 *
 	 * @return int Inserted Property Id.
 	 */
-	public function insert_or_update_property( $property_data ) {
-		$property_table_name = $this->wpdb->prefix . WL_PROPERTY_TABLE_NAME;
-		$property_id         = array_key_exists( 'property_id', $property_data ) ? $property_data['property_id'] : null;
+	public function insert_or_update_property($property_data) {
+		$property_table_name = $this->wpdb->prefix.WL_PROPERTY_TABLE_NAME;
+		$property_id         = array_key_exists('property_id', $property_data) ? $property_data['property_id'] : null;
 
-		if ( $this->check_if_row_exists( $property_table_name, 'property_id', $property_id ) ) {
+		if ($this->check_if_row_exists($property_table_name, 'property_id', $property_id)) {
 			$this->wpdb->update(
 				$property_table_name,
 				$property_data,
-				array( 'property_id' => $property_id )
+				array('property_id' => $property_id)
 			);
 		} else {
-			$this->wpdb->insert( $property_table_name, $property_data );
+			$this->wpdb->insert($property_table_name, $property_data);
 			$property_id = $this->wpdb->insert_id;
 		}
 
@@ -319,8 +319,8 @@ 
 	 *
 	 * @return array Returns single mapping table row..
 	 */
-	public function get_mapping_item_data( $mapping_id ) {
-		$mapping_table_name = $this->wpdb->prefix . WL_MAPPING_TABLE_NAME;
+	public function get_mapping_item_data($mapping_id) {
+		$mapping_table_name = $this->wpdb->prefix.WL_MAPPING_TABLE_NAME;
 
 		return $this->wpdb->get_row(
 			$this->wpdb->prepare(
@@ -338,10 +338,10 @@ 
 	 *
 	 * @return int|false The number of rows updated, or false on error.
 	 */
-	public function delete_property( $property_id ) {
-		$property_table_name = $this->wpdb->prefix . WL_PROPERTY_TABLE_NAME;
+	public function delete_property($property_id) {
+		$property_table_name = $this->wpdb->prefix.WL_PROPERTY_TABLE_NAME;
 
-		return $this->wpdb->delete( $property_table_name, array( 'property_id' => $property_id ) );
+		return $this->wpdb->delete($property_table_name, array('property_id' => $property_id));
 	}
 
 }

src/wordlift/mappings/class-mappings-transform-function.php

Indentation +25 added lines, -25 removed lines

@@ -15,32 +15,32 @@
  */
 interface Mappings_Transform_Function {
 
-	/**
-	 * Returns unique name of the transform function.
-	 *
-	 * @return string $name Unique name of the transform function, it should not be repeated
-	 * for any other transform function.
-	 */
-	public function get_name();
+    /**
+     * Returns unique name of the transform function.
+     *
+     * @return string $name Unique name of the transform function, it should not be repeated
+     * for any other transform function.
+     */
+    public function get_name();
 
-	/**
-	 * Returns label of the transform function.
-	 *
-	 * @return string $label Label of the transform function to be used in UI, need not
-	 * be unique.
-	 */
-	public function get_label();
+    /**
+     * Returns label of the transform function.
+     *
+     * @return string $label Label of the transform function to be used in UI, need not
+     * be unique.
+     */
+    public function get_label();
 
-	/**
-	 * Transform data and map to the desired keys.
-	 *
-	 * @param array|string $data An Associative Array containing raw data or string.
-	 * @param array $jsonld The JSON-LD structure.
-	 * @param int[] $references An array of post IDs referenced by the JSON-LD structure.
-	 * @param int $post_id The post ID.
-	 *
-	 * @return array|string Return Mapped data.
-	 */
-	public function transform_data( $data, $jsonld, &$references, $post_id );
+    /**
+     * Transform data and map to the desired keys.
+     *
+     * @param array|string $data An Associative Array containing raw data or string.
+     * @param array $jsonld The JSON-LD structure.
+     * @param int[] $references An array of post IDs referenced by the JSON-LD structure.
+     * @param int $post_id The post ID.
+     *
+     * @return array|string Return Mapped data.
+     */
+    public function transform_data( $data, $jsonld, &$references, $post_id );
 
 }

Spacing +1 added lines, -1 removed lines

@@ -41,6 +41,6 @@
 	 *
 	 * @return array|string Return Mapped data.
 	 */
-	public function transform_data( $data, $jsonld, &$references, $post_id );
+	public function transform_data($data, $jsonld, &$references, $post_id);
 
 }