insideout10 / wordlift-plugin

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);
 
 	}