insideout10 / wordlift-plugin

src/wordlift/metabox/field/class-wordlift-metabox-field-duration.php

Indentation +37 added lines, -37 removed lines

@@ -20,42 +20,42 @@
  */
 class Wordlift_Metabox_Field_Duration extends Wl_Metabox_Field_date {
 
-	/**
-	 * {@inheritdoc}
-	 */
-	public function __construct( $args, $id, $type ) {
-		parent::__construct( $args, $id, $type );
-
-		$this->date_format = 'H:i';
-		$this->timepicker  = true;
-		$this->no_calendar = true;
-
-	}
-
-	/**
-	 * Sanitize a single value. Called from $this->sanitize_data. Default sanitization excludes empty values.
-	 * make sure the value is either empty, an integer representing valid number of minutes
-	 * or an HH:MM time format.
-	 *
-	 * @param mixed $value The value being sanitized.
-	 *
-	 * @return mixed Returns sanitized value, or null.
-	 */
-	public function sanitize_data_filter( $value ) {
-
-		if ( $value !== null && '' !== $value ) {         // do not use 'empty()' -> https://www.virendrachandak.com/techtalk/php-isset-vs-empty-vs-is_null/ .
-			preg_match(
-				'#((([01]?[0-9]{1}|2[0-3]{1}):)?[0-5]{1})?[0-9]{1}#',
-				trim( $value ),
-				$matches
-			);
-
-			if ( count( $matches ) > 0 ) {
-				return $matches[0];
-			}
-		}
-
-		return null;
-	}
+    /**
+     * {@inheritdoc}
+     */
+    public function __construct( $args, $id, $type ) {
+        parent::__construct( $args, $id, $type );
+
+        $this->date_format = 'H:i';
+        $this->timepicker  = true;
+        $this->no_calendar = true;
+
+    }
+
+    /**
+     * Sanitize a single value. Called from $this->sanitize_data. Default sanitization excludes empty values.
+     * make sure the value is either empty, an integer representing valid number of minutes
+     * or an HH:MM time format.
+     *
+     * @param mixed $value The value being sanitized.
+     *
+     * @return mixed Returns sanitized value, or null.
+     */
+    public function sanitize_data_filter( $value ) {
+
+        if ( $value !== null && '' !== $value ) {         // do not use 'empty()' -> https://www.virendrachandak.com/techtalk/php-isset-vs-empty-vs-is_null/ .
+            preg_match(
+                '#((([01]?[0-9]{1}|2[0-3]{1}):)?[0-5]{1})?[0-9]{1}#',
+                trim( $value ),
+                $matches
+            );
+
+            if ( count( $matches ) > 0 ) {
+                return $matches[0];
+            }
+        }
+
+        return null;
+    }
 
 }

Spacing +6 added lines, -6 removed lines

@@ -23,8 +23,8 @@ 
 	/**
 	 * {@inheritdoc}
 	 */
-	public function __construct( $args, $id, $type ) {
-		parent::__construct( $args, $id, $type );
+	public function __construct($args, $id, $type) {
+		parent::__construct($args, $id, $type);
 
 		$this->date_format = 'H:i';
 		$this->timepicker  = true;
@@ -41,16 +41,16 @@ 
 	 *
 	 * @return mixed Returns sanitized value, or null.
 	 */
-	public function sanitize_data_filter( $value ) {
+	public function sanitize_data_filter($value) {
 
-		if ( $value !== null && '' !== $value ) {         // do not use 'empty()' -> https://www.virendrachandak.com/techtalk/php-isset-vs-empty-vs-is_null/ .
+		if ($value !== null && '' !== $value) {         // do not use 'empty()' -> https://www.virendrachandak.com/techtalk/php-isset-vs-empty-vs-is_null/ .
 			preg_match(
 				'#((([01]?[0-9]{1}|2[0-3]{1}):)?[0-5]{1})?[0-9]{1}#',
-				trim( $value ),
+				trim($value),
 				$matches
 			);
 
-			if ( count( $matches ) > 0 ) {
+			if (count($matches) > 0) {
 				return $matches[0];
 			}
 		}

src/wordlift/metabox/field/store/class-store-factory.php

Indentation +13 added lines, -13 removed lines

@@ -11,19 +11,19 @@
 
 class Store_Factory {
 
-	/**
-	 * @param $type
-	 *
-	 * @return Store
-	 */
-	public static function get_instance( $type ) {
+    /**
+     * @param $type
+     *
+     * @return Store
+     */
+    public static function get_instance( $type ) {
 
-		if ( Object_Type_Enum::POST === $type ) {
-			return new Post_Store();
-		} elseif ( Object_Type_Enum::TERM === $type ) {
-			return new Term_Store();
-		}
-		return null;
-	}
+        if ( Object_Type_Enum::POST === $type ) {
+            return new Post_Store();
+        } elseif ( Object_Type_Enum::TERM === $type ) {
+            return new Term_Store();
+        }
+        return null;
+    }
 
 }

Spacing +3 added lines, -3 removed lines

@@ -16,11 +16,11 @@
 	 *
 	 * @return Store
 	 */
-	public static function get_instance( $type ) {
+	public static function get_instance($type) {
 
-		if ( Object_Type_Enum::POST === $type ) {
+		if (Object_Type_Enum::POST === $type) {
 			return new Post_Store();
-		} elseif ( Object_Type_Enum::TERM === $type ) {
+		} elseif (Object_Type_Enum::TERM === $type) {
 			return new Term_Store();
 		}
 		return null;

src/wordlift/metabox/field/store/class-post-store.php

Indentation +21 added lines, -21 removed lines

@@ -9,28 +9,28 @@
 
 class Post_Store implements Store {
 
-	public static function get_data( $post_id, $meta_key ) {
-		return get_post_meta( $post_id, $meta_key );
-	}
+    public static function get_data( $post_id, $meta_key ) {
+        return get_post_meta( $post_id, $meta_key );
+    }
 
-	public static function save_data( $post_id, $meta_key, $cardinality, $values ) {
-		$entity_id = $post_id;
-		// Take away old values.
-		delete_post_meta( $entity_id, $meta_key );
-		// insert new values, respecting cardinality.
-		$single = ( 1 === $cardinality );
-		foreach ( $values as $value ) {
-			// To avoid duplicate values
-			delete_post_meta( $entity_id, $meta_key, $value );
-			add_post_meta( $entity_id, $meta_key, $value, $single );
-		}
-	}
+    public static function save_data( $post_id, $meta_key, $cardinality, $values ) {
+        $entity_id = $post_id;
+        // Take away old values.
+        delete_post_meta( $entity_id, $meta_key );
+        // insert new values, respecting cardinality.
+        $single = ( 1 === $cardinality );
+        foreach ( $values as $value ) {
+            // To avoid duplicate values
+            delete_post_meta( $entity_id, $meta_key, $value );
+            add_post_meta( $entity_id, $meta_key, $value, $single );
+        }
+    }
 
-	public static function delete_meta( $id, $meta_key ) {
-		delete_post_meta( $id, $meta_key );
-	}
+    public static function delete_meta( $id, $meta_key ) {
+        delete_post_meta( $id, $meta_key );
+    }
 
-	public static function add_meta( $id, $meta_key, $meta_value, $is_unique ) {
-		add_post_meta( $id, $meta_key, $meta_value, $is_unique );
-	}
+    public static function add_meta( $id, $meta_key, $meta_value, $is_unique ) {
+        add_post_meta( $id, $meta_key, $meta_value, $is_unique );
+    }
 }

Spacing +12 added lines, -12 removed lines

@@ -9,28 +9,28 @@
 
 class Post_Store implements Store {
 
-	public static function get_data( $post_id, $meta_key ) {
-		return get_post_meta( $post_id, $meta_key );
+	public static function get_data($post_id, $meta_key) {
+		return get_post_meta($post_id, $meta_key);
 	}
 
-	public static function save_data( $post_id, $meta_key, $cardinality, $values ) {
+	public static function save_data($post_id, $meta_key, $cardinality, $values) {
 		$entity_id = $post_id;
 		// Take away old values.
-		delete_post_meta( $entity_id, $meta_key );
+		delete_post_meta($entity_id, $meta_key);
 		// insert new values, respecting cardinality.
-		$single = ( 1 === $cardinality );
-		foreach ( $values as $value ) {
+		$single = (1 === $cardinality);
+		foreach ($values as $value) {
 			// To avoid duplicate values
-			delete_post_meta( $entity_id, $meta_key, $value );
-			add_post_meta( $entity_id, $meta_key, $value, $single );
+			delete_post_meta($entity_id, $meta_key, $value);
+			add_post_meta($entity_id, $meta_key, $value, $single);
 		}
 	}
 
-	public static function delete_meta( $id, $meta_key ) {
-		delete_post_meta( $id, $meta_key );
+	public static function delete_meta($id, $meta_key) {
+		delete_post_meta($id, $meta_key);
 	}
 
-	public static function add_meta( $id, $meta_key, $meta_value, $is_unique ) {
-		add_post_meta( $id, $meta_key, $meta_value, $is_unique );
+	public static function add_meta($id, $meta_key, $meta_value, $is_unique) {
+		add_post_meta($id, $meta_key, $meta_value, $is_unique);
 	}
 }

src/wordlift/metabox/field/store/class-term-store.php

Indentation +20 added lines, -20 removed lines

@@ -9,30 +9,30 @@
 
 class Term_Store implements Store {
 
-	public static function get_data( $term_id, $meta_key ) {
-		return get_term_meta( $term_id, $meta_key );
-	}
+    public static function get_data( $term_id, $meta_key ) {
+        return get_term_meta( $term_id, $meta_key );
+    }
 
-	public static function save_data( $term_id, $meta_key, $cardinality, $values ) {
+    public static function save_data( $term_id, $meta_key, $cardinality, $values ) {
 
-		// Take away old values.
-		delete_term_meta( $term_id, $meta_key );
+        // Take away old values.
+        delete_term_meta( $term_id, $meta_key );
 
-		// insert new values, respecting cardinality.
-		$single = ( 1 === $cardinality );
-		foreach ( $values as $value ) {
-			// To avoid duplicate values
-			delete_term_meta( $term_id, $meta_key, $value );
-			add_term_meta( $term_id, $meta_key, $value, $single );
-		}
+        // insert new values, respecting cardinality.
+        $single = ( 1 === $cardinality );
+        foreach ( $values as $value ) {
+            // To avoid duplicate values
+            delete_term_meta( $term_id, $meta_key, $value );
+            add_term_meta( $term_id, $meta_key, $value, $single );
+        }
 
-	}
+    }
 
-	public static function delete_meta( $id, $meta_key ) {
-		delete_term_meta( $id, $meta_key );
-	}
+    public static function delete_meta( $id, $meta_key ) {
+        delete_term_meta( $id, $meta_key );
+    }
 
-	public static function add_meta( $id, $meta_key, $meta_value, $is_unique ) {
-		add_term_meta( $id, $meta_key, $meta_value, $is_unique );
-	}
+    public static function add_meta( $id, $meta_key, $meta_value, $is_unique ) {
+        add_term_meta( $id, $meta_key, $meta_value, $is_unique );
+    }
 }

Spacing +12 added lines, -12 removed lines

@@ -9,30 +9,30 @@
 
 class Term_Store implements Store {
 
-	public static function get_data( $term_id, $meta_key ) {
-		return get_term_meta( $term_id, $meta_key );
+	public static function get_data($term_id, $meta_key) {
+		return get_term_meta($term_id, $meta_key);
 	}
 
-	public static function save_data( $term_id, $meta_key, $cardinality, $values ) {
+	public static function save_data($term_id, $meta_key, $cardinality, $values) {
 
 		// Take away old values.
-		delete_term_meta( $term_id, $meta_key );
+		delete_term_meta($term_id, $meta_key);
 
 		// insert new values, respecting cardinality.
-		$single = ( 1 === $cardinality );
-		foreach ( $values as $value ) {
+		$single = (1 === $cardinality);
+		foreach ($values as $value) {
 			// To avoid duplicate values
-			delete_term_meta( $term_id, $meta_key, $value );
-			add_term_meta( $term_id, $meta_key, $value, $single );
+			delete_term_meta($term_id, $meta_key, $value);
+			add_term_meta($term_id, $meta_key, $value, $single);
 		}
 
 	}
 
-	public static function delete_meta( $id, $meta_key ) {
-		delete_term_meta( $id, $meta_key );
+	public static function delete_meta($id, $meta_key) {
+		delete_term_meta($id, $meta_key);
 	}
 
-	public static function add_meta( $id, $meta_key, $meta_value, $is_unique ) {
-		add_term_meta( $id, $meta_key, $meta_value, $is_unique );
+	public static function add_meta($id, $meta_key, $meta_value, $is_unique) {
+		add_term_meta($id, $meta_key, $meta_value, $is_unique);
 	}
 }

src/wordlift/metabox/field/store/class-store.php

Indentation +4 added lines, -4 removed lines

@@ -8,12 +8,12 @@
 
 interface Store {
 
-	public static function get_data( $post_id, $meta_key );
+    public static function get_data( $post_id, $meta_key );
 
-	public static function save_data( $post_id, $meta_key, $cardinality, $values );
+    public static function save_data( $post_id, $meta_key, $cardinality, $values );
 
-	public static function delete_meta( $id, $meta_key );
+    public static function delete_meta( $id, $meta_key );
 
-	public static function add_meta( $id, $meta_key, $meta_value, $is_unique );
+    public static function add_meta( $id, $meta_key, $meta_value, $is_unique );
 
 }

Spacing +4 added lines, -4 removed lines

@@ -8,12 +8,12 @@
 
 interface Store {
 
-	public static function get_data( $post_id, $meta_key );
+	public static function get_data($post_id, $meta_key);
 
-	public static function save_data( $post_id, $meta_key, $cardinality, $values );
+	public static function save_data($post_id, $meta_key, $cardinality, $values);
 
-	public static function delete_meta( $id, $meta_key );
+	public static function delete_meta($id, $meta_key);
 
-	public static function add_meta( $id, $meta_key, $meta_value, $is_unique );
+	public static function add_meta($id, $meta_key, $meta_value, $is_unique);
 
 }

src/wordlift/metabox/field/class-field.php

Indentation +89 added lines, -89 removed lines

@@ -6,103 +6,103 @@
 namespace Wordlift\Metabox\Field;
 
 interface Field {
-	/**
-	 * Return nonce HTML.
-	 *
-	 * Overwrite this method in a child class to obtain custom behaviour.
-	 */
-	public function html_nonce();
+    /**
+     * Return nonce HTML.
+     *
+     * Overwrite this method in a child class to obtain custom behaviour.
+     */
+    public function html_nonce();
 
-	/**
-	 * Verify nonce.
-	 *
-	 * Overwrite this method in a child class to obtain custom behaviour.
-	 *
-	 * @return bool Nonce verification.
-	 */
-	public function verify_nonce();
+    /**
+     * Verify nonce.
+     *
+     * Overwrite this method in a child class to obtain custom behaviour.
+     *
+     * @return bool Nonce verification.
+     */
+    public function verify_nonce();
 
-	/**
-	 * Load data from DB and store the resulting array in $this->data.
-	 *
-	 * Overwrite this method in a child class to obtain custom behaviour.
-	 */
-	public function get_data();
+    /**
+     * Load data from DB and store the resulting array in $this->data.
+     *
+     * Overwrite this method in a child class to obtain custom behaviour.
+     */
+    public function get_data();
 
-	/**
-	 * Sanitizes data before saving to DB. Default sanitization trashes empty
-	 * values.
-	 *
-	 * Stores the sanitized values into $this->data so they can be later processed.
-	 * Overwrite this method in a child class to obtain custom behaviour.
-	 *
-	 * @param array $values Array of values to be sanitized and then stored into
-	 *                      $this->data.
-	 *
-	 * @return array | string Return sanitized data.
-	 */
-	public function sanitize_data( $values );
+    /**
+     * Sanitizes data before saving to DB. Default sanitization trashes empty
+     * values.
+     *
+     * Stores the sanitized values into $this->data so they can be later processed.
+     * Overwrite this method in a child class to obtain custom behaviour.
+     *
+     * @param array $values Array of values to be sanitized and then stored into
+     *                      $this->data.
+     *
+     * @return array | string Return sanitized data.
+     */
+    public function sanitize_data( $values );
 
-	/**
-	 * Sanitize a single value. Called from $this->sanitize_data. Default
-	 * sanitization excludes empty values.
-	 *
-	 * Overwrite this method in a child class to obtain custom behaviour.
-	 *
-	 * @param string $value The value to sanitize.
-	 *
-	 * @return mixed Returns sanitized value, or null.
-	 */
-	public function sanitize_data_filter( $value );
+    /**
+     * Sanitize a single value. Called from $this->sanitize_data. Default
+     * sanitization excludes empty values.
+     *
+     * Overwrite this method in a child class to obtain custom behaviour.
+     *
+     * @param string $value The value to sanitize.
+     *
+     * @return mixed Returns sanitized value, or null.
+     */
+    public function sanitize_data_filter( $value );
 
-	/**
-	 * Save data to DB.
-	 *
-	 * Overwrite this method in a child class to obtain custom behaviour.
-	 *
-	 * @param array $values Array of values to be sanitized and then stored into $this->data.
-	 */
-	public function save_data( $values );
+    /**
+     * Save data to DB.
+     *
+     * Overwrite this method in a child class to obtain custom behaviour.
+     *
+     * @param array $values Array of values to be sanitized and then stored into $this->data.
+     */
+    public function save_data( $values );
 
-	/**
-	 * Returns the HTML tag that will contain the Field. By default the we
-	 * return a <div> with data- attributes on cardinality and expected types.
-	 *
-	 * It is useful to provide data- attributes for the JS scripts.
-	 *
-	 * Overwrite this method in a child class to obtain custom behaviour.
-	 */
-	public function html_wrapper_open();
+    /**
+     * Returns the HTML tag that will contain the Field. By default the we
+     * return a <div> with data- attributes on cardinality and expected types.
+     *
+     * It is useful to provide data- attributes for the JS scripts.
+     *
+     * Overwrite this method in a child class to obtain custom behaviour.
+     */
+    public function html_wrapper_open();
 
-	/**
-	 * Returns Field HTML (nonce included).
-	 *
-	 * Overwrite this method (or methods called from this method) in a child
-	 * class to obtain custom behaviour.
-	 *
-	 * The HTML fragment includes the following parts:
-	 * * html wrapper open.
-	 * * heading.
-	 * * nonce.
-	 * * stored values.
-	 * * an empty input when there are no stored values.
-	 * * an add button to add more values.
-	 * * html wrapper close.
-	 */
-	public function html();
+    /**
+     * Returns Field HTML (nonce included).
+     *
+     * Overwrite this method (or methods called from this method) in a child
+     * class to obtain custom behaviour.
+     *
+     * The HTML fragment includes the following parts:
+     * * html wrapper open.
+     * * heading.
+     * * nonce.
+     * * stored values.
+     * * an empty input when there are no stored values.
+     * * an add button to add more values.
+     * * html wrapper close.
+     */
+    public function html();
 
-	/**
-	 * Return a single <input> tag for the Field.
-	 *
-	 * @param mixed $value Input value.
-	 *
-	 * @return string The html code fragment.
-	 */
-	public function html_input( $value );
+    /**
+     * Return a single <input> tag for the Field.
+     *
+     * @param mixed $value Input value.
+     *
+     * @return string The html code fragment.
+     */
+    public function html_input( $value );
 
-	/**
-	 * Returns closing for the wrapper HTML tag.
-	 */
-	public function html_wrapper_close();
+    /**
+     * Returns closing for the wrapper HTML tag.
+     */
+    public function html_wrapper_close();
 
 }

Spacing +4 added lines, -4 removed lines

@@ -41,7 +41,7 @@ 
 	 *
 	 * @return array | string Return sanitized data.
 	 */
-	public function sanitize_data( $values );
+	public function sanitize_data($values);
 
 	/**
 	 * Sanitize a single value. Called from $this->sanitize_data. Default
@@ -53,7 +53,7 @@ 
 	 *
 	 * @return mixed Returns sanitized value, or null.
 	 */
-	public function sanitize_data_filter( $value );
+	public function sanitize_data_filter($value);
 
 	/**
 	 * Save data to DB.
@@ -62,7 +62,7 @@ 
 	 *
 	 * @param array $values Array of values to be sanitized and then stored into $this->data.
 	 */
-	public function save_data( $values );
+	public function save_data($values);
 
 	/**
 	 * Returns the HTML tag that will contain the Field. By default the we
@@ -98,7 +98,7 @@ 
 	 *
 	 * @return string The html code fragment.
 	 */
-	public function html_input( $value );
+	public function html_input($value);
 
 	/**
 	 * Returns closing for the wrapper HTML tag.

src/wordlift/metabox/field/class-wl-metabox-field-date.php

Indentation +83 added lines, -83 removed lines

@@ -21,63 +21,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, $id, $type ) {
-		parent::__construct( $args, $id, $type );
-
-		$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, $id, $type ) {
+        parent::__construct( $args, $id, $type );
+
+        $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"
@@ -92,35 +92,35 @@ 
 			</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  = wp_json_encode( $this->timepicker );
-		$date_format = wp_json_encode( $this->date_format );
-		$no_calendar = wp_json_encode( $this->no_calendar );
+        // Should the widget include time picker?
+        $timepicker  = wp_json_encode( $this->timepicker );
+        $date_format = wp_json_encode( $this->date_format );
+        $no_calendar = wp_json_encode( $this->no_calendar );
 
-		// Set up the datetimepicker.
-		//
-		// See https://github.com/trentrichardson/jQuery-Timepicker-Addon
-		// See in http://trentrichardson.com/examples/timepicker.
+        // Set up the datetimepicker.
+        //
+        // See https://github.com/trentrichardson/jQuery-Timepicker-Addon
+        // See in http://trentrichardson.com/examples/timepicker.
 
-		$js = wp_json_encode(
-			array(
-				'enableTime' => $timepicker,
-				'noCalendar' => $no_calendar,
-				'time_24hr'  => true,
-				'dateFormat' => $date_format,
-			)
-		);
+        $js = wp_json_encode(
+            array(
+                'enableTime' => $timepicker,
+                'noCalendar' => $no_calendar,
+                'time_24hr'  => true,
+                'dateFormat' => $date_format,
+            )
+        );
 
         // phpcs:ignore WordPress.PHP.NoSilencedErrors.Discouraged
-		@ob_start();
-		?>
+        @ob_start();
+        ?>
 		<script type='text/javascript'>
 			(function ($) {
 				$(function () {
@@ -129,11 +129,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 +13 added lines, -13 removed lines

@@ -51,13 +51,13 @@ 
 	/**
 	 * {@inheritdoc}
 	 */
-	public function __construct( $args, $id, $type ) {
-		parent::__construct( $args, $id, $type );
+	public function __construct($args, $id, $type) {
+		parent::__construct($args, $id, $type);
 
 		$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 {
@@ -72,23 +72,23 @@ 
 	 *
 	 * @return string
 	 */
-	public function html_input( $date ) {
+	public function html_input($date) {
 
-		$this->log->debug( "Creating date input with date value $date..." );
+		$this->log->debug("Creating date input with date value $date...");
 
 		ob_start();
 		?>
 		<div class="wl-input-wrapper">
 			<input
 					type="text"
-					class="<?php echo esc_attr( $this->meta_name ); ?>"
-					name="wl_metaboxes[<?php echo esc_attr( $this->meta_name ); ?>][]"
-					value="<?php echo esc_attr( $date ); ?>"
+					class="<?php echo esc_attr($this->meta_name); ?>"
+					name="wl_metaboxes[<?php echo esc_attr($this->meta_name); ?>][]"
+					value="<?php echo esc_attr($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
@@ -100,9 +100,9 @@ 
 	public function html_wrapper_close() {
 
 		// Should the widget include time picker?
-		$timepicker  = wp_json_encode( $this->timepicker );
-		$date_format = wp_json_encode( $this->date_format );
-		$no_calendar = wp_json_encode( $this->no_calendar );
+		$timepicker  = wp_json_encode($this->timepicker);
+		$date_format = wp_json_encode($this->date_format);
+		$no_calendar = wp_json_encode($this->no_calendar);
 
 		// Set up the datetimepicker.
 		//
@@ -124,7 +124,7 @@ 
 		<script type='text/javascript'>
 			(function ($) {
 				$(function () {
-					$('.<?php echo esc_js( $this->meta_name ); ?>[type=text]').flatpickr(<?php echo esc_html( $js ); ?>);
+					$('.<?php echo esc_js($this->meta_name); ?>[type=text]').flatpickr(<?php echo esc_html($js); ?>);
 				});
 			})(jQuery);
 		</script>

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

Indentation +9 added lines, -9 removed lines

@@ -20,13 +20,13 @@ 
  */
 class Wordlift_Metabox_Field_Multiline extends Wl_Metabox_Field {
 
-	/**
-	 * @inheritdoc
-	 */
-	public function html_input( $text ) {
+    /**
+     * @inheritdoc
+     */
+    public function html_input( $text ) {
         // phpcs:ignore WordPress.PHP.NoSilencedErrors.Discouraged
-		@ob_start();
-		?>
+        @ob_start();
+        ?>
 			<div class="wl-input-wrapper">
 				<textarea
 					id="<?php echo esc_attr( $this->meta_name ); ?>"
@@ -40,8 +40,8 @@ 
 				</button>
 			</div>
 		<?php
-		$html = ob_get_clean();
+        $html = ob_get_clean();
 
-		return $html;
-	}
+        return $html;
+    }
 }

Spacing +6 added lines, -6 removed lines

@@ -23,20 +23,20 @@
 	/**
 	 * @inheritdoc
 	 */
-	public function html_input( $text ) {
+	public function html_input($text) {
         // phpcs:ignore WordPress.PHP.NoSilencedErrors.Discouraged
 		@ob_start();
 		?>
 			<div class="wl-input-wrapper">
 				<textarea
-					id="<?php echo esc_attr( $this->meta_name ); ?>"
-					class="<?php echo esc_attr( $this->meta_name ); ?>"
-					name="wl_metaboxes[<?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 esc_attr($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

src/wordlift/metabox/field/class-wl-metabox-field-address.php

Indentation +64 added lines, -64 removed lines

@@ -9,79 +9,79 @@ 
  */
 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, $id, $type ) {
+    /**
+     * 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, $id, $type ) {
 
-		$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 ), $id, $type );
-		}
+        // 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 ), $id, $type );
+        }
 
-		// $_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() {
+    /**
+     * Returns Field HTML (nonce included).
+     *
+     * @return string Field HTML
+     */
+    public function html() {
         // phpcs:ignore WordPress.PHP.NoSilencedErrors.Discouraged
-		@ob_start();
-		// Open main <div> for the Field, then insert label and nonce
-		?>
+        @ob_start();
+        // Open main <div> for the Field, then insert label and nonce
+        ?>
 			<div class='wl-field'>
 			<h3><?php echo esc_attr( $this->label ); ?></h3>
 			<?php $this->html_nonce( true ); ?>
 
 		<?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 esc_attr( $this->meta_name ); ?>][<?php echo esc_attr( $subfield->meta_name ); ?>]"
@@ -98,12 +98,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;
+    }
 }

Spacing +17 added lines, -17 removed lines

@@ -25,15 +25,15 @@ 
 	 * The structure of $args is:
 	 * array( 'address' => array( ... array of subfields ... ) )
 	 */
-	public function __construct( $args, $id, $type ) {
+	public function __construct($args, $id, $type) {
 
-		$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 ), $id, $type );
+		foreach (reset($args) as $key => $subfield) {
+			$this->subfields[] = new Wl_Metabox_Field(array($key => $subfield), $id, $type);
 		}
 
 		// $_POST array key in which we will pass the values
@@ -45,7 +45,7 @@ 
 	 */
 	public function get_data() {
 
-		foreach ( $this->subfields as $subfield ) {
+		foreach ($this->subfields as $subfield) {
 			$subfield->get_data();
 		}
 	}
@@ -55,11 +55,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);
 		}
 	}
 
@@ -74,26 +74,26 @@ 
 		// Open main <div> for the Field, then insert label and nonce
 		?>
 			<div class='wl-field'>
-			<h3><?php echo esc_attr( $this->label ); ?></h3>
-			<?php $this->html_nonce( true ); ?>
+			<h3><?php echo esc_attr($this->label); ?></h3>
+			<?php $this->html_nonce(true); ?>
 
 		<?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
-					for="wl_metaboxes[<?php echo esc_attr( $this->meta_name ); ?>][<?php echo esc_attr( $subfield->meta_name ); ?>]"
+					for="wl_metaboxes[<?php echo esc_attr($this->meta_name); ?>][<?php echo esc_attr($subfield->meta_name); ?>]"
 					style="display:inline-block; width:20%;"
 				>
-					<?php echo esc_html( $subfield->label ); ?>
+					<?php echo esc_html($subfield->label); ?>
 				</label>
 
 				<input
 					type="text"
-					name="wl_metaboxes[<?php echo esc_attr( $this->meta_name ); ?>][<?php echo esc_attr( $subfield->meta_name ); ?>]"
-					value="<?php echo esc_attr( $value ); ?>"
+					name="wl_metaboxes[<?php echo esc_attr($this->meta_name); ?>][<?php echo esc_attr($subfield->meta_name); ?>]"
+					value="<?php echo esc_attr($value); ?>"
 					style="width:78%;"
 				/>
 			</div>