insideout10 / wordlift-plugin

src/wordlift/dataset/class-sync-service.php

Indentation +244 added lines, -244 removed lines

@@ -8,251 +8,251 @@
 use Wordlift\Object_Type_Enum;
 
 class Sync_Service {
-	const JSONLD_HASH = 'jsonld_hash';
-	const SYNCED_GMT = 'synced_gmt';
-
-	/**
-	 * @var \Wordlift_Log_Service
-	 */
-	private $log;
-
-	/**
-	 * @var Api_Service
-	 */
-	private $api_service;
-
-	/**
-	 * @var Jsonld_Service
-	 */
-	private $jsonld_service;
-
-	/**
-	 * @var Sync_Background_Process
-	 */
-	private $sync_background_process;
-
-	/**
-	 * The number of posts processed in one call.
-	 *
-	 * @var int The batch size.
-	 */
-	private $batch_size;
-
-	/**
-	 * @var Sync_Object_Adapter_Factory
-	 */
-	private $sync_object_adapter_factory;
-
-	/**
-	 * @var Sync_Service
-	 */
-	private static $instance;
-	private $entity_service;
-
-	/**
-	 * Constructor.
-	 *
-	 * @param Api_Service $api_service The {@link Api_Service} used to communicate with the remote APIs.
-	 * @param Sync_Object_Adapter_Factory $sync_object_adapter_factory
-	 * @param Jsonld_Service $jsonld_service
-	 * @param \Wordlift_Entity_Service $entity_service
-	 */
-	public function __construct( $api_service, $sync_object_adapter_factory, $jsonld_service, $entity_service ) {
-
-		$this->log = \Wordlift_Log_Service::get_logger( get_class() );
-
-		$this->api_service                 = $api_service;
-		$this->sync_object_adapter_factory = $sync_object_adapter_factory;
-		$this->jsonld_service              = $jsonld_service;
-		$this->entity_service              = $entity_service;
-		$this->batch_size                  = 10;
-
-		// You need to initialize this early, otherwise the Background Process isn't registered in AJAX calls.
+    const JSONLD_HASH = 'jsonld_hash';
+    const SYNCED_GMT = 'synced_gmt';
+
+    /**
+     * @var \Wordlift_Log_Service
+     */
+    private $log;
+
+    /**
+     * @var Api_Service
+     */
+    private $api_service;
+
+    /**
+     * @var Jsonld_Service
+     */
+    private $jsonld_service;
+
+    /**
+     * @var Sync_Background_Process
+     */
+    private $sync_background_process;
+
+    /**
+     * The number of posts processed in one call.
+     *
+     * @var int The batch size.
+     */
+    private $batch_size;
+
+    /**
+     * @var Sync_Object_Adapter_Factory
+     */
+    private $sync_object_adapter_factory;
+
+    /**
+     * @var Sync_Service
+     */
+    private static $instance;
+    private $entity_service;
+
+    /**
+     * Constructor.
+     *
+     * @param Api_Service $api_service The {@link Api_Service} used to communicate with the remote APIs.
+     * @param Sync_Object_Adapter_Factory $sync_object_adapter_factory
+     * @param Jsonld_Service $jsonld_service
+     * @param \Wordlift_Entity_Service $entity_service
+     */
+    public function __construct( $api_service, $sync_object_adapter_factory, $jsonld_service, $entity_service ) {
+
+        $this->log = \Wordlift_Log_Service::get_logger( get_class() );
+
+        $this->api_service                 = $api_service;
+        $this->sync_object_adapter_factory = $sync_object_adapter_factory;
+        $this->jsonld_service              = $jsonld_service;
+        $this->entity_service              = $entity_service;
+        $this->batch_size                  = 10;
+
+        // You need to initialize this early, otherwise the Background Process isn't registered in AJAX calls.
 //		$this->sync_background_process = new Sync_Background_Process( $this );;
 
-		// Exclude the JSONLD_HASH meta key from those that require a resync.
-		add_filter( 'wl_dataset__sync_hooks__ignored_meta_keys', function ( $args ) {
-			$args[] = Sync_Service::JSONLD_HASH;
-			$args[] = Sync_Service::SYNCED_GMT;
-
-			return $args;
-		} );
-
-		self::$instance = $this;
-	}
-
-	public static function get_instance() {
-		return self::$instance;
-	}
-
-	/**
-	 * @param int $type
-	 * @param int $object_id
-	 *
-	 * @return array|false
-	 * @throws Exception
-	 */
-	public function sync_one( $type, $object_id ) {
-
-		$object = $this->sync_object_adapter_factory->create( $type, $object_id );
-
-		return $this->sync_many( array( $object ) );
-	}
-
-	/**
-	 * @param $type string Post or User.
-	 * @param $object_id  int Post or User id
-	 * @param $uri string Entity uri , This needs to be supplied before deletion, if we
-	 * get it from meta it might not be available.
-	 *
-	 * @return bool
-	 */
-	public function delete_one( $type, $object_id, $uri ) {
-		// Entity URL isn't set, bail out.
-		if ( empty( $uri ) ) {
-			return false;
-		}
-
-		$response = $this->api_service->request(
-			'DELETE', sprintf( '/middleware/dataset?uri=%s', rawurlencode( $uri ) ) );
-
-
-		// Update the sync date in case of success, otherwise log an error.
-		if ( ! $response->is_success() ) {
-			return false;
-		}
-
-		/**
-		 * Allow 3rd parties to run additional sync work.
-		 */
-		do_action( 'wl_sync__delete_one', $type, $object_id, $uri );
-
-		return true;
-	}
-
-	/**
-	 * @param Sync_Object_Adapter[] $objects
-	 * @param bool $force Force synchronization even if the json-ld hash hasn't changed.
-	 *
-	 * @return bool
-	 * @throws Exception
-	 */
-	public function sync_many( $objects, $force = false ) {
-
-		$hashes   = array();
-		$payloads = array();
-		/** @var Sync_Object_Adapter $object */
-		foreach ( $objects as $object ) {
-			// Bail out if no payload.
-			$payload_as_string = $this->get_payload_as_string( $object );
-			if ( empty( $payload_as_string ) ) {
-				continue;
-			}
-			$new_hash = sha1( $payload_as_string );
-			$old_hash = $object->get_value( self::JSONLD_HASH );
-
-			// JSON-LD hasn't changed, bail out.
-			$should_sync = $force || $new_hash !== $old_hash;
-			if ( ! apply_filters( 'wl_dataset__sync_service__sync_item', $should_sync, $object, $payload_as_string ) ) {
-				continue;
-			}
-
-			// Collect the hashes and the payloads.
-			$hashes[]   = array( $object, $new_hash, $payload_as_string );
-			$payloads[] = $payload_as_string;
-		}
-
-		// Bail out if payloads are empty.
-		if ( empty( $payloads ) ) {
-			return false;
-		}
-
-		$blocking = apply_filters( 'wl_feature__enable__sync-blocking', false );
-		$response = $this->api_service->request(
-			'POST', '/middleware/dataset/batch',
-			array( 'Content-Type' => 'application/json', ),
-			// Put the payload in a JSON array w/o decoding/encoding again.
-			'[ ' . implode( ', ', $payloads ) . ' ]',
-			$blocking ? 60 : 0.001,
-			null,
-			array( 'blocking' => $blocking ) );
-
-		// Update the sync date in case of success, otherwise log an error.
-		if ( $blocking && ! $response->is_success() ) {
-			return false;
-		}
-
-		// If successful update the hashes and sync datetime.
-		foreach ( $hashes as $hash ) {
-			$object   = $hash[0];
-			$new_hash = $hash[1];
-
-			$object->set_values( array(
-				self::JSONLD_HASH => $new_hash,
-				self::SYNCED_GMT  => current_time( 'mysql', true ),
-			) );
-		}
-
-
-		/**
-		 * Allow 3rd parties to run additional sync work.
-		 */
-		do_action( 'wl_sync__sync_many', $hashes );
-
-		return true;
-	}
-
-	/**
-	 * @param Sync_Object_Adapter $object
-	 *
-	 * @return false|string
-	 * @throws Exception
-	 */
-	private function get_payload_as_string( $object ) {
-		$type             = $object->get_type();
-		$object_id        = $object->get_object_id();
-		$jsonld_as_string = wp_json_encode( apply_filters( 'wl_dataset__sync_service__sync_item__jsonld',
-			$this->jsonld_service->get( $type, $object_id ), $type, $object_id ), 64 ); // JSON_UNESCAPED_SLASHES
-		$uri              = $this->entity_service->get_uri( $object_id, $type );
-
-		// Entity URL isn't set, bail out.
-		if ( empty( $uri ) ) {
-			return false;
-		}
-
-		return wp_json_encode( array(
-			'uri'     => $uri,
-			'model'   => $jsonld_as_string,
-			'private' => ! ( $object->is_public() && $object->is_published() ),
-		), 64 ); // JSON_UNESCAPED_SLASHES
-	}
-
-	/**
-	 * @param $post_id
-	 *
-	 * @todo Complete the delete item.
-	 */
-	public function delete_item( $post_id ) {
-		$uri = $this->entity_service->get_uri( $post_id, Object_Type_Enum::POST );
-
-		if ( ! isset( $uri ) ) {
-			return;
-		}
-
-		// Make a request to the remote endpoint.
-		$response = $this->api_service->request(
-			'DELETE', '/middleware/dataset?uri=' . rawurlencode( $uri ),
-			array( 'Content-Type' => 'application/ld+json', ) );
-
-	}
-
-	public function get_batch_size() {
-
-		return $this->batch_size;
-	}
-
-	public function delete_all() {
-		$this->api_service->request( 'DELETE', '/middleware/dataset/all' );
-	}
+        // Exclude the JSONLD_HASH meta key from those that require a resync.
+        add_filter( 'wl_dataset__sync_hooks__ignored_meta_keys', function ( $args ) {
+            $args[] = Sync_Service::JSONLD_HASH;
+            $args[] = Sync_Service::SYNCED_GMT;
+
+            return $args;
+        } );
+
+        self::$instance = $this;
+    }
+
+    public static function get_instance() {
+        return self::$instance;
+    }
+
+    /**
+     * @param int $type
+     * @param int $object_id
+     *
+     * @return array|false
+     * @throws Exception
+     */
+    public function sync_one( $type, $object_id ) {
+
+        $object = $this->sync_object_adapter_factory->create( $type, $object_id );
+
+        return $this->sync_many( array( $object ) );
+    }
+
+    /**
+     * @param $type string Post or User.
+     * @param $object_id  int Post or User id
+     * @param $uri string Entity uri , This needs to be supplied before deletion, if we
+     * get it from meta it might not be available.
+     *
+     * @return bool
+     */
+    public function delete_one( $type, $object_id, $uri ) {
+        // Entity URL isn't set, bail out.
+        if ( empty( $uri ) ) {
+            return false;
+        }
+
+        $response = $this->api_service->request(
+            'DELETE', sprintf( '/middleware/dataset?uri=%s', rawurlencode( $uri ) ) );
+
+
+        // Update the sync date in case of success, otherwise log an error.
+        if ( ! $response->is_success() ) {
+            return false;
+        }
+
+        /**
+         * Allow 3rd parties to run additional sync work.
+         */
+        do_action( 'wl_sync__delete_one', $type, $object_id, $uri );
+
+        return true;
+    }
+
+    /**
+     * @param Sync_Object_Adapter[] $objects
+     * @param bool $force Force synchronization even if the json-ld hash hasn't changed.
+     *
+     * @return bool
+     * @throws Exception
+     */
+    public function sync_many( $objects, $force = false ) {
+
+        $hashes   = array();
+        $payloads = array();
+        /** @var Sync_Object_Adapter $object */
+        foreach ( $objects as $object ) {
+            // Bail out if no payload.
+            $payload_as_string = $this->get_payload_as_string( $object );
+            if ( empty( $payload_as_string ) ) {
+                continue;
+            }
+            $new_hash = sha1( $payload_as_string );
+            $old_hash = $object->get_value( self::JSONLD_HASH );
+
+            // JSON-LD hasn't changed, bail out.
+            $should_sync = $force || $new_hash !== $old_hash;
+            if ( ! apply_filters( 'wl_dataset__sync_service__sync_item', $should_sync, $object, $payload_as_string ) ) {
+                continue;
+            }
+
+            // Collect the hashes and the payloads.
+            $hashes[]   = array( $object, $new_hash, $payload_as_string );
+            $payloads[] = $payload_as_string;
+        }
+
+        // Bail out if payloads are empty.
+        if ( empty( $payloads ) ) {
+            return false;
+        }
+
+        $blocking = apply_filters( 'wl_feature__enable__sync-blocking', false );
+        $response = $this->api_service->request(
+            'POST', '/middleware/dataset/batch',
+            array( 'Content-Type' => 'application/json', ),
+            // Put the payload in a JSON array w/o decoding/encoding again.
+            '[ ' . implode( ', ', $payloads ) . ' ]',
+            $blocking ? 60 : 0.001,
+            null,
+            array( 'blocking' => $blocking ) );
+
+        // Update the sync date in case of success, otherwise log an error.
+        if ( $blocking && ! $response->is_success() ) {
+            return false;
+        }
+
+        // If successful update the hashes and sync datetime.
+        foreach ( $hashes as $hash ) {
+            $object   = $hash[0];
+            $new_hash = $hash[1];
+
+            $object->set_values( array(
+                self::JSONLD_HASH => $new_hash,
+                self::SYNCED_GMT  => current_time( 'mysql', true ),
+            ) );
+        }
+
+
+        /**
+         * Allow 3rd parties to run additional sync work.
+         */
+        do_action( 'wl_sync__sync_many', $hashes );
+
+        return true;
+    }
+
+    /**
+     * @param Sync_Object_Adapter $object
+     *
+     * @return false|string
+     * @throws Exception
+     */
+    private function get_payload_as_string( $object ) {
+        $type             = $object->get_type();
+        $object_id        = $object->get_object_id();
+        $jsonld_as_string = wp_json_encode( apply_filters( 'wl_dataset__sync_service__sync_item__jsonld',
+            $this->jsonld_service->get( $type, $object_id ), $type, $object_id ), 64 ); // JSON_UNESCAPED_SLASHES
+        $uri              = $this->entity_service->get_uri( $object_id, $type );
+
+        // Entity URL isn't set, bail out.
+        if ( empty( $uri ) ) {
+            return false;
+        }
+
+        return wp_json_encode( array(
+            'uri'     => $uri,
+            'model'   => $jsonld_as_string,
+            'private' => ! ( $object->is_public() && $object->is_published() ),
+        ), 64 ); // JSON_UNESCAPED_SLASHES
+    }
+
+    /**
+     * @param $post_id
+     *
+     * @todo Complete the delete item.
+     */
+    public function delete_item( $post_id ) {
+        $uri = $this->entity_service->get_uri( $post_id, Object_Type_Enum::POST );
+
+        if ( ! isset( $uri ) ) {
+            return;
+        }
+
+        // Make a request to the remote endpoint.
+        $response = $this->api_service->request(
+            'DELETE', '/middleware/dataset?uri=' . rawurlencode( $uri ),
+            array( 'Content-Type' => 'application/ld+json', ) );
+
+    }
+
+    public function get_batch_size() {
+
+        return $this->batch_size;
+    }
+
+    public function delete_all() {
+        $this->api_service->request( 'DELETE', '/middleware/dataset/all' );
+    }
 
 }

Spacing +44 added lines, -44 removed lines

@@ -57,9 +57,9 @@ 
 	 * @param Jsonld_Service $jsonld_service
 	 * @param \Wordlift_Entity_Service $entity_service
 	 */
-	public function __construct( $api_service, $sync_object_adapter_factory, $jsonld_service, $entity_service ) {
+	public function __construct($api_service, $sync_object_adapter_factory, $jsonld_service, $entity_service) {
 
-		$this->log = \Wordlift_Log_Service::get_logger( get_class() );
+		$this->log = \Wordlift_Log_Service::get_logger(get_class());
 
 		$this->api_service                 = $api_service;
 		$this->sync_object_adapter_factory = $sync_object_adapter_factory;
@@ -71,7 +71,7 @@ 
 //		$this->sync_background_process = new Sync_Background_Process( $this );;
 
 		// Exclude the JSONLD_HASH meta key from those that require a resync.
-		add_filter( 'wl_dataset__sync_hooks__ignored_meta_keys', function ( $args ) {
+		add_filter('wl_dataset__sync_hooks__ignored_meta_keys', function($args) {
 			$args[] = Sync_Service::JSONLD_HASH;
 			$args[] = Sync_Service::SYNCED_GMT;
 
@@ -92,11 +92,11 @@ 
 	 * @return array|false
 	 * @throws Exception
 	 */
-	public function sync_one( $type, $object_id ) {
+	public function sync_one($type, $object_id) {
 
-		$object = $this->sync_object_adapter_factory->create( $type, $object_id );
+		$object = $this->sync_object_adapter_factory->create($type, $object_id);
 
-		return $this->sync_many( array( $object ) );
+		return $this->sync_many(array($object));
 	}
 
 	/**
@@ -107,25 +107,25 @@ 
 	 *
 	 * @return bool
 	 */
-	public function delete_one( $type, $object_id, $uri ) {
+	public function delete_one($type, $object_id, $uri) {
 		// Entity URL isn't set, bail out.
-		if ( empty( $uri ) ) {
+		if (empty($uri)) {
 			return false;
 		}
 
 		$response = $this->api_service->request(
-			'DELETE', sprintf( '/middleware/dataset?uri=%s', rawurlencode( $uri ) ) );
+			'DELETE', sprintf('/middleware/dataset?uri=%s', rawurlencode($uri)) );
 
 
 		// Update the sync date in case of success, otherwise log an error.
-		if ( ! $response->is_success() ) {
+		if ( ! $response->is_success()) {
 			return false;
 		}
 
 		/**
 		 * Allow 3rd parties to run additional sync work.
 		 */
-		do_action( 'wl_sync__delete_one', $type, $object_id, $uri );
+		do_action('wl_sync__delete_one', $type, $object_id, $uri);
 
 		return true;
 	}
@@ -137,67 +137,67 @@ 
 	 * @return bool
 	 * @throws Exception
 	 */
-	public function sync_many( $objects, $force = false ) {
+	public function sync_many($objects, $force = false) {
 
 		$hashes   = array();
 		$payloads = array();
 		/** @var Sync_Object_Adapter $object */
-		foreach ( $objects as $object ) {
+		foreach ($objects as $object) {
 			// Bail out if no payload.
-			$payload_as_string = $this->get_payload_as_string( $object );
-			if ( empty( $payload_as_string ) ) {
+			$payload_as_string = $this->get_payload_as_string($object);
+			if (empty($payload_as_string)) {
 				continue;
 			}
-			$new_hash = sha1( $payload_as_string );
-			$old_hash = $object->get_value( self::JSONLD_HASH );
+			$new_hash = sha1($payload_as_string);
+			$old_hash = $object->get_value(self::JSONLD_HASH);
 
 			// JSON-LD hasn't changed, bail out.
 			$should_sync = $force || $new_hash !== $old_hash;
-			if ( ! apply_filters( 'wl_dataset__sync_service__sync_item', $should_sync, $object, $payload_as_string ) ) {
+			if ( ! apply_filters('wl_dataset__sync_service__sync_item', $should_sync, $object, $payload_as_string)) {
 				continue;
 			}
 
 			// Collect the hashes and the payloads.
-			$hashes[]   = array( $object, $new_hash, $payload_as_string );
+			$hashes[]   = array($object, $new_hash, $payload_as_string);
 			$payloads[] = $payload_as_string;
 		}
 
 		// Bail out if payloads are empty.
-		if ( empty( $payloads ) ) {
+		if (empty($payloads)) {
 			return false;
 		}
 
-		$blocking = apply_filters( 'wl_feature__enable__sync-blocking', false );
+		$blocking = apply_filters('wl_feature__enable__sync-blocking', false);
 		$response = $this->api_service->request(
 			'POST', '/middleware/dataset/batch',
-			array( 'Content-Type' => 'application/json', ),
+			array('Content-Type' => 'application/json',),
 			// Put the payload in a JSON array w/o decoding/encoding again.
-			'[ ' . implode( ', ', $payloads ) . ' ]',
+			'[ '.implode(', ', $payloads).' ]',
 			$blocking ? 60 : 0.001,
 			null,
-			array( 'blocking' => $blocking ) );
+			array('blocking' => $blocking) );
 
 		// Update the sync date in case of success, otherwise log an error.
-		if ( $blocking && ! $response->is_success() ) {
+		if ($blocking && ! $response->is_success()) {
 			return false;
 		}
 
 		// If successful update the hashes and sync datetime.
-		foreach ( $hashes as $hash ) {
+		foreach ($hashes as $hash) {
 			$object   = $hash[0];
 			$new_hash = $hash[1];
 
-			$object->set_values( array(
+			$object->set_values(array(
 				self::JSONLD_HASH => $new_hash,
-				self::SYNCED_GMT  => current_time( 'mysql', true ),
-			) );
+				self::SYNCED_GMT  => current_time('mysql', true),
+			));
 		}
 
 
 		/**
 		 * Allow 3rd parties to run additional sync work.
 		 */
-		do_action( 'wl_sync__sync_many', $hashes );
+		do_action('wl_sync__sync_many', $hashes);
 
 		return true;
 	}
@@ -208,23 +208,23 @@ 
 	 * @return false|string
 	 * @throws Exception
 	 */
-	private function get_payload_as_string( $object ) {
+	private function get_payload_as_string($object) {
 		$type             = $object->get_type();
 		$object_id        = $object->get_object_id();
-		$jsonld_as_string = wp_json_encode( apply_filters( 'wl_dataset__sync_service__sync_item__jsonld',
-			$this->jsonld_service->get( $type, $object_id ), $type, $object_id ), 64 ); // JSON_UNESCAPED_SLASHES
-		$uri              = $this->entity_service->get_uri( $object_id, $type );
+		$jsonld_as_string = wp_json_encode(apply_filters('wl_dataset__sync_service__sync_item__jsonld',
+			$this->jsonld_service->get($type, $object_id), $type, $object_id), 64); // JSON_UNESCAPED_SLASHES
+		$uri              = $this->entity_service->get_uri($object_id, $type);
 
 		// Entity URL isn't set, bail out.
-		if ( empty( $uri ) ) {
+		if (empty($uri)) {
 			return false;
 		}
 
-		return wp_json_encode( array(
+		return wp_json_encode(array(
 			'uri'     => $uri,
 			'model'   => $jsonld_as_string,
-			'private' => ! ( $object->is_public() && $object->is_published() ),
-		), 64 ); // JSON_UNESCAPED_SLASHES
+			'private' => ! ($object->is_public() && $object->is_published()),
+		), 64); // JSON_UNESCAPED_SLASHES
 	}
 
 	/**
@@ -232,17 +232,17 @@ 
 	 *
 	 * @todo Complete the delete item.
 	 */
-	public function delete_item( $post_id ) {
-		$uri = $this->entity_service->get_uri( $post_id, Object_Type_Enum::POST );
+	public function delete_item($post_id) {
+		$uri = $this->entity_service->get_uri($post_id, Object_Type_Enum::POST);
 
-		if ( ! isset( $uri ) ) {
+		if ( ! isset($uri)) {
 			return;
 		}
 
 		// Make a request to the remote endpoint.
 		$response = $this->api_service->request(
-			'DELETE', '/middleware/dataset?uri=' . rawurlencode( $uri ),
-			array( 'Content-Type' => 'application/ld+json', ) );
+			'DELETE', '/middleware/dataset?uri='.rawurlencode($uri),
+			array('Content-Type' => 'application/ld+json',) );
 
 	}
 
@@ -252,7 +252,7 @@ 
 	}
 
 	public function delete_all() {
-		$this->api_service->request( 'DELETE', '/middleware/dataset/all' );
+		$this->api_service->request('DELETE', '/middleware/dataset/all');
 	}
 
 }

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

Indentation +125 added lines, -125 removed lines

@@ -14,130 +14,130 @@
  */
 class Wordlift_Redirect_Service {
 
-	const LOD_ENDPOINT = 'http://www.lodview.it';
-	const LOD_HOST = 'www.lodview.it';
-
-	/**
-	 * The Log service.
-	 *
-	 * @since 3.2.0
-	 * @access private
-	 * @var \Wordlift_Log_Service $log_service The Log service.
-	 */
-	private $log_service;
-
-	/**
-	 * The Entity URI service.
-	 *
-	 * @since 3.2.0
-	 * @access private
-	 * @var Wordlift_Entity_Uri_Service $entity_uri_service The Entity service.
-	 */
-	private $entity_uri_service;
-
-	/**
-	 * A singleton instance of the Redirect service (useful for unit tests).
-	 *
-	 * @since 3.2.0
-	 * @access private
-	 * @var \Wordlift_Redirect_Service $instance The singleton instance.
-	 */
-	private static $instance;
-
-	/**
-	 * Create a Wordlift_Redirect_Service instance.
-	 *
-	 * @param Wordlift_Entity_Uri_Service $entity_uri_service The Entity service.
-	 *
-	 * @since 3.2.0
-	 *
-	 */
-	public function __construct( $entity_uri_service ) {
-
-		$this->log_service = Wordlift_Log_Service::get_logger( 'Wordlift_Redirect_Service' );
-
-		$this->entity_uri_service = $entity_uri_service;
-
-		self::$instance = $this;
-
-	}
-
-	/**
-	 * Get the singleton instance of the Wordlift_Redirect_Service
-	 *
-	 * @return \Wordlift_Redirect_Service The singleton instance of the Wordlift_Redirect_Service.
-	 * @since 3.2.0
-	 *
-	 */
-	public static function get_instance() {
-
-		return self::$instance;
-	}
-
-	/**
-	 * Perform redirect depending on entity uri and target
-	 *
-	 * @since 3.2.0
-	 */
-	public function ajax_redirect() {
-
-		// Check the `uri` parameter.
-		if ( ! ( $entity_uri = filter_input( INPUT_GET, 'uri', FILTER_VALIDATE_URL ) ) ) {
-			wp_die( __( 'Invalid URI.', 'wordlift' ), __( 'Invalid URI.', 'wordlift' ), array(
-				'response'  => 400,
-				'back_link' => true,
-			) );
-		}
-
-		// Check the `to` parameter.
-		if ( ! ( $target = filter_input( INPUT_GET, 'to' ) ) ) {
-			wp_die( __( 'Invalid `to` parameter.', 'wordlift' ), __( 'Invalid `to` parameter.', 'wordlift' ), array(
-				'response'  => 400,
-				'back_link' => true,
-			) );
-		}
-
-		/** @var Wordpress_Content $content */
-		$content = Wordpress_Content_Service::get_instance()
-		                                    ->get_by_entity_id_or_same_as( $entity_uri );
-
-		if ( ! isset( $content ) ) {
-			wp_die( __( 'Entity not found.', 'wordlift' ), __( 'Entity not found.', 'wordlift' ), array(
-				'response'  => 404,
-				'back_link' => true,
-			) );
-		}
-
-		switch ( $target ) {
-			case 'edit':
-				$redirect_url = $content->get_edit_link();
-				break;
-			case 'lod':
-				$redirect_url = self::LOD_ENDPOINT . '/lodview/?IRI=' . urlencode( $entity_uri );
-				break;
-			case 'permalink':
-				$redirect_url = $content->get_permalink();
-				break;
-			default:
-				wp_die( 'Unsupported redirect target.' );
-		}
-
-		// Perform the redirect
-		wp_safe_redirect( $redirect_url );
-		exit;
-	}
-
-	/**
-	 * Register custom allowed hosts.
-	 * @see https://developer.wordpress.org/reference/functions/wp_safe_redirect/
-	 *
-	 * @since 3.2.0
-	 *
-	 * @return array permalink.
-	 */
-	public function allowed_redirect_hosts( $content ) {
-
-		return array_merge( $content, array( self::LOD_HOST ) );
-	}
+    const LOD_ENDPOINT = 'http://www.lodview.it';
+    const LOD_HOST = 'www.lodview.it';
+
+    /**
+     * The Log service.
+     *
+     * @since 3.2.0
+     * @access private
+     * @var \Wordlift_Log_Service $log_service The Log service.
+     */
+    private $log_service;
+
+    /**
+     * The Entity URI service.
+     *
+     * @since 3.2.0
+     * @access private
+     * @var Wordlift_Entity_Uri_Service $entity_uri_service The Entity service.
+     */
+    private $entity_uri_service;
+
+    /**
+     * A singleton instance of the Redirect service (useful for unit tests).
+     *
+     * @since 3.2.0
+     * @access private
+     * @var \Wordlift_Redirect_Service $instance The singleton instance.
+     */
+    private static $instance;
+
+    /**
+     * Create a Wordlift_Redirect_Service instance.
+     *
+     * @param Wordlift_Entity_Uri_Service $entity_uri_service The Entity service.
+     *
+     * @since 3.2.0
+     *
+     */
+    public function __construct( $entity_uri_service ) {
+
+        $this->log_service = Wordlift_Log_Service::get_logger( 'Wordlift_Redirect_Service' );
+
+        $this->entity_uri_service = $entity_uri_service;
+
+        self::$instance = $this;
+
+    }
+
+    /**
+     * Get the singleton instance of the Wordlift_Redirect_Service
+     *
+     * @return \Wordlift_Redirect_Service The singleton instance of the Wordlift_Redirect_Service.
+     * @since 3.2.0
+     *
+     */
+    public static function get_instance() {
+
+        return self::$instance;
+    }
+
+    /**
+     * Perform redirect depending on entity uri and target
+     *
+     * @since 3.2.0
+     */
+    public function ajax_redirect() {
+
+        // Check the `uri` parameter.
+        if ( ! ( $entity_uri = filter_input( INPUT_GET, 'uri', FILTER_VALIDATE_URL ) ) ) {
+            wp_die( __( 'Invalid URI.', 'wordlift' ), __( 'Invalid URI.', 'wordlift' ), array(
+                'response'  => 400,
+                'back_link' => true,
+            ) );
+        }
+
+        // Check the `to` parameter.
+        if ( ! ( $target = filter_input( INPUT_GET, 'to' ) ) ) {
+            wp_die( __( 'Invalid `to` parameter.', 'wordlift' ), __( 'Invalid `to` parameter.', 'wordlift' ), array(
+                'response'  => 400,
+                'back_link' => true,
+            ) );
+        }
+
+        /** @var Wordpress_Content $content */
+        $content = Wordpress_Content_Service::get_instance()
+                                            ->get_by_entity_id_or_same_as( $entity_uri );
+
+        if ( ! isset( $content ) ) {
+            wp_die( __( 'Entity not found.', 'wordlift' ), __( 'Entity not found.', 'wordlift' ), array(
+                'response'  => 404,
+                'back_link' => true,
+            ) );
+        }
+
+        switch ( $target ) {
+            case 'edit':
+                $redirect_url = $content->get_edit_link();
+                break;
+            case 'lod':
+                $redirect_url = self::LOD_ENDPOINT . '/lodview/?IRI=' . urlencode( $entity_uri );
+                break;
+            case 'permalink':
+                $redirect_url = $content->get_permalink();
+                break;
+            default:
+                wp_die( 'Unsupported redirect target.' );
+        }
+
+        // Perform the redirect
+        wp_safe_redirect( $redirect_url );
+        exit;
+    }
+
+    /**
+     * Register custom allowed hosts.
+     * @see https://developer.wordpress.org/reference/functions/wp_safe_redirect/
+     *
+     * @since 3.2.0
+     *
+     * @return array permalink.
+     */
+    public function allowed_redirect_hosts( $content ) {
+
+        return array_merge( $content, array( self::LOD_HOST ) );
+    }
 
 }

Spacing +18 added lines, -18 removed lines

@@ -52,9 +52,9 @@ 
 	 * @since 3.2.0
 	 *
 	 */
-	public function __construct( $entity_uri_service ) {
+	public function __construct($entity_uri_service) {
 
-		$this->log_service = Wordlift_Log_Service::get_logger( 'Wordlift_Redirect_Service' );
+		$this->log_service = Wordlift_Log_Service::get_logger('Wordlift_Redirect_Service');
 
 		$this->entity_uri_service = $entity_uri_service;
 
@@ -82,48 +82,48 @@ 
 	public function ajax_redirect() {
 
 		// Check the `uri` parameter.
-		if ( ! ( $entity_uri = filter_input( INPUT_GET, 'uri', FILTER_VALIDATE_URL ) ) ) {
-			wp_die( __( 'Invalid URI.', 'wordlift' ), __( 'Invalid URI.', 'wordlift' ), array(
+		if ( ! ($entity_uri = filter_input(INPUT_GET, 'uri', FILTER_VALIDATE_URL))) {
+			wp_die(__('Invalid URI.', 'wordlift'), __('Invalid URI.', 'wordlift'), array(
 				'response'  => 400,
 				'back_link' => true,
-			) );
+			));
 		}
 
 		// Check the `to` parameter.
-		if ( ! ( $target = filter_input( INPUT_GET, 'to' ) ) ) {
-			wp_die( __( 'Invalid `to` parameter.', 'wordlift' ), __( 'Invalid `to` parameter.', 'wordlift' ), array(
+		if ( ! ($target = filter_input(INPUT_GET, 'to'))) {
+			wp_die(__('Invalid `to` parameter.', 'wordlift'), __('Invalid `to` parameter.', 'wordlift'), array(
 				'response'  => 400,
 				'back_link' => true,
-			) );
+			));
 		}
 
 		/** @var Wordpress_Content $content */
 		$content = Wordpress_Content_Service::get_instance()
-		                                    ->get_by_entity_id_or_same_as( $entity_uri );
+		                                    ->get_by_entity_id_or_same_as($entity_uri);
 
-		if ( ! isset( $content ) ) {
-			wp_die( __( 'Entity not found.', 'wordlift' ), __( 'Entity not found.', 'wordlift' ), array(
+		if ( ! isset($content)) {
+			wp_die(__('Entity not found.', 'wordlift'), __('Entity not found.', 'wordlift'), array(
 				'response'  => 404,
 				'back_link' => true,
-			) );
+			));
 		}
 
-		switch ( $target ) {
+		switch ($target) {
 			case 'edit':
 				$redirect_url = $content->get_edit_link();
 				break;
 			case 'lod':
-				$redirect_url = self::LOD_ENDPOINT . '/lodview/?IRI=' . urlencode( $entity_uri );
+				$redirect_url = self::LOD_ENDPOINT.'/lodview/?IRI='.urlencode($entity_uri);
 				break;
 			case 'permalink':
 				$redirect_url = $content->get_permalink();
 				break;
 			default:
-				wp_die( 'Unsupported redirect target.' );
+				wp_die('Unsupported redirect target.');
 		}
 
 		// Perform the redirect
-		wp_safe_redirect( $redirect_url );
+		wp_safe_redirect($redirect_url);
 		exit;
 	}
 
@@ -135,9 +135,9 @@ 
 	 *
 	 * @return array permalink.
 	 */
-	public function allowed_redirect_hosts( $content ) {
+	public function allowed_redirect_hosts($content) {
 
-		return array_merge( $content, array( self::LOD_HOST ) );
+		return array_merge($content, array(self::LOD_HOST));
 	}
 
 }

src/wordlift/content/wordpress/class-wordpress-content.php

Indentation +57 added lines, -57 removed lines

@@ -10,70 +10,70 @@
 
 class Wordpress_Content implements Content {
 
-	/**
-	 * The actual content.
-	 *
-	 * @var WP_Post|WP_Term|WP_User $bag
-	 */
-	private $bag;
+    /**
+     * The actual content.
+     *
+     * @var WP_Post|WP_Term|WP_User $bag
+     */
+    private $bag;
 
-	/**
-	 * Creates a WordPress content instance.
-	 *
-	 * @param WP_Post|WP_Term|WP_User $bag
-	 */
-	public function __construct( $bag ) {
-		$this->bag = $bag;
-	}
+    /**
+     * Creates a WordPress content instance.
+     *
+     * @param WP_Post|WP_Term|WP_User $bag
+     */
+    public function __construct( $bag ) {
+        $this->bag = $bag;
+    }
 
-	/**
-	 * The actual content.
-	 *
-	 * @return WP_Post|WP_Term|WP_User
-	 */
-	function get_bag() {
-		return $this->bag;
-	}
+    /**
+     * The actual content.
+     *
+     * @return WP_Post|WP_Term|WP_User
+     */
+    function get_bag() {
+        return $this->bag;
+    }
 
-	function get_id() {
-		switch ( get_class( $this->bag ) ) {
-			case 'WP_Post':
-			case 'WP_User':
-				return $this->bag->ID;
-			case 'WP_Term':
-				return $this->bag->term_id;
-		}
+    function get_id() {
+        switch ( get_class( $this->bag ) ) {
+            case 'WP_Post':
+            case 'WP_User':
+                return $this->bag->ID;
+            case 'WP_Term':
+                return $this->bag->term_id;
+        }
 
-		return null;
-	}
+        return null;
+    }
 
-	function get_object_type_enum() {
-		return Object_Type_Enum::from_wordpress_instance( $this->bag );
-	}
+    function get_object_type_enum() {
+        return Object_Type_Enum::from_wordpress_instance( $this->bag );
+    }
 
-	function get_permalink() {
-		switch ( get_class( $this->bag ) ) {
-			case 'WP_Post':
-				return get_permalink( $this->get_bag()->ID );
-			case 'WP_User':
-				return get_author_posts_url( $this->get_bag()->ID );
-			case 'WP_Term':
-				return get_term_link( $this->bag->term_id );
-		}
+    function get_permalink() {
+        switch ( get_class( $this->bag ) ) {
+            case 'WP_Post':
+                return get_permalink( $this->get_bag()->ID );
+            case 'WP_User':
+                return get_author_posts_url( $this->get_bag()->ID );
+            case 'WP_Term':
+                return get_term_link( $this->bag->term_id );
+        }
 
-		return null;
-	}
+        return null;
+    }
 
-	function get_edit_link() {
-		switch ( get_class( $this->bag ) ) {
-			case 'WP_Post':
-				return get_edit_post_link( $this->get_bag()->ID );
-			case 'WP_User':
-				return get_edit_user_link( $this->get_bag()->ID );
-			case 'WP_Term':
-				return get_edit_term_link( $this->bag->term_id );
-		}
+    function get_edit_link() {
+        switch ( get_class( $this->bag ) ) {
+            case 'WP_Post':
+                return get_edit_post_link( $this->get_bag()->ID );
+            case 'WP_User':
+                return get_edit_user_link( $this->get_bag()->ID );
+            case 'WP_Term':
+                return get_edit_term_link( $this->bag->term_id );
+        }
 
-		return null;
-	}
+        return null;
+    }
 }
\ No newline at end of file

Spacing +11 added lines, -11 removed lines

@@ -22,7 +22,7 @@ 
 	 *
 	 * @param WP_Post|WP_Term|WP_User $bag
 	 */
-	public function __construct( $bag ) {
+	public function __construct($bag) {
 		$this->bag = $bag;
 	}
 
@@ -36,7 +36,7 @@ 
 	}
 
 	function get_id() {
-		switch ( get_class( $this->bag ) ) {
+		switch (get_class($this->bag)) {
 			case 'WP_Post':
 			case 'WP_User':
 				return $this->bag->ID;
@@ -48,30 +48,30 @@ 
 	}
 
 	function get_object_type_enum() {
-		return Object_Type_Enum::from_wordpress_instance( $this->bag );
+		return Object_Type_Enum::from_wordpress_instance($this->bag);
 	}
 
 	function get_permalink() {
-		switch ( get_class( $this->bag ) ) {
+		switch (get_class($this->bag)) {
 			case 'WP_Post':
-				return get_permalink( $this->get_bag()->ID );
+				return get_permalink($this->get_bag()->ID);
 			case 'WP_User':
-				return get_author_posts_url( $this->get_bag()->ID );
+				return get_author_posts_url($this->get_bag()->ID);
 			case 'WP_Term':
-				return get_term_link( $this->bag->term_id );
+				return get_term_link($this->bag->term_id);
 		}
 
 		return null;
 	}
 
 	function get_edit_link() {
-		switch ( get_class( $this->bag ) ) {
+		switch (get_class($this->bag)) {
 			case 'WP_Post':
-				return get_edit_post_link( $this->get_bag()->ID );
+				return get_edit_post_link($this->get_bag()->ID);
 			case 'WP_User':
-				return get_edit_user_link( $this->get_bag()->ID );
+				return get_edit_user_link($this->get_bag()->ID);
 			case 'WP_Term':
-				return get_edit_term_link( $this->bag->term_id );
+				return get_edit_term_link($this->bag->term_id);
 		}
 
 		return null;

src/wordlift/content/class-content.php

Indentation +32 added lines, -32 removed lines

@@ -4,37 +4,37 @@
 
 interface Content {
 
-	/**
-	 * Get the actual content.
-	 *
-	 * @return mixed Get the actual content.
-	 */
-	function get_bag();
-
-	/**
-	 * Get the content id.
-	 *
-	 * @return mixed
-	 */
-	function get_id();
-
-	/**
-	 * Get the content type.
-	 *
-	 * @return mixed
-	 */
-	function get_object_type_enum();
-
-	/**
-	 * Get the permalink.
-	 * @return string
-	 */
-	function get_permalink();
-
-	/**
-	 * Get the edit link.
-	 * @return string
-	 */
-	function get_edit_link();
+    /**
+     * Get the actual content.
+     *
+     * @return mixed Get the actual content.
+     */
+    function get_bag();
+
+    /**
+     * Get the content id.
+     *
+     * @return mixed
+     */
+    function get_id();
+
+    /**
+     * Get the content type.
+     *
+     * @return mixed
+     */
+    function get_object_type_enum();
+
+    /**
+     * Get the permalink.
+     * @return string
+     */
+    function get_permalink();
+
+    /**
+     * Get the edit link.
+     * @return string
+     */
+    function get_edit_link();
 
 }

src/wordlift/link/class-object-link-provider.php

Indentation +113 added lines, -113 removed lines

@@ -11,119 +11,119 @@
 use Wordlift\Object_Type_Enum;
 
 class Object_Link_Provider extends Singleton {
-	/**
-	 * @var array<Link>
-	 */
-	private $link_providers;
-
-	/**
-	 * @return Object_Link_Provider
-	 */
-	public static function get_instance() {
-		return parent::get_instance();
-	}
-
-	public function __construct() {
-		parent::__construct();
-		$this->link_providers = array(
-			Object_Type_Enum::POST => Post_Link::get_instance(),
-			Object_Type_Enum::TERM => Term_Link::get_instance()
-		);
-	}
-
-	/**
-	 * @param $id int
-	 * @param $label_to_be_ignored string
-	 * @param $object_type int
-	 *
-	 * @return string
-	 */
-	public function get_link_title( $id, $label_to_be_ignored, $object_type ) {
-		$provider = $this->get_provider( $object_type );
-		if ( ! $provider ) {
-			return '';
-		}
-
-		return $provider->get_link_title( $id, $label_to_be_ignored );
-	}
-
-
-	/**
-	 * Return the object type by the entity uri.
-	 * @return int which can be any of the {@link Object_Type_Enum} values.
-	 */
-	public function get_object_type( $uri ) {
-
-		$link_providers = $this->link_providers;
-		foreach ( $link_providers as $type => $provider ) {
-			/**
-			 * @var $provider Link
-			 */
-			$id = $provider->get_id( $uri );
-			if ( $id ) {
-				return $type;
-			}
-		}
-
-		return Object_Type_Enum::UNKNOWN;
-	}
-
-	public function get_same_as_uris( $id, $object_type ) {
-
-		$provider = $this->get_provider( $object_type );
-		if ( ! $provider ) {
-			return array();
-		}
-
-		return $provider->get_same_as_uris( $id );
-	}
-
-	/**
-	 * @param $object_type
-	 *
-	 * @return mixed|Link
-	 */
-	private function get_provider( $object_type ) {
-
-		if ( ! array_key_exists( $object_type, $this->link_providers ) ) {
-			return false;
-		}
-
-		return $this->link_providers[ $object_type ];
-	}
-
-
-	public function get_permalink( $id, $object_type ) {
-		$provider = $this->get_provider( $object_type );
-		if ( ! $provider ) {
-			return false;
-		}
-
-		return $provider->get_permalink( $id );
-	}
-
-	/**
-	 * Return the edit term page link.
-	 *
-	 * @param $object_id
-	 * @param $uri
-	 *
-	 * @return string | false
-	 * @since 3.32.0
-	 */
-	public function get_edit_page_link( $object_id, $uri ) {
-
-		$object_type = $this->get_object_type( $uri );
-
-		$provider = $this->get_provider( $object_type );
-
-		if ( ! $provider ) {
-			return false;
-		}
-
-		return $provider->get_edit_page_link( $object_id );
-
-	}
+    /**
+     * @var array<Link>
+     */
+    private $link_providers;
+
+    /**
+     * @return Object_Link_Provider
+     */
+    public static function get_instance() {
+        return parent::get_instance();
+    }
+
+    public function __construct() {
+        parent::__construct();
+        $this->link_providers = array(
+            Object_Type_Enum::POST => Post_Link::get_instance(),
+            Object_Type_Enum::TERM => Term_Link::get_instance()
+        );
+    }
+
+    /**
+     * @param $id int
+     * @param $label_to_be_ignored string
+     * @param $object_type int
+     *
+     * @return string
+     */
+    public function get_link_title( $id, $label_to_be_ignored, $object_type ) {
+        $provider = $this->get_provider( $object_type );
+        if ( ! $provider ) {
+            return '';
+        }
+
+        return $provider->get_link_title( $id, $label_to_be_ignored );
+    }
+
+
+    /**
+     * Return the object type by the entity uri.
+     * @return int which can be any of the {@link Object_Type_Enum} values.
+     */
+    public function get_object_type( $uri ) {
+
+        $link_providers = $this->link_providers;
+        foreach ( $link_providers as $type => $provider ) {
+            /**
+             * @var $provider Link
+             */
+            $id = $provider->get_id( $uri );
+            if ( $id ) {
+                return $type;
+            }
+        }
+
+        return Object_Type_Enum::UNKNOWN;
+    }
+
+    public function get_same_as_uris( $id, $object_type ) {
+
+        $provider = $this->get_provider( $object_type );
+        if ( ! $provider ) {
+            return array();
+        }
+
+        return $provider->get_same_as_uris( $id );
+    }
+
+    /**
+     * @param $object_type
+     *
+     * @return mixed|Link
+     */
+    private function get_provider( $object_type ) {
+
+        if ( ! array_key_exists( $object_type, $this->link_providers ) ) {
+            return false;
+        }
+
+        return $this->link_providers[ $object_type ];
+    }
+
+
+    public function get_permalink( $id, $object_type ) {
+        $provider = $this->get_provider( $object_type );
+        if ( ! $provider ) {
+            return false;
+        }
+
+        return $provider->get_permalink( $id );
+    }
+
+    /**
+     * Return the edit term page link.
+     *
+     * @param $object_id
+     * @param $uri
+     *
+     * @return string | false
+     * @since 3.32.0
+     */
+    public function get_edit_page_link( $object_id, $uri ) {
+
+        $object_type = $this->get_object_type( $uri );
+
+        $provider = $this->get_provider( $object_type );
+
+        if ( ! $provider ) {
+            return false;
+        }
+
+        return $provider->get_edit_page_link( $object_id );
+
+    }
 
 
 }
\ No newline at end of file

Spacing +24 added lines, -24 removed lines

@@ -38,13 +38,13 @@ 
 	 *
 	 * @return string
 	 */
-	public function get_link_title( $id, $label_to_be_ignored, $object_type ) {
-		$provider = $this->get_provider( $object_type );
-		if ( ! $provider ) {
+	public function get_link_title($id, $label_to_be_ignored, $object_type) {
+		$provider = $this->get_provider($object_type);
+		if ( ! $provider) {
 			return '';
 		}
 
-		return $provider->get_link_title( $id, $label_to_be_ignored );
+		return $provider->get_link_title($id, $label_to_be_ignored);
 	}
 
 
@@ -52,15 +52,15 @@ 
 	 * Return the object type by the entity uri.
 	 * @return int which can be any of the {@link Object_Type_Enum} values.
 	 */
-	public function get_object_type( $uri ) {
+	public function get_object_type($uri) {
 
 		$link_providers = $this->link_providers;
-		foreach ( $link_providers as $type => $provider ) {
+		foreach ($link_providers as $type => $provider) {
 			/**
 			 * @var $provider Link
 			 */
-			$id = $provider->get_id( $uri );
-			if ( $id ) {
+			$id = $provider->get_id($uri);
+			if ($id) {
 				return $type;
 			}
 		}
@@ -68,14 +68,14 @@ 
 		return Object_Type_Enum::UNKNOWN;
 	}
 
-	public function get_same_as_uris( $id, $object_type ) {
+	public function get_same_as_uris($id, $object_type) {
 
-		$provider = $this->get_provider( $object_type );
-		if ( ! $provider ) {
+		$provider = $this->get_provider($object_type);
+		if ( ! $provider) {
 			return array();
 		}
 
-		return $provider->get_same_as_uris( $id );
+		return $provider->get_same_as_uris($id);
 	}
 
 	/**
@@ -83,23 +83,23 @@ 
 	 *
 	 * @return mixed|Link
 	 */
-	private function get_provider( $object_type ) {
+	private function get_provider($object_type) {
 
-		if ( ! array_key_exists( $object_type, $this->link_providers ) ) {
+		if ( ! array_key_exists($object_type, $this->link_providers)) {
 			return false;
 		}
 
-		return $this->link_providers[ $object_type ];
+		return $this->link_providers[$object_type];
 	}
 
 
-	public function get_permalink( $id, $object_type ) {
-		$provider = $this->get_provider( $object_type );
-		if ( ! $provider ) {
+	public function get_permalink($id, $object_type) {
+		$provider = $this->get_provider($object_type);
+		if ( ! $provider) {
 			return false;
 		}
 
-		return $provider->get_permalink( $id );
+		return $provider->get_permalink($id);
 	}
 
 	/**
@@ -111,17 +111,17 @@ 
 	 * @return string | false
 	 * @since 3.32.0
 	 */
-	public function get_edit_page_link( $object_id, $uri ) {
+	public function get_edit_page_link($object_id, $uri) {
 
-		$object_type = $this->get_object_type( $uri );
+		$object_type = $this->get_object_type($uri);
 
-		$provider = $this->get_provider( $object_type );
+		$provider = $this->get_provider($object_type);
 
-		if ( ! $provider ) {
+		if ( ! $provider) {
 			return false;
 		}
 
-		return $provider->get_edit_page_link( $object_id );
+		return $provider->get_edit_page_link($object_id);
 
 	}