insideout10 / wordlift-plugin

src/wordlift/vocabulary-terms/jsonld/class-jsonld-generator.php

Indentation +146 added lines, -146 removed lines

@@ -11,151 +11,151 @@
 
 class Jsonld_Generator {
 
-	/**
-	 * @var \Wordlift_Entity_Type_Service
-	 */
-	private $entity_type_service;
-	/**
-	 * @var \Wordlift_Property_Getter
-	 */
-	private $property_getter;
-	/**
-	 * @var Type_Service
-	 */
-	private $term_entity_type_service;
-	/**
-	 * @var \Wordlift_Entity_Service
-	 */
-	private $entity_service;
-
-	public function __construct( $entity_type_service, $property_getter ) {
-		$this->entity_type_service      = $entity_type_service;
-		$this->property_getter          = $property_getter;
-		$this->term_entity_type_service = Type_Service::get_instance();
-		$this->entity_service           = \Wordlift_Entity_Service::get_instance();
-	}
-
-	public function init() {
-		add_filter( 'wl_term_jsonld_array', array( $this, 'wl_term_jsonld_array' ), 10, 2 );
-	}
-
-	public function wl_term_jsonld_array( $data, $term_id ) {
-		$jsonld     = $data['jsonld'];
-		$references = $data['references'];
-
-		$term_jsonld_data = $this->get_jsonld_data_for_term( $term_id );
-
-		// Return early if we dont have the entity data
-		// for the term.
-		if ( ! $term_jsonld_data ) {
-			return $data;
-		}
-
-		$term_jsonld = $term_jsonld_data['jsonld'];
-
-		$references = array_merge( $references, $term_jsonld_data['references'] );
-
-		array_unshift( $jsonld, $term_jsonld );
-
-		return array(
-			'jsonld'     => $jsonld,
-			'references' => $references,
-		);
-	}
-
-	private function get_jsonld_data_for_term( $term_id ) {
-
-		$id = $this->entity_service->get_uri( $term_id, Object_Type_Enum::TERM );
-
-		// If we don't have a dataset  URI, then don't publish the term data
-		// on this page.
-		if ( ! $id ) {
-			return false;
-		}
-
-		$references = array();
-		$term       = get_term( $term_id );
-		$permalink  = get_term_link( $term );
-
-		$custom_fields = $this->entity_type_service->get_custom_fields_for_term( $term_id );
-		$term          = get_term( $term_id );
-		$jsonld        = array(
-			'@context'    => 'http://schema.org',
-			'name'        => $term->name,
-			'@type'       => $this->term_entity_type_service->get_entity_types_labels( $term_id ),
-			'@id'         => $id,
-			'description' => $term->description,
-		);
-
-		if ( ! $custom_fields || ! is_array( $custom_fields ) ) {
-			return $jsonld;
-		}
-
-		foreach ( $custom_fields as $key => $value ) {
-			$name  = $this->relative_to_schema_context( $value['predicate'] );
-			$value = $this->property_getter->get( $term_id, $key, Object_Type_Enum::TERM );
-			$value = $this->process_value( $value, $references );
-			if ( ! isset( $value ) ||
-				 is_array( $value ) && empty( $value ) ||
-				 is_string( $value ) && empty( $value ) ) {
-				continue;
-			}
-			$jsonld[ $name ] = $value;
-
-		}
-
-		if ( $permalink ) {
-			$jsonld['mainEntityOfPage'] = $permalink;
-		}
-
-		return apply_filters(
-			'wl_no_vocabulary_term_jsonld_array',
-			array(
-				'jsonld'     => $jsonld,
-				'references' => $references,
-			),
-			$term_id
-		);
-
-	}
-
-	private function relative_to_schema_context( $predicate ) {
-		return str_replace( 'http://schema.org/', '', $predicate );
-	}
-
-	private function process_value( $value, &$references ) {
-
-		if ( is_array( $value )
-			 && count( $value ) > 0
-			 && $value[0] instanceof \Wordlift_Property_Entity_Reference ) {
-
-			// All of the references from the custom fields are post references.
-			$references = array_merge(
-				$references,
-				array_map(
-					function ( $property_entity_reference ) {
-						/**
-						 * @var $property_entity_reference \Wordlift_Property_Entity_Reference
-						 */
-						return $property_entity_reference->to_reference();
-					},
-					$value
-				)
-			);
-
-			return array_map(
-				function ( $reference ) {
-					/**
-					 * @var $reference \Wordlift_Property_Entity_Reference
-					 */
-					return array( '@id' => $reference->get_url() );
-				},
-				$value
-			);
-
-		}
-
-		return $value;
-	}
+    /**
+     * @var \Wordlift_Entity_Type_Service
+     */
+    private $entity_type_service;
+    /**
+     * @var \Wordlift_Property_Getter
+     */
+    private $property_getter;
+    /**
+     * @var Type_Service
+     */
+    private $term_entity_type_service;
+    /**
+     * @var \Wordlift_Entity_Service
+     */
+    private $entity_service;
+
+    public function __construct( $entity_type_service, $property_getter ) {
+        $this->entity_type_service      = $entity_type_service;
+        $this->property_getter          = $property_getter;
+        $this->term_entity_type_service = Type_Service::get_instance();
+        $this->entity_service           = \Wordlift_Entity_Service::get_instance();
+    }
+
+    public function init() {
+        add_filter( 'wl_term_jsonld_array', array( $this, 'wl_term_jsonld_array' ), 10, 2 );
+    }
+
+    public function wl_term_jsonld_array( $data, $term_id ) {
+        $jsonld     = $data['jsonld'];
+        $references = $data['references'];
+
+        $term_jsonld_data = $this->get_jsonld_data_for_term( $term_id );
+
+        // Return early if we dont have the entity data
+        // for the term.
+        if ( ! $term_jsonld_data ) {
+            return $data;
+        }
+
+        $term_jsonld = $term_jsonld_data['jsonld'];
+
+        $references = array_merge( $references, $term_jsonld_data['references'] );
+
+        array_unshift( $jsonld, $term_jsonld );
+
+        return array(
+            'jsonld'     => $jsonld,
+            'references' => $references,
+        );
+    }
+
+    private function get_jsonld_data_for_term( $term_id ) {
+
+        $id = $this->entity_service->get_uri( $term_id, Object_Type_Enum::TERM );
+
+        // If we don't have a dataset  URI, then don't publish the term data
+        // on this page.
+        if ( ! $id ) {
+            return false;
+        }
+
+        $references = array();
+        $term       = get_term( $term_id );
+        $permalink  = get_term_link( $term );
+
+        $custom_fields = $this->entity_type_service->get_custom_fields_for_term( $term_id );
+        $term          = get_term( $term_id );
+        $jsonld        = array(
+            '@context'    => 'http://schema.org',
+            'name'        => $term->name,
+            '@type'       => $this->term_entity_type_service->get_entity_types_labels( $term_id ),
+            '@id'         => $id,
+            'description' => $term->description,
+        );
+
+        if ( ! $custom_fields || ! is_array( $custom_fields ) ) {
+            return $jsonld;
+        }
+
+        foreach ( $custom_fields as $key => $value ) {
+            $name  = $this->relative_to_schema_context( $value['predicate'] );
+            $value = $this->property_getter->get( $term_id, $key, Object_Type_Enum::TERM );
+            $value = $this->process_value( $value, $references );
+            if ( ! isset( $value ) ||
+                 is_array( $value ) && empty( $value ) ||
+                 is_string( $value ) && empty( $value ) ) {
+                continue;
+            }
+            $jsonld[ $name ] = $value;
+
+        }
+
+        if ( $permalink ) {
+            $jsonld['mainEntityOfPage'] = $permalink;
+        }
+
+        return apply_filters(
+            'wl_no_vocabulary_term_jsonld_array',
+            array(
+                'jsonld'     => $jsonld,
+                'references' => $references,
+            ),
+            $term_id
+        );
+
+    }
+
+    private function relative_to_schema_context( $predicate ) {
+        return str_replace( 'http://schema.org/', '', $predicate );
+    }
+
+    private function process_value( $value, &$references ) {
+
+        if ( is_array( $value )
+             && count( $value ) > 0
+             && $value[0] instanceof \Wordlift_Property_Entity_Reference ) {
+
+            // All of the references from the custom fields are post references.
+            $references = array_merge(
+                $references,
+                array_map(
+                    function ( $property_entity_reference ) {
+                        /**
+                         * @var $property_entity_reference \Wordlift_Property_Entity_Reference
+                         */
+                        return $property_entity_reference->to_reference();
+                    },
+                    $value
+                )
+            );
+
+            return array_map(
+                function ( $reference ) {
+                    /**
+                     * @var $reference \Wordlift_Property_Entity_Reference
+                     */
+                    return array( '@id' => $reference->get_url() );
+                },
+                $value
+            );
+
+        }
+
+        return $value;
+    }
 
 }

Spacing +34 added lines, -34 removed lines

@@ -28,7 +28,7 @@ 
 	 */
 	private $entity_service;
 
-	public function __construct( $entity_type_service, $property_getter ) {
+	public function __construct($entity_type_service, $property_getter) {
 		$this->entity_type_service      = $entity_type_service;
 		$this->property_getter          = $property_getter;
 		$this->term_entity_type_service = Type_Service::get_instance();
@@ -36,26 +36,26 @@ 
 	}
 
 	public function init() {
-		add_filter( 'wl_term_jsonld_array', array( $this, 'wl_term_jsonld_array' ), 10, 2 );
+		add_filter('wl_term_jsonld_array', array($this, 'wl_term_jsonld_array'), 10, 2);
 	}
 
-	public function wl_term_jsonld_array( $data, $term_id ) {
+	public function wl_term_jsonld_array($data, $term_id) {
 		$jsonld     = $data['jsonld'];
 		$references = $data['references'];
 
-		$term_jsonld_data = $this->get_jsonld_data_for_term( $term_id );
+		$term_jsonld_data = $this->get_jsonld_data_for_term($term_id);
 
 		// Return early if we dont have the entity data
 		// for the term.
-		if ( ! $term_jsonld_data ) {
+		if ( ! $term_jsonld_data) {
 			return $data;
 		}
 
 		$term_jsonld = $term_jsonld_data['jsonld'];
 
-		$references = array_merge( $references, $term_jsonld_data['references'] );
+		$references = array_merge($references, $term_jsonld_data['references']);
 
-		array_unshift( $jsonld, $term_jsonld );
+		array_unshift($jsonld, $term_jsonld);
 
 		return array(
 			'jsonld'     => $jsonld,
@@ -63,48 +63,48 @@ 
 		);
 	}
 
-	private function get_jsonld_data_for_term( $term_id ) {
+	private function get_jsonld_data_for_term($term_id) {
 
-		$id = $this->entity_service->get_uri( $term_id, Object_Type_Enum::TERM );
+		$id = $this->entity_service->get_uri($term_id, Object_Type_Enum::TERM);
 
 		// If we don't have a dataset  URI, then don't publish the term data
 		// on this page.
-		if ( ! $id ) {
+		if ( ! $id) {
 			return false;
 		}
 
 		$references = array();
-		$term       = get_term( $term_id );
-		$permalink  = get_term_link( $term );
+		$term       = get_term($term_id);
+		$permalink  = get_term_link($term);
 
-		$custom_fields = $this->entity_type_service->get_custom_fields_for_term( $term_id );
-		$term          = get_term( $term_id );
+		$custom_fields = $this->entity_type_service->get_custom_fields_for_term($term_id);
+		$term          = get_term($term_id);
 		$jsonld        = array(
 			'@context'    => 'http://schema.org',
 			'name'        => $term->name,
-			'@type'       => $this->term_entity_type_service->get_entity_types_labels( $term_id ),
+			'@type'       => $this->term_entity_type_service->get_entity_types_labels($term_id),
 			'@id'         => $id,
 			'description' => $term->description,
 		);
 
-		if ( ! $custom_fields || ! is_array( $custom_fields ) ) {
+		if ( ! $custom_fields || ! is_array($custom_fields)) {
 			return $jsonld;
 		}
 
-		foreach ( $custom_fields as $key => $value ) {
-			$name  = $this->relative_to_schema_context( $value['predicate'] );
-			$value = $this->property_getter->get( $term_id, $key, Object_Type_Enum::TERM );
-			$value = $this->process_value( $value, $references );
-			if ( ! isset( $value ) ||
-				 is_array( $value ) && empty( $value ) ||
-				 is_string( $value ) && empty( $value ) ) {
+		foreach ($custom_fields as $key => $value) {
+			$name  = $this->relative_to_schema_context($value['predicate']);
+			$value = $this->property_getter->get($term_id, $key, Object_Type_Enum::TERM);
+			$value = $this->process_value($value, $references);
+			if ( ! isset($value) ||
+				 is_array($value) && empty($value) ||
+				 is_string($value) && empty($value)) {
 				continue;
 			}
-			$jsonld[ $name ] = $value;
+			$jsonld[$name] = $value;
 
 		}
 
-		if ( $permalink ) {
+		if ($permalink) {
 			$jsonld['mainEntityOfPage'] = $permalink;
 		}
 
@@ -119,21 +119,21 @@ 
 
 	}
 
-	private function relative_to_schema_context( $predicate ) {
-		return str_replace( 'http://schema.org/', '', $predicate );
+	private function relative_to_schema_context($predicate) {
+		return str_replace('http://schema.org/', '', $predicate);
 	}
 
-	private function process_value( $value, &$references ) {
+	private function process_value($value, &$references) {
 
-		if ( is_array( $value )
-			 && count( $value ) > 0
-			 && $value[0] instanceof \Wordlift_Property_Entity_Reference ) {
+		if (is_array($value)
+			 && count($value) > 0
+			 && $value[0] instanceof \Wordlift_Property_Entity_Reference) {
 
 			// All of the references from the custom fields are post references.
 			$references = array_merge(
 				$references,
 				array_map(
-					function ( $property_entity_reference ) {
+					function($property_entity_reference) {
 						/**
 						 * @var $property_entity_reference \Wordlift_Property_Entity_Reference
 						 */
@@ -144,11 +144,11 @@ 
 			);
 
 			return array_map(
-				function ( $reference ) {
+				function($reference) {
 					/**
 					 * @var $reference \Wordlift_Property_Entity_Reference
 					 */
-					return array( '@id' => $reference->get_url() );
+					return array('@id' => $reference->get_url());
 				},
 				$value
 			);

src/modules/core/wordlift-core-entity-api.php

Indentation +112 added lines, -112 removed lines

@@ -19,81 +19,81 @@ 
  */
 function wl_entity_get_by_title( $title, $autocomplete = false, $include_alias = true, $limit = false, $schema_types = array() ) {
 
-	global $wpdb;
-
-	$schema_type_query = '';
-
-	if ( $schema_types ) {
-		$schema_type_query = ' AND t.name IN (' . join(
-			',',
-			array_map(
-				function ( $schema_type ) {
-					return "'" . esc_sql( $schema_type ) . "'";
-				},
-				$schema_types
-			)
-		) . ')';
-	}
-
-	// Search by substring
-	if ( $autocomplete ) {
-		$title = '%' . $title . '%';
-	}
-
-	// The title is a LIKE query.
-	$query = 'SELECT DISTINCT p.ID AS id, p.post_title AS title, t.name AS schema_type_name, t.slug AS type_slug'
-			 . " FROM $wpdb->posts p, $wpdb->term_taxonomy tt, $wpdb->term_relationships tr, $wpdb->terms t"
-			 . '  WHERE p.post_title LIKE %s'
-			 . '   AND t.term_id = tt.term_id'
-			 . '   AND tt.taxonomy = %s'
-			 . '   AND tt.term_taxonomy_id = tr.term_taxonomy_id'
-			 . '   AND tr.object_id = p.ID'
-			 // Ensure we don't load entities from the trash, see https://github.com/insideout10/wordlift-plugin/issues/278.
-			 . "   AND p.post_status != 'trash'"
-			 . $schema_type_query;
-
-	$params = array(
-		$title,
-		Wordlift_Entity_Type_Taxonomy_Service::TAXONOMY_NAME,
-	);
-
-	if ( $include_alias ) {
-
-		$query .= ' UNION'
-				  . "  SELECT DISTINCT p.ID AS id, CONCAT( m.meta_value, ' (', p.post_title, ')' ) AS title, t.name AS schema_type_name, t.slug AS type_slug"
-				  . "  FROM $wpdb->posts p, $wpdb->term_taxonomy tt, $wpdb->term_relationships tr, $wpdb->terms t, $wpdb->postmeta m"
-				  . '   WHERE m.meta_key = %s AND m.meta_value LIKE %s'
-				  . '    AND m.post_id = p.ID'
-				  . '    AND t.term_id = tt.term_id'
-				  . '    AND tt.taxonomy = %s'
-				  . '    AND tt.term_taxonomy_id = tr.term_taxonomy_id'
-				  . '    AND tr.object_id = p.ID'
-				  // Ensure we don't load entities from the trash, see https://github.com/insideout10/wordlift-plugin/issues/278.
-				  . "    AND p.post_status != 'trash'"
-				  . $schema_type_query;
-
-		$params = array_merge(
-			$params,
-			array(
-				Wordlift_Entity_Service::ALTERNATIVE_LABEL_META_KEY,
-				$title,
-				Wordlift_Entity_Type_Taxonomy_Service::TAXONOMY_NAME,
-			)
-		);
-
-	}
-
-	if ( $limit ) {
-		$query   .= ' LIMIT %d';
-		$params[] = $limit;
-	}
-
-	return $wpdb->get_results(
-		$wpdb->prepare(
-			$query, // phpcs:ignore WordPress.DB.PreparedSQL.NotPrepared
-			$params
-		)
-	);
+    global $wpdb;
+
+    $schema_type_query = '';
+
+    if ( $schema_types ) {
+        $schema_type_query = ' AND t.name IN (' . join(
+            ',',
+            array_map(
+                function ( $schema_type ) {
+                    return "'" . esc_sql( $schema_type ) . "'";
+                },
+                $schema_types
+            )
+        ) . ')';
+    }
+
+    // Search by substring
+    if ( $autocomplete ) {
+        $title = '%' . $title . '%';
+    }
+
+    // The title is a LIKE query.
+    $query = 'SELECT DISTINCT p.ID AS id, p.post_title AS title, t.name AS schema_type_name, t.slug AS type_slug'
+                . " FROM $wpdb->posts p, $wpdb->term_taxonomy tt, $wpdb->term_relationships tr, $wpdb->terms t"
+                . '  WHERE p.post_title LIKE %s'
+                . '   AND t.term_id = tt.term_id'
+                . '   AND tt.taxonomy = %s'
+                . '   AND tt.term_taxonomy_id = tr.term_taxonomy_id'
+                . '   AND tr.object_id = p.ID'
+                // Ensure we don't load entities from the trash, see https://github.com/insideout10/wordlift-plugin/issues/278.
+                . "   AND p.post_status != 'trash'"
+                . $schema_type_query;
+
+    $params = array(
+        $title,
+        Wordlift_Entity_Type_Taxonomy_Service::TAXONOMY_NAME,
+    );
+
+    if ( $include_alias ) {
+
+        $query .= ' UNION'
+                    . "  SELECT DISTINCT p.ID AS id, CONCAT( m.meta_value, ' (', p.post_title, ')' ) AS title, t.name AS schema_type_name, t.slug AS type_slug"
+                    . "  FROM $wpdb->posts p, $wpdb->term_taxonomy tt, $wpdb->term_relationships tr, $wpdb->terms t, $wpdb->postmeta m"
+                    . '   WHERE m.meta_key = %s AND m.meta_value LIKE %s'
+                    . '    AND m.post_id = p.ID'
+                    . '    AND t.term_id = tt.term_id'
+                    . '    AND tt.taxonomy = %s'
+                    . '    AND tt.term_taxonomy_id = tr.term_taxonomy_id'
+                    . '    AND tr.object_id = p.ID'
+                    // Ensure we don't load entities from the trash, see https://github.com/insideout10/wordlift-plugin/issues/278.
+                    . "    AND p.post_status != 'trash'"
+                    . $schema_type_query;
+
+        $params = array_merge(
+            $params,
+            array(
+                Wordlift_Entity_Service::ALTERNATIVE_LABEL_META_KEY,
+                $title,
+                Wordlift_Entity_Type_Taxonomy_Service::TAXONOMY_NAME,
+            )
+        );
+
+    }
+
+    if ( $limit ) {
+        $query   .= ' LIMIT %d';
+        $params[] = $limit;
+    }
+
+    return $wpdb->get_results(
+        $wpdb->prepare(
+            $query, // phpcs:ignore WordPress.DB.PreparedSQL.NotPrepared
+            $params
+        )
+    );
 }
 
 /**
@@ -103,43 +103,43 @@ 
  */
 function wl_entity_ajax_get_by_title() {
 
-	// `wl_entity_metaboxes_utilities.js` still uses `GET`.
-	//
-	// See https://github.com/insideout10/wordlift-plugin/issues/438.
-	// Get the title to search.
-	if ( empty( $_POST['title'] ) && empty( $_GET['title'] ) ) {  //phpcs:ignore WordPress.Security.NonceVerification.Recommended,WordPress.Security.NonceVerification.Missing
-		// phpcs:ignore WordPress.PHP.NoSilencedErrors.Discouraged
-		@ob_clean();
-		wp_send_json_error( 'The title parameter is required.' );
-	}
-
-	// `wl_entity_metaboxes_utilities.js` still uses `GET`.
-	//
-	// See https://github.com/insideout10/wordlift-plugin/issues/438.
-	$title = sanitize_text_field( wp_unslash( $_POST['title'] ? $_POST['title'] : $_GET['title'] ) ); //phpcs:ignore WordPress.Security.NonceVerification.Recommended,WordPress.Security.NonceVerification.Missing
-
-	// Are we searching for a specific title or for a containing title?
-	$autocomplete = isset( $_GET['autocomplete'] ); //phpcs:ignore WordPress.Security.NonceVerification.Recommended
-
-	// Are we searching also for the aliases?
-	$include_alias = isset( $_GET['alias'] ); //phpcs:ignore WordPress.Security.NonceVerification.Recommended
-
-	// Get the edit link.
-	$post_type_object = get_post_type_object( Wordlift_Entity_Service::TYPE_NAME );
-	$edit_link        = $post_type_object->_edit_link . '&action=edit';
-
-	// Prepare the response with the edit link.
-	$response = array(
-		'edit_link' => $edit_link,
-		'results'   => wl_entity_get_by_title( $title, $autocomplete, $include_alias ),
-	);
-
-	// Clean any buffer.
-	// phpcs:ignore WordPress.PHP.NoSilencedErrors.Discouraged
-	@ob_clean();
-
-	// Send the success response.
-	wp_send_json_success( $response );
+    // `wl_entity_metaboxes_utilities.js` still uses `GET`.
+    //
+    // See https://github.com/insideout10/wordlift-plugin/issues/438.
+    // Get the title to search.
+    if ( empty( $_POST['title'] ) && empty( $_GET['title'] ) ) {  //phpcs:ignore WordPress.Security.NonceVerification.Recommended,WordPress.Security.NonceVerification.Missing
+        // phpcs:ignore WordPress.PHP.NoSilencedErrors.Discouraged
+        @ob_clean();
+        wp_send_json_error( 'The title parameter is required.' );
+    }
+
+    // `wl_entity_metaboxes_utilities.js` still uses `GET`.
+    //
+    // See https://github.com/insideout10/wordlift-plugin/issues/438.
+    $title = sanitize_text_field( wp_unslash( $_POST['title'] ? $_POST['title'] : $_GET['title'] ) ); //phpcs:ignore WordPress.Security.NonceVerification.Recommended,WordPress.Security.NonceVerification.Missing
+
+    // Are we searching for a specific title or for a containing title?
+    $autocomplete = isset( $_GET['autocomplete'] ); //phpcs:ignore WordPress.Security.NonceVerification.Recommended
+
+    // Are we searching also for the aliases?
+    $include_alias = isset( $_GET['alias'] ); //phpcs:ignore WordPress.Security.NonceVerification.Recommended
+
+    // Get the edit link.
+    $post_type_object = get_post_type_object( Wordlift_Entity_Service::TYPE_NAME );
+    $edit_link        = $post_type_object->_edit_link . '&action=edit';
+
+    // Prepare the response with the edit link.
+    $response = array(
+        'edit_link' => $edit_link,
+        'results'   => wl_entity_get_by_title( $title, $autocomplete, $include_alias ),
+    );
+
+    // Clean any buffer.
+    // phpcs:ignore WordPress.PHP.NoSilencedErrors.Discouraged
+    @ob_clean();
+
+    // Send the success response.
+    wp_send_json_success( $response );
 
 }
 

Spacing +20 added lines, -20 removed lines

@@ -17,27 +17,27 @@ 
  * @return array An array of WP_Post instances.
  * @since 3.0.0
  */
-function wl_entity_get_by_title( $title, $autocomplete = false, $include_alias = true, $limit = false, $schema_types = array() ) {
+function wl_entity_get_by_title($title, $autocomplete = false, $include_alias = true, $limit = false, $schema_types = array()) {
 
 	global $wpdb;
 
 	$schema_type_query = '';
 
-	if ( $schema_types ) {
-		$schema_type_query = ' AND t.name IN (' . join(
+	if ($schema_types) {
+		$schema_type_query = ' AND t.name IN ('.join(
 			',',
 			array_map(
-				function ( $schema_type ) {
-					return "'" . esc_sql( $schema_type ) . "'";
+				function($schema_type) {
+					return "'".esc_sql($schema_type)."'";
 				},
 				$schema_types
 			)
-		) . ')';
+		).')';
 	}
 
 	// Search by substring
-	if ( $autocomplete ) {
-		$title = '%' . $title . '%';
+	if ($autocomplete) {
+		$title = '%'.$title.'%';
 	}
 
 	// The title is a LIKE query.
@@ -57,7 +57,7 @@ 
 		Wordlift_Entity_Type_Taxonomy_Service::TAXONOMY_NAME,
 	);
 
-	if ( $include_alias ) {
+	if ($include_alias) {
 
 		$query .= ' UNION'
 				  . "  SELECT DISTINCT p.ID AS id, CONCAT( m.meta_value, ' (', p.post_title, ')' ) AS title, t.name AS schema_type_name, t.slug AS type_slug"
@@ -83,7 +83,7 @@ 
 
 	}
 
-	if ( $limit ) {
+	if ($limit) {
 		$query   .= ' LIMIT %d';
 		$params[] = $limit;
 	}
@@ -107,31 +107,31 @@ 
 	//
 	// See https://github.com/insideout10/wordlift-plugin/issues/438.
 	// Get the title to search.
-	if ( empty( $_POST['title'] ) && empty( $_GET['title'] ) ) {  //phpcs:ignore WordPress.Security.NonceVerification.Recommended,WordPress.Security.NonceVerification.Missing
+	if (empty($_POST['title']) && empty($_GET['title'])) {  //phpcs:ignore WordPress.Security.NonceVerification.Recommended,WordPress.Security.NonceVerification.Missing
 		// phpcs:ignore WordPress.PHP.NoSilencedErrors.Discouraged
 		@ob_clean();
-		wp_send_json_error( 'The title parameter is required.' );
+		wp_send_json_error('The title parameter is required.');
 	}
 
 	// `wl_entity_metaboxes_utilities.js` still uses `GET`.
 	//
 	// See https://github.com/insideout10/wordlift-plugin/issues/438.
-	$title = sanitize_text_field( wp_unslash( $_POST['title'] ? $_POST['title'] : $_GET['title'] ) ); //phpcs:ignore WordPress.Security.NonceVerification.Recommended,WordPress.Security.NonceVerification.Missing
+	$title = sanitize_text_field(wp_unslash($_POST['title'] ? $_POST['title'] : $_GET['title'])); //phpcs:ignore WordPress.Security.NonceVerification.Recommended,WordPress.Security.NonceVerification.Missing
 
 	// Are we searching for a specific title or for a containing title?
-	$autocomplete = isset( $_GET['autocomplete'] ); //phpcs:ignore WordPress.Security.NonceVerification.Recommended
+	$autocomplete = isset($_GET['autocomplete']); //phpcs:ignore WordPress.Security.NonceVerification.Recommended
 
 	// Are we searching also for the aliases?
-	$include_alias = isset( $_GET['alias'] ); //phpcs:ignore WordPress.Security.NonceVerification.Recommended
+	$include_alias = isset($_GET['alias']); //phpcs:ignore WordPress.Security.NonceVerification.Recommended
 
 	// Get the edit link.
-	$post_type_object = get_post_type_object( Wordlift_Entity_Service::TYPE_NAME );
-	$edit_link        = $post_type_object->_edit_link . '&action=edit';
+	$post_type_object = get_post_type_object(Wordlift_Entity_Service::TYPE_NAME);
+	$edit_link        = $post_type_object->_edit_link.'&action=edit';
 
 	// Prepare the response with the edit link.
 	$response = array(
 		'edit_link' => $edit_link,
-		'results'   => wl_entity_get_by_title( $title, $autocomplete, $include_alias ),
+		'results'   => wl_entity_get_by_title($title, $autocomplete, $include_alias),
 	);
 
 	// Clean any buffer.
@@ -139,8 +139,8 @@ 
 	@ob_clean();
 
 	// Send the success response.
-	wp_send_json_success( $response );
+	wp_send_json_success($response);
 
 }
 
-add_action( 'wp_ajax_entity_by_title', 'wl_entity_ajax_get_by_title' );
+add_action('wp_ajax_entity_by_title', 'wl_entity_ajax_get_by_title');

src/modules/common/load.php

Indentation +2 added lines, -2 removed lines

@@ -5,9 +5,9 @@
 
 // Autoloader for dependencies.
 if ( file_exists( __DIR__ . '/third-party/vendor/scoper-autoload.php' ) ) {
-	require __DIR__ . '/third-party/vendor/scoper-autoload.php';
+    require __DIR__ . '/third-party/vendor/scoper-autoload.php';
 }
 
 if ( file_exists( __DIR__ . '/vendor/autoload.php' ) ) {
-	require __DIR__ . '/vendor/autoload.php';
+    require __DIR__ . '/vendor/autoload.php';
 }

Spacing +4 added lines, -4 removed lines

@@ -4,10 +4,10 @@
  */
 
 // Autoloader for dependencies.
-if ( file_exists( __DIR__ . '/third-party/vendor/scoper-autoload.php' ) ) {
-	require __DIR__ . '/third-party/vendor/scoper-autoload.php';
+if (file_exists(__DIR__.'/third-party/vendor/scoper-autoload.php')) {
+	require __DIR__.'/third-party/vendor/scoper-autoload.php';
 }
 
-if ( file_exists( __DIR__ . '/vendor/autoload.php' ) ) {
-	require __DIR__ . '/vendor/autoload.php';
+if (file_exists(__DIR__.'/vendor/autoload.php')) {
+	require __DIR__.'/vendor/autoload.php';
 }

src/modules/pods/includes/Context.php

Indentation +55 added lines, -55 removed lines

@@ -4,60 +4,60 @@
 
 class Context {
 
-	const POST       = 0;
-	const TERM       = 1;
-	const ADMIN_AJAX = 2;
-	const UNKNOWN    = -1;
-	/**
-	 * @var $object_type int
-	 */
-	private $object_type;
-	/**
-	 * @var $identifier int
-	 */
-	private $identifier;
-	/**
-	 * @var $custom_fields Schema_Field_Group[]
-	 */
-	private $custom_fields;
-
-	private static $pod_types_map = array(
-		self::POST => 'post_type',
-		self::TERM => 'taxonomy',
-	);
-
-	/**
-	 * @param int           $object_type
-	 * @param $identifier
-	 * @param $custom_fields
-	 */
-	public function __construct( $object_type, $identifier, $custom_fields ) {
-		$this->object_type   = $object_type;
-		$this->identifier    = $identifier;
-		$this->custom_fields = $custom_fields;
-	}
-
-	/**
-	 * @return int
-	 */
-	public function get_object_type() {
-		return $this->object_type;
-	}
-
-	public function get_pod_name() {
-		if ( self::POST === $this->object_type ) {
-			return get_post_type( $this->identifier );
-		} elseif ( self::TERM === $this->object_type ) {
-			return get_term( $this->identifier )->taxonomy;
-		}
-	}
-
-	public function get_pod_type() {
-		return self::$pod_types_map[ $this->object_type ];
-	}
-
-	public function get_custom_fields() {
-		return $this->custom_fields;
-	}
+    const POST       = 0;
+    const TERM       = 1;
+    const ADMIN_AJAX = 2;
+    const UNKNOWN    = -1;
+    /**
+     * @var $object_type int
+     */
+    private $object_type;
+    /**
+     * @var $identifier int
+     */
+    private $identifier;
+    /**
+     * @var $custom_fields Schema_Field_Group[]
+     */
+    private $custom_fields;
+
+    private static $pod_types_map = array(
+        self::POST => 'post_type',
+        self::TERM => 'taxonomy',
+    );
+
+    /**
+     * @param int           $object_type
+     * @param $identifier
+     * @param $custom_fields
+     */
+    public function __construct( $object_type, $identifier, $custom_fields ) {
+        $this->object_type   = $object_type;
+        $this->identifier    = $identifier;
+        $this->custom_fields = $custom_fields;
+    }
+
+    /**
+     * @return int
+     */
+    public function get_object_type() {
+        return $this->object_type;
+    }
+
+    public function get_pod_name() {
+        if ( self::POST === $this->object_type ) {
+            return get_post_type( $this->identifier );
+        } elseif ( self::TERM === $this->object_type ) {
+            return get_term( $this->identifier )->taxonomy;
+        }
+    }
+
+    public function get_pod_type() {
+        return self::$pod_types_map[ $this->object_type ];
+    }
+
+    public function get_custom_fields() {
+        return $this->custom_fields;
+    }
 
 }

Spacing +6 added lines, -6 removed lines

@@ -31,7 +31,7 @@ 
 	 * @param $identifier
 	 * @param $custom_fields
 	 */
-	public function __construct( $object_type, $identifier, $custom_fields ) {
+	public function __construct($object_type, $identifier, $custom_fields) {
 		$this->object_type   = $object_type;
 		$this->identifier    = $identifier;
 		$this->custom_fields = $custom_fields;
@@ -45,15 +45,15 @@ 
 	}
 
 	public function get_pod_name() {
-		if ( self::POST === $this->object_type ) {
-			return get_post_type( $this->identifier );
-		} elseif ( self::TERM === $this->object_type ) {
-			return get_term( $this->identifier )->taxonomy;
+		if (self::POST === $this->object_type) {
+			return get_post_type($this->identifier);
+		} elseif (self::TERM === $this->object_type) {
+			return get_term($this->identifier)->taxonomy;
 		}
 	}
 
 	public function get_pod_type() {
-		return self::$pod_types_map[ $this->object_type ];
+		return self::$pod_types_map[$this->object_type];
 	}
 
 	public function get_custom_fields() {

src/modules/pods/includes/FieldDefinition/FieldDefinition.php

Indentation +6 added lines, -6 removed lines

@@ -4,12 +4,12 @@
 
 interface FieldDefinition {
 
-	/**
-	 * Register the fields for the provided context in constructor.
-	 *
-	 * @return void
-	 */
-	public function register();
+    /**
+     * Register the fields for the provided context in constructor.
+     *
+     * @return void
+     */
+    public function register();
 
 }
 

src/modules/pods/includes/FieldDefinition/AllPodsDefiniton.php

Indentation +22 added lines, -22 removed lines

@@ -10,27 +10,27 @@
  */
 class AllPodsDefiniton extends AbstractFieldDefiniton {
 
-	public function register() {
-
-		add_action( 'init', array( $this, 'register_on_all_valid_post_types' ) );
-		add_action( 'setup_theme', array( $this, 'register_on_all_supported_taxonomies' ) );
-	}
-
-	public function register_on_all_supported_taxonomies() {
-		$context    = $this->schema->get();
-		$taxonomies = Terms_Compat::get_public_taxonomies();
-		foreach ( $taxonomies as $taxonomy ) {
-			$this->register_pod( $taxonomy, 'taxonomy', $context );
-
-		}
-	}
-
-	public function register_on_all_valid_post_types() {
-		$context         = $this->schema->get();
-		$supported_types = \Wordlift_Entity_Service::valid_entity_post_types();
-		foreach ( $supported_types as $supported_type ) {
-			$this->register_pod( $supported_type, 'post_type', $context );
-		}
-	}
+    public function register() {
+
+        add_action( 'init', array( $this, 'register_on_all_valid_post_types' ) );
+        add_action( 'setup_theme', array( $this, 'register_on_all_supported_taxonomies' ) );
+    }
+
+    public function register_on_all_supported_taxonomies() {
+        $context    = $this->schema->get();
+        $taxonomies = Terms_Compat::get_public_taxonomies();
+        foreach ( $taxonomies as $taxonomy ) {
+            $this->register_pod( $taxonomy, 'taxonomy', $context );
+
+        }
+    }
+
+    public function register_on_all_valid_post_types() {
+        $context         = $this->schema->get();
+        $supported_types = \Wordlift_Entity_Service::valid_entity_post_types();
+        foreach ( $supported_types as $supported_type ) {
+            $this->register_pod( $supported_type, 'post_type', $context );
+        }
+    }
 
 }

Spacing +6 added lines, -6 removed lines

@@ -12,15 +12,15 @@ 
 
 	public function register() {
 
-		add_action( 'init', array( $this, 'register_on_all_valid_post_types' ) );
-		add_action( 'setup_theme', array( $this, 'register_on_all_supported_taxonomies' ) );
+		add_action('init', array($this, 'register_on_all_valid_post_types'));
+		add_action('setup_theme', array($this, 'register_on_all_supported_taxonomies'));
 	}
 
 	public function register_on_all_supported_taxonomies() {
 		$context    = $this->schema->get();
 		$taxonomies = Terms_Compat::get_public_taxonomies();
-		foreach ( $taxonomies as $taxonomy ) {
-			$this->register_pod( $taxonomy, 'taxonomy', $context );
+		foreach ($taxonomies as $taxonomy) {
+			$this->register_pod($taxonomy, 'taxonomy', $context);
 
 		}
 	}
@@ -28,8 +28,8 @@ 
 	public function register_on_all_valid_post_types() {
 		$context         = $this->schema->get();
 		$supported_types = \Wordlift_Entity_Service::valid_entity_post_types();
-		foreach ( $supported_types as $supported_type ) {
-			$this->register_pod( $supported_type, 'post_type', $context );
+		foreach ($supported_types as $supported_type) {
+			$this->register_pod($supported_type, 'post_type', $context);
 		}
 	}
 

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

Indentation +393 added lines, -393 removed lines

@@ -23,419 +23,419 @@
  */
 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->to_reference();
-
-							// 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->to_reference();
+
+                            // phpcs:ignore VariableAnalysis.CodeAnalysis.VariableAnalysis.UndefinedVariable
+                                $references_infos[] = array( 'reference' => $item );
+
+                                return array( '@id' => $url );
+                        }
+
+                        return $converter->relative_to_context( $item );
+                    },
+                    $value
+                )
+            );
+
+        }
+
+    }
 
 }

Spacing +59 added lines, -59 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,12 +423,12 @@ 
 							  $references[] = $item->to_reference();
 
 							// phpcs:ignore VariableAnalysis.CodeAnalysis.VariableAnalysis.UndefinedVariable
-							  $references_infos[] = array( 'reference' => $item );
+							  $references_infos[] = array('reference' => $item);
 
-							  return array( '@id' => $url );
+							  return array('@id' => $url);
 						}
 
-						return $converter->relative_to_context( $item );
+						return $converter->relative_to_context($item);
 					},
 					$value
 				)

src/modules/pods/includes/Schema.php

Indentation +94 added lines, -94 removed lines

@@ -4,99 +4,99 @@
 
 class Schema {
 
-	public function get_context_type() {
-
-		if ( isset( $_REQUEST['post'] ) || isset( $_REQUEST['post_ID'] ) ) { // phpcs:ignore WordPress.Security.NonceVerification.Recommended
-			return Context::POST;
-		}
-		if ( isset( $_REQUEST['tag_ID'] ) ) { // phpcs:ignore WordPress.Security.NonceVerification.Recommended
-			return Context::TERM;
-		}
-
-		if ( is_admin() && ! defined( 'DOING_AJAX' ) ) {
-			return Context::ADMIN_AJAX;
-		}
-
-		return Context::UNKNOWN;
-	}
-
-	/**
-	 * @return Context
-	 */
-	public function get() {
-		// we need to identify the context to filter the results.
-
-		$identifier = isset( $_REQUEST['post'] ) ? sanitize_text_field( wp_unslash( $_REQUEST['post'] ) ) // phpcs:ignore WordPress.Security.NonceVerification.Recommended
-			: sanitize_text_field( wp_unslash( isset( $_REQUEST['post_ID'] ) ? $_REQUEST['post_ID'] : '' ) ); // phpcs:ignore WordPress.Security.NonceVerification.Recommended
-
-		if ( $identifier ) {
-			// If post identifier, get schema.
-			return new Context( Context::POST, $identifier, $this->get_fields_for_post( $identifier ) );
-		}
-
-		$identifier = isset( $_REQUEST['tag_ID'] ) ? sanitize_text_field( wp_unslash( $_REQUEST['tag_ID'] ) ) : ''; // phpcs:ignore WordPress.Security.NonceVerification.Recommended
-
-		if ( $identifier ) {
-			// If post identifier, get schema.
-			return new Context( Context::TERM, $identifier, $this->get_fields_for_term( $identifier ) );
-		}
-
-		if ( is_admin() && defined( 'DOING_AJAX' ) ) {
-			return new Context( Context::ADMIN_AJAX, null, $this->get_all_fields() );
-		}
-
-		return new Context( Context::UNKNOWN, null, null );
-
-	}
-
-	/**
-	 * @return Schema_Field_Group[]
-	 */
-	private function get_fields_for_post( $identifier ) {
-		$types          = \Wordlift_Entity_Type_Service::get_instance()->get_names( $identifier );
-		$schema_classes = \Wordlift_Schema_Service::get_instance();
-
-		return array_map(
-			function ( $schema_type ) use ( $schema_classes ) {
-				$data = $schema_classes->get_schema( strtolower( $schema_type ) );
-
-				return new Schema_Field_Group( $schema_type, $data['custom_fields'] );
-			},
-			$types
-		);
-	}
-
-	/**
-	 * @return Schema_Field_Group[]
-	 */
-	private function get_fields_for_term( $identifier ) {
-		$term_entity_types = get_term_meta( (int) $identifier, \Wordlift_Entity_Type_Taxonomy_Service::TAXONOMY_NAME );
-		$schema_classes    = \Wordlift_Schema_Service::get_instance();
-
-		return array_map(
-			function ( $schema_type ) use ( $schema_classes ) {
-				$data = $schema_classes->get_schema( strtolower( $schema_type ) );
-
-				return new Schema_Field_Group( $schema_type, $data['custom_fields'] );
-			},
-			$term_entity_types
-		);
-	}
-
-	private function get_all_fields() {
-		$schema_classes   = \Wordlift_Schema_Service::get_instance();
-		$all_schema_slugs = $schema_classes->get_all_schema_slugs();
-
-		return array_filter(
-			array_map(
-				function ( $schema_type ) use ( $schema_classes ) {
-					$data = $schema_classes->get_schema( strtolower( $schema_type ) );
-
-					return new Schema_Field_Group( $schema_type, $data['custom_fields'] );
-				},
-				$all_schema_slugs
-			)
-		);
-	}
+    public function get_context_type() {
+
+        if ( isset( $_REQUEST['post'] ) || isset( $_REQUEST['post_ID'] ) ) { // phpcs:ignore WordPress.Security.NonceVerification.Recommended
+            return Context::POST;
+        }
+        if ( isset( $_REQUEST['tag_ID'] ) ) { // phpcs:ignore WordPress.Security.NonceVerification.Recommended
+            return Context::TERM;
+        }
+
+        if ( is_admin() && ! defined( 'DOING_AJAX' ) ) {
+            return Context::ADMIN_AJAX;
+        }
+
+        return Context::UNKNOWN;
+    }
+
+    /**
+     * @return Context
+     */
+    public function get() {
+        // we need to identify the context to filter the results.
+
+        $identifier = isset( $_REQUEST['post'] ) ? sanitize_text_field( wp_unslash( $_REQUEST['post'] ) ) // phpcs:ignore WordPress.Security.NonceVerification.Recommended
+            : sanitize_text_field( wp_unslash( isset( $_REQUEST['post_ID'] ) ? $_REQUEST['post_ID'] : '' ) ); // phpcs:ignore WordPress.Security.NonceVerification.Recommended
+
+        if ( $identifier ) {
+            // If post identifier, get schema.
+            return new Context( Context::POST, $identifier, $this->get_fields_for_post( $identifier ) );
+        }
+
+        $identifier = isset( $_REQUEST['tag_ID'] ) ? sanitize_text_field( wp_unslash( $_REQUEST['tag_ID'] ) ) : ''; // phpcs:ignore WordPress.Security.NonceVerification.Recommended
+
+        if ( $identifier ) {
+            // If post identifier, get schema.
+            return new Context( Context::TERM, $identifier, $this->get_fields_for_term( $identifier ) );
+        }
+
+        if ( is_admin() && defined( 'DOING_AJAX' ) ) {
+            return new Context( Context::ADMIN_AJAX, null, $this->get_all_fields() );
+        }
+
+        return new Context( Context::UNKNOWN, null, null );
+
+    }
+
+    /**
+     * @return Schema_Field_Group[]
+     */
+    private function get_fields_for_post( $identifier ) {
+        $types          = \Wordlift_Entity_Type_Service::get_instance()->get_names( $identifier );
+        $schema_classes = \Wordlift_Schema_Service::get_instance();
+
+        return array_map(
+            function ( $schema_type ) use ( $schema_classes ) {
+                $data = $schema_classes->get_schema( strtolower( $schema_type ) );
+
+                return new Schema_Field_Group( $schema_type, $data['custom_fields'] );
+            },
+            $types
+        );
+    }
+
+    /**
+     * @return Schema_Field_Group[]
+     */
+    private function get_fields_for_term( $identifier ) {
+        $term_entity_types = get_term_meta( (int) $identifier, \Wordlift_Entity_Type_Taxonomy_Service::TAXONOMY_NAME );
+        $schema_classes    = \Wordlift_Schema_Service::get_instance();
+
+        return array_map(
+            function ( $schema_type ) use ( $schema_classes ) {
+                $data = $schema_classes->get_schema( strtolower( $schema_type ) );
+
+                return new Schema_Field_Group( $schema_type, $data['custom_fields'] );
+            },
+            $term_entity_types
+        );
+    }
+
+    private function get_all_fields() {
+        $schema_classes   = \Wordlift_Schema_Service::get_instance();
+        $all_schema_slugs = $schema_classes->get_all_schema_slugs();
+
+        return array_filter(
+            array_map(
+                function ( $schema_type ) use ( $schema_classes ) {
+                    $data = $schema_classes->get_schema( strtolower( $schema_type ) );
+
+                    return new Schema_Field_Group( $schema_type, $data['custom_fields'] );
+                },
+                $all_schema_slugs
+            )
+        );
+    }
 
 }

Spacing +26 added lines, -26 removed lines

@@ -6,14 +6,14 @@ 
 
 	public function get_context_type() {
 
-		if ( isset( $_REQUEST['post'] ) || isset( $_REQUEST['post_ID'] ) ) { // phpcs:ignore WordPress.Security.NonceVerification.Recommended
+		if (isset($_REQUEST['post']) || isset($_REQUEST['post_ID'])) { // phpcs:ignore WordPress.Security.NonceVerification.Recommended
 			return Context::POST;
 		}
-		if ( isset( $_REQUEST['tag_ID'] ) ) { // phpcs:ignore WordPress.Security.NonceVerification.Recommended
+		if (isset($_REQUEST['tag_ID'])) { // phpcs:ignore WordPress.Security.NonceVerification.Recommended
 			return Context::TERM;
 		}
 
-		if ( is_admin() && ! defined( 'DOING_AJAX' ) ) {
+		if (is_admin() && ! defined('DOING_AJAX')) {
 			return Context::ADMIN_AJAX;
 		}
 
@@ -26,41 +26,41 @@ 
 	public function get() {
 		// we need to identify the context to filter the results.
 
-		$identifier = isset( $_REQUEST['post'] ) ? sanitize_text_field( wp_unslash( $_REQUEST['post'] ) ) // phpcs:ignore WordPress.Security.NonceVerification.Recommended
-			: sanitize_text_field( wp_unslash( isset( $_REQUEST['post_ID'] ) ? $_REQUEST['post_ID'] : '' ) ); // phpcs:ignore WordPress.Security.NonceVerification.Recommended
+		$identifier = isset($_REQUEST['post']) ? sanitize_text_field(wp_unslash($_REQUEST['post'])) // phpcs:ignore WordPress.Security.NonceVerification.Recommended
+			: sanitize_text_field(wp_unslash(isset($_REQUEST['post_ID']) ? $_REQUEST['post_ID'] : '')); // phpcs:ignore WordPress.Security.NonceVerification.Recommended
 
-		if ( $identifier ) {
+		if ($identifier) {
 			// If post identifier, get schema.
-			return new Context( Context::POST, $identifier, $this->get_fields_for_post( $identifier ) );
+			return new Context(Context::POST, $identifier, $this->get_fields_for_post($identifier));
 		}
 
-		$identifier = isset( $_REQUEST['tag_ID'] ) ? sanitize_text_field( wp_unslash( $_REQUEST['tag_ID'] ) ) : ''; // phpcs:ignore WordPress.Security.NonceVerification.Recommended
+		$identifier = isset($_REQUEST['tag_ID']) ? sanitize_text_field(wp_unslash($_REQUEST['tag_ID'])) : ''; // phpcs:ignore WordPress.Security.NonceVerification.Recommended
 
-		if ( $identifier ) {
+		if ($identifier) {
 			// If post identifier, get schema.
-			return new Context( Context::TERM, $identifier, $this->get_fields_for_term( $identifier ) );
+			return new Context(Context::TERM, $identifier, $this->get_fields_for_term($identifier));
 		}
 
-		if ( is_admin() && defined( 'DOING_AJAX' ) ) {
-			return new Context( Context::ADMIN_AJAX, null, $this->get_all_fields() );
+		if (is_admin() && defined('DOING_AJAX')) {
+			return new Context(Context::ADMIN_AJAX, null, $this->get_all_fields());
 		}
 
-		return new Context( Context::UNKNOWN, null, null );
+		return new Context(Context::UNKNOWN, null, null);
 
 	}
 
 	/**
 	 * @return Schema_Field_Group[]
 	 */
-	private function get_fields_for_post( $identifier ) {
-		$types          = \Wordlift_Entity_Type_Service::get_instance()->get_names( $identifier );
+	private function get_fields_for_post($identifier) {
+		$types          = \Wordlift_Entity_Type_Service::get_instance()->get_names($identifier);
 		$schema_classes = \Wordlift_Schema_Service::get_instance();
 
 		return array_map(
-			function ( $schema_type ) use ( $schema_classes ) {
-				$data = $schema_classes->get_schema( strtolower( $schema_type ) );
+			function($schema_type) use ($schema_classes) {
+				$data = $schema_classes->get_schema(strtolower($schema_type));
 
-				return new Schema_Field_Group( $schema_type, $data['custom_fields'] );
+				return new Schema_Field_Group($schema_type, $data['custom_fields']);
 			},
 			$types
 		);
@@ -69,15 +69,15 @@ 
 	/**
 	 * @return Schema_Field_Group[]
 	 */
-	private function get_fields_for_term( $identifier ) {
-		$term_entity_types = get_term_meta( (int) $identifier, \Wordlift_Entity_Type_Taxonomy_Service::TAXONOMY_NAME );
+	private function get_fields_for_term($identifier) {
+		$term_entity_types = get_term_meta((int) $identifier, \Wordlift_Entity_Type_Taxonomy_Service::TAXONOMY_NAME);
 		$schema_classes    = \Wordlift_Schema_Service::get_instance();
 
 		return array_map(
-			function ( $schema_type ) use ( $schema_classes ) {
-				$data = $schema_classes->get_schema( strtolower( $schema_type ) );
+			function($schema_type) use ($schema_classes) {
+				$data = $schema_classes->get_schema(strtolower($schema_type));
 
-				return new Schema_Field_Group( $schema_type, $data['custom_fields'] );
+				return new Schema_Field_Group($schema_type, $data['custom_fields']);
 			},
 			$term_entity_types
 		);
@@ -89,10 +89,10 @@ 
 
 		return array_filter(
 			array_map(
-				function ( $schema_type ) use ( $schema_classes ) {
-					$data = $schema_classes->get_schema( strtolower( $schema_type ) );
+				function($schema_type) use ($schema_classes) {
+					$data = $schema_classes->get_schema(strtolower($schema_type));
 
-					return new Schema_Field_Group( $schema_type, $data['custom_fields'] );
+					return new Schema_Field_Group($schema_type, $data['custom_fields']);
 				},
 				$all_schema_slugs
 			)

src/modules/pods/includes/WlEntityField/Filters.php

Indentation +189 added lines, -189 removed lines

@@ -7,196 +7,196 @@
 
 class Filters {
 
-	const FIELD_NAME = 'wlentity';
-
-	public function __construct() {
-
-		pods_register_related_object( self::FIELD_NAME, 'WordLift Entity', array( 'simple' => false ) );
-		add_filter( 'pods_form_ui_field_pick_ajax', array( $this, 'ajax_filter' ), 10, 4 );
-		add_filter( 'pods_api_get_table_info', array( $this, 'table_info_filter' ), 10, 6 );
-		add_filter( 'pods_field_pick_object_data', array( $this, 'field_options_filter' ), 10, 7 );
-		add_filter( 'pods_field_dfv_data', array( $this, 'data_filter' ), 10, 2 );
-		add_filter( 'pods_field_pick_data_ajax', array( $this, 'admin_ajax_filter' ), 10, 4 );
-
-		add_action(
-			'pods_meta_save_taxonomy',
-			function ( $data, $pod, $id, $groups, $term_id ) {
-				$this->save_field( 'term', $term_id, $groups );
-			},
-			10,
-			5
-		);
-
-		add_action(
-			'pods_meta_save_post',
-			function ( $data, $pod, $id, $groups ) {
-				$this->save_field( 'post', $id, $groups );
-			},
-			10,
-			4
-		);
-
-	}
-
-	private function save_field( $type, $identifier, $groups ) {
-
-		$entity_fields = $this->filter_entity_fields( $groups );
-
-		foreach ( $entity_fields as $entity_field ) {
-			delete_metadata( $type, $identifier, $entity_field );
-			$key = sprintf( 'pods_meta_%s', $entity_field );
-
-			$data = filter_var_array( $_REQUEST, array( $key => array( 'flags' => FILTER_REQUIRE_ARRAY ) ) ); // phpcs:ignore WordPress.Security.NonceVerification.Recommended
-
-			if ( ! $data ) {
-				continue;
-			}
-			$values = $data[ $key ];
-
-			foreach ( $values as $value ) {
-				add_metadata( $type, $identifier, $entity_field, $value );
-			}
-		}
-
-	}
-
-	private function filter_entity_fields( $groups ) {
-
-		$pods = json_decode( wp_json_encode( $groups ), true );
-
-		$fields = array_reduce(
-			$pods,
-			function ( $carry, $item ) {
-				return array_merge( $carry, $item['fields'] );
-			},
-			array()
-		);
-
-		return array_map(
-			function ( $item ) {
-				return $item['name'];
-			},
-			array_filter(
-				$fields,
-				function ( $item ) {
-					return is_array( $item ) && isset( $item['pick_object'] ) && self::FIELD_NAME === $item['pick_object'];
-				}
-			)
-		);
-	}
-
-	public function wl_pods_transform_data_for_pick_field( $item ) {
-
-		$content = $item->get_content();
-
-		return array(
-			'id'        => sprintf( '%s_%d', Object_Type_Enum::to_string( $content->get_object_type_enum() ), $content->get_id() ),
-			'icon'      => 'https:\/\/wordlift.localhost\/wp-content\/plugins\/wordlift\/images\/svg\/wl-vocabulary-icon.svg',
-			'name'      => $item->get_title() . ' (' . $item->get_schema_type() . ')',
-			'edit_link' => $content->get_edit_link(),
-			'link'      => $content->get_permalink(),
-			'selected'  => false,
-		);
-	}
-
-	public function admin_ajax_filter( $data, $name, $_, $field ) {
-
-		if ( ( ! $field instanceof \Pods\Whatsit\Field ) || $field->get_arg( 'pick_object', false ) !== self::FIELD_NAME ) {
-			return $data;
-		}
-
-		$query         = sanitize_text_field( wp_unslash( isset( $_REQUEST['query'] ) ? $_REQUEST['query'] : '' ) ); // phpcs:ignore WordPress.Security.NonceVerification.Recommended
-		$query_service = Entity_Query_Service::get_instance();
-
-		return array_map(
-			array(
-				$this,
-				'wl_pods_transform_data_for_pick_field',
-			),
-			$query_service->query( $query, $field->get_arg( 'supported_schema_types', array( 'Thing' ) ) )
-		);
-
-	}
-
-	public function data_filter( $data, $args ) {
-
-		$args_arr   = json_decode( wp_json_encode( $args ), true );
-		$field_data = $args_arr['options'];
-
-		if ( ! isset( $field_data['pick_object'] ) || self::FIELD_NAME !== $field_data['pick_object'] ) {
-			return $data;
-		}
-
-		if ( ! isset( $args_arr['pod']['data']['pod_data']['type'] )
-			 || ! is_string( $args_arr['pod']['data']['pod_data']['type'] ) ) {
-			return $data;
-		}
-
-		$name       = $field_data['name'];
-		$identifier = $args->id;
-		$type       = $args_arr['pod']['data']['pod_data']['type'];
+    const FIELD_NAME = 'wlentity';
+
+    public function __construct() {
+
+        pods_register_related_object( self::FIELD_NAME, 'WordLift Entity', array( 'simple' => false ) );
+        add_filter( 'pods_form_ui_field_pick_ajax', array( $this, 'ajax_filter' ), 10, 4 );
+        add_filter( 'pods_api_get_table_info', array( $this, 'table_info_filter' ), 10, 6 );
+        add_filter( 'pods_field_pick_object_data', array( $this, 'field_options_filter' ), 10, 7 );
+        add_filter( 'pods_field_dfv_data', array( $this, 'data_filter' ), 10, 2 );
+        add_filter( 'pods_field_pick_data_ajax', array( $this, 'admin_ajax_filter' ), 10, 4 );
+
+        add_action(
+            'pods_meta_save_taxonomy',
+            function ( $data, $pod, $id, $groups, $term_id ) {
+                $this->save_field( 'term', $term_id, $groups );
+            },
+            10,
+            5
+        );
+
+        add_action(
+            'pods_meta_save_post',
+            function ( $data, $pod, $id, $groups ) {
+                $this->save_field( 'post', $id, $groups );
+            },
+            10,
+            4
+        );
+
+    }
+
+    private function save_field( $type, $identifier, $groups ) {
+
+        $entity_fields = $this->filter_entity_fields( $groups );
+
+        foreach ( $entity_fields as $entity_field ) {
+            delete_metadata( $type, $identifier, $entity_field );
+            $key = sprintf( 'pods_meta_%s', $entity_field );
+
+            $data = filter_var_array( $_REQUEST, array( $key => array( 'flags' => FILTER_REQUIRE_ARRAY ) ) ); // phpcs:ignore WordPress.Security.NonceVerification.Recommended
+
+            if ( ! $data ) {
+                continue;
+            }
+            $values = $data[ $key ];
+
+            foreach ( $values as $value ) {
+                add_metadata( $type, $identifier, $entity_field, $value );
+            }
+        }
+
+    }
+
+    private function filter_entity_fields( $groups ) {
+
+        $pods = json_decode( wp_json_encode( $groups ), true );
+
+        $fields = array_reduce(
+            $pods,
+            function ( $carry, $item ) {
+                return array_merge( $carry, $item['fields'] );
+            },
+            array()
+        );
+
+        return array_map(
+            function ( $item ) {
+                return $item['name'];
+            },
+            array_filter(
+                $fields,
+                function ( $item ) {
+                    return is_array( $item ) && isset( $item['pick_object'] ) && self::FIELD_NAME === $item['pick_object'];
+                }
+            )
+        );
+    }
+
+    public function wl_pods_transform_data_for_pick_field( $item ) {
+
+        $content = $item->get_content();
+
+        return array(
+            'id'        => sprintf( '%s_%d', Object_Type_Enum::to_string( $content->get_object_type_enum() ), $content->get_id() ),
+            'icon'      => 'https:\/\/wordlift.localhost\/wp-content\/plugins\/wordlift\/images\/svg\/wl-vocabulary-icon.svg',
+            'name'      => $item->get_title() . ' (' . $item->get_schema_type() . ')',
+            'edit_link' => $content->get_edit_link(),
+            'link'      => $content->get_permalink(),
+            'selected'  => false,
+        );
+    }
+
+    public function admin_ajax_filter( $data, $name, $_, $field ) {
+
+        if ( ( ! $field instanceof \Pods\Whatsit\Field ) || $field->get_arg( 'pick_object', false ) !== self::FIELD_NAME ) {
+            return $data;
+        }
+
+        $query         = sanitize_text_field( wp_unslash( isset( $_REQUEST['query'] ) ? $_REQUEST['query'] : '' ) ); // phpcs:ignore WordPress.Security.NonceVerification.Recommended
+        $query_service = Entity_Query_Service::get_instance();
+
+        return array_map(
+            array(
+                $this,
+                'wl_pods_transform_data_for_pick_field',
+            ),
+            $query_service->query( $query, $field->get_arg( 'supported_schema_types', array( 'Thing' ) ) )
+        );
+
+    }
+
+    public function data_filter( $data, $args ) {
+
+        $args_arr   = json_decode( wp_json_encode( $args ), true );
+        $field_data = $args_arr['options'];
+
+        if ( ! isset( $field_data['pick_object'] ) || self::FIELD_NAME !== $field_data['pick_object'] ) {
+            return $data;
+        }
+
+        if ( ! isset( $args_arr['pod']['data']['pod_data']['type'] )
+             || ! is_string( $args_arr['pod']['data']['pod_data']['type'] ) ) {
+            return $data;
+        }
+
+        $name       = $field_data['name'];
+        $identifier = $args->id;
+        $type       = $args_arr['pod']['data']['pod_data']['type'];
 
-		if ( 'post_type' === $type ) {
-			$data['fieldValue'] = get_post_meta( $identifier, $name );
-		} elseif ( 'taxonomy' === $type ) {
-			$data['fieldValue'] = get_term_meta( $identifier, $name );
-		}
-
-		return $data;
-	}
-
-	public function ajax_filter( $result, $name, $value, $field_options ) {
-
-		if ( ! isset( $field_options['pick_object'] ) ) {
-			return $result;
-		}
-
-		return self::FIELD_NAME === $field_options['pick_object'];
-	}
-
-	public function table_info_filter( $info, $object_type, $object, $name, $pod, $field ) {
-
-		if ( $field === null || self::FIELD_NAME !== $field->get_arg( 'pick_object', false ) ) {
-			return $info;
-		}
-		// We need to return an non empty array here to prevent pods from querying a table.
-		// This is necessary to prevent errors on ui.
-		return array( 'foo' => 'bar' );
-	}
-
-	public function field_options_filter( $_, $name, $value, $options, $pod, $id, $object_params ) {
-
-		$object_params = json_decode( wp_json_encode( $object_params ), true );
-
-		$query_service = Entity_Query_Service::get_instance();
-
-		if ( is_array( $object_params ) && isset( $object_params['options']['pick_object'] )
-			 && is_string( $object_params['options']['pick_object'] )
-			 && self::FIELD_NAME === $object_params['options']['pick_object']
-			 && isset( $object_params['pod']['data']['pod_data']['type'] )
-			 && is_string( $object_params['pod']['data']['pod_data']['type'] ) ) {
-
-			$type            = $object_params['pod']['data']['pod_data']['type'];
-			$linked_entities = array();
-			if ( 'post_type' === $type ) {
-				$linked_entities = get_post_meta( $id, $name );
-			} elseif ( 'taxonomy' === $type ) {
-				$linked_entities = get_term_meta( $id, $name );
-			}
-			$data            = array();
-			$linked_entities = $query_service->get( $linked_entities );
-			foreach ( $linked_entities as $linked_entity ) {
-				$content     = $linked_entity->get_content();
-				$id          = sprintf( '%s_%d', Object_Type_Enum::to_string( $content->get_object_type_enum() ), $content->get_id() );
-				$text        = $linked_entity->get_title() . ' (' . $linked_entity->get_schema_type() . ')';
-				$data[ $id ] = $text;
-			}
-
-			return $data;
-		}
-
-		return $_;
+        if ( 'post_type' === $type ) {
+            $data['fieldValue'] = get_post_meta( $identifier, $name );
+        } elseif ( 'taxonomy' === $type ) {
+            $data['fieldValue'] = get_term_meta( $identifier, $name );
+        }
+
+        return $data;
+    }
+
+    public function ajax_filter( $result, $name, $value, $field_options ) {
+
+        if ( ! isset( $field_options['pick_object'] ) ) {
+            return $result;
+        }
+
+        return self::FIELD_NAME === $field_options['pick_object'];
+    }
+
+    public function table_info_filter( $info, $object_type, $object, $name, $pod, $field ) {
+
+        if ( $field === null || self::FIELD_NAME !== $field->get_arg( 'pick_object', false ) ) {
+            return $info;
+        }
+        // We need to return an non empty array here to prevent pods from querying a table.
+        // This is necessary to prevent errors on ui.
+        return array( 'foo' => 'bar' );
+    }
+
+    public function field_options_filter( $_, $name, $value, $options, $pod, $id, $object_params ) {
+
+        $object_params = json_decode( wp_json_encode( $object_params ), true );
+
+        $query_service = Entity_Query_Service::get_instance();
+
+        if ( is_array( $object_params ) && isset( $object_params['options']['pick_object'] )
+             && is_string( $object_params['options']['pick_object'] )
+             && self::FIELD_NAME === $object_params['options']['pick_object']
+             && isset( $object_params['pod']['data']['pod_data']['type'] )
+             && is_string( $object_params['pod']['data']['pod_data']['type'] ) ) {
+
+            $type            = $object_params['pod']['data']['pod_data']['type'];
+            $linked_entities = array();
+            if ( 'post_type' === $type ) {
+                $linked_entities = get_post_meta( $id, $name );
+            } elseif ( 'taxonomy' === $type ) {
+                $linked_entities = get_term_meta( $id, $name );
+            }
+            $data            = array();
+            $linked_entities = $query_service->get( $linked_entities );
+            foreach ( $linked_entities as $linked_entity ) {
+                $content     = $linked_entity->get_content();
+                $id          = sprintf( '%s_%d', Object_Type_Enum::to_string( $content->get_object_type_enum() ), $content->get_id() );
+                $text        = $linked_entity->get_title() . ' (' . $linked_entity->get_schema_type() . ')';
+                $data[ $id ] = $text;
+            }
+
+            return $data;
+        }
+
+        return $_;
 
-	}
+    }
 
 }

Spacing +63 added lines, -63 removed lines

@@ -11,17 +11,17 @@ 
 
 	public function __construct() {
 
-		pods_register_related_object( self::FIELD_NAME, 'WordLift Entity', array( 'simple' => false ) );
-		add_filter( 'pods_form_ui_field_pick_ajax', array( $this, 'ajax_filter' ), 10, 4 );
-		add_filter( 'pods_api_get_table_info', array( $this, 'table_info_filter' ), 10, 6 );
-		add_filter( 'pods_field_pick_object_data', array( $this, 'field_options_filter' ), 10, 7 );
-		add_filter( 'pods_field_dfv_data', array( $this, 'data_filter' ), 10, 2 );
-		add_filter( 'pods_field_pick_data_ajax', array( $this, 'admin_ajax_filter' ), 10, 4 );
+		pods_register_related_object(self::FIELD_NAME, 'WordLift Entity', array('simple' => false));
+		add_filter('pods_form_ui_field_pick_ajax', array($this, 'ajax_filter'), 10, 4);
+		add_filter('pods_api_get_table_info', array($this, 'table_info_filter'), 10, 6);
+		add_filter('pods_field_pick_object_data', array($this, 'field_options_filter'), 10, 7);
+		add_filter('pods_field_dfv_data', array($this, 'data_filter'), 10, 2);
+		add_filter('pods_field_pick_data_ajax', array($this, 'admin_ajax_filter'), 10, 4);
 
 		add_action(
 			'pods_meta_save_taxonomy',
-			function ( $data, $pod, $id, $groups, $term_id ) {
-				$this->save_field( 'term', $term_id, $groups );
+			function($data, $pod, $id, $groups, $term_id) {
+				$this->save_field('term', $term_id, $groups);
 			},
 			10,
 			5
@@ -29,8 +29,8 @@ 
 
 		add_action(
 			'pods_meta_save_post',
-			function ( $data, $pod, $id, $groups ) {
-				$this->save_field( 'post', $id, $groups );
+			function($data, $pod, $id, $groups) {
+				$this->save_field('post', $id, $groups);
 			},
 			10,
 			4
@@ -38,74 +38,74 @@ 
 
 	}
 
-	private function save_field( $type, $identifier, $groups ) {
+	private function save_field($type, $identifier, $groups) {
 
-		$entity_fields = $this->filter_entity_fields( $groups );
+		$entity_fields = $this->filter_entity_fields($groups);
 
-		foreach ( $entity_fields as $entity_field ) {
-			delete_metadata( $type, $identifier, $entity_field );
-			$key = sprintf( 'pods_meta_%s', $entity_field );
+		foreach ($entity_fields as $entity_field) {
+			delete_metadata($type, $identifier, $entity_field);
+			$key = sprintf('pods_meta_%s', $entity_field);
 
-			$data = filter_var_array( $_REQUEST, array( $key => array( 'flags' => FILTER_REQUIRE_ARRAY ) ) ); // phpcs:ignore WordPress.Security.NonceVerification.Recommended
+			$data = filter_var_array($_REQUEST, array($key => array('flags' => FILTER_REQUIRE_ARRAY))); // phpcs:ignore WordPress.Security.NonceVerification.Recommended
 
-			if ( ! $data ) {
+			if ( ! $data) {
 				continue;
 			}
-			$values = $data[ $key ];
+			$values = $data[$key];
 
-			foreach ( $values as $value ) {
-				add_metadata( $type, $identifier, $entity_field, $value );
+			foreach ($values as $value) {
+				add_metadata($type, $identifier, $entity_field, $value);
 			}
 		}
 
 	}
 
-	private function filter_entity_fields( $groups ) {
+	private function filter_entity_fields($groups) {
 
-		$pods = json_decode( wp_json_encode( $groups ), true );
+		$pods = json_decode(wp_json_encode($groups), true);
 
 		$fields = array_reduce(
 			$pods,
-			function ( $carry, $item ) {
-				return array_merge( $carry, $item['fields'] );
+			function($carry, $item) {
+				return array_merge($carry, $item['fields']);
 			},
 			array()
 		);
 
 		return array_map(
-			function ( $item ) {
+			function($item) {
 				return $item['name'];
 			},
 			array_filter(
 				$fields,
-				function ( $item ) {
-					return is_array( $item ) && isset( $item['pick_object'] ) && self::FIELD_NAME === $item['pick_object'];
+				function($item) {
+					return is_array($item) && isset($item['pick_object']) && self::FIELD_NAME === $item['pick_object'];
 				}
 			)
 		);
 	}
 
-	public function wl_pods_transform_data_for_pick_field( $item ) {
+	public function wl_pods_transform_data_for_pick_field($item) {
 
 		$content = $item->get_content();
 
 		return array(
-			'id'        => sprintf( '%s_%d', Object_Type_Enum::to_string( $content->get_object_type_enum() ), $content->get_id() ),
+			'id'        => sprintf('%s_%d', Object_Type_Enum::to_string($content->get_object_type_enum()), $content->get_id()),
 			'icon'      => 'https:\/\/wordlift.localhost\/wp-content\/plugins\/wordlift\/images\/svg\/wl-vocabulary-icon.svg',
-			'name'      => $item->get_title() . ' (' . $item->get_schema_type() . ')',
+			'name'      => $item->get_title().' ('.$item->get_schema_type().')',
 			'edit_link' => $content->get_edit_link(),
 			'link'      => $content->get_permalink(),
 			'selected'  => false,
 		);
 	}
 
-	public function admin_ajax_filter( $data, $name, $_, $field ) {
+	public function admin_ajax_filter($data, $name, $_, $field) {
 
-		if ( ( ! $field instanceof \Pods\Whatsit\Field ) || $field->get_arg( 'pick_object', false ) !== self::FIELD_NAME ) {
+		if (( ! $field instanceof \Pods\Whatsit\Field) || $field->get_arg('pick_object', false) !== self::FIELD_NAME) {
 			return $data;
 		}
 
-		$query         = sanitize_text_field( wp_unslash( isset( $_REQUEST['query'] ) ? $_REQUEST['query'] : '' ) ); // phpcs:ignore WordPress.Security.NonceVerification.Recommended
+		$query         = sanitize_text_field(wp_unslash(isset($_REQUEST['query']) ? $_REQUEST['query'] : '')); // phpcs:ignore WordPress.Security.NonceVerification.Recommended
 		$query_service = Entity_Query_Service::get_instance();
 
 		return array_map(
@@ -113,22 +113,22 @@ 
 				$this,
 				'wl_pods_transform_data_for_pick_field',
 			),
-			$query_service->query( $query, $field->get_arg( 'supported_schema_types', array( 'Thing' ) ) )
+			$query_service->query($query, $field->get_arg('supported_schema_types', array('Thing')))
 		);
 
 	}
 
-	public function data_filter( $data, $args ) {
+	public function data_filter($data, $args) {
 
-		$args_arr   = json_decode( wp_json_encode( $args ), true );
+		$args_arr   = json_decode(wp_json_encode($args), true);
 		$field_data = $args_arr['options'];
 
-		if ( ! isset( $field_data['pick_object'] ) || self::FIELD_NAME !== $field_data['pick_object'] ) {
+		if ( ! isset($field_data['pick_object']) || self::FIELD_NAME !== $field_data['pick_object']) {
 			return $data;
 		}
 
-		if ( ! isset( $args_arr['pod']['data']['pod_data']['type'] )
-			 || ! is_string( $args_arr['pod']['data']['pod_data']['type'] ) ) {
+		if ( ! isset($args_arr['pod']['data']['pod_data']['type'])
+			 || ! is_string($args_arr['pod']['data']['pod_data']['type'])) {
 			return $data;
 		}
 
@@ -136,60 +136,60 @@ 
 		$identifier = $args->id;
 		$type       = $args_arr['pod']['data']['pod_data']['type'];
 
-		if ( 'post_type' === $type ) {
-			$data['fieldValue'] = get_post_meta( $identifier, $name );
-		} elseif ( 'taxonomy' === $type ) {
-			$data['fieldValue'] = get_term_meta( $identifier, $name );
+		if ('post_type' === $type) {
+			$data['fieldValue'] = get_post_meta($identifier, $name);
+		} elseif ('taxonomy' === $type) {
+			$data['fieldValue'] = get_term_meta($identifier, $name);
 		}
 
 		return $data;
 	}
 
-	public function ajax_filter( $result, $name, $value, $field_options ) {
+	public function ajax_filter($result, $name, $value, $field_options) {
 
-		if ( ! isset( $field_options['pick_object'] ) ) {
+		if ( ! isset($field_options['pick_object'])) {
 			return $result;
 		}
 
 		return self::FIELD_NAME === $field_options['pick_object'];
 	}
 
-	public function table_info_filter( $info, $object_type, $object, $name, $pod, $field ) {
+	public function table_info_filter($info, $object_type, $object, $name, $pod, $field) {
 
-		if ( $field === null || self::FIELD_NAME !== $field->get_arg( 'pick_object', false ) ) {
+		if ($field === null || self::FIELD_NAME !== $field->get_arg('pick_object', false)) {
 			return $info;
 		}
 		// We need to return an non empty array here to prevent pods from querying a table.
 		// This is necessary to prevent errors on ui.
-		return array( 'foo' => 'bar' );
+		return array('foo' => 'bar');
 	}
 
-	public function field_options_filter( $_, $name, $value, $options, $pod, $id, $object_params ) {
+	public function field_options_filter($_, $name, $value, $options, $pod, $id, $object_params) {
 
-		$object_params = json_decode( wp_json_encode( $object_params ), true );
+		$object_params = json_decode(wp_json_encode($object_params), true);
 
 		$query_service = Entity_Query_Service::get_instance();
 
-		if ( is_array( $object_params ) && isset( $object_params['options']['pick_object'] )
-			 && is_string( $object_params['options']['pick_object'] )
+		if (is_array($object_params) && isset($object_params['options']['pick_object'])
+			 && is_string($object_params['options']['pick_object'])
 			 && self::FIELD_NAME === $object_params['options']['pick_object']
-			 && isset( $object_params['pod']['data']['pod_data']['type'] )
-			 && is_string( $object_params['pod']['data']['pod_data']['type'] ) ) {
+			 && isset($object_params['pod']['data']['pod_data']['type'])
+			 && is_string($object_params['pod']['data']['pod_data']['type'])) {
 
 			$type            = $object_params['pod']['data']['pod_data']['type'];
 			$linked_entities = array();
-			if ( 'post_type' === $type ) {
-				$linked_entities = get_post_meta( $id, $name );
-			} elseif ( 'taxonomy' === $type ) {
-				$linked_entities = get_term_meta( $id, $name );
+			if ('post_type' === $type) {
+				$linked_entities = get_post_meta($id, $name);
+			} elseif ('taxonomy' === $type) {
+				$linked_entities = get_term_meta($id, $name);
 			}
 			$data            = array();
-			$linked_entities = $query_service->get( $linked_entities );
-			foreach ( $linked_entities as $linked_entity ) {
+			$linked_entities = $query_service->get($linked_entities);
+			foreach ($linked_entities as $linked_entity) {
 				$content     = $linked_entity->get_content();
-				$id          = sprintf( '%s_%d', Object_Type_Enum::to_string( $content->get_object_type_enum() ), $content->get_id() );
-				$text        = $linked_entity->get_title() . ' (' . $linked_entity->get_schema_type() . ')';
-				$data[ $id ] = $text;
+				$id          = sprintf('%s_%d', Object_Type_Enum::to_string($content->get_object_type_enum()), $content->get_id());
+				$text        = $linked_entity->get_title().' ('.$linked_entity->get_schema_type().')';
+				$data[$id] = $text;
 			}
 
 			return $data;