insideout10 / wordlift-plugin

src/includes/class-wordlift-entity-type-taxonomy-service.php

Indentation +201 added lines, -201 removed lines

@@ -14,211 +14,211 @@
  */
 class Wordlift_Entity_Type_Taxonomy_Service {
 
-	/**
-	 * The WordPress taxonomy name.
-	 *
-	 * @since 1.0.0
-	 */
-	const TAXONOMY_NAME = 'wl_entity_type';
-
-	/**
-	 * Register the taxonomies.
-	 *
-	 * @since 3.23.6 we hook to `wp_get_object_terms` to ensure that a term is returned when a post is queries for the
-	 *               `wl_entity_type` taxonomy.
-	 * @since 3.18.0
-	 */
-	public function init() {
-
-		$labels = array(
-			'name'              => _x( 'Entity Types', 'taxonomy general name', 'wordlift' ),
-			'singular_name'     => _x( 'Entity Type', 'taxonomy singular name', 'wordlift' ),
-			'search_items'      => __( 'Search Entity Types', 'wordlift' ),
-			'all_items'         => __( 'All Entity Types', 'wordlift' ),
-			'parent_item'       => __( 'Parent Entity Type', 'wordlift' ),
-			'parent_item_colon' => __( 'Parent Entity Type:', 'wordlift' ),
-			'edit_item'         => __( 'Edit Entity Type', 'wordlift' ),
-			'update_item'       => __( 'Update Entity Type', 'wordlift' ),
-			'add_new_item'      => __( 'Add New Entity Type', 'wordlift' ),
-			'new_item_name'     => __( 'New Entity Type', 'wordlift' ),
-			'menu_name'         => __( 'Entity Types', 'wordlift' ),
-		);
-
-		// Take away GUI for taxonomy editing.
-		// TODO: read capabilities when editing of the WL <-> schema.org mapping is possible.
-		$capabilities = array(
-			// We enable editors to change the title/description of terms:
-			//
-			// @see https://github.com/insideout10/wordlift-plugin/issues/398.
-			'manage_terms' => 'manage_options',
-			'edit_terms'   => 'wl_entity_type_edit_term',
-			'delete_terms' => 'wl_entity_type_delete_term',
-			'assign_terms' => 'edit_posts',
-		);
-
-		$args = array(
-			'labels'             => $labels,
-			'capabilities'       => $capabilities,
-			'hierarchical'       => true,
-			// phpcs:ignore WordPress.NamingConventions.ValidHookName.UseUnderscores
-			'show_admin_column'  => apply_filters( 'wl_feature__enable__entity-types-taxonomy', true ),
-			// phpcs:ignore WordPress.NamingConventions.ValidHookName.UseUnderscores
-			'show_in_rest'       => apply_filters( 'wl_feature__enable__entity-types-taxonomy', true ),
-			'show_in_quick_edit' => false,
-			'publicly_queryable' => false,
-		);
-
-		/*
+    /**
+     * The WordPress taxonomy name.
+     *
+     * @since 1.0.0
+     */
+    const TAXONOMY_NAME = 'wl_entity_type';
+
+    /**
+     * Register the taxonomies.
+     *
+     * @since 3.23.6 we hook to `wp_get_object_terms` to ensure that a term is returned when a post is queries for the
+     *               `wl_entity_type` taxonomy.
+     * @since 3.18.0
+     */
+    public function init() {
+
+        $labels = array(
+            'name'              => _x( 'Entity Types', 'taxonomy general name', 'wordlift' ),
+            'singular_name'     => _x( 'Entity Type', 'taxonomy singular name', 'wordlift' ),
+            'search_items'      => __( 'Search Entity Types', 'wordlift' ),
+            'all_items'         => __( 'All Entity Types', 'wordlift' ),
+            'parent_item'       => __( 'Parent Entity Type', 'wordlift' ),
+            'parent_item_colon' => __( 'Parent Entity Type:', 'wordlift' ),
+            'edit_item'         => __( 'Edit Entity Type', 'wordlift' ),
+            'update_item'       => __( 'Update Entity Type', 'wordlift' ),
+            'add_new_item'      => __( 'Add New Entity Type', 'wordlift' ),
+            'new_item_name'     => __( 'New Entity Type', 'wordlift' ),
+            'menu_name'         => __( 'Entity Types', 'wordlift' ),
+        );
+
+        // Take away GUI for taxonomy editing.
+        // TODO: read capabilities when editing of the WL <-> schema.org mapping is possible.
+        $capabilities = array(
+            // We enable editors to change the title/description of terms:
+            //
+            // @see https://github.com/insideout10/wordlift-plugin/issues/398.
+            'manage_terms' => 'manage_options',
+            'edit_terms'   => 'wl_entity_type_edit_term',
+            'delete_terms' => 'wl_entity_type_delete_term',
+            'assign_terms' => 'edit_posts',
+        );
+
+        $args = array(
+            'labels'             => $labels,
+            'capabilities'       => $capabilities,
+            'hierarchical'       => true,
+            // phpcs:ignore WordPress.NamingConventions.ValidHookName.UseUnderscores
+            'show_admin_column'  => apply_filters( 'wl_feature__enable__entity-types-taxonomy', true ),
+            // phpcs:ignore WordPress.NamingConventions.ValidHookName.UseUnderscores
+            'show_in_rest'       => apply_filters( 'wl_feature__enable__entity-types-taxonomy', true ),
+            'show_in_quick_edit' => false,
+            'publicly_queryable' => false,
+        );
+
+        /*
 		 * If `All Entity Types` is enabled, use the new metabox.
 		 *
 		 * @see https://github.com/insideout10/wordlift-plugin/issues/835
 		 * @since 3.20.0
 		 */
-		// phpcs:ignore WordPress.NamingConventions.ValidHookName.UseUnderscores
-		if ( apply_filters( 'wl_feature__enable__all-entity-types', WL_ALL_ENTITY_TYPES ) ) {
-			// phpcs:ignore WordPress.NamingConventions.ValidHookName.UseUnderscores
-			$args['meta_box_cb'] = apply_filters( 'wl_feature__enable__entity-types-taxonomy', true ) ? array(
-				'Wordlift_Admin_Schemaorg_Taxonomy_Metabox',
-				'render',
-			) : false;
-		}
-
-		register_taxonomy(
-			self::TAXONOMY_NAME, // Taxonomy name.
-			Wordlift_Entity_Service::valid_entity_post_types(), // Taxonomy post types.
-			$args // Taxonomy args.
-		);
-
-		/**
-		 * Register meta wl_entities_gutenberg for use in Gutenberg
-		 */
-		register_meta(
-			'post',
-			'wl_entities_gutenberg',
-			array(
-				'show_in_rest' => true,
-				'single'       => true,
-				'type'         => 'string',
-			)
-		);
-
-		/**
-		 * Register meta _wl_alt_label for use in Gutenberg
-		 */
-		// register_meta( 'post', Wordlift_Entity_Service::ALTERNATIVE_LABEL_META_KEY, array(
-		// 'object_subtype' => '',
-		// 'show_in_rest'   => true,
-		// 'single'         => false,
-		// 'type'           => 'string',
-		// 'auth_callback'  => function () {
-		// return current_user_can( 'edit_posts' );
-		// }
-		// ) );
-
-		// see #1364: add custom fields support for CPTs that are valid entity post types in order to be able to
-		// query for synonyms in Block Editor.
-		add_filter(
-			'register_post_type_args',
-			function ( $args, $post_type ) {
-				if ( in_array( $post_type, Wordlift_Entity_Service::valid_entity_post_types(), true ) ) {
-					if ( ! isset( $args['supports'] ) ) {
-						$args['supports'] = array();
-					}
-					$args['supports'][] = 'custom-fields';
-				}
-
-				return $args;
-			},
-			10,
-			2
-		);
-
-		// Add filter to change the metabox CSS class.
-		add_filter( 'postbox_classes_entity_wl_entity_typediv', 'wl_admin_metaboxes_add_css_class' );
-
-		// Add a filter to preset the object term if none is set.
-		//
-		// DO NOT hook to `wp_get_object_terms`, because `wp_get_object_terms` returns imploded values for SQL queries.
-		//
-		// @see https://github.com/insideout10/wordlift-plugin/issues/995
-		// @since 3.23.6
-		add_filter( 'get_object_terms', array( $this, 'get_object_terms' ), 10, 4 );
-
-		/**
-		 * Exclude sitemap creation for wl_entity_type taxonomy in Yoast
-		 *
-		 * @since 3.30.1
-		 */
-		add_filter( 'wpseo_sitemap_exclude_taxonomy', array( $this, 'wpseo_sitemap_exclude_taxonomy' ), 10, 2 );
-
-	}
-
-	/**
-	 * Hook to the `get_object_terms` filter.
-	 *
-	 * We check if our taxonomy is requested and whether a term has been returned. If no term has been returned we
-	 * preset `Article` for posts/pages and 'Thing' for everything else and we query the terms again.
-	 *
-	 * @param array    $terms Array of terms for the given object or objects.
-	 * @param int[]    $object_ids Array of object IDs for which terms were retrieved.
-	 * @param string[] $taxonomies Array of taxonomy names from which terms were retrieved.
-	 * @param array    $args Array of arguments for retrieving terms for the given
-	 *                                object(s). See get_object_terms() for details.
-	 *
-	 * @return array|WP_Error
-	 * @since 3.23.6
-	 */
-	public function get_object_terms( $terms, $object_ids, $taxonomies, $args ) {
-		// Get our entity type.
-		$entity_type = self::TAXONOMY_NAME;
-
-		// Check if this is a query for our entity type, that no terms have been found and that we have an article
-		// term to preset in case.
-		if ( ! taxonomy_exists( $entity_type )
-			 || array( $entity_type ) !== (array) $taxonomies
-			 || ! empty( $terms )
-			 || ! term_exists( 'article', $entity_type )
-			 || ! term_exists( 'thing', $entity_type ) ) {
-
-			// Return the input value.
-			return $terms;
-		}
-
-		// Avoid nested calls in case of issues.
-		remove_filter( 'get_object_terms', array( $this, 'get_object_terms' ), 10 );
-
-		// Set the default term for all the queried object.
-		foreach ( (array) $object_ids as $object_id ) {
-			$post_type = get_post_type( $object_id );
-			if ( Wordlift_Entity_Type_Service::is_valid_entity_post_type( $post_type ) ) {
-				// Set the term to article for posts and pages, or to thing for everything else.
-				$uris = Wordlift_Entity_Type_Adapter::get_entity_types( $post_type );
-				foreach ( $uris as $uri ) {
-					// set the uri based on post type.
-					if ( 'http://schema.org/Article' === $uri || 'http://schema.org/Thing' === $uri ) {
-						$uri = Wordlift_Entity_Service::TYPE_NAME === $post_type ?
-							'http://schema.org/Thing' : 'http://schema.org/Article';
-					}
-					Wordlift_Entity_Type_Service::get_instance()->set( $object_id, $uri );
-				}
-			}
-		}
-
-		// Finally return the object terms.
-		$terms = wp_get_object_terms( $object_ids, $taxonomies, $args );
-
-		// Re-enable nested calls in case of issues.
-		add_filter( 'get_object_terms', array( $this, 'get_object_terms' ), 10, 4 );
-
-		return $terms;
-	}
-
-	public function wpseo_sitemap_exclude_taxonomy( $exclude, $tax ) {
-		if ( self::TAXONOMY_NAME === $tax ) {
-			return true;
-		}
-
-		return $exclude;
-	}
+        // phpcs:ignore WordPress.NamingConventions.ValidHookName.UseUnderscores
+        if ( apply_filters( 'wl_feature__enable__all-entity-types', WL_ALL_ENTITY_TYPES ) ) {
+            // phpcs:ignore WordPress.NamingConventions.ValidHookName.UseUnderscores
+            $args['meta_box_cb'] = apply_filters( 'wl_feature__enable__entity-types-taxonomy', true ) ? array(
+                'Wordlift_Admin_Schemaorg_Taxonomy_Metabox',
+                'render',
+            ) : false;
+        }
+
+        register_taxonomy(
+            self::TAXONOMY_NAME, // Taxonomy name.
+            Wordlift_Entity_Service::valid_entity_post_types(), // Taxonomy post types.
+            $args // Taxonomy args.
+        );
+
+        /**
+         * Register meta wl_entities_gutenberg for use in Gutenberg
+         */
+        register_meta(
+            'post',
+            'wl_entities_gutenberg',
+            array(
+                'show_in_rest' => true,
+                'single'       => true,
+                'type'         => 'string',
+            )
+        );
+
+        /**
+         * Register meta _wl_alt_label for use in Gutenberg
+         */
+        // register_meta( 'post', Wordlift_Entity_Service::ALTERNATIVE_LABEL_META_KEY, array(
+        // 'object_subtype' => '',
+        // 'show_in_rest'   => true,
+        // 'single'         => false,
+        // 'type'           => 'string',
+        // 'auth_callback'  => function () {
+        // return current_user_can( 'edit_posts' );
+        // }
+        // ) );
+
+        // see #1364: add custom fields support for CPTs that are valid entity post types in order to be able to
+        // query for synonyms in Block Editor.
+        add_filter(
+            'register_post_type_args',
+            function ( $args, $post_type ) {
+                if ( in_array( $post_type, Wordlift_Entity_Service::valid_entity_post_types(), true ) ) {
+                    if ( ! isset( $args['supports'] ) ) {
+                        $args['supports'] = array();
+                    }
+                    $args['supports'][] = 'custom-fields';
+                }
+
+                return $args;
+            },
+            10,
+            2
+        );
+
+        // Add filter to change the metabox CSS class.
+        add_filter( 'postbox_classes_entity_wl_entity_typediv', 'wl_admin_metaboxes_add_css_class' );
+
+        // Add a filter to preset the object term if none is set.
+        //
+        // DO NOT hook to `wp_get_object_terms`, because `wp_get_object_terms` returns imploded values for SQL queries.
+        //
+        // @see https://github.com/insideout10/wordlift-plugin/issues/995
+        // @since 3.23.6
+        add_filter( 'get_object_terms', array( $this, 'get_object_terms' ), 10, 4 );
+
+        /**
+         * Exclude sitemap creation for wl_entity_type taxonomy in Yoast
+         *
+         * @since 3.30.1
+         */
+        add_filter( 'wpseo_sitemap_exclude_taxonomy', array( $this, 'wpseo_sitemap_exclude_taxonomy' ), 10, 2 );
+
+    }
+
+    /**
+     * Hook to the `get_object_terms` filter.
+     *
+     * We check if our taxonomy is requested and whether a term has been returned. If no term has been returned we
+     * preset `Article` for posts/pages and 'Thing' for everything else and we query the terms again.
+     *
+     * @param array    $terms Array of terms for the given object or objects.
+     * @param int[]    $object_ids Array of object IDs for which terms were retrieved.
+     * @param string[] $taxonomies Array of taxonomy names from which terms were retrieved.
+     * @param array    $args Array of arguments for retrieving terms for the given
+     *                                object(s). See get_object_terms() for details.
+     *
+     * @return array|WP_Error
+     * @since 3.23.6
+     */
+    public function get_object_terms( $terms, $object_ids, $taxonomies, $args ) {
+        // Get our entity type.
+        $entity_type = self::TAXONOMY_NAME;
+
+        // Check if this is a query for our entity type, that no terms have been found and that we have an article
+        // term to preset in case.
+        if ( ! taxonomy_exists( $entity_type )
+             || array( $entity_type ) !== (array) $taxonomies
+             || ! empty( $terms )
+             || ! term_exists( 'article', $entity_type )
+             || ! term_exists( 'thing', $entity_type ) ) {
+
+            // Return the input value.
+            return $terms;
+        }
+
+        // Avoid nested calls in case of issues.
+        remove_filter( 'get_object_terms', array( $this, 'get_object_terms' ), 10 );
+
+        // Set the default term for all the queried object.
+        foreach ( (array) $object_ids as $object_id ) {
+            $post_type = get_post_type( $object_id );
+            if ( Wordlift_Entity_Type_Service::is_valid_entity_post_type( $post_type ) ) {
+                // Set the term to article for posts and pages, or to thing for everything else.
+                $uris = Wordlift_Entity_Type_Adapter::get_entity_types( $post_type );
+                foreach ( $uris as $uri ) {
+                    // set the uri based on post type.
+                    if ( 'http://schema.org/Article' === $uri || 'http://schema.org/Thing' === $uri ) {
+                        $uri = Wordlift_Entity_Service::TYPE_NAME === $post_type ?
+                            'http://schema.org/Thing' : 'http://schema.org/Article';
+                    }
+                    Wordlift_Entity_Type_Service::get_instance()->set( $object_id, $uri );
+                }
+            }
+        }
+
+        // Finally return the object terms.
+        $terms = wp_get_object_terms( $object_ids, $taxonomies, $args );
+
+        // Re-enable nested calls in case of issues.
+        add_filter( 'get_object_terms', array( $this, 'get_object_terms' ), 10, 4 );
+
+        return $terms;
+    }
+
+    public function wpseo_sitemap_exclude_taxonomy( $exclude, $tax ) {
+        if ( self::TAXONOMY_NAME === $tax ) {
+            return true;
+        }
+
+        return $exclude;
+    }
 
 }

Spacing +39 added lines, -39 removed lines

@@ -31,17 +31,17 @@ 
 	public function init() {
 
 		$labels = array(
-			'name'              => _x( 'Entity Types', 'taxonomy general name', 'wordlift' ),
-			'singular_name'     => _x( 'Entity Type', 'taxonomy singular name', 'wordlift' ),
-			'search_items'      => __( 'Search Entity Types', 'wordlift' ),
-			'all_items'         => __( 'All Entity Types', 'wordlift' ),
-			'parent_item'       => __( 'Parent Entity Type', 'wordlift' ),
-			'parent_item_colon' => __( 'Parent Entity Type:', 'wordlift' ),
-			'edit_item'         => __( 'Edit Entity Type', 'wordlift' ),
-			'update_item'       => __( 'Update Entity Type', 'wordlift' ),
-			'add_new_item'      => __( 'Add New Entity Type', 'wordlift' ),
-			'new_item_name'     => __( 'New Entity Type', 'wordlift' ),
-			'menu_name'         => __( 'Entity Types', 'wordlift' ),
+			'name'              => _x('Entity Types', 'taxonomy general name', 'wordlift'),
+			'singular_name'     => _x('Entity Type', 'taxonomy singular name', 'wordlift'),
+			'search_items'      => __('Search Entity Types', 'wordlift'),
+			'all_items'         => __('All Entity Types', 'wordlift'),
+			'parent_item'       => __('Parent Entity Type', 'wordlift'),
+			'parent_item_colon' => __('Parent Entity Type:', 'wordlift'),
+			'edit_item'         => __('Edit Entity Type', 'wordlift'),
+			'update_item'       => __('Update Entity Type', 'wordlift'),
+			'add_new_item'      => __('Add New Entity Type', 'wordlift'),
+			'new_item_name'     => __('New Entity Type', 'wordlift'),
+			'menu_name'         => __('Entity Types', 'wordlift'),
 		);
 
 		// Take away GUI for taxonomy editing.
@@ -61,9 +61,9 @@ 
 			'capabilities'       => $capabilities,
 			'hierarchical'       => true,
 			// phpcs:ignore WordPress.NamingConventions.ValidHookName.UseUnderscores
-			'show_admin_column'  => apply_filters( 'wl_feature__enable__entity-types-taxonomy', true ),
+			'show_admin_column'  => apply_filters('wl_feature__enable__entity-types-taxonomy', true),
 			// phpcs:ignore WordPress.NamingConventions.ValidHookName.UseUnderscores
-			'show_in_rest'       => apply_filters( 'wl_feature__enable__entity-types-taxonomy', true ),
+			'show_in_rest'       => apply_filters('wl_feature__enable__entity-types-taxonomy', true),
 			'show_in_quick_edit' => false,
 			'publicly_queryable' => false,
 		);
@@ -75,9 +75,9 @@ 
 		 * @since 3.20.0
 		 */
 		// phpcs:ignore WordPress.NamingConventions.ValidHookName.UseUnderscores
-		if ( apply_filters( 'wl_feature__enable__all-entity-types', WL_ALL_ENTITY_TYPES ) ) {
+		if (apply_filters('wl_feature__enable__all-entity-types', WL_ALL_ENTITY_TYPES)) {
 			// phpcs:ignore WordPress.NamingConventions.ValidHookName.UseUnderscores
-			$args['meta_box_cb'] = apply_filters( 'wl_feature__enable__entity-types-taxonomy', true ) ? array(
+			$args['meta_box_cb'] = apply_filters('wl_feature__enable__entity-types-taxonomy', true) ? array(
 				'Wordlift_Admin_Schemaorg_Taxonomy_Metabox',
 				'render',
 			) : false;
@@ -119,9 +119,9 @@ 
 		// query for synonyms in Block Editor.
 		add_filter(
 			'register_post_type_args',
-			function ( $args, $post_type ) {
-				if ( in_array( $post_type, Wordlift_Entity_Service::valid_entity_post_types(), true ) ) {
-					if ( ! isset( $args['supports'] ) ) {
+			function($args, $post_type) {
+				if (in_array($post_type, Wordlift_Entity_Service::valid_entity_post_types(), true)) {
+					if ( ! isset($args['supports'])) {
 						$args['supports'] = array();
 					}
 					$args['supports'][] = 'custom-fields';
@@ -134,7 +134,7 @@ 
 		);
 
 		// Add filter to change the metabox CSS class.
-		add_filter( 'postbox_classes_entity_wl_entity_typediv', 'wl_admin_metaboxes_add_css_class' );
+		add_filter('postbox_classes_entity_wl_entity_typediv', 'wl_admin_metaboxes_add_css_class');
 
 		// Add a filter to preset the object term if none is set.
 		//
@@ -142,14 +142,14 @@ 
 		//
 		// @see https://github.com/insideout10/wordlift-plugin/issues/995
 		// @since 3.23.6
-		add_filter( 'get_object_terms', array( $this, 'get_object_terms' ), 10, 4 );
+		add_filter('get_object_terms', array($this, 'get_object_terms'), 10, 4);
 
 		/**
 		 * Exclude sitemap creation for wl_entity_type taxonomy in Yoast
 		 *
 		 * @since 3.30.1
 		 */
-		add_filter( 'wpseo_sitemap_exclude_taxonomy', array( $this, 'wpseo_sitemap_exclude_taxonomy' ), 10, 2 );
+		add_filter('wpseo_sitemap_exclude_taxonomy', array($this, 'wpseo_sitemap_exclude_taxonomy'), 10, 2);
 
 	}
 
@@ -168,53 +168,53 @@ 
 	 * @return array|WP_Error
 	 * @since 3.23.6
 	 */
-	public function get_object_terms( $terms, $object_ids, $taxonomies, $args ) {
+	public function get_object_terms($terms, $object_ids, $taxonomies, $args) {
 		// Get our entity type.
 		$entity_type = self::TAXONOMY_NAME;
 
 		// Check if this is a query for our entity type, that no terms have been found and that we have an article
 		// term to preset in case.
-		if ( ! taxonomy_exists( $entity_type )
-			 || array( $entity_type ) !== (array) $taxonomies
-			 || ! empty( $terms )
-			 || ! term_exists( 'article', $entity_type )
-			 || ! term_exists( 'thing', $entity_type ) ) {
+		if ( ! taxonomy_exists($entity_type)
+			 || array($entity_type) !== (array) $taxonomies
+			 || ! empty($terms)
+			 || ! term_exists('article', $entity_type)
+			 || ! term_exists('thing', $entity_type)) {
 
 			// Return the input value.
 			return $terms;
 		}
 
 		// Avoid nested calls in case of issues.
-		remove_filter( 'get_object_terms', array( $this, 'get_object_terms' ), 10 );
+		remove_filter('get_object_terms', array($this, 'get_object_terms'), 10);
 
 		// Set the default term for all the queried object.
-		foreach ( (array) $object_ids as $object_id ) {
-			$post_type = get_post_type( $object_id );
-			if ( Wordlift_Entity_Type_Service::is_valid_entity_post_type( $post_type ) ) {
+		foreach ((array) $object_ids as $object_id) {
+			$post_type = get_post_type($object_id);
+			if (Wordlift_Entity_Type_Service::is_valid_entity_post_type($post_type)) {
 				// Set the term to article for posts and pages, or to thing for everything else.
-				$uris = Wordlift_Entity_Type_Adapter::get_entity_types( $post_type );
-				foreach ( $uris as $uri ) {
+				$uris = Wordlift_Entity_Type_Adapter::get_entity_types($post_type);
+				foreach ($uris as $uri) {
 					// set the uri based on post type.
-					if ( 'http://schema.org/Article' === $uri || 'http://schema.org/Thing' === $uri ) {
+					if ('http://schema.org/Article' === $uri || 'http://schema.org/Thing' === $uri) {
 						$uri = Wordlift_Entity_Service::TYPE_NAME === $post_type ?
 							'http://schema.org/Thing' : 'http://schema.org/Article';
 					}
-					Wordlift_Entity_Type_Service::get_instance()->set( $object_id, $uri );
+					Wordlift_Entity_Type_Service::get_instance()->set($object_id, $uri);
 				}
 			}
 		}
 
 		// Finally return the object terms.
-		$terms = wp_get_object_terms( $object_ids, $taxonomies, $args );
+		$terms = wp_get_object_terms($object_ids, $taxonomies, $args);
 
 		// Re-enable nested calls in case of issues.
-		add_filter( 'get_object_terms', array( $this, 'get_object_terms' ), 10, 4 );
+		add_filter('get_object_terms', array($this, 'get_object_terms'), 10, 4);
 
 		return $terms;
 	}
 
-	public function wpseo_sitemap_exclude_taxonomy( $exclude, $tax ) {
-		if ( self::TAXONOMY_NAME === $tax ) {
+	public function wpseo_sitemap_exclude_taxonomy($exclude, $tax) {
+		if (self::TAXONOMY_NAME === $tax) {
 			return true;
 		}
 

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

Indentation +182 added lines, -182 removed lines

@@ -16,187 +16,187 @@
  */
 class Wordlift_Attachment_Service {
 
-	/**
-	 * Create a {@link Wordlift_Attachment_Service} instance.
-	 *
-	 * @since 3.20.0
-	 */
-	protected function __construct() {
-
-	}
-
-	private static $instance = null;
-
-	/**
-	 * Get the singleton instance.
-	 *
-	 * @return \Wordlift_Attachment_Service The singleton instance.
-	 * @since 3.20.0
-	 */
-	public static function get_instance() {
-
-		if ( ! isset( self::$instance ) ) {
-			self::$instance = new self();
-		}
-
-		return self::$instance;
-	}
-
-	/**
-	 * Get an attachment ID given a URL.
-	 *
-	 * Inspired from https://wpscholar.com/blog/get-attachment-id-from-wp-image-url/
-	 *
-	 * @param string $url The attachment URL.
-	 *
-	 * @return int|false Attachment ID on success, false on failure
-	 * @since 3.10.0
-	 */
-	public function get_attachment_id( $url ) {
-
-		// Get the upload directory data, we need the base URL to check whether
-		// the URL we received is within WP.
-		$dir = wp_upload_dir();
-
-		// Get the filename, the extension is kept.
-		if ( 1 !== preg_match( "@^{$dir['baseurl']}/(.+?)(?:-\d+x\d+)?(\.\w+)$@", $url, $matches ) ) {
-			return false;
-		}
-
-		$filename = $matches[1] . $matches[2];
-
-		// The following query is CPU-intensive, we need to review it.
-		//
-		// See https://github.com/insideout10/wordlift-plugin/issues/689.
-		//
-		// Query for attachments with the specified filename.
-		$query = new WP_Query(
-			array(
-				'post_type'           => 'attachment',
-				'post_status'         => 'inherit',
-				'fields'              => 'ids',
-				'meta_query'          => array(
-					array(
-						'value'   => $filename,
-						'compare' => '=',
-						'key'     => '_wp_attached_file',
-					),
-				),
-				'posts_per_page'      => 1,
-				'ignore_sticky_posts' => true,
-			)
-		);
-
-		// If there are no posts, return.
-		if ( $query->have_posts() ) {
-			return $query->posts[0];
-			// foreach ( $query->posts as $attachment_id ) {
-			//
-			// Get the attachment metadata, we need the filename.
-			// $metadata          = wp_get_attachment_metadata( $attachment_id );
-			// $original_filename = basename( $metadata['file'] );
-			//
-			// Get the cropped filenames, or an empty array in case there are no files.
-			// $sizes_filenames = isset( $metadata['sizes'] ) ? wp_list_pluck( $metadata['sizes'], 'file' ) : array();
-			//
-			// If the provided filename matches the attachment filename (or one of its resized images), return the id.
-			// if ( $original_filename === $filename || in_array( $filename, $sizes_filenames ) ) {
-			// return $attachment_id;
-			// }
-			// }
-		}
-
-		// If we got here, we couldn't find any attachment.
-		return false;
-	}
-
-	/**
-	 * Get images embedded in the post content.
-	 *
-	 * @param string $content The post content.
-	 *
-	 * @return array An array of attachment ids.
-	 * @since 3.10.0
-	 */
-	public function get_image_embeds( $content ) {
-
-		// Go over all the images included in the post content, check if they are
-		// in the DB, and if so include them.
-		$images = array();
-		if ( false === preg_match_all( '#<img [^>]*src="([^\\">]*)"[^>]*>#', $content, $images ) ) {
-			return array();
-		}
-
-		// Map the image URLs to attachment ids.
-		$that = $this;
-		$ids  = array_map(
-			function ( $url ) use ( $that ) {
-				return $that->get_attachment_id( $url );
-			},
-			$images[1]
-		);
-
-		// Filter out not found ids (i.e. id is false).
-		return array_filter(
-			$ids,
-			function ( $item ) {
-				return false !== $item;
-			}
-		);
-	}
-
-	/**
-	 * Get images linked via the `gallery` shortcode.
-	 *
-	 * @param \WP_Post $post A {@link WP_Post} instance.
-	 *
-	 * @return array An array of attachment ids.
-	 * @since 3.10.0
-	 */
-	public function get_gallery( $post ) {
-
-		// @todo: the `gallery` shortcode has an `exclude` attribute which isn't
-		// checked at the moment.
-
-		// Prepare the return value.
-		$ids = array();
-
-		// As the above for images in galleries.
-		// Code inspired by http://wordpress.stackexchange.com/questions/80408/how-to-get-page-post-gallery-attachment-images-in-order-they-are-set-in-backend
-		$pattern = get_shortcode_regex();
-
-		if ( preg_match_all( '/' . $pattern . '/s', $post->post_content, $matches )
-			 && array_key_exists( 2, $matches )
-			 && in_array( 'gallery', $matches[2], true )
-		) {
-
-			$keys = array_keys( $matches[2], 'gallery', true );
-
-			foreach ( $keys as $key ) {
-				$atts = shortcode_parse_atts( $matches[3][ $key ] );
-
-				if ( is_array( $atts ) && array_key_exists( 'ids', $atts ) ) {
-					// gallery images insert explicitly by their ids.
-
-					foreach ( explode( ',', $atts['ids'] ) as $attachment_id ) {
-						// Since we do not check for actual image existence
-						// when generating the json content, check it now.
-						if ( wp_get_attachment_image_src( $attachment_id, 'full' ) ) {
-							$ids[ $attachment_id ] = true;
-						}
-					}
-				} else {
-					// gallery shortcode with no ids uses all the images
-					// attached to the post.
-					$images = get_attached_media( 'image', $post->ID );
-					foreach ( $images as $attachment ) {
-						$ids[ $attachment->ID ] = true;
-					}
-				}
-			}
-		}
-
-		return array_keys( $ids );
-	}
+    /**
+     * Create a {@link Wordlift_Attachment_Service} instance.
+     *
+     * @since 3.20.0
+     */
+    protected function __construct() {
+
+    }
+
+    private static $instance = null;
+
+    /**
+     * Get the singleton instance.
+     *
+     * @return \Wordlift_Attachment_Service The singleton instance.
+     * @since 3.20.0
+     */
+    public static function get_instance() {
+
+        if ( ! isset( self::$instance ) ) {
+            self::$instance = new self();
+        }
+
+        return self::$instance;
+    }
+
+    /**
+     * Get an attachment ID given a URL.
+     *
+     * Inspired from https://wpscholar.com/blog/get-attachment-id-from-wp-image-url/
+     *
+     * @param string $url The attachment URL.
+     *
+     * @return int|false Attachment ID on success, false on failure
+     * @since 3.10.0
+     */
+    public function get_attachment_id( $url ) {
+
+        // Get the upload directory data, we need the base URL to check whether
+        // the URL we received is within WP.
+        $dir = wp_upload_dir();
+
+        // Get the filename, the extension is kept.
+        if ( 1 !== preg_match( "@^{$dir['baseurl']}/(.+?)(?:-\d+x\d+)?(\.\w+)$@", $url, $matches ) ) {
+            return false;
+        }
+
+        $filename = $matches[1] . $matches[2];
+
+        // The following query is CPU-intensive, we need to review it.
+        //
+        // See https://github.com/insideout10/wordlift-plugin/issues/689.
+        //
+        // Query for attachments with the specified filename.
+        $query = new WP_Query(
+            array(
+                'post_type'           => 'attachment',
+                'post_status'         => 'inherit',
+                'fields'              => 'ids',
+                'meta_query'          => array(
+                    array(
+                        'value'   => $filename,
+                        'compare' => '=',
+                        'key'     => '_wp_attached_file',
+                    ),
+                ),
+                'posts_per_page'      => 1,
+                'ignore_sticky_posts' => true,
+            )
+        );
+
+        // If there are no posts, return.
+        if ( $query->have_posts() ) {
+            return $query->posts[0];
+            // foreach ( $query->posts as $attachment_id ) {
+            //
+            // Get the attachment metadata, we need the filename.
+            // $metadata          = wp_get_attachment_metadata( $attachment_id );
+            // $original_filename = basename( $metadata['file'] );
+            //
+            // Get the cropped filenames, or an empty array in case there are no files.
+            // $sizes_filenames = isset( $metadata['sizes'] ) ? wp_list_pluck( $metadata['sizes'], 'file' ) : array();
+            //
+            // If the provided filename matches the attachment filename (or one of its resized images), return the id.
+            // if ( $original_filename === $filename || in_array( $filename, $sizes_filenames ) ) {
+            // return $attachment_id;
+            // }
+            // }
+        }
+
+        // If we got here, we couldn't find any attachment.
+        return false;
+    }
+
+    /**
+     * Get images embedded in the post content.
+     *
+     * @param string $content The post content.
+     *
+     * @return array An array of attachment ids.
+     * @since 3.10.0
+     */
+    public function get_image_embeds( $content ) {
+
+        // Go over all the images included in the post content, check if they are
+        // in the DB, and if so include them.
+        $images = array();
+        if ( false === preg_match_all( '#<img [^>]*src="([^\\">]*)"[^>]*>#', $content, $images ) ) {
+            return array();
+        }
+
+        // Map the image URLs to attachment ids.
+        $that = $this;
+        $ids  = array_map(
+            function ( $url ) use ( $that ) {
+                return $that->get_attachment_id( $url );
+            },
+            $images[1]
+        );
+
+        // Filter out not found ids (i.e. id is false).
+        return array_filter(
+            $ids,
+            function ( $item ) {
+                return false !== $item;
+            }
+        );
+    }
+
+    /**
+     * Get images linked via the `gallery` shortcode.
+     *
+     * @param \WP_Post $post A {@link WP_Post} instance.
+     *
+     * @return array An array of attachment ids.
+     * @since 3.10.0
+     */
+    public function get_gallery( $post ) {
+
+        // @todo: the `gallery` shortcode has an `exclude` attribute which isn't
+        // checked at the moment.
+
+        // Prepare the return value.
+        $ids = array();
+
+        // As the above for images in galleries.
+        // Code inspired by http://wordpress.stackexchange.com/questions/80408/how-to-get-page-post-gallery-attachment-images-in-order-they-are-set-in-backend
+        $pattern = get_shortcode_regex();
+
+        if ( preg_match_all( '/' . $pattern . '/s', $post->post_content, $matches )
+             && array_key_exists( 2, $matches )
+             && in_array( 'gallery', $matches[2], true )
+        ) {
+
+            $keys = array_keys( $matches[2], 'gallery', true );
+
+            foreach ( $keys as $key ) {
+                $atts = shortcode_parse_atts( $matches[3][ $key ] );
+
+                if ( is_array( $atts ) && array_key_exists( 'ids', $atts ) ) {
+                    // gallery images insert explicitly by their ids.
+
+                    foreach ( explode( ',', $atts['ids'] ) as $attachment_id ) {
+                        // Since we do not check for actual image existence
+                        // when generating the json content, check it now.
+                        if ( wp_get_attachment_image_src( $attachment_id, 'full' ) ) {
+                            $ids[ $attachment_id ] = true;
+                        }
+                    }
+                } else {
+                    // gallery shortcode with no ids uses all the images
+                    // attached to the post.
+                    $images = get_attached_media( 'image', $post->ID );
+                    foreach ( $images as $attachment ) {
+                        $ids[ $attachment->ID ] = true;
+                    }
+                }
+            }
+        }
+
+        return array_keys( $ids );
+    }
 
 }

Spacing +25 added lines, -25 removed lines

@@ -35,7 +35,7 @@ 
 	 */
 	public static function get_instance() {
 
-		if ( ! isset( self::$instance ) ) {
+		if ( ! isset(self::$instance)) {
 			self::$instance = new self();
 		}
 
@@ -52,18 +52,18 @@ 
 	 * @return int|false Attachment ID on success, false on failure
 	 * @since 3.10.0
 	 */
-	public function get_attachment_id( $url ) {
+	public function get_attachment_id($url) {
 
 		// Get the upload directory data, we need the base URL to check whether
 		// the URL we received is within WP.
 		$dir = wp_upload_dir();
 
 		// Get the filename, the extension is kept.
-		if ( 1 !== preg_match( "@^{$dir['baseurl']}/(.+?)(?:-\d+x\d+)?(\.\w+)$@", $url, $matches ) ) {
+		if (1 !== preg_match("@^{$dir['baseurl']}/(.+?)(?:-\d+x\d+)?(\.\w+)$@", $url, $matches)) {
 			return false;
 		}
 
-		$filename = $matches[1] . $matches[2];
+		$filename = $matches[1].$matches[2];
 
 		// The following query is CPU-intensive, we need to review it.
 		//
@@ -88,7 +88,7 @@ 
 		);
 
 		// If there are no posts, return.
-		if ( $query->have_posts() ) {
+		if ($query->have_posts()) {
 			return $query->posts[0];
 			// foreach ( $query->posts as $attachment_id ) {
 			//
@@ -118,20 +118,20 @@ 
 	 * @return array An array of attachment ids.
 	 * @since 3.10.0
 	 */
-	public function get_image_embeds( $content ) {
+	public function get_image_embeds($content) {
 
 		// Go over all the images included in the post content, check if they are
 		// in the DB, and if so include them.
 		$images = array();
-		if ( false === preg_match_all( '#<img [^>]*src="([^\\">]*)"[^>]*>#', $content, $images ) ) {
+		if (false === preg_match_all('#<img [^>]*src="([^\\">]*)"[^>]*>#', $content, $images)) {
 			return array();
 		}
 
 		// Map the image URLs to attachment ids.
 		$that = $this;
 		$ids  = array_map(
-			function ( $url ) use ( $that ) {
-				return $that->get_attachment_id( $url );
+			function($url) use ($that) {
+				return $that->get_attachment_id($url);
 			},
 			$images[1]
 		);
@@ -139,7 +139,7 @@ 
 		// Filter out not found ids (i.e. id is false).
 		return array_filter(
 			$ids,
-			function ( $item ) {
+			function($item) {
 				return false !== $item;
 			}
 		);
@@ -153,7 +153,7 @@ 
 	 * @return array An array of attachment ids.
 	 * @since 3.10.0
 	 */
-	public function get_gallery( $post ) {
+	public function get_gallery($post) {
 
 		// @todo: the `gallery` shortcode has an `exclude` attribute which isn't
 		// checked at the moment.
@@ -165,38 +165,38 @@ 
 		// Code inspired by http://wordpress.stackexchange.com/questions/80408/how-to-get-page-post-gallery-attachment-images-in-order-they-are-set-in-backend
 		$pattern = get_shortcode_regex();
 
-		if ( preg_match_all( '/' . $pattern . '/s', $post->post_content, $matches )
-			 && array_key_exists( 2, $matches )
-			 && in_array( 'gallery', $matches[2], true )
+		if (preg_match_all('/'.$pattern.'/s', $post->post_content, $matches)
+			 && array_key_exists(2, $matches)
+			 && in_array('gallery', $matches[2], true)
 		) {
 
-			$keys = array_keys( $matches[2], 'gallery', true );
+			$keys = array_keys($matches[2], 'gallery', true);
 
-			foreach ( $keys as $key ) {
-				$atts = shortcode_parse_atts( $matches[3][ $key ] );
+			foreach ($keys as $key) {
+				$atts = shortcode_parse_atts($matches[3][$key]);
 
-				if ( is_array( $atts ) && array_key_exists( 'ids', $atts ) ) {
+				if (is_array($atts) && array_key_exists('ids', $atts)) {
 					// gallery images insert explicitly by their ids.
 
-					foreach ( explode( ',', $atts['ids'] ) as $attachment_id ) {
+					foreach (explode(',', $atts['ids']) as $attachment_id) {
 						// Since we do not check for actual image existence
 						// when generating the json content, check it now.
-						if ( wp_get_attachment_image_src( $attachment_id, 'full' ) ) {
-							$ids[ $attachment_id ] = true;
+						if (wp_get_attachment_image_src($attachment_id, 'full')) {
+							$ids[$attachment_id] = true;
 						}
 					}
 				} else {
 					// gallery shortcode with no ids uses all the images
 					// attached to the post.
-					$images = get_attached_media( 'image', $post->ID );
-					foreach ( $images as $attachment ) {
-						$ids[ $attachment->ID ] = true;
+					$images = get_attached_media('image', $post->ID);
+					foreach ($images as $attachment) {
+						$ids[$attachment->ID] = true;
 					}
 				}
 			}
 		}
 
-		return array_keys( $ids );
+		return array_keys($ids);
 	}
 
 }

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

Indentation +264 added lines, -264 removed lines

@@ -17,274 +17,274 @@
  */
 class Wordlift_Jsonld_Service {
 
-	/**
-	 * A {@link Wordlift_Entity_Service} instance.
-	 *
-	 * @since  3.8.0
-	 * @access private
-	 * @var Wordlift_Entity_Service $entity_service A {@link Wordlift_Entity_Service} instance.
-	 */
-	private $entity_service;
-
-	/**
-	 * A {@link Wordlift_Term_JsonLd_Adapter} instance.
-	 *
-	 * @since  3.32.0
-	 * @access private
-	 * @var Wordlift_Term_JsonLd_Adapter $entity_service A {@link Wordlift_Term_JsonLd_Adapter} instance.
-	 */
-	private $term_jsonld_adapter;
-
-	/**
-	 * A {@link Wordlift_Post_Converter} instance.
-	 *
-	 * @since  3.8.0
-	 * @access private
-	 * @var \Wordlift_Post_Converter A {@link Wordlift_Post_Converter} instance.
-	 */
-	private $converter;
-
-	/**
-	 * A {@link Wordlift_Website_Jsonld_Converter} instance.
-	 *
-	 * @since  3.14.0
-	 * @access private
-	 * @var \Wordlift_Website_Jsonld_Converter A {@link Wordlift_Website_Jsonld_Converter} instance.
-	 */
-	private $website_converter;
-
-	/**
-	 * The singleton instance for the JSON-LD service.
-	 *
-	 * @since 3.15.1
-	 *
-	 * @var \Wordlift_Jsonld_Service $instance The singleton instance for the JSON-LD service.
-	 */
-	private static $instance;
-
-	/**
-	 * Create a JSON-LD service.
-	 *
-	 * @param \Wordlift_Entity_Service           $entity_service A {@link Wordlift_Entity_Service} instance.
-	 * @param \Wordlift_Post_Converter           $converter A {@link Wordlift_Uri_To_Jsonld_Converter} instance.
-	 * @param \Wordlift_Website_Jsonld_Converter $website_converter A {@link Wordlift_Website_Jsonld_Converter} instance.
-	 * @param \Wordlift_Term_JsonLd_Adapter      $term_jsonld_adapter
-	 *
-	 * @since 3.8.0
-	 */
-	public function __construct( $entity_service, $converter, $website_converter, $term_jsonld_adapter ) {
-
-		$this->entity_service      = $entity_service;
-		$this->converter           = $converter;
-		$this->website_converter   = $website_converter;
-		$this->term_jsonld_adapter = $term_jsonld_adapter;
-		self::$instance            = $this;
-
-	}
-
-	/**
-	 * Get the singleton instance for the JSON-LD service.
-	 *
-	 * @return \Wordlift_Jsonld_Service The singleton instance for the JSON-LD service.
-	 * @since 3.15.1
-	 */
-	public static function get_instance() {
-
-		return self::$instance;
-	}
-
-	/**
-	 * Process calls to the AJAX 'wl_jsonld' endpoint.
-	 *
-	 * @since 3.8.0
-	 */
-	public function get() {
-		// Clear the buffer to be sure someone doesn't mess with our response.
-		//
-		// See https://github.com/insideout10/wordlift-plugin/issues/406.
-		// See https://codex.wordpress.org/AJAX_in_Plugins.
+    /**
+     * A {@link Wordlift_Entity_Service} instance.
+     *
+     * @since  3.8.0
+     * @access private
+     * @var Wordlift_Entity_Service $entity_service A {@link Wordlift_Entity_Service} instance.
+     */
+    private $entity_service;
+
+    /**
+     * A {@link Wordlift_Term_JsonLd_Adapter} instance.
+     *
+     * @since  3.32.0
+     * @access private
+     * @var Wordlift_Term_JsonLd_Adapter $entity_service A {@link Wordlift_Term_JsonLd_Adapter} instance.
+     */
+    private $term_jsonld_adapter;
+
+    /**
+     * A {@link Wordlift_Post_Converter} instance.
+     *
+     * @since  3.8.0
+     * @access private
+     * @var \Wordlift_Post_Converter A {@link Wordlift_Post_Converter} instance.
+     */
+    private $converter;
+
+    /**
+     * A {@link Wordlift_Website_Jsonld_Converter} instance.
+     *
+     * @since  3.14.0
+     * @access private
+     * @var \Wordlift_Website_Jsonld_Converter A {@link Wordlift_Website_Jsonld_Converter} instance.
+     */
+    private $website_converter;
+
+    /**
+     * The singleton instance for the JSON-LD service.
+     *
+     * @since 3.15.1
+     *
+     * @var \Wordlift_Jsonld_Service $instance The singleton instance for the JSON-LD service.
+     */
+    private static $instance;
+
+    /**
+     * Create a JSON-LD service.
+     *
+     * @param \Wordlift_Entity_Service           $entity_service A {@link Wordlift_Entity_Service} instance.
+     * @param \Wordlift_Post_Converter           $converter A {@link Wordlift_Uri_To_Jsonld_Converter} instance.
+     * @param \Wordlift_Website_Jsonld_Converter $website_converter A {@link Wordlift_Website_Jsonld_Converter} instance.
+     * @param \Wordlift_Term_JsonLd_Adapter      $term_jsonld_adapter
+     *
+     * @since 3.8.0
+     */
+    public function __construct( $entity_service, $converter, $website_converter, $term_jsonld_adapter ) {
+
+        $this->entity_service      = $entity_service;
+        $this->converter           = $converter;
+        $this->website_converter   = $website_converter;
+        $this->term_jsonld_adapter = $term_jsonld_adapter;
+        self::$instance            = $this;
+
+    }
+
+    /**
+     * Get the singleton instance for the JSON-LD service.
+     *
+     * @return \Wordlift_Jsonld_Service The singleton instance for the JSON-LD service.
+     * @since 3.15.1
+     */
+    public static function get_instance() {
+
+        return self::$instance;
+    }
+
+    /**
+     * Process calls to the AJAX 'wl_jsonld' endpoint.
+     *
+     * @since 3.8.0
+     */
+    public function get() {
+        // Clear the buffer to be sure someone doesn't mess with our response.
+        //
+        // See https://github.com/insideout10/wordlift-plugin/issues/406.
+        // See https://codex.wordpress.org/AJAX_in_Plugins.
         // phpcs:ignore WordPress.PHP.NoSilencedErrors.Discouraged
-		@ob_clean();
-
-		// Get the parameter from the request.
-		$is_homepage = isset( $_REQUEST['homepage'] ); //phpcs:ignore WordPress.Security.NonceVerification.Recommended
-		$post_id     = isset( $_REQUEST['id'] ) && is_numeric( $_REQUEST['id'] ) ? intval( $_REQUEST['id'] ) : null; //phpcs:ignore WordPress.Security.NonceVerification.Recommended
-
-		// Send the generated JSON-LD.
-		$this->send_jsonld( $this->get_jsonld( $is_homepage, $post_id ) );
-
-	}
-
-	/**
-	 * A close of WP's own `wp_send_json` function which uses `application/ld+json` as content type.
-	 *
-	 * @param mixed $response Variable (usually an array or object) to encode as JSON,
-	 *                           then print and die.
-	 *
-	 * @since 3.18.5
-	 */
-	private function send_jsonld( $response ) {
+        @ob_clean();
+
+        // Get the parameter from the request.
+        $is_homepage = isset( $_REQUEST['homepage'] ); //phpcs:ignore WordPress.Security.NonceVerification.Recommended
+        $post_id     = isset( $_REQUEST['id'] ) && is_numeric( $_REQUEST['id'] ) ? intval( $_REQUEST['id'] ) : null; //phpcs:ignore WordPress.Security.NonceVerification.Recommended
+
+        // Send the generated JSON-LD.
+        $this->send_jsonld( $this->get_jsonld( $is_homepage, $post_id ) );
+
+    }
+
+    /**
+     * A close of WP's own `wp_send_json` function which uses `application/ld+json` as content type.
+     *
+     * @param mixed $response Variable (usually an array or object) to encode as JSON,
+     *                           then print and die.
+     *
+     * @since 3.18.5
+     */
+    private function send_jsonld( $response ) {
         // phpcs:ignore WordPress.PHP.NoSilencedErrors.Discouraged
-		@header( 'Content-Type: application/ld+json; charset=' . get_option( 'blog_charset' ) );
-		echo wp_json_encode( $response );
-		if ( apply_filters( 'wp_doing_ajax', defined( 'DOING_AJAX' ) && DOING_AJAX ) ) {
-			wp_die();
-		} else {
-			die;
-		}
-	}
-
-	/**
-	 * Get the JSON-LD.
-	 *
-	 * @param bool     $is_homepage Whether the JSON-LD for the homepage is being requested.
-	 * @param int|null $post_id The JSON-LD for the specified {@link WP_Post} id.
-	 * @param int      $context A context for the JSON-LD generation, valid values in Jsonld_Context_Enum.
-	 *
-	 * @return array A JSON-LD structure.
-	 * @since 3.15.1
-	 */
-	public function get_jsonld( $is_homepage = false, $post_id = null, $context = Jsonld_Context_Enum::UNKNOWN ) {
-
-		// Tell NewRelic to ignore us, otherwise NewRelic customers might receive
-		// e-mails with a low apdex score.
-		//
-		// See https://github.com/insideout10/wordlift-plugin/issues/521
-		Wordlift_NewRelic_Adapter::ignore_apdex();
-
-		// Switch to Website converter if is home page.
-		if ( $is_homepage ) {
-			/**
-			 * Filter: 'wordlift_disable_website_json_ld' - Allow disabling of the json+ld output.
-			 *
-			 * @since  3.14.0
-			 * @api    bool $display_search Whether or not to display json+ld search on the frontend.
-			 */
-			if ( apply_filters( 'wordlift_disable_website_json_ld', false ) ) {
-				return array();
-			}
-
-			// Set a reference to the website_converter.
-			$website_converter = $this->website_converter;
-
-			// Send JSON-LD.
-			return $website_converter->create_schema();
-		}
-
-		// If no id has been provided return an empty array.
-		if ( ! isset( $post_id ) ) {
-			return array();
-		}
-
-		// An array of references which is captured when converting an URI to a
-		// json which we gather to further expand our json-ld.
-		$references       = array();
-		$references_infos = array();
-
-		// Set a reference to the entity_to_jsonld_converter to use in the closures.
-		$entity_to_jsonld_converter = $this->converter;
-
-		$jsonld = array( $entity_to_jsonld_converter->convert( $post_id, $references, $references_infos ) );
-
-		$that                       = $this;
-		$expanded_references_jsonld = array_map(
-			function ( $item ) use ( $context, $entity_to_jsonld_converter, &$references_infos, $that ) {
-				// "2nd level properties" may not output here, e.g. a post
-				// mentioning an event, located in a place: the place is referenced
-				// via the `@id` but no other properties are loaded.
-				$ignored = array();
-				if ( $item instanceof Term_Reference ) {
+        @header( 'Content-Type: application/ld+json; charset=' . get_option( 'blog_charset' ) );
+        echo wp_json_encode( $response );
+        if ( apply_filters( 'wp_doing_ajax', defined( 'DOING_AJAX' ) && DOING_AJAX ) ) {
+            wp_die();
+        } else {
+            die;
+        }
+    }
+
+    /**
+     * Get the JSON-LD.
+     *
+     * @param bool     $is_homepage Whether the JSON-LD for the homepage is being requested.
+     * @param int|null $post_id The JSON-LD for the specified {@link WP_Post} id.
+     * @param int      $context A context for the JSON-LD generation, valid values in Jsonld_Context_Enum.
+     *
+     * @return array A JSON-LD structure.
+     * @since 3.15.1
+     */
+    public function get_jsonld( $is_homepage = false, $post_id = null, $context = Jsonld_Context_Enum::UNKNOWN ) {
+
+        // Tell NewRelic to ignore us, otherwise NewRelic customers might receive
+        // e-mails with a low apdex score.
+        //
+        // See https://github.com/insideout10/wordlift-plugin/issues/521
+        Wordlift_NewRelic_Adapter::ignore_apdex();
+
+        // Switch to Website converter if is home page.
+        if ( $is_homepage ) {
+            /**
+             * Filter: 'wordlift_disable_website_json_ld' - Allow disabling of the json+ld output.
+             *
+             * @since  3.14.0
+             * @api    bool $display_search Whether or not to display json+ld search on the frontend.
+             */
+            if ( apply_filters( 'wordlift_disable_website_json_ld', false ) ) {
+                return array();
+            }
+
+            // Set a reference to the website_converter.
+            $website_converter = $this->website_converter;
+
+            // Send JSON-LD.
+            return $website_converter->create_schema();
+        }
+
+        // If no id has been provided return an empty array.
+        if ( ! isset( $post_id ) ) {
+            return array();
+        }
+
+        // An array of references which is captured when converting an URI to a
+        // json which we gather to further expand our json-ld.
+        $references       = array();
+        $references_infos = array();
+
+        // Set a reference to the entity_to_jsonld_converter to use in the closures.
+        $entity_to_jsonld_converter = $this->converter;
+
+        $jsonld = array( $entity_to_jsonld_converter->convert( $post_id, $references, $references_infos ) );
+
+        $that                       = $this;
+        $expanded_references_jsonld = array_map(
+            function ( $item ) use ( $context, $entity_to_jsonld_converter, &$references_infos, $that ) {
+                // "2nd level properties" may not output here, e.g. a post
+                // mentioning an event, located in a place: the place is referenced
+                // via the `@id` but no other properties are loaded.
+                $ignored = array();
+                if ( $item instanceof Term_Reference ) {
                     // phpcs:ignore VariableAnalysis.CodeAnalysis.VariableAnalysis.UndefinedVariable
-					  $term_jsonld = $that->term_jsonld_adapter->get( $item->get_id(), $context );
-
-					  // For term references, we publish a jsonld array on the page, use only the first item.
-					  return count( $term_jsonld ) > 0 ? $term_jsonld[0] : false;
-				} elseif ( $item instanceof Post_Reference ) {
-					$item = $item->get_id();
-				}
-
-				// phpcs:ignore VariableAnalysis.CodeAnalysis.VariableAnalysis.UndefinedVariable
-				return $entity_to_jsonld_converter->convert( $item, $ignored, $references_infos );
-			},
-			array_unique( $references )
-		);
-
-		// Convert each URI to a JSON-LD array, while gathering referenced entities.
-		// in the references array.
-		$jsonld = array_merge(
-			$jsonld,
-			// Convert each URI in the references array to JSON-LD. We don't output
-			// entities already output above (hence the array_diff).
-			array_filter( $expanded_references_jsonld )
-		);
-
-		$required_references = array_filter(
-			$references_infos,
-			function ( $item ) use ( $references ) {
-
-				return isset( $item['reference'] ) &&
-				   // Check that the reference is required
-				   $item['reference']->get_required() &&
-				   // Check that the reference isn't being output already.
-				   ! in_array( $item['reference']->get_id(), $references, true );
-			}
-		);
-
-		$jsonld = array_merge(
-			$jsonld,
-			array_filter(
-				array_map(
-					function ( $item ) use ( $references, $entity_to_jsonld_converter ) {
-
-						if ( ! isset( $item['reference'] ) ) {
-							  return null;
-						}
-
-						$post_id = $item['reference']->get_id();
-						if ( in_array( $post_id, $references, true ) ) {
-							return null;
-						}
-
-						$references[] = $post_id;
-
-						return $entity_to_jsonld_converter->convert( $post_id, $references );
-					},
-					$required_references
-				)
-			)
-		);
-
-		/**
-		 * Filter name: wl_after_get_jsonld
-		 *
-		 * @return array
-		 * @since 3.27.2
-		 * @var $jsonld array The final jsonld before outputting to page.
-		 * @var $post_id int The post id for which the jsonld is generated.
-		 */
-		$jsonld = apply_filters( 'wl_after_get_jsonld', $jsonld, $post_id, $context );
-
-		return $jsonld;
-	}
-
-	/**
-	 * Write the JSON-LD in the head.
-	 *
-	 * This function isn't actually used, but may be used to quickly enable writing the JSON-LD synchronously to the
-	 * document head, using the `wp_head` hook.
-	 *
-	 * @since 3.18.5
-	 */
-	public function wp_head() {
-
-		// Determine whether this is the home page or whether we're displaying a single post.
-		$is_homepage = is_home() || is_front_page();
-		$post_id     = is_singular() ? get_the_ID() : null;
-
-		$jsonld = wp_json_encode( $this->get_jsonld( $is_homepage, $post_id, Jsonld_Context_Enum::PAGE ) );
-		?>
+                        $term_jsonld = $that->term_jsonld_adapter->get( $item->get_id(), $context );
+
+                        // For term references, we publish a jsonld array on the page, use only the first item.
+                        return count( $term_jsonld ) > 0 ? $term_jsonld[0] : false;
+                } elseif ( $item instanceof Post_Reference ) {
+                    $item = $item->get_id();
+                }
+
+                // phpcs:ignore VariableAnalysis.CodeAnalysis.VariableAnalysis.UndefinedVariable
+                return $entity_to_jsonld_converter->convert( $item, $ignored, $references_infos );
+            },
+            array_unique( $references )
+        );
+
+        // Convert each URI to a JSON-LD array, while gathering referenced entities.
+        // in the references array.
+        $jsonld = array_merge(
+            $jsonld,
+            // Convert each URI in the references array to JSON-LD. We don't output
+            // entities already output above (hence the array_diff).
+            array_filter( $expanded_references_jsonld )
+        );
+
+        $required_references = array_filter(
+            $references_infos,
+            function ( $item ) use ( $references ) {
+
+                return isset( $item['reference'] ) &&
+                   // Check that the reference is required
+                   $item['reference']->get_required() &&
+                   // Check that the reference isn't being output already.
+                   ! in_array( $item['reference']->get_id(), $references, true );
+            }
+        );
+
+        $jsonld = array_merge(
+            $jsonld,
+            array_filter(
+                array_map(
+                    function ( $item ) use ( $references, $entity_to_jsonld_converter ) {
+
+                        if ( ! isset( $item['reference'] ) ) {
+                                return null;
+                        }
+
+                        $post_id = $item['reference']->get_id();
+                        if ( in_array( $post_id, $references, true ) ) {
+                            return null;
+                        }
+
+                        $references[] = $post_id;
+
+                        return $entity_to_jsonld_converter->convert( $post_id, $references );
+                    },
+                    $required_references
+                )
+            )
+        );
+
+        /**
+         * Filter name: wl_after_get_jsonld
+         *
+         * @return array
+         * @since 3.27.2
+         * @var $jsonld array The final jsonld before outputting to page.
+         * @var $post_id int The post id for which the jsonld is generated.
+         */
+        $jsonld = apply_filters( 'wl_after_get_jsonld', $jsonld, $post_id, $context );
+
+        return $jsonld;
+    }
+
+    /**
+     * Write the JSON-LD in the head.
+     *
+     * This function isn't actually used, but may be used to quickly enable writing the JSON-LD synchronously to the
+     * document head, using the `wp_head` hook.
+     *
+     * @since 3.18.5
+     */
+    public function wp_head() {
+
+        // Determine whether this is the home page or whether we're displaying a single post.
+        $is_homepage = is_home() || is_front_page();
+        $post_id     = is_singular() ? get_the_ID() : null;
+
+        $jsonld = wp_json_encode( $this->get_jsonld( $is_homepage, $post_id, Jsonld_Context_Enum::PAGE ) );
+        ?>
 		<script type="application/ld+json"><?php echo esc_html( $jsonld ); ?></script>
 													  <?php
-	}
+    }
 
 }

Spacing +31 added lines, -31 removed lines

@@ -72,7 +72,7 @@ 
 	 *
 	 * @since 3.8.0
 	 */
-	public function __construct( $entity_service, $converter, $website_converter, $term_jsonld_adapter ) {
+	public function __construct($entity_service, $converter, $website_converter, $term_jsonld_adapter) {
 
 		$this->entity_service      = $entity_service;
 		$this->converter           = $converter;
@@ -107,11 +107,11 @@ 
 		@ob_clean();
 
 		// Get the parameter from the request.
-		$is_homepage = isset( $_REQUEST['homepage'] ); //phpcs:ignore WordPress.Security.NonceVerification.Recommended
-		$post_id     = isset( $_REQUEST['id'] ) && is_numeric( $_REQUEST['id'] ) ? intval( $_REQUEST['id'] ) : null; //phpcs:ignore WordPress.Security.NonceVerification.Recommended
+		$is_homepage = isset($_REQUEST['homepage']); //phpcs:ignore WordPress.Security.NonceVerification.Recommended
+		$post_id     = isset($_REQUEST['id']) && is_numeric($_REQUEST['id']) ? intval($_REQUEST['id']) : null; //phpcs:ignore WordPress.Security.NonceVerification.Recommended
 
 		// Send the generated JSON-LD.
-		$this->send_jsonld( $this->get_jsonld( $is_homepage, $post_id ) );
+		$this->send_jsonld($this->get_jsonld($is_homepage, $post_id));
 
 	}
 
@@ -123,11 +123,11 @@ 
 	 *
 	 * @since 3.18.5
 	 */
-	private function send_jsonld( $response ) {
+	private function send_jsonld($response) {
         // phpcs:ignore WordPress.PHP.NoSilencedErrors.Discouraged
-		@header( 'Content-Type: application/ld+json; charset=' . get_option( 'blog_charset' ) );
-		echo wp_json_encode( $response );
-		if ( apply_filters( 'wp_doing_ajax', defined( 'DOING_AJAX' ) && DOING_AJAX ) ) {
+		@header('Content-Type: application/ld+json; charset='.get_option('blog_charset'));
+		echo wp_json_encode($response);
+		if (apply_filters('wp_doing_ajax', defined('DOING_AJAX') && DOING_AJAX)) {
 			wp_die();
 		} else {
 			die;
@@ -144,7 +144,7 @@ 
 	 * @return array A JSON-LD structure.
 	 * @since 3.15.1
 	 */
-	public function get_jsonld( $is_homepage = false, $post_id = null, $context = Jsonld_Context_Enum::UNKNOWN ) {
+	public function get_jsonld($is_homepage = false, $post_id = null, $context = Jsonld_Context_Enum::UNKNOWN) {
 
 		// Tell NewRelic to ignore us, otherwise NewRelic customers might receive
 		// e-mails with a low apdex score.
@@ -153,14 +153,14 @@ 
 		Wordlift_NewRelic_Adapter::ignore_apdex();
 
 		// Switch to Website converter if is home page.
-		if ( $is_homepage ) {
+		if ($is_homepage) {
 			/**
 			 * Filter: 'wordlift_disable_website_json_ld' - Allow disabling of the json+ld output.
 			 *
 			 * @since  3.14.0
 			 * @api    bool $display_search Whether or not to display json+ld search on the frontend.
 			 */
-			if ( apply_filters( 'wordlift_disable_website_json_ld', false ) ) {
+			if (apply_filters('wordlift_disable_website_json_ld', false)) {
 				return array();
 			}
 
@@ -172,7 +172,7 @@ 
 		}
 
 		// If no id has been provided return an empty array.
-		if ( ! isset( $post_id ) ) {
+		if ( ! isset($post_id)) {
 			return array();
 		}
 
@@ -184,29 +184,29 @@ 
 		// Set a reference to the entity_to_jsonld_converter to use in the closures.
 		$entity_to_jsonld_converter = $this->converter;
 
-		$jsonld = array( $entity_to_jsonld_converter->convert( $post_id, $references, $references_infos ) );
+		$jsonld = array($entity_to_jsonld_converter->convert($post_id, $references, $references_infos));
 
 		$that                       = $this;
 		$expanded_references_jsonld = array_map(
-			function ( $item ) use ( $context, $entity_to_jsonld_converter, &$references_infos, $that ) {
+			function($item) use ($context, $entity_to_jsonld_converter, &$references_infos, $that) {
 				// "2nd level properties" may not output here, e.g. a post
 				// mentioning an event, located in a place: the place is referenced
 				// via the `@id` but no other properties are loaded.
 				$ignored = array();
-				if ( $item instanceof Term_Reference ) {
+				if ($item instanceof Term_Reference) {
                     // phpcs:ignore VariableAnalysis.CodeAnalysis.VariableAnalysis.UndefinedVariable
-					  $term_jsonld = $that->term_jsonld_adapter->get( $item->get_id(), $context );
+					  $term_jsonld = $that->term_jsonld_adapter->get($item->get_id(), $context);
 
 					  // For term references, we publish a jsonld array on the page, use only the first item.
-					  return count( $term_jsonld ) > 0 ? $term_jsonld[0] : false;
-				} elseif ( $item instanceof Post_Reference ) {
+					  return count($term_jsonld) > 0 ? $term_jsonld[0] : false;
+				} elseif ($item instanceof Post_Reference) {
 					$item = $item->get_id();
 				}
 
 				// phpcs:ignore VariableAnalysis.CodeAnalysis.VariableAnalysis.UndefinedVariable
-				return $entity_to_jsonld_converter->convert( $item, $ignored, $references_infos );
+				return $entity_to_jsonld_converter->convert($item, $ignored, $references_infos);
 			},
-			array_unique( $references )
+			array_unique($references)
 		);
 
 		// Convert each URI to a JSON-LD array, while gathering referenced entities.
@@ -215,18 +215,18 @@ 
 			$jsonld,
 			// Convert each URI in the references array to JSON-LD. We don't output
 			// entities already output above (hence the array_diff).
-			array_filter( $expanded_references_jsonld )
+			array_filter($expanded_references_jsonld)
 		);
 
 		$required_references = array_filter(
 			$references_infos,
-			function ( $item ) use ( $references ) {
+			function($item) use ($references) {
 
-				return isset( $item['reference'] ) &&
+				return isset($item['reference']) &&
 				   // Check that the reference is required
 				   $item['reference']->get_required() &&
 				   // Check that the reference isn't being output already.
-				   ! in_array( $item['reference']->get_id(), $references, true );
+				   ! in_array($item['reference']->get_id(), $references, true);
 			}
 		);
 
@@ -234,20 +234,20 @@ 
 			$jsonld,
 			array_filter(
 				array_map(
-					function ( $item ) use ( $references, $entity_to_jsonld_converter ) {
+					function($item) use ($references, $entity_to_jsonld_converter) {
 
-						if ( ! isset( $item['reference'] ) ) {
+						if ( ! isset($item['reference'])) {
 							  return null;
 						}
 
 						$post_id = $item['reference']->get_id();
-						if ( in_array( $post_id, $references, true ) ) {
+						if (in_array($post_id, $references, true)) {
 							return null;
 						}
 
 						$references[] = $post_id;
 
-						return $entity_to_jsonld_converter->convert( $post_id, $references );
+						return $entity_to_jsonld_converter->convert($post_id, $references);
 					},
 					$required_references
 				)
@@ -262,7 +262,7 @@ 
 		 * @var $jsonld array The final jsonld before outputting to page.
 		 * @var $post_id int The post id for which the jsonld is generated.
 		 */
-		$jsonld = apply_filters( 'wl_after_get_jsonld', $jsonld, $post_id, $context );
+		$jsonld = apply_filters('wl_after_get_jsonld', $jsonld, $post_id, $context);
 
 		return $jsonld;
 	}
@@ -281,9 +281,9 @@ 
 		$is_homepage = is_home() || is_front_page();
 		$post_id     = is_singular() ? get_the_ID() : null;
 
-		$jsonld = wp_json_encode( $this->get_jsonld( $is_homepage, $post_id, Jsonld_Context_Enum::PAGE ) );
+		$jsonld = wp_json_encode($this->get_jsonld($is_homepage, $post_id, Jsonld_Context_Enum::PAGE));
 		?>
-		<script type="application/ld+json"><?php echo esc_html( $jsonld ); ?></script>
+		<script type="application/ld+json"><?php echo esc_html($jsonld); ?></script>
 													  <?php
 	}
 

src/includes/class-wordlift-abstract-post-to-jsonld-converter.php

Indentation +395 added lines, -395 removed lines

@@ -23,421 +23,421 @@
  */
 abstract class Wordlift_Abstract_Post_To_Jsonld_Converter implements Wordlift_Post_Converter {
 
-	/**
-	 * The JSON-LD context.
-	 *
-	 * @since 3.10.0
-	 */
-	const CONTEXT = 'http://schema.org';
-
-	/**
-	 * A {@link Wordlift_Entity_Type_Service} instance.
-	 *
-	 * @since  3.10.0
-	 * @access protected
-	 * @var \Wordlift_Entity_Type_Service $entity_type_service A {@link Wordlift_Entity_Type_Service} instance.
-	 */
-	protected $entity_type_service;
-
-	/**
-	 * A {@link Wordlift_User_Service} instance.
-	 *
-	 * @since  3.10.0
-	 * @access private
-	 * @var \Wordlift_User_Service $user_service A {@link Wordlift_User_Service} instance.
-	 */
-	protected $user_service;
-
-	/**
-	 * A {@link Wordlift_Attachment_Service} instance.
-	 *
-	 * @since  3.10.0
-	 * @access private
-	 * @var \Wordlift_Attachment_Service $attachment_service A {@link Wordlift_Attachment_Service} instance.
-	 */
-	protected $attachment_service;
-
-	/**
-	 * @var Wordlift_Property_Getter
-	 */
-	private $property_getter;
-
-	/**
-	 * Wordlift_Post_To_Jsonld_Converter constructor.
-	 *
-	 * @param \Wordlift_Entity_Type_Service $entity_type_service A {@link Wordlift_Entity_Type_Service} instance.
-	 * @param \Wordlift_User_Service        $user_service A {@link Wordlift_User_Service} instance.
-	 * @param \Wordlift_Attachment_Service  $attachment_service A {@link Wordlift_Attachment_Service} instance.
-	 * @param \Wordlift_Property_Getter     $property_getter
-	 *
-	 * @since 3.10.0
-	 */
-	public function __construct( $entity_type_service, $user_service, $attachment_service, $property_getter ) {
-		$this->entity_type_service = $entity_type_service;
-		$this->user_service        = $user_service;
-		$this->attachment_service  = $attachment_service;
-		$this->property_getter     = $property_getter;
-	}
-
-	/**
-	 * Convert the provided {@link WP_Post} to a JSON-LD array. Any entity reference
-	 * found while processing the post is set in the $references array.
-	 *
-	 * @param int   $post_id The post id.
-	 * @param array $references An array of entity references.
-	 * @param array $references_infos
-	 *
-	 * @return array A JSON-LD array.
-	 * @since 3.10.0
-	 */
-	// phpcs:ignore VariableAnalysis.CodeAnalysis.VariableAnalysis.UnusedVariable
-	public function convert( $post_id, &$references = array(), &$references_infos = array() ) {
-
-		// Get the post instance.
-		$post = get_post( $post_id );
-		if ( null === $post ) {
-			// Post not found.
-			return null;
-		}
-
-		// Get the post URI @id.
-		$id = Wordlift_Entity_Service::get_instance()->get_uri( $post->ID );
-		if ( $id === null ) {
-			$id = 'get_uri returned null, dataset is ' . wl_configuration_get_redlink_dataset_uri();
-		}
-
-		/*
+    /**
+     * The JSON-LD context.
+     *
+     * @since 3.10.0
+     */
+    const CONTEXT = 'http://schema.org';
+
+    /**
+     * A {@link Wordlift_Entity_Type_Service} instance.
+     *
+     * @since  3.10.0
+     * @access protected
+     * @var \Wordlift_Entity_Type_Service $entity_type_service A {@link Wordlift_Entity_Type_Service} instance.
+     */
+    protected $entity_type_service;
+
+    /**
+     * A {@link Wordlift_User_Service} instance.
+     *
+     * @since  3.10.0
+     * @access private
+     * @var \Wordlift_User_Service $user_service A {@link Wordlift_User_Service} instance.
+     */
+    protected $user_service;
+
+    /**
+     * A {@link Wordlift_Attachment_Service} instance.
+     *
+     * @since  3.10.0
+     * @access private
+     * @var \Wordlift_Attachment_Service $attachment_service A {@link Wordlift_Attachment_Service} instance.
+     */
+    protected $attachment_service;
+
+    /**
+     * @var Wordlift_Property_Getter
+     */
+    private $property_getter;
+
+    /**
+     * Wordlift_Post_To_Jsonld_Converter constructor.
+     *
+     * @param \Wordlift_Entity_Type_Service $entity_type_service A {@link Wordlift_Entity_Type_Service} instance.
+     * @param \Wordlift_User_Service        $user_service A {@link Wordlift_User_Service} instance.
+     * @param \Wordlift_Attachment_Service  $attachment_service A {@link Wordlift_Attachment_Service} instance.
+     * @param \Wordlift_Property_Getter     $property_getter
+     *
+     * @since 3.10.0
+     */
+    public function __construct( $entity_type_service, $user_service, $attachment_service, $property_getter ) {
+        $this->entity_type_service = $entity_type_service;
+        $this->user_service        = $user_service;
+        $this->attachment_service  = $attachment_service;
+        $this->property_getter     = $property_getter;
+    }
+
+    /**
+     * Convert the provided {@link WP_Post} to a JSON-LD array. Any entity reference
+     * found while processing the post is set in the $references array.
+     *
+     * @param int   $post_id The post id.
+     * @param array $references An array of entity references.
+     * @param array $references_infos
+     *
+     * @return array A JSON-LD array.
+     * @since 3.10.0
+     */
+    // phpcs:ignore VariableAnalysis.CodeAnalysis.VariableAnalysis.UnusedVariable
+    public function convert( $post_id, &$references = array(), &$references_infos = array() ) {
+
+        // Get the post instance.
+        $post = get_post( $post_id );
+        if ( null === $post ) {
+            // Post not found.
+            return null;
+        }
+
+        // Get the post URI @id.
+        $id = Wordlift_Entity_Service::get_instance()->get_uri( $post->ID );
+        if ( $id === null ) {
+            $id = 'get_uri returned null, dataset is ' . wl_configuration_get_redlink_dataset_uri();
+        }
+
+        /*
 		 * The `types` variable holds one or more entity types. The `type` variable isn't used anymore.
 		 *
 		 * @since 3.20.0 We support more than one entity type.
 		 *
 		 * @see https://github.com/insideout10/wordlift-plugin/issues/835
 		 */
-		// Get the entity @type. We consider `post` BlogPostings.
-		// $type = $this->entity_type_service->get( $post_id );
-		$types = $this->entity_type_service->get_names( $post_id );
-		$type  = self::make_one( $types );
-
-		// Prepare the response.
-		$jsonld = array(
-			'@context'    => self::CONTEXT,
-			'@id'         => $id,
-			'@type'       => $type,
-			'description' => Wordlift_Post_Excerpt_Helper::get_text_excerpt( $post ),
-		);
-
-		// Set the `mainEntityOfPage` property if the post has some contents.
-		/*
+        // Get the entity @type. We consider `post` BlogPostings.
+        // $type = $this->entity_type_service->get( $post_id );
+        $types = $this->entity_type_service->get_names( $post_id );
+        $type  = self::make_one( $types );
+
+        // Prepare the response.
+        $jsonld = array(
+            '@context'    => self::CONTEXT,
+            '@id'         => $id,
+            '@type'       => $type,
+            'description' => Wordlift_Post_Excerpt_Helper::get_text_excerpt( $post ),
+        );
+
+        // Set the `mainEntityOfPage` property if the post has some contents.
+        /*
 		 * Apply the `wl_post_content` filter, in case 3rd parties want to change the post content, e.g.
 		 * because the content is written elsewhere.
 		 *
 		 * @since 3.20.0
 		 */
-		$post_content = apply_filters( 'wl_post_content', $post->post_content, $post );
-		if ( ! empty( $post_content ) ) {
-			// We're setting the `mainEntityOfPage` to signal which one is the
-			// main entity for the specified URL. It might be however that the
-			// post/page is actually about another specific entity. How WL deals
-			// with that hasn't been defined yet (see https://github.com/insideout10/wordlift-plugin/issues/451).
-			//
-			// See http://schema.org/mainEntityOfPage
-			//
-			// No need to specify `'@type' => 'WebPage'.
-			//
-			// See https://github.com/insideout10/wordlift-plugin/issues/451.
-			$jsonld['mainEntityOfPage'] = get_the_permalink( $post->ID );
-
-			/**
-			 * @see https://github.com/insideout10/wordlift-plugin/issues/1207
-			 *
-			 * @since 3.27.7
-			 */
-			if ( in_array( $type, array( 'Occupation', 'OccupationAggregationByEmployer' ), true ) ) {
-				$jsonld['mainEntityOfPage'] = array(
-					'@type'        => 'WebPage',
-					'lastReviewed' => get_post_time( 'Y-m-d\TH:i:sP', true, $post, false ),
-				);
-			}
-		};
-
-		$this->set_images( $this->attachment_service, $post, $jsonld );
-
-		// Get the entities referenced by this post and set it to the `references`
-		// array so that the caller can do further processing, such as printing out
-		// more of those references.
-		$references_without_locations = Object_Relation_Service::get_instance()
-															   ->get_references( $post_id, Object_Type_Enum::POST );
-
-		/*
+        $post_content = apply_filters( 'wl_post_content', $post->post_content, $post );
+        if ( ! empty( $post_content ) ) {
+            // We're setting the `mainEntityOfPage` to signal which one is the
+            // main entity for the specified URL. It might be however that the
+            // post/page is actually about another specific entity. How WL deals
+            // with that hasn't been defined yet (see https://github.com/insideout10/wordlift-plugin/issues/451).
+            //
+            // See http://schema.org/mainEntityOfPage
+            //
+            // No need to specify `'@type' => 'WebPage'.
+            //
+            // See https://github.com/insideout10/wordlift-plugin/issues/451.
+            $jsonld['mainEntityOfPage'] = get_the_permalink( $post->ID );
+
+            /**
+             * @see https://github.com/insideout10/wordlift-plugin/issues/1207
+             *
+             * @since 3.27.7
+             */
+            if ( in_array( $type, array( 'Occupation', 'OccupationAggregationByEmployer' ), true ) ) {
+                $jsonld['mainEntityOfPage'] = array(
+                    '@type'        => 'WebPage',
+                    'lastReviewed' => get_post_time( 'Y-m-d\TH:i:sP', true, $post, false ),
+                );
+            }
+        };
+
+        $this->set_images( $this->attachment_service, $post, $jsonld );
+
+        // Get the entities referenced by this post and set it to the `references`
+        // array so that the caller can do further processing, such as printing out
+        // more of those references.
+        $references_without_locations = Object_Relation_Service::get_instance()
+                                                                ->get_references( $post_id, Object_Type_Enum::POST );
+
+        /*
 		 * Add the locations to the references.
 		 *
 		 * @since 3.19.5
 		 *
 		 * @see https://github.com/insideout10/wordlift-plugin/issues/858.
 		 */
-		// A reference to use in closure.
-		$entity_type_service = $this->entity_type_service;
-		$locations           = array_reduce(
-			$references_without_locations,
-			function ( $carry, $reference ) use ( $entity_type_service ) {
-				/**
-				 * @var $reference Reference
-				 */
-				// @see https://schema.org/location for the schema.org types using the `location` property.
-				if ( ! $entity_type_service->has_entity_type( $reference->get_id(), 'http://schema.org/Action' )
-				 && ! $entity_type_service->has_entity_type( $reference->get_id(), 'http://schema.org/Event' )
-				 && ! $entity_type_service->has_entity_type( $reference->get_id(), 'http://schema.org/Organization' ) ) {
-
-					return $carry;
-				}
-				$post_location_ids        = get_post_meta( $reference->get_id(), Wordlift_Schema_Service::FIELD_LOCATION );
-				$post_location_references = array_map(
-					function ( $post_id ) {
-						return new Post_Reference( $post_id );
-					},
-					$post_location_ids
-				);
-
-				return array_merge( $carry, $post_location_references );
-			},
-			array()
-		);
-
-		// Merge the references with the referenced locations if any.
-		// phpcs:ignore VariableAnalysis.CodeAnalysis.VariableAnalysis.UnusedVariable
-		$references = array_merge( $references_without_locations, $locations );
-
-		return $jsonld;
-	}
-
-	/**
-	 * If the provided value starts with the schema.org context, we remove the schema.org
-	 * part since it is set with the '@context'.
-	 *
-	 * @param string $value The property value.
-	 *
-	 * @return string The property value without the context.
-	 * @since 3.10.0
-	 */
-	public function relative_to_context( $value ) {
-
-		return 0 === strpos( $value, self::CONTEXT . '/' ) ? substr( $value, strlen( self::CONTEXT ) + 1 ) : $value;
-	}
-
-	/**
-	 * Set the images, by looking for embedded images, for images loaded via the
-	 * gallery and for the featured image.
-	 *
-	 * Uses the cache service to store the results of this function for a day.
-	 *
-	 * @param $attachment_service Wordlift_Attachment_Service
-	 * @param WP_Post                                        $post The target {@link WP_Post}.
-	 * @param array                                          $jsonld The JSON-LD array.
-	 *
-	 * @since 3.10.0
-	 */
-	public static function set_images( $attachment_service, $post, &$jsonld ) {
-
-		// Prepare the attachment ids array.
-		$ids = array();
-
-		// Set the thumbnail id as first attachment id, if any.
-		$thumbnail_id = get_post_thumbnail_id( $post->ID );
-		if ( '' !== $thumbnail_id && 0 !== $thumbnail_id ) {
-			$ids[] = $thumbnail_id;
-		}
-
-		// For the time being the following is being removed since the query
-		// initiated by `get_image_embeds` is consuming lots of CPU.
-		//
-		// See https://github.com/insideout10/wordlift-plugin/issues/689.
-		//
-		// Get the embeds, removing existing ids.
-		// phpcs:ignore WordPress.NamingConventions.ValidHookName.UseUnderscores
-		if ( apply_filters( 'wl_feature__enable__image-embeds', false ) ) {
-			$embeds = array_diff( $attachment_service->get_image_embeds( $post->post_content ), $ids );
-		} else {
-			$embeds = array();
-		}
-
-		// Get the gallery, removing existing ids.
-		$gallery = array_diff( $attachment_service->get_gallery( $post ), $ids, $embeds );
-
-		// Map the attachment ids to images' data structured for schema.org use.
-		$images_with_sizes = array_filter(
-			array_reduce(
-				array_merge( $ids, $embeds, $gallery ),
-				function ( $carry, $item ) {
-					/*
+        // A reference to use in closure.
+        $entity_type_service = $this->entity_type_service;
+        $locations           = array_reduce(
+            $references_without_locations,
+            function ( $carry, $reference ) use ( $entity_type_service ) {
+                /**
+                 * @var $reference Reference
+                 */
+                // @see https://schema.org/location for the schema.org types using the `location` property.
+                if ( ! $entity_type_service->has_entity_type( $reference->get_id(), 'http://schema.org/Action' )
+                 && ! $entity_type_service->has_entity_type( $reference->get_id(), 'http://schema.org/Event' )
+                 && ! $entity_type_service->has_entity_type( $reference->get_id(), 'http://schema.org/Organization' ) ) {
+
+                    return $carry;
+                }
+                $post_location_ids        = get_post_meta( $reference->get_id(), Wordlift_Schema_Service::FIELD_LOCATION );
+                $post_location_references = array_map(
+                    function ( $post_id ) {
+                        return new Post_Reference( $post_id );
+                    },
+                    $post_location_ids
+                );
+
+                return array_merge( $carry, $post_location_references );
+            },
+            array()
+        );
+
+        // Merge the references with the referenced locations if any.
+        // phpcs:ignore VariableAnalysis.CodeAnalysis.VariableAnalysis.UnusedVariable
+        $references = array_merge( $references_without_locations, $locations );
+
+        return $jsonld;
+    }
+
+    /**
+     * If the provided value starts with the schema.org context, we remove the schema.org
+     * part since it is set with the '@context'.
+     *
+     * @param string $value The property value.
+     *
+     * @return string The property value without the context.
+     * @since 3.10.0
+     */
+    public function relative_to_context( $value ) {
+
+        return 0 === strpos( $value, self::CONTEXT . '/' ) ? substr( $value, strlen( self::CONTEXT ) + 1 ) : $value;
+    }
+
+    /**
+     * Set the images, by looking for embedded images, for images loaded via the
+     * gallery and for the featured image.
+     *
+     * Uses the cache service to store the results of this function for a day.
+     *
+     * @param $attachment_service Wordlift_Attachment_Service
+     * @param WP_Post                                        $post The target {@link WP_Post}.
+     * @param array                                          $jsonld The JSON-LD array.
+     *
+     * @since 3.10.0
+     */
+    public static function set_images( $attachment_service, $post, &$jsonld ) {
+
+        // Prepare the attachment ids array.
+        $ids = array();
+
+        // Set the thumbnail id as first attachment id, if any.
+        $thumbnail_id = get_post_thumbnail_id( $post->ID );
+        if ( '' !== $thumbnail_id && 0 !== $thumbnail_id ) {
+            $ids[] = $thumbnail_id;
+        }
+
+        // For the time being the following is being removed since the query
+        // initiated by `get_image_embeds` is consuming lots of CPU.
+        //
+        // See https://github.com/insideout10/wordlift-plugin/issues/689.
+        //
+        // Get the embeds, removing existing ids.
+        // phpcs:ignore WordPress.NamingConventions.ValidHookName.UseUnderscores
+        if ( apply_filters( 'wl_feature__enable__image-embeds', false ) ) {
+            $embeds = array_diff( $attachment_service->get_image_embeds( $post->post_content ), $ids );
+        } else {
+            $embeds = array();
+        }
+
+        // Get the gallery, removing existing ids.
+        $gallery = array_diff( $attachment_service->get_gallery( $post ), $ids, $embeds );
+
+        // Map the attachment ids to images' data structured for schema.org use.
+        $images_with_sizes = array_filter(
+            array_reduce(
+                array_merge( $ids, $embeds, $gallery ),
+                function ( $carry, $item ) {
+                    /*
 					* @todo: we're not sure that we're getting attachment data here, we
 					* should filter `false`s.
 					*/
 
-					$sources = array_merge(
-						Wordlift_Image_Service::get_sources( $item ),
-						array( wp_get_attachment_image_src( $item, 'full' ) )
-					);
-
-					$sources_with_image = array_filter(
-						$sources,
-						function ( $source ) {
-							return ! empty( $source[0] );
-						}
-					);
-
-					// Get the attachment data.
-					// $attachment = wp_get_attachment_image_src( $item, 'full' );
-
-					// var_dump( array( "sources-$item" => Wordlift_Image_Service::get_sources( $item ) ) );
-
-					// Bail if image is not found.
-					// In some cases, you can delete the image from the database
-					// or from uploads dir, but the image id still exists as featured image
-					// or in [gallery] shortcode.
-					// if ( empty( $attachment[0] ) ) {
-					if ( empty( $sources_with_image ) ) {
-						return $carry;
-					}
-
-					// Merge the arrays.
-					return array_merge(
-						$carry,
-						$sources_with_image
-					);
-				},
-				// Initial array.
-
-				array()
-			)
-		);
-
-		// Refactor data as per schema.org specifications.
-		$images = array_map(
-			function ( $attachment ) {
-				return Wordlift_Abstract_Post_To_Jsonld_Converter::set_image_size(
-					array(
-						'@type' => 'ImageObject',
-						'url'   => $attachment[0],
-					),
-					$attachment
-				);
-			},
-			$images_with_sizes
-		);
-
-		// Add images if present.
-		if ( 0 < count( $images ) ) {
-			$jsonld['image'] = $images;
-		}
-
-	}
-
-	/**
-	 * If the provided array of values contains only one value, then one single
-	 * value is returned, otherwise the original array is returned.
-	 *
-	 * @param array $value An array of values.
-	 *
-	 * @return mixed|array A single value or the original array.
-	 * @since 3.20.0 The function has been moved from {@link Wordlift_Entity_Post_To_Jsonld_Converter} to
-	 *  {@link Wordlift_Abstract_Post_To_Jsonld_Converter}.
-	 * @since  3.8.0
-	 * @access private
-	 */
-	protected static function make_one( $value ) {
-
-		return 1 === count( $value ) ? $value[0] : $value;
-	}
-
-	/**
-	 * Process the provided array by adding the width / height if the values
-	 * are available and are greater than 0.
-	 *
-	 * @param array $image The `ImageObject` array.
-	 * @param array $attachment The attachment array.
-	 *
-	 * @return array The enriched `ImageObject` array.
-	 * @since 3.14.0
-	 */
-	public static function set_image_size( $image, $attachment ) {
-
-		// If you specify a "width" or "height" value you should leave out
-		// 'px'. For example: "width":"4608px" should be "width":"4608".
-		//
-		// See https://github.com/insideout10/wordlift-plugin/issues/451.
-		if ( isset( $attachment[1] ) && is_numeric( $attachment[1] ) && 0 < $attachment[1] ) {
-			$image['width'] = $attachment[1];
-		}
-
-		if ( isset( $attachment[2] ) && is_numeric( $attachment[2] ) && 0 < $attachment[2] ) {
-			$image['height'] = $attachment[2];
-		}
-
-		return $image;
-	}
-
-	/**
-	 * Add data to the JSON-LD using the `custom_fields` array which contains the definitions of property
-	 * for the post entity type.
-	 *
-	 * @param array   $jsonld The JSON-LD array.
-	 * @param array   $fields The entity types field array.
-	 * @param WP_Post $post The target {@link WP_Post} instance.
-	 * @param array   $references The references array.
-	 *
-	 * @since 3.20.0 This code moved from the above function `convert`, used for entity types defined in
-	 *  the {@link Wordlift_Schema_Service} class.
-	 */
-	protected function process_type_custom_fields( &$jsonld, $fields, $post, &$references, &$references_infos ) {
-
-		// Set a reference to use in closures.
-		$converter = $this;
-
-		// Try each field on the entity.
-		foreach ( $fields as $key => $value ) {
-
-			// Get the predicate.
-			$name = $this->relative_to_context( $value['predicate'] );
-
-			// Get the value, the property service will get the right extractor
-			// for that property.
-			$value = $this->property_getter->get( $post->ID, $key, Object_Type_Enum::POST );
-
-			if ( empty( $value ) ) {
-				continue;
-			}
-
-			// Map the value to the property name.
-			// If we got an array with just one value, we return that one value.
-			// If we got a Wordlift_Property_Entity_Reference we get the URL.
-			$jsonld[ $name ] = self::make_one(
-				array_map(
-					function ( $item ) use ( $converter, &$references, &$references_infos ) {
-
-						if ( $item instanceof Wordlift_Property_Entity_Reference ) {
-
-							  $url = $item->get_url();
-
-							  // The refactored converters require the entity id.
-							// phpcs:ignore VariableAnalysis.CodeAnalysis.VariableAnalysis.UndefinedVariable
-							  $references[] = $item->get_id();
-
-							// phpcs:ignore VariableAnalysis.CodeAnalysis.VariableAnalysis.UndefinedVariable
-							  $references_infos[] = array( 'reference' => $item );
-
-							  return array(
-								  '@id' => $url,
-							  );
-						}
-
-						return $converter->relative_to_context( $item );
-					},
-					$value
-				)
-			);
-
-		}
-
-	}
+                    $sources = array_merge(
+                        Wordlift_Image_Service::get_sources( $item ),
+                        array( wp_get_attachment_image_src( $item, 'full' ) )
+                    );
+
+                    $sources_with_image = array_filter(
+                        $sources,
+                        function ( $source ) {
+                            return ! empty( $source[0] );
+                        }
+                    );
+
+                    // Get the attachment data.
+                    // $attachment = wp_get_attachment_image_src( $item, 'full' );
+
+                    // var_dump( array( "sources-$item" => Wordlift_Image_Service::get_sources( $item ) ) );
+
+                    // Bail if image is not found.
+                    // In some cases, you can delete the image from the database
+                    // or from uploads dir, but the image id still exists as featured image
+                    // or in [gallery] shortcode.
+                    // if ( empty( $attachment[0] ) ) {
+                    if ( empty( $sources_with_image ) ) {
+                        return $carry;
+                    }
+
+                    // Merge the arrays.
+                    return array_merge(
+                        $carry,
+                        $sources_with_image
+                    );
+                },
+                // Initial array.
+
+                array()
+            )
+        );
+
+        // Refactor data as per schema.org specifications.
+        $images = array_map(
+            function ( $attachment ) {
+                return Wordlift_Abstract_Post_To_Jsonld_Converter::set_image_size(
+                    array(
+                        '@type' => 'ImageObject',
+                        'url'   => $attachment[0],
+                    ),
+                    $attachment
+                );
+            },
+            $images_with_sizes
+        );
+
+        // Add images if present.
+        if ( 0 < count( $images ) ) {
+            $jsonld['image'] = $images;
+        }
+
+    }
+
+    /**
+     * If the provided array of values contains only one value, then one single
+     * value is returned, otherwise the original array is returned.
+     *
+     * @param array $value An array of values.
+     *
+     * @return mixed|array A single value or the original array.
+     * @since 3.20.0 The function has been moved from {@link Wordlift_Entity_Post_To_Jsonld_Converter} to
+     *  {@link Wordlift_Abstract_Post_To_Jsonld_Converter}.
+     * @since  3.8.0
+     * @access private
+     */
+    protected static function make_one( $value ) {
+
+        return 1 === count( $value ) ? $value[0] : $value;
+    }
+
+    /**
+     * Process the provided array by adding the width / height if the values
+     * are available and are greater than 0.
+     *
+     * @param array $image The `ImageObject` array.
+     * @param array $attachment The attachment array.
+     *
+     * @return array The enriched `ImageObject` array.
+     * @since 3.14.0
+     */
+    public static function set_image_size( $image, $attachment ) {
+
+        // If you specify a "width" or "height" value you should leave out
+        // 'px'. For example: "width":"4608px" should be "width":"4608".
+        //
+        // See https://github.com/insideout10/wordlift-plugin/issues/451.
+        if ( isset( $attachment[1] ) && is_numeric( $attachment[1] ) && 0 < $attachment[1] ) {
+            $image['width'] = $attachment[1];
+        }
+
+        if ( isset( $attachment[2] ) && is_numeric( $attachment[2] ) && 0 < $attachment[2] ) {
+            $image['height'] = $attachment[2];
+        }
+
+        return $image;
+    }
+
+    /**
+     * Add data to the JSON-LD using the `custom_fields` array which contains the definitions of property
+     * for the post entity type.
+     *
+     * @param array   $jsonld The JSON-LD array.
+     * @param array   $fields The entity types field array.
+     * @param WP_Post $post The target {@link WP_Post} instance.
+     * @param array   $references The references array.
+     *
+     * @since 3.20.0 This code moved from the above function `convert`, used for entity types defined in
+     *  the {@link Wordlift_Schema_Service} class.
+     */
+    protected function process_type_custom_fields( &$jsonld, $fields, $post, &$references, &$references_infos ) {
+
+        // Set a reference to use in closures.
+        $converter = $this;
+
+        // Try each field on the entity.
+        foreach ( $fields as $key => $value ) {
+
+            // Get the predicate.
+            $name = $this->relative_to_context( $value['predicate'] );
+
+            // Get the value, the property service will get the right extractor
+            // for that property.
+            $value = $this->property_getter->get( $post->ID, $key, Object_Type_Enum::POST );
+
+            if ( empty( $value ) ) {
+                continue;
+            }
+
+            // Map the value to the property name.
+            // If we got an array with just one value, we return that one value.
+            // If we got a Wordlift_Property_Entity_Reference we get the URL.
+            $jsonld[ $name ] = self::make_one(
+                array_map(
+                    function ( $item ) use ( $converter, &$references, &$references_infos ) {
+
+                        if ( $item instanceof Wordlift_Property_Entity_Reference ) {
+
+                                $url = $item->get_url();
+
+                                // The refactored converters require the entity id.
+                            // phpcs:ignore VariableAnalysis.CodeAnalysis.VariableAnalysis.UndefinedVariable
+                                $references[] = $item->get_id();
+
+                            // phpcs:ignore VariableAnalysis.CodeAnalysis.VariableAnalysis.UndefinedVariable
+                                $references_infos[] = array( 'reference' => $item );
+
+                                return array(
+                                    '@id' => $url,
+                                );
+                        }
+
+                        return $converter->relative_to_context( $item );
+                    },
+                    $value
+                )
+            );
+
+        }
+
+    }
 
 }

Spacing +58 added lines, -58 removed lines

@@ -72,7 +72,7 @@ 
 	 *
 	 * @since 3.10.0
 	 */
-	public function __construct( $entity_type_service, $user_service, $attachment_service, $property_getter ) {
+	public function __construct($entity_type_service, $user_service, $attachment_service, $property_getter) {
 		$this->entity_type_service = $entity_type_service;
 		$this->user_service        = $user_service;
 		$this->attachment_service  = $attachment_service;
@@ -91,19 +91,19 @@ 
 	 * @since 3.10.0
 	 */
 	// phpcs:ignore VariableAnalysis.CodeAnalysis.VariableAnalysis.UnusedVariable
-	public function convert( $post_id, &$references = array(), &$references_infos = array() ) {
+	public function convert($post_id, &$references = array(), &$references_infos = array()) {
 
 		// Get the post instance.
-		$post = get_post( $post_id );
-		if ( null === $post ) {
+		$post = get_post($post_id);
+		if (null === $post) {
 			// Post not found.
 			return null;
 		}
 
 		// Get the post URI @id.
-		$id = Wordlift_Entity_Service::get_instance()->get_uri( $post->ID );
-		if ( $id === null ) {
-			$id = 'get_uri returned null, dataset is ' . wl_configuration_get_redlink_dataset_uri();
+		$id = Wordlift_Entity_Service::get_instance()->get_uri($post->ID);
+		if ($id === null) {
+			$id = 'get_uri returned null, dataset is '.wl_configuration_get_redlink_dataset_uri();
 		}
 
 		/*
@@ -115,15 +115,15 @@ 
 		 */
 		// Get the entity @type. We consider `post` BlogPostings.
 		// $type = $this->entity_type_service->get( $post_id );
-		$types = $this->entity_type_service->get_names( $post_id );
-		$type  = self::make_one( $types );
+		$types = $this->entity_type_service->get_names($post_id);
+		$type  = self::make_one($types);
 
 		// Prepare the response.
 		$jsonld = array(
 			'@context'    => self::CONTEXT,
 			'@id'         => $id,
 			'@type'       => $type,
-			'description' => Wordlift_Post_Excerpt_Helper::get_text_excerpt( $post ),
+			'description' => Wordlift_Post_Excerpt_Helper::get_text_excerpt($post),
 		);
 
 		// Set the `mainEntityOfPage` property if the post has some contents.
@@ -133,8 +133,8 @@ 
 		 *
 		 * @since 3.20.0
 		 */
-		$post_content = apply_filters( 'wl_post_content', $post->post_content, $post );
-		if ( ! empty( $post_content ) ) {
+		$post_content = apply_filters('wl_post_content', $post->post_content, $post);
+		if ( ! empty($post_content)) {
 			// We're setting the `mainEntityOfPage` to signal which one is the
 			// main entity for the specified URL. It might be however that the
 			// post/page is actually about another specific entity. How WL deals
@@ -145,28 +145,28 @@ 
 			// No need to specify `'@type' => 'WebPage'.
 			//
 			// See https://github.com/insideout10/wordlift-plugin/issues/451.
-			$jsonld['mainEntityOfPage'] = get_the_permalink( $post->ID );
+			$jsonld['mainEntityOfPage'] = get_the_permalink($post->ID);
 
 			/**
 			 * @see https://github.com/insideout10/wordlift-plugin/issues/1207
 			 *
 			 * @since 3.27.7
 			 */
-			if ( in_array( $type, array( 'Occupation', 'OccupationAggregationByEmployer' ), true ) ) {
+			if (in_array($type, array('Occupation', 'OccupationAggregationByEmployer'), true)) {
 				$jsonld['mainEntityOfPage'] = array(
 					'@type'        => 'WebPage',
-					'lastReviewed' => get_post_time( 'Y-m-d\TH:i:sP', true, $post, false ),
+					'lastReviewed' => get_post_time('Y-m-d\TH:i:sP', true, $post, false),
 				);
 			}
 		};
 
-		$this->set_images( $this->attachment_service, $post, $jsonld );
+		$this->set_images($this->attachment_service, $post, $jsonld);
 
 		// Get the entities referenced by this post and set it to the `references`
 		// array so that the caller can do further processing, such as printing out
 		// more of those references.
 		$references_without_locations = Object_Relation_Service::get_instance()
-															   ->get_references( $post_id, Object_Type_Enum::POST );
+															   ->get_references($post_id, Object_Type_Enum::POST);
 
 		/*
 		 * Add the locations to the references.
@@ -179,33 +179,33 @@ 
 		$entity_type_service = $this->entity_type_service;
 		$locations           = array_reduce(
 			$references_without_locations,
-			function ( $carry, $reference ) use ( $entity_type_service ) {
+			function($carry, $reference) use ($entity_type_service) {
 				/**
 				 * @var $reference Reference
 				 */
 				// @see https://schema.org/location for the schema.org types using the `location` property.
-				if ( ! $entity_type_service->has_entity_type( $reference->get_id(), 'http://schema.org/Action' )
-				 && ! $entity_type_service->has_entity_type( $reference->get_id(), 'http://schema.org/Event' )
-				 && ! $entity_type_service->has_entity_type( $reference->get_id(), 'http://schema.org/Organization' ) ) {
+				if ( ! $entity_type_service->has_entity_type($reference->get_id(), 'http://schema.org/Action')
+				 && ! $entity_type_service->has_entity_type($reference->get_id(), 'http://schema.org/Event')
+				 && ! $entity_type_service->has_entity_type($reference->get_id(), 'http://schema.org/Organization')) {
 
 					return $carry;
 				}
-				$post_location_ids        = get_post_meta( $reference->get_id(), Wordlift_Schema_Service::FIELD_LOCATION );
+				$post_location_ids        = get_post_meta($reference->get_id(), Wordlift_Schema_Service::FIELD_LOCATION);
 				$post_location_references = array_map(
-					function ( $post_id ) {
-						return new Post_Reference( $post_id );
+					function($post_id) {
+						return new Post_Reference($post_id);
 					},
 					$post_location_ids
 				);
 
-				return array_merge( $carry, $post_location_references );
+				return array_merge($carry, $post_location_references);
 			},
 			array()
 		);
 
 		// Merge the references with the referenced locations if any.
 		// phpcs:ignore VariableAnalysis.CodeAnalysis.VariableAnalysis.UnusedVariable
-		$references = array_merge( $references_without_locations, $locations );
+		$references = array_merge($references_without_locations, $locations);
 
 		return $jsonld;
 	}
@@ -219,9 +219,9 @@ 
 	 * @return string The property value without the context.
 	 * @since 3.10.0
 	 */
-	public function relative_to_context( $value ) {
+	public function relative_to_context($value) {
 
-		return 0 === strpos( $value, self::CONTEXT . '/' ) ? substr( $value, strlen( self::CONTEXT ) + 1 ) : $value;
+		return 0 === strpos($value, self::CONTEXT.'/') ? substr($value, strlen(self::CONTEXT) + 1) : $value;
 	}
 
 	/**
@@ -236,14 +236,14 @@ 
 	 *
 	 * @since 3.10.0
 	 */
-	public static function set_images( $attachment_service, $post, &$jsonld ) {
+	public static function set_images($attachment_service, $post, &$jsonld) {
 
 		// Prepare the attachment ids array.
 		$ids = array();
 
 		// Set the thumbnail id as first attachment id, if any.
-		$thumbnail_id = get_post_thumbnail_id( $post->ID );
-		if ( '' !== $thumbnail_id && 0 !== $thumbnail_id ) {
+		$thumbnail_id = get_post_thumbnail_id($post->ID);
+		if ('' !== $thumbnail_id && 0 !== $thumbnail_id) {
 			$ids[] = $thumbnail_id;
 		}
 
@@ -254,34 +254,34 @@ 
 		//
 		// Get the embeds, removing existing ids.
 		// phpcs:ignore WordPress.NamingConventions.ValidHookName.UseUnderscores
-		if ( apply_filters( 'wl_feature__enable__image-embeds', false ) ) {
-			$embeds = array_diff( $attachment_service->get_image_embeds( $post->post_content ), $ids );
+		if (apply_filters('wl_feature__enable__image-embeds', false)) {
+			$embeds = array_diff($attachment_service->get_image_embeds($post->post_content), $ids);
 		} else {
 			$embeds = array();
 		}
 
 		// Get the gallery, removing existing ids.
-		$gallery = array_diff( $attachment_service->get_gallery( $post ), $ids, $embeds );
+		$gallery = array_diff($attachment_service->get_gallery($post), $ids, $embeds);
 
 		// Map the attachment ids to images' data structured for schema.org use.
 		$images_with_sizes = array_filter(
 			array_reduce(
-				array_merge( $ids, $embeds, $gallery ),
-				function ( $carry, $item ) {
+				array_merge($ids, $embeds, $gallery),
+				function($carry, $item) {
 					/*
 					* @todo: we're not sure that we're getting attachment data here, we
 					* should filter `false`s.
 					*/
 
 					$sources = array_merge(
-						Wordlift_Image_Service::get_sources( $item ),
-						array( wp_get_attachment_image_src( $item, 'full' ) )
+						Wordlift_Image_Service::get_sources($item),
+						array(wp_get_attachment_image_src($item, 'full'))
 					);
 
 					$sources_with_image = array_filter(
 						$sources,
-						function ( $source ) {
-							return ! empty( $source[0] );
+						function($source) {
+							return ! empty($source[0]);
 						}
 					);
 
@@ -295,7 +295,7 @@ 
 					// or from uploads dir, but the image id still exists as featured image
 					// or in [gallery] shortcode.
 					// if ( empty( $attachment[0] ) ) {
-					if ( empty( $sources_with_image ) ) {
+					if (empty($sources_with_image)) {
 						return $carry;
 					}
 
@@ -313,7 +313,7 @@ 
 
 		// Refactor data as per schema.org specifications.
 		$images = array_map(
-			function ( $attachment ) {
+			function($attachment) {
 				return Wordlift_Abstract_Post_To_Jsonld_Converter::set_image_size(
 					array(
 						'@type' => 'ImageObject',
@@ -326,7 +326,7 @@ 
 		);
 
 		// Add images if present.
-		if ( 0 < count( $images ) ) {
+		if (0 < count($images)) {
 			$jsonld['image'] = $images;
 		}
 
@@ -344,9 +344,9 @@ 
 	 * @since  3.8.0
 	 * @access private
 	 */
-	protected static function make_one( $value ) {
+	protected static function make_one($value) {
 
-		return 1 === count( $value ) ? $value[0] : $value;
+		return 1 === count($value) ? $value[0] : $value;
 	}
 
 	/**
@@ -359,17 +359,17 @@ 
 	 * @return array The enriched `ImageObject` array.
 	 * @since 3.14.0
 	 */
-	public static function set_image_size( $image, $attachment ) {
+	public static function set_image_size($image, $attachment) {
 
 		// If you specify a "width" or "height" value you should leave out
 		// 'px'. For example: "width":"4608px" should be "width":"4608".
 		//
 		// See https://github.com/insideout10/wordlift-plugin/issues/451.
-		if ( isset( $attachment[1] ) && is_numeric( $attachment[1] ) && 0 < $attachment[1] ) {
+		if (isset($attachment[1]) && is_numeric($attachment[1]) && 0 < $attachment[1]) {
 			$image['width'] = $attachment[1];
 		}
 
-		if ( isset( $attachment[2] ) && is_numeric( $attachment[2] ) && 0 < $attachment[2] ) {
+		if (isset($attachment[2]) && is_numeric($attachment[2]) && 0 < $attachment[2]) {
 			$image['height'] = $attachment[2];
 		}
 
@@ -388,33 +388,33 @@ 
 	 * @since 3.20.0 This code moved from the above function `convert`, used for entity types defined in
 	 *  the {@link Wordlift_Schema_Service} class.
 	 */
-	protected function process_type_custom_fields( &$jsonld, $fields, $post, &$references, &$references_infos ) {
+	protected function process_type_custom_fields(&$jsonld, $fields, $post, &$references, &$references_infos) {
 
 		// Set a reference to use in closures.
 		$converter = $this;
 
 		// Try each field on the entity.
-		foreach ( $fields as $key => $value ) {
+		foreach ($fields as $key => $value) {
 
 			// Get the predicate.
-			$name = $this->relative_to_context( $value['predicate'] );
+			$name = $this->relative_to_context($value['predicate']);
 
 			// Get the value, the property service will get the right extractor
 			// for that property.
-			$value = $this->property_getter->get( $post->ID, $key, Object_Type_Enum::POST );
+			$value = $this->property_getter->get($post->ID, $key, Object_Type_Enum::POST);
 
-			if ( empty( $value ) ) {
+			if (empty($value)) {
 				continue;
 			}
 
 			// Map the value to the property name.
 			// If we got an array with just one value, we return that one value.
 			// If we got a Wordlift_Property_Entity_Reference we get the URL.
-			$jsonld[ $name ] = self::make_one(
+			$jsonld[$name] = self::make_one(
 				array_map(
-					function ( $item ) use ( $converter, &$references, &$references_infos ) {
+					function($item) use ($converter, &$references, &$references_infos) {
 
-						if ( $item instanceof Wordlift_Property_Entity_Reference ) {
+						if ($item instanceof Wordlift_Property_Entity_Reference) {
 
 							  $url = $item->get_url();
 
@@ -423,14 +423,14 @@ 
 							  $references[] = $item->get_id();
 
 							// phpcs:ignore VariableAnalysis.CodeAnalysis.VariableAnalysis.UndefinedVariable
-							  $references_infos[] = array( 'reference' => $item );
+							  $references_infos[] = array('reference' => $item);
 
 							  return array(
 								  '@id' => $url,
 							  );
 						}
 
-						return $converter->relative_to_context( $item );
+						return $converter->relative_to_context($item);
 					},
 					$value
 				)

src/includes/class-wordlift-category-taxonomy-service.php

Indentation +60 added lines, -60 removed lines

@@ -18,75 +18,75 @@
  */
 class Wordlift_Category_Taxonomy_Service {
 
-	/**
-	 * The {@link Wordlift_Entity_Post_Type_Service} instance.
-	 *
-	 * @since  3.11.0
-	 * @access private
-	 * @var \Wordlift_Entity_Post_Type_Service $entity_post_type_service The {@link Wordlift_Entity_Post_Type_Service} instance.
-	 */
-	private $entity_post_type_service;
+    /**
+     * The {@link Wordlift_Entity_Post_Type_Service} instance.
+     *
+     * @since  3.11.0
+     * @access private
+     * @var \Wordlift_Entity_Post_Type_Service $entity_post_type_service The {@link Wordlift_Entity_Post_Type_Service} instance.
+     */
+    private $entity_post_type_service;
 
-	/**
-	 * Create a {@link Wordlift_Category_Taxonomy_Service} instance.
-	 *
-	 * @since 3.11.0
-	 *
-	 * @param \Wordlift_Entity_Post_Type_Service $entity_post_type_service The {@link Wordlift_Entity_Post_Type_Service} instance.
-	 */
-	public function __construct( $entity_post_type_service ) {
+    /**
+     * Create a {@link Wordlift_Category_Taxonomy_Service} instance.
+     *
+     * @since 3.11.0
+     *
+     * @param \Wordlift_Entity_Post_Type_Service $entity_post_type_service The {@link Wordlift_Entity_Post_Type_Service} instance.
+     */
+    public function __construct( $entity_post_type_service ) {
 
-		$this->entity_post_type_service = $entity_post_type_service;
+        $this->entity_post_type_service = $entity_post_type_service;
 
-	}
+    }
 
-	/**
-	 * Set the entity post types as one to be included in archive pages.
-	 *
-	 * In order to have entities show up in standard WP categories (Posts categories)
-	 * we configure the `entity` post type, but we also need to alter the main
-	 * WP query (which by default queries posts only) to include the `entities`.
-	 *
-	 * @since 3.11.0
-	 *
-	 * @param WP_Query $query WP's {@link WP_Query} instance.
-	 */
-	public function pre_get_posts( $query ) {
+    /**
+     * Set the entity post types as one to be included in archive pages.
+     *
+     * In order to have entities show up in standard WP categories (Posts categories)
+     * we configure the `entity` post type, but we also need to alter the main
+     * WP query (which by default queries posts only) to include the `entities`.
+     *
+     * @since 3.11.0
+     *
+     * @param WP_Query $query WP's {@link WP_Query} instance.
+     */
+    public function pre_get_posts( $query ) {
 
-		// Only for the main query, avoid problems with widgets and what not.
-		if ( ! $query->is_main_query() ) {
-			return;
-		}
+        // Only for the main query, avoid problems with widgets and what not.
+        if ( ! $query->is_main_query() ) {
+            return;
+        }
 
-		// We don't want to alter the query if we're in the admin UI, if this is
-		// not a category query, or if the `suppress_filters` is set.
-		//
-		// Note that it is unlikely for `suppress_filter` to be set on the front
-		// end, but let's be safe if it is set the calling code assumes no
-		// modifications of queries.
-		//
-		// is_admin is needed, otherwise category based post filters will show
-		// both types and at the current release (4.7) it causes PHP errors.
-		if ( is_admin() || ! is_category() || ! empty( $query->query_vars['suppress_filters'] ) ) {
-			return;
-		}
+        // We don't want to alter the query if we're in the admin UI, if this is
+        // not a category query, or if the `suppress_filters` is set.
+        //
+        // Note that it is unlikely for `suppress_filter` to be set on the front
+        // end, but let's be safe if it is set the calling code assumes no
+        // modifications of queries.
+        //
+        // is_admin is needed, otherwise category based post filters will show
+        // both types and at the current release (4.7) it causes PHP errors.
+        if ( is_admin() || ! is_category() || ! empty( $query->query_vars['suppress_filters'] ) ) {
+            return;
+        }
 
-		// Check the current post types, maybe the category archive pages
-		// are already associated with other post types.
-		//
-		// If `post_type` isn't set, WP assumes `post` by default.
-		$post_types = $query->get( 'post_type' );
-		$post_types = (array) ( $post_types ? $post_types : 'post' );
+        // Check the current post types, maybe the category archive pages
+        // are already associated with other post types.
+        //
+        // If `post_type` isn't set, WP assumes `post` by default.
+        $post_types = $query->get( 'post_type' );
+        $post_types = (array) ( $post_types ? $post_types : 'post' );
 
-		// Add the entities post type only if the post post type is used in the query
-		// since we only want `entities` to appear alongside posts.
-		if ( in_array( 'post', $post_types, true ) ) {
-			$post_types[] = $this->entity_post_type_service->get_post_type();
-		}
+        // Add the entities post type only if the post post type is used in the query
+        // since we only want `entities` to appear alongside posts.
+        if ( in_array( 'post', $post_types, true ) ) {
+            $post_types[] = $this->entity_post_type_service->get_post_type();
+        }
 
-		// Update the query post types.
-		$query->set( 'post_type', $post_types );
+        // Update the query post types.
+        $query->set( 'post_type', $post_types );
 
-	}
+    }
 
 }

Spacing +8 added lines, -8 removed lines

@@ -34,7 +34,7 @@ 
 	 *
 	 * @param \Wordlift_Entity_Post_Type_Service $entity_post_type_service The {@link Wordlift_Entity_Post_Type_Service} instance.
 	 */
-	public function __construct( $entity_post_type_service ) {
+	public function __construct($entity_post_type_service) {
 
 		$this->entity_post_type_service = $entity_post_type_service;
 
@@ -51,10 +51,10 @@ 
 	 *
 	 * @param WP_Query $query WP's {@link WP_Query} instance.
 	 */
-	public function pre_get_posts( $query ) {
+	public function pre_get_posts($query) {
 
 		// Only for the main query, avoid problems with widgets and what not.
-		if ( ! $query->is_main_query() ) {
+		if ( ! $query->is_main_query()) {
 			return;
 		}
 
@@ -67,7 +67,7 @@ 
 		//
 		// is_admin is needed, otherwise category based post filters will show
 		// both types and at the current release (4.7) it causes PHP errors.
-		if ( is_admin() || ! is_category() || ! empty( $query->query_vars['suppress_filters'] ) ) {
+		if (is_admin() || ! is_category() || ! empty($query->query_vars['suppress_filters'])) {
 			return;
 		}
 
@@ -75,17 +75,17 @@ 
 		// are already associated with other post types.
 		//
 		// If `post_type` isn't set, WP assumes `post` by default.
-		$post_types = $query->get( 'post_type' );
-		$post_types = (array) ( $post_types ? $post_types : 'post' );
+		$post_types = $query->get('post_type');
+		$post_types = (array) ($post_types ? $post_types : 'post');
 
 		// Add the entities post type only if the post post type is used in the query
 		// since we only want `entities` to appear alongside posts.
-		if ( in_array( 'post', $post_types, true ) ) {
+		if (in_array('post', $post_types, true)) {
 			$post_types[] = $this->entity_post_type_service->get_post_type();
 		}
 
 		// Update the query post types.
-		$query->set( 'post_type', $post_types );
+		$query->set('post_type', $post_types);
 
 	}
 

src/includes/class-wordlift-entity-type-adapter.php

Indentation +141 added lines, -141 removed lines

@@ -20,146 +20,146 @@
  */
 class Wordlift_Entity_Type_Adapter {
 
-	/**
-	 * A {@link Wordlift_Log_Service} instance.
-	 *
-	 * @since  3.15.0
-	 * @access private
-	 * @var \Wordlift_Log_Service $log A {@link Wordlift_Log_Service} instance.
-	 */
-	private $log;
-
-	/**
-	 * The {@link Wordlift_Entity_Type_Service} instance.
-	 *
-	 * @since  3.15.0
-	 * @access private
-	 * @var \Wordlift_Entity_Type_Service $entity_type_service The {@link Wordlift_Entity_Type_Service} instance.
-	 */
-	private $entity_type_service;
-
-	/**
-	 * Create a {@link Wordlift_Entity_Type_Adapter} instance.
-	 *
-	 * @param \Wordlift_Entity_Type_Service $entity_type_service The {@link Wordlift_Entity_Type_Service} instance.
-	 *
-	 * @since 3.15.0
-	 */
-	public function __construct( $entity_type_service ) {
-
-		$this->log = Wordlift_Log_Service::get_logger( 'Wordlift_Entity_Type_Adapter' );
-
-		$this->entity_type_service = $entity_type_service;
-
-		add_filter(
-			'wl_default_entity_type_for_post_type',
-			array(
-				$this,
-				'default_entity_type_callback',
-			),
-			10,
-			2
-		);
-
-	}
-
-	/**
-	 * @param $entity_type string Entity type
-	 * @param $post_type string The post type
-	 *
-	 * @return string The default entity type depending on the post type.
-	 */
-	public function default_entity_type_callback( $entity_type, $post_type ) {
-		if ( 'product' === $post_type ) {
-			return 'http://schema.org/Product';
-		}
-
-		return $entity_type;
-	}
-
-	/**
-	 * Hook to `save_post`.
-	 *
-	 * This function is called when the `save_post` action is run. The
-	 * function checks whether an Entity Type term (from the Entity Types
-	 * taxonomy) is already assigned and, if not, it calls {@link Wordlift_Entity_Type_Service}
-	 * to set the default taxonomy term.
-	 *
-	 * @param int     $post_id The {@link WP_Post}'s id.
-	 * @param WP_Post $post The {@link WP_Post} instance.
-	 *
-	 * @since 3.15.0
-	 */
-	public function save_post( $post_id, $post ) {
-
-		if ( ! Wordlift_Entity_Type_Service::is_valid_entity_post_type( $post->post_type ) ) {
-			$this->log->debug( "Ignoring `{$post->post_type}` post type." );
-
-			// Bail out if the post can not be an entity.
-			return;
-		}
-
-		// Bail out if the post already has an entity type.
-		if ( apply_filters( 'wl_entity_type_adapter__save_post__has_entity_type', $this->entity_type_service->has_entity_type( $post_id ) ) ) {
-			$this->log->debug( "Post $post_id has already an entity type." );
-
-			return;
-		}
-
-		$this->log->debug( "Setting `Article` entity type for post $post_id." );
-
-		// Finally set the default entity type.
-
-		// For entities use a Thing
-		if ( 'entity' === $post->post_type ) {
-			$this->entity_type_service->set( $post_id, 'http://schema.org/Thing' );
-		} else {
-			// Get the entity types.
-			$entity_types = self::get_entity_types( $post->post_type );
-
-			// Set the entity type.
-			foreach ( $entity_types as $entity_type ) {
-				$this->entity_type_service->set( $post_id, $entity_type, false );
-			}
-		}
-
-	}
-
-	/**
-	 * Get the entity types for a post type.
-	 *
-	 * @param string $post_type The post type.
-	 *
-	 * @return array An array of entity types.
-	 * @since 3.20.0
-	 */
-	public static function get_entity_types( $post_type ) {
-
-		/**
-		 * Get the default entity type.
-		 *
-		 * @param string $entity_type The preset entity type.
-		 * @param string $post_type The post type.
-		 *
-		 * @since 3.20.0
-		 */
-		$default_entity_type = apply_filters( 'wl_default_entity_type_for_post_type', 'http://schema.org/Article', $post_type );
-
-		/**
-		 * Get the default entity types.
-		 *
-		 * Adding support to assign more than one entity type.
-		 *
-		 * @param array $entity_types The default entity types.
-		 * @param string $post_type The post type.
-		 *
-		 * @since 3.20.0
-		 *
-		 * @see Wordlift_Mapping_Service
-		 */
-		$entity_types = apply_filters( 'wl_default_entity_types_for_post_type', array( $default_entity_type ), $post_type );
-
-		return $entity_types;
-	}
+    /**
+     * A {@link Wordlift_Log_Service} instance.
+     *
+     * @since  3.15.0
+     * @access private
+     * @var \Wordlift_Log_Service $log A {@link Wordlift_Log_Service} instance.
+     */
+    private $log;
+
+    /**
+     * The {@link Wordlift_Entity_Type_Service} instance.
+     *
+     * @since  3.15.0
+     * @access private
+     * @var \Wordlift_Entity_Type_Service $entity_type_service The {@link Wordlift_Entity_Type_Service} instance.
+     */
+    private $entity_type_service;
+
+    /**
+     * Create a {@link Wordlift_Entity_Type_Adapter} instance.
+     *
+     * @param \Wordlift_Entity_Type_Service $entity_type_service The {@link Wordlift_Entity_Type_Service} instance.
+     *
+     * @since 3.15.0
+     */
+    public function __construct( $entity_type_service ) {
+
+        $this->log = Wordlift_Log_Service::get_logger( 'Wordlift_Entity_Type_Adapter' );
+
+        $this->entity_type_service = $entity_type_service;
+
+        add_filter(
+            'wl_default_entity_type_for_post_type',
+            array(
+                $this,
+                'default_entity_type_callback',
+            ),
+            10,
+            2
+        );
+
+    }
+
+    /**
+     * @param $entity_type string Entity type
+     * @param $post_type string The post type
+     *
+     * @return string The default entity type depending on the post type.
+     */
+    public function default_entity_type_callback( $entity_type, $post_type ) {
+        if ( 'product' === $post_type ) {
+            return 'http://schema.org/Product';
+        }
+
+        return $entity_type;
+    }
+
+    /**
+     * Hook to `save_post`.
+     *
+     * This function is called when the `save_post` action is run. The
+     * function checks whether an Entity Type term (from the Entity Types
+     * taxonomy) is already assigned and, if not, it calls {@link Wordlift_Entity_Type_Service}
+     * to set the default taxonomy term.
+     *
+     * @param int     $post_id The {@link WP_Post}'s id.
+     * @param WP_Post $post The {@link WP_Post} instance.
+     *
+     * @since 3.15.0
+     */
+    public function save_post( $post_id, $post ) {
+
+        if ( ! Wordlift_Entity_Type_Service::is_valid_entity_post_type( $post->post_type ) ) {
+            $this->log->debug( "Ignoring `{$post->post_type}` post type." );
+
+            // Bail out if the post can not be an entity.
+            return;
+        }
+
+        // Bail out if the post already has an entity type.
+        if ( apply_filters( 'wl_entity_type_adapter__save_post__has_entity_type', $this->entity_type_service->has_entity_type( $post_id ) ) ) {
+            $this->log->debug( "Post $post_id has already an entity type." );
+
+            return;
+        }
+
+        $this->log->debug( "Setting `Article` entity type for post $post_id." );
+
+        // Finally set the default entity type.
+
+        // For entities use a Thing
+        if ( 'entity' === $post->post_type ) {
+            $this->entity_type_service->set( $post_id, 'http://schema.org/Thing' );
+        } else {
+            // Get the entity types.
+            $entity_types = self::get_entity_types( $post->post_type );
+
+            // Set the entity type.
+            foreach ( $entity_types as $entity_type ) {
+                $this->entity_type_service->set( $post_id, $entity_type, false );
+            }
+        }
+
+    }
+
+    /**
+     * Get the entity types for a post type.
+     *
+     * @param string $post_type The post type.
+     *
+     * @return array An array of entity types.
+     * @since 3.20.0
+     */
+    public static function get_entity_types( $post_type ) {
+
+        /**
+         * Get the default entity type.
+         *
+         * @param string $entity_type The preset entity type.
+         * @param string $post_type The post type.
+         *
+         * @since 3.20.0
+         */
+        $default_entity_type = apply_filters( 'wl_default_entity_type_for_post_type', 'http://schema.org/Article', $post_type );
+
+        /**
+         * Get the default entity types.
+         *
+         * Adding support to assign more than one entity type.
+         *
+         * @param array $entity_types The default entity types.
+         * @param string $post_type The post type.
+         *
+         * @since 3.20.0
+         *
+         * @see Wordlift_Mapping_Service
+         */
+        $entity_types = apply_filters( 'wl_default_entity_types_for_post_type', array( $default_entity_type ), $post_type );
+
+        return $entity_types;
+    }
 
 }

Spacing +18 added lines, -18 removed lines

@@ -45,9 +45,9 @@ 
 	 *
 	 * @since 3.15.0
 	 */
-	public function __construct( $entity_type_service ) {
+	public function __construct($entity_type_service) {
 
-		$this->log = Wordlift_Log_Service::get_logger( 'Wordlift_Entity_Type_Adapter' );
+		$this->log = Wordlift_Log_Service::get_logger('Wordlift_Entity_Type_Adapter');
 
 		$this->entity_type_service = $entity_type_service;
 
@@ -69,8 +69,8 @@ 
 	 *
 	 * @return string The default entity type depending on the post type.
 	 */
-	public function default_entity_type_callback( $entity_type, $post_type ) {
-		if ( 'product' === $post_type ) {
+	public function default_entity_type_callback($entity_type, $post_type) {
+		if ('product' === $post_type) {
 			return 'http://schema.org/Product';
 		}
 
@@ -90,36 +90,36 @@ 
 	 *
 	 * @since 3.15.0
 	 */
-	public function save_post( $post_id, $post ) {
+	public function save_post($post_id, $post) {
 
-		if ( ! Wordlift_Entity_Type_Service::is_valid_entity_post_type( $post->post_type ) ) {
-			$this->log->debug( "Ignoring `{$post->post_type}` post type." );
+		if ( ! Wordlift_Entity_Type_Service::is_valid_entity_post_type($post->post_type)) {
+			$this->log->debug("Ignoring `{$post->post_type}` post type.");
 
 			// Bail out if the post can not be an entity.
 			return;
 		}
 
 		// Bail out if the post already has an entity type.
-		if ( apply_filters( 'wl_entity_type_adapter__save_post__has_entity_type', $this->entity_type_service->has_entity_type( $post_id ) ) ) {
-			$this->log->debug( "Post $post_id has already an entity type." );
+		if (apply_filters('wl_entity_type_adapter__save_post__has_entity_type', $this->entity_type_service->has_entity_type($post_id))) {
+			$this->log->debug("Post $post_id has already an entity type.");
 
 			return;
 		}
 
-		$this->log->debug( "Setting `Article` entity type for post $post_id." );
+		$this->log->debug("Setting `Article` entity type for post $post_id.");
 
 		// Finally set the default entity type.
 
 		// For entities use a Thing
-		if ( 'entity' === $post->post_type ) {
-			$this->entity_type_service->set( $post_id, 'http://schema.org/Thing' );
+		if ('entity' === $post->post_type) {
+			$this->entity_type_service->set($post_id, 'http://schema.org/Thing');
 		} else {
 			// Get the entity types.
-			$entity_types = self::get_entity_types( $post->post_type );
+			$entity_types = self::get_entity_types($post->post_type);
 
 			// Set the entity type.
-			foreach ( $entity_types as $entity_type ) {
-				$this->entity_type_service->set( $post_id, $entity_type, false );
+			foreach ($entity_types as $entity_type) {
+				$this->entity_type_service->set($post_id, $entity_type, false);
 			}
 		}
 
@@ -133,7 +133,7 @@ 
 	 * @return array An array of entity types.
 	 * @since 3.20.0
 	 */
-	public static function get_entity_types( $post_type ) {
+	public static function get_entity_types($post_type) {
 
 		/**
 		 * Get the default entity type.
@@ -143,7 +143,7 @@ 
 		 *
 		 * @since 3.20.0
 		 */
-		$default_entity_type = apply_filters( 'wl_default_entity_type_for_post_type', 'http://schema.org/Article', $post_type );
+		$default_entity_type = apply_filters('wl_default_entity_type_for_post_type', 'http://schema.org/Article', $post_type);
 
 		/**
 		 * Get the default entity types.
@@ -157,7 +157,7 @@ 
 		 *
 		 * @see Wordlift_Mapping_Service
 		 */
-		$entity_types = apply_filters( 'wl_default_entity_types_for_post_type', array( $default_entity_type ), $post_type );
+		$entity_types = apply_filters('wl_default_entity_types_for_post_type', array($default_entity_type), $post_type);
 
 		return $entity_types;
 	}

src/includes/class-wordlift-deactivator-feedback.php

Indentation +141 added lines, -141 removed lines

@@ -21,145 +21,145 @@
  */
 class Wordlift_Deactivator_Feedback {
 
-	/**
-	 * A {@link Wordlift_Log_Service} instance.
-	 *
-	 * @since  3.19.0
-	 * @access private
-	 * @var \Wordlift_Log_Service $log A {@link Wordlift_Log_Service} instance.
-	 */
-	private $log;
-
-	/**
-	 * Wordlift_Deactivator_Feedback constructor.
-	 *
-	 * @since 3.19.0
-	 */
-	public function __construct() {
-
-		$this->log = Wordlift_Log_Service::get_logger( 'Wordlift_Deactivator_Feedback' );
-
-	}
-
-	/**
-	 * Checks whether we have permissions to show the popup.
-	 *
-	 * @return  bool `true` if we have permissions, false otherwise.
-	 * @version 3.19.0
-	 */
-	private function has_permission_to_show_popup() {
-		// Get the current page.
-		global $pagenow;
-
-		// Bail if the user doesn't have permissions
-		// or if it's not the plugins page.
-		if ( 'plugins.php' !== $pagenow ) {
-			return false;
-		}
-
-		// Get the user preferences. We shouldn't show the feedback popup
-		// if we don't have permissions for that.
-		$user_preferences = Wordlift_Configuration_Service::get_instance()->get_diagnostic_preferences();
-
-		// Bail. We don't have preferences to show the popup.
-		if ( 'yes' !== $user_preferences ) {
-			return false;
-		}
-
-		return true;
-	}
-
-	/**
-	 * Render the feedback popup in the footer.
-	 *
-	 * @return  void
-	 * @version 3.19.0
-	 */
-	public function render_feedback_popup() {
-		// Bail if we don't have permissions to show the popup.
-		if ( ! $this->has_permission_to_show_popup() ) {
-			return;
-		}
-		// Include the partial.
-		include plugin_dir_path( __FILE__ ) . '../admin/partials/wordlift-admin-deactivation-feedback-popup.php';
-	}
-
-	/**
-	 * Enqueue required popup scripts and styles.
-	 *
-	 * @return  void
-	 * @version 3.19.0
-	 */
-	public function enqueue_popup_scripts() {
-		// Bail if we don't have permissions to show the popup.
-		if ( ! $this->has_permission_to_show_popup() ) {
-			return;
-		}
-
-		wp_enqueue_style( 'wordlift-admin-feedback-popup', plugin_dir_url( __DIR__ ) . 'admin/css/wordlift-admin-feedback-popup.css', array(), WORDLIFT_VERSION );
-		wp_enqueue_script( 'wordlift-admin-feedback-popup', plugin_dir_url( __DIR__ ) . 'admin/js/wordlift-admin-feedback-popup.js', array( 'jquery' ), WORDLIFT_VERSION, false );
-	}
-
-	/**
-	 * Handle the deactivation ajax call
-	 * and perform a request to external server.
-	 *
-	 * @return  void
-	 * @version 3.19.0
-	 */
-	public function wl_deactivation_feedback() {
-		// Bail if the nonce is not valid.
-		if (
-			empty( $_POST['wl_deactivation_feedback_nonce'] ) || // The nonce doens't exists.
-			! wp_verify_nonce( sanitize_text_field( wp_unslash( $_POST['wl_deactivation_feedback_nonce'] ) ), 'wl_deactivation_feedback_nonce' ) // The nonce is invalid.
-		) {
-			wp_send_json_error( __( 'Nonce Security Check Failed!', 'wordlift' ) );
-		}
-
-		// We allow user to deactivate without providing a reason
-		// so bail and send success response.
-		if ( empty( $_POST['code'] ) ) {
-			wp_send_json_success();
-		}
-
-		$plugin_data = get_plugin_data( plugin_dir_path( __DIR__ ) . 'wordlift.php', false, false );
-
-		// Prepare the options.
-		$options = array(
-			// The deactivation reason.
-			'code'             => sanitize_text_field( wp_unslash( (string) $_POST['code'] ) ),
-			// Additional information if provided.
-			'details'          => ( ! empty( $_POST['details'] ) ) ? sanitize_text_field( wp_unslash( (string) $_POST['details'] ) ) : '',
-			// The website url.
-			'url'              => get_bloginfo( 'url' ),
-			// WP version.
-			'wordpressVersion' => get_bloginfo( 'version' ),
-			// WL version.
-			'wordliftVersion'  => $plugin_data['Version'],
-			// The admin email.
-			'email'            => get_bloginfo( 'admin_email' ),
-		);
-
-		$response = wp_remote_post(
-			Wordlift_Configuration_Service::get_instance()->get_deactivation_feedback_url(),
-			array(
-				'method'  => 'POST',
-				'body'    => wp_json_encode( $options ),
-				'headers' => array( 'Content-Type' => 'application/json; charset=utf-8' ),
-			)
-		);
-
-		$code    = wp_remote_retrieve_response_code( $response );
-		$message = wp_remote_retrieve_response_message( $response );
-
-		// Add message to the error log if the response code is not 200.
-		if ( 201 !== $code ) {
-			// Write the error in the logs.
-			$this->log->error( 'An error occurred while requesting a feedback endpoint error_code: ' . $code . ' message: ' . $message );
-		}
-
-		// We should send success message even when the feedback is not
-		// send, because otherwise the plugin cannot be deactivated.
-		wp_send_json_success();
-	}
+    /**
+     * A {@link Wordlift_Log_Service} instance.
+     *
+     * @since  3.19.0
+     * @access private
+     * @var \Wordlift_Log_Service $log A {@link Wordlift_Log_Service} instance.
+     */
+    private $log;
+
+    /**
+     * Wordlift_Deactivator_Feedback constructor.
+     *
+     * @since 3.19.0
+     */
+    public function __construct() {
+
+        $this->log = Wordlift_Log_Service::get_logger( 'Wordlift_Deactivator_Feedback' );
+
+    }
+
+    /**
+     * Checks whether we have permissions to show the popup.
+     *
+     * @return  bool `true` if we have permissions, false otherwise.
+     * @version 3.19.0
+     */
+    private function has_permission_to_show_popup() {
+        // Get the current page.
+        global $pagenow;
+
+        // Bail if the user doesn't have permissions
+        // or if it's not the plugins page.
+        if ( 'plugins.php' !== $pagenow ) {
+            return false;
+        }
+
+        // Get the user preferences. We shouldn't show the feedback popup
+        // if we don't have permissions for that.
+        $user_preferences = Wordlift_Configuration_Service::get_instance()->get_diagnostic_preferences();
+
+        // Bail. We don't have preferences to show the popup.
+        if ( 'yes' !== $user_preferences ) {
+            return false;
+        }
+
+        return true;
+    }
+
+    /**
+     * Render the feedback popup in the footer.
+     *
+     * @return  void
+     * @version 3.19.0
+     */
+    public function render_feedback_popup() {
+        // Bail if we don't have permissions to show the popup.
+        if ( ! $this->has_permission_to_show_popup() ) {
+            return;
+        }
+        // Include the partial.
+        include plugin_dir_path( __FILE__ ) . '../admin/partials/wordlift-admin-deactivation-feedback-popup.php';
+    }
+
+    /**
+     * Enqueue required popup scripts and styles.
+     *
+     * @return  void
+     * @version 3.19.0
+     */
+    public function enqueue_popup_scripts() {
+        // Bail if we don't have permissions to show the popup.
+        if ( ! $this->has_permission_to_show_popup() ) {
+            return;
+        }
+
+        wp_enqueue_style( 'wordlift-admin-feedback-popup', plugin_dir_url( __DIR__ ) . 'admin/css/wordlift-admin-feedback-popup.css', array(), WORDLIFT_VERSION );
+        wp_enqueue_script( 'wordlift-admin-feedback-popup', plugin_dir_url( __DIR__ ) . 'admin/js/wordlift-admin-feedback-popup.js', array( 'jquery' ), WORDLIFT_VERSION, false );
+    }
+
+    /**
+     * Handle the deactivation ajax call
+     * and perform a request to external server.
+     *
+     * @return  void
+     * @version 3.19.0
+     */
+    public function wl_deactivation_feedback() {
+        // Bail if the nonce is not valid.
+        if (
+            empty( $_POST['wl_deactivation_feedback_nonce'] ) || // The nonce doens't exists.
+            ! wp_verify_nonce( sanitize_text_field( wp_unslash( $_POST['wl_deactivation_feedback_nonce'] ) ), 'wl_deactivation_feedback_nonce' ) // The nonce is invalid.
+        ) {
+            wp_send_json_error( __( 'Nonce Security Check Failed!', 'wordlift' ) );
+        }
+
+        // We allow user to deactivate without providing a reason
+        // so bail and send success response.
+        if ( empty( $_POST['code'] ) ) {
+            wp_send_json_success();
+        }
+
+        $plugin_data = get_plugin_data( plugin_dir_path( __DIR__ ) . 'wordlift.php', false, false );
+
+        // Prepare the options.
+        $options = array(
+            // The deactivation reason.
+            'code'             => sanitize_text_field( wp_unslash( (string) $_POST['code'] ) ),
+            // Additional information if provided.
+            'details'          => ( ! empty( $_POST['details'] ) ) ? sanitize_text_field( wp_unslash( (string) $_POST['details'] ) ) : '',
+            // The website url.
+            'url'              => get_bloginfo( 'url' ),
+            // WP version.
+            'wordpressVersion' => get_bloginfo( 'version' ),
+            // WL version.
+            'wordliftVersion'  => $plugin_data['Version'],
+            // The admin email.
+            'email'            => get_bloginfo( 'admin_email' ),
+        );
+
+        $response = wp_remote_post(
+            Wordlift_Configuration_Service::get_instance()->get_deactivation_feedback_url(),
+            array(
+                'method'  => 'POST',
+                'body'    => wp_json_encode( $options ),
+                'headers' => array( 'Content-Type' => 'application/json; charset=utf-8' ),
+            )
+        );
+
+        $code    = wp_remote_retrieve_response_code( $response );
+        $message = wp_remote_retrieve_response_message( $response );
+
+        // Add message to the error log if the response code is not 200.
+        if ( 201 !== $code ) {
+            // Write the error in the logs.
+            $this->log->error( 'An error occurred while requesting a feedback endpoint error_code: ' . $code . ' message: ' . $message );
+        }
+
+        // We should send success message even when the feedback is not
+        // send, because otherwise the plugin cannot be deactivated.
+        wp_send_json_success();
+    }
 }

Spacing +24 added lines, -24 removed lines

@@ -37,7 +37,7 @@ 
 	 */
 	public function __construct() {
 
-		$this->log = Wordlift_Log_Service::get_logger( 'Wordlift_Deactivator_Feedback' );
+		$this->log = Wordlift_Log_Service::get_logger('Wordlift_Deactivator_Feedback');
 
 	}
 
@@ -53,7 +53,7 @@ 
 
 		// Bail if the user doesn't have permissions
 		// or if it's not the plugins page.
-		if ( 'plugins.php' !== $pagenow ) {
+		if ('plugins.php' !== $pagenow) {
 			return false;
 		}
 
@@ -62,7 +62,7 @@ 
 		$user_preferences = Wordlift_Configuration_Service::get_instance()->get_diagnostic_preferences();
 
 		// Bail. We don't have preferences to show the popup.
-		if ( 'yes' !== $user_preferences ) {
+		if ('yes' !== $user_preferences) {
 			return false;
 		}
 
@@ -77,11 +77,11 @@ 
 	 */
 	public function render_feedback_popup() {
 		// Bail if we don't have permissions to show the popup.
-		if ( ! $this->has_permission_to_show_popup() ) {
+		if ( ! $this->has_permission_to_show_popup()) {
 			return;
 		}
 		// Include the partial.
-		include plugin_dir_path( __FILE__ ) . '../admin/partials/wordlift-admin-deactivation-feedback-popup.php';
+		include plugin_dir_path(__FILE__).'../admin/partials/wordlift-admin-deactivation-feedback-popup.php';
 	}
 
 	/**
@@ -92,12 +92,12 @@ 
 	 */
 	public function enqueue_popup_scripts() {
 		// Bail if we don't have permissions to show the popup.
-		if ( ! $this->has_permission_to_show_popup() ) {
+		if ( ! $this->has_permission_to_show_popup()) {
 			return;
 		}
 
-		wp_enqueue_style( 'wordlift-admin-feedback-popup', plugin_dir_url( __DIR__ ) . 'admin/css/wordlift-admin-feedback-popup.css', array(), WORDLIFT_VERSION );
-		wp_enqueue_script( 'wordlift-admin-feedback-popup', plugin_dir_url( __DIR__ ) . 'admin/js/wordlift-admin-feedback-popup.js', array( 'jquery' ), WORDLIFT_VERSION, false );
+		wp_enqueue_style('wordlift-admin-feedback-popup', plugin_dir_url(__DIR__).'admin/css/wordlift-admin-feedback-popup.css', array(), WORDLIFT_VERSION);
+		wp_enqueue_script('wordlift-admin-feedback-popup', plugin_dir_url(__DIR__).'admin/js/wordlift-admin-feedback-popup.js', array('jquery'), WORDLIFT_VERSION, false);
 	}
 
 	/**
@@ -110,52 +110,52 @@ 
 	public function wl_deactivation_feedback() {
 		// Bail if the nonce is not valid.
 		if (
-			empty( $_POST['wl_deactivation_feedback_nonce'] ) || // The nonce doens't exists.
-			! wp_verify_nonce( sanitize_text_field( wp_unslash( $_POST['wl_deactivation_feedback_nonce'] ) ), 'wl_deactivation_feedback_nonce' ) // The nonce is invalid.
+			empty($_POST['wl_deactivation_feedback_nonce']) || // The nonce doens't exists.
+			! wp_verify_nonce(sanitize_text_field(wp_unslash($_POST['wl_deactivation_feedback_nonce'])), 'wl_deactivation_feedback_nonce') // The nonce is invalid.
 		) {
-			wp_send_json_error( __( 'Nonce Security Check Failed!', 'wordlift' ) );
+			wp_send_json_error(__('Nonce Security Check Failed!', 'wordlift'));
 		}
 
 		// We allow user to deactivate without providing a reason
 		// so bail and send success response.
-		if ( empty( $_POST['code'] ) ) {
+		if (empty($_POST['code'])) {
 			wp_send_json_success();
 		}
 
-		$plugin_data = get_plugin_data( plugin_dir_path( __DIR__ ) . 'wordlift.php', false, false );
+		$plugin_data = get_plugin_data(plugin_dir_path(__DIR__).'wordlift.php', false, false);
 
 		// Prepare the options.
 		$options = array(
 			// The deactivation reason.
-			'code'             => sanitize_text_field( wp_unslash( (string) $_POST['code'] ) ),
+			'code'             => sanitize_text_field(wp_unslash((string) $_POST['code'])),
 			// Additional information if provided.
-			'details'          => ( ! empty( $_POST['details'] ) ) ? sanitize_text_field( wp_unslash( (string) $_POST['details'] ) ) : '',
+			'details'          => ( ! empty($_POST['details'])) ? sanitize_text_field(wp_unslash((string) $_POST['details'])) : '',
 			// The website url.
-			'url'              => get_bloginfo( 'url' ),
+			'url'              => get_bloginfo('url'),
 			// WP version.
-			'wordpressVersion' => get_bloginfo( 'version' ),
+			'wordpressVersion' => get_bloginfo('version'),
 			// WL version.
 			'wordliftVersion'  => $plugin_data['Version'],
 			// The admin email.
-			'email'            => get_bloginfo( 'admin_email' ),
+			'email'            => get_bloginfo('admin_email'),
 		);
 
 		$response = wp_remote_post(
 			Wordlift_Configuration_Service::get_instance()->get_deactivation_feedback_url(),
 			array(
 				'method'  => 'POST',
-				'body'    => wp_json_encode( $options ),
-				'headers' => array( 'Content-Type' => 'application/json; charset=utf-8' ),
+				'body'    => wp_json_encode($options),
+				'headers' => array('Content-Type' => 'application/json; charset=utf-8'),
 			)
 		);
 
-		$code    = wp_remote_retrieve_response_code( $response );
-		$message = wp_remote_retrieve_response_message( $response );
+		$code    = wp_remote_retrieve_response_code($response);
+		$message = wp_remote_retrieve_response_message($response);
 
 		// Add message to the error log if the response code is not 200.
-		if ( 201 !== $code ) {
+		if (201 !== $code) {
 			// Write the error in the logs.
-			$this->log->error( 'An error occurred while requesting a feedback endpoint error_code: ' . $code . ' message: ' . $message );
+			$this->log->error('An error occurred while requesting a feedback endpoint error_code: '.$code.' message: '.$message);
 		}
 
 		// We should send success message even when the feedback is not

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

Indentation +1235 added lines, -1235 removed lines

@@ -18,1240 +18,1240 @@
  */
 class Wordlift_Schema_Service {
 
-	/**
-	 * The 'location created' field name.
-	 *
-	 * @since 3.5.0
-	 */
-	const FIELD_LOCATION_CREATED = 'wl_location_created';
-
-	/**
-	 * The 'topic' field name.
-	 *
-	 * @since 3.5.0
-	 */
-	const FIELD_TOPIC = 'wl_topic';
-
-	/**
-	 * The 'author' field name.
-	 *
-	 * @since 3.1.0
-	 */
-	const FIELD_AUTHOR = 'wl_author';
-
-	/**
-	 * The 'same as' field name.
-	 *
-	 * @since 3.1.0
-	 */
-	const FIELD_SAME_AS = 'entity_same_as';
-
-	/**
-	 * The 'date start' field name.
-	 *
-	 * @since 3.1.0
-	 */
-	const FIELD_DATE_START = 'wl_cal_date_start';
-
-	/**
-	 * The 'date end' field name.
-	 *
-	 * @since 3.1.0
-	 */
-	const FIELD_DATE_END = 'wl_cal_date_end';
-
-	/**
-	 * The 'location' field name.
-	 *
-	 * @since 3.1.0
-	 */
-	const FIELD_LOCATION = 'wl_location';
-
-	/**
-	 * The 'founder' field name.
-	 *
-	 * @since 3.1.0
-	 */
-	const FIELD_FOUNDER = 'wl_founder';
-
-	/**
-	 * The 'knows' field name.
-	 *
-	 * @since 3.1.0
-	 */
-	const FIELD_KNOWS = 'wl_knows';
-
-	/**
-	 * The 'birth date' field name.
-	 *
-	 * @since 3.1.0
-	 */
-	const FIELD_BIRTH_DATE = 'wl_birth_date';
-
-	/**
-	 * The 'birth place' field name.
-	 *
-	 * @since 3.1.0
-	 */
-	const FIELD_BIRTH_PLACE = 'wl_birth_place';
-
-	/**
-	 * The 'latitude' field name.
-	 *
-	 * @since 3.1.0
-	 */
-	const FIELD_GEO_LATITUDE = 'wl_geo_latitude';
-
-	/**
-	 * The 'longitude' field name.
-	 *
-	 * @since 3.1.0
-	 */
-	const FIELD_GEO_LONGITUDE = 'wl_geo_longitude';
-
-	/**
-	 * The 'streetAddress' field name.
-	 *
-	 * @since 3.1.0
-	 */
-	const FIELD_ADDRESS = 'wl_address';
-
-	/**
-	 * The 'postOfficeBoxNumber' field name.
-	 *
-	 * @since 3.3.0
-	 */
-	const FIELD_ADDRESS_PO_BOX = 'wl_address_post_office_box';
-
-	/**
-	 * The 'postalCode' field name.
-	 *
-	 * @since 3.3.0
-	 */
-	const FIELD_ADDRESS_POSTAL_CODE = 'wl_address_postal_code';
-
-	/**
-	 * The 'addressLocality' field name.
-	 *
-	 * @since 3.3.0
-	 */
-	const FIELD_ADDRESS_LOCALITY = 'wl_address_locality';
-	/**
-	 * The 'addressRegion' field name.
-	 *
-	 * @since 3.3.0
-	 */
-	const FIELD_ADDRESS_REGION = 'wl_address_region';
-
-	/**
-	 * The 'addressCountry' field name.
-	 *
-	 * @since 3.3.0
-	 */
-	const FIELD_ADDRESS_COUNTRY = 'wl_address_country';
-
-	/**
-	 * The 'entity type' field name.
-	 *
-	 * @since 3.1.0
-	 */
-	const FIELD_ENTITY_TYPE = 'wl_entity_type_uri';
-
-	/**
-	 * The 'email' field name.
-	 *
-	 * @since 3.2.0
-	 */
-	const FIELD_EMAIL = 'wl_email';
-
-	/**
-	 * The 'affiliation' field name.
-	 *
-	 * @since 3.2.0
-	 */
-	const FIELD_AFFILIATION = 'wl_affiliation';
-
-	/**
-	 * The 'telephone' field name.
-	 *
-	 * @since 3.8.0
-	 */
-	const FIELD_TELEPHONE = 'wl_schema_telephone';
-
-	/**
-	 * The 'legalName' field name.
-	 *
-	 * @since 3.12.0
-	 */
-	const FIELD_LEGAL_NAME = 'wl_schema_legal_name';
-
-	/**
-	 * The 'recipeCuisine' field name.
-	 *
-	 * @since 3.14.0
-	 */
-	const FIELD_RECIPE_CUISINE = 'wl_schema_recipe_cuisine';
-
-	/**
-	 * The 'recipeIngredient' field name.
-	 *
-	 * @since 3.14.0
-	 */
-	const FIELD_RECIPE_INGREDIENT = 'wl_schema_recipe_ingredient';
-
-	/**
-	 * The 'calories' field name.
-	 *
-	 * @since 3.14.0
-	 */
-	const FIELD_NUTRITION_INFO_CALORIES = 'wl_schema_nutrition_information_calories';
-
-	/**
-	 * The 'recipeInstructions' field name.
-	 *
-	 * @since 3.14.0
-	 */
-	const FIELD_RECIPE_INSTRUCTIONS = 'wl_schema_recipe_instructions';
-
-	/**
-	 * The 'recipeYield' field name.
-	 *
-	 * @since 3.14.0
-	 */
-	const FIELD_RECIPE_YIELD = 'wl_schema_recipe_yield';
-
-	/**
-	 * The 'prepTime' field name.
-	 *
-	 * @since 3.14.0
-	 */
-	const FIELD_PREP_TIME = 'wl_schema_prep_time';
-
-	/**
-	 * The 'cookTime' field name.
-	 *
-	 * @since 3.14.0
-	 */
-	const FIELD_COOK_TIME = 'wl_schema_cook_time';
-
-	/**
-	 * The 'totalTime' field name.
-	 *
-	 * @since 3.14.0
-	 */
-	const FIELD_TOTAL_TIME = 'wl_schema_total_time';
-
-	/**
-	 * The 'performer' field name.
-	 *
-	 * @since 3.18.0
-	 */
-	const FIELD_PERFORMER = 'wl_schema_performer';
-
-	/**
-	 * The 'offers' field name.
-	 *
-	 * @since 3.18.0
-	 */
-	const FIELD_OFFERS = 'wl_schema_offers';
-
-	/**
-	 * The 'availablity' field name.
-	 *
-	 * @since 3.18.0
-	 */
-	const FIELD_AVAILABILITY = 'wl_schema_availability';
-
-	/**
-	 * The 'inventoryLevel' field name.
-	 *
-	 * @since 3.18.0
-	 */
-	const FIELD_INVENTORY_LEVEL = 'wl_schema_inventory_level';
-
-	/**
-	 * The 'price' field name.
-	 *
-	 * @since 3.18.0
-	 */
-	const FIELD_PRICE = 'wl_schema_price';
-
-	/**
-	 * The 'priceCurrency' field name.
-	 *
-	 * @since 3.18.0
-	 */
-	const FIELD_PRICE_CURRENCY = 'wl_schema_price_currency';
-
-	/**
-	 * The 'availabilityStarts' field name.
-	 *
-	 * @since 3.18.0
-	 */
-	const FIELD_AVAILABILITY_STARTS = 'wl_schema_availability_starts';
-
-	/**
-	 * The 'availabilityEnds' field name.
-	 *
-	 * @since 3.18.0
-	 */
-	const FIELD_AVAILABILITY_ENDS = 'wl_schema_availability_ends';
-
-	/**
-	 * The 'validFrom' field name.
-	 *
-	 * @since 3.18.0
-	 */
-	const FIELD_VALID_FROM = 'wl_schema_valid_from';
-
-	/**
-	 * The 'priceValidUntil' field name.
-	 *
-	 * @since 3.18.0
-	 */
-	const FIELD_PRICE_VALID_UNTIL = 'wl_schema_valid_until';
-
-	/**
-	 * The 'itemOffered' field name.
-	 *
-	 * @since 3.18.0
-	 */
-	const FIELD_ITEM_OFFERED = 'wl_schema_item_offered';
-
-	/**
-	 * The 'URI' data type name.
-	 *
-	 * @since 3.1.0
-	 */
-	const DATA_TYPE_URI = 'uri';
-
-	/**
-	 * The 'date' data type name.
-	 *
-	 * @since 3.1.0
-	 */
-	const DATA_TYPE_DATE = 'date';
-
-	/**
-	 * The 'time' data type name.
-	 *
-	 * @since 3.14.0
-	 */
-	const DATA_TYPE_DURATION = 'duration';
-
-	/**
-	 * The 'double' data type name.
-	 *
-	 * @since 3.1.0
-	 */
-	const DATA_TYPE_DOUBLE = 'double';
-
-	/**
-	 * The 'string' data type name.
-	 *
-	 * @since 3.1.0
-	 */
-	const DATA_TYPE_STRING = 'string';
-
-	/**
-	 * The multiline text data type name.
-	 *
-	 * @since 3.14.0
-	 */
-	const DATA_TYPE_MULTILINE = 'multiline';
-
-	/**
-	 * The schema.org Event type URI.
-	 *
-	 * @since 3.1.0
-	 */
-	const SCHEMA_EVENT_TYPE = 'http://schema.org/Event';
-
-	/**
-	 * The schema.org Offer type URI.
-	 *
-	 * @since 3.18.0
-	 */
-	const SCHEMA_OFFER_TYPE = 'http://schema.org/Offer';
-
-	/**
-	 * WordLift's schema.
-	 *
-	 * @since  3.1.0
-	 * @access private
-	 * @var array $schema WordLift's schema.
-	 */
-	private $schema;
-
-	/**
-	 * The Log service.
-	 *
-	 * @since  3.1.0
-	 * @access private
-	 * @var \Wordlift_Log_Service $log The Log service.
-	 */
-	private $log;
-
-	/**
-	 * Wordlift_Schema_Service constructor.
-	 *
-	 * @since 3.1.0
-	 */
-	protected function __construct() {
-
-		$this->log = Wordlift_Log_Service::get_logger( 'Wordlift_Schema_Service' );
-
-		/**
-		 * Alter the configured schemas.
-		 *
-		 * Enable 3rd parties to alter WordLift's schemas array.
-		 *
-		 * @param array $schemas The array of schemas.
-		 *
-		 * @since  3.19.1
-		 */
-		$this->schema = apply_filters(
-			'wl_schemas',
-			array(
-				'article'        => $this->get_article_schema(),
-				'thing'          => $this->get_thing_schema(),
-				'creative-work'  => $this->get_creative_work_schema(),
-				'event'          => $this->get_event_schema(),
-				'organization'   => $this->get_organization_schema(),
-				'person'         => $this->get_person_schema(),
-				'place'          => $this->get_place_schema(),
-				'local-business' => $this->get_local_business_schema(),
-				'recipe'         => $this->get_recipe_schema(),
-				'web-page'       => $this->get_web_page_schema(),
-				'offer'          => $this->get_offer_schema(),
-			)
-		);
-
-		// Create a singleton instance of the Schema service, useful to provide static functions to global functions.
-		self::$instance = $this;
-
-	}
-
-	/**
-	 * The Schema service singleton instance.
-	 *
-	 * @since  3.1.0
-	 * @access private
-	 * @var Wordlift_Schema_Service $instance The Schema service singleton instance.
-	 */
-	private static $instance = null;
-
-	/**
-	 * Get a reference to the Schema service.
-	 *
-	 * @return Wordlift_Schema_Service A reference to the Schema service.
-	 * @since 3.1.0
-	 */
-	public static function get_instance() {
-		if ( ! isset( self::$instance ) ) {
-			self::$instance = new self();
-		}
-
-		return self::$instance;
-	}
-
-	/**
-	 * Get the properties for a field with the specified key. The key is used as
-	 * meta key when the field's value is stored in WordPress meta data table.
-	 *
-	 * @param string $key The field's key.
-	 *
-	 * @return null|array An array of field's properties or null if the field is not found.
-	 * @since 3.6.0
-	 */
-	public function get_field( $key ) {
-
-		// Parse each schema's fields until we find the one we're looking for, then
-		// return its properties.
-		// phpcs:ignore VariableAnalysis.CodeAnalysis.VariableAnalysis.UnusedVariable
-		foreach ( $this->schema as $_ => $schema ) {
-
-			if ( ! isset( $schema['custom_fields'] ) ) {
-				break;
-			}
-
-			foreach ( $schema['custom_fields'] as $field => $props ) {
-				if ( $key === $field ) {
-					return $props;
-				}
-			}
-		}
-
-		return null;
-	}
-
-	/**
-	 * Get the WordLift's schema.
-	 *
-	 * @param string $name The schema name.
-	 *
-	 * @return array|null An array with the schema configuration or NULL if the schema is not found.
-	 *
-	 * @since 3.1.0
-	 */
-	public function get_schema( $name ) {
-		// Check if the schema exists and, if not, return NULL.
-		if ( ! isset( $this->schema[ $name ] ) ) {
-			return null;
-		}
-
-		// Return the requested schema.
-		return $this->schema[ $name ];
-	}
-
-	/**
-	 * Get the WordLift's schema trough schema type uri.
-	 *
-	 * @param string $uri The schema uri.
-	 *
-	 * @return array|null An array with the schema configuration or NULL if the schema is not found.
-	 *
-	 * @since 3.3.0
-	 */
-	public function get_schema_by_uri( $uri ) {
-
-		// phpcs:ignore VariableAnalysis.CodeAnalysis.VariableAnalysis.UnusedVariable
-		foreach ( $this->schema as $name => $schema ) {
-			if ( $schema['uri'] === $uri ) {
-				return $schema;
-			}
-		}
-
-		return null;
-	}
-
-	/**
-	 * Get the 'thing' schema.
-	 *
-	 * @return array An array with the schema configuration.
-	 *
-	 * @since 3.1.0
-	 */
-	private function get_thing_schema() {
-
-		return array(
-			'css_class'     => 'wl-thing',
-			'uri'           => 'http://schema.org/Thing',
-			'same_as'       => array( '*' ),
-			// set as default.
-			'custom_fields' => array(
-				self::FIELD_SAME_AS => array(
-					'predicate'   => 'http://schema.org/sameAs',
-					'type'        => self::DATA_TYPE_URI,
-					'export_type' => 'http://schema.org/Thing',
-					'constraints' => array(
-						'cardinality' => INF,
-					),
-					// We need a custom metabox.
-					'input_field' => 'sameas',
-				),
-				// Add the schema:url property.
-				Wordlift_Schema_Url_Property_Service::META_KEY => Wordlift_Schema_Url_Property_Service::get_instance()
-																									  ->get_compat_definition(),
-			),
-			// {{sameAs}} not present in the microdata template,
-			// because it is treated separately in *wl_content_embed_item_microdata*
-			'templates'     => array(
-				'subtitle' => '{{id}}',
-			),
-		);
-
-	}
-
-	/**
-	 * Get the 'web-page' schema.
-	 *
-	 * @return array An array with the schema configuration.
-	 *
-	 * @since 3.18.0
-	 */
-	private function get_web_page_schema() {
-
-		return array(
-			'css_class' => 'wl-webpage',
-			'uri'       => 'http://schema.org/WebPage',
-		);
-
-	}
-
-	/**
-	 * Get the 'creative work' schema.
-	 *
-	 * @return array An array with the schema configuration.
-	 *
-	 * @since 3.1.0
-	 */
-	private function get_creative_work_schema() {
-
-		$schema = array(
-			'label'         => 'CreativeWork',
-			'description'   => 'A creative work (or a Music Album).',
-			'parents'       => array( 'thing' ),
-			// Give term slug as parent.
-			'css_class'     => 'wl-creative-work',
-			'uri'           => 'http://schema.org/CreativeWork',
-			'same_as'       => array(
-				'http://schema.org/MusicAlbum',
-				'http://schema.org/Product',
-			),
-			'custom_fields' => array(
-				self::FIELD_AUTHOR => array(
-					'predicate'   => 'http://schema.org/author',
-					'type'        => self::DATA_TYPE_URI,
-					'export_type' => 'http://schema.org/Person',
-					'constraints' => array(
-						'uri_type'    => array( 'Person', 'Organization' ),
-						'cardinality' => INF,
-					),
-				),
-			),
-			'templates'     => array(
-				'subtitle' => '{{id}}',
-			),
-		);
-
-		// Merge the custom fields with those provided by the thing schema.
-		$parent_schema           = $this->get_thing_schema();
-		$schema['custom_fields'] = array_merge( $schema['custom_fields'], $parent_schema['custom_fields'] );
-
-		return $schema;
-	}
-
-	/**
-	 * Get the 'event' schema.
-	 *
-	 * @return array An array with the schema configuration.
-	 *
-	 * @since 3.1.0
-	 */
-	private function get_event_schema() {
-
-		$schema = array(
-			'label'         => 'Event',
-			'description'   => 'An event . ',
-			'parents'       => array( 'thing' ),
-			'css_class'     => 'wl-event',
-			'uri'           => self::SCHEMA_EVENT_TYPE,
-			'same_as'       => array( 'http://dbpedia.org/ontology/Event' ),
-			'custom_fields' => array(
-				self::FIELD_DATE_START => array(
-					'predicate'   => 'http://schema.org/startDate',
-					'type'        => self::DATA_TYPE_DATE,
-					'export_type' => 'xsd:dateTime',
-					'constraints' => '',
-				),
-				self::FIELD_DATE_END   => array(
-					'predicate'   => 'http://schema.org/endDate',
-					'type'        => self::DATA_TYPE_DATE,
-					'export_type' => 'xsd:dateTime',
-					'constraints' => '',
-				),
-				self::FIELD_LOCATION   => array(
-					'predicate'   => 'http://schema.org/location',
-					'type'        => self::DATA_TYPE_URI,
-					'export_type' => 'http://schema.org/PostalAddress',
-					'constraints' => array(
-						'uri_type'    => array( 'Place', 'LocalBusiness' ),
-						'cardinality' => INF,
-					),
-				),
-				self::FIELD_PERFORMER  => array(
-					'predicate'   => 'http://schema.org/performer',
-					'type'        => self::DATA_TYPE_URI,
-					'export_type' => 'http://schema.org/Person',
-					'constraints' => array(
-						'uri_type'    => array( 'Person', 'Organization' ),
-						'cardinality' => INF,
-					),
-				),
-				self::FIELD_OFFERS     => array(
-					'predicate'   => 'http://schema.org/offers',
-					'type'        => self::DATA_TYPE_URI,
-					'export_type' => 'http://schema.org/Offer',
-					'constraints' => array(
-						'uri_type'    => array( 'Offer' ),
-						'cardinality' => INF,
-					),
-				),
-			),
-			'templates'     => array(
-				'subtitle' => '{{id}}',
-			),
-		);
-
-		// Merge the custom fields with those provided by the thing schema.
-		$parent_schema           = $this->get_thing_schema();
-		$schema['custom_fields'] = array_merge( $schema['custom_fields'], $parent_schema['custom_fields'] );
-
-		return $schema;
-	}
-
-	/**
-	 * Get the 'organization' schema.
-	 *
-	 * @return array An array with the schema configuration.
-	 *
-	 * @since 3.1.0
-	 */
-	private function get_organization_schema() {
-
-		$schema = array(
-			'label'         => 'Organization',
-			'description'   => 'An organization, including a government or a newspaper.',
-			'parents'       => array( 'thing' ),
-			'css_class'     => 'wl-organization',
-			'uri'           => 'http://schema.org/Organization',
-			'same_as'       => array(
-				'http://rdf.freebase.com/ns/organization.organization',
-				'http://rdf.freebase.com/ns/government.government',
-				'http://schema.org/Newspaper',
-			),
-			'custom_fields' => array(
-				self::FIELD_LEGAL_NAME          => array(
-					'predicate'   => 'http://schema.org/legalName',
-					'type'        => self::DATA_TYPE_STRING,
-					'export_type' => 'xsd:string',
-					'constraints' => '',
-				),
-				self::FIELD_FOUNDER             => array(
-					'predicate'   => 'http://schema.org/founder',
-					'type'        => self::DATA_TYPE_URI,
-					'export_type' => 'http://schema.org/Person',
-					'constraints' => array(
-						'uri_type'    => 'Person',
-						'cardinality' => INF,
-					),
-				),
-				self::FIELD_ADDRESS             => array(
-					'predicate'   => 'http://schema.org/streetAddress',
-					'type'        => self::DATA_TYPE_STRING,
-					'export_type' => 'xsd:string',
-					'constraints' => '',
-					// To build custom metabox.
-					'input_field' => 'address',
-				),
-				self::FIELD_ADDRESS_PO_BOX      => array(
-					'predicate'   => 'http://schema.org/postOfficeBoxNumber',
-					'type'        => self::DATA_TYPE_STRING,
-					'export_type' => 'xsd:string',
-					'constraints' => '',
-					// To build custom metabox.
-					'input_field' => 'address',
-				),
-				self::FIELD_ADDRESS_POSTAL_CODE => array(
-					'predicate'   => 'http://schema.org/postalCode',
-					'type'        => self::DATA_TYPE_STRING,
-					'export_type' => 'xsd:string',
-					'constraints' => '',
-					// To build custom metabox.
-					'input_field' => 'address',
-				),
-				self::FIELD_ADDRESS_LOCALITY    => array(
-					'predicate'   => 'http://schema.org/addressLocality',
-					'type'        => self::DATA_TYPE_STRING,
-					'export_type' => 'xsd:string',
-					'constraints' => '',
-					// To build custom metabox.
-					'input_field' => 'address',
-				),
-				self::FIELD_ADDRESS_REGION      => array(
-					'predicate'   => 'http://schema.org/addressRegion',
-					'type'        => self::DATA_TYPE_STRING,
-					'export_type' => 'xsd:string',
-					'constraints' => '',
-					// To build custom metabox.
-					'input_field' => 'address',
-				),
-				self::FIELD_ADDRESS_COUNTRY     => array(
-					'predicate'   => 'http://schema.org/addressCountry',
-					'type'        => self::DATA_TYPE_STRING,
-					'export_type' => 'xsd:string',
-					'constraints' => '',
-					// To build custom metabox.
-					'input_field' => 'address',
-				),
-				self::FIELD_EMAIL               => array(
-					'predicate'   => 'http://schema.org/email',
-					'type'        => self::DATA_TYPE_STRING,
-					'export_type' => 'xsd:string',
-					'constraints' => '',
-				),
-				self::FIELD_TELEPHONE           => array(
-					'predicate'   => 'http://schema.org/telephone',
-					'type'        => self::DATA_TYPE_STRING,
-					'export_type' => 'xsd:string',
-					'constraints' => '',
-				),
-			),
-			'templates'     => array(
-				'subtitle' => '{{id}}',
-			),
-		);
-
-		// Merge the custom fields with those provided by the thing schema.
-		$parent_schema           = $this->get_thing_schema();
-		$schema['custom_fields'] = array_merge( $schema['custom_fields'], $parent_schema['custom_fields'] );
-
-		return $schema;
-	}
-
-	/**
-	 * Get the 'person' schema.
-	 *
-	 * @return array An array with the schema configuration.
-	 *
-	 * @since 3.1.0
-	 */
-	private function get_person_schema() {
-
-		$schema = array(
-			'label'         => 'Person',
-			'description'   => 'A person (or a music artist).',
-			'parents'       => array( 'thing' ),
-			'css_class'     => 'wl-person',
-			'uri'           => 'http://schema.org/Person',
-			'same_as'       => array(
-				'http://rdf.freebase.com/ns/people.person',
-				'http://rdf.freebase.com/ns/music.artist',
-				'http://dbpedia.org/class/yago/LivingPeople',
-			),
-			'custom_fields' => array(
-				self::FIELD_KNOWS       => array(
-					'predicate'   => 'http://schema.org/knows',
-					'type'        => self::DATA_TYPE_URI,
-					'export_type' => 'http://schema.org/Person',
-					'constraints' => array(
-						'uri_type'    => 'Person',
-						'cardinality' => INF,
-					),
-				),
-				self::FIELD_BIRTH_DATE  => array(
-					'predicate'   => 'http://schema.org/birthDate',
-					'type'        => self::DATA_TYPE_DATE,
-					'export_type' => 'xsd:date',
-					'constraints' => '',
-				),
-				self::FIELD_BIRTH_PLACE => array(
-					'predicate'   => 'http://schema.org/birthPlace',
-					'type'        => self::DATA_TYPE_URI,
-					'export_type' => 'http://schema.org/Place',
-					'constraints' => array(
-						'uri_type' => 'Place',
-					),
-				),
-				self::FIELD_AFFILIATION => array(
-					'predicate'   => 'http://schema.org/affiliation',
-					'type'        => self::DATA_TYPE_URI,
-					'export_type' => 'http://schema.org/Organization',
-					'constraints' => array(
-						'uri_type'    => array(
-							'Organization',
-							'LocalBusiness',
-						),
-						'cardinality' => INF,
-					),
-				),
-				self::FIELD_EMAIL       => array(
-					'predicate'   => 'http://schema.org/email',
-					'type'        => self::DATA_TYPE_STRING,
-					'export_type' => 'xsd:string',
-					'constraints' => array(
-						'cardinality' => INF,
-					),
-				),
-			),
-			'templates'     => array(
-				'subtitle' => '{{id}}',
-			),
-		);
-
-		// Merge the custom fields with those provided by the thing schema.
-		$parent_schema           = $this->get_thing_schema();
-		$schema['custom_fields'] = array_merge( $schema['custom_fields'], $parent_schema['custom_fields'] );
-
-		return $schema;
-
-	}
-
-	/**
-	 * Get the 'place' schema.
-	 *
-	 * @return array An array with the schema configuration.
-	 *
-	 * @since 3.1.0
-	 */
-	private function get_place_schema() {
-
-		$schema = array(
-			'label'         => 'Place',
-			'description'   => 'A place.',
-			'parents'       => array( 'thing' ),
-			'css_class'     => 'wl-place',
-			'uri'           => 'http://schema.org/Place',
-			'same_as'       => array(
-				'http://rdf.freebase.com/ns/location.location',
-				'http://www.opengis.net/gml/_Feature',
-			),
-			'custom_fields' => array(
-				self::FIELD_GEO_LATITUDE        => array(
-					'predicate'   => 'http://schema.org/latitude',
-					'type'        => self::DATA_TYPE_DOUBLE,
-					'export_type' => 'xsd:double',
-					'constraints' => '',
-					// To build custom metabox.
-					'input_field' => 'coordinates',
-				),
-				self::FIELD_GEO_LONGITUDE       => array(
-					'predicate'   => 'http://schema.org/longitude',
-					'type'        => self::DATA_TYPE_DOUBLE,
-					'export_type' => 'xsd:double',
-					'constraints' => '',
-					// To build custom metabox.
-					'input_field' => 'coordinates',
-				),
-				self::FIELD_ADDRESS             => array(
-					'predicate'   => 'http://schema.org/streetAddress',
-					'type'        => self::DATA_TYPE_STRING,
-					'export_type' => 'xsd:string',
-					'constraints' => '',
-					// To build custom metabox.
-					'input_field' => 'address',
-				),
-				self::FIELD_ADDRESS_PO_BOX      => array(
-					'predicate'   => 'http://schema.org/postOfficeBoxNumber',
-					'type'        => self::DATA_TYPE_STRING,
-					'export_type' => 'xsd:string',
-					'constraints' => '',
-					// To build custom metabox.
-					'input_field' => 'address',
-				),
-				self::FIELD_ADDRESS_POSTAL_CODE => array(
-					'predicate'   => 'http://schema.org/postalCode',
-					'type'        => self::DATA_TYPE_STRING,
-					'export_type' => 'xsd:string',
-					'constraints' => '',
-					// To build custom metabox.
-					'input_field' => 'address',
-				),
-				self::FIELD_ADDRESS_LOCALITY    => array(
-					'predicate'   => 'http://schema.org/addressLocality',
-					'type'        => self::DATA_TYPE_STRING,
-					'export_type' => 'xsd:string',
-					'constraints' => '',
-					// To build custom metabox.
-					'input_field' => 'address',
-				),
-				self::FIELD_ADDRESS_REGION      => array(
-					'predicate'   => 'http://schema.org/addressRegion',
-					'type'        => self::DATA_TYPE_STRING,
-					'export_type' => 'xsd:string',
-					'constraints' => '',
-					// To build custom metabox.
-					'input_field' => 'address',
-				),
-				self::FIELD_ADDRESS_COUNTRY     => array(
-					'predicate'   => 'http://schema.org/addressCountry',
-					'type'        => self::DATA_TYPE_STRING,
-					'export_type' => 'xsd:string',
-					'constraints' => '',
-					// To build custom metabox.
-					'input_field' => 'address',
-				),
-			),
-			'templates'     => array(
-				'subtitle' => '{{id}}',
-			),
-		);
-
-		// Merge the custom fields with those provided by the thing schema.
-		$parent_schema           = $this->get_thing_schema();
-		$schema['custom_fields'] = array_merge( $schema['custom_fields'], $parent_schema['custom_fields'] );
-
-		return $schema;
-	}
-
-	/**
-	 * Get the 'local business' schema.
-	 *
-	 * @return array An array with the schema configuration.
-	 *
-	 * @since 3.1.0
-	 */
-	private function get_local_business_schema() {
-
-		$schema = array(
-			'label'         => 'LocalBusiness',
-			'description'   => 'A local business.',
-			'parents'       => array( 'place', 'organization' ),
-			'css_class'     => 'wl-local-business',
-			'uri'           => 'http://schema.org/LocalBusiness',
-			'same_as'       => array(
-				'http://rdf.freebase.com/ns/business/business_location',
-				'https://schema.org/Store',
-			),
-			'custom_fields' => array(),
-			'templates'     => array(
-				'subtitle' => '{{id}}',
-			),
-		);
-
-		// Merge the custom fields with those provided by the place and organization schema.
-		$place_schema            = $this->get_place_schema();
-		$organization_schema     = $this->get_organization_schema();
-		$schema['custom_fields'] = array_merge( $schema['custom_fields'], $place_schema['custom_fields'], $organization_schema['custom_fields'] );
-
-		return $schema;
-	}
-
-	/**
-	 * Get the 'recipe' schema.
-	 *
-	 * @return array An array with the schema configuration.
-	 *
-	 * @since 3.14.0
-	 */
-	private function get_recipe_schema() {
-
-		$schema = array(
-			'label'         => 'Recipe',
-			'description'   => 'A Recipe.',
-			'parents'       => array( 'CreativeWork' ),
-			'css_class'     => 'wl-recipe',
-			'uri'           => 'http://schema.org/Recipe',
-			'same_as'       => array(),
-			'templates'     => array(
-				'subtitle' => '{{id}}',
-			),
-			'custom_fields' => array(
-				self::FIELD_RECIPE_CUISINE          => array(
-					'predicate'   => 'http://schema.org/recipeCuisine',
-					'type'        => self::DATA_TYPE_STRING,
-					'export_type' => 'xsd:string',
-					'constraints' => '',
-					'metabox'     => array(
-						'label' => __( 'Recipe cuisine', 'wordlift' ),
-					),
-				),
-				self::FIELD_RECIPE_INGREDIENT       => array(
-					'predicate'   => 'http://schema.org/recipeIngredient',
-					'type'        => self::DATA_TYPE_STRING,
-					'export_type' => 'xsd:string',
-					'constraints' => array(
-						'cardinality' => INF,
-					),
-					'metabox'     => array(
-						'label' => __( 'Recipe ingredient', 'wordlift' ),
-					),
-				),
-				self::FIELD_RECIPE_INSTRUCTIONS     => array(
-					'predicate'   => 'http://schema.org/recipeInstructions',
-					'type'        => self::DATA_TYPE_MULTILINE,
-					'export_type' => 'xsd:string',
-					'constraints' => '',
-					'metabox'     => array(
-						'class' => 'Wordlift_Metabox_Field_Multiline',
-						'label' => __( 'Recipe instructions', 'wordlift' ),
-					),
-				),
-				self::FIELD_RECIPE_YIELD            => array(
-					'predicate'   => 'http://schema.org/recipeYield',
-					'type'        => self::DATA_TYPE_STRING,
-					'export_type' => 'xsd:string',
-					'constraints' => '',
-					'metabox'     => array(
-						'label' => __( 'Recipe number of servings', 'wordlift' ),
-					),
-				),
-				self::FIELD_RECIPE_INGREDIENT       => array(
-					'predicate'   => 'http://schema.org/recipeIngredient',
-					'type'        => self::DATA_TYPE_STRING,
-					'export_type' => 'xsd:string',
-					'constraints' => array(
-						'cardinality' => INF,
-					),
-					'metabox'     => array(
-						'label' => __( 'Recipe ingredient', 'wordlift' ),
-					),
-				),
-				self::FIELD_NUTRITION_INFO_CALORIES => array(
-					'predicate'   => 'http://schema.org/calories',
-					'type'        => self::DATA_TYPE_STRING,
-					'export_type' => 'xsd:string',
-					'constraints' => '',
-					'metabox'     => array(
-						'label' => __( 'Calories (e.g. 240 calories)', 'wordlift' ),
-					),
-				),
-				self::FIELD_PREP_TIME               => array(
-					'predicate'   => 'http://schema.org/prepTime',
-					'type'        => self::DATA_TYPE_DURATION,
-					'export_type' => 'xsd:time',
-					'constraints' => '',
-					'metabox'     => array(
-						'class' => 'Wordlift_Metabox_Field_Duration',
-						'label' => __( 'Recipe preparation time (e.g. 1:30)', 'wordlift' ),
-					),
-				),
-				self::FIELD_COOK_TIME               => array(
-					'predicate'   => 'http://schema.org/cookTime',
-					'type'        => self::DATA_TYPE_DURATION,
-					'export_type' => 'xsd:time',
-					'constraints' => '',
-					'metabox'     => array(
-						'class' => 'Wordlift_Metabox_Field_Duration',
-						'label' => __( 'Recipe cook time (e.g. 1:30)', 'wordlift' ),
-					),
-				),
-				self::FIELD_TOTAL_TIME              => array(
-					'predicate'   => 'http://schema.org/totalTime',
-					'type'        => self::DATA_TYPE_DURATION,
-					'export_type' => 'xsd:time',
-					'constraints' => '',
-					'metabox'     => array(
-						'class' => 'Wordlift_Metabox_Field_Duration',
-						'label' => __( 'Recipe total time (e.g. 1:30)', 'wordlift' ),
-					),
-				),
-			),
-		);
-
-		// Merge the custom fields with those provided by the parent schema.
-		$parent_schema           = $this->get_creative_work_schema();
-		$schema['custom_fields'] = array_merge( $schema['custom_fields'], $parent_schema['custom_fields'] );
-
-		return $schema;
-	}
-
-	/**
-	 * Get the 'offer' schema.
-	 *
-	 * @return array An array with the schema configuration.
-	 *
-	 * @since 3.18.0
-	 */
-	private function get_offer_schema() {
-
-		$schema = array(
-			'label'         => 'Offer',
-			'description'   => 'An offer. ',
-			'parents'       => array( 'thing' ),
-			'css_class'     => 'wl-offer',
-			'uri'           => self::SCHEMA_OFFER_TYPE,
-			'same_as'       => array(),
-			'templates'     => array(
-				'subtitle' => '{{id}}',
-			),
-			'custom_fields' => array(
-				self::FIELD_AVAILABILITY        => array(
-					'predicate'   => 'http://schema.org/availability',
-					'type'        => self::DATA_TYPE_STRING,
-					'export_type' => 'xsd:string',
-					'metabox'     => array(
-						'class' => 'Wordlift_Metabox_Field_Select',
-					),
-					'options'     => array(
-						'Discontinued'        => esc_html__( 'Discontinued', 'wordlift' ),
-						'InStock'             => esc_html__( 'In Stock', 'wordlift' ),
-						'InStoreOnly'         => esc_html__( 'In Store Only', 'wordlift' ),
-						'LimitedAvailability' => esc_html__( 'Limited Availability', 'wordlift' ),
-						'OnlineOnly'          => esc_html__( 'Online Only', 'wordlift' ),
-						'OutOfStock'          => esc_html__( 'Out of Stock', 'wordlift' ),
-						'PreOrder'            => esc_html__( 'Pre Order', 'wordlift' ),
-						'PreSale'             => esc_html__( 'Pre Sale', 'wordlift' ),
-						'SoldOut'             => esc_html__( 'Sold Out', 'wordlift' ),
-					),
-				),
-				self::FIELD_PRICE               => array(
-					'predicate'   => 'http://schema.org/price',
-					'type'        => self::DATA_TYPE_STRING,
-					'export_type' => 'xsd:integer',
-					'metabox'     => array(
-						'class' => 'Wordlift_Metabox_Field_Integer',
-					),
-				),
-				self::FIELD_PRICE_CURRENCY      => array(
-					'predicate'   => 'http://schema.org/priceCurrency',
-					'type'        => self::DATA_TYPE_STRING,
-					'export_type' => 'xsd:string',
-				),
-				self::FIELD_AVAILABILITY_STARTS => array(
-					'predicate'   => 'http://schema.org/availabilityStarts',
-					'type'        => self::DATA_TYPE_DATE,
-					'export_type' => 'xsd:dateTime',
-				),
-				self::FIELD_AVAILABILITY_ENDS   => array(
-					'predicate'   => 'http://schema.org/availabilityEnds',
-					'type'        => self::DATA_TYPE_DATE,
-					'export_type' => 'xsd:dateTime',
-				),
-				self::FIELD_INVENTORY_LEVEL     => array(
-					'predicate'   => 'http://schema.org/inventoryLevel',
-					'type'        => self::DATA_TYPE_STRING,
-					'export_type' => 'xsd:integer',
-					'metabox'     => array(
-						'class' => 'Wordlift_Metabox_Field_Integer',
-					),
-				),
-				self::FIELD_VALID_FROM          => array(
-					'predicate'   => 'http://schema.org/validFrom',
-					'type'        => self::DATA_TYPE_DATE,
-					'export_type' => 'xsd:dateTime',
-				),
-				self::FIELD_PRICE_VALID_UNTIL   => array(
-					'predicate'   => 'http://schema.org/priceValidUntil',
-					'type'        => self::DATA_TYPE_DATE,
-					'export_type' => 'xsd:dateTime',
-				),
-				self::FIELD_ITEM_OFFERED        => array(
-					'predicate'   => 'http://schema.org/itemOffered',
-					'type'        => self::DATA_TYPE_URI,
-					'export_type' => 'http://schema.org/Thing',
-					'constraints' => array(
-						'uri_type'    => array(
-							'Event',
-							'Thing',
-						),
-						'cardinality' => INF,
-					),
-				),
-			),
-		);
-
-		// Merge the custom fields with those provided by the thing schema.
-		$parent_schema           = $this->get_thing_schema();
-		$schema['custom_fields'] = array_merge( $schema['custom_fields'], $parent_schema['custom_fields'] );
-
-		return $schema;
-	}
-
-	/**
-	 * Get the 'article' schema.
-	 *
-	 * @return array An array with the schema configuration.
-	 *
-	 * @since 3.15.0
-	 */
-	private function get_article_schema() {
-
-		$schema = array(
-			'label'         => 'Article',
-			'description'   => 'An Article.',
-			'parents'       => array(),
-			'css_class'     => 'wl-article',
-			'uri'           => 'http://schema.org/Article',
-			'same_as'       => array(),
-			'templates'     => array(
-				'subtitle' => '{{id}}',
-			),
-			'custom_fields' => array(),
-		);
-
-		return $schema;
-	}
+    /**
+     * The 'location created' field name.
+     *
+     * @since 3.5.0
+     */
+    const FIELD_LOCATION_CREATED = 'wl_location_created';
+
+    /**
+     * The 'topic' field name.
+     *
+     * @since 3.5.0
+     */
+    const FIELD_TOPIC = 'wl_topic';
+
+    /**
+     * The 'author' field name.
+     *
+     * @since 3.1.0
+     */
+    const FIELD_AUTHOR = 'wl_author';
+
+    /**
+     * The 'same as' field name.
+     *
+     * @since 3.1.0
+     */
+    const FIELD_SAME_AS = 'entity_same_as';
+
+    /**
+     * The 'date start' field name.
+     *
+     * @since 3.1.0
+     */
+    const FIELD_DATE_START = 'wl_cal_date_start';
+
+    /**
+     * The 'date end' field name.
+     *
+     * @since 3.1.0
+     */
+    const FIELD_DATE_END = 'wl_cal_date_end';
+
+    /**
+     * The 'location' field name.
+     *
+     * @since 3.1.0
+     */
+    const FIELD_LOCATION = 'wl_location';
+
+    /**
+     * The 'founder' field name.
+     *
+     * @since 3.1.0
+     */
+    const FIELD_FOUNDER = 'wl_founder';
+
+    /**
+     * The 'knows' field name.
+     *
+     * @since 3.1.0
+     */
+    const FIELD_KNOWS = 'wl_knows';
+
+    /**
+     * The 'birth date' field name.
+     *
+     * @since 3.1.0
+     */
+    const FIELD_BIRTH_DATE = 'wl_birth_date';
+
+    /**
+     * The 'birth place' field name.
+     *
+     * @since 3.1.0
+     */
+    const FIELD_BIRTH_PLACE = 'wl_birth_place';
+
+    /**
+     * The 'latitude' field name.
+     *
+     * @since 3.1.0
+     */
+    const FIELD_GEO_LATITUDE = 'wl_geo_latitude';
+
+    /**
+     * The 'longitude' field name.
+     *
+     * @since 3.1.0
+     */
+    const FIELD_GEO_LONGITUDE = 'wl_geo_longitude';
+
+    /**
+     * The 'streetAddress' field name.
+     *
+     * @since 3.1.0
+     */
+    const FIELD_ADDRESS = 'wl_address';
+
+    /**
+     * The 'postOfficeBoxNumber' field name.
+     *
+     * @since 3.3.0
+     */
+    const FIELD_ADDRESS_PO_BOX = 'wl_address_post_office_box';
+
+    /**
+     * The 'postalCode' field name.
+     *
+     * @since 3.3.0
+     */
+    const FIELD_ADDRESS_POSTAL_CODE = 'wl_address_postal_code';
+
+    /**
+     * The 'addressLocality' field name.
+     *
+     * @since 3.3.0
+     */
+    const FIELD_ADDRESS_LOCALITY = 'wl_address_locality';
+    /**
+     * The 'addressRegion' field name.
+     *
+     * @since 3.3.0
+     */
+    const FIELD_ADDRESS_REGION = 'wl_address_region';
+
+    /**
+     * The 'addressCountry' field name.
+     *
+     * @since 3.3.0
+     */
+    const FIELD_ADDRESS_COUNTRY = 'wl_address_country';
+
+    /**
+     * The 'entity type' field name.
+     *
+     * @since 3.1.0
+     */
+    const FIELD_ENTITY_TYPE = 'wl_entity_type_uri';
+
+    /**
+     * The 'email' field name.
+     *
+     * @since 3.2.0
+     */
+    const FIELD_EMAIL = 'wl_email';
+
+    /**
+     * The 'affiliation' field name.
+     *
+     * @since 3.2.0
+     */
+    const FIELD_AFFILIATION = 'wl_affiliation';
+
+    /**
+     * The 'telephone' field name.
+     *
+     * @since 3.8.0
+     */
+    const FIELD_TELEPHONE = 'wl_schema_telephone';
+
+    /**
+     * The 'legalName' field name.
+     *
+     * @since 3.12.0
+     */
+    const FIELD_LEGAL_NAME = 'wl_schema_legal_name';
+
+    /**
+     * The 'recipeCuisine' field name.
+     *
+     * @since 3.14.0
+     */
+    const FIELD_RECIPE_CUISINE = 'wl_schema_recipe_cuisine';
+
+    /**
+     * The 'recipeIngredient' field name.
+     *
+     * @since 3.14.0
+     */
+    const FIELD_RECIPE_INGREDIENT = 'wl_schema_recipe_ingredient';
+
+    /**
+     * The 'calories' field name.
+     *
+     * @since 3.14.0
+     */
+    const FIELD_NUTRITION_INFO_CALORIES = 'wl_schema_nutrition_information_calories';
+
+    /**
+     * The 'recipeInstructions' field name.
+     *
+     * @since 3.14.0
+     */
+    const FIELD_RECIPE_INSTRUCTIONS = 'wl_schema_recipe_instructions';
+
+    /**
+     * The 'recipeYield' field name.
+     *
+     * @since 3.14.0
+     */
+    const FIELD_RECIPE_YIELD = 'wl_schema_recipe_yield';
+
+    /**
+     * The 'prepTime' field name.
+     *
+     * @since 3.14.0
+     */
+    const FIELD_PREP_TIME = 'wl_schema_prep_time';
+
+    /**
+     * The 'cookTime' field name.
+     *
+     * @since 3.14.0
+     */
+    const FIELD_COOK_TIME = 'wl_schema_cook_time';
+
+    /**
+     * The 'totalTime' field name.
+     *
+     * @since 3.14.0
+     */
+    const FIELD_TOTAL_TIME = 'wl_schema_total_time';
+
+    /**
+     * The 'performer' field name.
+     *
+     * @since 3.18.0
+     */
+    const FIELD_PERFORMER = 'wl_schema_performer';
+
+    /**
+     * The 'offers' field name.
+     *
+     * @since 3.18.0
+     */
+    const FIELD_OFFERS = 'wl_schema_offers';
+
+    /**
+     * The 'availablity' field name.
+     *
+     * @since 3.18.0
+     */
+    const FIELD_AVAILABILITY = 'wl_schema_availability';
+
+    /**
+     * The 'inventoryLevel' field name.
+     *
+     * @since 3.18.0
+     */
+    const FIELD_INVENTORY_LEVEL = 'wl_schema_inventory_level';
+
+    /**
+     * The 'price' field name.
+     *
+     * @since 3.18.0
+     */
+    const FIELD_PRICE = 'wl_schema_price';
+
+    /**
+     * The 'priceCurrency' field name.
+     *
+     * @since 3.18.0
+     */
+    const FIELD_PRICE_CURRENCY = 'wl_schema_price_currency';
+
+    /**
+     * The 'availabilityStarts' field name.
+     *
+     * @since 3.18.0
+     */
+    const FIELD_AVAILABILITY_STARTS = 'wl_schema_availability_starts';
+
+    /**
+     * The 'availabilityEnds' field name.
+     *
+     * @since 3.18.0
+     */
+    const FIELD_AVAILABILITY_ENDS = 'wl_schema_availability_ends';
+
+    /**
+     * The 'validFrom' field name.
+     *
+     * @since 3.18.0
+     */
+    const FIELD_VALID_FROM = 'wl_schema_valid_from';
+
+    /**
+     * The 'priceValidUntil' field name.
+     *
+     * @since 3.18.0
+     */
+    const FIELD_PRICE_VALID_UNTIL = 'wl_schema_valid_until';
+
+    /**
+     * The 'itemOffered' field name.
+     *
+     * @since 3.18.0
+     */
+    const FIELD_ITEM_OFFERED = 'wl_schema_item_offered';
+
+    /**
+     * The 'URI' data type name.
+     *
+     * @since 3.1.0
+     */
+    const DATA_TYPE_URI = 'uri';
+
+    /**
+     * The 'date' data type name.
+     *
+     * @since 3.1.0
+     */
+    const DATA_TYPE_DATE = 'date';
+
+    /**
+     * The 'time' data type name.
+     *
+     * @since 3.14.0
+     */
+    const DATA_TYPE_DURATION = 'duration';
+
+    /**
+     * The 'double' data type name.
+     *
+     * @since 3.1.0
+     */
+    const DATA_TYPE_DOUBLE = 'double';
+
+    /**
+     * The 'string' data type name.
+     *
+     * @since 3.1.0
+     */
+    const DATA_TYPE_STRING = 'string';
+
+    /**
+     * The multiline text data type name.
+     *
+     * @since 3.14.0
+     */
+    const DATA_TYPE_MULTILINE = 'multiline';
+
+    /**
+     * The schema.org Event type URI.
+     *
+     * @since 3.1.0
+     */
+    const SCHEMA_EVENT_TYPE = 'http://schema.org/Event';
+
+    /**
+     * The schema.org Offer type URI.
+     *
+     * @since 3.18.0
+     */
+    const SCHEMA_OFFER_TYPE = 'http://schema.org/Offer';
+
+    /**
+     * WordLift's schema.
+     *
+     * @since  3.1.0
+     * @access private
+     * @var array $schema WordLift's schema.
+     */
+    private $schema;
+
+    /**
+     * The Log service.
+     *
+     * @since  3.1.0
+     * @access private
+     * @var \Wordlift_Log_Service $log The Log service.
+     */
+    private $log;
+
+    /**
+     * Wordlift_Schema_Service constructor.
+     *
+     * @since 3.1.0
+     */
+    protected function __construct() {
+
+        $this->log = Wordlift_Log_Service::get_logger( 'Wordlift_Schema_Service' );
+
+        /**
+         * Alter the configured schemas.
+         *
+         * Enable 3rd parties to alter WordLift's schemas array.
+         *
+         * @param array $schemas The array of schemas.
+         *
+         * @since  3.19.1
+         */
+        $this->schema = apply_filters(
+            'wl_schemas',
+            array(
+                'article'        => $this->get_article_schema(),
+                'thing'          => $this->get_thing_schema(),
+                'creative-work'  => $this->get_creative_work_schema(),
+                'event'          => $this->get_event_schema(),
+                'organization'   => $this->get_organization_schema(),
+                'person'         => $this->get_person_schema(),
+                'place'          => $this->get_place_schema(),
+                'local-business' => $this->get_local_business_schema(),
+                'recipe'         => $this->get_recipe_schema(),
+                'web-page'       => $this->get_web_page_schema(),
+                'offer'          => $this->get_offer_schema(),
+            )
+        );
+
+        // Create a singleton instance of the Schema service, useful to provide static functions to global functions.
+        self::$instance = $this;
+
+    }
+
+    /**
+     * The Schema service singleton instance.
+     *
+     * @since  3.1.0
+     * @access private
+     * @var Wordlift_Schema_Service $instance The Schema service singleton instance.
+     */
+    private static $instance = null;
+
+    /**
+     * Get a reference to the Schema service.
+     *
+     * @return Wordlift_Schema_Service A reference to the Schema service.
+     * @since 3.1.0
+     */
+    public static function get_instance() {
+        if ( ! isset( self::$instance ) ) {
+            self::$instance = new self();
+        }
+
+        return self::$instance;
+    }
+
+    /**
+     * Get the properties for a field with the specified key. The key is used as
+     * meta key when the field's value is stored in WordPress meta data table.
+     *
+     * @param string $key The field's key.
+     *
+     * @return null|array An array of field's properties or null if the field is not found.
+     * @since 3.6.0
+     */
+    public function get_field( $key ) {
+
+        // Parse each schema's fields until we find the one we're looking for, then
+        // return its properties.
+        // phpcs:ignore VariableAnalysis.CodeAnalysis.VariableAnalysis.UnusedVariable
+        foreach ( $this->schema as $_ => $schema ) {
+
+            if ( ! isset( $schema['custom_fields'] ) ) {
+                break;
+            }
+
+            foreach ( $schema['custom_fields'] as $field => $props ) {
+                if ( $key === $field ) {
+                    return $props;
+                }
+            }
+        }
+
+        return null;
+    }
+
+    /**
+     * Get the WordLift's schema.
+     *
+     * @param string $name The schema name.
+     *
+     * @return array|null An array with the schema configuration or NULL if the schema is not found.
+     *
+     * @since 3.1.0
+     */
+    public function get_schema( $name ) {
+        // Check if the schema exists and, if not, return NULL.
+        if ( ! isset( $this->schema[ $name ] ) ) {
+            return null;
+        }
+
+        // Return the requested schema.
+        return $this->schema[ $name ];
+    }
+
+    /**
+     * Get the WordLift's schema trough schema type uri.
+     *
+     * @param string $uri The schema uri.
+     *
+     * @return array|null An array with the schema configuration or NULL if the schema is not found.
+     *
+     * @since 3.3.0
+     */
+    public function get_schema_by_uri( $uri ) {
+
+        // phpcs:ignore VariableAnalysis.CodeAnalysis.VariableAnalysis.UnusedVariable
+        foreach ( $this->schema as $name => $schema ) {
+            if ( $schema['uri'] === $uri ) {
+                return $schema;
+            }
+        }
+
+        return null;
+    }
+
+    /**
+     * Get the 'thing' schema.
+     *
+     * @return array An array with the schema configuration.
+     *
+     * @since 3.1.0
+     */
+    private function get_thing_schema() {
+
+        return array(
+            'css_class'     => 'wl-thing',
+            'uri'           => 'http://schema.org/Thing',
+            'same_as'       => array( '*' ),
+            // set as default.
+            'custom_fields' => array(
+                self::FIELD_SAME_AS => array(
+                    'predicate'   => 'http://schema.org/sameAs',
+                    'type'        => self::DATA_TYPE_URI,
+                    'export_type' => 'http://schema.org/Thing',
+                    'constraints' => array(
+                        'cardinality' => INF,
+                    ),
+                    // We need a custom metabox.
+                    'input_field' => 'sameas',
+                ),
+                // Add the schema:url property.
+                Wordlift_Schema_Url_Property_Service::META_KEY => Wordlift_Schema_Url_Property_Service::get_instance()
+                                                                                                        ->get_compat_definition(),
+            ),
+            // {{sameAs}} not present in the microdata template,
+            // because it is treated separately in *wl_content_embed_item_microdata*
+            'templates'     => array(
+                'subtitle' => '{{id}}',
+            ),
+        );
+
+    }
+
+    /**
+     * Get the 'web-page' schema.
+     *
+     * @return array An array with the schema configuration.
+     *
+     * @since 3.18.0
+     */
+    private function get_web_page_schema() {
+
+        return array(
+            'css_class' => 'wl-webpage',
+            'uri'       => 'http://schema.org/WebPage',
+        );
+
+    }
+
+    /**
+     * Get the 'creative work' schema.
+     *
+     * @return array An array with the schema configuration.
+     *
+     * @since 3.1.0
+     */
+    private function get_creative_work_schema() {
+
+        $schema = array(
+            'label'         => 'CreativeWork',
+            'description'   => 'A creative work (or a Music Album).',
+            'parents'       => array( 'thing' ),
+            // Give term slug as parent.
+            'css_class'     => 'wl-creative-work',
+            'uri'           => 'http://schema.org/CreativeWork',
+            'same_as'       => array(
+                'http://schema.org/MusicAlbum',
+                'http://schema.org/Product',
+            ),
+            'custom_fields' => array(
+                self::FIELD_AUTHOR => array(
+                    'predicate'   => 'http://schema.org/author',
+                    'type'        => self::DATA_TYPE_URI,
+                    'export_type' => 'http://schema.org/Person',
+                    'constraints' => array(
+                        'uri_type'    => array( 'Person', 'Organization' ),
+                        'cardinality' => INF,
+                    ),
+                ),
+            ),
+            'templates'     => array(
+                'subtitle' => '{{id}}',
+            ),
+        );
+
+        // Merge the custom fields with those provided by the thing schema.
+        $parent_schema           = $this->get_thing_schema();
+        $schema['custom_fields'] = array_merge( $schema['custom_fields'], $parent_schema['custom_fields'] );
+
+        return $schema;
+    }
+
+    /**
+     * Get the 'event' schema.
+     *
+     * @return array An array with the schema configuration.
+     *
+     * @since 3.1.0
+     */
+    private function get_event_schema() {
+
+        $schema = array(
+            'label'         => 'Event',
+            'description'   => 'An event . ',
+            'parents'       => array( 'thing' ),
+            'css_class'     => 'wl-event',
+            'uri'           => self::SCHEMA_EVENT_TYPE,
+            'same_as'       => array( 'http://dbpedia.org/ontology/Event' ),
+            'custom_fields' => array(
+                self::FIELD_DATE_START => array(
+                    'predicate'   => 'http://schema.org/startDate',
+                    'type'        => self::DATA_TYPE_DATE,
+                    'export_type' => 'xsd:dateTime',
+                    'constraints' => '',
+                ),
+                self::FIELD_DATE_END   => array(
+                    'predicate'   => 'http://schema.org/endDate',
+                    'type'        => self::DATA_TYPE_DATE,
+                    'export_type' => 'xsd:dateTime',
+                    'constraints' => '',
+                ),
+                self::FIELD_LOCATION   => array(
+                    'predicate'   => 'http://schema.org/location',
+                    'type'        => self::DATA_TYPE_URI,
+                    'export_type' => 'http://schema.org/PostalAddress',
+                    'constraints' => array(
+                        'uri_type'    => array( 'Place', 'LocalBusiness' ),
+                        'cardinality' => INF,
+                    ),
+                ),
+                self::FIELD_PERFORMER  => array(
+                    'predicate'   => 'http://schema.org/performer',
+                    'type'        => self::DATA_TYPE_URI,
+                    'export_type' => 'http://schema.org/Person',
+                    'constraints' => array(
+                        'uri_type'    => array( 'Person', 'Organization' ),
+                        'cardinality' => INF,
+                    ),
+                ),
+                self::FIELD_OFFERS     => array(
+                    'predicate'   => 'http://schema.org/offers',
+                    'type'        => self::DATA_TYPE_URI,
+                    'export_type' => 'http://schema.org/Offer',
+                    'constraints' => array(
+                        'uri_type'    => array( 'Offer' ),
+                        'cardinality' => INF,
+                    ),
+                ),
+            ),
+            'templates'     => array(
+                'subtitle' => '{{id}}',
+            ),
+        );
+
+        // Merge the custom fields with those provided by the thing schema.
+        $parent_schema           = $this->get_thing_schema();
+        $schema['custom_fields'] = array_merge( $schema['custom_fields'], $parent_schema['custom_fields'] );
+
+        return $schema;
+    }
+
+    /**
+     * Get the 'organization' schema.
+     *
+     * @return array An array with the schema configuration.
+     *
+     * @since 3.1.0
+     */
+    private function get_organization_schema() {
+
+        $schema = array(
+            'label'         => 'Organization',
+            'description'   => 'An organization, including a government or a newspaper.',
+            'parents'       => array( 'thing' ),
+            'css_class'     => 'wl-organization',
+            'uri'           => 'http://schema.org/Organization',
+            'same_as'       => array(
+                'http://rdf.freebase.com/ns/organization.organization',
+                'http://rdf.freebase.com/ns/government.government',
+                'http://schema.org/Newspaper',
+            ),
+            'custom_fields' => array(
+                self::FIELD_LEGAL_NAME          => array(
+                    'predicate'   => 'http://schema.org/legalName',
+                    'type'        => self::DATA_TYPE_STRING,
+                    'export_type' => 'xsd:string',
+                    'constraints' => '',
+                ),
+                self::FIELD_FOUNDER             => array(
+                    'predicate'   => 'http://schema.org/founder',
+                    'type'        => self::DATA_TYPE_URI,
+                    'export_type' => 'http://schema.org/Person',
+                    'constraints' => array(
+                        'uri_type'    => 'Person',
+                        'cardinality' => INF,
+                    ),
+                ),
+                self::FIELD_ADDRESS             => array(
+                    'predicate'   => 'http://schema.org/streetAddress',
+                    'type'        => self::DATA_TYPE_STRING,
+                    'export_type' => 'xsd:string',
+                    'constraints' => '',
+                    // To build custom metabox.
+                    'input_field' => 'address',
+                ),
+                self::FIELD_ADDRESS_PO_BOX      => array(
+                    'predicate'   => 'http://schema.org/postOfficeBoxNumber',
+                    'type'        => self::DATA_TYPE_STRING,
+                    'export_type' => 'xsd:string',
+                    'constraints' => '',
+                    // To build custom metabox.
+                    'input_field' => 'address',
+                ),
+                self::FIELD_ADDRESS_POSTAL_CODE => array(
+                    'predicate'   => 'http://schema.org/postalCode',
+                    'type'        => self::DATA_TYPE_STRING,
+                    'export_type' => 'xsd:string',
+                    'constraints' => '',
+                    // To build custom metabox.
+                    'input_field' => 'address',
+                ),
+                self::FIELD_ADDRESS_LOCALITY    => array(
+                    'predicate'   => 'http://schema.org/addressLocality',
+                    'type'        => self::DATA_TYPE_STRING,
+                    'export_type' => 'xsd:string',
+                    'constraints' => '',
+                    // To build custom metabox.
+                    'input_field' => 'address',
+                ),
+                self::FIELD_ADDRESS_REGION      => array(
+                    'predicate'   => 'http://schema.org/addressRegion',
+                    'type'        => self::DATA_TYPE_STRING,
+                    'export_type' => 'xsd:string',
+                    'constraints' => '',
+                    // To build custom metabox.
+                    'input_field' => 'address',
+                ),
+                self::FIELD_ADDRESS_COUNTRY     => array(
+                    'predicate'   => 'http://schema.org/addressCountry',
+                    'type'        => self::DATA_TYPE_STRING,
+                    'export_type' => 'xsd:string',
+                    'constraints' => '',
+                    // To build custom metabox.
+                    'input_field' => 'address',
+                ),
+                self::FIELD_EMAIL               => array(
+                    'predicate'   => 'http://schema.org/email',
+                    'type'        => self::DATA_TYPE_STRING,
+                    'export_type' => 'xsd:string',
+                    'constraints' => '',
+                ),
+                self::FIELD_TELEPHONE           => array(
+                    'predicate'   => 'http://schema.org/telephone',
+                    'type'        => self::DATA_TYPE_STRING,
+                    'export_type' => 'xsd:string',
+                    'constraints' => '',
+                ),
+            ),
+            'templates'     => array(
+                'subtitle' => '{{id}}',
+            ),
+        );
+
+        // Merge the custom fields with those provided by the thing schema.
+        $parent_schema           = $this->get_thing_schema();
+        $schema['custom_fields'] = array_merge( $schema['custom_fields'], $parent_schema['custom_fields'] );
+
+        return $schema;
+    }
+
+    /**
+     * Get the 'person' schema.
+     *
+     * @return array An array with the schema configuration.
+     *
+     * @since 3.1.0
+     */
+    private function get_person_schema() {
+
+        $schema = array(
+            'label'         => 'Person',
+            'description'   => 'A person (or a music artist).',
+            'parents'       => array( 'thing' ),
+            'css_class'     => 'wl-person',
+            'uri'           => 'http://schema.org/Person',
+            'same_as'       => array(
+                'http://rdf.freebase.com/ns/people.person',
+                'http://rdf.freebase.com/ns/music.artist',
+                'http://dbpedia.org/class/yago/LivingPeople',
+            ),
+            'custom_fields' => array(
+                self::FIELD_KNOWS       => array(
+                    'predicate'   => 'http://schema.org/knows',
+                    'type'        => self::DATA_TYPE_URI,
+                    'export_type' => 'http://schema.org/Person',
+                    'constraints' => array(
+                        'uri_type'    => 'Person',
+                        'cardinality' => INF,
+                    ),
+                ),
+                self::FIELD_BIRTH_DATE  => array(
+                    'predicate'   => 'http://schema.org/birthDate',
+                    'type'        => self::DATA_TYPE_DATE,
+                    'export_type' => 'xsd:date',
+                    'constraints' => '',
+                ),
+                self::FIELD_BIRTH_PLACE => array(
+                    'predicate'   => 'http://schema.org/birthPlace',
+                    'type'        => self::DATA_TYPE_URI,
+                    'export_type' => 'http://schema.org/Place',
+                    'constraints' => array(
+                        'uri_type' => 'Place',
+                    ),
+                ),
+                self::FIELD_AFFILIATION => array(
+                    'predicate'   => 'http://schema.org/affiliation',
+                    'type'        => self::DATA_TYPE_URI,
+                    'export_type' => 'http://schema.org/Organization',
+                    'constraints' => array(
+                        'uri_type'    => array(
+                            'Organization',
+                            'LocalBusiness',
+                        ),
+                        'cardinality' => INF,
+                    ),
+                ),
+                self::FIELD_EMAIL       => array(
+                    'predicate'   => 'http://schema.org/email',
+                    'type'        => self::DATA_TYPE_STRING,
+                    'export_type' => 'xsd:string',
+                    'constraints' => array(
+                        'cardinality' => INF,
+                    ),
+                ),
+            ),
+            'templates'     => array(
+                'subtitle' => '{{id}}',
+            ),
+        );
+
+        // Merge the custom fields with those provided by the thing schema.
+        $parent_schema           = $this->get_thing_schema();
+        $schema['custom_fields'] = array_merge( $schema['custom_fields'], $parent_schema['custom_fields'] );
+
+        return $schema;
+
+    }
+
+    /**
+     * Get the 'place' schema.
+     *
+     * @return array An array with the schema configuration.
+     *
+     * @since 3.1.0
+     */
+    private function get_place_schema() {
+
+        $schema = array(
+            'label'         => 'Place',
+            'description'   => 'A place.',
+            'parents'       => array( 'thing' ),
+            'css_class'     => 'wl-place',
+            'uri'           => 'http://schema.org/Place',
+            'same_as'       => array(
+                'http://rdf.freebase.com/ns/location.location',
+                'http://www.opengis.net/gml/_Feature',
+            ),
+            'custom_fields' => array(
+                self::FIELD_GEO_LATITUDE        => array(
+                    'predicate'   => 'http://schema.org/latitude',
+                    'type'        => self::DATA_TYPE_DOUBLE,
+                    'export_type' => 'xsd:double',
+                    'constraints' => '',
+                    // To build custom metabox.
+                    'input_field' => 'coordinates',
+                ),
+                self::FIELD_GEO_LONGITUDE       => array(
+                    'predicate'   => 'http://schema.org/longitude',
+                    'type'        => self::DATA_TYPE_DOUBLE,
+                    'export_type' => 'xsd:double',
+                    'constraints' => '',
+                    // To build custom metabox.
+                    'input_field' => 'coordinates',
+                ),
+                self::FIELD_ADDRESS             => array(
+                    'predicate'   => 'http://schema.org/streetAddress',
+                    'type'        => self::DATA_TYPE_STRING,
+                    'export_type' => 'xsd:string',
+                    'constraints' => '',
+                    // To build custom metabox.
+                    'input_field' => 'address',
+                ),
+                self::FIELD_ADDRESS_PO_BOX      => array(
+                    'predicate'   => 'http://schema.org/postOfficeBoxNumber',
+                    'type'        => self::DATA_TYPE_STRING,
+                    'export_type' => 'xsd:string',
+                    'constraints' => '',
+                    // To build custom metabox.
+                    'input_field' => 'address',
+                ),
+                self::FIELD_ADDRESS_POSTAL_CODE => array(
+                    'predicate'   => 'http://schema.org/postalCode',
+                    'type'        => self::DATA_TYPE_STRING,
+                    'export_type' => 'xsd:string',
+                    'constraints' => '',
+                    // To build custom metabox.
+                    'input_field' => 'address',
+                ),
+                self::FIELD_ADDRESS_LOCALITY    => array(
+                    'predicate'   => 'http://schema.org/addressLocality',
+                    'type'        => self::DATA_TYPE_STRING,
+                    'export_type' => 'xsd:string',
+                    'constraints' => '',
+                    // To build custom metabox.
+                    'input_field' => 'address',
+                ),
+                self::FIELD_ADDRESS_REGION      => array(
+                    'predicate'   => 'http://schema.org/addressRegion',
+                    'type'        => self::DATA_TYPE_STRING,
+                    'export_type' => 'xsd:string',
+                    'constraints' => '',
+                    // To build custom metabox.
+                    'input_field' => 'address',
+                ),
+                self::FIELD_ADDRESS_COUNTRY     => array(
+                    'predicate'   => 'http://schema.org/addressCountry',
+                    'type'        => self::DATA_TYPE_STRING,
+                    'export_type' => 'xsd:string',
+                    'constraints' => '',
+                    // To build custom metabox.
+                    'input_field' => 'address',
+                ),
+            ),
+            'templates'     => array(
+                'subtitle' => '{{id}}',
+            ),
+        );
+
+        // Merge the custom fields with those provided by the thing schema.
+        $parent_schema           = $this->get_thing_schema();
+        $schema['custom_fields'] = array_merge( $schema['custom_fields'], $parent_schema['custom_fields'] );
+
+        return $schema;
+    }
+
+    /**
+     * Get the 'local business' schema.
+     *
+     * @return array An array with the schema configuration.
+     *
+     * @since 3.1.0
+     */
+    private function get_local_business_schema() {
+
+        $schema = array(
+            'label'         => 'LocalBusiness',
+            'description'   => 'A local business.',
+            'parents'       => array( 'place', 'organization' ),
+            'css_class'     => 'wl-local-business',
+            'uri'           => 'http://schema.org/LocalBusiness',
+            'same_as'       => array(
+                'http://rdf.freebase.com/ns/business/business_location',
+                'https://schema.org/Store',
+            ),
+            'custom_fields' => array(),
+            'templates'     => array(
+                'subtitle' => '{{id}}',
+            ),
+        );
+
+        // Merge the custom fields with those provided by the place and organization schema.
+        $place_schema            = $this->get_place_schema();
+        $organization_schema     = $this->get_organization_schema();
+        $schema['custom_fields'] = array_merge( $schema['custom_fields'], $place_schema['custom_fields'], $organization_schema['custom_fields'] );
+
+        return $schema;
+    }
+
+    /**
+     * Get the 'recipe' schema.
+     *
+     * @return array An array with the schema configuration.
+     *
+     * @since 3.14.0
+     */
+    private function get_recipe_schema() {
+
+        $schema = array(
+            'label'         => 'Recipe',
+            'description'   => 'A Recipe.',
+            'parents'       => array( 'CreativeWork' ),
+            'css_class'     => 'wl-recipe',
+            'uri'           => 'http://schema.org/Recipe',
+            'same_as'       => array(),
+            'templates'     => array(
+                'subtitle' => '{{id}}',
+            ),
+            'custom_fields' => array(
+                self::FIELD_RECIPE_CUISINE          => array(
+                    'predicate'   => 'http://schema.org/recipeCuisine',
+                    'type'        => self::DATA_TYPE_STRING,
+                    'export_type' => 'xsd:string',
+                    'constraints' => '',
+                    'metabox'     => array(
+                        'label' => __( 'Recipe cuisine', 'wordlift' ),
+                    ),
+                ),
+                self::FIELD_RECIPE_INGREDIENT       => array(
+                    'predicate'   => 'http://schema.org/recipeIngredient',
+                    'type'        => self::DATA_TYPE_STRING,
+                    'export_type' => 'xsd:string',
+                    'constraints' => array(
+                        'cardinality' => INF,
+                    ),
+                    'metabox'     => array(
+                        'label' => __( 'Recipe ingredient', 'wordlift' ),
+                    ),
+                ),
+                self::FIELD_RECIPE_INSTRUCTIONS     => array(
+                    'predicate'   => 'http://schema.org/recipeInstructions',
+                    'type'        => self::DATA_TYPE_MULTILINE,
+                    'export_type' => 'xsd:string',
+                    'constraints' => '',
+                    'metabox'     => array(
+                        'class' => 'Wordlift_Metabox_Field_Multiline',
+                        'label' => __( 'Recipe instructions', 'wordlift' ),
+                    ),
+                ),
+                self::FIELD_RECIPE_YIELD            => array(
+                    'predicate'   => 'http://schema.org/recipeYield',
+                    'type'        => self::DATA_TYPE_STRING,
+                    'export_type' => 'xsd:string',
+                    'constraints' => '',
+                    'metabox'     => array(
+                        'label' => __( 'Recipe number of servings', 'wordlift' ),
+                    ),
+                ),
+                self::FIELD_RECIPE_INGREDIENT       => array(
+                    'predicate'   => 'http://schema.org/recipeIngredient',
+                    'type'        => self::DATA_TYPE_STRING,
+                    'export_type' => 'xsd:string',
+                    'constraints' => array(
+                        'cardinality' => INF,
+                    ),
+                    'metabox'     => array(
+                        'label' => __( 'Recipe ingredient', 'wordlift' ),
+                    ),
+                ),
+                self::FIELD_NUTRITION_INFO_CALORIES => array(
+                    'predicate'   => 'http://schema.org/calories',
+                    'type'        => self::DATA_TYPE_STRING,
+                    'export_type' => 'xsd:string',
+                    'constraints' => '',
+                    'metabox'     => array(
+                        'label' => __( 'Calories (e.g. 240 calories)', 'wordlift' ),
+                    ),
+                ),
+                self::FIELD_PREP_TIME               => array(
+                    'predicate'   => 'http://schema.org/prepTime',
+                    'type'        => self::DATA_TYPE_DURATION,
+                    'export_type' => 'xsd:time',
+                    'constraints' => '',
+                    'metabox'     => array(
+                        'class' => 'Wordlift_Metabox_Field_Duration',
+                        'label' => __( 'Recipe preparation time (e.g. 1:30)', 'wordlift' ),
+                    ),
+                ),
+                self::FIELD_COOK_TIME               => array(
+                    'predicate'   => 'http://schema.org/cookTime',
+                    'type'        => self::DATA_TYPE_DURATION,
+                    'export_type' => 'xsd:time',
+                    'constraints' => '',
+                    'metabox'     => array(
+                        'class' => 'Wordlift_Metabox_Field_Duration',
+                        'label' => __( 'Recipe cook time (e.g. 1:30)', 'wordlift' ),
+                    ),
+                ),
+                self::FIELD_TOTAL_TIME              => array(
+                    'predicate'   => 'http://schema.org/totalTime',
+                    'type'        => self::DATA_TYPE_DURATION,
+                    'export_type' => 'xsd:time',
+                    'constraints' => '',
+                    'metabox'     => array(
+                        'class' => 'Wordlift_Metabox_Field_Duration',
+                        'label' => __( 'Recipe total time (e.g. 1:30)', 'wordlift' ),
+                    ),
+                ),
+            ),
+        );
+
+        // Merge the custom fields with those provided by the parent schema.
+        $parent_schema           = $this->get_creative_work_schema();
+        $schema['custom_fields'] = array_merge( $schema['custom_fields'], $parent_schema['custom_fields'] );
+
+        return $schema;
+    }
+
+    /**
+     * Get the 'offer' schema.
+     *
+     * @return array An array with the schema configuration.
+     *
+     * @since 3.18.0
+     */
+    private function get_offer_schema() {
+
+        $schema = array(
+            'label'         => 'Offer',
+            'description'   => 'An offer. ',
+            'parents'       => array( 'thing' ),
+            'css_class'     => 'wl-offer',
+            'uri'           => self::SCHEMA_OFFER_TYPE,
+            'same_as'       => array(),
+            'templates'     => array(
+                'subtitle' => '{{id}}',
+            ),
+            'custom_fields' => array(
+                self::FIELD_AVAILABILITY        => array(
+                    'predicate'   => 'http://schema.org/availability',
+                    'type'        => self::DATA_TYPE_STRING,
+                    'export_type' => 'xsd:string',
+                    'metabox'     => array(
+                        'class' => 'Wordlift_Metabox_Field_Select',
+                    ),
+                    'options'     => array(
+                        'Discontinued'        => esc_html__( 'Discontinued', 'wordlift' ),
+                        'InStock'             => esc_html__( 'In Stock', 'wordlift' ),
+                        'InStoreOnly'         => esc_html__( 'In Store Only', 'wordlift' ),
+                        'LimitedAvailability' => esc_html__( 'Limited Availability', 'wordlift' ),
+                        'OnlineOnly'          => esc_html__( 'Online Only', 'wordlift' ),
+                        'OutOfStock'          => esc_html__( 'Out of Stock', 'wordlift' ),
+                        'PreOrder'            => esc_html__( 'Pre Order', 'wordlift' ),
+                        'PreSale'             => esc_html__( 'Pre Sale', 'wordlift' ),
+                        'SoldOut'             => esc_html__( 'Sold Out', 'wordlift' ),
+                    ),
+                ),
+                self::FIELD_PRICE               => array(
+                    'predicate'   => 'http://schema.org/price',
+                    'type'        => self::DATA_TYPE_STRING,
+                    'export_type' => 'xsd:integer',
+                    'metabox'     => array(
+                        'class' => 'Wordlift_Metabox_Field_Integer',
+                    ),
+                ),
+                self::FIELD_PRICE_CURRENCY      => array(
+                    'predicate'   => 'http://schema.org/priceCurrency',
+                    'type'        => self::DATA_TYPE_STRING,
+                    'export_type' => 'xsd:string',
+                ),
+                self::FIELD_AVAILABILITY_STARTS => array(
+                    'predicate'   => 'http://schema.org/availabilityStarts',
+                    'type'        => self::DATA_TYPE_DATE,
+                    'export_type' => 'xsd:dateTime',
+                ),
+                self::FIELD_AVAILABILITY_ENDS   => array(
+                    'predicate'   => 'http://schema.org/availabilityEnds',
+                    'type'        => self::DATA_TYPE_DATE,
+                    'export_type' => 'xsd:dateTime',
+                ),
+                self::FIELD_INVENTORY_LEVEL     => array(
+                    'predicate'   => 'http://schema.org/inventoryLevel',
+                    'type'        => self::DATA_TYPE_STRING,
+                    'export_type' => 'xsd:integer',
+                    'metabox'     => array(
+                        'class' => 'Wordlift_Metabox_Field_Integer',
+                    ),
+                ),
+                self::FIELD_VALID_FROM          => array(
+                    'predicate'   => 'http://schema.org/validFrom',
+                    'type'        => self::DATA_TYPE_DATE,
+                    'export_type' => 'xsd:dateTime',
+                ),
+                self::FIELD_PRICE_VALID_UNTIL   => array(
+                    'predicate'   => 'http://schema.org/priceValidUntil',
+                    'type'        => self::DATA_TYPE_DATE,
+                    'export_type' => 'xsd:dateTime',
+                ),
+                self::FIELD_ITEM_OFFERED        => array(
+                    'predicate'   => 'http://schema.org/itemOffered',
+                    'type'        => self::DATA_TYPE_URI,
+                    'export_type' => 'http://schema.org/Thing',
+                    'constraints' => array(
+                        'uri_type'    => array(
+                            'Event',
+                            'Thing',
+                        ),
+                        'cardinality' => INF,
+                    ),
+                ),
+            ),
+        );
+
+        // Merge the custom fields with those provided by the thing schema.
+        $parent_schema           = $this->get_thing_schema();
+        $schema['custom_fields'] = array_merge( $schema['custom_fields'], $parent_schema['custom_fields'] );
+
+        return $schema;
+    }
+
+    /**
+     * Get the 'article' schema.
+     *
+     * @return array An array with the schema configuration.
+     *
+     * @since 3.15.0
+     */
+    private function get_article_schema() {
+
+        $schema = array(
+            'label'         => 'Article',
+            'description'   => 'An Article.',
+            'parents'       => array(),
+            'css_class'     => 'wl-article',
+            'uri'           => 'http://schema.org/Article',
+            'same_as'       => array(),
+            'templates'     => array(
+                'subtitle' => '{{id}}',
+            ),
+            'custom_fields' => array(),
+        );
+
+        return $schema;
+    }
 
 }

Spacing +53 added lines, -53 removed lines

@@ -399,7 +399,7 @@ 
 	 */
 	protected function __construct() {
 
-		$this->log = Wordlift_Log_Service::get_logger( 'Wordlift_Schema_Service' );
+		$this->log = Wordlift_Log_Service::get_logger('Wordlift_Schema_Service');
 
 		/**
 		 * Alter the configured schemas.
@@ -448,7 +448,7 @@ 
 	 * @since 3.1.0
 	 */
 	public static function get_instance() {
-		if ( ! isset( self::$instance ) ) {
+		if ( ! isset(self::$instance)) {
 			self::$instance = new self();
 		}
 
@@ -464,19 +464,19 @@ 
 	 * @return null|array An array of field's properties or null if the field is not found.
 	 * @since 3.6.0
 	 */
-	public function get_field( $key ) {
+	public function get_field($key) {
 
 		// Parse each schema's fields until we find the one we're looking for, then
 		// return its properties.
 		// phpcs:ignore VariableAnalysis.CodeAnalysis.VariableAnalysis.UnusedVariable
-		foreach ( $this->schema as $_ => $schema ) {
+		foreach ($this->schema as $_ => $schema) {
 
-			if ( ! isset( $schema['custom_fields'] ) ) {
+			if ( ! isset($schema['custom_fields'])) {
 				break;
 			}
 
-			foreach ( $schema['custom_fields'] as $field => $props ) {
-				if ( $key === $field ) {
+			foreach ($schema['custom_fields'] as $field => $props) {
+				if ($key === $field) {
 					return $props;
 				}
 			}
@@ -494,14 +494,14 @@ 
 	 *
 	 * @since 3.1.0
 	 */
-	public function get_schema( $name ) {
+	public function get_schema($name) {
 		// Check if the schema exists and, if not, return NULL.
-		if ( ! isset( $this->schema[ $name ] ) ) {
+		if ( ! isset($this->schema[$name])) {
 			return null;
 		}
 
 		// Return the requested schema.
-		return $this->schema[ $name ];
+		return $this->schema[$name];
 	}
 
 	/**
@@ -513,11 +513,11 @@ 
 	 *
 	 * @since 3.3.0
 	 */
-	public function get_schema_by_uri( $uri ) {
+	public function get_schema_by_uri($uri) {
 
 		// phpcs:ignore VariableAnalysis.CodeAnalysis.VariableAnalysis.UnusedVariable
-		foreach ( $this->schema as $name => $schema ) {
-			if ( $schema['uri'] === $uri ) {
+		foreach ($this->schema as $name => $schema) {
+			if ($schema['uri'] === $uri) {
 				return $schema;
 			}
 		}
@@ -537,7 +537,7 @@ 
 		return array(
 			'css_class'     => 'wl-thing',
 			'uri'           => 'http://schema.org/Thing',
-			'same_as'       => array( '*' ),
+			'same_as'       => array('*'),
 			// set as default.
 			'custom_fields' => array(
 				self::FIELD_SAME_AS => array(
@@ -591,7 +591,7 @@ 
 		$schema = array(
 			'label'         => 'CreativeWork',
 			'description'   => 'A creative work (or a Music Album).',
-			'parents'       => array( 'thing' ),
+			'parents'       => array('thing'),
 			// Give term slug as parent.
 			'css_class'     => 'wl-creative-work',
 			'uri'           => 'http://schema.org/CreativeWork',
@@ -605,7 +605,7 @@ 
 					'type'        => self::DATA_TYPE_URI,
 					'export_type' => 'http://schema.org/Person',
 					'constraints' => array(
-						'uri_type'    => array( 'Person', 'Organization' ),
+						'uri_type'    => array('Person', 'Organization'),
 						'cardinality' => INF,
 					),
 				),
@@ -617,7 +617,7 @@ 
 
 		// Merge the custom fields with those provided by the thing schema.
 		$parent_schema           = $this->get_thing_schema();
-		$schema['custom_fields'] = array_merge( $schema['custom_fields'], $parent_schema['custom_fields'] );
+		$schema['custom_fields'] = array_merge($schema['custom_fields'], $parent_schema['custom_fields']);
 
 		return $schema;
 	}
@@ -634,10 +634,10 @@ 
 		$schema = array(
 			'label'         => 'Event',
 			'description'   => 'An event . ',
-			'parents'       => array( 'thing' ),
+			'parents'       => array('thing'),
 			'css_class'     => 'wl-event',
 			'uri'           => self::SCHEMA_EVENT_TYPE,
-			'same_as'       => array( 'http://dbpedia.org/ontology/Event' ),
+			'same_as'       => array('http://dbpedia.org/ontology/Event'),
 			'custom_fields' => array(
 				self::FIELD_DATE_START => array(
 					'predicate'   => 'http://schema.org/startDate',
@@ -656,7 +656,7 @@ 
 					'type'        => self::DATA_TYPE_URI,
 					'export_type' => 'http://schema.org/PostalAddress',
 					'constraints' => array(
-						'uri_type'    => array( 'Place', 'LocalBusiness' ),
+						'uri_type'    => array('Place', 'LocalBusiness'),
 						'cardinality' => INF,
 					),
 				),
@@ -665,7 +665,7 @@ 
 					'type'        => self::DATA_TYPE_URI,
 					'export_type' => 'http://schema.org/Person',
 					'constraints' => array(
-						'uri_type'    => array( 'Person', 'Organization' ),
+						'uri_type'    => array('Person', 'Organization'),
 						'cardinality' => INF,
 					),
 				),
@@ -674,7 +674,7 @@ 
 					'type'        => self::DATA_TYPE_URI,
 					'export_type' => 'http://schema.org/Offer',
 					'constraints' => array(
-						'uri_type'    => array( 'Offer' ),
+						'uri_type'    => array('Offer'),
 						'cardinality' => INF,
 					),
 				),
@@ -686,7 +686,7 @@ 
 
 		// Merge the custom fields with those provided by the thing schema.
 		$parent_schema           = $this->get_thing_schema();
-		$schema['custom_fields'] = array_merge( $schema['custom_fields'], $parent_schema['custom_fields'] );
+		$schema['custom_fields'] = array_merge($schema['custom_fields'], $parent_schema['custom_fields']);
 
 		return $schema;
 	}
@@ -703,7 +703,7 @@ 
 		$schema = array(
 			'label'         => 'Organization',
 			'description'   => 'An organization, including a government or a newspaper.',
-			'parents'       => array( 'thing' ),
+			'parents'       => array('thing'),
 			'css_class'     => 'wl-organization',
 			'uri'           => 'http://schema.org/Organization',
 			'same_as'       => array(
@@ -795,7 +795,7 @@ 
 
 		// Merge the custom fields with those provided by the thing schema.
 		$parent_schema           = $this->get_thing_schema();
-		$schema['custom_fields'] = array_merge( $schema['custom_fields'], $parent_schema['custom_fields'] );
+		$schema['custom_fields'] = array_merge($schema['custom_fields'], $parent_schema['custom_fields']);
 
 		return $schema;
 	}
@@ -812,7 +812,7 @@ 
 		$schema = array(
 			'label'         => 'Person',
 			'description'   => 'A person (or a music artist).',
-			'parents'       => array( 'thing' ),
+			'parents'       => array('thing'),
 			'css_class'     => 'wl-person',
 			'uri'           => 'http://schema.org/Person',
 			'same_as'       => array(
@@ -872,7 +872,7 @@ 
 
 		// Merge the custom fields with those provided by the thing schema.
 		$parent_schema           = $this->get_thing_schema();
-		$schema['custom_fields'] = array_merge( $schema['custom_fields'], $parent_schema['custom_fields'] );
+		$schema['custom_fields'] = array_merge($schema['custom_fields'], $parent_schema['custom_fields']);
 
 		return $schema;
 
@@ -890,7 +890,7 @@ 
 		$schema = array(
 			'label'         => 'Place',
 			'description'   => 'A place.',
-			'parents'       => array( 'thing' ),
+			'parents'       => array('thing'),
 			'css_class'     => 'wl-place',
 			'uri'           => 'http://schema.org/Place',
 			'same_as'       => array(
@@ -970,7 +970,7 @@ 
 
 		// Merge the custom fields with those provided by the thing schema.
 		$parent_schema           = $this->get_thing_schema();
-		$schema['custom_fields'] = array_merge( $schema['custom_fields'], $parent_schema['custom_fields'] );
+		$schema['custom_fields'] = array_merge($schema['custom_fields'], $parent_schema['custom_fields']);
 
 		return $schema;
 	}
@@ -987,7 +987,7 @@ 
 		$schema = array(
 			'label'         => 'LocalBusiness',
 			'description'   => 'A local business.',
-			'parents'       => array( 'place', 'organization' ),
+			'parents'       => array('place', 'organization'),
 			'css_class'     => 'wl-local-business',
 			'uri'           => 'http://schema.org/LocalBusiness',
 			'same_as'       => array(
@@ -1003,7 +1003,7 @@ 
 		// Merge the custom fields with those provided by the place and organization schema.
 		$place_schema            = $this->get_place_schema();
 		$organization_schema     = $this->get_organization_schema();
-		$schema['custom_fields'] = array_merge( $schema['custom_fields'], $place_schema['custom_fields'], $organization_schema['custom_fields'] );
+		$schema['custom_fields'] = array_merge($schema['custom_fields'], $place_schema['custom_fields'], $organization_schema['custom_fields']);
 
 		return $schema;
 	}
@@ -1020,7 +1020,7 @@ 
 		$schema = array(
 			'label'         => 'Recipe',
 			'description'   => 'A Recipe.',
-			'parents'       => array( 'CreativeWork' ),
+			'parents'       => array('CreativeWork'),
 			'css_class'     => 'wl-recipe',
 			'uri'           => 'http://schema.org/Recipe',
 			'same_as'       => array(),
@@ -1034,7 +1034,7 @@ 
 					'export_type' => 'xsd:string',
 					'constraints' => '',
 					'metabox'     => array(
-						'label' => __( 'Recipe cuisine', 'wordlift' ),
+						'label' => __('Recipe cuisine', 'wordlift'),
 					),
 				),
 				self::FIELD_RECIPE_INGREDIENT       => array(
@@ -1045,7 +1045,7 @@ 
 						'cardinality' => INF,
 					),
 					'metabox'     => array(
-						'label' => __( 'Recipe ingredient', 'wordlift' ),
+						'label' => __('Recipe ingredient', 'wordlift'),
 					),
 				),
 				self::FIELD_RECIPE_INSTRUCTIONS     => array(
@@ -1055,7 +1055,7 @@ 
 					'constraints' => '',
 					'metabox'     => array(
 						'class' => 'Wordlift_Metabox_Field_Multiline',
-						'label' => __( 'Recipe instructions', 'wordlift' ),
+						'label' => __('Recipe instructions', 'wordlift'),
 					),
 				),
 				self::FIELD_RECIPE_YIELD            => array(
@@ -1064,7 +1064,7 @@ 
 					'export_type' => 'xsd:string',
 					'constraints' => '',
 					'metabox'     => array(
-						'label' => __( 'Recipe number of servings', 'wordlift' ),
+						'label' => __('Recipe number of servings', 'wordlift'),
 					),
 				),
 				self::FIELD_RECIPE_INGREDIENT       => array(
@@ -1075,7 +1075,7 @@ 
 						'cardinality' => INF,
 					),
 					'metabox'     => array(
-						'label' => __( 'Recipe ingredient', 'wordlift' ),
+						'label' => __('Recipe ingredient', 'wordlift'),
 					),
 				),
 				self::FIELD_NUTRITION_INFO_CALORIES => array(
@@ -1084,7 +1084,7 @@ 
 					'export_type' => 'xsd:string',
 					'constraints' => '',
 					'metabox'     => array(
-						'label' => __( 'Calories (e.g. 240 calories)', 'wordlift' ),
+						'label' => __('Calories (e.g. 240 calories)', 'wordlift'),
 					),
 				),
 				self::FIELD_PREP_TIME               => array(
@@ -1094,7 +1094,7 @@ 
 					'constraints' => '',
 					'metabox'     => array(
 						'class' => 'Wordlift_Metabox_Field_Duration',
-						'label' => __( 'Recipe preparation time (e.g. 1:30)', 'wordlift' ),
+						'label' => __('Recipe preparation time (e.g. 1:30)', 'wordlift'),
 					),
 				),
 				self::FIELD_COOK_TIME               => array(
@@ -1104,7 +1104,7 @@ 
 					'constraints' => '',
 					'metabox'     => array(
 						'class' => 'Wordlift_Metabox_Field_Duration',
-						'label' => __( 'Recipe cook time (e.g. 1:30)', 'wordlift' ),
+						'label' => __('Recipe cook time (e.g. 1:30)', 'wordlift'),
 					),
 				),
 				self::FIELD_TOTAL_TIME              => array(
@@ -1114,7 +1114,7 @@ 
 					'constraints' => '',
 					'metabox'     => array(
 						'class' => 'Wordlift_Metabox_Field_Duration',
-						'label' => __( 'Recipe total time (e.g. 1:30)', 'wordlift' ),
+						'label' => __('Recipe total time (e.g. 1:30)', 'wordlift'),
 					),
 				),
 			),
@@ -1122,7 +1122,7 @@ 
 
 		// Merge the custom fields with those provided by the parent schema.
 		$parent_schema           = $this->get_creative_work_schema();
-		$schema['custom_fields'] = array_merge( $schema['custom_fields'], $parent_schema['custom_fields'] );
+		$schema['custom_fields'] = array_merge($schema['custom_fields'], $parent_schema['custom_fields']);
 
 		return $schema;
 	}
@@ -1139,7 +1139,7 @@ 
 		$schema = array(
 			'label'         => 'Offer',
 			'description'   => 'An offer. ',
-			'parents'       => array( 'thing' ),
+			'parents'       => array('thing'),
 			'css_class'     => 'wl-offer',
 			'uri'           => self::SCHEMA_OFFER_TYPE,
 			'same_as'       => array(),
@@ -1155,15 +1155,15 @@ 
 						'class' => 'Wordlift_Metabox_Field_Select',
 					),
 					'options'     => array(
-						'Discontinued'        => esc_html__( 'Discontinued', 'wordlift' ),
-						'InStock'             => esc_html__( 'In Stock', 'wordlift' ),
-						'InStoreOnly'         => esc_html__( 'In Store Only', 'wordlift' ),
-						'LimitedAvailability' => esc_html__( 'Limited Availability', 'wordlift' ),
-						'OnlineOnly'          => esc_html__( 'Online Only', 'wordlift' ),
-						'OutOfStock'          => esc_html__( 'Out of Stock', 'wordlift' ),
-						'PreOrder'            => esc_html__( 'Pre Order', 'wordlift' ),
-						'PreSale'             => esc_html__( 'Pre Sale', 'wordlift' ),
-						'SoldOut'             => esc_html__( 'Sold Out', 'wordlift' ),
+						'Discontinued'        => esc_html__('Discontinued', 'wordlift'),
+						'InStock'             => esc_html__('In Stock', 'wordlift'),
+						'InStoreOnly'         => esc_html__('In Store Only', 'wordlift'),
+						'LimitedAvailability' => esc_html__('Limited Availability', 'wordlift'),
+						'OnlineOnly'          => esc_html__('Online Only', 'wordlift'),
+						'OutOfStock'          => esc_html__('Out of Stock', 'wordlift'),
+						'PreOrder'            => esc_html__('Pre Order', 'wordlift'),
+						'PreSale'             => esc_html__('Pre Sale', 'wordlift'),
+						'SoldOut'             => esc_html__('Sold Out', 'wordlift'),
 					),
 				),
 				self::FIELD_PRICE               => array(
@@ -1224,7 +1224,7 @@ 
 
 		// Merge the custom fields with those provided by the thing schema.
 		$parent_schema           = $this->get_thing_schema();
-		$schema['custom_fields'] = array_merge( $schema['custom_fields'], $parent_schema['custom_fields'] );
+		$schema['custom_fields'] = array_merge($schema['custom_fields'], $parent_schema['custom_fields']);
 
 		return $schema;
 	}

src/includes/class-wordlift-loader.php

Indentation +102 added lines, -102 removed lines

@@ -23,107 +23,107 @@
  */
 class Wordlift_Loader {
 
-	/**
-	 * The array of actions registered with WordPress.
-	 *
-	 * @since    1.0.0
-	 * @access   protected
-	 * @var      array    $actions    The actions registered with WordPress to fire when the plugin loads.
-	 */
-	protected $actions;
-
-	/**
-	 * The array of filters registered with WordPress.
-	 *
-	 * @since    1.0.0
-	 * @access   protected
-	 * @var      array    $filters    The filters registered with WordPress to fire when the plugin loads.
-	 */
-	protected $filters;
-
-	/**
-	 * Initialize the collections used to maintain the actions and filters.
-	 *
-	 * @since    1.0.0
-	 */
-	public function __construct() {
-
-		$this->actions = array();
-		$this->filters = array();
-
-	}
-
-	/**
-	 * Add a new action to the collection to be registered with WordPress.
-	 *
-	 * @since    1.0.0
-	 * @param    string $hook             The name of the WordPress action that is being registered.
-	 * @param    object $component        A reference to the instance of the object on which the action is defined.
-	 * @param    string $callback         The name of the function definition on the $component.
-	 * @param    int    $priority         Optional. he priority at which the function should be fired. Default is 10.
-	 * @param    int    $accepted_args    Optional. The number of arguments that should be passed to the $callback. Default is 1.
-	 */
-	public function add_action( $hook, $component, $callback, $priority = 10, $accepted_args = 1 ) {
-		add_action( $hook, array( $component, $callback ), $priority, $accepted_args );
-	}
-
-	/**
-	 * Add a new filter to the collection to be registered with WordPress.
-	 *
-	 * @since    1.0.0
-	 * @param    string $hook             The name of the WordPress filter that is being registered.
-	 * @param    object $component        A reference to the instance of the object on which the filter is defined.
-	 * @param    string $callback         The name of the function definition on the $component.
-	 * @param    int    $priority         Optional. he priority at which the function should be fired. Default is 10.
-	 * @param    int    $accepted_args    Optional. The number of arguments that should be passed to the $callback. Default is 1
-	 */
-	public function add_filter( $hook, $component, $callback, $priority = 10, $accepted_args = 1 ) {
-		add_filter( $hook, array( $component, $callback ), $priority, $accepted_args );
-	}
-
-	/**
-	 * A utility function that is used to register the actions and hooks into a single
-	 * collection.
-	 *
-	 * @since    1.0.0
-	 * @access   private
-	 * @param    array  $hooks            The collection of hooks that is being registered (that is, actions or filters).
-	 * @param    string $hook             The name of the WordPress filter that is being registered.
-	 * @param    object $component        A reference to the instance of the object on which the filter is defined.
-	 * @param    string $callback         The name of the function definition on the $component.
-	 * @param    int    $priority         The priority at which the function should be fired.
-	 * @param    int    $accepted_args    The number of arguments that should be passed to the $callback.
-	 * @return   array                                  The collection of actions and filters registered with WordPress.
-	 */
-	private function add( $hooks, $hook, $component, $callback, $priority, $accepted_args ) {
-
-		$hooks[] = array(
-			'hook'          => $hook,
-			'component'     => $component,
-			'callback'      => $callback,
-			'priority'      => $priority,
-			'accepted_args' => $accepted_args,
-		);
-
-		return $hooks;
-
-	}
-
-	/**
-	 * Register the filters and actions with WordPress.
-	 *
-	 * @since    1.0.0
-	 */
-	public function run() {
-
-		foreach ( $this->filters as $hook ) {
-			add_filter( $hook['hook'], array( $hook['component'], $hook['callback'] ), $hook['priority'], $hook['accepted_args'] );
-		}
-
-		foreach ( $this->actions as $hook ) {
-			add_action( $hook['hook'], array( $hook['component'], $hook['callback'] ), $hook['priority'], $hook['accepted_args'] );
-		}
-
-	}
+    /**
+     * The array of actions registered with WordPress.
+     *
+     * @since    1.0.0
+     * @access   protected
+     * @var      array    $actions    The actions registered with WordPress to fire when the plugin loads.
+     */
+    protected $actions;
+
+    /**
+     * The array of filters registered with WordPress.
+     *
+     * @since    1.0.0
+     * @access   protected
+     * @var      array    $filters    The filters registered with WordPress to fire when the plugin loads.
+     */
+    protected $filters;
+
+    /**
+     * Initialize the collections used to maintain the actions and filters.
+     *
+     * @since    1.0.0
+     */
+    public function __construct() {
+
+        $this->actions = array();
+        $this->filters = array();
+
+    }
+
+    /**
+     * Add a new action to the collection to be registered with WordPress.
+     *
+     * @since    1.0.0
+     * @param    string $hook             The name of the WordPress action that is being registered.
+     * @param    object $component        A reference to the instance of the object on which the action is defined.
+     * @param    string $callback         The name of the function definition on the $component.
+     * @param    int    $priority         Optional. he priority at which the function should be fired. Default is 10.
+     * @param    int    $accepted_args    Optional. The number of arguments that should be passed to the $callback. Default is 1.
+     */
+    public function add_action( $hook, $component, $callback, $priority = 10, $accepted_args = 1 ) {
+        add_action( $hook, array( $component, $callback ), $priority, $accepted_args );
+    }
+
+    /**
+     * Add a new filter to the collection to be registered with WordPress.
+     *
+     * @since    1.0.0
+     * @param    string $hook             The name of the WordPress filter that is being registered.
+     * @param    object $component        A reference to the instance of the object on which the filter is defined.
+     * @param    string $callback         The name of the function definition on the $component.
+     * @param    int    $priority         Optional. he priority at which the function should be fired. Default is 10.
+     * @param    int    $accepted_args    Optional. The number of arguments that should be passed to the $callback. Default is 1
+     */
+    public function add_filter( $hook, $component, $callback, $priority = 10, $accepted_args = 1 ) {
+        add_filter( $hook, array( $component, $callback ), $priority, $accepted_args );
+    }
+
+    /**
+     * A utility function that is used to register the actions and hooks into a single
+     * collection.
+     *
+     * @since    1.0.0
+     * @access   private
+     * @param    array  $hooks            The collection of hooks that is being registered (that is, actions or filters).
+     * @param    string $hook             The name of the WordPress filter that is being registered.
+     * @param    object $component        A reference to the instance of the object on which the filter is defined.
+     * @param    string $callback         The name of the function definition on the $component.
+     * @param    int    $priority         The priority at which the function should be fired.
+     * @param    int    $accepted_args    The number of arguments that should be passed to the $callback.
+     * @return   array                                  The collection of actions and filters registered with WordPress.
+     */
+    private function add( $hooks, $hook, $component, $callback, $priority, $accepted_args ) {
+
+        $hooks[] = array(
+            'hook'          => $hook,
+            'component'     => $component,
+            'callback'      => $callback,
+            'priority'      => $priority,
+            'accepted_args' => $accepted_args,
+        );
+
+        return $hooks;
+
+    }
+
+    /**
+     * Register the filters and actions with WordPress.
+     *
+     * @since    1.0.0
+     */
+    public function run() {
+
+        foreach ( $this->filters as $hook ) {
+            add_filter( $hook['hook'], array( $hook['component'], $hook['callback'] ), $hook['priority'], $hook['accepted_args'] );
+        }
+
+        foreach ( $this->actions as $hook ) {
+            add_action( $hook['hook'], array( $hook['component'], $hook['callback'] ), $hook['priority'], $hook['accepted_args'] );
+        }
+
+    }
 
 }

Spacing +9 added lines, -9 removed lines

@@ -63,8 +63,8 @@ 
 	 * @param    int    $priority         Optional. he priority at which the function should be fired. Default is 10.
 	 * @param    int    $accepted_args    Optional. The number of arguments that should be passed to the $callback. Default is 1.
 	 */
-	public function add_action( $hook, $component, $callback, $priority = 10, $accepted_args = 1 ) {
-		add_action( $hook, array( $component, $callback ), $priority, $accepted_args );
+	public function add_action($hook, $component, $callback, $priority = 10, $accepted_args = 1) {
+		add_action($hook, array($component, $callback), $priority, $accepted_args);
 	}
 
 	/**
@@ -77,8 +77,8 @@ 
 	 * @param    int    $priority         Optional. he priority at which the function should be fired. Default is 10.
 	 * @param    int    $accepted_args    Optional. The number of arguments that should be passed to the $callback. Default is 1
 	 */
-	public function add_filter( $hook, $component, $callback, $priority = 10, $accepted_args = 1 ) {
-		add_filter( $hook, array( $component, $callback ), $priority, $accepted_args );
+	public function add_filter($hook, $component, $callback, $priority = 10, $accepted_args = 1) {
+		add_filter($hook, array($component, $callback), $priority, $accepted_args);
 	}
 
 	/**
@@ -95,7 +95,7 @@ 
 	 * @param    int    $accepted_args    The number of arguments that should be passed to the $callback.
 	 * @return   array                                  The collection of actions and filters registered with WordPress.
 	 */
-	private function add( $hooks, $hook, $component, $callback, $priority, $accepted_args ) {
+	private function add($hooks, $hook, $component, $callback, $priority, $accepted_args) {
 
 		$hooks[] = array(
 			'hook'          => $hook,
@@ -116,12 +116,12 @@ 
 	 */
 	public function run() {
 
-		foreach ( $this->filters as $hook ) {
-			add_filter( $hook['hook'], array( $hook['component'], $hook['callback'] ), $hook['priority'], $hook['accepted_args'] );
+		foreach ($this->filters as $hook) {
+			add_filter($hook['hook'], array($hook['component'], $hook['callback']), $hook['priority'], $hook['accepted_args']);
 		}
 
-		foreach ( $this->actions as $hook ) {
-			add_action( $hook['hook'], array( $hook['component'], $hook['callback'] ), $hook['priority'], $hook['accepted_args'] );
+		foreach ($this->actions as $hook) {
+			add_action($hook['hook'], array($hook['component'], $hook['callback']), $hook['priority'], $hook['accepted_args']);
 		}
 
 	}