insideout10 / wordlift-plugin

src/includes/class-wordlift-query-builder.php

Indentation +305 added lines, -305 removed lines

@@ -7,310 +7,310 @@
  */
 class Wordlift_Query_Builder {
 
-	/**
-	 * The INSERT statement template.
-	 *
-	 * @since 3.1.7
-	 */
-	const INSERT = 'INSERT DATA { %s };';
-
-	/**
-	 * The DELETE statement template (it repeats the statements in the WHERE clause.
-	 *
-	 * @since 3.1.7
-	 */
-	const DELETE = 'DELETE { %s } WHERE { %1$s };';
-
-	/**
-	 * Tell the statement function to guess the object type (URI, value or parameter).
-	 *
-	 * @since 3.1.7
-	 */
-	const OBJECT_AUTO = - 1;
-
-	/**
-	 * Tell the statement function that the object is a URI.
-	 *
-	 * @since 3.1.7
-	 */
-	const OBJECT_URI = 0;
-
-	/**
-	 * Tell the statement function that the object is a value.
-	 *
-	 * @since 3.1.7
-	 */
-	const OBJECT_VALUE = 1;
-
-	/**
-	 * Tell the statement function that the object is a parameter.
-	 *
-	 * @since 3.1.7
-	 */
-	const OBJECT_PARAMETER = 2;
-
-	/**
-	 * The RDFS type.
-	 *
-	 * @since 3.1.7
-	 */
-	const RDFS_TYPE_URI = 'http://www.w3.org/1999/02/22-rdf-syntax-ns#type';
-
-	/**
-	 * The schema.org/Person type.
-	 *
-	 * @since 3.1.7
-	 */
-	const SCHEMA_PERSON_URI = 'http://schema.org/Person';
-
-	/**
-	 * The schema.org given name predicate.
-	 *
-	 * @since 3.1.7
-	 */
-	const SCHEMA_GIVEN_NAME_URI = 'http://schema.org/givenName';
-
-	/**
-	 * The schema.org family name predicate.
-	 *
-	 * @since 3.1.7
-	 */
-	const SCHEMA_FAMILY_NAME_URI = 'http://schema.org/familyName';
-
-	/**
-	 * The schema.org url predicate.
-	 *
-	 * @since 3.1.7
-	 */
-	const SCHEMA_URL_URI = 'http://schema.org/url';
-
-	/**
-	 * @since 3.14.0
-	 */
-	const SCHEMA_IMAGE_URI = 'http://schema.org/image';
-
-	/**
-	 * The location created predicate.
-	 *
-	 * @since 3.14.0
-	 */
-	const SCHEMA_LOCATION_CREATED_URI = 'http://schema.org/locationCreated';
-
-	/**
-	 * @since 3.14.0
-	 */
-	const SCHEMA_AUTHOR_URI = 'http://schema.org/author';
-
-	/**
-	 * @since 3.14.0
-	 */
-	const SCHEMA_INTERACTION_COUNT_URI = 'http://schema.org/interactionCount';
-
-	/**
-	 * @since 3.14.0
-	 */
-	const DCTERMS_SUBJECT_URI = 'http://purl.org/dc/terms/subject';
-
-	/**
-	 * @since 3.14.0
-	 */
-	const DCTERMS_REFERENCES_URI = 'http://purl.org/dc/terms/references';
-
-	/**
-	 * The RDF label.
-	 *
-	 * @since 3.1.7
-	 */
-	const RDFS_LABEL_URI = 'http://www.w3.org/2000/01/rdf-schema#label';
-
-	/**
-	 * Hold the template (INSERT or DELETE).
-	 *
-	 * @since  3.1.7
-	 * @access private
-	 * @var string $template The query template.
-	 */
-	private $template;
-
-	/**
-	 * An array of statements (in the form of subject, predicate, object).
-	 *
-	 * @since  3.1.7
-	 * @access private
-	 * @var array $statements An array of statements.
-	 */
-	private $statements = array();
-
-	/**
-	 * Create a new instance of the Query builder (compatible with PHP 5.3).
-	 *
-	 * @since 3.1.7
-	 * @return Wordlift_Query_Builder A new instance of the Query builder.
-	 */
-	public static function new_instance() {
-
-		return new Wordlift_Query_Builder();
-	}
-
-	/**
-	 * Set the query to INSERT.
-	 *
-	 * @since 3.1.7
-	 * @return Wordlift_Query_Builder The Query builder.
-	 */
-	public function insert() {
-
-		$this->template = self::INSERT;
-
-		return $this;
-	}
-
-	/**
-	 * Set the query to DELETE.
-	 *
-	 * @since 3.1.7
-	 * @return $this \Wordlift_Query_Builder The Query builder.
-	 */
-	public function delete() {
-
-		$this->template = self::DELETE;
-
-		return $this;
-	}
-
-	/**
-	 * Set the query to SELECT.
-	 *
-	 * @since 3.12.2
-	 *
-	 * @param string $props The list of properties to read.
-	 *
-	 * @return $this \Wordlift_Query_Builder The Query builder.
-	 */
-	public function select( $props = '*' ) {
-
-		$this->template = "SELECT $props WHERE { %s }";
-
-		return $this;
-	}
-
-	/**
-	 * Add a statement.
-	 *
-	 * @since 3.1.7
-	 *
-	 * @param string      $subject     The subject of the statement (must be a URI).
-	 * @param string      $predicate   The predicate (must be a URI).
-	 * @param string      $object      The object, can be a URI or a value.
-	 * @param int         $object_type The object type, either a {@link OBJECT_URI} or a value {@link OBJECT_VALUE}. If set to {@link OBJECT_AUTO}, the Query builder will try to guess.
-	 * @param string|null $data_type   The data type (or null).
-	 * @param string|null $language    The language code (or null).
-	 *
-	 * @return $this \Wordlift_Query_Builder The Query builder.
-	 */
-	public function statement( $subject, $predicate, $object, $object_type = self::OBJECT_AUTO, $data_type = null, $language = null ) {
-
-		// If no value has been provided, we don't set any statement.
-		if ( empty( $object ) ) {
-			return $this;
-		}
-
-		// Guess the subject type.
-		$subject_value_type = $this->guess_subject_type( $subject );
-
-		// Get the object type if set, otherwise try to guess it.
-		$object_value_type = ( self::OBJECT_AUTO === $object_type ? $this->guess_object_type( $predicate, $object ) : $object_type );
-
-		// Prepare the statement template.
-		$template =
-			// Subject as a parameter, no `<`, `>`.
-			( self::OBJECT_PARAMETER === $subject_value_type ? '%1$s' : '<%1$s>' ) .
-			// Predicate.
-			' <%2$s> ' .
-			// Object.
-			( self::OBJECT_URI === $object_value_type ? '<%3$s>' :
-				( self::OBJECT_PARAMETER === $object_value_type ? '%3$s' :
-					// self::OBJECT_VALUE === $object_value_type
-					'"%3$s"' . ( isset( $data_type ) ? '^^%4$s' : '' ) . ( isset( $language ) ? '@%5$s' : '' ) ) );
-
-		// Escape the subject, predicate and object.
-		$escaped_subject   = Wordlift_Sparql_Service::escape_uri( $subject );
-		$escaped_predicate = Wordlift_Sparql_Service::escape_uri( $predicate );
-		$escaped_object    = ( self::OBJECT_URI === $object_value_type ? Wordlift_Sparql_Service::escape_uri( $object ) : Wordlift_Sparql_Service::escape( $object ) );
-
-		// Prepare the statement and add it to the list of statements.
-		$this->statements[] = sprintf( $template, $escaped_subject, $escaped_predicate, $escaped_object, $data_type, $language );
-
-		return $this;
-	}
-
-	/**
-	 * Build the query.
-	 *
-	 * @since 3.1.7
-	 * @return string The query string.
-	 */
-	public function build() {
-
-		// If there are no statements return an empty string.
-		if ( 0 === count( $this->statements ) ) {
-			return '';
-		}
-
-		return sprintf( $this->template, implode( ' . ', $this->statements ) ) . "\n";
-	}
-
-	/**
-	 * Guess the statement object type.
-	 *
-	 * @since 3.1.7
-	 *
-	 * @param string $predicate The predicate.
-	 * @param string $object    The object.
-	 *
-	 * @return int {@link Wordlift_Query_Builder::OBJECT_URI} if the Query builder thinks the object must be an URI, {@link Wordlift_Query_Builder::OBJECT_VALUE} otherwise.
-	 */
-	private function guess_object_type( $predicate, $object ) {
-
-		// If the object starts with a question mark, it's a parameter.
-		if ( 0 === strpos( $object, '?' ) ) {
-			return self::OBJECT_PARAMETER;
-		}
-
-		// Guess based on the predicate.
-		switch ( $predicate ) {
-
-			case self::DCTERMS_REFERENCES_URI:
-			case self::DCTERMS_SUBJECT_URI:
-			case self::RDFS_TYPE_URI:
-			case self::SCHEMA_AUTHOR_URI:
-			case self::SCHEMA_LOCATION_CREATED_URI:
-			case self::SCHEMA_URL_URI:
-			case self::SCHEMA_IMAGE_URI:
-				return self::OBJECT_URI;
-
-		}
-
-		return self::OBJECT_VALUE;
-	}
-
-	/**
-	 * Guess the subject type.
-	 *
-	 * @since 3.12.3
-	 *
-	 * @param string $subject The subject string.
-	 *
-	 * @return int {@link Wordlift_Query_Builder::OBJECT_PARAMETER} if the Query builder thinks the subject is a parameter (starts with ?), otherwise {@link Wordlift_Query_Builder::OBJECT_URI}.
-	 */
-	private function guess_subject_type( $subject ) {
-
-		// If the object starts with a question mark, it's a parameter.
-		if ( 0 === strpos( $subject, '?' ) ) {
-			return self::OBJECT_PARAMETER;
-		}
-
-		return self::OBJECT_URI;
-	}
+    /**
+     * The INSERT statement template.
+     *
+     * @since 3.1.7
+     */
+    const INSERT = 'INSERT DATA { %s };';
+
+    /**
+     * The DELETE statement template (it repeats the statements in the WHERE clause.
+     *
+     * @since 3.1.7
+     */
+    const DELETE = 'DELETE { %s } WHERE { %1$s };';
+
+    /**
+     * Tell the statement function to guess the object type (URI, value or parameter).
+     *
+     * @since 3.1.7
+     */
+    const OBJECT_AUTO = - 1;
+
+    /**
+     * Tell the statement function that the object is a URI.
+     *
+     * @since 3.1.7
+     */
+    const OBJECT_URI = 0;
+
+    /**
+     * Tell the statement function that the object is a value.
+     *
+     * @since 3.1.7
+     */
+    const OBJECT_VALUE = 1;
+
+    /**
+     * Tell the statement function that the object is a parameter.
+     *
+     * @since 3.1.7
+     */
+    const OBJECT_PARAMETER = 2;
+
+    /**
+     * The RDFS type.
+     *
+     * @since 3.1.7
+     */
+    const RDFS_TYPE_URI = 'http://www.w3.org/1999/02/22-rdf-syntax-ns#type';
+
+    /**
+     * The schema.org/Person type.
+     *
+     * @since 3.1.7
+     */
+    const SCHEMA_PERSON_URI = 'http://schema.org/Person';
+
+    /**
+     * The schema.org given name predicate.
+     *
+     * @since 3.1.7
+     */
+    const SCHEMA_GIVEN_NAME_URI = 'http://schema.org/givenName';
+
+    /**
+     * The schema.org family name predicate.
+     *
+     * @since 3.1.7
+     */
+    const SCHEMA_FAMILY_NAME_URI = 'http://schema.org/familyName';
+
+    /**
+     * The schema.org url predicate.
+     *
+     * @since 3.1.7
+     */
+    const SCHEMA_URL_URI = 'http://schema.org/url';
+
+    /**
+     * @since 3.14.0
+     */
+    const SCHEMA_IMAGE_URI = 'http://schema.org/image';
+
+    /**
+     * The location created predicate.
+     *
+     * @since 3.14.0
+     */
+    const SCHEMA_LOCATION_CREATED_URI = 'http://schema.org/locationCreated';
+
+    /**
+     * @since 3.14.0
+     */
+    const SCHEMA_AUTHOR_URI = 'http://schema.org/author';
+
+    /**
+     * @since 3.14.0
+     */
+    const SCHEMA_INTERACTION_COUNT_URI = 'http://schema.org/interactionCount';
+
+    /**
+     * @since 3.14.0
+     */
+    const DCTERMS_SUBJECT_URI = 'http://purl.org/dc/terms/subject';
+
+    /**
+     * @since 3.14.0
+     */
+    const DCTERMS_REFERENCES_URI = 'http://purl.org/dc/terms/references';
+
+    /**
+     * The RDF label.
+     *
+     * @since 3.1.7
+     */
+    const RDFS_LABEL_URI = 'http://www.w3.org/2000/01/rdf-schema#label';
+
+    /**
+     * Hold the template (INSERT or DELETE).
+     *
+     * @since  3.1.7
+     * @access private
+     * @var string $template The query template.
+     */
+    private $template;
+
+    /**
+     * An array of statements (in the form of subject, predicate, object).
+     *
+     * @since  3.1.7
+     * @access private
+     * @var array $statements An array of statements.
+     */
+    private $statements = array();
+
+    /**
+     * Create a new instance of the Query builder (compatible with PHP 5.3).
+     *
+     * @since 3.1.7
+     * @return Wordlift_Query_Builder A new instance of the Query builder.
+     */
+    public static function new_instance() {
+
+        return new Wordlift_Query_Builder();
+    }
+
+    /**
+     * Set the query to INSERT.
+     *
+     * @since 3.1.7
+     * @return Wordlift_Query_Builder The Query builder.
+     */
+    public function insert() {
+
+        $this->template = self::INSERT;
+
+        return $this;
+    }
+
+    /**
+     * Set the query to DELETE.
+     *
+     * @since 3.1.7
+     * @return $this \Wordlift_Query_Builder The Query builder.
+     */
+    public function delete() {
+
+        $this->template = self::DELETE;
+
+        return $this;
+    }
+
+    /**
+     * Set the query to SELECT.
+     *
+     * @since 3.12.2
+     *
+     * @param string $props The list of properties to read.
+     *
+     * @return $this \Wordlift_Query_Builder The Query builder.
+     */
+    public function select( $props = '*' ) {
+
+        $this->template = "SELECT $props WHERE { %s }";
+
+        return $this;
+    }
+
+    /**
+     * Add a statement.
+     *
+     * @since 3.1.7
+     *
+     * @param string      $subject     The subject of the statement (must be a URI).
+     * @param string      $predicate   The predicate (must be a URI).
+     * @param string      $object      The object, can be a URI or a value.
+     * @param int         $object_type The object type, either a {@link OBJECT_URI} or a value {@link OBJECT_VALUE}. If set to {@link OBJECT_AUTO}, the Query builder will try to guess.
+     * @param string|null $data_type   The data type (or null).
+     * @param string|null $language    The language code (or null).
+     *
+     * @return $this \Wordlift_Query_Builder The Query builder.
+     */
+    public function statement( $subject, $predicate, $object, $object_type = self::OBJECT_AUTO, $data_type = null, $language = null ) {
+
+        // If no value has been provided, we don't set any statement.
+        if ( empty( $object ) ) {
+            return $this;
+        }
+
+        // Guess the subject type.
+        $subject_value_type = $this->guess_subject_type( $subject );
+
+        // Get the object type if set, otherwise try to guess it.
+        $object_value_type = ( self::OBJECT_AUTO === $object_type ? $this->guess_object_type( $predicate, $object ) : $object_type );
+
+        // Prepare the statement template.
+        $template =
+            // Subject as a parameter, no `<`, `>`.
+            ( self::OBJECT_PARAMETER === $subject_value_type ? '%1$s' : '<%1$s>' ) .
+            // Predicate.
+            ' <%2$s> ' .
+            // Object.
+            ( self::OBJECT_URI === $object_value_type ? '<%3$s>' :
+                ( self::OBJECT_PARAMETER === $object_value_type ? '%3$s' :
+                    // self::OBJECT_VALUE === $object_value_type
+                    '"%3$s"' . ( isset( $data_type ) ? '^^%4$s' : '' ) . ( isset( $language ) ? '@%5$s' : '' ) ) );
+
+        // Escape the subject, predicate and object.
+        $escaped_subject   = Wordlift_Sparql_Service::escape_uri( $subject );
+        $escaped_predicate = Wordlift_Sparql_Service::escape_uri( $predicate );
+        $escaped_object    = ( self::OBJECT_URI === $object_value_type ? Wordlift_Sparql_Service::escape_uri( $object ) : Wordlift_Sparql_Service::escape( $object ) );
+
+        // Prepare the statement and add it to the list of statements.
+        $this->statements[] = sprintf( $template, $escaped_subject, $escaped_predicate, $escaped_object, $data_type, $language );
+
+        return $this;
+    }
+
+    /**
+     * Build the query.
+     *
+     * @since 3.1.7
+     * @return string The query string.
+     */
+    public function build() {
+
+        // If there are no statements return an empty string.
+        if ( 0 === count( $this->statements ) ) {
+            return '';
+        }
+
+        return sprintf( $this->template, implode( ' . ', $this->statements ) ) . "\n";
+    }
+
+    /**
+     * Guess the statement object type.
+     *
+     * @since 3.1.7
+     *
+     * @param string $predicate The predicate.
+     * @param string $object    The object.
+     *
+     * @return int {@link Wordlift_Query_Builder::OBJECT_URI} if the Query builder thinks the object must be an URI, {@link Wordlift_Query_Builder::OBJECT_VALUE} otherwise.
+     */
+    private function guess_object_type( $predicate, $object ) {
+
+        // If the object starts with a question mark, it's a parameter.
+        if ( 0 === strpos( $object, '?' ) ) {
+            return self::OBJECT_PARAMETER;
+        }
+
+        // Guess based on the predicate.
+        switch ( $predicate ) {
+
+            case self::DCTERMS_REFERENCES_URI:
+            case self::DCTERMS_SUBJECT_URI:
+            case self::RDFS_TYPE_URI:
+            case self::SCHEMA_AUTHOR_URI:
+            case self::SCHEMA_LOCATION_CREATED_URI:
+            case self::SCHEMA_URL_URI:
+            case self::SCHEMA_IMAGE_URI:
+                return self::OBJECT_URI;
+
+        }
+
+        return self::OBJECT_VALUE;
+    }
+
+    /**
+     * Guess the subject type.
+     *
+     * @since 3.12.3
+     *
+     * @param string $subject The subject string.
+     *
+     * @return int {@link Wordlift_Query_Builder::OBJECT_PARAMETER} if the Query builder thinks the subject is a parameter (starts with ?), otherwise {@link Wordlift_Query_Builder::OBJECT_URI}.
+     */
+    private function guess_subject_type( $subject ) {
+
+        // If the object starts with a question mark, it's a parameter.
+        if ( 0 === strpos( $subject, '?' ) ) {
+            return self::OBJECT_PARAMETER;
+        }
+
+        return self::OBJECT_URI;
+    }
 
 }

Spacing +20 added lines, -22 removed lines

@@ -187,7 +187,7 @@ 
 	 *
 	 * @return $this \Wordlift_Query_Builder The Query builder.
 	 */
-	public function select( $props = '*' ) {
+	public function select($props = '*') {
 
 		$this->template = "SELECT $props WHERE { %s }";
 
@@ -208,38 +208,36 @@ 
 	 *
 	 * @return $this \Wordlift_Query_Builder The Query builder.
 	 */
-	public function statement( $subject, $predicate, $object, $object_type = self::OBJECT_AUTO, $data_type = null, $language = null ) {
+	public function statement($subject, $predicate, $object, $object_type = self::OBJECT_AUTO, $data_type = null, $language = null) {
 
 		// If no value has been provided, we don't set any statement.
-		if ( empty( $object ) ) {
+		if (empty($object)) {
 			return $this;
 		}
 
 		// Guess the subject type.
-		$subject_value_type = $this->guess_subject_type( $subject );
+		$subject_value_type = $this->guess_subject_type($subject);
 
 		// Get the object type if set, otherwise try to guess it.
-		$object_value_type = ( self::OBJECT_AUTO === $object_type ? $this->guess_object_type( $predicate, $object ) : $object_type );
+		$object_value_type = (self::OBJECT_AUTO === $object_type ? $this->guess_object_type($predicate, $object) : $object_type);
 
 		// Prepare the statement template.
 		$template =
 			// Subject as a parameter, no `<`, `>`.
-			( self::OBJECT_PARAMETER === $subject_value_type ? '%1$s' : '<%1$s>' ) .
+			(self::OBJECT_PARAMETER === $subject_value_type ? '%1$s' : '<%1$s>').
 			// Predicate.
-			' <%2$s> ' .
+			' <%2$s> '.
 			// Object.
-			( self::OBJECT_URI === $object_value_type ? '<%3$s>' :
-				( self::OBJECT_PARAMETER === $object_value_type ? '%3$s' :
-					// self::OBJECT_VALUE === $object_value_type
-					'"%3$s"' . ( isset( $data_type ) ? '^^%4$s' : '' ) . ( isset( $language ) ? '@%5$s' : '' ) ) );
+			(self::OBJECT_URI === $object_value_type ? '<%3$s>' : (self::OBJECT_PARAMETER === $object_value_type ? '%3$s' : // self::OBJECT_VALUE === $object_value_type
+					'"%3$s"'.(isset($data_type) ? '^^%4$s' : '').(isset($language) ? '@%5$s' : '')));
 
 		// Escape the subject, predicate and object.
-		$escaped_subject   = Wordlift_Sparql_Service::escape_uri( $subject );
-		$escaped_predicate = Wordlift_Sparql_Service::escape_uri( $predicate );
-		$escaped_object    = ( self::OBJECT_URI === $object_value_type ? Wordlift_Sparql_Service::escape_uri( $object ) : Wordlift_Sparql_Service::escape( $object ) );
+		$escaped_subject   = Wordlift_Sparql_Service::escape_uri($subject);
+		$escaped_predicate = Wordlift_Sparql_Service::escape_uri($predicate);
+		$escaped_object    = (self::OBJECT_URI === $object_value_type ? Wordlift_Sparql_Service::escape_uri($object) : Wordlift_Sparql_Service::escape($object));
 
 		// Prepare the statement and add it to the list of statements.
-		$this->statements[] = sprintf( $template, $escaped_subject, $escaped_predicate, $escaped_object, $data_type, $language );
+		$this->statements[] = sprintf($template, $escaped_subject, $escaped_predicate, $escaped_object, $data_type, $language);
 
 		return $this;
 	}
@@ -253,11 +251,11 @@ 
 	public function build() {
 
 		// If there are no statements return an empty string.
-		if ( 0 === count( $this->statements ) ) {
+		if (0 === count($this->statements)) {
 			return '';
 		}
 
-		return sprintf( $this->template, implode( ' . ', $this->statements ) ) . "\n";
+		return sprintf($this->template, implode(' . ', $this->statements))."\n";
 	}
 
 	/**
@@ -270,15 +268,15 @@ 
 	 *
 	 * @return int {@link Wordlift_Query_Builder::OBJECT_URI} if the Query builder thinks the object must be an URI, {@link Wordlift_Query_Builder::OBJECT_VALUE} otherwise.
 	 */
-	private function guess_object_type( $predicate, $object ) {
+	private function guess_object_type($predicate, $object) {
 
 		// If the object starts with a question mark, it's a parameter.
-		if ( 0 === strpos( $object, '?' ) ) {
+		if (0 === strpos($object, '?')) {
 			return self::OBJECT_PARAMETER;
 		}
 
 		// Guess based on the predicate.
-		switch ( $predicate ) {
+		switch ($predicate) {
 
 			case self::DCTERMS_REFERENCES_URI:
 			case self::DCTERMS_SUBJECT_URI:
@@ -303,10 +301,10 @@ 
 	 *
 	 * @return int {@link Wordlift_Query_Builder::OBJECT_PARAMETER} if the Query builder thinks the subject is a parameter (starts with ?), otherwise {@link Wordlift_Query_Builder::OBJECT_URI}.
 	 */
-	private function guess_subject_type( $subject ) {
+	private function guess_subject_type($subject) {
 
 		// If the object starts with a question mark, it's a parameter.
-		if ( 0 === strpos( $subject, '?' ) ) {
+		if (0 === strpos($subject, '?')) {
 			return self::OBJECT_PARAMETER;
 		}
 

src/admin/wordlift_admin_edit_post.php

Indentation +16 added lines, -16 removed lines

@@ -16,23 +16,23 @@
  */
 function wl_admin_permalink_html( $html, $post_id, $new_title, $new_slug ) {
 
-	// If the post is published, add the button to view Redlink's linked data.
-	if ( 'publish' == get_post_status( $post_id ) ) {
-		if ( $uri = wl_get_entity_uri( $post_id ) ) {
-			$uri_esc       = esc_attr( wl_get_entity_uri( $post_id ) );
-			$lod_view_href = 'http://lodview.it/lodview/?IRI=' . $uri_esc;
-			$html .= "<span id='view-post-btn'><a href='$lod_view_href' class='button button-small wl-button' target='_blank'>" .
-			         esc_html__( 'View Linked Data', 'wordlift' ) .
-			         "</a></span>\n";
-		}
-		$html .= "<span id='view-post-btn'><a href='" . WL_CONFIG_TEST_GOOGLE_RICH_SNIPPETS_URL .
-		         urlencode( get_permalink( $post_id ) ) .
-		         "' class='button button-small wl-button' target='_blank'>" .
-		         esc_html__( 'Test Google Rich Snippets', 'wordlift' ) .
-		         "</a></span>\n";
-	}
+    // If the post is published, add the button to view Redlink's linked data.
+    if ( 'publish' == get_post_status( $post_id ) ) {
+        if ( $uri = wl_get_entity_uri( $post_id ) ) {
+            $uri_esc       = esc_attr( wl_get_entity_uri( $post_id ) );
+            $lod_view_href = 'http://lodview.it/lodview/?IRI=' . $uri_esc;
+            $html .= "<span id='view-post-btn'><a href='$lod_view_href' class='button button-small wl-button' target='_blank'>" .
+                        esc_html__( 'View Linked Data', 'wordlift' ) .
+                        "</a></span>\n";
+        }
+        $html .= "<span id='view-post-btn'><a href='" . WL_CONFIG_TEST_GOOGLE_RICH_SNIPPETS_URL .
+                    urlencode( get_permalink( $post_id ) ) .
+                    "' class='button button-small wl-button' target='_blank'>" .
+                    esc_html__( 'Test Google Rich Snippets', 'wordlift' ) .
+                    "</a></span>\n";
+    }
 
-	return $html;
+    return $html;
 }
 
 add_filter( 'get_sample_permalink_html', 'wl_admin_permalink_html', PHP_INT_MAX, 4 );

Spacing +12 added lines, -12 removed lines

@@ -14,25 +14,25 @@
  *
  * @return The enhanced html.
  */
-function wl_admin_permalink_html( $html, $post_id, $new_title, $new_slug ) {
+function wl_admin_permalink_html($html, $post_id, $new_title, $new_slug) {
 
 	// If the post is published, add the button to view Redlink's linked data.
-	if ( 'publish' == get_post_status( $post_id ) ) {
-		if ( $uri = wl_get_entity_uri( $post_id ) ) {
-			$uri_esc       = esc_attr( wl_get_entity_uri( $post_id ) );
-			$lod_view_href = 'http://lodview.it/lodview/?IRI=' . $uri_esc;
-			$html .= "<span id='view-post-btn'><a href='$lod_view_href' class='button button-small wl-button' target='_blank'>" .
-			         esc_html__( 'View Linked Data', 'wordlift' ) .
+	if ('publish' == get_post_status($post_id)) {
+		if ($uri = wl_get_entity_uri($post_id)) {
+			$uri_esc       = esc_attr(wl_get_entity_uri($post_id));
+			$lod_view_href = 'http://lodview.it/lodview/?IRI='.$uri_esc;
+			$html .= "<span id='view-post-btn'><a href='$lod_view_href' class='button button-small wl-button' target='_blank'>".
+			         esc_html__('View Linked Data', 'wordlift').
 			         "</a></span>\n";
 		}
-		$html .= "<span id='view-post-btn'><a href='" . WL_CONFIG_TEST_GOOGLE_RICH_SNIPPETS_URL .
-		         urlencode( get_permalink( $post_id ) ) .
-		         "' class='button button-small wl-button' target='_blank'>" .
-		         esc_html__( 'Test Google Rich Snippets', 'wordlift' ) .
+		$html .= "<span id='view-post-btn'><a href='".WL_CONFIG_TEST_GOOGLE_RICH_SNIPPETS_URL.
+		         urlencode(get_permalink($post_id)).
+		         "' class='button button-small wl-button' target='_blank'>".
+		         esc_html__('Test Google Rich Snippets', 'wordlift').
 		         "</a></span>\n";
 	}
 
 	return $html;
 }
 
-add_filter( 'get_sample_permalink_html', 'wl_admin_permalink_html', PHP_INT_MAX, 4 );
+add_filter('get_sample_permalink_html', 'wl_admin_permalink_html', PHP_INT_MAX, 4);