insideout10 / wordlift-plugin

src/install/class-wordlift-install-3-49-2.php

Indentation +11 added lines, -11 removed lines

@@ -5,20 +5,20 @@
  */
 class Wordlift_Install_3_49_2 extends Wordlift_Install {
 
-	/**
-	 * {@inheritdoc}
-	 */
-	protected static $version = '3.49.2';
+    /**
+     * {@inheritdoc}
+     */
+    protected static $version = '3.49.2';
 
-	public function install() {
+    public function install() {
 
-		global $wpdb;
+        global $wpdb;
 
-		$wpdb->query( "DELETE FROM $wpdb->termmeta WHERE meta_key = 'wl_mentions_count'" );
-		$wpdb->query( "DELETE FROM $wpdb->termmeta WHERE meta_key = 'wl_about_count'" );
-		$wpdb->query( "DELETE FROM $wpdb->postmeta WHERE meta_key = 'wl_mentions_count'" );
-		$wpdb->query( "DELETE FROM $wpdb->postmeta WHERE meta_key = 'wl_about_count'" );
+        $wpdb->query( "DELETE FROM $wpdb->termmeta WHERE meta_key = 'wl_mentions_count'" );
+        $wpdb->query( "DELETE FROM $wpdb->termmeta WHERE meta_key = 'wl_about_count'" );
+        $wpdb->query( "DELETE FROM $wpdb->postmeta WHERE meta_key = 'wl_mentions_count'" );
+        $wpdb->query( "DELETE FROM $wpdb->postmeta WHERE meta_key = 'wl_about_count'" );
 
-	}
+    }
 
 }

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

Indentation +193 added lines, -193 removed lines

@@ -18,201 +18,201 @@
  */
 class Wordlift_Install_Service {
 
-	/**
-	 * A {@link Wordlift_Log_Service} instance.
-	 *
-	 * @since  3.18.0
-	 * @access private
-	 * @var \Wordlift_Log_Service $log A {@link Wordlift_Log_Service} instance.
-	 */
-	private $log;
-
-	/**
-	 * The singleton instance.
-	 *
-	 * @since  3.18.0
-	 * @access private
-	 * @var \Wordlift_Install_Service $instance A {@link Wordlift_Install_Service} instance.
-	 */
-	private static $instance;
-
-	/**
-	 * @var array
-	 */
-	private $installs;
-
-	/**
-	 * Wordlift_Install_Service constructor.
-	 *
-	 * @since 3.18.0
-	 */
-	public function __construct() {
-
-		/** Installs. */
-		require_once plugin_dir_path( __DIR__ ) . 'install/class-wordlift-install.php';
-		require_once plugin_dir_path( __DIR__ ) . 'install/class-wordlift-install-1-0-0.php';
-		require_once plugin_dir_path( __DIR__ ) . 'install/class-wordlift-install-3-10-0.php';
-		require_once plugin_dir_path( __DIR__ ) . 'install/class-wordlift-install-3-12-0.php';
-		require_once plugin_dir_path( __DIR__ ) . 'install/class-wordlift-install-3-14-0.php';
-		require_once plugin_dir_path( __DIR__ ) . 'install/class-wordlift-install-3-15-0.php';
-		require_once plugin_dir_path( __DIR__ ) . 'install/class-wordlift-install-3-18-0.php';
-		require_once plugin_dir_path( __DIR__ ) . 'install/class-wordlift-install-3-18-3.php';
-		require_once plugin_dir_path( __DIR__ ) . 'install/class-wordlift-install-3-19-5.php';
-		require_once plugin_dir_path( __DIR__ ) . 'install/class-wordlift-install-3-20-0.php';
-		require_once plugin_dir_path( __DIR__ ) . 'install/class-wordlift-install-3-23-4.php';
-		require_once plugin_dir_path( __DIR__ ) . 'install/class-wordlift-install-3-24-2.php';
-		require_once plugin_dir_path( __DIR__ ) . 'install/class-wordlift-install-all-entity-types.php';
-		require_once plugin_dir_path( __DIR__ ) . 'install/class-wordlift-install-package-type.php';
-		require_once plugin_dir_path( __DIR__ ) . 'install/class-wordlift-install-3-25-0.php';
-		require_once plugin_dir_path( __DIR__ ) . 'install/class-wordlift-install-3-27-0.php';
-		require_once plugin_dir_path( __DIR__ ) . 'install/class-wordlift-install-3-27-1.php';
-		require_once plugin_dir_path( __DIR__ ) . 'install/class-wordlift-install-3-28-0.php';
-		require_once plugin_dir_path( __DIR__ ) . 'install/class-wordlift-install-3-32-0.php';
-		require_once plugin_dir_path( __DIR__ ) . 'install/class-wordlift-install-3-33-9.php';
-		require_once plugin_dir_path( __DIR__ ) . 'install/class-wordlift-install-3-36-0.php';
-		require_once plugin_dir_path( __DIR__ ) . 'install/class-wordlift-install-3-38-5.php';
-		require_once plugin_dir_path( __DIR__ ) . 'install/class-wordlift-install-3-39-1.php';
-		require_once plugin_dir_path( __DIR__ ) . 'install/class-wordlift-install-3-40-1.php';
-		require_once plugin_dir_path( __DIR__ ) . 'install/class-wordlift-install-3-40-2.php';
-		require_once plugin_dir_path( __DIR__ ) . 'install/class-wordlift-install-3-41-0.php';
-		require_once plugin_dir_path( __DIR__ ) . 'install/class-wordlift-install-3-42-0.php';
-		require_once plugin_dir_path( __DIR__ ) . 'install/class-wordlift-install-3-44-1.php';
-		require_once plugin_dir_path( __DIR__ ) . 'install/class-wordlift-install-3-44-4.php';
-		require_once plugin_dir_path( __DIR__ ) . 'install/class-wordlift-install-3-45-0.php';
-		require_once plugin_dir_path( __DIR__ ) . 'install/class-wordlift-install-3-45-1.php';
-		require_once plugin_dir_path( __DIR__ ) . 'install/class-wordlift-install-3-49-2.php';
-
-		// Get the install services.
-		$this->installs = array(
-			new Wordlift_Install_1_0_0(),
-			new Wordlift_Install_3_10_0(),
-			new Wordlift_Install_3_12_0(),
-			new Wordlift_Install_3_14_0(),
-			new Wordlift_Install_3_15_0(),
-			new Wordlift_Install_3_18_0(),
-			new Wordlift_Install_3_18_3(),
-			new Wordlift_Install_3_19_5(),
-			new Wordlift_Install_3_20_0(),
-
-			/*
+    /**
+     * A {@link Wordlift_Log_Service} instance.
+     *
+     * @since  3.18.0
+     * @access private
+     * @var \Wordlift_Log_Service $log A {@link Wordlift_Log_Service} instance.
+     */
+    private $log;
+
+    /**
+     * The singleton instance.
+     *
+     * @since  3.18.0
+     * @access private
+     * @var \Wordlift_Install_Service $instance A {@link Wordlift_Install_Service} instance.
+     */
+    private static $instance;
+
+    /**
+     * @var array
+     */
+    private $installs;
+
+    /**
+     * Wordlift_Install_Service constructor.
+     *
+     * @since 3.18.0
+     */
+    public function __construct() {
+
+        /** Installs. */
+        require_once plugin_dir_path( __DIR__ ) . 'install/class-wordlift-install.php';
+        require_once plugin_dir_path( __DIR__ ) . 'install/class-wordlift-install-1-0-0.php';
+        require_once plugin_dir_path( __DIR__ ) . 'install/class-wordlift-install-3-10-0.php';
+        require_once plugin_dir_path( __DIR__ ) . 'install/class-wordlift-install-3-12-0.php';
+        require_once plugin_dir_path( __DIR__ ) . 'install/class-wordlift-install-3-14-0.php';
+        require_once plugin_dir_path( __DIR__ ) . 'install/class-wordlift-install-3-15-0.php';
+        require_once plugin_dir_path( __DIR__ ) . 'install/class-wordlift-install-3-18-0.php';
+        require_once plugin_dir_path( __DIR__ ) . 'install/class-wordlift-install-3-18-3.php';
+        require_once plugin_dir_path( __DIR__ ) . 'install/class-wordlift-install-3-19-5.php';
+        require_once plugin_dir_path( __DIR__ ) . 'install/class-wordlift-install-3-20-0.php';
+        require_once plugin_dir_path( __DIR__ ) . 'install/class-wordlift-install-3-23-4.php';
+        require_once plugin_dir_path( __DIR__ ) . 'install/class-wordlift-install-3-24-2.php';
+        require_once plugin_dir_path( __DIR__ ) . 'install/class-wordlift-install-all-entity-types.php';
+        require_once plugin_dir_path( __DIR__ ) . 'install/class-wordlift-install-package-type.php';
+        require_once plugin_dir_path( __DIR__ ) . 'install/class-wordlift-install-3-25-0.php';
+        require_once plugin_dir_path( __DIR__ ) . 'install/class-wordlift-install-3-27-0.php';
+        require_once plugin_dir_path( __DIR__ ) . 'install/class-wordlift-install-3-27-1.php';
+        require_once plugin_dir_path( __DIR__ ) . 'install/class-wordlift-install-3-28-0.php';
+        require_once plugin_dir_path( __DIR__ ) . 'install/class-wordlift-install-3-32-0.php';
+        require_once plugin_dir_path( __DIR__ ) . 'install/class-wordlift-install-3-33-9.php';
+        require_once plugin_dir_path( __DIR__ ) . 'install/class-wordlift-install-3-36-0.php';
+        require_once plugin_dir_path( __DIR__ ) . 'install/class-wordlift-install-3-38-5.php';
+        require_once plugin_dir_path( __DIR__ ) . 'install/class-wordlift-install-3-39-1.php';
+        require_once plugin_dir_path( __DIR__ ) . 'install/class-wordlift-install-3-40-1.php';
+        require_once plugin_dir_path( __DIR__ ) . 'install/class-wordlift-install-3-40-2.php';
+        require_once plugin_dir_path( __DIR__ ) . 'install/class-wordlift-install-3-41-0.php';
+        require_once plugin_dir_path( __DIR__ ) . 'install/class-wordlift-install-3-42-0.php';
+        require_once plugin_dir_path( __DIR__ ) . 'install/class-wordlift-install-3-44-1.php';
+        require_once plugin_dir_path( __DIR__ ) . 'install/class-wordlift-install-3-44-4.php';
+        require_once plugin_dir_path( __DIR__ ) . 'install/class-wordlift-install-3-45-0.php';
+        require_once plugin_dir_path( __DIR__ ) . 'install/class-wordlift-install-3-45-1.php';
+        require_once plugin_dir_path( __DIR__ ) . 'install/class-wordlift-install-3-49-2.php';
+
+        // Get the install services.
+        $this->installs = array(
+            new Wordlift_Install_1_0_0(),
+            new Wordlift_Install_3_10_0(),
+            new Wordlift_Install_3_12_0(),
+            new Wordlift_Install_3_14_0(),
+            new Wordlift_Install_3_15_0(),
+            new Wordlift_Install_3_18_0(),
+            new Wordlift_Install_3_18_3(),
+            new Wordlift_Install_3_19_5(),
+            new Wordlift_Install_3_20_0(),
+
+            /*
 			 * This should be enabled with #852.
 			 */
-			new Wordlift_Install_Package_Type(),
-			new Wordlift_Install_3_23_4(),
-			new Wordlift_Install_3_24_2(),
-			new Wordlift_Install_3_25_0(),
-			new Wordlift_Install_3_27_0(),
-			new Wordlift_Install_3_27_1(),
-			new Wordlift_Install_3_28_0(),
-			// Add column to represent term
-			new Wordlift_Install_3_32_0(),
-			// Add the entities table.
-			new Wordlift_Install_3_33_9(),
-			// When woocommerce extension installed, acf4so should be installed automatically.
-			new Wordlift_Install_3_36_0(),
-
-			new Wordlift_Install_3_38_5(),
-			new Wordlift_Install_3_39_1(),
-
-			// See #1621.
-			new Wordlift_Install_3_40_1(),
-
-			// @link https://github.com/insideout10/wordlift-plugin/issues/1627
-			new Wordlift_Install_3_40_2(),
-
-			new Wordlift_Install_3_41_0(),
-			new Wordlift_Install_3_42_0(),
-
-			new Wordlift_Install_3_44_1(),
-			new Wordlift_Install_3_44_4(),
-
-			new Wordlift_Install_3_45_0(),
-			new Wordlift_Install_3_45_1(),
-
-			new Wordlift_Install_3_49_2(),
-		);
-		self::$instance = $this;
-
-		$this->log = Wordlift_Log_Service::get_logger( get_class() );
-
-		add_action( 'init', array( $this, 'install' ) );
-
-	}
-
-	/**
-	 * Get the singleton instance.
-	 *
-	 * @since 3.18.0
-	 */
-	public static function get_instance() {
-
-		return self::$instance;
-	}
-
-	/**
-	 * Loop thought all versions and install the updates.
-	 *
-	 * @return void
-	 * @since 3.18.0
-	 *
-	 * @since 3.20.0 use a transient to avoid concurrent installation calls.
-	 */
-	public function install() {
-
-		$version = null;
-
-		if ( $this->install_required() && false === get_transient( '_wl_installing' ) ) {
-			set_transient( '_wl_installing', true, 5 * MINUTE_IN_SECONDS );
-			/** @var Wordlift_Install $install */
-			foreach ( $this->installs as $install ) {
-				// Get the install version.
-				$version = $install->get_version();
-
-				if ( version_compare( $version, $this->get_current_version(), '>' )
-					 || $install->must_install() ) {
-					$class_name = get_class( $install );
-
-					$this->log->info( "Current version is {$this->get_current_version()}, installing $class_name..." );
-					// Install version.
-					$install->install();
-
-					$this->log->info( "$class_name installed." );
-
-					// Bump the version.
-					update_option( 'wl_db_version', $version );
-				}
-			}
-
-			// phpcs:ignore WordPress.PHP.NoSilencedErrors.Discouraged
-			@delete_transient( '_wl_installing' );
-
-		}
-
-	}
-
-	private function install_required() {
-
-		/** @var Wordlift_Install $install */
-		foreach ( $this->installs as $install ) {
-			// Get the install version.
-			$version = $install->get_version();
-
-			if ( version_compare( $version, $this->get_current_version(), '>' )
-				 || $install->must_install() ) {
-				return true;
-			}
-		}
-
-		return false;
-	}
+            new Wordlift_Install_Package_Type(),
+            new Wordlift_Install_3_23_4(),
+            new Wordlift_Install_3_24_2(),
+            new Wordlift_Install_3_25_0(),
+            new Wordlift_Install_3_27_0(),
+            new Wordlift_Install_3_27_1(),
+            new Wordlift_Install_3_28_0(),
+            // Add column to represent term
+            new Wordlift_Install_3_32_0(),
+            // Add the entities table.
+            new Wordlift_Install_3_33_9(),
+            // When woocommerce extension installed, acf4so should be installed automatically.
+            new Wordlift_Install_3_36_0(),
+
+            new Wordlift_Install_3_38_5(),
+            new Wordlift_Install_3_39_1(),
+
+            // See #1621.
+            new Wordlift_Install_3_40_1(),
+
+            // @link https://github.com/insideout10/wordlift-plugin/issues/1627
+            new Wordlift_Install_3_40_2(),
+
+            new Wordlift_Install_3_41_0(),
+            new Wordlift_Install_3_42_0(),
+
+            new Wordlift_Install_3_44_1(),
+            new Wordlift_Install_3_44_4(),
+
+            new Wordlift_Install_3_45_0(),
+            new Wordlift_Install_3_45_1(),
+
+            new Wordlift_Install_3_49_2(),
+        );
+        self::$instance = $this;
+
+        $this->log = Wordlift_Log_Service::get_logger( get_class() );
+
+        add_action( 'init', array( $this, 'install' ) );
+
+    }
+
+    /**
+     * Get the singleton instance.
+     *
+     * @since 3.18.0
+     */
+    public static function get_instance() {
+
+        return self::$instance;
+    }
+
+    /**
+     * Loop thought all versions and install the updates.
+     *
+     * @return void
+     * @since 3.18.0
+     *
+     * @since 3.20.0 use a transient to avoid concurrent installation calls.
+     */
+    public function install() {
+
+        $version = null;
+
+        if ( $this->install_required() && false === get_transient( '_wl_installing' ) ) {
+            set_transient( '_wl_installing', true, 5 * MINUTE_IN_SECONDS );
+            /** @var Wordlift_Install $install */
+            foreach ( $this->installs as $install ) {
+                // Get the install version.
+                $version = $install->get_version();
+
+                if ( version_compare( $version, $this->get_current_version(), '>' )
+                     || $install->must_install() ) {
+                    $class_name = get_class( $install );
+
+                    $this->log->info( "Current version is {$this->get_current_version()}, installing $class_name..." );
+                    // Install version.
+                    $install->install();
+
+                    $this->log->info( "$class_name installed." );
+
+                    // Bump the version.
+                    update_option( 'wl_db_version', $version );
+                }
+            }
+
+            // phpcs:ignore WordPress.PHP.NoSilencedErrors.Discouraged
+            @delete_transient( '_wl_installing' );
+
+        }
+
+    }
+
+    private function install_required() {
+
+        /** @var Wordlift_Install $install */
+        foreach ( $this->installs as $install ) {
+            // Get the install version.
+            $version = $install->get_version();
+
+            if ( version_compare( $version, $this->get_current_version(), '>' )
+                 || $install->must_install() ) {
+                return true;
+            }
+        }
+
+        return false;
+    }
 
-	/**
-	 * Retrieve the current db version.
-	 *
-	 * @return type
-	 */
-	private function get_current_version() {
-		return get_option( 'wl_db_version', '0.0.0' );
-	}
+    /**
+     * Retrieve the current db version.
+     *
+     * @return type
+     */
+    private function get_current_version() {
+        return get_option( 'wl_db_version', '0.0.0' );
+    }
 
 }

src/modules/events/includes/Term_Entity/Events_Term_Entity_Jsonld.php

Indentation +200 added lines, -200 removed lines

@@ -13,204 +13,204 @@
  */
 class Events_Term_Entity_Jsonld {
 
-	/**
-	 * The {@link Api_Service} used to communicate with the remote APIs.
-	 *
-	 * @access private
-	 * @var Api_Service
-	 */
-	private $api_service;
-
-	/**
-	 * @param Api_Service $api_service
-	 */
-	public function __construct( Api_Service $api_service ) {
-		$this->api_service = $api_service;
-	}
-
-	/**
-	 * Register hooks.
-	 */
-	public function register_hooks() {
-		add_filter( 'wl_term_jsonld_array', array( $this, 'set_events_request' ), 90, 3 );
-	}
-
-	/**
-	 * Set term jsonld array event.
-	 *
-	 * @param $data array An array containing jsonld for term and entities.
-	 * @param $term_id int The term id for which the jsonld is generated.
-	 * @param $context int A context for the JSON-LD generation, valid values in Jsonld_Context_Enum
-	 *
-	 * @return array
-	 */
-	public function set_events_request( $data, $term_id, $context ) {
-		$jsonld_arr = $data['jsonld'];
-		if ( $this->should_return_early( $context ) ) {
-			return $data;
-		}
-
-		// Bail out if we don't have a term URL.
-		$term_url = $this->get_term_url( $term_id );
-		if ( ! is_string( $term_url ) ) {
-			return $data;
-		}
-
-		$counts = $this->get_initial_counts( $term_id );
-
-		$change_status = $this->update_counts_if_necessary( $jsonld_arr, $counts, $term_id );
-
-		if ( $change_status ) {
-			$this->send_api_request( $counts, $term_url );
-		}
-
-		return $data;
-	}
-
-	/**
-	 * If context is not PAGE, return early.
-	 *
-	 * @param $context int A context for the JSON-LD generation, valid values in Jsonld_Context_Enum
-	 *
-	 * @return bool
-	 */
-	private function should_return_early( $context ) {
-		return Jsonld_Context_Enum::PAGE !== $context;
-	}
-
-	/**
-	 * Fetch the initial 'about' and 'mentions' counts from term meta.
-	 *
-	 * @param $term_id int The term id for which the jsonld is generated.
-	 *
-	 * @return int[]
-	 */
-	private function get_initial_counts( $term_id ) {
-		return array(
-			'about'    => get_term_meta( $term_id, 'wl_about_count', true ) ? (int) get_term_meta( $term_id, 'wl_about_count', true ) : 0,
-			'mentions' => get_term_meta( $term_id, 'wl_mentions_count', true )
-				? (int) get_term_meta( $term_id, 'wl_mentions_count', true )
-				: 0,
-		);
-	}
-
-	/**
-	 * Update counts if necessary.
-	 *
-	 * @param $jsonld_arr
-	 * @param $counts
-	 * @param $term_id
-	 *
-	 * @return bool
-	 */
-	private function update_counts_if_necessary( $jsonld_arr, &$counts, $term_id ) {
-		// Flag to indicate if we should make an API request.
-		$change_status = false;
-
-		// If the $jsonld_arr is empty but the counts were previously more than 0.
-		if ( empty( $jsonld_arr[0] ) ) {
-			return $this->reset_counts_if_non_zero( $counts, $term_id );
-		}
-
-		// Get data from the array.
-		$data_arr = $jsonld_arr[0];
-
-		// Iterate over the counts array.
-		foreach ( $counts as $type => $type_count ) {
-			// Check if data has 'about' or 'mentions' and the count is different from the existing meta value.
-			if ( isset( $data_arr[ $type ] ) ) {
-				$new_count = count( $data_arr[ $type ] );
-				if ( $type_count !== $new_count ) {
-					// Set flag to true if counts have changed.
-					$change_status = true;
-
-					// Update the counts array with new count.
-					$counts[ $type ] = $new_count;
-
-					// Update term meta with new count.
-					update_term_meta( $term_id, 'wl_' . $type . '_count', $new_count );
-					continue;
-				}
-			}
-
-			if ( ! isset( $data_arr[ $type ] ) && $type_count > 0 ) {
-				// If the 'about' or 'mentions' has become empty, set it to 0.
-				$change_status   = true;
-				$counts[ $type ] = 0;
-				update_term_meta( $term_id, 'wl_' . $type . '_count', 0 );
-			}
-		}
-
-		return $change_status;
-	}
-
-	/**
-	 * Reset counts if non zero.
-	 *
-	 * @param $counts
-	 * @param $term_id
-	 *
-	 * @return bool
-	 */
-	private function reset_counts_if_non_zero( &$counts, $term_id ) {
-		$change_status = false;
-
-		foreach ( $counts as $type => $type_count ) {
-			if ( $type_count > 0 ) {
-				$change_status   = true;
-				$counts[ $type ] = 0;
-				update_term_meta( $term_id, 'wl_' . $type . '_count', 0 );
-			}
-		}
-
-		return $change_status;
-	}
-
-	/**
-	 * Send api request.
-	 *
-	 * @param $counts
-	 * @param $term_url string The term URL.
-	 */
-	private function send_api_request( $counts, $term_url ) {
-		// If the count has changed, make the API request.
-		$this->api_service->request(
-			'POST',
-			'/plugin/events',
-			array( 'Content-Type' => 'application/json' ),
-			wp_json_encode(
-				array(
-					'source' => 'jsonld',
-					'args'   => array(
-						array( 'about_count' => $counts['about'] ),
-						array( 'mentions_count' => $counts['mentions'] ),
-					),
-					'url'    => $term_url,
-				)
-			),
-			0.001,
-			null,
-			array( 'blocking' => false )
-		);
-	}
-
-	/**
-	 * Get term url.
-	 *
-	 * @param $id
-	 *
-	 * @return array|false|int|mixed|string|\WP_Error|\WP_Term|null
-	 */
-	private function get_term_url( $id ) {
-		if ( null === $id ) {
-			return isset( $_SERVER['REQUEST_URI'] ) ? filter_var( wp_unslash( $_SERVER['REQUEST_URI'] ), FILTER_SANITIZE_URL ) : '';
-		}
-
-		$maybe_url = get_term_meta( $id, Wordlift_Url_Property_Service::META_KEY, true );
-		if ( ! empty( $maybe_url ) ) {
-			return $maybe_url;
-		}
-
-		return get_term_link( $id );
-	}
+    /**
+     * The {@link Api_Service} used to communicate with the remote APIs.
+     *
+     * @access private
+     * @var Api_Service
+     */
+    private $api_service;
+
+    /**
+     * @param Api_Service $api_service
+     */
+    public function __construct( Api_Service $api_service ) {
+        $this->api_service = $api_service;
+    }
+
+    /**
+     * Register hooks.
+     */
+    public function register_hooks() {
+        add_filter( 'wl_term_jsonld_array', array( $this, 'set_events_request' ), 90, 3 );
+    }
+
+    /**
+     * Set term jsonld array event.
+     *
+     * @param $data array An array containing jsonld for term and entities.
+     * @param $term_id int The term id for which the jsonld is generated.
+     * @param $context int A context for the JSON-LD generation, valid values in Jsonld_Context_Enum
+     *
+     * @return array
+     */
+    public function set_events_request( $data, $term_id, $context ) {
+        $jsonld_arr = $data['jsonld'];
+        if ( $this->should_return_early( $context ) ) {
+            return $data;
+        }
+
+        // Bail out if we don't have a term URL.
+        $term_url = $this->get_term_url( $term_id );
+        if ( ! is_string( $term_url ) ) {
+            return $data;
+        }
+
+        $counts = $this->get_initial_counts( $term_id );
+
+        $change_status = $this->update_counts_if_necessary( $jsonld_arr, $counts, $term_id );
+
+        if ( $change_status ) {
+            $this->send_api_request( $counts, $term_url );
+        }
+
+        return $data;
+    }
+
+    /**
+     * If context is not PAGE, return early.
+     *
+     * @param $context int A context for the JSON-LD generation, valid values in Jsonld_Context_Enum
+     *
+     * @return bool
+     */
+    private function should_return_early( $context ) {
+        return Jsonld_Context_Enum::PAGE !== $context;
+    }
+
+    /**
+     * Fetch the initial 'about' and 'mentions' counts from term meta.
+     *
+     * @param $term_id int The term id for which the jsonld is generated.
+     *
+     * @return int[]
+     */
+    private function get_initial_counts( $term_id ) {
+        return array(
+            'about'    => get_term_meta( $term_id, 'wl_about_count', true ) ? (int) get_term_meta( $term_id, 'wl_about_count', true ) : 0,
+            'mentions' => get_term_meta( $term_id, 'wl_mentions_count', true )
+                ? (int) get_term_meta( $term_id, 'wl_mentions_count', true )
+                : 0,
+        );
+    }
+
+    /**
+     * Update counts if necessary.
+     *
+     * @param $jsonld_arr
+     * @param $counts
+     * @param $term_id
+     *
+     * @return bool
+     */
+    private function update_counts_if_necessary( $jsonld_arr, &$counts, $term_id ) {
+        // Flag to indicate if we should make an API request.
+        $change_status = false;
+
+        // If the $jsonld_arr is empty but the counts were previously more than 0.
+        if ( empty( $jsonld_arr[0] ) ) {
+            return $this->reset_counts_if_non_zero( $counts, $term_id );
+        }
+
+        // Get data from the array.
+        $data_arr = $jsonld_arr[0];
+
+        // Iterate over the counts array.
+        foreach ( $counts as $type => $type_count ) {
+            // Check if data has 'about' or 'mentions' and the count is different from the existing meta value.
+            if ( isset( $data_arr[ $type ] ) ) {
+                $new_count = count( $data_arr[ $type ] );
+                if ( $type_count !== $new_count ) {
+                    // Set flag to true if counts have changed.
+                    $change_status = true;
+
+                    // Update the counts array with new count.
+                    $counts[ $type ] = $new_count;
+
+                    // Update term meta with new count.
+                    update_term_meta( $term_id, 'wl_' . $type . '_count', $new_count );
+                    continue;
+                }
+            }
+
+            if ( ! isset( $data_arr[ $type ] ) && $type_count > 0 ) {
+                // If the 'about' or 'mentions' has become empty, set it to 0.
+                $change_status   = true;
+                $counts[ $type ] = 0;
+                update_term_meta( $term_id, 'wl_' . $type . '_count', 0 );
+            }
+        }
+
+        return $change_status;
+    }
+
+    /**
+     * Reset counts if non zero.
+     *
+     * @param $counts
+     * @param $term_id
+     *
+     * @return bool
+     */
+    private function reset_counts_if_non_zero( &$counts, $term_id ) {
+        $change_status = false;
+
+        foreach ( $counts as $type => $type_count ) {
+            if ( $type_count > 0 ) {
+                $change_status   = true;
+                $counts[ $type ] = 0;
+                update_term_meta( $term_id, 'wl_' . $type . '_count', 0 );
+            }
+        }
+
+        return $change_status;
+    }
+
+    /**
+     * Send api request.
+     *
+     * @param $counts
+     * @param $term_url string The term URL.
+     */
+    private function send_api_request( $counts, $term_url ) {
+        // If the count has changed, make the API request.
+        $this->api_service->request(
+            'POST',
+            '/plugin/events',
+            array( 'Content-Type' => 'application/json' ),
+            wp_json_encode(
+                array(
+                    'source' => 'jsonld',
+                    'args'   => array(
+                        array( 'about_count' => $counts['about'] ),
+                        array( 'mentions_count' => $counts['mentions'] ),
+                    ),
+                    'url'    => $term_url,
+                )
+            ),
+            0.001,
+            null,
+            array( 'blocking' => false )
+        );
+    }
+
+    /**
+     * Get term url.
+     *
+     * @param $id
+     *
+     * @return array|false|int|mixed|string|\WP_Error|\WP_Term|null
+     */
+    private function get_term_url( $id ) {
+        if ( null === $id ) {
+            return isset( $_SERVER['REQUEST_URI'] ) ? filter_var( wp_unslash( $_SERVER['REQUEST_URI'] ), FILTER_SANITIZE_URL ) : '';
+        }
+
+        $maybe_url = get_term_meta( $id, Wordlift_Url_Property_Service::META_KEY, true );
+        if ( ! empty( $maybe_url ) ) {
+            return $maybe_url;
+        }
+
+        return get_term_link( $id );
+    }
 }

src/modules/events/includes/Post_Entity/Events_Post_Entity_Jsonld.php

Indentation +210 added lines, -210 removed lines

@@ -13,219 +13,219 @@
  */
 class Events_Post_Entity_Jsonld {
 
-	/**
-	 * The {@link Api_Service} used to communicate with the remote APIs.
-	 *
-	 * @access private
-	 * @var Api_Service
-	 */
-	private $api_service;
-
-	/**
-	 * @param Api_Service $api_service
-	 */
-	public function __construct( Api_Service $api_service ) {
-		$this->api_service = $api_service;
-	}
-
-	/**
-	 * Register hooks.
-	 */
-	public function register_hooks() {
-		add_filter( 'wl_after_get_jsonld', array( $this, 'handle_after_get_jsonld' ), 90, 3 );
-		add_filter( 'wprm_recipe_metadata', array( $this, 'handle_recipe_metadata' ), PHP_INT_MAX - 100, 2 );
-	}
-
-	public function handle_after_get_jsonld( $jsonld_arr, $post_id, $context ) {
-
-		if (
-			// We're not in a page context.
-			Jsonld_Context_Enum::PAGE !== $context ||
-			// It's a Recipe post.
-			$this->is_a_recipe_post( $post_id )
-		) {
-			return $jsonld_arr;
-		}
-
-		return $this->set_events_request( $jsonld_arr, $post_id );
-	}
-
-	private function is_a_recipe_post( $post_id ) {
-		global $wpdb;
-
-		return $wpdb->get_var(
-			$wpdb->prepare(
-				"
+    /**
+     * The {@link Api_Service} used to communicate with the remote APIs.
+     *
+     * @access private
+     * @var Api_Service
+     */
+    private $api_service;
+
+    /**
+     * @param Api_Service $api_service
+     */
+    public function __construct( Api_Service $api_service ) {
+        $this->api_service = $api_service;
+    }
+
+    /**
+     * Register hooks.
+     */
+    public function register_hooks() {
+        add_filter( 'wl_after_get_jsonld', array( $this, 'handle_after_get_jsonld' ), 90, 3 );
+        add_filter( 'wprm_recipe_metadata', array( $this, 'handle_recipe_metadata' ), PHP_INT_MAX - 100, 2 );
+    }
+
+    public function handle_after_get_jsonld( $jsonld_arr, $post_id, $context ) {
+
+        if (
+            // We're not in a page context.
+            Jsonld_Context_Enum::PAGE !== $context ||
+            // It's a Recipe post.
+            $this->is_a_recipe_post( $post_id )
+        ) {
+            return $jsonld_arr;
+        }
+
+        return $this->set_events_request( $jsonld_arr, $post_id );
+    }
+
+    private function is_a_recipe_post( $post_id ) {
+        global $wpdb;
+
+        return $wpdb->get_var(
+            $wpdb->prepare(
+                "
 			SELECT COUNT(1) 
 			FROM $wpdb->postmeta 
 			WHERE meta_key = 'wprm_parent_post_id' 
 			AND meta_value = %d
 		",
-				$post_id
-			)
-		);
-	}
-
-	/**
-	 * @param $value
-	 * @param WPRM_Recipe $recipe
-	 *
-	 * @return mixed
-	 */
-	public function handle_recipe_metadata( $value, $recipe ) {
-		// We only handle the parent post.
-		$parent_post_id = $recipe->parent_post_id();
-		if ( is_numeric( $parent_post_id ) && 0 < $parent_post_id ) {
-			$this->set_events_request( array( $value ), $parent_post_id, Jsonld_Context_Enum::PAGE );
-		}
-
-		return $value;
-	}
-
-	/**
-	 * Set events request.
-	 *
-	 * @param $jsonld_arr array The final jsonld before outputting to page.
-	 * @param $post_id int The post id for which the jsonld is generated.
-	 *
-	 * @return array
-	 */
-	private function set_events_request( $jsonld_arr, $post_id ) {
-
-		// Bail out if we can't get a permalink.
-		$permalink = get_permalink( $post_id );
-		if ( false === $permalink ) {
-			return $jsonld_arr;
-		}
-
-		$counts = $this->get_initial_counts_post( $post_id );
-
-		$change_status = $this->update_counts_if_necessary_post( $jsonld_arr, $counts, $post_id );
-
-		if ( $change_status ) {
-			$this->send_api_request_post( $counts, $permalink );
-		}
-
-		return $jsonld_arr;
-	}
-
-	/**
-	 * Fetch the initial 'about' and 'mentions' counts from post meta.
-	 *
-	 * @param $post_id int The post id for which the jsonld is generated.
-	 *
-	 * @return int[]
-	 */
-	private function get_initial_counts_post( $post_id ) {
-		return array(
-			'about'    => get_post_meta( $post_id, 'wl_about_count', true ) ? (int) get_post_meta( $post_id, 'wl_about_count', true ) : 0,
-			'mentions' => get_post_meta( $post_id, 'wl_mentions_count', true )
-				? (int) get_post_meta( $post_id, 'wl_mentions_count', true )
-				: 0,
-		);
-	}
-
-	/**
-	 * Update counts if necessary post.
-	 *
-	 * @param $jsonld_arr
-	 * @param $counts
-	 * @param $post_id
-	 *
-	 * @return bool
-	 */
-	private function update_counts_if_necessary_post( $jsonld_arr, &$counts, $post_id ) {
-		// Flag to indicate if we should make an API request.
-		$change_status = false;
-
-		// If the $jsonld_arr is empty but the counts were previously more than 0.
-		if ( empty( $jsonld_arr[0] ) ) {
-			return $this->reset_counts_if_non_zero_post( $counts, $post_id );
-		}
-
-		// Get data from the array.
-		$data = $jsonld_arr[0];
-
-		// Iterate over the counts array.
-		foreach ( $counts as $type => $type_count ) {
-			// Check if data has 'about' or 'mentions' and the count is different from the existing meta value.
-			if ( isset( $data[ $type ] ) ) {
-				$new_count = count( $data[ $type ] );
-				if ( $type_count !== $new_count ) {
-					// Set flag to true if counts have changed.
-					$change_status = true;
-
-					// Update the counts array with new count.
-					$counts[ $type ] = $new_count;
-
-					// Update post meta with new count.
-					update_post_meta( $post_id, 'wl_' . $type . '_count', $new_count );
-					continue;
-				}
-			}
-
-			if ( ! isset( $data[ $type ] ) && $type_count > 0 ) {
-				// If the 'about' or 'mentions' has become empty, set it to 0.
-				$change_status   = true;
-				$counts[ $type ] = 0;
-				update_post_meta( $post_id, 'wl_' . $type . '_count', 0 );
-			}
-		}
-
-		return $change_status;
-	}
-
-	/**
-	 * Reset counts if non zero post.
-	 *
-	 * @param $counts
-	 * @param $post_id
-	 *
-	 * @return bool
-	 */
-	private function reset_counts_if_non_zero_post( &$counts, $post_id ) {
-		$change_status = false;
-
-		foreach ( $counts as $type => $type_count ) {
-			if ( $type_count > 0 ) {
-				$change_status   = true;
-				$counts[ $type ] = 0;
-				update_post_meta( $post_id, 'wl_' . $type . '_count', 0 );
-			}
-		}
-
-		return $change_status;
-	}
-
-	/**
-	 * Send api request post.
-	 *
-	 * @param $counts
-	 * @param $permalink string The web page URL
-	 */
-	private function send_api_request_post( $counts, $permalink ) {
-		// If the count has changed, make the API request.
-		// phpcs:ignore WordPress.NamingConventions.ValidHookName.UseUnderscores
-		$blocking = apply_filters( 'wl_feature__enable__sync-blocking', false );
-
-		// Apply the filter to the request args
-		$this->api_service->request(
-			'POST',
-			'/plugin/events',
-			array( 'Content-Type' => 'application/json' ),
-			wp_json_encode(
-				array(
-					'source' => 'jsonld',
-					'args'   => array(
-						array( 'about_count' => $counts['about'] ),
-						array( 'mentions_count' => $counts['mentions'] ),
-					),
-					'url'    => $permalink,
-				)
-			),
-			$blocking ? 60 : 0.001,
-			null,
-			array( 'blocking' => $blocking )
-		);
-	}
+                $post_id
+            )
+        );
+    }
+
+    /**
+     * @param $value
+     * @param WPRM_Recipe $recipe
+     *
+     * @return mixed
+     */
+    public function handle_recipe_metadata( $value, $recipe ) {
+        // We only handle the parent post.
+        $parent_post_id = $recipe->parent_post_id();
+        if ( is_numeric( $parent_post_id ) && 0 < $parent_post_id ) {
+            $this->set_events_request( array( $value ), $parent_post_id, Jsonld_Context_Enum::PAGE );
+        }
+
+        return $value;
+    }
+
+    /**
+     * Set events request.
+     *
+     * @param $jsonld_arr array The final jsonld before outputting to page.
+     * @param $post_id int The post id for which the jsonld is generated.
+     *
+     * @return array
+     */
+    private function set_events_request( $jsonld_arr, $post_id ) {
+
+        // Bail out if we can't get a permalink.
+        $permalink = get_permalink( $post_id );
+        if ( false === $permalink ) {
+            return $jsonld_arr;
+        }
+
+        $counts = $this->get_initial_counts_post( $post_id );
+
+        $change_status = $this->update_counts_if_necessary_post( $jsonld_arr, $counts, $post_id );
+
+        if ( $change_status ) {
+            $this->send_api_request_post( $counts, $permalink );
+        }
+
+        return $jsonld_arr;
+    }
+
+    /**
+     * Fetch the initial 'about' and 'mentions' counts from post meta.
+     *
+     * @param $post_id int The post id for which the jsonld is generated.
+     *
+     * @return int[]
+     */
+    private function get_initial_counts_post( $post_id ) {
+        return array(
+            'about'    => get_post_meta( $post_id, 'wl_about_count', true ) ? (int) get_post_meta( $post_id, 'wl_about_count', true ) : 0,
+            'mentions' => get_post_meta( $post_id, 'wl_mentions_count', true )
+                ? (int) get_post_meta( $post_id, 'wl_mentions_count', true )
+                : 0,
+        );
+    }
+
+    /**
+     * Update counts if necessary post.
+     *
+     * @param $jsonld_arr
+     * @param $counts
+     * @param $post_id
+     *
+     * @return bool
+     */
+    private function update_counts_if_necessary_post( $jsonld_arr, &$counts, $post_id ) {
+        // Flag to indicate if we should make an API request.
+        $change_status = false;
+
+        // If the $jsonld_arr is empty but the counts were previously more than 0.
+        if ( empty( $jsonld_arr[0] ) ) {
+            return $this->reset_counts_if_non_zero_post( $counts, $post_id );
+        }
+
+        // Get data from the array.
+        $data = $jsonld_arr[0];
+
+        // Iterate over the counts array.
+        foreach ( $counts as $type => $type_count ) {
+            // Check if data has 'about' or 'mentions' and the count is different from the existing meta value.
+            if ( isset( $data[ $type ] ) ) {
+                $new_count = count( $data[ $type ] );
+                if ( $type_count !== $new_count ) {
+                    // Set flag to true if counts have changed.
+                    $change_status = true;
+
+                    // Update the counts array with new count.
+                    $counts[ $type ] = $new_count;
+
+                    // Update post meta with new count.
+                    update_post_meta( $post_id, 'wl_' . $type . '_count', $new_count );
+                    continue;
+                }
+            }
+
+            if ( ! isset( $data[ $type ] ) && $type_count > 0 ) {
+                // If the 'about' or 'mentions' has become empty, set it to 0.
+                $change_status   = true;
+                $counts[ $type ] = 0;
+                update_post_meta( $post_id, 'wl_' . $type . '_count', 0 );
+            }
+        }
+
+        return $change_status;
+    }
+
+    /**
+     * Reset counts if non zero post.
+     *
+     * @param $counts
+     * @param $post_id
+     *
+     * @return bool
+     */
+    private function reset_counts_if_non_zero_post( &$counts, $post_id ) {
+        $change_status = false;
+
+        foreach ( $counts as $type => $type_count ) {
+            if ( $type_count > 0 ) {
+                $change_status   = true;
+                $counts[ $type ] = 0;
+                update_post_meta( $post_id, 'wl_' . $type . '_count', 0 );
+            }
+        }
+
+        return $change_status;
+    }
+
+    /**
+     * Send api request post.
+     *
+     * @param $counts
+     * @param $permalink string The web page URL
+     */
+    private function send_api_request_post( $counts, $permalink ) {
+        // If the count has changed, make the API request.
+        // phpcs:ignore WordPress.NamingConventions.ValidHookName.UseUnderscores
+        $blocking = apply_filters( 'wl_feature__enable__sync-blocking', false );
+
+        // Apply the filter to the request args
+        $this->api_service->request(
+            'POST',
+            '/plugin/events',
+            array( 'Content-Type' => 'application/json' ),
+            wp_json_encode(
+                array(
+                    'source' => 'jsonld',
+                    'args'   => array(
+                        array( 'about_count' => $counts['about'] ),
+                        array( 'mentions_count' => $counts['mentions'] ),
+                    ),
+                    'url'    => $permalink,
+                )
+            ),
+            $blocking ? 60 : 0.001,
+            null,
+            array( 'blocking' => $blocking )
+        );
+    }
 }

src/modules/food-kg/includes/Preconditions.php

Indentation +34 added lines, -34 removed lines

@@ -4,39 +4,39 @@
 
 class Preconditions {
 
-	/**
-	 * @var Notices
-	 */
-	private $notices;
-
-	public function __construct( Notices $notices ) {
-		$this->notices = $notices;
-	}
-
-	public function pass() {
-		return $this->has_prerequisites() && $this->check_version();
-	}
-
-	private function has_prerequisites() {
-		return defined( 'WPRM_VERSION' )
-			   && class_exists( 'WP_Recipe_Maker' )
-		       // phpcs:ignore WordPress.NamingConventions.ValidHookName.UseUnderscores
-			   && apply_filters( 'wl_feature__enable__food-kg', false );
-	}
-
-	private function check_version() {
-		$check = version_compare( WPRM_VERSION, '8.1.0', '>=' )
-				&& version_compare( WPRM_VERSION, '8.11.0', '<' );
-
-		if ( ! $check ) {
-			$this->notices->queue(
-				'warning',
-				/* translators: 1: minimum supported WPRM version, 2: maximum supported WPRM version, 3: Detected WP Recipe Maker version. */
-				sprintf( __( 'WordLift Food KG support requires WP Recipe Maker %1$s-%2$s, %3$s found.', 'wordlift' ), '8.1', '8.10', WPRM_VERSION )
-			);
-		}
-
-		return $check;
-	}
+    /**
+     * @var Notices
+     */
+    private $notices;
+
+    public function __construct( Notices $notices ) {
+        $this->notices = $notices;
+    }
+
+    public function pass() {
+        return $this->has_prerequisites() && $this->check_version();
+    }
+
+    private function has_prerequisites() {
+        return defined( 'WPRM_VERSION' )
+               && class_exists( 'WP_Recipe_Maker' )
+                // phpcs:ignore WordPress.NamingConventions.ValidHookName.UseUnderscores
+               && apply_filters( 'wl_feature__enable__food-kg', false );
+    }
+
+    private function check_version() {
+        $check = version_compare( WPRM_VERSION, '8.1.0', '>=' )
+                && version_compare( WPRM_VERSION, '8.11.0', '<' );
+
+        if ( ! $check ) {
+            $this->notices->queue(
+                'warning',
+                /* translators: 1: minimum supported WPRM version, 2: maximum supported WPRM version, 3: Detected WP Recipe Maker version. */
+                sprintf( __( 'WordLift Food KG support requires WP Recipe Maker %1$s-%2$s, %3$s found.', 'wordlift' ), '8.1', '8.10', WPRM_VERSION )
+            );
+        }
+
+        return $check;
+    }
 
 }