insideout10 / wordlift-plugin

src/includes/linked-data/rendition/intf-wordlift-sparql-tuple-rendition.php

Indentation +20 added lines, -20 removed lines

@@ -11,25 +11,25 @@
  * Define the {@link Wordlift_Sparql_Tuple_Rendition} interface.
  */
 interface Wordlift_Sparql_Tuple_Rendition {
-	/**
-	 * Get tuple representations for the specified {@link WP_Post}.
-	 *
-	 * @since 3.18.0
-	 *
-	 * @param int $post_id The {@link WP_Post}'s id.
-	 *
-	 * @return array An array of triples.
-	 */
-	public function get_insert_triples( $post_id );
+    /**
+     * Get tuple representations for the specified {@link WP_Post}.
+     *
+     * @since 3.18.0
+     *
+     * @param int $post_id The {@link WP_Post}'s id.
+     *
+     * @return array An array of triples.
+     */
+    public function get_insert_triples( $post_id );
 
-	/**
-	 * Get delete statement for current post uri.
-	 *
-	 * @since 3.18.0
-	 *
-	 * @param int $post_id The post id.
-	 *
-	 * @return array An array of delete triples (`<...> <...> ?o` and `?s <...> <...>`).
-	 */
-	public function get_delete_triples( $post_id );
+    /**
+     * Get delete statement for current post uri.
+     *
+     * @since 3.18.0
+     *
+     * @param int $post_id The post id.
+     *
+     * @return array An array of delete triples (`<...> <...> ?o` and `?s <...> <...>`).
+     */
+    public function get_delete_triples( $post_id );
 }

Spacing +2 added lines, -2 removed lines

@@ -20,7 +20,7 @@ 
 	 *
 	 * @return array An array of triples.
 	 */
-	public function get_insert_triples( $post_id );
+	public function get_insert_triples($post_id);
 
 	/**
 	 * Get delete statement for current post uri.
@@ -31,5 +31,5 @@ 
 	 *
 	 * @return array An array of delete triples (`<...> <...> ?o` and `?s <...> <...>`).
 	 */
-	public function get_delete_triples( $post_id );
+	public function get_delete_triples($post_id);
 }

src/includes/linked-data/rendition/class-wordlift-default-sparql-tuple-rendition.php

Indentation +141 added lines, -141 removed lines

@@ -19,145 +19,145 @@
  */
 class Wordlift_Default_Sparql_Tuple_Rendition implements Wordlift_Sparql_Tuple_Rendition {
 
-	/**
-	 * A {@link Wordlift_Storage} instance to read a property.
-	 *
-	 * @since  3.15.0
-	 * @access private
-	 * @var \Wordlift_Storage $storage A {@link Wordlift_Storage} instance to
-	 *                                 read a property.
-	 */
-	private $storage;
-
-	/**
-	 * The data type (or null if not set).
-	 *
-	 * @since  3.15.0
-	 * @access private
-	 * @var string|null $data_type The data type (or null if not set).
-	 */
-	private $data_type;
-
-	/**
-	 * The language (or null if not set).
-	 *
-	 * @since  3.15.0
-	 * @access private
-	 * @var string|null $language The language (or null if not set).
-	 */
-	private $language;
-
-	/**
-	 * The {@link Wordlift_Entity_Service} instance.
-	 *
-	 * @since  3.15.0
-	 * @access private
-	 * @var \Wordlift_Entity_Service $entity_service The {@link Wordlift_Entity_Service} instance.
-	 */
-	private $entity_service;
-
-	/**
-	 * The predicate URI.
-	 *
-	 * @since  3.15.0
-	 * @access private
-	 * @var string $predicate The predicate URI.
-	 */
-	private $predicate;
-
-	/**
-	 * The URI suffix.
-	 *
-	 * @since 3.18.0
-	 * @access private
-	 * @var null|string $uri_suffix The URI suffix or null if no suffix is required.
-	 */
-	private $uri_suffix;
-
-	/**
-	 * Create a {@link Wordlift_Sparql_Tuple_Rendition} instance.
-	 *
-	 * @since 3.15.0
-	 *
-	 * @param \Wordlift_Entity_Service $entity_service The {@link Wordlift_Entity_Service}
-	 *                                                 instance.
-	 * @param \Wordlift_Storage        $storage        The {@link Wordlift_Storage}
-	 *                                                 instance.
-	 * @param string                   $predicate      The predicate URI.
-	 * @param string|null              $data_type      The data type or null.
-	 * @param string|null              $language       The language code or null.
-	 * @param string|null              $uri_suffix     The URI suffix, used for example for 2nd level entities, like location.
-	 */
-	public function __construct( $entity_service, $storage, $predicate, $data_type = null, $language = null, $uri_suffix = null ) {
-
-		$this->entity_service = $entity_service;
-		$this->storage        = $storage;
-		$this->predicate      = $predicate;
-		$this->data_type      = $data_type;
-		$this->language       = $language;
-		$this->uri_suffix     = ( ! is_null( $uri_suffix ) ) ? $uri_suffix : '';
-
-	}
-
-	/**
-	 * Get tuple representations for the specified {@link WP_Post}.
-	 *
-	 * @since 3.15.0
-	 *
-	 * @param int $post_id The {@link WP_Post}'s id.
-	 *
-	 * @return array An array of tuples.
-	 */
-	public function get_insert_triples( $post_id ) {
-		// Get the entity URI.
-		$uri = $this->entity_service->get_uri( $post_id ) . $this->uri_suffix;
-
-		// Get the predicate, data type and language.
-		$predicate = $this->predicate;
-		$data_type = $this->data_type;
-		$language  = $this->language;
-
-		// Filter out empty values.
-		$values = array_filter( (array) $this->storage->get( $post_id ), function ( $item ) {
-			return ! empty( $item );
-		} );
-
-		// Map the values to tuples.
-		return array_map( function ( $item ) use ( $uri, $predicate, $data_type, $language ) {
-
-			return sprintf( '<%s> <%s> %s . ',
-				Wordlift_Sparql_Service::escape_uri( $uri ),
-				Wordlift_Sparql_Service::escape_uri( $predicate ),
-				Wordlift_Sparql_Service::format( $item, $data_type, $language )
-			);
-		}, $values );
-	}
-
-	/**
-	 * Get the delete statement for current post id.
-	 *
-	 * @since 3.18.0
-	 *
-	 * @param int $post_id The post id.
-	 *
-	 * @return array An array containing delete statements for both
-	 *               the uri as subject and object.
-	 */
-	public function get_delete_triples( $post_id ) {
-		// Get the entity URI.
-		$uri = $this->entity_service->get_uri( $post_id ) . $this->uri_suffix;
-
-		return array(
-			// As object.
-			sprintf( '<%s> <%s> ?o',
-				Wordlift_Sparql_Service::escape_uri( $uri ),
-				Wordlift_Sparql_Service::escape_uri( $this->predicate )
-			),
-			// As subject.
-			sprintf( '?s <%s> <%s>',
-				Wordlift_Sparql_Service::escape_uri( $this->predicate ),
-				Wordlift_Sparql_Service::escape_uri( $uri )
-			),
-		);
-	}
+    /**
+     * A {@link Wordlift_Storage} instance to read a property.
+     *
+     * @since  3.15.0
+     * @access private
+     * @var \Wordlift_Storage $storage A {@link Wordlift_Storage} instance to
+     *                                 read a property.
+     */
+    private $storage;
+
+    /**
+     * The data type (or null if not set).
+     *
+     * @since  3.15.0
+     * @access private
+     * @var string|null $data_type The data type (or null if not set).
+     */
+    private $data_type;
+
+    /**
+     * The language (or null if not set).
+     *
+     * @since  3.15.0
+     * @access private
+     * @var string|null $language The language (or null if not set).
+     */
+    private $language;
+
+    /**
+     * The {@link Wordlift_Entity_Service} instance.
+     *
+     * @since  3.15.0
+     * @access private
+     * @var \Wordlift_Entity_Service $entity_service The {@link Wordlift_Entity_Service} instance.
+     */
+    private $entity_service;
+
+    /**
+     * The predicate URI.
+     *
+     * @since  3.15.0
+     * @access private
+     * @var string $predicate The predicate URI.
+     */
+    private $predicate;
+
+    /**
+     * The URI suffix.
+     *
+     * @since 3.18.0
+     * @access private
+     * @var null|string $uri_suffix The URI suffix or null if no suffix is required.
+     */
+    private $uri_suffix;
+
+    /**
+     * Create a {@link Wordlift_Sparql_Tuple_Rendition} instance.
+     *
+     * @since 3.15.0
+     *
+     * @param \Wordlift_Entity_Service $entity_service The {@link Wordlift_Entity_Service}
+     *                                                 instance.
+     * @param \Wordlift_Storage        $storage        The {@link Wordlift_Storage}
+     *                                                 instance.
+     * @param string                   $predicate      The predicate URI.
+     * @param string|null              $data_type      The data type or null.
+     * @param string|null              $language       The language code or null.
+     * @param string|null              $uri_suffix     The URI suffix, used for example for 2nd level entities, like location.
+     */
+    public function __construct( $entity_service, $storage, $predicate, $data_type = null, $language = null, $uri_suffix = null ) {
+
+        $this->entity_service = $entity_service;
+        $this->storage        = $storage;
+        $this->predicate      = $predicate;
+        $this->data_type      = $data_type;
+        $this->language       = $language;
+        $this->uri_suffix     = ( ! is_null( $uri_suffix ) ) ? $uri_suffix : '';
+
+    }
+
+    /**
+     * Get tuple representations for the specified {@link WP_Post}.
+     *
+     * @since 3.15.0
+     *
+     * @param int $post_id The {@link WP_Post}'s id.
+     *
+     * @return array An array of tuples.
+     */
+    public function get_insert_triples( $post_id ) {
+        // Get the entity URI.
+        $uri = $this->entity_service->get_uri( $post_id ) . $this->uri_suffix;
+
+        // Get the predicate, data type and language.
+        $predicate = $this->predicate;
+        $data_type = $this->data_type;
+        $language  = $this->language;
+
+        // Filter out empty values.
+        $values = array_filter( (array) $this->storage->get( $post_id ), function ( $item ) {
+            return ! empty( $item );
+        } );
+
+        // Map the values to tuples.
+        return array_map( function ( $item ) use ( $uri, $predicate, $data_type, $language ) {
+
+            return sprintf( '<%s> <%s> %s . ',
+                Wordlift_Sparql_Service::escape_uri( $uri ),
+                Wordlift_Sparql_Service::escape_uri( $predicate ),
+                Wordlift_Sparql_Service::format( $item, $data_type, $language )
+            );
+        }, $values );
+    }
+
+    /**
+     * Get the delete statement for current post id.
+     *
+     * @since 3.18.0
+     *
+     * @param int $post_id The post id.
+     *
+     * @return array An array containing delete statements for both
+     *               the uri as subject and object.
+     */
+    public function get_delete_triples( $post_id ) {
+        // Get the entity URI.
+        $uri = $this->entity_service->get_uri( $post_id ) . $this->uri_suffix;
+
+        return array(
+            // As object.
+            sprintf( '<%s> <%s> ?o',
+                Wordlift_Sparql_Service::escape_uri( $uri ),
+                Wordlift_Sparql_Service::escape_uri( $this->predicate )
+            ),
+            // As subject.
+            sprintf( '?s <%s> <%s>',
+                Wordlift_Sparql_Service::escape_uri( $this->predicate ),
+                Wordlift_Sparql_Service::escape_uri( $uri )
+            ),
+        );
+    }
 }

Spacing +20 added lines, -20 removed lines

@@ -88,14 +88,14 @@ 
 	 * @param string|null              $language       The language code or null.
 	 * @param string|null              $uri_suffix     The URI suffix, used for example for 2nd level entities, like location.
 	 */
-	public function __construct( $entity_service, $storage, $predicate, $data_type = null, $language = null, $uri_suffix = null ) {
+	public function __construct($entity_service, $storage, $predicate, $data_type = null, $language = null, $uri_suffix = null) {
 
 		$this->entity_service = $entity_service;
 		$this->storage        = $storage;
 		$this->predicate      = $predicate;
 		$this->data_type      = $data_type;
 		$this->language       = $language;
-		$this->uri_suffix     = ( ! is_null( $uri_suffix ) ) ? $uri_suffix : '';
+		$this->uri_suffix     = ( ! is_null($uri_suffix)) ? $uri_suffix : '';
 
 	}
 
@@ -108,9 +108,9 @@ 
 	 *
 	 * @return array An array of tuples.
 	 */
-	public function get_insert_triples( $post_id ) {
+	public function get_insert_triples($post_id) {
 		// Get the entity URI.
-		$uri = $this->entity_service->get_uri( $post_id ) . $this->uri_suffix;
+		$uri = $this->entity_service->get_uri($post_id).$this->uri_suffix;
 
 		// Get the predicate, data type and language.
 		$predicate = $this->predicate;
@@ -118,19 +118,19 @@ 
 		$language  = $this->language;
 
 		// Filter out empty values.
-		$values = array_filter( (array) $this->storage->get( $post_id ), function ( $item ) {
-			return ! empty( $item );
+		$values = array_filter((array) $this->storage->get($post_id), function($item) {
+			return ! empty($item);
 		} );
 
 		// Map the values to tuples.
-		return array_map( function ( $item ) use ( $uri, $predicate, $data_type, $language ) {
+		return array_map(function($item) use ($uri, $predicate, $data_type, $language) {
 
-			return sprintf( '<%s> <%s> %s . ',
-				Wordlift_Sparql_Service::escape_uri( $uri ),
-				Wordlift_Sparql_Service::escape_uri( $predicate ),
-				Wordlift_Sparql_Service::format( $item, $data_type, $language )
+			return sprintf('<%s> <%s> %s . ',
+				Wordlift_Sparql_Service::escape_uri($uri),
+				Wordlift_Sparql_Service::escape_uri($predicate),
+				Wordlift_Sparql_Service::format($item, $data_type, $language)
 			);
-		}, $values );
+		}, $values);
 	}
 
 	/**
@@ -143,20 +143,20 @@ 
 	 * @return array An array containing delete statements for both
 	 *               the uri as subject and object.
 	 */
-	public function get_delete_triples( $post_id ) {
+	public function get_delete_triples($post_id) {
 		// Get the entity URI.
-		$uri = $this->entity_service->get_uri( $post_id ) . $this->uri_suffix;
+		$uri = $this->entity_service->get_uri($post_id).$this->uri_suffix;
 
 		return array(
 			// As object.
-			sprintf( '<%s> <%s> ?o',
-				Wordlift_Sparql_Service::escape_uri( $uri ),
-				Wordlift_Sparql_Service::escape_uri( $this->predicate )
+			sprintf('<%s> <%s> ?o',
+				Wordlift_Sparql_Service::escape_uri($uri),
+				Wordlift_Sparql_Service::escape_uri($this->predicate)
 			),
 			// As subject.
-			sprintf( '?s <%s> <%s>',
-				Wordlift_Sparql_Service::escape_uri( $this->predicate ),
-				Wordlift_Sparql_Service::escape_uri( $uri )
+			sprintf('?s <%s> <%s>',
+				Wordlift_Sparql_Service::escape_uri($this->predicate),
+				Wordlift_Sparql_Service::escape_uri($uri)
 			),
 		);
 	}

src/includes/linked-data/rendition/class-wordlift-sparql-tuple-rendition-factory.php

Spacing +5 added lines, -5 removed lines

@@ -35,7 +35,7 @@ 
 	 *
 	 * @param \Wordlift_Entity_Service $entity_service The {@link Wordlift_Entity_Service} instance.
 	 */
-	public function __construct( $entity_service ) {
+	public function __construct($entity_service) {
 
 		$this->entity_service = $entity_service;
 
@@ -54,9 +54,9 @@ 
 	 *
 	 * @return \Wordlift_Default_Sparql_Tuple_Rendition A {@link Wordlift_Default_Sparql_Tuple_Rendition} instance.
 	 */
-	public function create( $storage, $predicate, $data_type = null, $language = null, $uri_suffix = null ) {
+	public function create($storage, $predicate, $data_type = null, $language = null, $uri_suffix = null) {
 
-		return new Wordlift_Default_Sparql_Tuple_Rendition( $this->entity_service, $storage, $predicate, $data_type, $language, $uri_suffix );
+		return new Wordlift_Default_Sparql_Tuple_Rendition($this->entity_service, $storage, $predicate, $data_type, $language, $uri_suffix);
 	}
 
 	/**
@@ -69,8 +69,8 @@ 
 	 *
 	 * @return \Wordlift_Address_Sparql_Tuple_Rendition A {@link Wordlift_Address_Sparql_Tuple_Rendition} instance.
 	 */
-	public function create_address( $storage, $language = null ) {
+	public function create_address($storage, $language = null) {
 
-		return new Wordlift_Address_Sparql_Tuple_Rendition( $this->entity_service, $this, $storage, $language );
+		return new Wordlift_Address_Sparql_Tuple_Rendition($this->entity_service, $this, $storage, $language);
 	}
 }

Indentation +66 added lines, -66 removed lines

@@ -19,81 +19,81 @@
  */
 class Wordlift_Sparql_Tuple_Rendition_Factory {
 
-	/**
-	 * The {@link Wordlift_Entity_Service} instance.
-	 *
-	 * @since  3.15.0
-	 * @access private
-	 * @var \Wordlift_Entity_Service $entity_service The {@link Wordlift_Entity_Service} instance.
-	 */
-	private $entity_service;
+    /**
+     * The {@link Wordlift_Entity_Service} instance.
+     *
+     * @since  3.15.0
+     * @access private
+     * @var \Wordlift_Entity_Service $entity_service The {@link Wordlift_Entity_Service} instance.
+     */
+    private $entity_service;
 
-	/**
-	 * The {@link Wordlift_Sparql_Tuple_Rendition_Factory} singleton instance.
-	 *
-	 * @since 3.19.1
-	 * @access private
-	 * @var \Wordlift_Sparql_Tuple_Rendition_Factory $instance The {@link Wordlift_Sparql_Tuple_Rendition_Factory} singleton instance.
-	 */
-	private static $instance;
+    /**
+     * The {@link Wordlift_Sparql_Tuple_Rendition_Factory} singleton instance.
+     *
+     * @since 3.19.1
+     * @access private
+     * @var \Wordlift_Sparql_Tuple_Rendition_Factory $instance The {@link Wordlift_Sparql_Tuple_Rendition_Factory} singleton instance.
+     */
+    private static $instance;
 
-	/**
-	 * Create a a {@link Wordlift_Sparql_Tuple_Rendition_Factory} instance.
-	 *
-	 * @since 3.15.0
-	 *
-	 * @param \Wordlift_Entity_Service $entity_service The {@link Wordlift_Entity_Service} instance.
-	 */
-	public function __construct( $entity_service ) {
+    /**
+     * Create a a {@link Wordlift_Sparql_Tuple_Rendition_Factory} instance.
+     *
+     * @since 3.15.0
+     *
+     * @param \Wordlift_Entity_Service $entity_service The {@link Wordlift_Entity_Service} instance.
+     */
+    public function __construct( $entity_service ) {
 
-		$this->entity_service = $entity_service;
+        $this->entity_service = $entity_service;
 
-		self::$instance = $this;
+        self::$instance = $this;
 
-	}
+    }
 
-	/**
-	 * Get the {@link Wordlift_Sparql_Tuple_Rendition_Factory} singleton instance.
-	 *
-	 * @since 3.19.1
-	 *
-	 * @return \Wordlift_Sparql_Tuple_Rendition_Factory The {@link Wordlift_Sparql_Tuple_Rendition_Factory} singleton instance.
-	 */
-	public static function get_instance() {
+    /**
+     * Get the {@link Wordlift_Sparql_Tuple_Rendition_Factory} singleton instance.
+     *
+     * @since 3.19.1
+     *
+     * @return \Wordlift_Sparql_Tuple_Rendition_Factory The {@link Wordlift_Sparql_Tuple_Rendition_Factory} singleton instance.
+     */
+    public static function get_instance() {
 
-		return self::$instance;
-	}
+        return self::$instance;
+    }
 
-	/**
-	 * Create a rendition.
-	 *
-	 * @since 3.15.0
-	 *
-	 * @param \Wordlift_Storage $storage A {@link Wordlift_Storage} instance.
-	 * @param string $predicate A predicate (e.g. `http://schema.org/name`).
-	 * @param string|null $data_type A data-type.
-	 * @param string|null $language A language code (e.g. `en`).
-	 * @param string|null $uri_suffix The URI suffix.
-	 *
-	 * @return \Wordlift_Default_Sparql_Tuple_Rendition A {@link Wordlift_Default_Sparql_Tuple_Rendition} instance.
-	 */
-	public function create( $storage, $predicate, $data_type = null, $language = null, $uri_suffix = null ) {
+    /**
+     * Create a rendition.
+     *
+     * @since 3.15.0
+     *
+     * @param \Wordlift_Storage $storage A {@link Wordlift_Storage} instance.
+     * @param string $predicate A predicate (e.g. `http://schema.org/name`).
+     * @param string|null $data_type A data-type.
+     * @param string|null $language A language code (e.g. `en`).
+     * @param string|null $uri_suffix The URI suffix.
+     *
+     * @return \Wordlift_Default_Sparql_Tuple_Rendition A {@link Wordlift_Default_Sparql_Tuple_Rendition} instance.
+     */
+    public function create( $storage, $predicate, $data_type = null, $language = null, $uri_suffix = null ) {
 
-		return new Wordlift_Default_Sparql_Tuple_Rendition( $this->entity_service, $storage, $predicate, $data_type, $language, $uri_suffix );
-	}
+        return new Wordlift_Default_Sparql_Tuple_Rendition( $this->entity_service, $storage, $predicate, $data_type, $language, $uri_suffix );
+    }
 
-	/**
-	 * Create an address rendition.
-	 *
-	 * @since 3.18.0
-	 *
-	 * @param \Wordlift_Storage $storage A {@link Wordlift_Storage} instance.
-	 * @param string|null $language A language code (e.g. `en`).
-	 *
-	 * @return \Wordlift_Address_Sparql_Tuple_Rendition A {@link Wordlift_Address_Sparql_Tuple_Rendition} instance.
-	 */
-	public function create_address( $storage, $language = null ) {
+    /**
+     * Create an address rendition.
+     *
+     * @since 3.18.0
+     *
+     * @param \Wordlift_Storage $storage A {@link Wordlift_Storage} instance.
+     * @param string|null $language A language code (e.g. `en`).
+     *
+     * @return \Wordlift_Address_Sparql_Tuple_Rendition A {@link Wordlift_Address_Sparql_Tuple_Rendition} instance.
+     */
+    public function create_address( $storage, $language = null ) {
 
-		return new Wordlift_Address_Sparql_Tuple_Rendition( $this->entity_service, $this, $storage, $language );
-	}
+        return new Wordlift_Address_Sparql_Tuple_Rendition( $this->entity_service, $this, $storage, $language );
+    }
 }

src/admin/WL_Metabox/class-wl-metabox-field-sameas.php

Doc Comments +2 added lines

@@ -81,6 +81,7 @@ 
 
 	/**
 	 * @inheritdoc
+	 * @param integer $count
 	 */
 	protected function get_add_button_html( $count ) {
 
@@ -99,6 +100,7 @@ 
 
 	/**
 	 * @inheritdoc
+	 * @param integer $count
 	 */
 	protected function get_stored_values_html( &$count ) {
 

Indentation +131 added lines, -131 removed lines

@@ -16,134 +16,134 @@ 
  */
 class WL_Metabox_Field_sameas extends WL_Metabox_Field {
 
-	/**
-	 * @inheritdoc
-	 */
-	public function __construct( $args ) {
-		parent::__construct( $args['sameas'] );
-	}
-
-	/**
-	 * @inheritdoc
-	 */
-	public function save_data( $values ) {
-		// The autocomplete select may send JSON arrays in input values.
-
-		// Only use mb_* functions when mbstring is available.
-		//
-		// See https://github.com/insideout10/wordlift-plugin/issues/693.
-		if ( extension_loaded( 'mbstring' ) ) {
-			mb_regex_encoding( 'UTF-8' );
-
-			$merged = array_reduce( (array) $values, function ( $carry, $item ) {
-				return array_merge( $carry, mb_split( "\x{2063}", wp_unslash( $item ) ) );
-			}, array() );
-		} else {
-			$merged = array_reduce( (array) $values, function ( $carry, $item ) {
-				return array_merge( $carry, preg_split( "/\x{2063}/u", wp_unslash( $item ) ) );
-			}, array() );
-		}
-
-		// Convert all escaped special characters to their original.
-		$merged = array_map( 'urldecode', $merged );
-
-		parent::save_data( $merged );
-	}
-
-	/**
-	 * @inheritdoc
-	 */
-	public function sanitize_data_filter( $value ) {
-
-		// Call our sanitizer helper.
-		return Wordlift_Sanitizer::sanitize_url( $value );
-	}
-
-	/**
-	 * @inheritdoc
-	 */
-	protected function get_heading_html() {
-
-		// Add the select html fragment after the heading.
-		return parent::get_heading_html()
-			   . $this->get_select_html();
-	}
-
-	/**
-	 * Get the select html fragment.
-	 *
-	 * @since 3.15.0
-	 * @return string The html fragment.
-	 */
-	private function get_select_html() {
-		// Return an element where the new Autocomplete Select will attach to.
-		return '<p>'
-			   . esc_html__( 'Use the search below to link this entity with equivalent entities in the linked data cloud.', 'wordlift' )
-			   . '<div id="wl-metabox-field-sameas"></div></p>';
-	}
-
-	/**
-	 * @inheritdoc
-	 */
-	protected function get_add_button_html( $count ) {
-
-		$placeholder = esc_attr_x( 'Type here the URL of an equivalent entity from another dataset.', 'sameAs metabox input', 'wordlift' );
-
-		return
-			"<label for='$this->meta_name'>"
-			. esc_html__( 'If you already know the URL of the entity that you would like to link, add it in the field below.', 'wordlift' )
-			. '</label>'
-			. '<div class="wl-input-wrapper">'
-			. "<input type='text' id='$this->meta_name' name='wl_metaboxes[$this->meta_name][]' placeholder='$placeholder' style='width:88%' />"
-			. '<button class="button wl-remove-input wl-button" type="button">Remove</button>'
-			. '</div>'
-			. parent::get_add_button_html( $count );
-	}
-
-	/**
-	 * @inheritdoc
-	 */
-	protected function get_stored_values_html( &$count ) {
-
-		return '<p>'
-			   . parent::get_stored_values_html( $count )
-			   . '</p>';
-	}
-
-	/**
-	 * @inheritdoc
-	 */
-	public function html() {
-
-		// Open main <div> for the Field.
-		$html = $this->html_wrapper_open();
-
-		// Label.
-		$html .= $this->get_heading_html();
-
-		// print nonce.
-		$html .= $this->html_nonce();
-
-		// print data loaded from DB.
-		$count = 0;
-
-		// If cardinality allows it, print button to add new values.
-		$html .= $this->get_add_button_html( $count );
-
-		$html .= $this->get_stored_values_html( $count );
-
-		// Close the HTML wrapper.
-		$html .= $this->html_wrapper_close();
-
-		return $html;
-	}
-
-	/**
-	 * @inheritdoc
-	 */
-	public function html_input( $value ) {
-		@ob_start();
-		?>
+    /**
+     * @inheritdoc
+     */
+    public function __construct( $args ) {
+        parent::__construct( $args['sameas'] );
+    }
+
+    /**
+     * @inheritdoc
+     */
+    public function save_data( $values ) {
+        // The autocomplete select may send JSON arrays in input values.
+
+        // Only use mb_* functions when mbstring is available.
+        //
+        // See https://github.com/insideout10/wordlift-plugin/issues/693.
+        if ( extension_loaded( 'mbstring' ) ) {
+            mb_regex_encoding( 'UTF-8' );
+
+            $merged = array_reduce( (array) $values, function ( $carry, $item ) {
+                return array_merge( $carry, mb_split( "\x{2063}", wp_unslash( $item ) ) );
+            }, array() );
+        } else {
+            $merged = array_reduce( (array) $values, function ( $carry, $item ) {
+                return array_merge( $carry, preg_split( "/\x{2063}/u", wp_unslash( $item ) ) );
+            }, array() );
+        }
+
+        // Convert all escaped special characters to their original.
+        $merged = array_map( 'urldecode', $merged );
+
+        parent::save_data( $merged );
+    }
+
+    /**
+     * @inheritdoc
+     */
+    public function sanitize_data_filter( $value ) {
+
+        // Call our sanitizer helper.
+        return Wordlift_Sanitizer::sanitize_url( $value );
+    }
+
+    /**
+     * @inheritdoc
+     */
+    protected function get_heading_html() {
+
+        // Add the select html fragment after the heading.
+        return parent::get_heading_html()
+                . $this->get_select_html();
+    }
+
+    /**
+     * Get the select html fragment.
+     *
+     * @since 3.15.0
+     * @return string The html fragment.
+     */
+    private function get_select_html() {
+        // Return an element where the new Autocomplete Select will attach to.
+        return '<p>'
+                . esc_html__( 'Use the search below to link this entity with equivalent entities in the linked data cloud.', 'wordlift' )
+                . '<div id="wl-metabox-field-sameas"></div></p>';
+    }
+
+    /**
+     * @inheritdoc
+     */
+    protected function get_add_button_html( $count ) {
+
+        $placeholder = esc_attr_x( 'Type here the URL of an equivalent entity from another dataset.', 'sameAs metabox input', 'wordlift' );
+
+        return
+            "<label for='$this->meta_name'>"
+            . esc_html__( 'If you already know the URL of the entity that you would like to link, add it in the field below.', 'wordlift' )
+            . '</label>'
+            . '<div class="wl-input-wrapper">'
+            . "<input type='text' id='$this->meta_name' name='wl_metaboxes[$this->meta_name][]' placeholder='$placeholder' style='width:88%' />"
+            . '<button class="button wl-remove-input wl-button" type="button">Remove</button>'
+            . '</div>'
+            . parent::get_add_button_html( $count );
+    }
+
+    /**
+     * @inheritdoc
+     */
+    protected function get_stored_values_html( &$count ) {
+
+        return '<p>'
+                . parent::get_stored_values_html( $count )
+                . '</p>';
+    }
+
+    /**
+     * @inheritdoc
+     */
+    public function html() {
+
+        // Open main <div> for the Field.
+        $html = $this->html_wrapper_open();
+
+        // Label.
+        $html .= $this->get_heading_html();
+
+        // print nonce.
+        $html .= $this->html_nonce();
+
+        // print data loaded from DB.
+        $count = 0;
+
+        // If cardinality allows it, print button to add new values.
+        $html .= $this->get_add_button_html( $count );
+
+        $html .= $this->get_stored_values_html( $count );
+
+        // Close the HTML wrapper.
+        $html .= $this->html_wrapper_close();
+
+        return $html;
+    }
+
+    /**
+     * @inheritdoc
+     */
+    public function html_input( $value ) {
+        @ob_start();
+        ?>
 			<div class="wl-input-wrapper wl-input-wrapper-readonly">
 				<input
 					type="text"
@@ -160,9 +160,9 @@ 
 			</div>
 		<?php
 
-		$html = ob_get_clean();
+        $html = ob_get_clean();
 
-		return $html;
-	}
+        return $html;
+    }
 
 }

Spacing +26 added lines, -26 removed lines

@@ -19,44 +19,44 @@ 
 	/**
 	 * @inheritdoc
 	 */
-	public function __construct( $args ) {
-		parent::__construct( $args['sameas'] );
+	public function __construct($args) {
+		parent::__construct($args['sameas']);
 	}
 
 	/**
 	 * @inheritdoc
 	 */
-	public function save_data( $values ) {
+	public function save_data($values) {
 		// The autocomplete select may send JSON arrays in input values.
 
 		// Only use mb_* functions when mbstring is available.
 		//
 		// See https://github.com/insideout10/wordlift-plugin/issues/693.
-		if ( extension_loaded( 'mbstring' ) ) {
-			mb_regex_encoding( 'UTF-8' );
+		if (extension_loaded('mbstring')) {
+			mb_regex_encoding('UTF-8');
 
-			$merged = array_reduce( (array) $values, function ( $carry, $item ) {
-				return array_merge( $carry, mb_split( "\x{2063}", wp_unslash( $item ) ) );
-			}, array() );
+			$merged = array_reduce((array) $values, function($carry, $item) {
+				return array_merge($carry, mb_split("\x{2063}", wp_unslash($item)));
+			}, array());
 		} else {
-			$merged = array_reduce( (array) $values, function ( $carry, $item ) {
-				return array_merge( $carry, preg_split( "/\x{2063}/u", wp_unslash( $item ) ) );
-			}, array() );
+			$merged = array_reduce((array) $values, function($carry, $item) {
+				return array_merge($carry, preg_split("/\x{2063}/u", wp_unslash($item)));
+			}, array());
 		}
 
 		// Convert all escaped special characters to their original.
-		$merged = array_map( 'urldecode', $merged );
+		$merged = array_map('urldecode', $merged);
 
-		parent::save_data( $merged );
+		parent::save_data($merged);
 	}
 
 	/**
 	 * @inheritdoc
 	 */
-	public function sanitize_data_filter( $value ) {
+	public function sanitize_data_filter($value) {
 
 		// Call our sanitizer helper.
-		return Wordlift_Sanitizer::sanitize_url( $value );
+		return Wordlift_Sanitizer::sanitize_url($value);
 	}
 
 	/**
@@ -78,35 +78,35 @@ 
 	private function get_select_html() {
 		// Return an element where the new Autocomplete Select will attach to.
 		return '<p>'
-			   . esc_html__( 'Use the search below to link this entity with equivalent entities in the linked data cloud.', 'wordlift' )
+			   . esc_html__('Use the search below to link this entity with equivalent entities in the linked data cloud.', 'wordlift')
 			   . '<div id="wl-metabox-field-sameas"></div></p>';
 	}
 
 	/**
 	 * @inheritdoc
 	 */
-	protected function get_add_button_html( $count ) {
+	protected function get_add_button_html($count) {
 
-		$placeholder = esc_attr_x( 'Type here the URL of an equivalent entity from another dataset.', 'sameAs metabox input', 'wordlift' );
+		$placeholder = esc_attr_x('Type here the URL of an equivalent entity from another dataset.', 'sameAs metabox input', 'wordlift');
 
 		return
 			"<label for='$this->meta_name'>"
-			. esc_html__( 'If you already know the URL of the entity that you would like to link, add it in the field below.', 'wordlift' )
+			. esc_html__('If you already know the URL of the entity that you would like to link, add it in the field below.', 'wordlift')
 			. '</label>'
 			. '<div class="wl-input-wrapper">'
 			. "<input type='text' id='$this->meta_name' name='wl_metaboxes[$this->meta_name][]' placeholder='$placeholder' style='width:88%' />"
 			. '<button class="button wl-remove-input wl-button" type="button">Remove</button>'
 			. '</div>'
-			. parent::get_add_button_html( $count );
+			. parent::get_add_button_html($count);
 	}
 
 	/**
 	 * @inheritdoc
 	 */
-	protected function get_stored_values_html( &$count ) {
+	protected function get_stored_values_html(&$count) {
 
 		return '<p>'
-			   . parent::get_stored_values_html( $count )
+			   . parent::get_stored_values_html($count)
 			   . '</p>';
 	}
 
@@ -128,9 +128,9 @@ 
 		$count = 0;
 
 		// If cardinality allows it, print button to add new values.
-		$html .= $this->get_add_button_html( $count );
+		$html .= $this->get_add_button_html($count);
 
-		$html .= $this->get_stored_values_html( $count );
+		$html .= $this->get_stored_values_html($count);
 
 		// Close the HTML wrapper.
 		$html .= $this->html_wrapper_close();
@@ -141,7 +141,7 @@ 
 	/**
 	 * @inheritdoc
 	 */
-	public function html_input( $value ) {
+	public function html_input($value) {
 		@ob_start();
 		?>
 			<div class="wl-input-wrapper wl-input-wrapper-readonly">
@@ -155,7 +155,7 @@ 
 				/>
 
 				<button class="button wl-remove-input wl-button" type="button">
-					<?php esc_html_e( 'Remove', 'wordlift' ); ?>
+					<?php esc_html_e('Remove', 'wordlift'); ?>
 				</button>
 			</div>
 		<?php

src/includes/properties/class-wordlift-property-getter-factory.php

Indentation +36 added lines, -36 removed lines

@@ -25,44 +25,44 @@
  */
 class Wordlift_Property_Getter_Factory {
 
-	/**
-	 * Create a {@link Wordlift_Property_Getter} instance.
-	 *
-	 * @since 3.8.0
-	 *
-	 * @param \Wordlift_Entity_Service $entity_service A {@link Wordlift_Entity_Service} instance.
-	 *
-	 * @return \Wordlift_Property_Getter A {@link Wordlift_Property_Getter} instance.
-	 */
-	public static function create( $entity_service ) {
+    /**
+     * Create a {@link Wordlift_Property_Getter} instance.
+     *
+     * @since 3.8.0
+     *
+     * @param \Wordlift_Entity_Service $entity_service A {@link Wordlift_Entity_Service} instance.
+     *
+     * @return \Wordlift_Property_Getter A {@link Wordlift_Property_Getter} instance.
+     */
+    public static function create( $entity_service ) {
 
-		$property_getter = new Wordlift_Property_Getter( new Wordlift_Simple_Property_Service() );
-		$property_getter->register( new Wordlift_Entity_Property_Service( $entity_service ), array(
-			Wordlift_Schema_Service::FIELD_FOUNDER,
-			Wordlift_Schema_Service::FIELD_AUTHOR,
-			Wordlift_Schema_Service::FIELD_KNOWS,
-			Wordlift_Schema_Service::FIELD_BIRTH_PLACE,
-			Wordlift_Schema_Service::FIELD_AFFILIATION,
-			Wordlift_Schema_Service::FIELD_PERFORMER,
-			Wordlift_Schema_Service::FIELD_OFFERS,
-			Wordlift_Schema_Service::FIELD_ITEM_OFFERED,
-		) );
-		$property_getter->register( new Wordlift_Location_Property_Service( $entity_service ), array(
-			Wordlift_Schema_Service::FIELD_LOCATION,
-		) );
-		$property_getter->register( new Wordlift_Url_Property_Service(), array( Wordlift_Url_Property_Service::META_KEY ) );
-		$property_getter->register( new Wordlift_Double_Property_Service(), array(
-			Wordlift_Schema_Service::FIELD_GEO_LATITUDE,
-			Wordlift_Schema_Service::FIELD_GEO_LONGITUDE,
-		) );
+        $property_getter = new Wordlift_Property_Getter( new Wordlift_Simple_Property_Service() );
+        $property_getter->register( new Wordlift_Entity_Property_Service( $entity_service ), array(
+            Wordlift_Schema_Service::FIELD_FOUNDER,
+            Wordlift_Schema_Service::FIELD_AUTHOR,
+            Wordlift_Schema_Service::FIELD_KNOWS,
+            Wordlift_Schema_Service::FIELD_BIRTH_PLACE,
+            Wordlift_Schema_Service::FIELD_AFFILIATION,
+            Wordlift_Schema_Service::FIELD_PERFORMER,
+            Wordlift_Schema_Service::FIELD_OFFERS,
+            Wordlift_Schema_Service::FIELD_ITEM_OFFERED,
+        ) );
+        $property_getter->register( new Wordlift_Location_Property_Service( $entity_service ), array(
+            Wordlift_Schema_Service::FIELD_LOCATION,
+        ) );
+        $property_getter->register( new Wordlift_Url_Property_Service(), array( Wordlift_Url_Property_Service::META_KEY ) );
+        $property_getter->register( new Wordlift_Double_Property_Service(), array(
+            Wordlift_Schema_Service::FIELD_GEO_LATITUDE,
+            Wordlift_Schema_Service::FIELD_GEO_LONGITUDE,
+        ) );
 
-		$property_getter->register( new Wordlift_Duration_Property_Service(), array(
-			Wordlift_Schema_Service::FIELD_PREP_TIME,
-			Wordlift_Schema_Service::FIELD_COOK_TIME,
-			Wordlift_Schema_Service::FIELD_TOTAL_TIME,
-		) );
+        $property_getter->register( new Wordlift_Duration_Property_Service(), array(
+            Wordlift_Schema_Service::FIELD_PREP_TIME,
+            Wordlift_Schema_Service::FIELD_COOK_TIME,
+            Wordlift_Schema_Service::FIELD_TOTAL_TIME,
+        ) );
 
-		return $property_getter;
-	}
+        return $property_getter;
+    }
 
 }

Spacing +18 added lines, -18 removed lines

@@ -7,13 +7,13 @@ 
  * @subpackage Wordlift/includes/properties
  */
 
-require_once( 'class-wordlift-property-getter.php' );
-require_once( 'class-wordlift-simple-property-service.php' );
-require_once( 'class-wordlift-entity-property-service.php' );
-require_once( 'class-wordlift-location-property-service.php' );
-require_once( 'class-wordlift-url-property-service.php' );
-require_once( 'class-wordlift-double-property-service.php' );
-require_once( 'class-wordlift-duration-property-service.php' );
+require_once('class-wordlift-property-getter.php');
+require_once('class-wordlift-simple-property-service.php');
+require_once('class-wordlift-entity-property-service.php');
+require_once('class-wordlift-location-property-service.php');
+require_once('class-wordlift-url-property-service.php');
+require_once('class-wordlift-double-property-service.php');
+require_once('class-wordlift-duration-property-service.php');
 
 /**
  * A Wordlift_Property_Getter_Factory, which instantiate a configured
@@ -34,10 +34,10 @@ 
 	 *
 	 * @return \Wordlift_Property_Getter A {@link Wordlift_Property_Getter} instance.
 	 */
-	public static function create( $entity_service ) {
+	public static function create($entity_service) {
 
-		$property_getter = new Wordlift_Property_Getter( new Wordlift_Simple_Property_Service() );
-		$property_getter->register( new Wordlift_Entity_Property_Service( $entity_service ), array(
+		$property_getter = new Wordlift_Property_Getter(new Wordlift_Simple_Property_Service());
+		$property_getter->register(new Wordlift_Entity_Property_Service($entity_service), array(
 			Wordlift_Schema_Service::FIELD_FOUNDER,
 			Wordlift_Schema_Service::FIELD_AUTHOR,
 			Wordlift_Schema_Service::FIELD_KNOWS,
@@ -46,21 +46,21 @@ 
 			Wordlift_Schema_Service::FIELD_PERFORMER,
 			Wordlift_Schema_Service::FIELD_OFFERS,
 			Wordlift_Schema_Service::FIELD_ITEM_OFFERED,
-		) );
-		$property_getter->register( new Wordlift_Location_Property_Service( $entity_service ), array(
+		));
+		$property_getter->register(new Wordlift_Location_Property_Service($entity_service), array(
 			Wordlift_Schema_Service::FIELD_LOCATION,
-		) );
-		$property_getter->register( new Wordlift_Url_Property_Service(), array( Wordlift_Url_Property_Service::META_KEY ) );
-		$property_getter->register( new Wordlift_Double_Property_Service(), array(
+		));
+		$property_getter->register(new Wordlift_Url_Property_Service(), array(Wordlift_Url_Property_Service::META_KEY));
+		$property_getter->register(new Wordlift_Double_Property_Service(), array(
 			Wordlift_Schema_Service::FIELD_GEO_LATITUDE,
 			Wordlift_Schema_Service::FIELD_GEO_LONGITUDE,
-		) );
+		));
 
-		$property_getter->register( new Wordlift_Duration_Property_Service(), array(
+		$property_getter->register(new Wordlift_Duration_Property_Service(), array(
 			Wordlift_Schema_Service::FIELD_PREP_TIME,
 			Wordlift_Schema_Service::FIELD_COOK_TIME,
 			Wordlift_Schema_Service::FIELD_TOTAL_TIME,
-		) );
+		));
 
 		return $property_getter;
 	}

src/admin/WL_Metabox/class-wordlift-metabox-field-select.php

Indentation +9 added lines, -9 removed lines

@@ -18,12 +18,12 @@ 
  */
 class Wordlift_Metabox_Field_Select extends WL_Metabox_Field {
 
-	/**
-	 * @inheritdoc
-	 */
-	public function html_input( $text ) {
-		@ob_start();
-		?>
+    /**
+     * @inheritdoc
+     */
+    public function html_input( $text ) {
+        @ob_start();
+        ?>
 		<div class="wl-input-wrapper">
 
 			<select name="wl_metaboxes[<?php echo $this->meta_name ?>]" id="<?php echo esc_attr( $this->meta_name ); ?>" style="width:88%;">
@@ -40,8 +40,8 @@ 
 		</div>
 		<?php
 
-		$html = ob_get_clean();
-		return $html;
-	}
+        $html = ob_get_clean();
+        return $html;
+    }
 
 }

Spacing +4 added lines, -4 removed lines

@@ -21,15 +21,15 @@
 	/**
 	 * @inheritdoc
 	 */
-	public function html_input( $text ) {
+	public function html_input($text) {
 		@ob_start();
 		?>
 		<div class="wl-input-wrapper">
 
-			<select name="wl_metaboxes[<?php echo $this->meta_name ?>]" id="<?php echo esc_attr( $this->meta_name ); ?>" style="width:88%;">
-				<?php foreach ( $this->raw_custom_field['options'] as $option => $label ): ?>
+			<select name="wl_metaboxes[<?php echo $this->meta_name ?>]" id="<?php echo esc_attr($this->meta_name); ?>" style="width:88%;">
+				<?php foreach ($this->raw_custom_field['options'] as $option => $label): ?>
 
-					<option value="<?php echo esc_attr( $option ); ?>" <?php selected( $text, $option ); ?>>
+					<option value="<?php echo esc_attr($option); ?>" <?php selected($text, $option); ?>>
 						<?php echo $label; ?>
 					</option>
 

src/admin/WL_Metabox/WL_Metabox_Field_date.php

Indentation +75 added lines, -75 removed lines

@@ -18,63 +18,63 @@ 
  */
 class WL_Metabox_Field_date extends WL_Metabox_Field {
 
-	/**
-	 * Attribute to distinguish between date formats, inferred from the schema property export type
-	 *
-	 * @since  3.2.0
-	 * @access protected
-	 * @var string $date_format The date format.
-	 */
-	protected $date_format;
-
-	/**
-	 * Boolean flag to decide if the calendar should include time or not
-	 *
-	 * @since  3.2.0
-	 * @access protected
-	 * @var boolean $timepicker A boolean flag.
-	 */
-	protected $timepicker;
-
-	/**
-	 * Whether the calendar should be displayed or not.
-	 *
-	 * @since  3.14.0
-	 * @access protected
-	 * @var boolean $no_calendar Whether the calendar should be displayed or not.
-	 */
-	protected $no_calendar;
-
-	/**
-	 * {@inheritdoc}
-	 */
-	public function __construct( $args ) {
-		parent::__construct( $args );
-
-		$this->no_calendar = false;
-
-		// Distinguish between date and datetime
-		if ( isset( $this->raw_custom_field['export_type'] ) && 'xsd:dateTime' === $this->raw_custom_field['export_type'] ) {
-			$this->date_format = 'Y/m/d H:i';
-			$this->timepicker  = true;
-		} else {
-			$this->date_format = 'Y/m/d';
-			$this->timepicker  = false;
-		}
-
-	}
-
-	/**
-	 * @param mixed $date
-	 *
-	 * @return string
-	 */
-	public function html_input( $date ) {
-
-		$this->log->debug("Creating date input with date value $date...");
-
-		ob_start();
-		?>
+    /**
+     * Attribute to distinguish between date formats, inferred from the schema property export type
+     *
+     * @since  3.2.0
+     * @access protected
+     * @var string $date_format The date format.
+     */
+    protected $date_format;
+
+    /**
+     * Boolean flag to decide if the calendar should include time or not
+     *
+     * @since  3.2.0
+     * @access protected
+     * @var boolean $timepicker A boolean flag.
+     */
+    protected $timepicker;
+
+    /**
+     * Whether the calendar should be displayed or not.
+     *
+     * @since  3.14.0
+     * @access protected
+     * @var boolean $no_calendar Whether the calendar should be displayed or not.
+     */
+    protected $no_calendar;
+
+    /**
+     * {@inheritdoc}
+     */
+    public function __construct( $args ) {
+        parent::__construct( $args );
+
+        $this->no_calendar = false;
+
+        // Distinguish between date and datetime
+        if ( isset( $this->raw_custom_field['export_type'] ) && 'xsd:dateTime' === $this->raw_custom_field['export_type'] ) {
+            $this->date_format = 'Y/m/d H:i';
+            $this->timepicker  = true;
+        } else {
+            $this->date_format = 'Y/m/d';
+            $this->timepicker  = false;
+        }
+
+    }
+
+    /**
+     * @param mixed $date
+     *
+     * @return string
+     */
+    public function html_input( $date ) {
+
+        $this->log->debug("Creating date input with date value $date...");
+
+        ob_start();
+        ?>
 			<div class="wl-input-wrapper">
 				<input
 					type="text"
@@ -89,24 +89,24 @@ 
 				</button>
 			</div>
 		<?php
-		$html = ob_get_clean();
+        $html = ob_get_clean();
 
-		return $html;
-	}
+        return $html;
+    }
 
-	public function html_wrapper_close() {
+    public function html_wrapper_close() {
 
-		// Should the widget include time picker?
-		$timepicker  = json_encode( $this->timepicker );
-		$date_format = json_encode( $this->date_format );
-		$no_calendar = json_encode( $this->no_calendar );
+        // Should the widget include time picker?
+        $timepicker  = json_encode( $this->timepicker );
+        $date_format = json_encode( $this->date_format );
+        $no_calendar = json_encode( $this->no_calendar );
 
-		// Set up the datetimepicker.
-		//
-		// See https://github.com/trentrichardson/jQuery-Timepicker-Addon
-		// See in http://trentrichardson.com/examples/timepicker.
-		@ob_start();
-		?>
+        // Set up the datetimepicker.
+        //
+        // See https://github.com/trentrichardson/jQuery-Timepicker-Addon
+        // See in http://trentrichardson.com/examples/timepicker.
+        @ob_start();
+        ?>
 			<script type='text/javascript'>
 				( function( $ ) {
 
@@ -122,11 +122,11 @@ 
 				} ) ( jQuery );
 			</script>
 		<?php
-		$html = ob_get_clean();
+        $html = ob_get_clean();
 
-		$html .= parent::html_wrapper_close();
+        $html .= parent::html_wrapper_close();
 
-		return $html;
-	}
+        return $html;
+    }
 
 }

Spacing +9 added lines, -9 removed lines

@@ -48,13 +48,13 @@ 
 	/**
 	 * {@inheritdoc}
 	 */
-	public function __construct( $args ) {
-		parent::__construct( $args );
+	public function __construct($args) {
+		parent::__construct($args);
 
 		$this->no_calendar = false;
 
 		// Distinguish between date and datetime
-		if ( isset( $this->raw_custom_field['export_type'] ) && 'xsd:dateTime' === $this->raw_custom_field['export_type'] ) {
+		if (isset($this->raw_custom_field['export_type']) && 'xsd:dateTime' === $this->raw_custom_field['export_type']) {
 			$this->date_format = 'Y/m/d H:i';
 			$this->timepicker  = true;
 		} else {
@@ -69,7 +69,7 @@ 
 	 *
 	 * @return string
 	 */
-	public function html_input( $date ) {
+	public function html_input($date) {
 
 		$this->log->debug("Creating date input with date value $date...");
 
@@ -78,14 +78,14 @@ 
 			<div class="wl-input-wrapper">
 				<input
 					type="text"
-					class="<?php echo esc_attr( $this->meta_name ); ?>"
+					class="<?php echo esc_attr($this->meta_name); ?>"
 					name="wl_metaboxes[<?php echo $this->meta_name ?>][]"
 					value="<?php echo $date ?>"
 					style="width:88%"
 				/>
 
 				<button class="button wl-remove-input wl-button" type="button">
-					<?php esc_html_e( 'Remove', 'wordlift' ); ?>
+					<?php esc_html_e('Remove', 'wordlift'); ?>
 				</button>
 			</div>
 		<?php
@@ -97,9 +97,9 @@ 
 	public function html_wrapper_close() {
 
 		// Should the widget include time picker?
-		$timepicker  = json_encode( $this->timepicker );
-		$date_format = json_encode( $this->date_format );
-		$no_calendar = json_encode( $this->no_calendar );
+		$timepicker  = json_encode($this->timepicker);
+		$date_format = json_encode($this->date_format);
+		$no_calendar = json_encode($this->no_calendar);
 
 		// Set up the datetimepicker.
 		//

src/admin/WL_Metabox/WL_Metabox_Field_address.php

Indentation +64 added lines, -64 removed lines

@@ -8,78 +8,78 @@ 
  */
 class WL_Metabox_Field_address extends WL_Metabox_Field {
 
-	/**
-	 * Sub-fields contained in the Field.
-	 *
-	 * @since 3.2.0
-	 * @access private
-	 * @var $subfields Array of WL_Metabox_Field objects, each dealing with a part of the http://schema.org/PostalAddress structure.
-	 */
-	private $subfields;
+    /**
+     * Sub-fields contained in the Field.
+     *
+     * @since 3.2.0
+     * @access private
+     * @var $subfields Array of WL_Metabox_Field objects, each dealing with a part of the http://schema.org/PostalAddress structure.
+     */
+    private $subfields;
 	
-	/**
-	 * Constructor.
-	 * 
-	 * @param array $args Set of fields containing info to build the subfields.
-	 * The structure of $args is:
-	 * array( 'address' => array( ... array of subfields ... ) )
-	 */
-	public function __construct( $args ) {
+    /**
+     * Constructor.
+     * 
+     * @param array $args Set of fields containing info to build the subfields.
+     * The structure of $args is:
+     * array( 'address' => array( ... array of subfields ... ) )
+     */
+    public function __construct( $args ) {
 		
-		$this->label = key( $args );
+        $this->label = key( $args );
 		
-		// leverage the WL_Metabox_Field class to build the subfields
-		$this->subfields = array();
-		// Loop over subfields. Using 'reset' to take the data contained in the first element of $args
-		foreach ( reset( $args ) as $key => $subfield ) {
-			$this->subfields[] = new WL_Metabox_Field( array( $key => $subfield ) );
-		}
+        // leverage the WL_Metabox_Field class to build the subfields
+        $this->subfields = array();
+        // Loop over subfields. Using 'reset' to take the data contained in the first element of $args
+        foreach ( reset( $args ) as $key => $subfield ) {
+            $this->subfields[] = new WL_Metabox_Field( array( $key => $subfield ) );
+        }
 		
-		// $_POST array key in which we will pass the values
-		$this->meta_name = 'wl_grouped_field_address';
-	}
+        // $_POST array key in which we will pass the values
+        $this->meta_name = 'wl_grouped_field_address';
+    }
 
-	/**
-	 * Load data from DB and store the resulting array in $this->data.
-	 */
-	public function get_data() {
+    /**
+     * Load data from DB and store the resulting array in $this->data.
+     */
+    public function get_data() {
 		
-		foreach ( $this->subfields as $subfield ) {
-			$subfield->get_data();
-		}
-	}
+        foreach ( $this->subfields as $subfield ) {
+            $subfield->get_data();
+        }
+    }
 
-	/**
-	 * Save data to DB.
-	 * 
-	 * @param array $values Values coming from $_POST and passed from WL_Metabox. We just send to each subfield its own value.
-	 */
-	public function save_data( $values ) {
+    /**
+     * Save data to DB.
+     * 
+     * @param array $values Values coming from $_POST and passed from WL_Metabox. We just send to each subfield its own value.
+     */
+    public function save_data( $values ) {
 		
-		foreach ( $this->subfields as $subfield ) {
-			$subfield_value = isset( $values[ $subfield->meta_name ] )? $values[ $subfield->meta_name ] : null;
-			$subfield->save_data( $subfield_value );
-		}
-	}
+        foreach ( $this->subfields as $subfield ) {
+            $subfield_value = isset( $values[ $subfield->meta_name ] )? $values[ $subfield->meta_name ] : null;
+            $subfield->save_data( $subfield_value );
+        }
+    }
 
-	/**
-	 * Returns Field HTML (nonce included).
-	 * 
-	 * @return string Field HTML
-	 */
-	public function html() {
-		@ob_start();
-		// Open main <div> for the Field, then insert label and nonce
-		?>
+    /**
+     * Returns Field HTML (nonce included).
+     * 
+     * @return string Field HTML
+     */
+    public function html() {
+        @ob_start();
+        // Open main <div> for the Field, then insert label and nonce
+        ?>
 			<div class='wl-field'>
 			<h3><?php echo $this->label ?></h3>
 			<?php echo $this->html_nonce() ?>
 
 		<?php
-		// print data loaded from DB
-		foreach ( $this->subfields as $subfield ) :
-			$value = isset( $subfield->data[0] )? $subfield->data[0] : '';
-		?>
+        // print data loaded from DB
+        foreach ( $this->subfields as $subfield ) :
+            $value = isset( $subfield->data[0] )? $subfield->data[0] : '';
+        ?>
 			<div class="wl-input-wrapper">
 				<label
 					for="wl_metaboxes[<?php echo $this->meta_name ?>][<?php echo $subfield->meta_name ?>]"
@@ -96,12 +96,12 @@ 
 				/>
 			</div>
 		<?php
-		endforeach;
-		$html = ob_get_clean();
+        endforeach;
+        $html = ob_get_clean();
 
-		// Close the HTML wrapper
-		$html .= $this->html_wrapper_close();
+        // Close the HTML wrapper
+        $html .= $this->html_wrapper_close();
 
-		return $html;
-	}
+        return $html;
+    }
 }
\ No newline at end of file

Spacing +11 added lines, -11 removed lines

@@ -24,15 +24,15 @@ 
 	 * The structure of $args is:
 	 * array( 'address' => array( ... array of subfields ... ) )
 	 */
-	public function __construct( $args ) {
+	public function __construct($args) {
 		
-		$this->label = key( $args );
+		$this->label = key($args);
 		
 		// leverage the WL_Metabox_Field class to build the subfields
 		$this->subfields = array();
 		// Loop over subfields. Using 'reset' to take the data contained in the first element of $args
-		foreach ( reset( $args ) as $key => $subfield ) {
-			$this->subfields[] = new WL_Metabox_Field( array( $key => $subfield ) );
+		foreach (reset($args) as $key => $subfield) {
+			$this->subfields[] = new WL_Metabox_Field(array($key => $subfield));
 		}
 		
 		// $_POST array key in which we will pass the values
@@ -44,7 +44,7 @@ 
 	 */
 	public function get_data() {
 		
-		foreach ( $this->subfields as $subfield ) {
+		foreach ($this->subfields as $subfield) {
 			$subfield->get_data();
 		}
 	}
@@ -54,11 +54,11 @@ 
 	 * 
 	 * @param array $values Values coming from $_POST and passed from WL_Metabox. We just send to each subfield its own value.
 	 */
-	public function save_data( $values ) {
+	public function save_data($values) {
 		
-		foreach ( $this->subfields as $subfield ) {
-			$subfield_value = isset( $values[ $subfield->meta_name ] )? $values[ $subfield->meta_name ] : null;
-			$subfield->save_data( $subfield_value );
+		foreach ($this->subfields as $subfield) {
+			$subfield_value = isset($values[$subfield->meta_name]) ? $values[$subfield->meta_name] : null;
+			$subfield->save_data($subfield_value);
 		}
 	}
 
@@ -77,8 +77,8 @@ 
 
 		<?php
 		// print data loaded from DB
-		foreach ( $this->subfields as $subfield ) :
-			$value = isset( $subfield->data[0] )? $subfield->data[0] : '';
+		foreach ($this->subfields as $subfield) :
+			$value = isset($subfield->data[0]) ? $subfield->data[0] : '';
 		?>
 			<div class="wl-input-wrapper">
 				<label

src/admin/WL_Metabox/class-wordlift-metabox-field-multiline.php

Indentation +9 added lines, -9 removed lines

@@ -18,12 +18,12 @@ 
  */
 class Wordlift_Metabox_Field_Multiline extends WL_Metabox_Field {
 
-	/**
-	 * @inheritdoc
-	 */
-	public function html_input( $text ) {
-		@ob_start();
-		?>
+    /**
+     * @inheritdoc
+     */
+    public function html_input( $text ) {
+        @ob_start();
+        ?>
 			<div class="wl-input-wrapper">
 				<textarea
 					id="<?php echo esc_attr( $this->meta_name ) ?>"
@@ -37,8 +37,8 @@ 
 				</button>
 			</div>
 		<?php
-		$html = ob_get_clean();
+        $html = ob_get_clean();
 
-		return $html;
-	}
+        return $html;
+    }
 }

Spacing +5 added lines, -5 removed lines

@@ -21,19 +21,19 @@
 	/**
 	 * @inheritdoc
 	 */
-	public function html_input( $text ) {
+	public function html_input($text) {
 		@ob_start();
 		?>
 			<div class="wl-input-wrapper">
 				<textarea
-					id="<?php echo esc_attr( $this->meta_name ) ?>"
-					class="<?php echo esc_attr( $this->meta_name ) ?>"
+					id="<?php echo esc_attr($this->meta_name) ?>"
+					class="<?php echo esc_attr($this->meta_name) ?>"
 					name="wl_metaboxes[<?php echo $this->meta_name ?>][]"
 					style="width:88%"
-				><?php echo esc_textarea( $text ) ?></textarea>
+				><?php echo esc_textarea($text) ?></textarea>
 
 				<button class="button wl-remove-input wl-button" type="button">
-					<?php esc_html_e( 'Remove', 'wordlift' ); ?>
+					<?php esc_html_e('Remove', 'wordlift'); ?>
 				</button>
 			</div>
 		<?php