insideout10 / wordlift-plugin

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

Doc Comments +1 added lines

@@ -86,6 +86,7 @@
 
 	/**
 	 * @inheritdoc
+	 * @param integer $count
 	 */
 	protected function get_add_button_html( $count ) {
 

Indentation +132 added lines, -132 removed lines

@@ -16,131 +16,131 @@ 
  */
 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 );
-
-		$merged = $this->filter_urls( $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.
-	 *
-	 * @return string The html fragment.
-	 * @since 3.15.0
-	 */
-	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
-  		'<button class="wl-add-input wl-add-input--link-button">'.esc_html__( 'Click here to manually add URLs', 'wordlift' ).'</button>'
+    /**
+     * @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 );
+
+        $merged = $this->filter_urls( $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.
+     *
+     * @return string The html fragment.
+     * @since 3.15.0
+     */
+    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
+            '<button class="wl-add-input wl-add-input--link-button">'.esc_html__( 'Click here to manually add URLs', 'wordlift' ).'</button>'
         .'<div style="display: none;"><div class="wl-input-wrapper">'
         ."<input type='text' id='$this->meta_name' name='wl_metaboxes[$this->meta_name][]' placeholder='$placeholder' />"
         . '<a href="#" class="wl-remove-input acf-icon -minus"></a>'
         .'</div></div>'
         . '<fieldset id="wl-input-container">'.$this->get_stored_values_html( $count ).'</fieldset>'
         .parent::get_add_custom_button_html( $count, 'Add Another URL', 'hide' );
-	}
+    }
 
-	/**
-	 * @inheritdoc
-	 */
-	protected function get_stored_values_html( &$count ) {
+    /**
+     * @inheritdoc
+     */
+    protected function get_stored_values_html( &$count ) {
 
-		return  parent::get_stored_values_html( $count );
-	}
+        return  parent::get_stored_values_html( $count );
+    }
 
-	/**
-	 * @inheritdoc
-	 */
-	public function html() {
+    /**
+     * @inheritdoc
+     */
+    public function html() {
 
-		// Open main <div> for the Field.
-		$html = $this->html_wrapper_open();
+        // Open main <div> for the Field.
+        $html = $this->html_wrapper_open();
 
-		// Label.
-		$html .= $this->get_heading_html();
+        // Label.
+        $html .= $this->get_heading_html();
 
-		// print nonce.
-		$html .= $this->html_nonce();
+        // print nonce.
+        $html .= $this->html_nonce();
 
-		// print data loaded from DB.
-		$count = 0;
+        // print data loaded from DB.
+        $count = 0;
 
-		// If cardinality allows it, print button to add new values.
-		$html .= $this->get_add_button_html( $count );
+        // If cardinality allows it, print button to add new values.
+        $html .= $this->get_add_button_html( $count );
 
-		// Close the HTML wrapper.
-		$html .= $this->html_wrapper_close();
+        // Close the HTML wrapper.
+        $html .= $this->html_wrapper_close();
 
-		return $html;
-	}
+        return $html;
+    }
 
-	/**
-	 * @inheritdoc
-	 */
-	public function html_input( $value ) {
-		@ob_start();
-		?>
+    /**
+     * @inheritdoc
+     */
+    public function html_input( $value ) {
+        @ob_start();
+        ?>
         <div class="wl-input-wrapper wl-input-wrapper-readonly">
             <input
                     type="text"
@@ -154,29 +154,29 @@ 
         </div>
 		<?php
 
-		$html = ob_get_clean();
-
-		return $html;
-	}
-
-	/**
-	 * @param array $urls
-	 *
-	 * @return array
-	 */
-	private function filter_urls( $urls ) {
-		$configuration_service = Wordlift_Configuration_Service::get_instance();
-		$dataset_uri           = $configuration_service->get_dataset_uri();
-
-		return array_filter( $urls, function ( $url ) use ( $dataset_uri ) {
-			$url_validation = filter_var( $url, FILTER_VALIDATE_URL );
-			if ( null === $dataset_uri ) {
-				return $url_validation;
-			}
-
-			// URLs should not start with local dataset uri.
-			return $url_validation && ( strpos( $url, $dataset_uri, 0 ) === false );
-		} );
-	}
+        $html = ob_get_clean();
+
+        return $html;
+    }
+
+    /**
+     * @param array $urls
+     *
+     * @return array
+     */
+    private function filter_urls( $urls ) {
+        $configuration_service = Wordlift_Configuration_Service::get_instance();
+        $dataset_uri           = $configuration_service->get_dataset_uri();
+
+        return array_filter( $urls, function ( $url ) use ( $dataset_uri ) {
+            $url_validation = filter_var( $url, FILTER_VALIDATE_URL );
+            if ( null === $dataset_uri ) {
+                return $url_validation;
+            }
+
+            // URLs should not start with local dataset uri.
+            return $url_validation && ( strpos( $url, $dataset_uri, 0 ) === false );
+        } );
+    }
 
 }

Spacing +31 added lines, -31 removed lines

@@ -19,46 +19,46 @@ 
 	/**
 	 * @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);
 
-		$merged = $this->filter_urls( $merged );
+		$merged = $this->filter_urls($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);
 	}
 
 	/**
@@ -80,33 +80,33 @@ 
 	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
-  		'<button class="wl-add-input wl-add-input--link-button">'.esc_html__( 'Click here to manually add URLs', 'wordlift' ).'</button>'
+  		'<button class="wl-add-input wl-add-input--link-button">'.esc_html__('Click here to manually add URLs', 'wordlift').'</button>'
         .'<div style="display: none;"><div class="wl-input-wrapper">'
         ."<input type='text' id='$this->meta_name' name='wl_metaboxes[$this->meta_name][]' placeholder='$placeholder' />"
         . '<a href="#" class="wl-remove-input acf-icon -minus"></a>'
         .'</div></div>'
-        . '<fieldset id="wl-input-container">'.$this->get_stored_values_html( $count ).'</fieldset>'
-        .parent::get_add_custom_button_html( $count, 'Add Another URL', 'hide' );
+        . '<fieldset id="wl-input-container">'.$this->get_stored_values_html($count).'</fieldset>'
+        .parent::get_add_custom_button_html($count, 'Add Another URL', 'hide');
 	}
 
 	/**
 	 * @inheritdoc
 	 */
-	protected function get_stored_values_html( &$count ) {
+	protected function get_stored_values_html(&$count) {
 
-		return  parent::get_stored_values_html( $count );
+		return  parent::get_stored_values_html($count);
 	}
 
 	/**
@@ -127,7 +127,7 @@ 
 		$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);
 
 		// Close the HTML wrapper.
 		$html .= $this->html_wrapper_close();
@@ -138,7 +138,7 @@ 
 	/**
 	 * @inheritdoc
 	 */
-	public function html_input( $value ) {
+	public function html_input($value) {
 		@ob_start();
 		?>
         <div class="wl-input-wrapper wl-input-wrapper-readonly">
@@ -164,18 +164,18 @@ 
 	 *
 	 * @return array
 	 */
-	private function filter_urls( $urls ) {
+	private function filter_urls($urls) {
 		$configuration_service = Wordlift_Configuration_Service::get_instance();
 		$dataset_uri           = $configuration_service->get_dataset_uri();
 
-		return array_filter( $urls, function ( $url ) use ( $dataset_uri ) {
-			$url_validation = filter_var( $url, FILTER_VALIDATE_URL );
-			if ( null === $dataset_uri ) {
+		return array_filter($urls, function($url) use ($dataset_uri) {
+			$url_validation = filter_var($url, FILTER_VALIDATE_URL);
+			if (null === $dataset_uri) {
 				return $url_validation;
 			}
 
 			// URLs should not start with local dataset uri.
-			return $url_validation && ( strpos( $url, $dataset_uri, 0 ) === false );
+			return $url_validation && (strpos($url, $dataset_uri, 0) === false);
 		} );
 	}
 

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

Doc Comments +1 added lines

@@ -432,6 +432,7 @@
 	 * @since 3.15.0
 	 *
 	 * @param int $count The current number of values.
+	 * @param string $label
 	 *
 	 * @return string The add button html code.
 	 */

Indentation +446 added lines, -446 removed lines

@@ -19,443 +19,443 @@ 
  */
 class WL_Metabox_Field {
 
-	/**
-	 * A {@link Wordlift_Log_Service} instance.
-	 *
-	 * @since  3.15.0
-	 * @access protected
-	 * @var \Wordlift_Log_Service $log A {@link Wordlift_Log_Service} instance.
-	 */
-	protected $log;
-
-	/**
-	 * The meta name for this field's value.
-	 *
-	 * @var string $meta_name The meta name for this field's value.
-	 */
-	public $meta_name;
-
-	/**
-	 * The custom field settings.
-	 *
-	 * @var null|array $raw_custom_field The custom field settings.
-	 */
-	public $raw_custom_field;
-
-	/**
-	 * The schema.org predicate.
-	 *
-	 * @var string $predicate The schema.org predicate.
-	 */
-	public $predicate;
-
-	/**
-	 * The field's label.
-	 *
-	 * @var string $label The field's label.
-	 */
-	public $label;
-
-	/**
-	 * The WordLift data type.
-	 *
-	 * @var string $expected_wl_type The WordLift data type.
-	 */
-	public $expected_wl_type;
-
-	/**
-	 * The RDF data type.
-	 *
-	 * @var string $expected_uri_type The RDF data type.
-	 */
-	public $expected_uri_type;
-
-	/**
-	 * The cardinality.
-	 *
-	 * @var int $cardinality The cardinality.
-	 */
-	public $cardinality;
-
-	/**
-	 * The current value.
-	 *
-	 * @var array $data The current value.
-	 */
-	public $data;
-
-	/**
-	 * The current {@link WP_Post} id.
-	 *
-	 * @since 3.15.3
-	 *
-	 * @var int The current {@link WP_Post} id.
-	 */
-	private $post_id;
-
-	/**
-	 * Create a {@link WL_Metabox_Field} instance.
-	 *
-	 * @param array $args An array of parameters.
-	 */
-	public function __construct( $args ) {
-
-		$this->log = Wordlift_Log_Service::get_logger( 'WL_Metabox_Field' );
-
-		if ( empty( $args ) ) {
-			return;
-		}
-
-		// Save a copy of the custom field's params.
-		$this->raw_custom_field = reset( $args );
-
-		// Extract meta name (post_meta key for the DB).
-		$this->meta_name = key( $args );
-
-		// Extract linked data predicate.
-		if ( isset( $this->raw_custom_field['predicate'] ) ) {
-			$this->predicate = $this->raw_custom_field['predicate'];
-		} else {
-			return;
-		}
-
-		// Extract human readable label.
-		$exploded_predicate = explode( '/', $this->predicate );
-
-		// Use the label defined for the property if set, otherwise the last part of the schema.org/xyz predicate.
-		$this->label = isset( $this->raw_custom_field['metabox']['label'] ) ? $this->raw_custom_field['metabox']['label'] : end( $exploded_predicate );
-
-		// Extract field constraints (numerosity, expected type).
-		// Default constaints: accept one string..
-		if ( isset( $this->raw_custom_field['type'] ) ) {
-			$this->expected_wl_type = $this->raw_custom_field['type'];
-		} else {
-			$this->expected_wl_type = Wordlift_Schema_Service::DATA_TYPE_STRING;
-		}
-
-		$this->cardinality = 1;
-		if ( isset( $this->raw_custom_field['constraints'] ) ) {
-
-			$constraints = $this->raw_custom_field['constraints'];
-
-			// Extract cardinality.
-			if ( isset( $constraints['cardinality'] ) ) {
-				$this->cardinality = $constraints['cardinality'];
-			}
-
-			// Which type of entity can we accept (e.g. Place, Event, ecc.)? .
-			if ( Wordlift_Schema_Service::DATA_TYPE_URI === $this->expected_wl_type && isset( $constraints['uri_type'] ) ) {
-				$this->expected_uri_type = is_array( $constraints['uri_type'] )
-					? $constraints['uri_type']
-					: array( $constraints['uri_type'] );
-			}
-		}
-
-		// Save early the post id to avoid other plugins messing up with it.
-		//
-		// See https://github.com/insideout10/wordlift-plugin/issues/665.
-		$this->post_id = get_the_ID();
-
-	}
-
-	/**
-	 * Return nonce HTML.
-	 *
-	 * Overwrite this method in a child class to obtain custom behaviour.
-	 */
-	public function html_nonce() {
-
-		return wp_nonce_field( 'wordlift_' . $this->meta_name . '_entity_box', 'wordlift_' . $this->meta_name . '_entity_box_nonce', true, false );
-	}
-
-	/**
-	 * Verify nonce.
-	 *
-	 * Overwrite this method in a child class to obtain custom behaviour.
-	 *
-	 * @return bool Nonce verification.
-	 */
-	public function verify_nonce() {
-
-		$nonce_name   = 'wordlift_' . $this->meta_name . '_entity_box_nonce';
-		$nonce_verify = 'wordlift_' . $this->meta_name . '_entity_box';
-
-		if ( ! isset( $_POST[ $nonce_name ] ) ) {
-			return false;
-		}
-
-		// Verify that the nonce is valid.
-		return wp_verify_nonce( $_POST[ $nonce_name ], $nonce_verify );
-	}
-
-	/**
-	 * 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() {
-
-		// Get the post id and load the data.
-		$post_id    = $this->post_id;
-		$this->data = get_post_meta( $post_id, $this->meta_name );
-
-	}
-
-	/**
-	 * 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.
-	 */
-	public function sanitize_data( $values ) {
-
-		$sanitized_data = array();
-
-		if ( ! is_array( $values ) ) {
-			$values = array( $values );
-		}
-
-		foreach ( $values as $value ) {
-			$sanitized_value = $this->sanitize_data_filter( $value );
-			if ( ! is_null( $sanitized_value ) ) {
-				$sanitized_data[] = $sanitized_value;
-			}
-		}
-
-		$this->data = $sanitized_data;
-	}
-
-	/**
-	 * 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 ) {
-
-		// TODO: all fields should provide their own sanitize which shouldn't
-		// be part of a UI class.
-
-		// If the field provides its own validation, use it.
-		if ( isset( $this->raw_custom_field['sanitize'] ) ) {
-			return call_user_func( $this->raw_custom_field['sanitize'], $value );
-		}
-
-		if ( ! is_null( $value ) && '' !== $value ) {         // do not use 'empty()' -> https://www.virendrachandak.com/techtalk/php-isset-vs-empty-vs-is_null/ .
-			return $value;
-		}
-
-		return null;
-	}
-
-	/**
-	 * 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 ) {
-
-		// Will sanitize data and store them in $field->data.
-		$this->sanitize_data( $values );
-
-		// Bail out, if the post id isn't set in the request or isn't numeric.
-		//
-		// See https://github.com/insideout10/wordlift-plugin/issues/665.
-		if ( ! isset( $_POST['post_ID'] ) || ! is_numeric( $_POST['post_ID'] ) ) {
-			return;
-		}
-
-		$entity_id = intval( $_POST['post_ID'] );
-
-		// Take away old values.
-		delete_post_meta( $entity_id, $this->meta_name );
-
-		// insert new values, respecting cardinality.
-		$single = ( 1 === $this->cardinality );
-		foreach ( $this->data as $value ) {
-			$this->log->trace( "Saving $value to $this->meta_name for entity $entity_id..." );
-			// To avoid duplicate values
-			delete_post_meta( $entity_id, $this->meta_name, $value );
-			$meta_id = add_post_meta( $entity_id, $this->meta_name, $value, $single );
-			$this->log->debug( "$value to $this->meta_name for entity $entity_id saved with id $meta_id." );
-		}
-	}
-
-	/**
-	 * 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() {
-
-		return "<div class='wl-field' data-cardinality='$this->cardinality'>";
-	}
-
-	/**
-	 * 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() {
-
-		// 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;
-		$html  .= $this->get_stored_values_html( $count );
-
-		// Print the empty <input> to add new values.
-		if ( 0 === $count ) { // } || $count < $this->cardinality ) { DO NOT print empty inputs unless requested by the editor since fields might support empty strings.
-			$this->log->debug( 'Going to print an empty HTML input...' );
-			$html .= $this->html_input( '' );    // Will print an empty <input>.
-			$count ++;
-		}
-
-		// If cardinality allows it, print button to add new values.
-		$html .= $this->get_add_button_html( $count );
-
-		// Close the HTML wrapper.
-		$html .= $this->html_wrapper_close();
-
-		return $html;
-	}
-
-	/**
-	 * Print the heading with the label for the metabox.
-	 *
-	 * @since 3.15.0
-	 * @return string The heading html fragment.
-	 */
-	protected function get_heading_html() {
-
-		return "<h3>$this->label</h3>";
-	}
-
-	/**
-	 * Print the stored values.
-	 *
-	 * @since 3.15.0
-	 *
-	 * @param int $count An output value: the number of printed values.
-	 *
-	 * @return string The html fragment.
-	 */
-	protected function get_stored_values_html( &$count ) {
-
-		$html = '';
-
-		// print data loaded from DB.
-		$count = 0;
-		if ( $this->data ) {
-			foreach ( $this->data as $value ) {
-				if ( $count < $this->cardinality ) {
-					$this->log->debug( "Going to print an HTML input #$count with $value..." );
-					$fragment = $this->html_input( $value );
-
-					// If the fragment is empty, continue to the next one. This is necessary because the
-					// metabox may reference an invalid value which would cause the metabox not to print,
-					// returning an empty html fragment.
-					//
-					// See https://github.com/insideout10/wordlift-plugin/issues/818
-					if ( '' === $fragment ) {
-						continue;
-					}
-
-					$html .= $fragment;
-					$count ++;
-				}
-			}
-		}
-
-		return $html;
-	}
-
-	/**
-	 * Get the add button html.
-	 *
-	 * This function is protected, allowing extending class to further customize
-	 * the add button html code.
-	 *
-	 * @since 3.15.0
-	 *
-	 * @param int $count The current number of values.
-	 *
-	 * @return string The add button html code.
-	 */
-	protected function get_add_button_html( $count ) {
-
-		// If cardinality allows it, print button to add new values.
-		if ( $count < $this->cardinality ) {
-			return '<button class="button wl-add-input wl-button" type="button">' . esc_html__( 'Add' ) . '</button>';
-		}
-
-		// Return an empty string.
-		return '';
-	}
-
-	/**
-	 * Get the add custom button html.
-	 *
-	 * This function is protected, allowing extending class to further customize
-	 * the add button html code.
-	 *
-	 * @since 3.15.0
-	 *
-	 * @param int $count The current number of values.
-	 *
-	 * @return string The add button html code.
-	 */
-	protected function get_add_custom_button_html( $count, $label, $class = '' ) {
-
-		// If cardinality allows it, print button to add new values.
-		if ( $count < $this->cardinality ) {
-			return '<button class="button wl-add-input wl-button wl-add-input--sameas-add-button '.$class.'" type="button">' . esc_html__( $label ) . '</button>';
-		}
-
-		// Return an empty string.
-		return '';
-	}
-
-	/**
-	 * Return a single <input> tag for the Field.
-	 *
-	 * @param mixed $value Input value.
-	 *
-	 * @return string The html code fragment.
-	 */
-	public function html_input( $value ) {
-		@ob_start();
-		?>
+    /**
+     * A {@link Wordlift_Log_Service} instance.
+     *
+     * @since  3.15.0
+     * @access protected
+     * @var \Wordlift_Log_Service $log A {@link Wordlift_Log_Service} instance.
+     */
+    protected $log;
+
+    /**
+     * The meta name for this field's value.
+     *
+     * @var string $meta_name The meta name for this field's value.
+     */
+    public $meta_name;
+
+    /**
+     * The custom field settings.
+     *
+     * @var null|array $raw_custom_field The custom field settings.
+     */
+    public $raw_custom_field;
+
+    /**
+     * The schema.org predicate.
+     *
+     * @var string $predicate The schema.org predicate.
+     */
+    public $predicate;
+
+    /**
+     * The field's label.
+     *
+     * @var string $label The field's label.
+     */
+    public $label;
+
+    /**
+     * The WordLift data type.
+     *
+     * @var string $expected_wl_type The WordLift data type.
+     */
+    public $expected_wl_type;
+
+    /**
+     * The RDF data type.
+     *
+     * @var string $expected_uri_type The RDF data type.
+     */
+    public $expected_uri_type;
+
+    /**
+     * The cardinality.
+     *
+     * @var int $cardinality The cardinality.
+     */
+    public $cardinality;
+
+    /**
+     * The current value.
+     *
+     * @var array $data The current value.
+     */
+    public $data;
+
+    /**
+     * The current {@link WP_Post} id.
+     *
+     * @since 3.15.3
+     *
+     * @var int The current {@link WP_Post} id.
+     */
+    private $post_id;
+
+    /**
+     * Create a {@link WL_Metabox_Field} instance.
+     *
+     * @param array $args An array of parameters.
+     */
+    public function __construct( $args ) {
+
+        $this->log = Wordlift_Log_Service::get_logger( 'WL_Metabox_Field' );
+
+        if ( empty( $args ) ) {
+            return;
+        }
+
+        // Save a copy of the custom field's params.
+        $this->raw_custom_field = reset( $args );
+
+        // Extract meta name (post_meta key for the DB).
+        $this->meta_name = key( $args );
+
+        // Extract linked data predicate.
+        if ( isset( $this->raw_custom_field['predicate'] ) ) {
+            $this->predicate = $this->raw_custom_field['predicate'];
+        } else {
+            return;
+        }
+
+        // Extract human readable label.
+        $exploded_predicate = explode( '/', $this->predicate );
+
+        // Use the label defined for the property if set, otherwise the last part of the schema.org/xyz predicate.
+        $this->label = isset( $this->raw_custom_field['metabox']['label'] ) ? $this->raw_custom_field['metabox']['label'] : end( $exploded_predicate );
+
+        // Extract field constraints (numerosity, expected type).
+        // Default constaints: accept one string..
+        if ( isset( $this->raw_custom_field['type'] ) ) {
+            $this->expected_wl_type = $this->raw_custom_field['type'];
+        } else {
+            $this->expected_wl_type = Wordlift_Schema_Service::DATA_TYPE_STRING;
+        }
+
+        $this->cardinality = 1;
+        if ( isset( $this->raw_custom_field['constraints'] ) ) {
+
+            $constraints = $this->raw_custom_field['constraints'];
+
+            // Extract cardinality.
+            if ( isset( $constraints['cardinality'] ) ) {
+                $this->cardinality = $constraints['cardinality'];
+            }
+
+            // Which type of entity can we accept (e.g. Place, Event, ecc.)? .
+            if ( Wordlift_Schema_Service::DATA_TYPE_URI === $this->expected_wl_type && isset( $constraints['uri_type'] ) ) {
+                $this->expected_uri_type = is_array( $constraints['uri_type'] )
+                    ? $constraints['uri_type']
+                    : array( $constraints['uri_type'] );
+            }
+        }
+
+        // Save early the post id to avoid other plugins messing up with it.
+        //
+        // See https://github.com/insideout10/wordlift-plugin/issues/665.
+        $this->post_id = get_the_ID();
+
+    }
+
+    /**
+     * Return nonce HTML.
+     *
+     * Overwrite this method in a child class to obtain custom behaviour.
+     */
+    public function html_nonce() {
+
+        return wp_nonce_field( 'wordlift_' . $this->meta_name . '_entity_box', 'wordlift_' . $this->meta_name . '_entity_box_nonce', true, false );
+    }
+
+    /**
+     * Verify nonce.
+     *
+     * Overwrite this method in a child class to obtain custom behaviour.
+     *
+     * @return bool Nonce verification.
+     */
+    public function verify_nonce() {
+
+        $nonce_name   = 'wordlift_' . $this->meta_name . '_entity_box_nonce';
+        $nonce_verify = 'wordlift_' . $this->meta_name . '_entity_box';
+
+        if ( ! isset( $_POST[ $nonce_name ] ) ) {
+            return false;
+        }
+
+        // Verify that the nonce is valid.
+        return wp_verify_nonce( $_POST[ $nonce_name ], $nonce_verify );
+    }
+
+    /**
+     * 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() {
+
+        // Get the post id and load the data.
+        $post_id    = $this->post_id;
+        $this->data = get_post_meta( $post_id, $this->meta_name );
+
+    }
+
+    /**
+     * 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.
+     */
+    public function sanitize_data( $values ) {
+
+        $sanitized_data = array();
+
+        if ( ! is_array( $values ) ) {
+            $values = array( $values );
+        }
+
+        foreach ( $values as $value ) {
+            $sanitized_value = $this->sanitize_data_filter( $value );
+            if ( ! is_null( $sanitized_value ) ) {
+                $sanitized_data[] = $sanitized_value;
+            }
+        }
+
+        $this->data = $sanitized_data;
+    }
+
+    /**
+     * 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 ) {
+
+        // TODO: all fields should provide their own sanitize which shouldn't
+        // be part of a UI class.
+
+        // If the field provides its own validation, use it.
+        if ( isset( $this->raw_custom_field['sanitize'] ) ) {
+            return call_user_func( $this->raw_custom_field['sanitize'], $value );
+        }
+
+        if ( ! is_null( $value ) && '' !== $value ) {         // do not use 'empty()' -> https://www.virendrachandak.com/techtalk/php-isset-vs-empty-vs-is_null/ .
+            return $value;
+        }
+
+        return null;
+    }
+
+    /**
+     * 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 ) {
+
+        // Will sanitize data and store them in $field->data.
+        $this->sanitize_data( $values );
+
+        // Bail out, if the post id isn't set in the request or isn't numeric.
+        //
+        // See https://github.com/insideout10/wordlift-plugin/issues/665.
+        if ( ! isset( $_POST['post_ID'] ) || ! is_numeric( $_POST['post_ID'] ) ) {
+            return;
+        }
+
+        $entity_id = intval( $_POST['post_ID'] );
+
+        // Take away old values.
+        delete_post_meta( $entity_id, $this->meta_name );
+
+        // insert new values, respecting cardinality.
+        $single = ( 1 === $this->cardinality );
+        foreach ( $this->data as $value ) {
+            $this->log->trace( "Saving $value to $this->meta_name for entity $entity_id..." );
+            // To avoid duplicate values
+            delete_post_meta( $entity_id, $this->meta_name, $value );
+            $meta_id = add_post_meta( $entity_id, $this->meta_name, $value, $single );
+            $this->log->debug( "$value to $this->meta_name for entity $entity_id saved with id $meta_id." );
+        }
+    }
+
+    /**
+     * 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() {
+
+        return "<div class='wl-field' data-cardinality='$this->cardinality'>";
+    }
+
+    /**
+     * 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() {
+
+        // 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;
+        $html  .= $this->get_stored_values_html( $count );
+
+        // Print the empty <input> to add new values.
+        if ( 0 === $count ) { // } || $count < $this->cardinality ) { DO NOT print empty inputs unless requested by the editor since fields might support empty strings.
+            $this->log->debug( 'Going to print an empty HTML input...' );
+            $html .= $this->html_input( '' );    // Will print an empty <input>.
+            $count ++;
+        }
+
+        // If cardinality allows it, print button to add new values.
+        $html .= $this->get_add_button_html( $count );
+
+        // Close the HTML wrapper.
+        $html .= $this->html_wrapper_close();
+
+        return $html;
+    }
+
+    /**
+     * Print the heading with the label for the metabox.
+     *
+     * @since 3.15.0
+     * @return string The heading html fragment.
+     */
+    protected function get_heading_html() {
+
+        return "<h3>$this->label</h3>";
+    }
+
+    /**
+     * Print the stored values.
+     *
+     * @since 3.15.0
+     *
+     * @param int $count An output value: the number of printed values.
+     *
+     * @return string The html fragment.
+     */
+    protected function get_stored_values_html( &$count ) {
+
+        $html = '';
+
+        // print data loaded from DB.
+        $count = 0;
+        if ( $this->data ) {
+            foreach ( $this->data as $value ) {
+                if ( $count < $this->cardinality ) {
+                    $this->log->debug( "Going to print an HTML input #$count with $value..." );
+                    $fragment = $this->html_input( $value );
+
+                    // If the fragment is empty, continue to the next one. This is necessary because the
+                    // metabox may reference an invalid value which would cause the metabox not to print,
+                    // returning an empty html fragment.
+                    //
+                    // See https://github.com/insideout10/wordlift-plugin/issues/818
+                    if ( '' === $fragment ) {
+                        continue;
+                    }
+
+                    $html .= $fragment;
+                    $count ++;
+                }
+            }
+        }
+
+        return $html;
+    }
+
+    /**
+     * Get the add button html.
+     *
+     * This function is protected, allowing extending class to further customize
+     * the add button html code.
+     *
+     * @since 3.15.0
+     *
+     * @param int $count The current number of values.
+     *
+     * @return string The add button html code.
+     */
+    protected function get_add_button_html( $count ) {
+
+        // If cardinality allows it, print button to add new values.
+        if ( $count < $this->cardinality ) {
+            return '<button class="button wl-add-input wl-button" type="button">' . esc_html__( 'Add' ) . '</button>';
+        }
+
+        // Return an empty string.
+        return '';
+    }
+
+    /**
+     * Get the add custom button html.
+     *
+     * This function is protected, allowing extending class to further customize
+     * the add button html code.
+     *
+     * @since 3.15.0
+     *
+     * @param int $count The current number of values.
+     *
+     * @return string The add button html code.
+     */
+    protected function get_add_custom_button_html( $count, $label, $class = '' ) {
+
+        // If cardinality allows it, print button to add new values.
+        if ( $count < $this->cardinality ) {
+            return '<button class="button wl-add-input wl-button wl-add-input--sameas-add-button '.$class.'" type="button">' . esc_html__( $label ) . '</button>';
+        }
+
+        // Return an empty string.
+        return '';
+    }
+
+    /**
+     * Return a single <input> tag for the Field.
+     *
+     * @param mixed $value Input value.
+     *
+     * @return string The html code fragment.
+     */
+    public function html_input( $value ) {
+        @ob_start();
+        ?>
         <div class="wl-input-wrapper">
             <input
                     type="text"
@@ -470,17 +470,17 @@ 
             </button>
         </div>
 		<?php
-		$html = ob_get_clean();
+        $html = ob_get_clean();
 
-		return $html;
-	}
+        return $html;
+    }
 
-	/**
-	 * Returns closing for the wrapper HTML tag.
-	 */
-	public function html_wrapper_close() {
+    /**
+     * Returns closing for the wrapper HTML tag.
+     */
+    public function html_wrapper_close() {
 
-		return '</div>';
-	}
+        return '</div>';
+    }
 
 }

Spacing +65 added lines, -65 removed lines

@@ -98,56 +98,56 @@ 
 	 *
 	 * @param array $args An array of parameters.
 	 */
-	public function __construct( $args ) {
+	public function __construct($args) {
 
-		$this->log = Wordlift_Log_Service::get_logger( 'WL_Metabox_Field' );
+		$this->log = Wordlift_Log_Service::get_logger('WL_Metabox_Field');
 
-		if ( empty( $args ) ) {
+		if (empty($args)) {
 			return;
 		}
 
 		// Save a copy of the custom field's params.
-		$this->raw_custom_field = reset( $args );
+		$this->raw_custom_field = reset($args);
 
 		// Extract meta name (post_meta key for the DB).
-		$this->meta_name = key( $args );
+		$this->meta_name = key($args);
 
 		// Extract linked data predicate.
-		if ( isset( $this->raw_custom_field['predicate'] ) ) {
+		if (isset($this->raw_custom_field['predicate'])) {
 			$this->predicate = $this->raw_custom_field['predicate'];
 		} else {
 			return;
 		}
 
 		// Extract human readable label.
-		$exploded_predicate = explode( '/', $this->predicate );
+		$exploded_predicate = explode('/', $this->predicate);
 
 		// Use the label defined for the property if set, otherwise the last part of the schema.org/xyz predicate.
-		$this->label = isset( $this->raw_custom_field['metabox']['label'] ) ? $this->raw_custom_field['metabox']['label'] : end( $exploded_predicate );
+		$this->label = isset($this->raw_custom_field['metabox']['label']) ? $this->raw_custom_field['metabox']['label'] : end($exploded_predicate);
 
 		// Extract field constraints (numerosity, expected type).
 		// Default constaints: accept one string..
-		if ( isset( $this->raw_custom_field['type'] ) ) {
+		if (isset($this->raw_custom_field['type'])) {
 			$this->expected_wl_type = $this->raw_custom_field['type'];
 		} else {
 			$this->expected_wl_type = Wordlift_Schema_Service::DATA_TYPE_STRING;
 		}
 
 		$this->cardinality = 1;
-		if ( isset( $this->raw_custom_field['constraints'] ) ) {
+		if (isset($this->raw_custom_field['constraints'])) {
 
 			$constraints = $this->raw_custom_field['constraints'];
 
 			// Extract cardinality.
-			if ( isset( $constraints['cardinality'] ) ) {
+			if (isset($constraints['cardinality'])) {
 				$this->cardinality = $constraints['cardinality'];
 			}
 
 			// Which type of entity can we accept (e.g. Place, Event, ecc.)? .
-			if ( Wordlift_Schema_Service::DATA_TYPE_URI === $this->expected_wl_type && isset( $constraints['uri_type'] ) ) {
-				$this->expected_uri_type = is_array( $constraints['uri_type'] )
+			if (Wordlift_Schema_Service::DATA_TYPE_URI === $this->expected_wl_type && isset($constraints['uri_type'])) {
+				$this->expected_uri_type = is_array($constraints['uri_type'])
 					? $constraints['uri_type']
-					: array( $constraints['uri_type'] );
+					: array($constraints['uri_type']);
 			}
 		}
 
@@ -165,7 +165,7 @@ 
 	 */
 	public function html_nonce() {
 
-		return wp_nonce_field( 'wordlift_' . $this->meta_name . '_entity_box', 'wordlift_' . $this->meta_name . '_entity_box_nonce', true, false );
+		return wp_nonce_field('wordlift_'.$this->meta_name.'_entity_box', 'wordlift_'.$this->meta_name.'_entity_box_nonce', true, false);
 	}
 
 	/**
@@ -177,15 +177,15 @@ 
 	 */
 	public function verify_nonce() {
 
-		$nonce_name   = 'wordlift_' . $this->meta_name . '_entity_box_nonce';
-		$nonce_verify = 'wordlift_' . $this->meta_name . '_entity_box';
+		$nonce_name   = 'wordlift_'.$this->meta_name.'_entity_box_nonce';
+		$nonce_verify = 'wordlift_'.$this->meta_name.'_entity_box';
 
-		if ( ! isset( $_POST[ $nonce_name ] ) ) {
+		if ( ! isset($_POST[$nonce_name])) {
 			return false;
 		}
 
 		// Verify that the nonce is valid.
-		return wp_verify_nonce( $_POST[ $nonce_name ], $nonce_verify );
+		return wp_verify_nonce($_POST[$nonce_name], $nonce_verify);
 	}
 
 	/**
@@ -197,7 +197,7 @@ 
 
 		// Get the post id and load the data.
 		$post_id    = $this->post_id;
-		$this->data = get_post_meta( $post_id, $this->meta_name );
+		$this->data = get_post_meta($post_id, $this->meta_name);
 
 	}
 
@@ -211,17 +211,17 @@ 
 	 * @param array $values Array of values to be sanitized and then stored into
 	 *                      $this->data.
 	 */
-	public function sanitize_data( $values ) {
+	public function sanitize_data($values) {
 
 		$sanitized_data = array();
 
-		if ( ! is_array( $values ) ) {
-			$values = array( $values );
+		if ( ! is_array($values)) {
+			$values = array($values);
 		}
 
-		foreach ( $values as $value ) {
-			$sanitized_value = $this->sanitize_data_filter( $value );
-			if ( ! is_null( $sanitized_value ) ) {
+		foreach ($values as $value) {
+			$sanitized_value = $this->sanitize_data_filter($value);
+			if ( ! is_null($sanitized_value)) {
 				$sanitized_data[] = $sanitized_value;
 			}
 		}
@@ -239,17 +239,17 @@ 
 	 *
 	 * @return mixed Returns sanitized value, or null.
 	 */
-	public function sanitize_data_filter( $value ) {
+	public function sanitize_data_filter($value) {
 
 		// TODO: all fields should provide their own sanitize which shouldn't
 		// be part of a UI class.
 
 		// If the field provides its own validation, use it.
-		if ( isset( $this->raw_custom_field['sanitize'] ) ) {
-			return call_user_func( $this->raw_custom_field['sanitize'], $value );
+		if (isset($this->raw_custom_field['sanitize'])) {
+			return call_user_func($this->raw_custom_field['sanitize'], $value);
 		}
 
-		if ( ! is_null( $value ) && '' !== $value ) {         // do not use 'empty()' -> https://www.virendrachandak.com/techtalk/php-isset-vs-empty-vs-is_null/ .
+		if ( ! is_null($value) && '' !== $value) {         // do not use 'empty()' -> https://www.virendrachandak.com/techtalk/php-isset-vs-empty-vs-is_null/ .
 			return $value;
 		}
 
@@ -263,31 +263,31 @@ 
 	 *
 	 * @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) {
 
 		// Will sanitize data and store them in $field->data.
-		$this->sanitize_data( $values );
+		$this->sanitize_data($values);
 
 		// Bail out, if the post id isn't set in the request or isn't numeric.
 		//
 		// See https://github.com/insideout10/wordlift-plugin/issues/665.
-		if ( ! isset( $_POST['post_ID'] ) || ! is_numeric( $_POST['post_ID'] ) ) {
+		if ( ! isset($_POST['post_ID']) || ! is_numeric($_POST['post_ID'])) {
 			return;
 		}
 
-		$entity_id = intval( $_POST['post_ID'] );
+		$entity_id = intval($_POST['post_ID']);
 
 		// Take away old values.
-		delete_post_meta( $entity_id, $this->meta_name );
+		delete_post_meta($entity_id, $this->meta_name);
 
 		// insert new values, respecting cardinality.
-		$single = ( 1 === $this->cardinality );
-		foreach ( $this->data as $value ) {
-			$this->log->trace( "Saving $value to $this->meta_name for entity $entity_id..." );
+		$single = (1 === $this->cardinality);
+		foreach ($this->data as $value) {
+			$this->log->trace("Saving $value to $this->meta_name for entity $entity_id...");
 			// To avoid duplicate values
-			delete_post_meta( $entity_id, $this->meta_name, $value );
-			$meta_id = add_post_meta( $entity_id, $this->meta_name, $value, $single );
-			$this->log->debug( "$value to $this->meta_name for entity $entity_id saved with id $meta_id." );
+			delete_post_meta($entity_id, $this->meta_name, $value);
+			$meta_id = add_post_meta($entity_id, $this->meta_name, $value, $single);
+			$this->log->debug("$value to $this->meta_name for entity $entity_id saved with id $meta_id.");
 		}
 	}
 
@@ -332,17 +332,17 @@ 
 
 		// print data loaded from DB.
 		$count = 0;
-		$html  .= $this->get_stored_values_html( $count );
+		$html .= $this->get_stored_values_html($count);
 
 		// Print the empty <input> to add new values.
-		if ( 0 === $count ) { // } || $count < $this->cardinality ) { DO NOT print empty inputs unless requested by the editor since fields might support empty strings.
-			$this->log->debug( 'Going to print an empty HTML input...' );
-			$html .= $this->html_input( '' );    // Will print an empty <input>.
-			$count ++;
+		if (0 === $count) { // } || $count < $this->cardinality ) { DO NOT print empty inputs unless requested by the editor since fields might support empty strings.
+			$this->log->debug('Going to print an empty HTML input...');
+			$html .= $this->html_input(''); // Will print an empty <input>.
+			$count++;
 		}
 
 		// If cardinality allows it, print button to add new values.
-		$html .= $this->get_add_button_html( $count );
+		$html .= $this->get_add_button_html($count);
 
 		// Close the HTML wrapper.
 		$html .= $this->html_wrapper_close();
@@ -370,29 +370,29 @@ 
 	 *
 	 * @return string The html fragment.
 	 */
-	protected function get_stored_values_html( &$count ) {
+	protected function get_stored_values_html(&$count) {
 
 		$html = '';
 
 		// print data loaded from DB.
 		$count = 0;
-		if ( $this->data ) {
-			foreach ( $this->data as $value ) {
-				if ( $count < $this->cardinality ) {
-					$this->log->debug( "Going to print an HTML input #$count with $value..." );
-					$fragment = $this->html_input( $value );
+		if ($this->data) {
+			foreach ($this->data as $value) {
+				if ($count < $this->cardinality) {
+					$this->log->debug("Going to print an HTML input #$count with $value...");
+					$fragment = $this->html_input($value);
 
 					// If the fragment is empty, continue to the next one. This is necessary because the
 					// metabox may reference an invalid value which would cause the metabox not to print,
 					// returning an empty html fragment.
 					//
 					// See https://github.com/insideout10/wordlift-plugin/issues/818
-					if ( '' === $fragment ) {
+					if ('' === $fragment) {
 						continue;
 					}
 
 					$html .= $fragment;
-					$count ++;
+					$count++;
 				}
 			}
 		}
@@ -412,11 +412,11 @@ 
 	 *
 	 * @return string The add button html code.
 	 */
-	protected function get_add_button_html( $count ) {
+	protected function get_add_button_html($count) {
 
 		// If cardinality allows it, print button to add new values.
-		if ( $count < $this->cardinality ) {
-			return '<button class="button wl-add-input wl-button" type="button">' . esc_html__( 'Add' ) . '</button>';
+		if ($count < $this->cardinality) {
+			return '<button class="button wl-add-input wl-button" type="button">'.esc_html__('Add').'</button>';
 		}
 
 		// Return an empty string.
@@ -435,11 +435,11 @@ 
 	 *
 	 * @return string The add button html code.
 	 */
-	protected function get_add_custom_button_html( $count, $label, $class = '' ) {
+	protected function get_add_custom_button_html($count, $label, $class = '') {
 
 		// If cardinality allows it, print button to add new values.
-		if ( $count < $this->cardinality ) {
-			return '<button class="button wl-add-input wl-button wl-add-input--sameas-add-button '.$class.'" type="button">' . esc_html__( $label ) . '</button>';
+		if ($count < $this->cardinality) {
+			return '<button class="button wl-add-input wl-button wl-add-input--sameas-add-button '.$class.'" type="button">'.esc_html__($label).'</button>';
 		}
 
 		// Return an empty string.
@@ -453,20 +453,20 @@ 
 	 *
 	 * @return string The html code fragment.
 	 */
-	public function html_input( $value ) {
+	public function html_input($value) {
 		@ob_start();
 		?>
         <div class="wl-input-wrapper">
             <input
                     type="text"
-                    id="<?php echo esc_attr( $this->meta_name ); ?>"
+                    id="<?php echo esc_attr($this->meta_name); ?>"
                     name="wl_metaboxes[<?php echo $this->meta_name ?>][]"
-                    value="<?php echo esc_attr( $value ); ?>"
+                    value="<?php echo esc_attr($value); ?>"
                     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