insideout10 / wordlift-plugin

src/includes/class-wordlift-property-service.php

Indentation +114 added lines, -114 removed lines

@@ -14,119 +14,119 @@
  */
 abstract class Wordlift_Property_Service {
 
-	// TODO: check that this is relative to the extending class.
-	protected static $instance;
-
-	public function __construct() {
-
-		static::$instance = $this;
-	}
-
-	/**
-	 * Get the field singleton.
-	 *
-	 * @since 3.6.0
-	 * @return \Wordlift_Schema_Url_Property_Service The singleton instance.
-	 */
-	public static function get_instance() {
-
-		return static::$instance;
-	}
-
-	/**
-	 * Get the value for the specified post/entity.
-	 *
-	 * @since 3.6.0
-	 *
-	 * @param int $post_id The post id.
-	 *
-	 * @return mixed
-	 */
-	public abstract function get( $post_id );
-
-	/**
-	 * Sanitize the provided value.
-	 *
-	 * @since 3.6.0
-	 *
-	 * @param mixed $value The value to sanitize.
-	 *
-	 * @return mixed|NULL The sanitized value or NULL avoid saving this value (see {@link WL_Metabox_Field}).
-	 */
-	public abstract function sanitize( $value );
-
-	/**
-	 * The RDF predicate for the property.
-	 *
-	 * @since 3.6.0
-	 * @return string The RDF predicate.
-	 */
-	public abstract function get_rdf_predicate();
-
-	/**
-	 * The RDF data type.
-	 *
-	 * @since 3.6.0
-	 * @return string The RDF data type.
-	 */
-	public abstract function get_rdf_data_type();
-
-	/**
-	 * The internal data type.
-	 *
-	 * @since 3.6.0
-	 * @return string The internal data type.
-	 */
-	public abstract function get_data_type();
-
-	/**
-	 * The cardinality.
-	 *
-	 * @since 3.6.0
-	 * @return mixed The cardinality.
-	 */
-	public abstract function get_cardinality();
-
-	/**
-	 * The metabox field class name.
-	 *
-	 * @since 3.6.0
-	 * @return string The metabox field class name.
-	 */
-	public abstract function get_metabox_class();
-
-	/**
-	 * The untranslated metabox field label.
-	 *
-	 * @since 3.6.0
-	 * @return string The untranslated metabox field label.
-	 */
-	public abstract function get_metabox_label();
-
-	/**
-	 * The definition of the property returned as a compatible array.
-	 *
-	 * @deprecated
-	 *
-	 * @since 3.6.0
-	 * @return array An array of property definitions.
-	 */
-	public function get_compat_definition() {
-
-		return array(
-			'type'        => $this->get_data_type(),
-			'predicate'   => $this->get_rdf_predicate(),
-			'export_type' => $this->get_rdf_data_type(),
-			'constraints' => array(
-				'cardinality' => $this->get_cardinality(),
-			),
-			// Use the standard metabox for these URI (the URI metabox creates local entities).
-			'metabox'     => array(
-				'class' => $this->get_metabox_class(),
-				'label' => $this->get_metabox_label(),
-			),
-			'sanitize'    => array( $this, 'sanitize' ),
-		);
-	}
+    // TODO: check that this is relative to the extending class.
+    protected static $instance;
+
+    public function __construct() {
+
+        static::$instance = $this;
+    }
+
+    /**
+     * Get the field singleton.
+     *
+     * @since 3.6.0
+     * @return \Wordlift_Schema_Url_Property_Service The singleton instance.
+     */
+    public static function get_instance() {
+
+        return static::$instance;
+    }
+
+    /**
+     * Get the value for the specified post/entity.
+     *
+     * @since 3.6.0
+     *
+     * @param int $post_id The post id.
+     *
+     * @return mixed
+     */
+    public abstract function get( $post_id );
+
+    /**
+     * Sanitize the provided value.
+     *
+     * @since 3.6.0
+     *
+     * @param mixed $value The value to sanitize.
+     *
+     * @return mixed|NULL The sanitized value or NULL avoid saving this value (see {@link WL_Metabox_Field}).
+     */
+    public abstract function sanitize( $value );
+
+    /**
+     * The RDF predicate for the property.
+     *
+     * @since 3.6.0
+     * @return string The RDF predicate.
+     */
+    public abstract function get_rdf_predicate();
+
+    /**
+     * The RDF data type.
+     *
+     * @since 3.6.0
+     * @return string The RDF data type.
+     */
+    public abstract function get_rdf_data_type();
+
+    /**
+     * The internal data type.
+     *
+     * @since 3.6.0
+     * @return string The internal data type.
+     */
+    public abstract function get_data_type();
+
+    /**
+     * The cardinality.
+     *
+     * @since 3.6.0
+     * @return mixed The cardinality.
+     */
+    public abstract function get_cardinality();
+
+    /**
+     * The metabox field class name.
+     *
+     * @since 3.6.0
+     * @return string The metabox field class name.
+     */
+    public abstract function get_metabox_class();
+
+    /**
+     * The untranslated metabox field label.
+     *
+     * @since 3.6.0
+     * @return string The untranslated metabox field label.
+     */
+    public abstract function get_metabox_label();
+
+    /**
+     * The definition of the property returned as a compatible array.
+     *
+     * @deprecated
+     *
+     * @since 3.6.0
+     * @return array An array of property definitions.
+     */
+    public function get_compat_definition() {
+
+        return array(
+            'type'        => $this->get_data_type(),
+            'predicate'   => $this->get_rdf_predicate(),
+            'export_type' => $this->get_rdf_data_type(),
+            'constraints' => array(
+                'cardinality' => $this->get_cardinality(),
+            ),
+            // Use the standard metabox for these URI (the URI metabox creates local entities).
+            'metabox'     => array(
+                'class' => $this->get_metabox_class(),
+                'label' => $this->get_metabox_label(),
+            ),
+            'sanitize'    => array( $this, 'sanitize' ),
+        );
+    }
 
 }

Spacing +3 added lines, -3 removed lines

@@ -42,7 +42,7 @@ 
 	 *
 	 * @return mixed
 	 */
-	public abstract function get( $post_id );
+	public abstract function get($post_id);
 
 	/**
 	 * Sanitize the provided value.
@@ -53,7 +53,7 @@ 
 	 *
 	 * @return mixed|NULL The sanitized value or NULL avoid saving this value (see {@link WL_Metabox_Field}).
 	 */
-	public abstract function sanitize( $value );
+	public abstract function sanitize($value);
 
 	/**
 	 * The RDF predicate for the property.
@@ -125,7 +125,7 @@ 
 				'class' => $this->get_metabox_class(),
 				'label' => $this->get_metabox_label(),
 			),
-			'sanitize'    => array( $this, 'sanitize' ),
+			'sanitize'    => array($this, 'sanitize'),
 		);
 	}
 

src/admin/class-wordlift-admin-batch-analysis-page.php

Indentation +69 added lines, -69 removed lines

@@ -18,74 +18,74 @@
  */
 class Wordlift_Batch_Analysis_Page extends Wordlift_Admin_Page {
 
-	/**
-	 * The {@link Wordlift_Batch_analysis_Service} instance.
-	 *
-	 * @since 3.14.0
-	 *
-	 * @var \Wordlift_Batch_Analysis_Service $batch_analysis_service The {@link Wordlift_Batch_analysis_Service} instance.
-	 */
-	public $batch_analysis_service;
-
-	/**
-	 * The {@link Wordlift_Batch_Analysis_page} instance.
-	 *
-	 * @since 3.14.0
-	 *
-	 * @param \Wordlift_Batch_Analysis_Service $batch_analysis_service The {@link Wordlift_Batch_analysis_Service} instance.
-	 */
-	public function __construct( $batch_analysis_service ) {
-
-		$this->batch_analysis_service = $batch_analysis_service;
-
-	}
-
-	/**
-	 * @inheritdoc
-	 */
-	function get_parent_slug() {
-
-		return 'wl_admin_menu';
-	}
-
-	/**
-	 * @inheritdoc
-	 */
-	function get_capability() {
-
-		return 'manage_options';
-	}
-
-	/**
-	 * @inheritdoc
-	 */
-	function get_page_title() {
-
-		return __( 'Batch Analysis', 'wordlift' );
-	}
-
-	/**
-	 * @inheritdoc
-	 */
-	function get_menu_title() {
-
-		return __( 'Batch Analysis', 'wordlift' );
-	}
-
-	/**
-	 * @inheritdoc
-	 */
-	function get_menu_slug() {
-
-		return 'wl_batch_analysis_menu';
-	}
-
-	/**
-	 * @inheritdoc
-	 */
-	function get_partial_name() {
-
-		return 'wordlift-admin-batch-analysis-page.php';
-	}
+    /**
+     * The {@link Wordlift_Batch_analysis_Service} instance.
+     *
+     * @since 3.14.0
+     *
+     * @var \Wordlift_Batch_Analysis_Service $batch_analysis_service The {@link Wordlift_Batch_analysis_Service} instance.
+     */
+    public $batch_analysis_service;
+
+    /**
+     * The {@link Wordlift_Batch_Analysis_page} instance.
+     *
+     * @since 3.14.0
+     *
+     * @param \Wordlift_Batch_Analysis_Service $batch_analysis_service The {@link Wordlift_Batch_analysis_Service} instance.
+     */
+    public function __construct( $batch_analysis_service ) {
+
+        $this->batch_analysis_service = $batch_analysis_service;
+
+    }
+
+    /**
+     * @inheritdoc
+     */
+    function get_parent_slug() {
+
+        return 'wl_admin_menu';
+    }
+
+    /**
+     * @inheritdoc
+     */
+    function get_capability() {
+
+        return 'manage_options';
+    }
+
+    /**
+     * @inheritdoc
+     */
+    function get_page_title() {
+
+        return __( 'Batch Analysis', 'wordlift' );
+    }
+
+    /**
+     * @inheritdoc
+     */
+    function get_menu_title() {
+
+        return __( 'Batch Analysis', 'wordlift' );
+    }
+
+    /**
+     * @inheritdoc
+     */
+    function get_menu_slug() {
+
+        return 'wl_batch_analysis_menu';
+    }
+
+    /**
+     * @inheritdoc
+     */
+    function get_partial_name() {
+
+        return 'wordlift-admin-batch-analysis-page.php';
+    }
 
 }

Spacing +3 added lines, -3 removed lines

@@ -34,7 +34,7 @@ 
 	 *
 	 * @param \Wordlift_Batch_Analysis_Service $batch_analysis_service The {@link Wordlift_Batch_analysis_Service} instance.
 	 */
-	public function __construct( $batch_analysis_service ) {
+	public function __construct($batch_analysis_service) {
 
 		$this->batch_analysis_service = $batch_analysis_service;
 
@@ -61,7 +61,7 @@ 
 	 */
 	function get_page_title() {
 
-		return __( 'Batch Analysis', 'wordlift' );
+		return __('Batch Analysis', 'wordlift');
 	}
 
 	/**
@@ -69,7 +69,7 @@ 
 	 */
 	function get_menu_title() {
 
-		return __( 'Batch Analysis', 'wordlift' );
+		return __('Batch Analysis', 'wordlift');
 	}
 
 	/**

src/modules/configuration/wordlift_configuration_settings.php

Indentation +28 added lines, -28 removed lines

@@ -11,7 +11,7 @@ 
  */
 function wl_configuration_get_key() {
 
-	return Wordlift_Configuration_Service::get_instance()->get_key();
+    return Wordlift_Configuration_Service::get_instance()->get_key();
 }
 
 /**
@@ -26,7 +26,7 @@ 
  */
 function wl_configuration_get_redlink_dataset_uri() {
 
-	return Wordlift_Configuration_Service::get_instance()->get_dataset_uri();
+    return Wordlift_Configuration_Service::get_instance()->get_dataset_uri();
 }
 
 /**
@@ -41,16 +41,16 @@ 
  */
 function wl_configuration_get_analyzer_url() {
 
-	// If the WordLift Key is set, we use WordLift.
-	$key = Wordlift_Configuration_Service::get_instance()->get_key();
+    // If the WordLift Key is set, we use WordLift.
+    $key = Wordlift_Configuration_Service::get_instance()->get_key();
 
-	// Return a NULL URL if the key isn't set.
-	if ( empty( $key ) ) {
-		return null;
-	}
+    // Return a NULL URL if the key isn't set.
+    if ( empty( $key ) ) {
+        return null;
+    }
 
-	return WL_CONFIG_WORDLIFT_API_URL_DEFAULT_VALUE . "analyses?key=$key"
-	       . ( defined( 'WL_EXCLUDE_IMAGES_REGEX' ) ? '&exclimage=' . urlencode( WL_EXCLUDE_IMAGES_REGEX ) : '' );
+    return WL_CONFIG_WORDLIFT_API_URL_DEFAULT_VALUE . "analyses?key=$key"
+            . ( defined( 'WL_EXCLUDE_IMAGES_REGEX' ) ? '&exclimage=' . urlencode( WL_EXCLUDE_IMAGES_REGEX ) : '' );
 
 }
 
@@ -63,14 +63,14 @@ 
  */
 function wl_configuration_get_query_select_url() {
 
-	// If the WordLift Key is set, we use WordLift.
-	$key = Wordlift_Configuration_Service::get_instance()->get_key();
+    // If the WordLift Key is set, we use WordLift.
+    $key = Wordlift_Configuration_Service::get_instance()->get_key();
 
-	if ( empty( $key ) ) {
-		return null;
-	}
+    if ( empty( $key ) ) {
+        return null;
+    }
 
-	return WL_CONFIG_WORDLIFT_API_URL_DEFAULT_VALUE . "datasets/key=$key/queries?q=";
+    return WL_CONFIG_WORDLIFT_API_URL_DEFAULT_VALUE . "datasets/key=$key/queries?q=";
 }
 
 /**
@@ -82,13 +82,13 @@ 
  */
 function wl_configuration_get_query_update_url() {
 
-	// If the WordLift Key is set, we use WordLift.
-	$key = Wordlift_Configuration_Service::get_instance()->get_key();
-	if ( empty( $key ) ) {
-		return null;
-	}
+    // If the WordLift Key is set, we use WordLift.
+    $key = Wordlift_Configuration_Service::get_instance()->get_key();
+    if ( empty( $key ) ) {
+        return null;
+    }
 
-	return WL_CONFIG_WORDLIFT_API_URL_DEFAULT_VALUE . "datasets/key=$key/queries";
+    return WL_CONFIG_WORDLIFT_API_URL_DEFAULT_VALUE . "datasets/key=$key/queries";
 }
 
 
@@ -101,11 +101,11 @@ 
  */
 function wl_configuration_get_dataset_index_url() {
 
-	// If the WordLift Key is set, we use WordLift.
-	$key = Wordlift_Configuration_Service::get_instance()->get_key();
-	if ( empty( $key ) ) {
-		return null;
-	}
+    // If the WordLift Key is set, we use WordLift.
+    $key = Wordlift_Configuration_Service::get_instance()->get_key();
+    if ( empty( $key ) ) {
+        return null;
+    }
 
-	return WL_CONFIG_WORDLIFT_API_URL_DEFAULT_VALUE . "datasets/key=$key/index";
+    return WL_CONFIG_WORDLIFT_API_URL_DEFAULT_VALUE . "datasets/key=$key/index";
 }

Spacing +9 added lines, -9 removed lines

@@ -45,12 +45,12 @@ 
 	$key = Wordlift_Configuration_Service::get_instance()->get_key();
 
 	// Return a NULL URL if the key isn't set.
-	if ( empty( $key ) ) {
+	if (empty($key)) {
 		return null;
 	}
 
-	return WL_CONFIG_WORDLIFT_API_URL_DEFAULT_VALUE . "analyses?key=$key"
-	       . ( defined( 'WL_EXCLUDE_IMAGES_REGEX' ) ? '&exclimage=' . urlencode( WL_EXCLUDE_IMAGES_REGEX ) : '' );
+	return WL_CONFIG_WORDLIFT_API_URL_DEFAULT_VALUE."analyses?key=$key"
+	       . (defined('WL_EXCLUDE_IMAGES_REGEX') ? '&exclimage='.urlencode(WL_EXCLUDE_IMAGES_REGEX) : '');
 
 }
 
@@ -66,11 +66,11 @@ 
 	// If the WordLift Key is set, we use WordLift.
 	$key = Wordlift_Configuration_Service::get_instance()->get_key();
 
-	if ( empty( $key ) ) {
+	if (empty($key)) {
 		return null;
 	}
 
-	return WL_CONFIG_WORDLIFT_API_URL_DEFAULT_VALUE . "datasets/key=$key/queries?q=";
+	return WL_CONFIG_WORDLIFT_API_URL_DEFAULT_VALUE."datasets/key=$key/queries?q=";
 }
 
 /**
@@ -84,11 +84,11 @@ 
 
 	// If the WordLift Key is set, we use WordLift.
 	$key = Wordlift_Configuration_Service::get_instance()->get_key();
-	if ( empty( $key ) ) {
+	if (empty($key)) {
 		return null;
 	}
 
-	return WL_CONFIG_WORDLIFT_API_URL_DEFAULT_VALUE . "datasets/key=$key/queries";
+	return WL_CONFIG_WORDLIFT_API_URL_DEFAULT_VALUE."datasets/key=$key/queries";
 }
 
 
@@ -103,9 +103,9 @@ 
 
 	// If the WordLift Key is set, we use WordLift.
 	$key = Wordlift_Configuration_Service::get_instance()->get_key();
-	if ( empty( $key ) ) {
+	if (empty($key)) {
 		return null;
 	}
 
-	return WL_CONFIG_WORDLIFT_API_URL_DEFAULT_VALUE . "datasets/key=$key/index";
+	return WL_CONFIG_WORDLIFT_API_URL_DEFAULT_VALUE."datasets/key=$key/index";
 }

src/admin/WL_Metabox/class-wordlift-metabox-field-duration.php

Indentation +36 added lines, -36 removed lines

@@ -18,41 +18,41 @@
  */
 class Wordlift_Metabox_Field_Duration extends WL_Metabox_Field_date {
 
-	/**
-	 * {@inheritdoc}
-	 */
-	public function __construct( $args ) {
-		parent::__construct( $args );
-
-		$this->date_format = 'H:i';
-		$this->timepicker  = true;
-		$this->no_calendar = true;
-
-	}
-
-	/**
-	 * Sanitize a single value. Called from $this->sanitize_data. Default sanitization excludes empty values.
-	 * make sure the value is either empty, an integer representing valid number of minutes
-	 * or an HH:MM time format.
-	 *
-	 * @param mixed $value The value being sanitized.
-	 *
-	 * @return mixed Returns sanitized value, or null.
-	 */
-	public function sanitize_data_filter( $value ) {
-
-		if ( ! is_null( $value ) && '' !== $value ) {         // do not use 'empty()' -> https://www.virendrachandak.com/techtalk/php-isset-vs-empty-vs-is_null/ .
-			preg_match( '#((([01]?[0-9]{1}|2[0-3]{1}):)?[0-5]{1})?[0-9]{1}#',
-				trim( $value ),
-				$matches
-			);
-
-			if ( count( $matches ) > 0 ) {
-				return $matches[0];
-			}
-		}
-
-		return null;
-	}
+    /**
+     * {@inheritdoc}
+     */
+    public function __construct( $args ) {
+        parent::__construct( $args );
+
+        $this->date_format = 'H:i';
+        $this->timepicker  = true;
+        $this->no_calendar = true;
+
+    }
+
+    /**
+     * Sanitize a single value. Called from $this->sanitize_data. Default sanitization excludes empty values.
+     * make sure the value is either empty, an integer representing valid number of minutes
+     * or an HH:MM time format.
+     *
+     * @param mixed $value The value being sanitized.
+     *
+     * @return mixed Returns sanitized value, or null.
+     */
+    public function sanitize_data_filter( $value ) {
+
+        if ( ! is_null( $value ) && '' !== $value ) {         // do not use 'empty()' -> https://www.virendrachandak.com/techtalk/php-isset-vs-empty-vs-is_null/ .
+            preg_match( '#((([01]?[0-9]{1}|2[0-3]{1}):)?[0-5]{1})?[0-9]{1}#',
+                trim( $value ),
+                $matches
+            );
+
+            if ( count( $matches ) > 0 ) {
+                return $matches[0];
+            }
+        }
+
+        return null;
+    }
 
 }

Spacing +7 added lines, -7 removed lines

@@ -21,8 +21,8 @@ 
 	/**
 	 * {@inheritdoc}
 	 */
-	public function __construct( $args ) {
-		parent::__construct( $args );
+	public function __construct($args) {
+		parent::__construct($args);
 
 		$this->date_format = 'H:i';
 		$this->timepicker  = true;
@@ -39,15 +39,15 @@ 
 	 *
 	 * @return mixed Returns sanitized value, or null.
 	 */
-	public function sanitize_data_filter( $value ) {
+	public function sanitize_data_filter($value) {
 
-		if ( ! is_null( $value ) && '' !== $value ) {         // do not use 'empty()' -> https://www.virendrachandak.com/techtalk/php-isset-vs-empty-vs-is_null/ .
-			preg_match( '#((([01]?[0-9]{1}|2[0-3]{1}):)?[0-5]{1})?[0-9]{1}#',
-				trim( $value ),
+		if ( ! is_null($value) && '' !== $value) {         // do not use 'empty()' -> https://www.virendrachandak.com/techtalk/php-isset-vs-empty-vs-is_null/ .
+			preg_match('#((([01]?[0-9]{1}|2[0-3]{1}):)?[0-5]{1})?[0-9]{1}#',
+				trim($value),
 				$matches
 			);
 
-			if ( count( $matches ) > 0 ) {
+			if (count($matches) > 0) {
 				return $matches[0];
 			}
 		}

src/includes/class-wordlift-key-validation-service.php

Indentation +72 added lines, -72 removed lines

@@ -17,77 +17,77 @@
  */
 class Wordlift_Key_Validation_Service {
 
-	/**
-	 * A {@link Wordlift_Log_Service} instance.
-	 *
-	 * @since  3.14.0
-	 * @access private
-	 * @var \Wordlift_Log_Service $log A {@link Wordlift_Log_Service} instance.
-	 */
-	private $log;
-
-	/**
-	 * The {@link Wordlift_Configuration_Service} instance.
-	 *
-	 * @since  3.14.0
-	 * @access private
-	 * @var \Wordlift_Configuration_Service $configuration_service The {@link Wordlift_Configuration_Service} instance.
-	 */
-	private $configuration_service;
-
-	/**
-	 * Create a {@link Wordlift_Key_Validation_Service} instance.
-	 *
-	 * @since 3.14.0
-	 *
-	 * @param \Wordlift_Configuration_Service $configuration_service The {@link Wordlift_Configuration_Service} instance.
-	 */
-	public function __construct( $configuration_service ) {
-
-		$this->log = Wordlift_Log_Service::get_logger( 'Wordlift_Key_Validation_Service' );
-
-		$this->configuration_service = $configuration_service;
-
-	}
-
-	/**
-	 * Validate the provided key.
-	 *
-	 * @since 3.9.0
-	 *
-	 * @param string $key WordLift's key to validate.
-	 *
-	 * @return bool True if the key is valid, otherwise false.
-	 */
-	public function is_valid( $key ) {
-
-		$this->log->debug( 'Validating key...' );
-
-		// Request the dataset URI as a way to validate the key
-		$response = wp_remote_get( $this->configuration_service->get_accounts_by_key_dataset_uri( $key ), unserialize( WL_REDLINK_API_HTTP_OPTIONS ) );
-
-		// If the response is valid, the key is valid.
-		return ! is_wp_error( $response ) && 200 === (int) $response['response']['code'];
-	}
-
-	/**
-	 * This function is hooked to the `wl_validate_key` AJAX call.
-	 *
-	 * @since 3.9.0
-	 */
-	public function validate_key() {
-
-		// Ensure we don't have garbage before us.
-		ob_clean();
-
-		// Check if we have a key.
-		if ( ! isset( $_POST['key'] ) ) {
-			wp_send_json_error( 'The key parameter is required.' );
-		}
-
-		// Set a response with valid set to true or false according to the key validity.
-		wp_send_json_success( array( 'valid' => $this->is_valid( $_POST['key'] ) ) );
-
-	}
+    /**
+     * A {@link Wordlift_Log_Service} instance.
+     *
+     * @since  3.14.0
+     * @access private
+     * @var \Wordlift_Log_Service $log A {@link Wordlift_Log_Service} instance.
+     */
+    private $log;
+
+    /**
+     * The {@link Wordlift_Configuration_Service} instance.
+     *
+     * @since  3.14.0
+     * @access private
+     * @var \Wordlift_Configuration_Service $configuration_service The {@link Wordlift_Configuration_Service} instance.
+     */
+    private $configuration_service;
+
+    /**
+     * Create a {@link Wordlift_Key_Validation_Service} instance.
+     *
+     * @since 3.14.0
+     *
+     * @param \Wordlift_Configuration_Service $configuration_service The {@link Wordlift_Configuration_Service} instance.
+     */
+    public function __construct( $configuration_service ) {
+
+        $this->log = Wordlift_Log_Service::get_logger( 'Wordlift_Key_Validation_Service' );
+
+        $this->configuration_service = $configuration_service;
+
+    }
+
+    /**
+     * Validate the provided key.
+     *
+     * @since 3.9.0
+     *
+     * @param string $key WordLift's key to validate.
+     *
+     * @return bool True if the key is valid, otherwise false.
+     */
+    public function is_valid( $key ) {
+
+        $this->log->debug( 'Validating key...' );
+
+        // Request the dataset URI as a way to validate the key
+        $response = wp_remote_get( $this->configuration_service->get_accounts_by_key_dataset_uri( $key ), unserialize( WL_REDLINK_API_HTTP_OPTIONS ) );
+
+        // If the response is valid, the key is valid.
+        return ! is_wp_error( $response ) && 200 === (int) $response['response']['code'];
+    }
+
+    /**
+     * This function is hooked to the `wl_validate_key` AJAX call.
+     *
+     * @since 3.9.0
+     */
+    public function validate_key() {
+
+        // Ensure we don't have garbage before us.
+        ob_clean();
+
+        // Check if we have a key.
+        if ( ! isset( $_POST['key'] ) ) {
+            wp_send_json_error( 'The key parameter is required.' );
+        }
+
+        // Set a response with valid set to true or false according to the key validity.
+        wp_send_json_success( array( 'valid' => $this->is_valid( $_POST['key'] ) ) );
+
+    }
 
 }

Spacing +9 added lines, -9 removed lines

@@ -42,9 +42,9 @@ 
 	 *
 	 * @param \Wordlift_Configuration_Service $configuration_service The {@link Wordlift_Configuration_Service} instance.
 	 */
-	public function __construct( $configuration_service ) {
+	public function __construct($configuration_service) {
 
-		$this->log = Wordlift_Log_Service::get_logger( 'Wordlift_Key_Validation_Service' );
+		$this->log = Wordlift_Log_Service::get_logger('Wordlift_Key_Validation_Service');
 
 		$this->configuration_service = $configuration_service;
 
@@ -59,15 +59,15 @@ 
 	 *
 	 * @return bool True if the key is valid, otherwise false.
 	 */
-	public function is_valid( $key ) {
+	public function is_valid($key) {
 
-		$this->log->debug( 'Validating key...' );
+		$this->log->debug('Validating key...');
 
 		// Request the dataset URI as a way to validate the key
-		$response = wp_remote_get( $this->configuration_service->get_accounts_by_key_dataset_uri( $key ), unserialize( WL_REDLINK_API_HTTP_OPTIONS ) );
+		$response = wp_remote_get($this->configuration_service->get_accounts_by_key_dataset_uri($key), unserialize(WL_REDLINK_API_HTTP_OPTIONS));
 
 		// If the response is valid, the key is valid.
-		return ! is_wp_error( $response ) && 200 === (int) $response['response']['code'];
+		return ! is_wp_error($response) && 200 === (int) $response['response']['code'];
 	}
 
 	/**
@@ -81,12 +81,12 @@ 
 		ob_clean();
 
 		// Check if we have a key.
-		if ( ! isset( $_POST['key'] ) ) {
-			wp_send_json_error( 'The key parameter is required.' );
+		if ( ! isset($_POST['key'])) {
+			wp_send_json_error('The key parameter is required.');
 		}
 
 		// Set a response with valid set to true or false according to the key validity.
-		wp_send_json_success( array( 'valid' => $this->is_valid( $_POST['key'] ) ) );
+		wp_send_json_success(array('valid' => $this->is_valid($_POST['key'])));
 
 	}
 

src/includes/properties/class-wordlift-duration-property-service.php

Indentation +13 added lines, -13 removed lines

@@ -19,25 +19,25 @@
  */
 class Wordlift_Duration_Property_Service extends Wordlift_Simple_Property_Service {
 
-	/**
-	 * {@inheritdoc}
-	 */
-	function get( $post_id, $meta_key ) {
+    /**
+     * {@inheritdoc}
+     */
+    function get( $post_id, $meta_key ) {
 
-		// Get the values and filter out the empty ones (or the ones with 00:00).
-		$values = array_filter( parent::get( $post_id, $meta_key ), function ( $item ) {
-			return ! empty( $item ) && '00:00' !== $item;
-		} );
+        // Get the values and filter out the empty ones (or the ones with 00:00).
+        $values = array_filter( parent::get( $post_id, $meta_key ), function ( $item ) {
+            return ! empty( $item ) && '00:00' !== $item;
+        } );
 
-		/*
+        /*
 		 * Map the value in the meta
 		 * The UI for the meta date enable two forms, a number of minutes
 		 * or an h:mm format.
 		 * Both needs to be adjusted to the iso format.
 		 */
-		return array_map( function ( $value ) {
-			return 'PT' . str_replace( ':', 'H', $value ) . 'M';
-		}, $values );
-	}
+        return array_map( function ( $value ) {
+            return 'PT' . str_replace( ':', 'H', $value ) . 'M';
+        }, $values );
+    }
 
 }

Spacing +6 added lines, -6 removed lines

@@ -22,11 +22,11 @@ 
 	/**
 	 * {@inheritdoc}
 	 */
-	function get( $post_id, $meta_key ) {
+	function get($post_id, $meta_key) {
 
 		// Get the values and filter out the empty ones (or the ones with 00:00).
-		$values = array_filter( parent::get( $post_id, $meta_key ), function ( $item ) {
-			return ! empty( $item ) && '00:00' !== $item;
+		$values = array_filter(parent::get($post_id, $meta_key), function($item) {
+			return ! empty($item) && '00:00' !== $item;
 		} );
 
 		/*
@@ -35,9 +35,9 @@ 
 		 * or an h:mm format.
 		 * Both needs to be adjusted to the iso format.
 		 */
-		return array_map( function ( $value ) {
-			return 'PT' . str_replace( ':', 'H', $value ) . 'M';
-		}, $values );
+		return array_map(function($value) {
+			return 'PT'.str_replace(':', 'H', $value).'M';
+		}, $values);
 	}
 
 }

src/includes/class-wordlift-sample-data-ajax-adapter.php

Indentation +40 added lines, -40 removed lines

@@ -17,58 +17,58 @@
  */
 class Wordlift_Sample_Data_Ajax_Adapter {
 
-	/**
-	 * The {@link Wordlift_Sample_Data_Service} instance.
-	 *
-	 * @since  3.12.0
-	 * @access private
-	 * @var \Wordlift_Sample_Data_Service $sample_data_service The {@link Wordlift_Sample_Data_Service} instance.
-	 */
-	private $sample_data_service;
+    /**
+     * The {@link Wordlift_Sample_Data_Service} instance.
+     *
+     * @since  3.12.0
+     * @access private
+     * @var \Wordlift_Sample_Data_Service $sample_data_service The {@link Wordlift_Sample_Data_Service} instance.
+     */
+    private $sample_data_service;
 
-	/**
-	 * Create a {@link Wordlift_Sample_Data_Ajax_Adapter} instance.
-	 *
-	 * @since 3.12.0
-	 *
-	 * @param \Wordlift_Sample_Data_Service $sample_data_service The {@link Wordlift_Sample_Data_Service} instance.
-	 */
-	function __construct( $sample_data_service ) {
+    /**
+     * Create a {@link Wordlift_Sample_Data_Ajax_Adapter} instance.
+     *
+     * @since 3.12.0
+     *
+     * @param \Wordlift_Sample_Data_Service $sample_data_service The {@link Wordlift_Sample_Data_Service} instance.
+     */
+    function __construct( $sample_data_service ) {
 
-		$this->sample_data_service = $sample_data_service;
+        $this->sample_data_service = $sample_data_service;
 
-	}
+    }
 
-	/**
-	 * Handle the `wl_sample_data_create` ajax action.
-	 *
-	 * @since 3.12.0
-	 */
-	function create() {
+    /**
+     * Handle the `wl_sample_data_create` ajax action.
+     *
+     * @since 3.12.0
+     */
+    function create() {
 
-		// Clean any potential garbage before us.
-		ob_clean();
+        // Clean any potential garbage before us.
+        ob_clean();
 
-		// Create the sample data.
-		$this->sample_data_service->create();
+        // Create the sample data.
+        $this->sample_data_service->create();
 
-		// Send success.
-		wp_send_json_success();
+        // Send success.
+        wp_send_json_success();
 
-	}
+    }
 
-	function delete() {
+    function delete() {
 
-		// Clean any potential garbage before us.
-		ob_clean();
+        // Clean any potential garbage before us.
+        ob_clean();
 
-		// Create the sample data.
-		$this->sample_data_service->delete();
+        // Create the sample data.
+        $this->sample_data_service->delete();
 
-		// Send success.
-		@header( 'Content-Disposition: inline' );
-		wp_send_json_success();
+        // Send success.
+        @header( 'Content-Disposition: inline' );
+        wp_send_json_success();
 
-	}
+    }
 
 }
\ No newline at end of file

Spacing +2 added lines, -2 removed lines

@@ -33,7 +33,7 @@ 
 	 *
 	 * @param \Wordlift_Sample_Data_Service $sample_data_service The {@link Wordlift_Sample_Data_Service} instance.
 	 */
-	function __construct( $sample_data_service ) {
+	function __construct($sample_data_service) {
 
 		$this->sample_data_service = $sample_data_service;
 
@@ -66,7 +66,7 @@ 
 		$this->sample_data_service->delete();
 
 		// Send success.
-		@header( 'Content-Disposition: inline' );
+		@header('Content-Disposition: inline');
 		wp_send_json_success();
 
 	}

src/admin/partials/wordlift-admin-batch-analysis-page.php

Indentation +43 added lines, -43 removed lines

@@ -11,68 +11,68 @@
 	<h2><?php esc_html_e( 'WordLift Batch Analyze Monitor', 'wordlift' ) ?></h2>
 
 	<?php
-	$queue        = $this->batch_analysis_service->waiting_for_analysis();
-	$submit_count = count( $queue );
-	?>
+    $queue        = $this->batch_analysis_service->waiting_for_analysis();
+    $submit_count = count( $queue );
+    ?>
 	<h3><?php esc_html_e( 'Posts in queue', 'wordlift' ) ?><?php echo " ($submit_count)"; ?></h3>
 	<?php
-	if ( empty( $queue ) ) {
-		esc_html_e( 'Nothing is currently in queue', 'wordlift' );
-	} else {
-		echo '<ul>';
-		foreach ( $queue as $pid ) {
-			$cancel_link = admin_url( "admin-ajax.php?action=wl_batch_analysis_cancel&post=$pid" );
-			?>
+    if ( empty( $queue ) ) {
+        esc_html_e( 'Nothing is currently in queue', 'wordlift' );
+    } else {
+        echo '<ul>';
+        foreach ( $queue as $pid ) {
+            $cancel_link = admin_url( "admin-ajax.php?action=wl_batch_analysis_cancel&post=$pid" );
+            ?>
 			<li><a href="<?php echo get_edit_post_link( $pid ) ?>"><?php
-					echo get_the_title( $pid ) ?></a> [<a
+                    echo get_the_title( $pid ) ?></a> [<a
 					href="<?php echo esc_attr( $cancel_link ); ?> "><?php esc_html_e( 'Cancel', 'wordlift' ); ?></a>]
 			</li>
 			<?php
-		}
-		echo '</ul>';
-	}
+        }
+        echo '</ul>';
+    }
 
-	$queue         = $this->batch_analysis_service->waiting_for_response();
-	$request_count = count( $queue );
-	?>
+    $queue         = $this->batch_analysis_service->waiting_for_response();
+    $request_count = count( $queue );
+    ?>
 	<h3><?php esc_html_e( 'Posts being processed', 'wordlift' ) ?><?php echo " ($request_count)"; ?></h3>
 	<?php
-	if ( empty( $queue ) ) {
-		esc_html_e( 'Nothing is currently being processed', 'wordlift' );
-	} else {
-		echo '<ul>';
-		foreach ( $queue as $pid ) {
-			$cancel_link = admin_url( "admin-ajax.php?action=wl_batch_analysis_cancel&post=$pid" );
-			?>
+    if ( empty( $queue ) ) {
+        esc_html_e( 'Nothing is currently being processed', 'wordlift' );
+    } else {
+        echo '<ul>';
+        foreach ( $queue as $pid ) {
+            $cancel_link = admin_url( "admin-ajax.php?action=wl_batch_analysis_cancel&post=$pid" );
+            ?>
 			<li><a href="<?php echo get_edit_post_link( $pid ) ?>"><?php
-					echo get_the_title( $pid ) ?></a> [<a
+                    echo get_the_title( $pid ) ?></a> [<a
 					href="<?php echo esc_attr( $cancel_link ); ?> "><?php esc_html_e( 'Cancel', 'wordlift' ); ?></a>]
 			</li>
 			<?php
-		}
-		echo '</ul>';
-	}
+        }
+        echo '</ul>';
+    }
 
-	$queue         = $this->batch_analysis_service->get_warnings();
-	$warning_count = count( $queue );
-	?>
+    $queue         = $this->batch_analysis_service->get_warnings();
+    $warning_count = count( $queue );
+    ?>
 
 	<h3><?php esc_html_e( 'Posts with warnings', 'wordlift' ) ?><?php echo " ($warning_count)"; ?></h3>
 	<?php
-	if ( empty( $queue ) ) {
-		esc_html_e( 'No warnings :-)', 'wordlift' );
-	} else {
-		echo '<ul>';
-		foreach ( $queue as $pid ) {
-			$cancel_link = admin_url( "admin-ajax.php?action=wl_batch_analysis_clear_warning&post=$pid" );
-			?>
+    if ( empty( $queue ) ) {
+        esc_html_e( 'No warnings :-)', 'wordlift' );
+    } else {
+        echo '<ul>';
+        foreach ( $queue as $pid ) {
+            $cancel_link = admin_url( "admin-ajax.php?action=wl_batch_analysis_clear_warning&post=$pid" );
+            ?>
 			<li><a href="<?php echo get_edit_post_link( $pid ) ?>"><?php
-					echo get_the_title( $pid ) ?></a> [<a
+                    echo get_the_title( $pid ) ?></a> [<a
 					href="<?php echo esc_attr( $cancel_link ); ?> "><?php esc_html_e( 'Clear warning', 'wordlift' ); ?></a>]
 			</li>
 			<?php
-		}
-		echo '</ul>';
-	}
-	?>
+        }
+        echo '</ul>';
+    }
+    ?>
 </div>

Spacing +28 added lines, -28 removed lines

@@ -8,24 +8,24 @@ 
 ?>
 
 <div class="wrap">
-	<h2><?php esc_html_e( 'WordLift Batch Analyze Monitor', 'wordlift' ) ?></h2>
+	<h2><?php esc_html_e('WordLift Batch Analyze Monitor', 'wordlift') ?></h2>
 
 	<?php
 	$queue        = $this->batch_analysis_service->waiting_for_analysis();
-	$submit_count = count( $queue );
+	$submit_count = count($queue);
 	?>
-	<h3><?php esc_html_e( 'Posts in queue', 'wordlift' ) ?><?php echo " ($submit_count)"; ?></h3>
+	<h3><?php esc_html_e('Posts in queue', 'wordlift') ?><?php echo " ($submit_count)"; ?></h3>
 	<?php
-	if ( empty( $queue ) ) {
-		esc_html_e( 'Nothing is currently in queue', 'wordlift' );
+	if (empty($queue)) {
+		esc_html_e('Nothing is currently in queue', 'wordlift');
 	} else {
 		echo '<ul>';
-		foreach ( $queue as $pid ) {
-			$cancel_link = admin_url( "admin-ajax.php?action=wl_batch_analysis_cancel&post=$pid" );
+		foreach ($queue as $pid) {
+			$cancel_link = admin_url("admin-ajax.php?action=wl_batch_analysis_cancel&post=$pid");
 			?>
-			<li><a href="<?php echo get_edit_post_link( $pid ) ?>"><?php
-					echo get_the_title( $pid ) ?></a> [<a
-					href="<?php echo esc_attr( $cancel_link ); ?> "><?php esc_html_e( 'Cancel', 'wordlift' ); ?></a>]
+			<li><a href="<?php echo get_edit_post_link($pid) ?>"><?php
+					echo get_the_title($pid) ?></a> [<a
+					href="<?php echo esc_attr($cancel_link); ?> "><?php esc_html_e('Cancel', 'wordlift'); ?></a>]
 			</li>
 			<?php
 		}
@@ -33,20 +33,20 @@ 
 	}
 
 	$queue         = $this->batch_analysis_service->waiting_for_response();
-	$request_count = count( $queue );
+	$request_count = count($queue);
 	?>
-	<h3><?php esc_html_e( 'Posts being processed', 'wordlift' ) ?><?php echo " ($request_count)"; ?></h3>
+	<h3><?php esc_html_e('Posts being processed', 'wordlift') ?><?php echo " ($request_count)"; ?></h3>
 	<?php
-	if ( empty( $queue ) ) {
-		esc_html_e( 'Nothing is currently being processed', 'wordlift' );
+	if (empty($queue)) {
+		esc_html_e('Nothing is currently being processed', 'wordlift');
 	} else {
 		echo '<ul>';
-		foreach ( $queue as $pid ) {
-			$cancel_link = admin_url( "admin-ajax.php?action=wl_batch_analysis_cancel&post=$pid" );
+		foreach ($queue as $pid) {
+			$cancel_link = admin_url("admin-ajax.php?action=wl_batch_analysis_cancel&post=$pid");
 			?>
-			<li><a href="<?php echo get_edit_post_link( $pid ) ?>"><?php
-					echo get_the_title( $pid ) ?></a> [<a
-					href="<?php echo esc_attr( $cancel_link ); ?> "><?php esc_html_e( 'Cancel', 'wordlift' ); ?></a>]
+			<li><a href="<?php echo get_edit_post_link($pid) ?>"><?php
+					echo get_the_title($pid) ?></a> [<a
+					href="<?php echo esc_attr($cancel_link); ?> "><?php esc_html_e('Cancel', 'wordlift'); ?></a>]
 			</li>
 			<?php
 		}
@@ -54,21 +54,21 @@ 
 	}
 
 	$queue         = $this->batch_analysis_service->get_warnings();
-	$warning_count = count( $queue );
+	$warning_count = count($queue);
 	?>
 
-	<h3><?php esc_html_e( 'Posts with warnings', 'wordlift' ) ?><?php echo " ($warning_count)"; ?></h3>
+	<h3><?php esc_html_e('Posts with warnings', 'wordlift') ?><?php echo " ($warning_count)"; ?></h3>
 	<?php
-	if ( empty( $queue ) ) {
-		esc_html_e( 'No warnings :-)', 'wordlift' );
+	if (empty($queue)) {
+		esc_html_e('No warnings :-)', 'wordlift');
 	} else {
 		echo '<ul>';
-		foreach ( $queue as $pid ) {
-			$cancel_link = admin_url( "admin-ajax.php?action=wl_batch_analysis_clear_warning&post=$pid" );
+		foreach ($queue as $pid) {
+			$cancel_link = admin_url("admin-ajax.php?action=wl_batch_analysis_clear_warning&post=$pid");
 			?>
-			<li><a href="<?php echo get_edit_post_link( $pid ) ?>"><?php
-					echo get_the_title( $pid ) ?></a> [<a
-					href="<?php echo esc_attr( $cancel_link ); ?> "><?php esc_html_e( 'Clear warning', 'wordlift' ); ?></a>]
+			<li><a href="<?php echo get_edit_post_link($pid) ?>"><?php
+					echo get_the_title($pid) ?></a> [<a
+					href="<?php echo esc_attr($cancel_link); ?> "><?php esc_html_e('Clear warning', 'wordlift'); ?></a>]
 			</li>
 			<?php
 		}

src/includes/class-wordlift-thumbnail-service.php

Indentation +122 added lines, -122 removed lines

@@ -7,127 +7,127 @@
  */
 class Wordlift_Thumbnail_Service {
 
-	/**
-	 * The Thumbnail id meta key.
-	 *
-	 * @since 3.1.5
-	 */
-	const THUMBNAIL_ID_META_KEY = '_thumbnail_id';
-
-	/**
-	 * The predicate used in RDF to describe the thumbnail.
-	 *
-	 * @since 3.1.5
-	 */
-	const THUMBNAIL_RDF_PREDICATE = 'http://schema.org/image';
-
-	/**
-	 * The Log service.
-	 *
-	 * @since  3.1.5
-	 * @access private
-	 * @var \Wordlift_Log_Service The Log service.
-	 */
-	private $log_service;
-
-	/**
-	 * Create an instance of the Thumbnail service.
-	 *
-	 * @since 3.1.5
-	 */
-	public function __construct() {
-
-		$this->log_service = Wordlift_Log_Service::get_logger( 'Wordlift_Thumbnail_Service' );
-
-	}
-
-	/**
-	 * Receive post meta events immediately after a post metadata has been deleted.
-	 *
-	 * @since 3.1.5
-	 *
-	 * @param array  $meta_ids    An array of deleted metadata entry IDs.
-	 * @param int    $object_id   Object ID.
-	 * @param string $meta_key    Meta key.
-	 * @param mixed  $_meta_value Meta value.
-	 */
-	public function deleted_post_meta( $meta_ids, $object_id, $meta_key, $_meta_value ) {
-
-		// Return if it's not the Thumbnail id meta key.
-		if ( self::THUMBNAIL_ID_META_KEY !== $meta_key ) {
-			return;
-		}
-
-		// Do not perform any action is the post is not published.
-		if ( 'publish' !== get_post_status( $object_id ) ) {
-			return;
-		}
-
-		// Get the post uri and return if it's null.
-		if ( null === ( $uri = wl_get_entity_uri( $object_id ) ) ) {
-			return;
-		}
-
-		// Prepare the query and execute it. We don't buffer the query since we're not going to reindex.
-		$query = sprintf( 'DELETE { <%s> <%s> ?o . } WHERE  { <%1$s> <%2$s> ?o . };', $uri, self::THUMBNAIL_RDF_PREDICATE );
-		if ( false === rl_execute_sparql_update_query( $query, false ) ) {
-
-			$this->log_service->error( "An error occurred removing the post thumbnail [ meta ids :: " . ( is_array( $meta_ids ) ? implode( ',', $meta_ids ) : $meta_ids ) . " ][ object id :: $object_id ][ meta key :: $meta_key ][ meta value :: " . ( is_array( $_meta_value ) ? implode( ',', $_meta_value ) : $_meta_value ) . " ][ query :: $query ]" );
-
-		}
-
-	}
-
-	/**
-	 * Receive post meta events immediately after a post metadata has been added.
-	 *
-	 * @since 3.1.5
-	 *
-	 * @param int    $mid         The meta ID after successful update.
-	 * @param int    $object_id   Object ID.
-	 * @param string $meta_key    Meta key.
-	 * @param mixed  $_meta_value Meta value.
-	 */
-	public function added_or_updated_post_meta( $mid, $object_id, $meta_key, $_meta_value ) {
-
-		// $this->log_service->trace( "A post meta has been updated [ meta id :: $mid ][ object id :: $object_id ][ meta key :: $meta_key ][ meta value :: " . var_export( $_meta_value, true ) . " ]" );
-
-		// Return if it's not the Thumbnail id meta key.
-		if ( self::THUMBNAIL_ID_META_KEY !== $meta_key ) {
-			return;
-		}
-
-		// The meta value must be the numeric id of the attachment. If it isn't, return.
-		if ( ! is_numeric( $_meta_value ) ) {
-			return;
-		}
-
-		// Do not perform any action is the post is not published.
-		if ( 'publish' !== get_post_status( $object_id ) ) {
-			return;
-		}
-
-		// Get the post uri and return if it's null.
-		if ( null === ( $uri = wl_get_entity_uri( $object_id ) ) ) {
-			return;
-		}
-
-		// Don't pollute the cache.
-		wp_cache_delete( $object_id, 'post_meta' );
-
-		// Get the attachment url and return if not found.
-		if ( false === ( $attachment_url = wp_get_attachment_url( $_meta_value ) ) ) {
-			return;
-		};
-
-		// Prepare the query and execute it. We don't buffer the query since we're not going to reindex.
-		$query = sprintf( 'DELETE { <%1$s> <%2$s> ?o . } WHERE  { <%1$s> <%2$s> ?o . }; INSERT DATA { <%1$s> <%2$s> <%3$s> . };', $uri, self::THUMBNAIL_RDF_PREDICATE, $attachment_url );
-		if ( false === rl_execute_sparql_update_query( $query, false ) ) {
-
-			$this->log_service->error( "An error occurred removing the post thumbnail [ meta ids :: $mid ][ object id :: $object_id ][ meta key :: $meta_key ][ meta value :: " . var_export( $_meta_value, true ) . " ][ query :: $query ]" );
-
-		}
-
-	}
+    /**
+     * The Thumbnail id meta key.
+     *
+     * @since 3.1.5
+     */
+    const THUMBNAIL_ID_META_KEY = '_thumbnail_id';
+
+    /**
+     * The predicate used in RDF to describe the thumbnail.
+     *
+     * @since 3.1.5
+     */
+    const THUMBNAIL_RDF_PREDICATE = 'http://schema.org/image';
+
+    /**
+     * The Log service.
+     *
+     * @since  3.1.5
+     * @access private
+     * @var \Wordlift_Log_Service The Log service.
+     */
+    private $log_service;
+
+    /**
+     * Create an instance of the Thumbnail service.
+     *
+     * @since 3.1.5
+     */
+    public function __construct() {
+
+        $this->log_service = Wordlift_Log_Service::get_logger( 'Wordlift_Thumbnail_Service' );
+
+    }
+
+    /**
+     * Receive post meta events immediately after a post metadata has been deleted.
+     *
+     * @since 3.1.5
+     *
+     * @param array  $meta_ids    An array of deleted metadata entry IDs.
+     * @param int    $object_id   Object ID.
+     * @param string $meta_key    Meta key.
+     * @param mixed  $_meta_value Meta value.
+     */
+    public function deleted_post_meta( $meta_ids, $object_id, $meta_key, $_meta_value ) {
+
+        // Return if it's not the Thumbnail id meta key.
+        if ( self::THUMBNAIL_ID_META_KEY !== $meta_key ) {
+            return;
+        }
+
+        // Do not perform any action is the post is not published.
+        if ( 'publish' !== get_post_status( $object_id ) ) {
+            return;
+        }
+
+        // Get the post uri and return if it's null.
+        if ( null === ( $uri = wl_get_entity_uri( $object_id ) ) ) {
+            return;
+        }
+
+        // Prepare the query and execute it. We don't buffer the query since we're not going to reindex.
+        $query = sprintf( 'DELETE { <%s> <%s> ?o . } WHERE  { <%1$s> <%2$s> ?o . };', $uri, self::THUMBNAIL_RDF_PREDICATE );
+        if ( false === rl_execute_sparql_update_query( $query, false ) ) {
+
+            $this->log_service->error( "An error occurred removing the post thumbnail [ meta ids :: " . ( is_array( $meta_ids ) ? implode( ',', $meta_ids ) : $meta_ids ) . " ][ object id :: $object_id ][ meta key :: $meta_key ][ meta value :: " . ( is_array( $_meta_value ) ? implode( ',', $_meta_value ) : $_meta_value ) . " ][ query :: $query ]" );
+
+        }
+
+    }
+
+    /**
+     * Receive post meta events immediately after a post metadata has been added.
+     *
+     * @since 3.1.5
+     *
+     * @param int    $mid         The meta ID after successful update.
+     * @param int    $object_id   Object ID.
+     * @param string $meta_key    Meta key.
+     * @param mixed  $_meta_value Meta value.
+     */
+    public function added_or_updated_post_meta( $mid, $object_id, $meta_key, $_meta_value ) {
+
+        // $this->log_service->trace( "A post meta has been updated [ meta id :: $mid ][ object id :: $object_id ][ meta key :: $meta_key ][ meta value :: " . var_export( $_meta_value, true ) . " ]" );
+
+        // Return if it's not the Thumbnail id meta key.
+        if ( self::THUMBNAIL_ID_META_KEY !== $meta_key ) {
+            return;
+        }
+
+        // The meta value must be the numeric id of the attachment. If it isn't, return.
+        if ( ! is_numeric( $_meta_value ) ) {
+            return;
+        }
+
+        // Do not perform any action is the post is not published.
+        if ( 'publish' !== get_post_status( $object_id ) ) {
+            return;
+        }
+
+        // Get the post uri and return if it's null.
+        if ( null === ( $uri = wl_get_entity_uri( $object_id ) ) ) {
+            return;
+        }
+
+        // Don't pollute the cache.
+        wp_cache_delete( $object_id, 'post_meta' );
+
+        // Get the attachment url and return if not found.
+        if ( false === ( $attachment_url = wp_get_attachment_url( $_meta_value ) ) ) {
+            return;
+        };
+
+        // Prepare the query and execute it. We don't buffer the query since we're not going to reindex.
+        $query = sprintf( 'DELETE { <%1$s> <%2$s> ?o . } WHERE  { <%1$s> <%2$s> ?o . }; INSERT DATA { <%1$s> <%2$s> <%3$s> . };', $uri, self::THUMBNAIL_RDF_PREDICATE, $attachment_url );
+        if ( false === rl_execute_sparql_update_query( $query, false ) ) {
+
+            $this->log_service->error( "An error occurred removing the post thumbnail [ meta ids :: $mid ][ object id :: $object_id ][ meta key :: $meta_key ][ meta value :: " . var_export( $_meta_value, true ) . " ][ query :: $query ]" );
+
+        }
+
+    }
 
 }

Spacing +18 added lines, -18 removed lines

@@ -37,7 +37,7 @@ 
 	 */
 	public function __construct() {
 
-		$this->log_service = Wordlift_Log_Service::get_logger( 'Wordlift_Thumbnail_Service' );
+		$this->log_service = Wordlift_Log_Service::get_logger('Wordlift_Thumbnail_Service');
 
 	}
 
@@ -51,28 +51,28 @@ 
 	 * @param string $meta_key    Meta key.
 	 * @param mixed  $_meta_value Meta value.
 	 */
-	public function deleted_post_meta( $meta_ids, $object_id, $meta_key, $_meta_value ) {
+	public function deleted_post_meta($meta_ids, $object_id, $meta_key, $_meta_value) {
 
 		// Return if it's not the Thumbnail id meta key.
-		if ( self::THUMBNAIL_ID_META_KEY !== $meta_key ) {
+		if (self::THUMBNAIL_ID_META_KEY !== $meta_key) {
 			return;
 		}
 
 		// Do not perform any action is the post is not published.
-		if ( 'publish' !== get_post_status( $object_id ) ) {
+		if ('publish' !== get_post_status($object_id)) {
 			return;
 		}
 
 		// Get the post uri and return if it's null.
-		if ( null === ( $uri = wl_get_entity_uri( $object_id ) ) ) {
+		if (null === ($uri = wl_get_entity_uri($object_id))) {
 			return;
 		}
 
 		// Prepare the query and execute it. We don't buffer the query since we're not going to reindex.
-		$query = sprintf( 'DELETE { <%s> <%s> ?o . } WHERE  { <%1$s> <%2$s> ?o . };', $uri, self::THUMBNAIL_RDF_PREDICATE );
-		if ( false === rl_execute_sparql_update_query( $query, false ) ) {
+		$query = sprintf('DELETE { <%s> <%s> ?o . } WHERE  { <%1$s> <%2$s> ?o . };', $uri, self::THUMBNAIL_RDF_PREDICATE);
+		if (false === rl_execute_sparql_update_query($query, false)) {
 
-			$this->log_service->error( "An error occurred removing the post thumbnail [ meta ids :: " . ( is_array( $meta_ids ) ? implode( ',', $meta_ids ) : $meta_ids ) . " ][ object id :: $object_id ][ meta key :: $meta_key ][ meta value :: " . ( is_array( $_meta_value ) ? implode( ',', $_meta_value ) : $_meta_value ) . " ][ query :: $query ]" );
+			$this->log_service->error("An error occurred removing the post thumbnail [ meta ids :: ".(is_array($meta_ids) ? implode(',', $meta_ids) : $meta_ids)." ][ object id :: $object_id ][ meta key :: $meta_key ][ meta value :: ".(is_array($_meta_value) ? implode(',', $_meta_value) : $_meta_value)." ][ query :: $query ]");
 
 		}
 
@@ -88,43 +88,43 @@ 
 	 * @param string $meta_key    Meta key.
 	 * @param mixed  $_meta_value Meta value.
 	 */
-	public function added_or_updated_post_meta( $mid, $object_id, $meta_key, $_meta_value ) {
+	public function added_or_updated_post_meta($mid, $object_id, $meta_key, $_meta_value) {
 
 		// $this->log_service->trace( "A post meta has been updated [ meta id :: $mid ][ object id :: $object_id ][ meta key :: $meta_key ][ meta value :: " . var_export( $_meta_value, true ) . " ]" );
 
 		// Return if it's not the Thumbnail id meta key.
-		if ( self::THUMBNAIL_ID_META_KEY !== $meta_key ) {
+		if (self::THUMBNAIL_ID_META_KEY !== $meta_key) {
 			return;
 		}
 
 		// The meta value must be the numeric id of the attachment. If it isn't, return.
-		if ( ! is_numeric( $_meta_value ) ) {
+		if ( ! is_numeric($_meta_value)) {
 			return;
 		}
 
 		// Do not perform any action is the post is not published.
-		if ( 'publish' !== get_post_status( $object_id ) ) {
+		if ('publish' !== get_post_status($object_id)) {
 			return;
 		}
 
 		// Get the post uri and return if it's null.
-		if ( null === ( $uri = wl_get_entity_uri( $object_id ) ) ) {
+		if (null === ($uri = wl_get_entity_uri($object_id))) {
 			return;
 		}
 
 		// Don't pollute the cache.
-		wp_cache_delete( $object_id, 'post_meta' );
+		wp_cache_delete($object_id, 'post_meta');
 
 		// Get the attachment url and return if not found.
-		if ( false === ( $attachment_url = wp_get_attachment_url( $_meta_value ) ) ) {
+		if (false === ($attachment_url = wp_get_attachment_url($_meta_value))) {
 			return;
 		};
 
 		// Prepare the query and execute it. We don't buffer the query since we're not going to reindex.
-		$query = sprintf( 'DELETE { <%1$s> <%2$s> ?o . } WHERE  { <%1$s> <%2$s> ?o . }; INSERT DATA { <%1$s> <%2$s> <%3$s> . };', $uri, self::THUMBNAIL_RDF_PREDICATE, $attachment_url );
-		if ( false === rl_execute_sparql_update_query( $query, false ) ) {
+		$query = sprintf('DELETE { <%1$s> <%2$s> ?o . } WHERE  { <%1$s> <%2$s> ?o . }; INSERT DATA { <%1$s> <%2$s> <%3$s> . };', $uri, self::THUMBNAIL_RDF_PREDICATE, $attachment_url);
+		if (false === rl_execute_sparql_update_query($query, false)) {
 
-			$this->log_service->error( "An error occurred removing the post thumbnail [ meta ids :: $mid ][ object id :: $object_id ][ meta key :: $meta_key ][ meta value :: " . var_export( $_meta_value, true ) . " ][ query :: $query ]" );
+			$this->log_service->error("An error occurred removing the post thumbnail [ meta ids :: $mid ][ object id :: $object_id ][ meta key :: $meta_key ][ meta value :: ".var_export($_meta_value, true)." ][ query :: $query ]");
 
 		}