insideout10 / wordlift-plugin

src/wordlift/videoobject/background-process/class-videos-data-source.php

Indentation +63 added lines, -63 removed lines

@@ -6,74 +6,74 @@
 
 class Videos_Data_Source extends Data_Source {
 
-	/**
-	 * @var \Wordlift_Log_Service
-	 */
-	private $log;
+    /**
+     * @var \Wordlift_Log_Service
+     */
+    private $log;
 
-	public function __construct( $state_storage_key ) {
-		parent::__construct( $state_storage_key );
-		$this->log = \Wordlift_Log_Service::get_logger( get_class() );
-	}
+    public function __construct( $state_storage_key ) {
+        parent::__construct( $state_storage_key );
+        $this->log = \Wordlift_Log_Service::get_logger( get_class() );
+    }
 
-	public function next() {
-		$this->log->debug( 'Received video data source index as ' . $this->get_state()->index );
-		$this->log->debug( 'Count set to ' . $this->get_batch_size() );
+    public function next() {
+        $this->log->debug( 'Received video data source index as ' . $this->get_state()->index );
+        $this->log->debug( 'Count set to ' . $this->get_batch_size() );
 
-		return get_posts(
-			array(
-				'fields'      => 'ids',
-				'post_status' => 'any',
-				'numberposts' => $this->get_batch_size(),
-				'offset'      => $this->get_state()->index,
-				/**
-				* @return array<string> An array of supported post types for the video import procedure.
-				* @since 3.31.4
-				* Filter name : wl_videoobject_procedure_post_types
-				*/
-			'post_type'       => array_unique(
-				apply_filters(
-					'wl_videoobject_procedure_post_types',
-					array(
-						'post',
-						'page',
-					)
-				)
-			),
-			)
-		);
-	}
+        return get_posts(
+            array(
+                'fields'      => 'ids',
+                'post_status' => 'any',
+                'numberposts' => $this->get_batch_size(),
+                'offset'      => $this->get_state()->index,
+                /**
+                 * @return array<string> An array of supported post types for the video import procedure.
+                 * @since 3.31.4
+                 * Filter name : wl_videoobject_procedure_post_types
+                 */
+            'post_type'       => array_unique(
+                apply_filters(
+                    'wl_videoobject_procedure_post_types',
+                    array(
+                        'post',
+                        'page',
+                    )
+                )
+            ),
+            )
+        );
+    }
 
-	public function count() {
-		return count(
-			get_posts(
-				array(
-					'fields'      => 'ids',
-					'numberposts' => - 1,
-					'post_status' => 'any',
-					/**
-					* @return array<string> An array of supported post types for the video import procedure.
-					* @since 3.31.4
-					* Filter name : wl_videoobject_procedure_post_types
-					*/
-					'post_type'   => array_unique(
-						apply_filters(
-							'wl_videoobject_procedure_post_types',
-							array(
-								'post',
-								'page',
-							)
-						)
-					),
-				)
-			)
-		);
+    public function count() {
+        return count(
+            get_posts(
+                array(
+                    'fields'      => 'ids',
+                    'numberposts' => - 1,
+                    'post_status' => 'any',
+                    /**
+                     * @return array<string> An array of supported post types for the video import procedure.
+                     * @since 3.31.4
+                     * Filter name : wl_videoobject_procedure_post_types
+                     */
+                    'post_type'   => array_unique(
+                        apply_filters(
+                            'wl_videoobject_procedure_post_types',
+                            array(
+                                'post',
+                                'page',
+                            )
+                        )
+                    ),
+                )
+            )
+        );
 
-	}
+    }
 
-	public function get_batch_size() {
-		// For now use only 5 in order to prevent exceeding api limit.
-		return 5;
-	}
+    public function get_batch_size() {
+        // For now use only 5 in order to prevent exceeding api limit.
+        return 5;
+    }
 
 }

Spacing +6 added lines, -6 removed lines

@@ -11,14 +11,14 @@ 
 	 */
 	private $log;
 
-	public function __construct( $state_storage_key ) {
-		parent::__construct( $state_storage_key );
-		$this->log = \Wordlift_Log_Service::get_logger( get_class() );
+	public function __construct($state_storage_key) {
+		parent::__construct($state_storage_key);
+		$this->log = \Wordlift_Log_Service::get_logger(get_class());
 	}
 
 	public function next() {
-		$this->log->debug( 'Received video data source index as ' . $this->get_state()->index );
-		$this->log->debug( 'Count set to ' . $this->get_batch_size() );
+		$this->log->debug('Received video data source index as '.$this->get_state()->index);
+		$this->log->debug('Count set to '.$this->get_batch_size());
 
 		return get_posts(
 			array(
@@ -49,7 +49,7 @@ 
 			get_posts(
 				array(
 					'fields'      => 'ids',
-					'numberposts' => - 1,
+					'numberposts' => -1,
 					'post_status' => 'any',
 					/**
 					* @return array<string> An array of supported post types for the video import procedure.

src/wordlift/videoobject/filters/class-post-filter.php

Indentation +21 added lines, -21 removed lines

@@ -10,32 +10,32 @@
  */
 class Post_Filter {
 
-	/**
-	 * @var Video_Processor
-	 */
-	private $video_processor;
+    /**
+     * @var Video_Processor
+     */
+    private $video_processor;
 
-	public function __construct( $video_processor ) {
-		$this->video_processor = $video_processor;
-	}
+    public function __construct( $video_processor ) {
+        $this->video_processor = $video_processor;
+    }
 
-	public function init() {
-		add_action( 'save_post', array( $this, 'save_post' ), 10, 2 );
-	}
+    public function init() {
+        add_action( 'save_post', array( $this, 'save_post' ), 10, 2 );
+    }
 
-	/**
-	 * @param $post_id int
-	 * @param $post \WP_Post
-	 * @param $update bool
-	 */
-	public function save_post( $post_id, $post ) {
+    /**
+     * @param $post_id int
+     * @param $post \WP_Post
+     * @param $update bool
+     */
+    public function save_post( $post_id, $post ) {
 
-		if ( wp_is_post_revision( $post_id ) || wp_is_post_autosave( $post_id ) ) {
-			return;
-		}
+        if ( wp_is_post_revision( $post_id ) || wp_is_post_autosave( $post_id ) ) {
+            return;
+        }
 
-		$this->video_processor->process_video_urls( $post, $post_id );
+        $this->video_processor->process_video_urls( $post, $post_id );
 
-	}
+    }
 
 }

Spacing +5 added lines, -5 removed lines

@@ -15,12 +15,12 @@ 
 	 */
 	private $video_processor;
 
-	public function __construct( $video_processor ) {
+	public function __construct($video_processor) {
 		$this->video_processor = $video_processor;
 	}
 
 	public function init() {
-		add_action( 'save_post', array( $this, 'save_post' ), 10, 2 );
+		add_action('save_post', array($this, 'save_post'), 10, 2);
 	}
 
 	/**
@@ -28,13 +28,13 @@ 
 	 * @param $post \WP_Post
 	 * @param $update bool
 	 */
-	public function save_post( $post_id, $post ) {
+	public function save_post($post_id, $post) {
 
-		if ( wp_is_post_revision( $post_id ) || wp_is_post_autosave( $post_id ) ) {
+		if (wp_is_post_revision($post_id) || wp_is_post_autosave($post_id)) {
 			return;
 		}
 
-		$this->video_processor->process_video_urls( $post, $post_id );
+		$this->video_processor->process_video_urls($post, $post_id);
 
 	}
 

src/wordlift/videoobject/filters/class-embed-shortcode-capture.php

Indentation +37 added lines, -37 removed lines

@@ -13,43 +13,43 @@
 
 class Embed_Shortcode_Capture {
 
-	public function init() {
-		add_filter( 'wl_videoobject_embedded_videos', array( $this, 'wl_videoobject_embedded_videos' ), 10, 2 );
-	}
-
-	public function wl_videoobject_embedded_videos( $embedded_videos, $post_id ) {
-
-		$post         = get_post( $post_id );
-		$post_content = $post->post_content;
-		$embed_regex  = get_shortcode_regex( array( 'embed' ) );
-
-		$matches = array();
-
-		preg_match_all( '/' . $embed_regex . '/', $post_content, $matches, PREG_SET_ORDER );
-
-		// The url is returned in index 5
-		$embed_shortcode_urls = array_filter(
-			array_map(
-				function ( $item ) {
-					if ( isset( $item[5] ) && is_string( $item[5] ) && $item[5] ) {
-						  return $item[5];
-					}
-					return false;
-				},
-				$matches
-			)
-		);
-
-		$embed_shortcode_videos = array_map(
-			function ( $url ) {
-				return new Default_Embedded_Video( $url );
-			},
-			$embed_shortcode_urls
-		);
-
-		return array_merge( $embedded_videos, $embed_shortcode_videos );
-
-	}
+    public function init() {
+        add_filter( 'wl_videoobject_embedded_videos', array( $this, 'wl_videoobject_embedded_videos' ), 10, 2 );
+    }
+
+    public function wl_videoobject_embedded_videos( $embedded_videos, $post_id ) {
+
+        $post         = get_post( $post_id );
+        $post_content = $post->post_content;
+        $embed_regex  = get_shortcode_regex( array( 'embed' ) );
+
+        $matches = array();
+
+        preg_match_all( '/' . $embed_regex . '/', $post_content, $matches, PREG_SET_ORDER );
+
+        // The url is returned in index 5
+        $embed_shortcode_urls = array_filter(
+            array_map(
+                function ( $item ) {
+                    if ( isset( $item[5] ) && is_string( $item[5] ) && $item[5] ) {
+                            return $item[5];
+                    }
+                    return false;
+                },
+                $matches
+            )
+        );
+
+        $embed_shortcode_videos = array_map(
+            function ( $url ) {
+                return new Default_Embedded_Video( $url );
+            },
+            $embed_shortcode_urls
+        );
+
+        return array_merge( $embedded_videos, $embed_shortcode_videos );
+
+    }
 
 }
 

Spacing +10 added lines, -10 removed lines

@@ -14,24 +14,24 @@ 
 class Embed_Shortcode_Capture {
 
 	public function init() {
-		add_filter( 'wl_videoobject_embedded_videos', array( $this, 'wl_videoobject_embedded_videos' ), 10, 2 );
+		add_filter('wl_videoobject_embedded_videos', array($this, 'wl_videoobject_embedded_videos'), 10, 2);
 	}
 
-	public function wl_videoobject_embedded_videos( $embedded_videos, $post_id ) {
+	public function wl_videoobject_embedded_videos($embedded_videos, $post_id) {
 
-		$post         = get_post( $post_id );
+		$post         = get_post($post_id);
 		$post_content = $post->post_content;
-		$embed_regex  = get_shortcode_regex( array( 'embed' ) );
+		$embed_regex  = get_shortcode_regex(array('embed'));
 
 		$matches = array();
 
-		preg_match_all( '/' . $embed_regex . '/', $post_content, $matches, PREG_SET_ORDER );
+		preg_match_all('/'.$embed_regex.'/', $post_content, $matches, PREG_SET_ORDER);
 
 		// The url is returned in index 5
 		$embed_shortcode_urls = array_filter(
 			array_map(
-				function ( $item ) {
-					if ( isset( $item[5] ) && is_string( $item[5] ) && $item[5] ) {
+				function($item) {
+					if (isset($item[5]) && is_string($item[5]) && $item[5]) {
 						  return $item[5];
 					}
 					return false;
@@ -41,13 +41,13 @@ 
 		);
 
 		$embed_shortcode_videos = array_map(
-			function ( $url ) {
-				return new Default_Embedded_Video( $url );
+			function($url) {
+				return new Default_Embedded_Video($url);
 			},
 			$embed_shortcode_urls
 		);
 
-		return array_merge( $embedded_videos, $embed_shortcode_videos );
+		return array_merge($embedded_videos, $embed_shortcode_videos);
 
 	}
 

src/wordlift/videoobject/filters/class-jw-player-capture.php

Indentation +28 added lines, -28 removed lines

@@ -13,33 +13,33 @@
 
 class Jw_Player_Capture {
 
-	public function init() {
-		add_filter( 'wl_videoobject_embedded_videos', array( $this, 'wl_videoobject_embedded_videos' ), 10, 2 );
-	}
-
-	public function wl_videoobject_embedded_videos( $embedded_videos, $post_id ) {
-		// we cant reliably determine count for external plugin without
-		// this method.
-		global $wpdb;
-		$video_ids = $wpdb->get_col(
-			$wpdb->prepare(
-				"SELECT meta_value FROM {$wpdb->postmeta} WHERE meta_key LIKE %s AND post_id=%d",
-				'_jwppp-video-url-%',
-				$post_id
-			)
-		);
-		if ( ! $video_ids ) {
-			return $embedded_videos;
-		}
-
-		$jw_player_videos = array_map(
-			function ( $video_id ) {
-				return new Default_Embedded_Video( 'https://cdn.jwplayer.com/v2/media/' . $video_id );
-			},
-			$video_ids
-		);
-
-		return array_merge( $embedded_videos, $jw_player_videos );
-	}
+    public function init() {
+        add_filter( 'wl_videoobject_embedded_videos', array( $this, 'wl_videoobject_embedded_videos' ), 10, 2 );
+    }
+
+    public function wl_videoobject_embedded_videos( $embedded_videos, $post_id ) {
+        // we cant reliably determine count for external plugin without
+        // this method.
+        global $wpdb;
+        $video_ids = $wpdb->get_col(
+            $wpdb->prepare(
+                "SELECT meta_value FROM {$wpdb->postmeta} WHERE meta_key LIKE %s AND post_id=%d",
+                '_jwppp-video-url-%',
+                $post_id
+            )
+        );
+        if ( ! $video_ids ) {
+            return $embedded_videos;
+        }
+
+        $jw_player_videos = array_map(
+            function ( $video_id ) {
+                return new Default_Embedded_Video( 'https://cdn.jwplayer.com/v2/media/' . $video_id );
+            },
+            $video_ids
+        );
+
+        return array_merge( $embedded_videos, $jw_player_videos );
+    }
 
 }

Spacing +6 added lines, -6 removed lines

@@ -14,10 +14,10 @@ 
 class Jw_Player_Capture {
 
 	public function init() {
-		add_filter( 'wl_videoobject_embedded_videos', array( $this, 'wl_videoobject_embedded_videos' ), 10, 2 );
+		add_filter('wl_videoobject_embedded_videos', array($this, 'wl_videoobject_embedded_videos'), 10, 2);
 	}
 
-	public function wl_videoobject_embedded_videos( $embedded_videos, $post_id ) {
+	public function wl_videoobject_embedded_videos($embedded_videos, $post_id) {
 		// we cant reliably determine count for external plugin without
 		// this method.
 		global $wpdb;
@@ -28,18 +28,18 @@ 
 				$post_id
 			)
 		);
-		if ( ! $video_ids ) {
+		if ( ! $video_ids) {
 			return $embedded_videos;
 		}
 
 		$jw_player_videos = array_map(
-			function ( $video_id ) {
-				return new Default_Embedded_Video( 'https://cdn.jwplayer.com/v2/media/' . $video_id );
+			function($video_id) {
+				return new Default_Embedded_Video('https://cdn.jwplayer.com/v2/media/'.$video_id);
 			},
 			$video_ids
 		);
 
-		return array_merge( $embedded_videos, $jw_player_videos );
+		return array_merge($embedded_videos, $jw_player_videos);
 	}
 
 }

src/wordlift/synonym/class-rest-field.php

Indentation +57 added lines, -57 removed lines

@@ -6,62 +6,62 @@
 
 class Rest_Field {
 
-	public function __construct() {
-		add_action( 'rest_api_init', array( $this, 'register_rest_field' ) );
-	}
-
-	public function register_rest_field() {
-
-		if ( ! function_exists( 'register_rest_field' ) ) {
-			return;
-		}
-
-		$post_types = Wordlift_Entity_Service::valid_entity_post_types();
-
-		foreach ( $post_types as $post_type ) {
-
-			register_rest_field(
-				$post_type,
-				\Wordlift_Entity_Service::ALTERNATIVE_LABEL_META_KEY,
-				array(
-					'get_callback'    => array( $this, 'get_value' ),
-					'update_callback' => array( $this, 'update_value' ),
-				)
-			);
-
-		}
-
-	}
-
-	/**
-	 * @param $meta_values array
-	 * @param $post \WP_Post
-	 * @param $meta_key string
-	 */
-	// phpcs:ignore VariableAnalysis.CodeAnalysis.VariableAnalysis.UnusedVariable
-	public function update_value( $meta_values, $post, $meta_key ) {
-
-		if ( ! is_array( $meta_values ) ) {
-			return;
-		}
-
-		$entity_service = Wordlift_Entity_Service::get_instance();
-
-		$entity_service->set_alternative_labels( $post->ID, $meta_values );
-	}
-
-	/**
-	 * @param $post array Post array.
-	 *
-	 * @return array|mixed
-	 */
-	public function get_value( $post ) {
-		$data = get_post_meta( (int) $post['id'], \Wordlift_Entity_Service::ALTERNATIVE_LABEL_META_KEY );
-		if ( ! is_array( $data ) ) {
-			return array();
-		}
-
-		return $data;
-	}
+    public function __construct() {
+        add_action( 'rest_api_init', array( $this, 'register_rest_field' ) );
+    }
+
+    public function register_rest_field() {
+
+        if ( ! function_exists( 'register_rest_field' ) ) {
+            return;
+        }
+
+        $post_types = Wordlift_Entity_Service::valid_entity_post_types();
+
+        foreach ( $post_types as $post_type ) {
+
+            register_rest_field(
+                $post_type,
+                \Wordlift_Entity_Service::ALTERNATIVE_LABEL_META_KEY,
+                array(
+                    'get_callback'    => array( $this, 'get_value' ),
+                    'update_callback' => array( $this, 'update_value' ),
+                )
+            );
+
+        }
+
+    }
+
+    /**
+     * @param $meta_values array
+     * @param $post \WP_Post
+     * @param $meta_key string
+     */
+    // phpcs:ignore VariableAnalysis.CodeAnalysis.VariableAnalysis.UnusedVariable
+    public function update_value( $meta_values, $post, $meta_key ) {
+
+        if ( ! is_array( $meta_values ) ) {
+            return;
+        }
+
+        $entity_service = Wordlift_Entity_Service::get_instance();
+
+        $entity_service->set_alternative_labels( $post->ID, $meta_values );
+    }
+
+    /**
+     * @param $post array Post array.
+     *
+     * @return array|mixed
+     */
+    public function get_value( $post ) {
+        $data = get_post_meta( (int) $post['id'], \Wordlift_Entity_Service::ALTERNATIVE_LABEL_META_KEY );
+        if ( ! is_array( $data ) ) {
+            return array();
+        }
+
+        return $data;
+    }
 
 }

Spacing +11 added lines, -11 removed lines

@@ -7,25 +7,25 @@ 
 class Rest_Field {
 
 	public function __construct() {
-		add_action( 'rest_api_init', array( $this, 'register_rest_field' ) );
+		add_action('rest_api_init', array($this, 'register_rest_field'));
 	}
 
 	public function register_rest_field() {
 
-		if ( ! function_exists( 'register_rest_field' ) ) {
+		if ( ! function_exists('register_rest_field')) {
 			return;
 		}
 
 		$post_types = Wordlift_Entity_Service::valid_entity_post_types();
 
-		foreach ( $post_types as $post_type ) {
+		foreach ($post_types as $post_type) {
 
 			register_rest_field(
 				$post_type,
 				\Wordlift_Entity_Service::ALTERNATIVE_LABEL_META_KEY,
 				array(
-					'get_callback'    => array( $this, 'get_value' ),
-					'update_callback' => array( $this, 'update_value' ),
+					'get_callback'    => array($this, 'get_value'),
+					'update_callback' => array($this, 'update_value'),
 				)
 			);
 
@@ -39,15 +39,15 @@ 
 	 * @param $meta_key string
 	 */
 	// phpcs:ignore VariableAnalysis.CodeAnalysis.VariableAnalysis.UnusedVariable
-	public function update_value( $meta_values, $post, $meta_key ) {
+	public function update_value($meta_values, $post, $meta_key) {
 
-		if ( ! is_array( $meta_values ) ) {
+		if ( ! is_array($meta_values)) {
 			return;
 		}
 
 		$entity_service = Wordlift_Entity_Service::get_instance();
 
-		$entity_service->set_alternative_labels( $post->ID, $meta_values );
+		$entity_service->set_alternative_labels($post->ID, $meta_values);
 	}
 
 	/**
@@ -55,9 +55,9 @@ 
 	 *
 	 * @return array|mixed
 	 */
-	public function get_value( $post ) {
-		$data = get_post_meta( (int) $post['id'], \Wordlift_Entity_Service::ALTERNATIVE_LABEL_META_KEY );
-		if ( ! is_array( $data ) ) {
+	public function get_value($post) {
+		$data = get_post_meta((int) $post['id'], \Wordlift_Entity_Service::ALTERNATIVE_LABEL_META_KEY);
+		if ( ! is_array($data)) {
 			return array();
 		}
 

src/wordlift/synonym/class-loader.php

Indentation +9 added lines, -9 removed lines

@@ -9,15 +9,15 @@
  */
 class Loader extends Default_Loader {
 
-	public function init_all_dependencies() {
-		new Rest_Field();
-	}
+    public function init_all_dependencies() {
+        new Rest_Field();
+    }
 
-	protected function get_feature_slug() {
-		return 'add-synonyms';
-	}
+    protected function get_feature_slug() {
+        return 'add-synonyms';
+    }
 
-	protected function get_feature_default_value() {
-		return true;
-	}
+    protected function get_feature_default_value() {
+        return true;
+    }
 }

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

Indentation +73 added lines, -73 removed lines

@@ -10,78 +10,78 @@
 
 class Jsonld_Service {
 
-	/**
-	 * @var Jsonld_Service
-	 */
-	private static $instance;
-
-	/**
-	 * @var Wordlift_Jsonld_Service
-	 */
-	private $legacy_jsonld_service;
-
-	/**
-	 * @var Wordlift_Term_JsonLd_Adapter
-	 */
-	private $term_jsonld_service;
-
-	/**
-	 * @var Jsonld_User_Service
-	 */
-	private $jsonld_user_service;
-
-	/**
-	 * Jsonld_Service constructor.
-	 *
-	 * @param Wordlift_Jsonld_Service      $legacy_jsonld_service
-	 * @param Wordlift_Term_JsonLd_Adapter $term_jsonld_adapter
-	 * @param Jsonld_User_Service          $jsonld_user_service
-	 *
-	 * @throws Exception when an error occurs.
-	 */
-	public function __construct( $legacy_jsonld_service, $term_jsonld_adapter, $jsonld_user_service ) {
-
-		Assertions::assert_of_type( $legacy_jsonld_service, 'Wordlift_Jsonld_Service' );
-		Assertions::assert_of_type( $term_jsonld_adapter, 'Wordlift_Term_JsonLd_Adapter' );
-		Assertions::assert_of_type( $jsonld_user_service, 'Wordlift\Jsonld\Jsonld_User_Service' );
-
-		$this->legacy_jsonld_service = $legacy_jsonld_service;
-		$this->term_jsonld_service   = $term_jsonld_adapter;
-		$this->jsonld_user_service   = $jsonld_user_service;
-		$this->jsonld_user_service->set_jsonld_service( $this );
-
-		self::$instance = $this;
-	}
-
-	public static function get_instance() {
-		return self::$instance;
-	}
-
-	/**
-	 * Get the JSON-LD structure for the specified type and id.
-	 *
-	 * @param int      $type The requested type, one of 'HOMEPAGE', 'POST' or 'TERM'. Default 'POST'.
-	 * @param int|null $id The id. Default `null`.
-	 * @param int      $context A context for the JSON-LD generation, valid values in Jsonld_Context_Enum.
-	 *
-	 * @return array The JSON-LD structure.
-	 * @throws Exception Throws an exception if the type isn't recognized.
-	 */
-	public function get( $type = Object_Type_Enum::POST, $id = null, $context = Jsonld_Context_Enum::UNKNOWN ) {
-
-		switch ( $type ) {
-			case Object_Type_Enum::HOMEPAGE:
-				return $this->legacy_jsonld_service->get_jsonld( true, $id, $context );
-			case Object_Type_Enum::POST:
-				return $this->legacy_jsonld_service->get_jsonld( false, $id, $context );
-			case Object_Type_Enum::TERM:
-				return $this->term_jsonld_service->get( $id, $context );
-			case Object_Type_Enum::USER:
-				return $this->jsonld_user_service->get( $id );
-			default:
-				throw new Exception( "Unknown type $type. Allowed types: 'HOMEPAGE', 'POST', 'TERM', 'USER'." );
-		}
-
-	}
+    /**
+     * @var Jsonld_Service
+     */
+    private static $instance;
+
+    /**
+     * @var Wordlift_Jsonld_Service
+     */
+    private $legacy_jsonld_service;
+
+    /**
+     * @var Wordlift_Term_JsonLd_Adapter
+     */
+    private $term_jsonld_service;
+
+    /**
+     * @var Jsonld_User_Service
+     */
+    private $jsonld_user_service;
+
+    /**
+     * Jsonld_Service constructor.
+     *
+     * @param Wordlift_Jsonld_Service      $legacy_jsonld_service
+     * @param Wordlift_Term_JsonLd_Adapter $term_jsonld_adapter
+     * @param Jsonld_User_Service          $jsonld_user_service
+     *
+     * @throws Exception when an error occurs.
+     */
+    public function __construct( $legacy_jsonld_service, $term_jsonld_adapter, $jsonld_user_service ) {
+
+        Assertions::assert_of_type( $legacy_jsonld_service, 'Wordlift_Jsonld_Service' );
+        Assertions::assert_of_type( $term_jsonld_adapter, 'Wordlift_Term_JsonLd_Adapter' );
+        Assertions::assert_of_type( $jsonld_user_service, 'Wordlift\Jsonld\Jsonld_User_Service' );
+
+        $this->legacy_jsonld_service = $legacy_jsonld_service;
+        $this->term_jsonld_service   = $term_jsonld_adapter;
+        $this->jsonld_user_service   = $jsonld_user_service;
+        $this->jsonld_user_service->set_jsonld_service( $this );
+
+        self::$instance = $this;
+    }
+
+    public static function get_instance() {
+        return self::$instance;
+    }
+
+    /**
+     * Get the JSON-LD structure for the specified type and id.
+     *
+     * @param int      $type The requested type, one of 'HOMEPAGE', 'POST' or 'TERM'. Default 'POST'.
+     * @param int|null $id The id. Default `null`.
+     * @param int      $context A context for the JSON-LD generation, valid values in Jsonld_Context_Enum.
+     *
+     * @return array The JSON-LD structure.
+     * @throws Exception Throws an exception if the type isn't recognized.
+     */
+    public function get( $type = Object_Type_Enum::POST, $id = null, $context = Jsonld_Context_Enum::UNKNOWN ) {
+
+        switch ( $type ) {
+            case Object_Type_Enum::HOMEPAGE:
+                return $this->legacy_jsonld_service->get_jsonld( true, $id, $context );
+            case Object_Type_Enum::POST:
+                return $this->legacy_jsonld_service->get_jsonld( false, $id, $context );
+            case Object_Type_Enum::TERM:
+                return $this->term_jsonld_service->get( $id, $context );
+            case Object_Type_Enum::USER:
+                return $this->jsonld_user_service->get( $id );
+            default:
+                throw new Exception( "Unknown type $type. Allowed types: 'HOMEPAGE', 'POST', 'TERM', 'USER'." );
+        }
+
+    }
 
 }

Spacing +12 added lines, -12 removed lines

@@ -39,16 +39,16 @@ 
 	 *
 	 * @throws Exception when an error occurs.
 	 */
-	public function __construct( $legacy_jsonld_service, $term_jsonld_adapter, $jsonld_user_service ) {
+	public function __construct($legacy_jsonld_service, $term_jsonld_adapter, $jsonld_user_service) {
 
-		Assertions::assert_of_type( $legacy_jsonld_service, 'Wordlift_Jsonld_Service' );
-		Assertions::assert_of_type( $term_jsonld_adapter, 'Wordlift_Term_JsonLd_Adapter' );
-		Assertions::assert_of_type( $jsonld_user_service, 'Wordlift\Jsonld\Jsonld_User_Service' );
+		Assertions::assert_of_type($legacy_jsonld_service, 'Wordlift_Jsonld_Service');
+		Assertions::assert_of_type($term_jsonld_adapter, 'Wordlift_Term_JsonLd_Adapter');
+		Assertions::assert_of_type($jsonld_user_service, 'Wordlift\Jsonld\Jsonld_User_Service');
 
 		$this->legacy_jsonld_service = $legacy_jsonld_service;
 		$this->term_jsonld_service   = $term_jsonld_adapter;
 		$this->jsonld_user_service   = $jsonld_user_service;
-		$this->jsonld_user_service->set_jsonld_service( $this );
+		$this->jsonld_user_service->set_jsonld_service($this);
 
 		self::$instance = $this;
 	}
@@ -67,19 +67,19 @@ 
 	 * @return array The JSON-LD structure.
 	 * @throws Exception Throws an exception if the type isn't recognized.
 	 */
-	public function get( $type = Object_Type_Enum::POST, $id = null, $context = Jsonld_Context_Enum::UNKNOWN ) {
+	public function get($type = Object_Type_Enum::POST, $id = null, $context = Jsonld_Context_Enum::UNKNOWN) {
 
-		switch ( $type ) {
+		switch ($type) {
 			case Object_Type_Enum::HOMEPAGE:
-				return $this->legacy_jsonld_service->get_jsonld( true, $id, $context );
+				return $this->legacy_jsonld_service->get_jsonld(true, $id, $context);
 			case Object_Type_Enum::POST:
-				return $this->legacy_jsonld_service->get_jsonld( false, $id, $context );
+				return $this->legacy_jsonld_service->get_jsonld(false, $id, $context);
 			case Object_Type_Enum::TERM:
-				return $this->term_jsonld_service->get( $id, $context );
+				return $this->term_jsonld_service->get($id, $context);
 			case Object_Type_Enum::USER:
-				return $this->jsonld_user_service->get( $id );
+				return $this->jsonld_user_service->get($id);
 			default:
-				throw new Exception( "Unknown type $type. Allowed types: 'HOMEPAGE', 'POST', 'TERM', 'USER'." );
+				throw new Exception("Unknown type $type. Allowed types: 'HOMEPAGE', 'POST', 'TERM', 'USER'.");
 		}
 
 	}

src/wordlift/jsonld/class-jsonld-article-wrapper.php

Indentation +128 added lines, -128 removed lines

@@ -6,142 +6,142 @@
 
 class Jsonld_Article_Wrapper {
 
-	public static $article_types = array(
-		'Article',
-		'AdvertiserContentArticle',
-		'NewsArticle',
-		'AnalysisNewsArticle',
-		'AskPublicNewsArticle',
-		'BackgroundNewsArticle',
-		'OpinionNewsArticle',
-		'ReportageNewsArticle',
-		'ReviewNewsArticle',
-		'Report',
-		'SatiricalArticle',
-		'ScholarlyArticle',
-		'MedicalScholarlyArticle',
-		'SocialMediaPosting',
-		'BlogPosting',
-		'LiveBlogPosting',
-		'DiscussionForumPosting',
-		'TechArticle',
-		'APIReference',
-	);
-
-	/**
-	 * @var Wordlift_Post_To_Jsonld_Converter
-	 */
-	private $post_to_jsonld_converter;
-	/**
-	 * @var \Wordlift_Cached_Post_Converter
-	 */
-	private $cached_postid_to_jsonld_converter;
-	/**
-	 * @var \Wordlift_Entity_Uri_Service
-	 */
-	private $entity_uri_service;
-
-	/**
-	 * Jsonld_Article_Wrapper constructor.
-	 *
-	 * @param Wordlift_Post_To_Jsonld_Converter $post_to_jsonld_converter
-	 * @param $cached_postid_to_jsonld_converter
-	 */
-	public function __construct( $post_to_jsonld_converter, $cached_postid_to_jsonld_converter ) {
-
-		$this->post_to_jsonld_converter = $post_to_jsonld_converter->new_instance_with_filters_disabled();
-
-		add_filter( 'wl_after_get_jsonld', array( $this, 'after_get_jsonld' ), PHP_INT_MAX - 100, 3 );
-
-		$this->cached_postid_to_jsonld_converter = $cached_postid_to_jsonld_converter;
-
-		$this->entity_uri_service = \Wordlift_Entity_Uri_Service::get_instance();
-	}
-
-	public function after_get_jsonld( $jsonld, $post_id, $context ) {
-
-		if ( Jsonld_Context_Enum::PAGE !== $context || ! is_array( $jsonld ) || ! isset( $jsonld[0] )
-			 || ! is_array( $jsonld[0] ) ) {
-			return $jsonld;
-		}
-
-		// Copy the 1st array element
-		$post_jsonld = $jsonld[0];
-
-		// Don't wrap in article if the json-ld is already about an Article (or its descendants). `@type` must be
-		// in the schema.org context, i.e. `Article`, not `http://schema.org/Article`.
-		if ( ! isset( $post_jsonld['@id'] ) || ! isset( $post_jsonld['@type'] ) || $this->is_article( $post_jsonld['@type'] ) ) {
-			return $jsonld;
-		}
-
-		// Convert the post as Article.
-		$article_jsonld = $this->post_to_jsonld_converter->convert( $post_id );
-
-		$article_jsonld['@id'] = $post_jsonld['@id'] . '#article';
-		// Reset the type, since by default the type assigned via the Entity Type taxonomy is used.
-		$article_jsonld['@type'] = 'Article';
-		$article_jsonld['about'] = array( '@id' => $post_jsonld['@id'] );
-
-		// Copy over the URLs.
-		if ( isset( $post_jsonld['url'] ) ) {
-			$article_jsonld['url'] = $post_jsonld['url'];
-		}
-
-		array_unshift( $jsonld, $article_jsonld );
-
-		$author_jsonld = $this->get_author_linked_entity( $article_jsonld );
-
-		/**
-		 * The author entities can be present in graph for some entity types
-		 * for Person and Organization, so check before we add it to graph.
-		 * reference : https://schema.org/author
-		 */
-		if ( $author_jsonld && ! $this->is_author_entity_present_in_graph( $jsonld, $article_jsonld['author']['@id'] ) ) {
-			$jsonld[] = $author_jsonld;
-		}
-
-		return $jsonld;
-	}
-
-	private function is_article( $schema_types ) {
-
-		$array_intersect = array_intersect( self::$article_types, (array) $schema_types );
-
-		return ! empty( $array_intersect );
-	}
-
-	private function get_author_linked_entity( $article_jsonld ) {
-		if ( ! array_key_exists( 'author', $article_jsonld ) ) {
-			return false;
-		}
-
-		$author = $article_jsonld['author'];
+    public static $article_types = array(
+        'Article',
+        'AdvertiserContentArticle',
+        'NewsArticle',
+        'AnalysisNewsArticle',
+        'AskPublicNewsArticle',
+        'BackgroundNewsArticle',
+        'OpinionNewsArticle',
+        'ReportageNewsArticle',
+        'ReviewNewsArticle',
+        'Report',
+        'SatiricalArticle',
+        'ScholarlyArticle',
+        'MedicalScholarlyArticle',
+        'SocialMediaPosting',
+        'BlogPosting',
+        'LiveBlogPosting',
+        'DiscussionForumPosting',
+        'TechArticle',
+        'APIReference',
+    );
+
+    /**
+     * @var Wordlift_Post_To_Jsonld_Converter
+     */
+    private $post_to_jsonld_converter;
+    /**
+     * @var \Wordlift_Cached_Post_Converter
+     */
+    private $cached_postid_to_jsonld_converter;
+    /**
+     * @var \Wordlift_Entity_Uri_Service
+     */
+    private $entity_uri_service;
+
+    /**
+     * Jsonld_Article_Wrapper constructor.
+     *
+     * @param Wordlift_Post_To_Jsonld_Converter $post_to_jsonld_converter
+     * @param $cached_postid_to_jsonld_converter
+     */
+    public function __construct( $post_to_jsonld_converter, $cached_postid_to_jsonld_converter ) {
+
+        $this->post_to_jsonld_converter = $post_to_jsonld_converter->new_instance_with_filters_disabled();
+
+        add_filter( 'wl_after_get_jsonld', array( $this, 'after_get_jsonld' ), PHP_INT_MAX - 100, 3 );
+
+        $this->cached_postid_to_jsonld_converter = $cached_postid_to_jsonld_converter;
+
+        $this->entity_uri_service = \Wordlift_Entity_Uri_Service::get_instance();
+    }
+
+    public function after_get_jsonld( $jsonld, $post_id, $context ) {
+
+        if ( Jsonld_Context_Enum::PAGE !== $context || ! is_array( $jsonld ) || ! isset( $jsonld[0] )
+             || ! is_array( $jsonld[0] ) ) {
+            return $jsonld;
+        }
+
+        // Copy the 1st array element
+        $post_jsonld = $jsonld[0];
+
+        // Don't wrap in article if the json-ld is already about an Article (or its descendants). `@type` must be
+        // in the schema.org context, i.e. `Article`, not `http://schema.org/Article`.
+        if ( ! isset( $post_jsonld['@id'] ) || ! isset( $post_jsonld['@type'] ) || $this->is_article( $post_jsonld['@type'] ) ) {
+            return $jsonld;
+        }
+
+        // Convert the post as Article.
+        $article_jsonld = $this->post_to_jsonld_converter->convert( $post_id );
+
+        $article_jsonld['@id'] = $post_jsonld['@id'] . '#article';
+        // Reset the type, since by default the type assigned via the Entity Type taxonomy is used.
+        $article_jsonld['@type'] = 'Article';
+        $article_jsonld['about'] = array( '@id' => $post_jsonld['@id'] );
+
+        // Copy over the URLs.
+        if ( isset( $post_jsonld['url'] ) ) {
+            $article_jsonld['url'] = $post_jsonld['url'];
+        }
+
+        array_unshift( $jsonld, $article_jsonld );
+
+        $author_jsonld = $this->get_author_linked_entity( $article_jsonld );
+
+        /**
+         * The author entities can be present in graph for some entity types
+         * for Person and Organization, so check before we add it to graph.
+         * reference : https://schema.org/author
+         */
+        if ( $author_jsonld && ! $this->is_author_entity_present_in_graph( $jsonld, $article_jsonld['author']['@id'] ) ) {
+            $jsonld[] = $author_jsonld;
+        }
+
+        return $jsonld;
+    }
+
+    private function is_article( $schema_types ) {
+
+        $array_intersect = array_intersect( self::$article_types, (array) $schema_types );
+
+        return ! empty( $array_intersect );
+    }
+
+    private function get_author_linked_entity( $article_jsonld ) {
+        if ( ! array_key_exists( 'author', $article_jsonld ) ) {
+            return false;
+        }
+
+        $author = $article_jsonld['author'];
 
-		if ( count( array_keys( $author ) ) !== 1 || ! array_key_exists( '@id', $author ) ) {
-			return false;
-		}
+        if ( count( array_keys( $author ) ) !== 1 || ! array_key_exists( '@id', $author ) ) {
+            return false;
+        }
 
-		$author_linked_entity_id = $author['@id'];
+        $author_linked_entity_id = $author['@id'];
 
-		$author_entity_post = $this->entity_uri_service->get_entity( $author_linked_entity_id );
+        $author_entity_post = $this->entity_uri_service->get_entity( $author_linked_entity_id );
 
-		if ( ! $author_entity_post instanceof \WP_Post ) {
-			return false;
-		}
+        if ( ! $author_entity_post instanceof \WP_Post ) {
+            return false;
+        }
 
-		return $this->cached_postid_to_jsonld_converter->convert( $author_entity_post->ID );
+        return $this->cached_postid_to_jsonld_converter->convert( $author_entity_post->ID );
 
-	}
+    }
 
-	private function is_author_entity_present_in_graph( $jsonld, $author_entity_id ) {
+    private function is_author_entity_present_in_graph( $jsonld, $author_entity_id ) {
 
-		foreach ( $jsonld as $item ) {
-			if ( $item && array_key_exists( '@id', $item ) && $item['@id'] === $author_entity_id ) {
-				return true;
-			}
-		}
+        foreach ( $jsonld as $item ) {
+            if ( $item && array_key_exists( '@id', $item ) && $item['@id'] === $author_entity_id ) {
+                return true;
+            }
+        }
 
-		return false;
-	}
+        return false;
+    }
 
 }

Spacing +25 added lines, -25 removed lines

@@ -47,21 +47,21 @@ 
 	 * @param Wordlift_Post_To_Jsonld_Converter $post_to_jsonld_converter
 	 * @param $cached_postid_to_jsonld_converter
 	 */
-	public function __construct( $post_to_jsonld_converter, $cached_postid_to_jsonld_converter ) {
+	public function __construct($post_to_jsonld_converter, $cached_postid_to_jsonld_converter) {
 
 		$this->post_to_jsonld_converter = $post_to_jsonld_converter->new_instance_with_filters_disabled();
 
-		add_filter( 'wl_after_get_jsonld', array( $this, 'after_get_jsonld' ), PHP_INT_MAX - 100, 3 );
+		add_filter('wl_after_get_jsonld', array($this, 'after_get_jsonld'), PHP_INT_MAX - 100, 3);
 
 		$this->cached_postid_to_jsonld_converter = $cached_postid_to_jsonld_converter;
 
 		$this->entity_uri_service = \Wordlift_Entity_Uri_Service::get_instance();
 	}
 
-	public function after_get_jsonld( $jsonld, $post_id, $context ) {
+	public function after_get_jsonld($jsonld, $post_id, $context) {
 
-		if ( Jsonld_Context_Enum::PAGE !== $context || ! is_array( $jsonld ) || ! isset( $jsonld[0] )
-			 || ! is_array( $jsonld[0] ) ) {
+		if (Jsonld_Context_Enum::PAGE !== $context || ! is_array($jsonld) || ! isset($jsonld[0])
+			 || ! is_array($jsonld[0])) {
 			return $jsonld;
 		}
 
@@ -70,73 +70,73 @@ 
 
 		// Don't wrap in article if the json-ld is already about an Article (or its descendants). `@type` must be
 		// in the schema.org context, i.e. `Article`, not `http://schema.org/Article`.
-		if ( ! isset( $post_jsonld['@id'] ) || ! isset( $post_jsonld['@type'] ) || $this->is_article( $post_jsonld['@type'] ) ) {
+		if ( ! isset($post_jsonld['@id']) || ! isset($post_jsonld['@type']) || $this->is_article($post_jsonld['@type'])) {
 			return $jsonld;
 		}
 
 		// Convert the post as Article.
-		$article_jsonld = $this->post_to_jsonld_converter->convert( $post_id );
+		$article_jsonld = $this->post_to_jsonld_converter->convert($post_id);
 
-		$article_jsonld['@id'] = $post_jsonld['@id'] . '#article';
+		$article_jsonld['@id'] = $post_jsonld['@id'].'#article';
 		// Reset the type, since by default the type assigned via the Entity Type taxonomy is used.
 		$article_jsonld['@type'] = 'Article';
-		$article_jsonld['about'] = array( '@id' => $post_jsonld['@id'] );
+		$article_jsonld['about'] = array('@id' => $post_jsonld['@id']);
 
 		// Copy over the URLs.
-		if ( isset( $post_jsonld['url'] ) ) {
+		if (isset($post_jsonld['url'])) {
 			$article_jsonld['url'] = $post_jsonld['url'];
 		}
 
-		array_unshift( $jsonld, $article_jsonld );
+		array_unshift($jsonld, $article_jsonld);
 
-		$author_jsonld = $this->get_author_linked_entity( $article_jsonld );
+		$author_jsonld = $this->get_author_linked_entity($article_jsonld);
 
 		/**
 		 * The author entities can be present in graph for some entity types
 		 * for Person and Organization, so check before we add it to graph.
 		 * reference : https://schema.org/author
 		 */
-		if ( $author_jsonld && ! $this->is_author_entity_present_in_graph( $jsonld, $article_jsonld['author']['@id'] ) ) {
+		if ($author_jsonld && ! $this->is_author_entity_present_in_graph($jsonld, $article_jsonld['author']['@id'])) {
 			$jsonld[] = $author_jsonld;
 		}
 
 		return $jsonld;
 	}
 
-	private function is_article( $schema_types ) {
+	private function is_article($schema_types) {
 
-		$array_intersect = array_intersect( self::$article_types, (array) $schema_types );
+		$array_intersect = array_intersect(self::$article_types, (array) $schema_types);
 
-		return ! empty( $array_intersect );
+		return ! empty($array_intersect);
 	}
 
-	private function get_author_linked_entity( $article_jsonld ) {
-		if ( ! array_key_exists( 'author', $article_jsonld ) ) {
+	private function get_author_linked_entity($article_jsonld) {
+		if ( ! array_key_exists('author', $article_jsonld)) {
 			return false;
 		}
 
 		$author = $article_jsonld['author'];
 
-		if ( count( array_keys( $author ) ) !== 1 || ! array_key_exists( '@id', $author ) ) {
+		if (count(array_keys($author)) !== 1 || ! array_key_exists('@id', $author)) {
 			return false;
 		}
 
 		$author_linked_entity_id = $author['@id'];
 
-		$author_entity_post = $this->entity_uri_service->get_entity( $author_linked_entity_id );
+		$author_entity_post = $this->entity_uri_service->get_entity($author_linked_entity_id);
 
-		if ( ! $author_entity_post instanceof \WP_Post ) {
+		if ( ! $author_entity_post instanceof \WP_Post) {
 			return false;
 		}
 
-		return $this->cached_postid_to_jsonld_converter->convert( $author_entity_post->ID );
+		return $this->cached_postid_to_jsonld_converter->convert($author_entity_post->ID);
 
 	}
 
-	private function is_author_entity_present_in_graph( $jsonld, $author_entity_id ) {
+	private function is_author_entity_present_in_graph($jsonld, $author_entity_id) {
 
-		foreach ( $jsonld as $item ) {
-			if ( $item && array_key_exists( '@id', $item ) && $item['@id'] === $author_entity_id ) {
+		foreach ($jsonld as $item) {
+			if ($item && array_key_exists('@id', $item) && $item['@id'] === $author_entity_id) {
 				return true;
 			}
 		}

src/wordlift/jsonld/class-jsonld-by-id-endpoint.php

Indentation +118 added lines, -118 removed lines

@@ -23,123 +23,123 @@
  */
 class Jsonld_By_Id_Endpoint {
 
-	/**
-	 * The {@link Wordlift_Jsonld_Service} instance.
-	 *
-	 * @var Wordlift_Jsonld_Service The {@link Wordlift_Jsonld_Service} instance.
-	 */
-	private $jsonld_service;
-
-	/**
-	 * @var \Wordlift_Entity_Uri_Service
-	 */
-	private $entity_uri_service;
-
-	/**
-	 * Jsonld_Endpoint constructor.
-	 *
-	 * @param \Wordlift_Jsonld_Service     $jsonld_service
-	 * @param \Wordlift_Entity_Uri_Service $entity_uri_service
-	 */
-	public function __construct( $jsonld_service, $entity_uri_service ) {
-
-		$this->jsonld_service     = $jsonld_service;
-		$this->entity_uri_service = $entity_uri_service;
-
-		add_action( 'rest_api_init', array( $this, 'register_routes' ) );
-
-	}
-
-	/**
-	 * Get the JSON-LD.
-	 *
-	 * @param \WP_REST_Request $request The incoming {@link \WP_REST_Request}.
-	 *
-	 * @return WP_REST_Response The outgoing {@link WP_REST_Response}.
-	 * @throws \Exception when an error occurs.
-	 */
-	public function jsonld_by_id( $request ) {
-
-		// Get the ids.
-		$ids = (array) $request->get_param( 'id' );
-
-		// Preload the URIs to reduce the number of DB roundtrips.
-		$this->entity_uri_service->preload_uris( array_map( 'urldecode', $ids ) );
-
-		$that = $this;
-
-		// Get the posts, filtering out those not found.
-		$posts = array_filter(
-			array_map(
-				function ( $item ) use ( $that ) {
-					return $that->entity_uri_service->get_entity( urldecode( $item ) );
-				},
-				$ids
-			)
-		);
-
-		// Get the posts' IDs and make the unique.
-		$post_ids = array_unique(
-			array_map(
-				function ( $item ) {
-					return $item->ID;
-				},
-				$posts
-			)
-		);
-
-		// Get the JSON-LD.
-		$data = array();
-		foreach ( $post_ids as $post_id ) {
-			$data = array_merge( $data, $that->jsonld_service->get_jsonld( false, $post_id ) );
-		}
-
-		// Add the WebSite fragment if requested.
-		if ( $request->get_param( 'website' ) ) {
-			$data[] = $this->jsonld_service->get_jsonld( true );
-		}
-
-		return Jsonld_Response_Helper::to_response( $data );
-	}
-
-	public function register_routes() {
-
-		register_rest_route(
-			WL_REST_ROUTE_DEFAULT_NAMESPACE,
-			'/jsonld',
-			array(
-				'methods'             => WP_REST_Server::READABLE,
-				'callback'            => array( $this, 'jsonld_by_id' ),
-				'permission_callback' => '__return_true',
-				'args'                => array(
-					'id'      => array(
-						'description'       => __( 'One ore more itemids (e.g. http://data.wordlift.io/wordlift).', 'wordlift' ),
-						// We expect an array of strings.
-						'type'              => 'array',
-						'items'             => array(
-							'type' => 'string',
-						),
-						// phpcs:ignore VariableAnalysis.CodeAnalysis.VariableAnalysis.UnusedVariable
-						'validate_callback' => function ( $values, $request, $param ) {
-
-							if ( ! is_array( $values ) ) {
-								return new WP_Error( 'rest_invalid_param', esc_html__( 'The id argument must be an array (try passing `id[]=...`.', 'wordlift' ), array( 'status' => 400 ) );
-							}
-
-							foreach ( $values as $value ) {
-								if ( 0 !== strpos( $value, 'http' ) ) {
-									return new WP_Error( 'rest_invalid_param', esc_html__( 'Ids must start with http.', 'wordlift' ), array( 'status' => 400 ) );
-								}
-							}
-						},
-					),
-					'website' => array(
-						'description' => __( 'Whether to include the WebSite markup.', 'wordlift' ),
-					),
-				),
-			)
-		);
-
-	}
+    /**
+     * The {@link Wordlift_Jsonld_Service} instance.
+     *
+     * @var Wordlift_Jsonld_Service The {@link Wordlift_Jsonld_Service} instance.
+     */
+    private $jsonld_service;
+
+    /**
+     * @var \Wordlift_Entity_Uri_Service
+     */
+    private $entity_uri_service;
+
+    /**
+     * Jsonld_Endpoint constructor.
+     *
+     * @param \Wordlift_Jsonld_Service     $jsonld_service
+     * @param \Wordlift_Entity_Uri_Service $entity_uri_service
+     */
+    public function __construct( $jsonld_service, $entity_uri_service ) {
+
+        $this->jsonld_service     = $jsonld_service;
+        $this->entity_uri_service = $entity_uri_service;
+
+        add_action( 'rest_api_init', array( $this, 'register_routes' ) );
+
+    }
+
+    /**
+     * Get the JSON-LD.
+     *
+     * @param \WP_REST_Request $request The incoming {@link \WP_REST_Request}.
+     *
+     * @return WP_REST_Response The outgoing {@link WP_REST_Response}.
+     * @throws \Exception when an error occurs.
+     */
+    public function jsonld_by_id( $request ) {
+
+        // Get the ids.
+        $ids = (array) $request->get_param( 'id' );
+
+        // Preload the URIs to reduce the number of DB roundtrips.
+        $this->entity_uri_service->preload_uris( array_map( 'urldecode', $ids ) );
+
+        $that = $this;
+
+        // Get the posts, filtering out those not found.
+        $posts = array_filter(
+            array_map(
+                function ( $item ) use ( $that ) {
+                    return $that->entity_uri_service->get_entity( urldecode( $item ) );
+                },
+                $ids
+            )
+        );
+
+        // Get the posts' IDs and make the unique.
+        $post_ids = array_unique(
+            array_map(
+                function ( $item ) {
+                    return $item->ID;
+                },
+                $posts
+            )
+        );
+
+        // Get the JSON-LD.
+        $data = array();
+        foreach ( $post_ids as $post_id ) {
+            $data = array_merge( $data, $that->jsonld_service->get_jsonld( false, $post_id ) );
+        }
+
+        // Add the WebSite fragment if requested.
+        if ( $request->get_param( 'website' ) ) {
+            $data[] = $this->jsonld_service->get_jsonld( true );
+        }
+
+        return Jsonld_Response_Helper::to_response( $data );
+    }
+
+    public function register_routes() {
+
+        register_rest_route(
+            WL_REST_ROUTE_DEFAULT_NAMESPACE,
+            '/jsonld',
+            array(
+                'methods'             => WP_REST_Server::READABLE,
+                'callback'            => array( $this, 'jsonld_by_id' ),
+                'permission_callback' => '__return_true',
+                'args'                => array(
+                    'id'      => array(
+                        'description'       => __( 'One ore more itemids (e.g. http://data.wordlift.io/wordlift).', 'wordlift' ),
+                        // We expect an array of strings.
+                        'type'              => 'array',
+                        'items'             => array(
+                            'type' => 'string',
+                        ),
+                        // phpcs:ignore VariableAnalysis.CodeAnalysis.VariableAnalysis.UnusedVariable
+                        'validate_callback' => function ( $values, $request, $param ) {
+
+                            if ( ! is_array( $values ) ) {
+                                return new WP_Error( 'rest_invalid_param', esc_html__( 'The id argument must be an array (try passing `id[]=...`.', 'wordlift' ), array( 'status' => 400 ) );
+                            }
+
+                            foreach ( $values as $value ) {
+                                if ( 0 !== strpos( $value, 'http' ) ) {
+                                    return new WP_Error( 'rest_invalid_param', esc_html__( 'Ids must start with http.', 'wordlift' ), array( 'status' => 400 ) );
+                                }
+                            }
+                        },
+                    ),
+                    'website' => array(
+                        'description' => __( 'Whether to include the WebSite markup.', 'wordlift' ),
+                    ),
+                ),
+            )
+        );
+
+    }
 
 }

Spacing +22 added lines, -22 removed lines

@@ -41,12 +41,12 @@ 
 	 * @param \Wordlift_Jsonld_Service     $jsonld_service
 	 * @param \Wordlift_Entity_Uri_Service $entity_uri_service
 	 */
-	public function __construct( $jsonld_service, $entity_uri_service ) {
+	public function __construct($jsonld_service, $entity_uri_service) {
 
 		$this->jsonld_service     = $jsonld_service;
 		$this->entity_uri_service = $entity_uri_service;
 
-		add_action( 'rest_api_init', array( $this, 'register_routes' ) );
+		add_action('rest_api_init', array($this, 'register_routes'));
 
 	}
 
@@ -58,21 +58,21 @@ 
 	 * @return WP_REST_Response The outgoing {@link WP_REST_Response}.
 	 * @throws \Exception when an error occurs.
 	 */
-	public function jsonld_by_id( $request ) {
+	public function jsonld_by_id($request) {
 
 		// Get the ids.
-		$ids = (array) $request->get_param( 'id' );
+		$ids = (array) $request->get_param('id');
 
 		// Preload the URIs to reduce the number of DB roundtrips.
-		$this->entity_uri_service->preload_uris( array_map( 'urldecode', $ids ) );
+		$this->entity_uri_service->preload_uris(array_map('urldecode', $ids));
 
 		$that = $this;
 
 		// Get the posts, filtering out those not found.
 		$posts = array_filter(
 			array_map(
-				function ( $item ) use ( $that ) {
-					return $that->entity_uri_service->get_entity( urldecode( $item ) );
+				function($item) use ($that) {
+					return $that->entity_uri_service->get_entity(urldecode($item));
 				},
 				$ids
 			)
@@ -81,7 +81,7 @@ 
 		// Get the posts' IDs and make the unique.
 		$post_ids = array_unique(
 			array_map(
-				function ( $item ) {
+				function($item) {
 					return $item->ID;
 				},
 				$posts
@@ -90,16 +90,16 @@ 
 
 		// Get the JSON-LD.
 		$data = array();
-		foreach ( $post_ids as $post_id ) {
-			$data = array_merge( $data, $that->jsonld_service->get_jsonld( false, $post_id ) );
+		foreach ($post_ids as $post_id) {
+			$data = array_merge($data, $that->jsonld_service->get_jsonld(false, $post_id));
 		}
 
 		// Add the WebSite fragment if requested.
-		if ( $request->get_param( 'website' ) ) {
-			$data[] = $this->jsonld_service->get_jsonld( true );
+		if ($request->get_param('website')) {
+			$data[] = $this->jsonld_service->get_jsonld(true);
 		}
 
-		return Jsonld_Response_Helper::to_response( $data );
+		return Jsonld_Response_Helper::to_response($data);
 	}
 
 	public function register_routes() {
@@ -109,32 +109,32 @@ 
 			'/jsonld',
 			array(
 				'methods'             => WP_REST_Server::READABLE,
-				'callback'            => array( $this, 'jsonld_by_id' ),
+				'callback'            => array($this, 'jsonld_by_id'),
 				'permission_callback' => '__return_true',
 				'args'                => array(
 					'id'      => array(
-						'description'       => __( 'One ore more itemids (e.g. http://data.wordlift.io/wordlift).', 'wordlift' ),
+						'description'       => __('One ore more itemids (e.g. http://data.wordlift.io/wordlift).', 'wordlift'),
 						// We expect an array of strings.
 						'type'              => 'array',
 						'items'             => array(
 							'type' => 'string',
 						),
 						// phpcs:ignore VariableAnalysis.CodeAnalysis.VariableAnalysis.UnusedVariable
-						'validate_callback' => function ( $values, $request, $param ) {
+						'validate_callback' => function($values, $request, $param) {
 
-							if ( ! is_array( $values ) ) {
-								return new WP_Error( 'rest_invalid_param', esc_html__( 'The id argument must be an array (try passing `id[]=...`.', 'wordlift' ), array( 'status' => 400 ) );
+							if ( ! is_array($values)) {
+								return new WP_Error('rest_invalid_param', esc_html__('The id argument must be an array (try passing `id[]=...`.', 'wordlift'), array('status' => 400));
 							}
 
-							foreach ( $values as $value ) {
-								if ( 0 !== strpos( $value, 'http' ) ) {
-									return new WP_Error( 'rest_invalid_param', esc_html__( 'Ids must start with http.', 'wordlift' ), array( 'status' => 400 ) );
+							foreach ($values as $value) {
+								if (0 !== strpos($value, 'http')) {
+									return new WP_Error('rest_invalid_param', esc_html__('Ids must start with http.', 'wordlift'), array('status' => 400));
 								}
 							}
 						},
 					),
 					'website' => array(
-						'description' => __( 'Whether to include the WebSite markup.', 'wordlift' ),
+						'description' => __('Whether to include the WebSite markup.', 'wordlift'),
 					),
 				),
 			)