insideout10 / wordlift-plugin

src/admin/elements/class-wordlift-admin-author-element.php

Indentation +120 added lines, -120 removed lines

@@ -19,125 +19,125 @@
  */
 class Wordlift_Admin_Author_Element implements Wordlift_Admin_Element {
 
-	/**
-	 * The {@link Wordlift_Publisher_Service} instance.
-	 *
-	 * @since  3.14.0
-	 * @access private
-	 * @var \Wordlift_Publisher_Service $publisher_service The {@link Wordlift_Publisher_Service} instance.
-	 */
-	private $publisher_service;
-
-	/**
-	 * A {@link Wordlift_Admin_Select2_Element} instance.
-	 *
-	 * @since  3.14.0
-	 * @access private
-	 * @var \Wordlift_Admin_Select2_Element $select_element A {@link Wordlift_Admin_Select2_Element} instance.
-	 */
-	private $select_element;
-
-	/**
-	 * Create a {@link Wordlift_Admin_Person_Element} instance.
-	 *
-	 * @param \Wordlift_Publisher_Service     $publisher_service The {@link Wordlift_Publisher_Service} instance.
-	 * @param \Wordlift_Admin_Select2_Element $select_element The {@link Wordlift_Admin_Select_Element} instance.
-	 *
-	 * @since 3.14.0
-	 */
-	public function __construct( $publisher_service, $select_element ) {
-
-		$this->publisher_service = $publisher_service;
-
-		// Child elements.
-		$this->select_element = $select_element;
-
-	}
-
-	/**
-	 * @inheritdoc
-	 */
-	public function render( $args ) {
-
-		// Parse the arguments and merge with default values.
-		$params = wp_parse_args(
-			$args,
-			array(
-				'id'             => uniqid( 'wl-input-' ),
-				'name'           => uniqid( 'wl-input-' ),
-				'current_entity' => 0,
-			)
-		);
-
-		$current_entity_id = $params['current_entity'];
-		$data              = $this->publisher_service->query();
-
-		// Set a default to show when no entity is associated and a way to unassign.
-		array_unshift(
-			$data,
-			array(
-				'id'            => '0',
-				'text'          => '<em>' . __( '(none)', 'wordlift' ) . '</em>',
-				'type'          => '',
-				'thumbnail_url' => plugin_dir_url( __DIR__ ) . 'images/pixel.png',
-			)
-		);
-
-		// Finally do the render, passing along also the current selected entity
-		// id and the options data.
-		return $this->do_render( $params, $current_entity_id, $data );
-	}
-
-	/**
-	 * Render the `select` using the provided parameters.
-	 *
-	 * @param array $params The array of parameters from the `render` function.
-	 * @param int   $current_post_id The currently selected {@link WP_Post} `id`.
-	 * @param array $data An array of Select2 options.
-	 *
-	 * @return \Wordlift_Admin_Author_Element $this Return this element.
-	 * @since 3.14.0
-	 */
-	protected function do_render( $params, $current_post_id, $data ) {
-
-		// Queue the script which will initialize the select and style it.
-		wp_enqueue_script( 'wl-author-element', plugin_dir_url( __DIR__ ) . 'js/1/author.js', array( 'wordlift-select2' ), WORDLIFT_VERSION, false );
-		wp_enqueue_style( 'wl-author-element', plugin_dir_url( __DIR__ ) . 'js/1/author.css', array(), WORDLIFT_VERSION );
-
-		// Prepare the URLs for entities which don't have logos.
-		$person_thumbnail_url       = plugin_dir_url( __DIR__ ) . '../images/person.png';
-		$organization_thumbnail_url = plugin_dir_url( __DIR__ ) . '../images/organization.png';
-
-		// Get the current post.
-		$current_post = $current_post_id ? get_post( $current_post_id ) : null;
-
-		// Finally render the Select.
-		$this->select_element->render(
-			array(
-				// Id.
-				'id'      => $params['id'],
-				// Name.
-				'name'    => $params['name'],
-				// Class names.
-				'class'   => 'wl-select2-element',
-				// The selected id.
-				'value'   => $current_post_id,
-				// The selected item (must be in the options for Select2 to display it).
-				'options' => $current_post ? array( $current_post->ID => $current_post->post_title ) : array(),
-				// Data attributes.
-				'data'    => array(
-					// The list of available options.
-					'wl-select2-data'               => wp_json_encode( $data ),
-					// The HTML template for each option.
-					'wl-select2-template-result'    => "<div class='wl-select2-result'><span class='wl-select2-thumbnail' style='background-image: url( <%- obj.thumbnail_url || ( 'Organization' === obj.type ? '$organization_thumbnail_url' : '$person_thumbnail_url' ) %> );'>&nbsp;</span><span class='wl-select2'><%- obj.text %></span><span class='wl-select2-type'><%- obj.type %></span></div>",
-					// The HTML template for the selected option.
-					'wl-select2-template-selection' => "<div class='wl-select2-selection'><span class='wl-select2-thumbnail' style='background-image: url( <%- obj.thumbnail_url || ( 'Organization' === obj.type ? '$organization_thumbnail_url' : '$person_thumbnail_url' ) %> );'>&nbsp;</span><span class='wl-select2'><%- obj.text %></span><span class='wl-select2-type'><%- obj.type %></span></div>",
-				),
-			)
-		);
-
-		// Finally return the element instance.
-		return $this;
-	}
+    /**
+     * The {@link Wordlift_Publisher_Service} instance.
+     *
+     * @since  3.14.0
+     * @access private
+     * @var \Wordlift_Publisher_Service $publisher_service The {@link Wordlift_Publisher_Service} instance.
+     */
+    private $publisher_service;
+
+    /**
+     * A {@link Wordlift_Admin_Select2_Element} instance.
+     *
+     * @since  3.14.0
+     * @access private
+     * @var \Wordlift_Admin_Select2_Element $select_element A {@link Wordlift_Admin_Select2_Element} instance.
+     */
+    private $select_element;
+
+    /**
+     * Create a {@link Wordlift_Admin_Person_Element} instance.
+     *
+     * @param \Wordlift_Publisher_Service     $publisher_service The {@link Wordlift_Publisher_Service} instance.
+     * @param \Wordlift_Admin_Select2_Element $select_element The {@link Wordlift_Admin_Select_Element} instance.
+     *
+     * @since 3.14.0
+     */
+    public function __construct( $publisher_service, $select_element ) {
+
+        $this->publisher_service = $publisher_service;
+
+        // Child elements.
+        $this->select_element = $select_element;
+
+    }
+
+    /**
+     * @inheritdoc
+     */
+    public function render( $args ) {
+
+        // Parse the arguments and merge with default values.
+        $params = wp_parse_args(
+            $args,
+            array(
+                'id'             => uniqid( 'wl-input-' ),
+                'name'           => uniqid( 'wl-input-' ),
+                'current_entity' => 0,
+            )
+        );
+
+        $current_entity_id = $params['current_entity'];
+        $data              = $this->publisher_service->query();
+
+        // Set a default to show when no entity is associated and a way to unassign.
+        array_unshift(
+            $data,
+            array(
+                'id'            => '0',
+                'text'          => '<em>' . __( '(none)', 'wordlift' ) . '</em>',
+                'type'          => '',
+                'thumbnail_url' => plugin_dir_url( __DIR__ ) . 'images/pixel.png',
+            )
+        );
+
+        // Finally do the render, passing along also the current selected entity
+        // id and the options data.
+        return $this->do_render( $params, $current_entity_id, $data );
+    }
+
+    /**
+     * Render the `select` using the provided parameters.
+     *
+     * @param array $params The array of parameters from the `render` function.
+     * @param int   $current_post_id The currently selected {@link WP_Post} `id`.
+     * @param array $data An array of Select2 options.
+     *
+     * @return \Wordlift_Admin_Author_Element $this Return this element.
+     * @since 3.14.0
+     */
+    protected function do_render( $params, $current_post_id, $data ) {
+
+        // Queue the script which will initialize the select and style it.
+        wp_enqueue_script( 'wl-author-element', plugin_dir_url( __DIR__ ) . 'js/1/author.js', array( 'wordlift-select2' ), WORDLIFT_VERSION, false );
+        wp_enqueue_style( 'wl-author-element', plugin_dir_url( __DIR__ ) . 'js/1/author.css', array(), WORDLIFT_VERSION );
+
+        // Prepare the URLs for entities which don't have logos.
+        $person_thumbnail_url       = plugin_dir_url( __DIR__ ) . '../images/person.png';
+        $organization_thumbnail_url = plugin_dir_url( __DIR__ ) . '../images/organization.png';
+
+        // Get the current post.
+        $current_post = $current_post_id ? get_post( $current_post_id ) : null;
+
+        // Finally render the Select.
+        $this->select_element->render(
+            array(
+                // Id.
+                'id'      => $params['id'],
+                // Name.
+                'name'    => $params['name'],
+                // Class names.
+                'class'   => 'wl-select2-element',
+                // The selected id.
+                'value'   => $current_post_id,
+                // The selected item (must be in the options for Select2 to display it).
+                'options' => $current_post ? array( $current_post->ID => $current_post->post_title ) : array(),
+                // Data attributes.
+                'data'    => array(
+                    // The list of available options.
+                    'wl-select2-data'               => wp_json_encode( $data ),
+                    // The HTML template for each option.
+                    'wl-select2-template-result'    => "<div class='wl-select2-result'><span class='wl-select2-thumbnail' style='background-image: url( <%- obj.thumbnail_url || ( 'Organization' === obj.type ? '$organization_thumbnail_url' : '$person_thumbnail_url' ) %> );'>&nbsp;</span><span class='wl-select2'><%- obj.text %></span><span class='wl-select2-type'><%- obj.type %></span></div>",
+                    // The HTML template for the selected option.
+                    'wl-select2-template-selection' => "<div class='wl-select2-selection'><span class='wl-select2-thumbnail' style='background-image: url( <%- obj.thumbnail_url || ( 'Organization' === obj.type ? '$organization_thumbnail_url' : '$person_thumbnail_url' ) %> );'>&nbsp;</span><span class='wl-select2'><%- obj.text %></span><span class='wl-select2-type'><%- obj.type %></span></div>",
+                ),
+            )
+        );
+
+        // Finally return the element instance.
+        return $this;
+    }
 
 }

Spacing +15 added lines, -15 removed lines

@@ -45,7 +45,7 @@ 
 	 *
 	 * @since 3.14.0
 	 */
-	public function __construct( $publisher_service, $select_element ) {
+	public function __construct($publisher_service, $select_element) {
 
 		$this->publisher_service = $publisher_service;
 
@@ -57,14 +57,14 @@ 
 	/**
 	 * @inheritdoc
 	 */
-	public function render( $args ) {
+	public function render($args) {
 
 		// Parse the arguments and merge with default values.
 		$params = wp_parse_args(
 			$args,
 			array(
-				'id'             => uniqid( 'wl-input-' ),
-				'name'           => uniqid( 'wl-input-' ),
+				'id'             => uniqid('wl-input-'),
+				'name'           => uniqid('wl-input-'),
 				'current_entity' => 0,
 			)
 		);
@@ -77,15 +77,15 @@ 
 			$data,
 			array(
 				'id'            => '0',
-				'text'          => '<em>' . __( '(none)', 'wordlift' ) . '</em>',
+				'text'          => '<em>'.__('(none)', 'wordlift').'</em>',
 				'type'          => '',
-				'thumbnail_url' => plugin_dir_url( __DIR__ ) . 'images/pixel.png',
+				'thumbnail_url' => plugin_dir_url(__DIR__).'images/pixel.png',
 			)
 		);
 
 		// Finally do the render, passing along also the current selected entity
 		// id and the options data.
-		return $this->do_render( $params, $current_entity_id, $data );
+		return $this->do_render($params, $current_entity_id, $data);
 	}
 
 	/**
@@ -98,18 +98,18 @@ 
 	 * @return \Wordlift_Admin_Author_Element $this Return this element.
 	 * @since 3.14.0
 	 */
-	protected function do_render( $params, $current_post_id, $data ) {
+	protected function do_render($params, $current_post_id, $data) {
 
 		// Queue the script which will initialize the select and style it.
-		wp_enqueue_script( 'wl-author-element', plugin_dir_url( __DIR__ ) . 'js/1/author.js', array( 'wordlift-select2' ), WORDLIFT_VERSION, false );
-		wp_enqueue_style( 'wl-author-element', plugin_dir_url( __DIR__ ) . 'js/1/author.css', array(), WORDLIFT_VERSION );
+		wp_enqueue_script('wl-author-element', plugin_dir_url(__DIR__).'js/1/author.js', array('wordlift-select2'), WORDLIFT_VERSION, false);
+		wp_enqueue_style('wl-author-element', plugin_dir_url(__DIR__).'js/1/author.css', array(), WORDLIFT_VERSION);
 
 		// Prepare the URLs for entities which don't have logos.
-		$person_thumbnail_url       = plugin_dir_url( __DIR__ ) . '../images/person.png';
-		$organization_thumbnail_url = plugin_dir_url( __DIR__ ) . '../images/organization.png';
+		$person_thumbnail_url       = plugin_dir_url(__DIR__).'../images/person.png';
+		$organization_thumbnail_url = plugin_dir_url(__DIR__).'../images/organization.png';
 
 		// Get the current post.
-		$current_post = $current_post_id ? get_post( $current_post_id ) : null;
+		$current_post = $current_post_id ? get_post($current_post_id) : null;
 
 		// Finally render the Select.
 		$this->select_element->render(
@@ -123,11 +123,11 @@ 
 				// The selected id.
 				'value'   => $current_post_id,
 				// The selected item (must be in the options for Select2 to display it).
-				'options' => $current_post ? array( $current_post->ID => $current_post->post_title ) : array(),
+				'options' => $current_post ? array($current_post->ID => $current_post->post_title) : array(),
 				// Data attributes.
 				'data'    => array(
 					// The list of available options.
-					'wl-select2-data'               => wp_json_encode( $data ),
+					'wl-select2-data'               => wp_json_encode($data),
 					// The HTML template for each option.
 					'wl-select2-template-result'    => "<div class='wl-select2-result'><span class='wl-select2-thumbnail' style='background-image: url( <%- obj.thumbnail_url || ( 'Organization' === obj.type ? '$organization_thumbnail_url' : '$person_thumbnail_url' ) %> );'>&nbsp;</span><span class='wl-select2'><%- obj.text %></span><span class='wl-select2-type'><%- obj.type %></span></div>",
 					// The HTML template for the selected option.

src/admin/elements/class-wordlift-admin-radio-input-element.php

Indentation +42 added lines, -42 removed lines

@@ -18,54 +18,54 @@ 
  */
 class Wordlift_Admin_Radio_Input_Element implements Wordlift_Admin_Element {
 
-	/**
-	 * Output the HTML for an input box type settings_page.
-	 *
-	 * @param array $args {
-	 *                           An array of arguments.
-	 *
-	 * @type string $name The name attribute of the input element. Mandatory.
-	 * @type string $id The id attribute of the input element. Optional,
-	 *                           randomly generated one is used if not supplied.
-	 * @type string $value The value of the input element. Optional, defaults
-	 *                           to empty string.
-	 * @type string $css_class The class attribute for the input element. If empty
-	 *                           string no class attribute will be added. Optional,
-	 *                           defaults to empty string.
-	 * @type string $description The description text to be displayed below the element.
-	 *                           Can include some HTML element. If empty string no
-	 *                           description will be displayed. Optional, defaults to
-	 *                           empty string.
-	 * }
-	 * @return $this|Wordlift_Admin_Element
-	 * @since      3.13.0
-	 */
-	public function render( $args ) {
-		/*
+    /**
+     * Output the HTML for an input box type settings_page.
+     *
+     * @param array $args {
+     *                           An array of arguments.
+     *
+     * @type string $name The name attribute of the input element. Mandatory.
+     * @type string $id The id attribute of the input element. Optional,
+     *                           randomly generated one is used if not supplied.
+     * @type string $value The value of the input element. Optional, defaults
+     *                           to empty string.
+     * @type string $css_class The class attribute for the input element. If empty
+     *                           string no class attribute will be added. Optional,
+     *                           defaults to empty string.
+     * @type string $description The description text to be displayed below the element.
+     *                           Can include some HTML element. If empty string no
+     *                           description will be displayed. Optional, defaults to
+     *                           empty string.
+     * }
+     * @return $this|Wordlift_Admin_Element
+     * @since      3.13.0
+     */
+    public function render( $args ) {
+        /*
 		 * Parse the arguments and merge with default values.
 		 * Name intentionally do not have a default as it has to be in SyncEvent
 		 * with form handling code
 		 */
-		$params = wp_parse_args(
-			$args,
-			array(
-				'id'          => uniqid( 'wl-input-' ),
-				'value'       => '',
-				'css_class'   => '',
-				'description' => '',
-			)
-		);
+        $params = wp_parse_args(
+            $args,
+            array(
+                'id'          => uniqid( 'wl-input-' ),
+                'value'       => '',
+                'css_class'   => '',
+                'description' => '',
+            )
+        );
 
-		// Set the readonly and class attributes and the description.
-		$value = $params['value'];
-		?>
+        // Set the readonly and class attributes and the description.
+        $value = $params['value'];
+        ?>
 
 		<input type="radio" id="<?php echo esc_attr( $params['id'] ); ?>"
 			   name="<?php echo esc_attr( $params['name'] ); ?>"
 			   value="yes" 
 			   <?php
-				if ( ! empty( $params['css_class'] ) ) {
-					?>
+                if ( ! empty( $params['css_class'] ) ) {
+                    ?>
 					 class="<?php echo esc_attr( $params['css_class'] ); ?>" <?php } ?>
 			<?php checked( $value, 'yes' ); ?>
 		/> Yes
@@ -73,8 +73,8 @@ 
 			   name="<?php echo esc_attr( $params['name'] ); ?>"
 			   value="no" 
 			   <?php
-				if ( ! empty( $params['css_class'] ) ) {
-					?>
+                if ( ! empty( $params['css_class'] ) ) {
+                    ?>
 					 class="<?php echo esc_attr( $params['css_class'] ); ?>" <?php } ?>
 			<?php checked( $value, 'no' ); ?>
 		/> No
@@ -82,7 +82,7 @@ 
 			<p><?php echo wp_kses( $params['description'], array( 'a' => array( 'href' => array() ) ) ); ?></p><?php } ?>
 		<?php
 
-		return $this;
-	}
+        return $this;
+    }
 
 }

Spacing +14 added lines, -14 removed lines

@@ -40,7 +40,7 @@ 
 	 * @return $this|Wordlift_Admin_Element
 	 * @since      3.13.0
 	 */
-	public function render( $args ) {
+	public function render($args) {
 		/*
 		 * Parse the arguments and merge with default values.
 		 * Name intentionally do not have a default as it has to be in SyncEvent
@@ -49,7 +49,7 @@ 
 		$params = wp_parse_args(
 			$args,
 			array(
-				'id'          => uniqid( 'wl-input-' ),
+				'id'          => uniqid('wl-input-'),
 				'value'       => '',
 				'css_class'   => '',
 				'description' => '',
@@ -60,26 +60,26 @@ 
 		$value = $params['value'];
 		?>
 
-		<input type="radio" id="<?php echo esc_attr( $params['id'] ); ?>"
-			   name="<?php echo esc_attr( $params['name'] ); ?>"
+		<input type="radio" id="<?php echo esc_attr($params['id']); ?>"
+			   name="<?php echo esc_attr($params['name']); ?>"
 			   value="yes" 
 			   <?php
-				if ( ! empty( $params['css_class'] ) ) {
+				if ( ! empty($params['css_class'])) {
 					?>
-					 class="<?php echo esc_attr( $params['css_class'] ); ?>" <?php } ?>
-			<?php checked( $value, 'yes' ); ?>
+					 class="<?php echo esc_attr($params['css_class']); ?>" <?php } ?>
+			<?php checked($value, 'yes'); ?>
 		/> Yes
-		<input type="radio" id="<?php echo esc_attr( $params['id'] ); ?>"
-			   name="<?php echo esc_attr( $params['name'] ); ?>"
+		<input type="radio" id="<?php echo esc_attr($params['id']); ?>"
+			   name="<?php echo esc_attr($params['name']); ?>"
 			   value="no" 
 			   <?php
-				if ( ! empty( $params['css_class'] ) ) {
+				if ( ! empty($params['css_class'])) {
 					?>
-					 class="<?php echo esc_attr( $params['css_class'] ); ?>" <?php } ?>
-			<?php checked( $value, 'no' ); ?>
+					 class="<?php echo esc_attr($params['css_class']); ?>" <?php } ?>
+			<?php checked($value, 'no'); ?>
 		/> No
-		<?php if ( ! empty( $params['description'] ) ) { ?>
-			<p><?php echo wp_kses( $params['description'], array( 'a' => array( 'href' => array() ) ) ); ?></p><?php } ?>
+		<?php if ( ! empty($params['description'])) { ?>
+			<p><?php echo wp_kses($params['description'], array('a' => array('href' => array()))); ?></p><?php } ?>
 		<?php
 
 		return $this;

src/admin/elements/class-wordlift-admin-country-select-element.php

Indentation +64 added lines, -64 removed lines

@@ -18,37 +18,37 @@ 
  */
 class Wordlift_Admin_Country_Select_Element extends Wordlift_Admin_Select_Element {
 
-	/**
-	 * Adds a filter that will add data `country-codes` attrbiute to the country select
-	 * to allow front-end validation.
-	 *
-	 * @since 3.18.0
-	 */
-	public function __construct() {
-		// Adds the country codes as data attribute to allow front-end validation.
-		add_filter( 'wl_admin_select_element_data_attributes', array( $this, 'add_country_codes_data' ), 10, 1 );
-	}
+    /**
+     * Adds a filter that will add data `country-codes` attrbiute to the country select
+     * to allow front-end validation.
+     *
+     * @since 3.18.0
+     */
+    public function __construct() {
+        // Adds the country codes as data attribute to allow front-end validation.
+        add_filter( 'wl_admin_select_element_data_attributes', array( $this, 'add_country_codes_data' ), 10, 1 );
+    }
 
-	/**
-	 * @inheritdoc
-	 */
-	public function render_options( $params ) {
-		// Print all the supported countries, preselecting the one configured
-		// in WP (or United Kingdom if not supported). We now use the `Wordlift_Countries`
-		// class which provides the list of countries supported by WordLift.
-		//
-		// https://github.com/insideout10/wordlift-plugin/issues/713
+    /**
+     * @inheritdoc
+     */
+    public function render_options( $params ) {
+        // Print all the supported countries, preselecting the one configured
+        // in WP (or United Kingdom if not supported). We now use the `Wordlift_Countries`
+        // class which provides the list of countries supported by WordLift.
+        //
+        // https://github.com/insideout10/wordlift-plugin/issues/713
 
-		$lang = ( isset( $params['lang'] ) ) ? $params['lang'] : false;
+        $lang = ( isset( $params['lang'] ) ) ? $params['lang'] : false;
 
-		// Get WordLift's supported countries.
-		$countries = Wordlift_Countries::get_countries( $lang );
+        // Get WordLift's supported countries.
+        $countries = Wordlift_Countries::get_countries( $lang );
 
-		// If we support WP's configured language, then use that, otherwise use English by default.
-		$language = isset( $countries[ $params['value'] ] ) ? $params['value'] : 'uk';
+        // If we support WP's configured language, then use that, otherwise use English by default.
+        $language = isset( $countries[ $params['value'] ] ) ? $params['value'] : 'uk';
 
-		foreach ( $countries as $code => $label ) :
-			?>
+        foreach ( $countries as $code => $label ) :
+            ?>
 			<option
 					value="<?php echo esc_attr( $code ); ?>"
 				<?php echo selected( $code, $language, false ); ?>
@@ -56,50 +56,50 @@ 
 				<?php echo esc_html( $label ); ?>
 			</option>
 			<?php
-		endforeach;
-	}
+        endforeach;
+    }
 
-	/**
-	 * Returns select options html.
-	 *
-	 * @since 3.18.0
-	 *
-	 * @return void Echoes select options or empty string if required params are not set.
-	 */
-	public function get_options_html() {
-		$html = '';
+    /**
+     * Returns select options html.
+     *
+     * @since 3.18.0
+     *
+     * @return void Echoes select options or empty string if required params are not set.
+     */
+    public function get_options_html() {
+        $html = '';
 
-		// Check whether the required params are set.
+        // Check whether the required params are set.
         // phpcs:ignore Standard.Category.SniffName.ErrorCode
-		if ( ! empty( $_POST['lang'] ) && ! empty( $_POST['value'] ) ) { //phpcs:ignore WordPress.Security.NonceVerification.Missing
-			ob_start();
-			// Get the new options.
-			// phpcs:ignore Standard.Category.SniffName.ErrorCode
-			$this->render_options( $_POST ); //phpcs:ignore WordPress.Security.NonceVerification.Missing
+        if ( ! empty( $_POST['lang'] ) && ! empty( $_POST['value'] ) ) { //phpcs:ignore WordPress.Security.NonceVerification.Missing
+            ob_start();
+            // Get the new options.
+            // phpcs:ignore Standard.Category.SniffName.ErrorCode
+            $this->render_options( $_POST ); //phpcs:ignore WordPress.Security.NonceVerification.Missing
 
-			$html = ob_get_clean();
-		}
+            $html = ob_get_clean();
+        }
 
-		// Return the html.
-		wp_send_json_success( $html );
-	}
+        // Return the html.
+        wp_send_json_success( $html );
+    }
 
-	/**
-	 * Modify the field data attributes by adding`country-code`
-	 * to existing attributes.
-	 *
-	 * @param array $attributes Current data attributes.
-	 *
-	 * @since 3.18.0
-	 *
-	 * @return array $attributes Modified attributes.
-	 */
-	public function add_country_codes_data( $attributes ) {
-		// Add the country codes.
-		$attributes['country-codes'] = wp_json_encode( Wordlift_Countries::get_codes() );
+    /**
+     * Modify the field data attributes by adding`country-code`
+     * to existing attributes.
+     *
+     * @param array $attributes Current data attributes.
+     *
+     * @since 3.18.0
+     *
+     * @return array $attributes Modified attributes.
+     */
+    public function add_country_codes_data( $attributes ) {
+        // Add the country codes.
+        $attributes['country-codes'] = wp_json_encode( Wordlift_Countries::get_codes() );
 
-		// Return the attributes.
-		return $attributes;
-	}
+        // Return the attributes.
+        return $attributes;
+    }
 
 }

Spacing +14 added lines, -14 removed lines

@@ -26,34 +26,34 @@ 
 	 */
 	public function __construct() {
 		// Adds the country codes as data attribute to allow front-end validation.
-		add_filter( 'wl_admin_select_element_data_attributes', array( $this, 'add_country_codes_data' ), 10, 1 );
+		add_filter('wl_admin_select_element_data_attributes', array($this, 'add_country_codes_data'), 10, 1);
 	}
 
 	/**
 	 * @inheritdoc
 	 */
-	public function render_options( $params ) {
+	public function render_options($params) {
 		// Print all the supported countries, preselecting the one configured
 		// in WP (or United Kingdom if not supported). We now use the `Wordlift_Countries`
 		// class which provides the list of countries supported by WordLift.
 		//
 		// https://github.com/insideout10/wordlift-plugin/issues/713
 
-		$lang = ( isset( $params['lang'] ) ) ? $params['lang'] : false;
+		$lang = (isset($params['lang'])) ? $params['lang'] : false;
 
 		// Get WordLift's supported countries.
-		$countries = Wordlift_Countries::get_countries( $lang );
+		$countries = Wordlift_Countries::get_countries($lang);
 
 		// If we support WP's configured language, then use that, otherwise use English by default.
-		$language = isset( $countries[ $params['value'] ] ) ? $params['value'] : 'uk';
+		$language = isset($countries[$params['value']]) ? $params['value'] : 'uk';
 
-		foreach ( $countries as $code => $label ) :
+		foreach ($countries as $code => $label) :
 			?>
 			<option
-					value="<?php echo esc_attr( $code ); ?>"
-				<?php echo selected( $code, $language, false ); ?>
+					value="<?php echo esc_attr($code); ?>"
+				<?php echo selected($code, $language, false); ?>
 			>
-				<?php echo esc_html( $label ); ?>
+				<?php echo esc_html($label); ?>
 			</option>
 			<?php
 		endforeach;
@@ -71,17 +71,17 @@ 
 
 		// Check whether the required params are set.
         // phpcs:ignore Standard.Category.SniffName.ErrorCode
-		if ( ! empty( $_POST['lang'] ) && ! empty( $_POST['value'] ) ) { //phpcs:ignore WordPress.Security.NonceVerification.Missing
+		if ( ! empty($_POST['lang']) && ! empty($_POST['value'])) { //phpcs:ignore WordPress.Security.NonceVerification.Missing
 			ob_start();
 			// Get the new options.
 			// phpcs:ignore Standard.Category.SniffName.ErrorCode
-			$this->render_options( $_POST ); //phpcs:ignore WordPress.Security.NonceVerification.Missing
+			$this->render_options($_POST); //phpcs:ignore WordPress.Security.NonceVerification.Missing
 
 			$html = ob_get_clean();
 		}
 
 		// Return the html.
-		wp_send_json_success( $html );
+		wp_send_json_success($html);
 	}
 
 	/**
@@ -94,9 +94,9 @@ 
 	 *
 	 * @return array $attributes Modified attributes.
 	 */
-	public function add_country_codes_data( $attributes ) {
+	public function add_country_codes_data($attributes) {
 		// Add the country codes.
-		$attributes['country-codes'] = wp_json_encode( Wordlift_Countries::get_codes() );
+		$attributes['country-codes'] = wp_json_encode(Wordlift_Countries::get_codes());
 
 		// Return the attributes.
 		return $attributes;

src/admin/elements/class-wordlift-admin-tabs-element.php

Indentation +41 added lines, -41 removed lines

@@ -18,48 +18,48 @@ 
  */
 class Wordlift_Admin_Tabs_Element implements Wordlift_Admin_Element {
 
-	/**
-	 * Render the element.
-	 *
-	 * @param array $args {
-	 *      Parameters controlling the html being output.
-	 *
-	 * @type integer $active The index of the active panel on first render
-	 *                              a zero based number of the tab actual placement
-	 *
-	 * @type array $tabs {
-	 *          The array of tabs to be rendered.
-	 *          The index of the elements is expected to be an ascending integers
-	 *          tabs with lower index values will be render first (on the left)
-	 *
-	 * @type string $label The label used for the tab.
-	 * @type callable $callback The callback to call to render the
-	 *                                      Tab "panel".
-	 * @type array $args The arguments array passed to the callback.
-	 *          }
-	 *      }
-	 *
-	 * @return \Wordlift_Admin_Element The element instance.
-	 * @since 3.11.0
-	 */
-	public function render( $args ) {
+    /**
+     * Render the element.
+     *
+     * @param array $args {
+     *      Parameters controlling the html being output.
+     *
+     * @type integer $active The index of the active panel on first render
+     *                              a zero based number of the tab actual placement
+     *
+     * @type array $tabs {
+     *          The array of tabs to be rendered.
+     *          The index of the elements is expected to be an ascending integers
+     *          tabs with lower index values will be render first (on the left)
+     *
+     * @type string $label The label used for the tab.
+     * @type callable $callback The callback to call to render the
+     *                                      Tab "panel".
+     * @type array $args The arguments array passed to the callback.
+     *          }
+     *      }
+     *
+     * @return \Wordlift_Admin_Element The element instance.
+     * @since 3.11.0
+     */
+    public function render( $args ) {
 
-		// Enqueue the jQuery UI Tabs script.
-		wp_enqueue_script( 'jquery-ui-tabs' );
+        // Enqueue the jQuery UI Tabs script.
+        wp_enqueue_script( 'jquery-ui-tabs' );
 
-		// Parse the arguments and merge with default values.
-		$params = wp_parse_args(
-			$args,
-			array(
-				'tabs'   => array(),
-				'active' => 0,
-			)
-		);
+        // Parse the arguments and merge with default values.
+        $params = wp_parse_args(
+            $args,
+            array(
+                'tabs'   => array(),
+                'active' => 0,
+            )
+        );
 
-		// Following is the HTML code:
-		// - the labels are printed, using the tab's `label`,
-		// - the panels are printed, using the tab's `callback`.
-		?>
+        // Following is the HTML code:
+        // - the labels are printed, using the tab's `label`,
+        // - the panels are printed, using the tab's `callback`.
+        ?>
 		<div
 				class="wl-tabs-element"
 				data-active="<?php echo esc_attr( $params['active'] ); ?>"
@@ -82,7 +82,7 @@ 
 
 		<?php
 
-		return $this;
-	}
+        return $this;
+    }
 
 }

Spacing +9 added lines, -9 removed lines

@@ -42,10 +42,10 @@ 
 	 * @return \Wordlift_Admin_Element The element instance.
 	 * @since 3.11.0
 	 */
-	public function render( $args ) {
+	public function render($args) {
 
 		// Enqueue the jQuery UI Tabs script.
-		wp_enqueue_script( 'jquery-ui-tabs' );
+		wp_enqueue_script('jquery-ui-tabs');
 
 		// Parse the arguments and merge with default values.
 		$params = wp_parse_args(
@@ -62,20 +62,20 @@ 
 		?>
 		<div
 				class="wl-tabs-element"
-				data-active="<?php echo esc_attr( $params['active'] ); ?>"
+				data-active="<?php echo esc_attr($params['active']); ?>"
 		>
 			<ul class="nav-tab-wrapper">
-				<?php foreach ( $params['tabs'] as $index => $tab ) : ?>
+				<?php foreach ($params['tabs'] as $index => $tab) : ?>
 					<li class="nav-tab">
-						<a href="#tabs-<?php echo esc_html( $index + 1 ); ?>">
-							<?php echo esc_html( $tab['label'] ); ?>
+						<a href="#tabs-<?php echo esc_html($index + 1); ?>">
+							<?php echo esc_html($tab['label']); ?>
 						</a>
 					</li>
 				<?php endforeach; ?>
 			</ul>
-			<?php foreach ( $params['tabs'] as $index => $tab ) : ?>
-				<div id="tabs-<?php echo esc_html( $index + 1 ); ?>">
-					<?php call_user_func( $tab['callback'], $tab['args'] ); ?>
+			<?php foreach ($params['tabs'] as $index => $tab) : ?>
+				<div id="tabs-<?php echo esc_html($index + 1); ?>">
+					<?php call_user_func($tab['callback'], $tab['args']); ?>
 				</div>
 			<?php endforeach; ?>
 		</div>

src/admin/elements/class-wordlift-admin-select2-element.php

Indentation +27 added lines, -27 removed lines

@@ -18,13 +18,13 @@ 
  */
 class Wordlift_Admin_Select2_Element extends Wordlift_Admin_Select_Element {
 
-	/**
-	 * @inheritdoc
-	 */
-	public function render_options( $params ) {
-		// Loop through all params and add the options.
-		foreach ( $params['options'] as $value => $label ) :
-			?>
+    /**
+     * @inheritdoc
+     */
+    public function render_options( $params ) {
+        // Loop through all params and add the options.
+        foreach ( $params['options'] as $value => $label ) :
+            ?>
 			<option
 					value="<?php echo esc_attr( $value ); ?>"
 				<?php selected( $params['value'], $value ); ?>
@@ -32,26 +32,26 @@ 
 				<?php echo esc_html( $label ); ?>
 			</option>
 			<?php
-		endforeach;
-	}
+        endforeach;
+    }
 
-	/**
-	 * @inheritdoc
-	 */
-	protected function enqueue_resources() {
-		// Enqueue select2 library js and css.
-		// Underscore is needed for Select2's `templateResult` and `templateSelection` templates.
-		wp_enqueue_script(
-			'wordlift-select2',
-			plugin_dir_url( __DIR__ ) . 'js/select2/js/select2' . ( ! defined( 'SCRIPT_DEBUG' ) || ! SCRIPT_DEBUG ? '.min' : '' ) . '.js',
-			array(
-				'jquery',
-				'underscore',
-			),
-			'4.0.3',
-			false
-		);
-		wp_enqueue_style( 'wordlift-select2', plugin_dir_url( __DIR__ ) . 'js/select2/css/select2' . ( ! defined( 'SCRIPT_DEBUG' ) || ! SCRIPT_DEBUG ? '.min' : '' ) . '.css', array(), '4.0.3' );
-	}
+    /**
+     * @inheritdoc
+     */
+    protected function enqueue_resources() {
+        // Enqueue select2 library js and css.
+        // Underscore is needed for Select2's `templateResult` and `templateSelection` templates.
+        wp_enqueue_script(
+            'wordlift-select2',
+            plugin_dir_url( __DIR__ ) . 'js/select2/js/select2' . ( ! defined( 'SCRIPT_DEBUG' ) || ! SCRIPT_DEBUG ? '.min' : '' ) . '.js',
+            array(
+                'jquery',
+                'underscore',
+            ),
+            '4.0.3',
+            false
+        );
+        wp_enqueue_style( 'wordlift-select2', plugin_dir_url( __DIR__ ) . 'js/select2/css/select2' . ( ! defined( 'SCRIPT_DEBUG' ) || ! SCRIPT_DEBUG ? '.min' : '' ) . '.css', array(), '4.0.3' );
+    }
 
 }

Spacing +7 added lines, -7 removed lines

@@ -21,15 +21,15 @@ 
 	/**
 	 * @inheritdoc
 	 */
-	public function render_options( $params ) {
+	public function render_options($params) {
 		// Loop through all params and add the options.
-		foreach ( $params['options'] as $value => $label ) :
+		foreach ($params['options'] as $value => $label) :
 			?>
 			<option
-					value="<?php echo esc_attr( $value ); ?>"
-				<?php selected( $params['value'], $value ); ?>
+					value="<?php echo esc_attr($value); ?>"
+				<?php selected($params['value'], $value); ?>
 			>
-				<?php echo esc_html( $label ); ?>
+				<?php echo esc_html($label); ?>
 			</option>
 			<?php
 		endforeach;
@@ -43,7 +43,7 @@ 
 		// Underscore is needed for Select2's `templateResult` and `templateSelection` templates.
 		wp_enqueue_script(
 			'wordlift-select2',
-			plugin_dir_url( __DIR__ ) . 'js/select2/js/select2' . ( ! defined( 'SCRIPT_DEBUG' ) || ! SCRIPT_DEBUG ? '.min' : '' ) . '.js',
+			plugin_dir_url(__DIR__).'js/select2/js/select2'.( ! defined('SCRIPT_DEBUG') || ! SCRIPT_DEBUG ? '.min' : '').'.js',
 			array(
 				'jquery',
 				'underscore',
@@ -51,7 +51,7 @@ 
 			'4.0.3',
 			false
 		);
-		wp_enqueue_style( 'wordlift-select2', plugin_dir_url( __DIR__ ) . 'js/select2/css/select2' . ( ! defined( 'SCRIPT_DEBUG' ) || ! SCRIPT_DEBUG ? '.min' : '' ) . '.css', array(), '4.0.3' );
+		wp_enqueue_style('wordlift-select2', plugin_dir_url(__DIR__).'js/select2/css/select2'.( ! defined('SCRIPT_DEBUG') || ! SCRIPT_DEBUG ? '.min' : '').'.css', array(), '4.0.3');
 	}
 
 }

src/admin/elements/class-wordlift-admin-language-select-element.php

Indentation +13 added lines, -13 removed lines

@@ -18,11 +18,11 @@ 
  */
 class Wordlift_Admin_Language_Select_Element extends Wordlift_Admin_Select_Element {
 
-	/**
-	 * @inheritdoc
-	 */
-	public function render_options( $params ) {
-		/*
+    /**
+     * @inheritdoc
+     */
+    public function render_options( $params ) {
+        /*
 		 * Print all the supported language, preselecting the one configured
 		 * in WP (or English if not supported). We now use the `Wordlift_Languages`
 		 * class which provides the list of languages supported by WordLift.
@@ -30,14 +30,14 @@ 
 		 * See https://github.com/insideout10/wordlift-plugin/issues/349
 		 */
 
-		// Get WordLift's supported languages.
-		$languages = Wordlift_Languages::get_languages();
+        // Get WordLift's supported languages.
+        $languages = Wordlift_Languages::get_languages();
 
-		// If we support WP's configured language, then use that, otherwise use English by default.
-		$language = isset( $languages[ $params['value'] ] ) ? $params['value'] : 'en';
+        // If we support WP's configured language, then use that, otherwise use English by default.
+        $language = isset( $languages[ $params['value'] ] ) ? $params['value'] : 'en';
 
-		foreach ( $languages as $code => $label ) :
-			?>
+        foreach ( $languages as $code => $label ) :
+            ?>
 			<option
 					value="<?php echo esc_attr( $code ); ?>"
 				<?php echo selected( $code, $language, false ); ?>
@@ -45,7 +45,7 @@ 
 				<?php echo esc_html( $label ); ?>
 			</option>
 			<?php
-		endforeach;
-	}
+        endforeach;
+    }
 
 }

Spacing +6 added lines, -6 removed lines

@@ -21,7 +21,7 @@ 
 	/**
 	 * @inheritdoc
 	 */
-	public function render_options( $params ) {
+	public function render_options($params) {
 		/*
 		 * Print all the supported language, preselecting the one configured
 		 * in WP (or English if not supported). We now use the `Wordlift_Languages`
@@ -34,15 +34,15 @@ 
 		$languages = Wordlift_Languages::get_languages();
 
 		// If we support WP's configured language, then use that, otherwise use English by default.
-		$language = isset( $languages[ $params['value'] ] ) ? $params['value'] : 'en';
+		$language = isset($languages[$params['value']]) ? $params['value'] : 'en';
 
-		foreach ( $languages as $code => $label ) :
+		foreach ($languages as $code => $label) :
 			?>
 			<option
-					value="<?php echo esc_attr( $code ); ?>"
-				<?php echo selected( $code, $language, false ); ?>
+					value="<?php echo esc_attr($code); ?>"
+				<?php echo selected($code, $language, false); ?>
 			>
-				<?php echo esc_html( $label ); ?>
+				<?php echo esc_html($label); ?>
 			</option>
 			<?php
 		endforeach;

src/admin/elements/class-wordlift-admin-publisher-element.php

Indentation +114 added lines, -114 removed lines

@@ -19,120 +19,120 @@ 
  */
 class Wordlift_Admin_Publisher_Element extends Wordlift_Admin_Author_Element {
 
-	/**
-	 * The {@link Wordlift_Publisher_Service} instance.
-	 *
-	 * @since  3.11.0
-	 * @access private
-	 * @var \Wordlift_Publisher_Service $publisher_service The {@link Wordlift_Publisher_Service} instance.
-	 */
-	private $publisher_service;
-
-	/**
-	 * @var Wordlift_Admin_Tabs_Element
-	 */
-	private $tabs_element;
-
-	/**
-	 * Create a {@link Wordlift_Admin_Publisher_Element} instance.
-	 *
-	 * @param \Wordlift_Publisher_Service     $publisher_service The {@link Wordlift_Publisher_Service} instance.
-	 * @param \Wordlift_Admin_Tabs_Element    $tabs_element The {@link Wordlift_Admin_Tabs_Element} instance.
-	 * @param \Wordlift_Admin_Select2_Element $select_element The {@link Wordlift_Admin_Select_Element} instance.
-	 *
-	 * @since 3.11.0
-	 */
-	public function __construct( $publisher_service, $tabs_element, $select_element ) {
-		parent::__construct( $publisher_service, $select_element );
-
-		$this->publisher_service = $publisher_service;
-
-		// Child elements.
-		$this->tabs_element = $tabs_element;
-
-	}
-
-	/**
-	 * @inheritdoc
-	 */
-	public function render( $args ) {
-
-		// Parse the arguments and merge with default values.
-		$params = wp_parse_args(
-			$args,
-			array(
-				'id'   => uniqid( 'wl-input-' ),
-				'name' => uniqid( 'wl-input-' ),
-			)
-		);
-
-		// Get the number of potential candidates as publishers.
-		$count = $this->publisher_service->count();
-
-		$this->tabs_element->render(
-			array(
-				'tabs'   => array(
-					array(
-						'label'    => __( 'Select an Existing Publisher', 'wordlift' ),
-						'callback' => array( $this, 'select' ),
-						'args'     => $params,
-					),
-					array(
-						'label'    => __( 'Create a New Publisher', 'wordlift' ),
-						'callback' => array( $this, 'create' ),
-						'args'     => $params,
-					),
-				),
-				// Set the default tab according to the number of potential publishers
-				// configured in WP: 0 = select, 1 = create.
-				'active' => 0 === $count ? 1 : 0,
-			)
-		);
-
-		// Finally return the element instance.
-		return $this;
-	}
-
-	/**
-	 * Render the publisher's select.
-	 *
-	 * @param array $params An array of parameters.
-	 *
-	 * @since 3.11.0
-	 */
-	public function select( $params ) {
-
-		// Get the configured publisher id. In case a publisher id is already configured
-		// this must be pre-loaded in the options.
-		$publisher_id = Wordlift_Configuration_Service::get_instance()->get_publisher_id();
-
-		// Get the publisher data.
-		$data = $this->publisher_service->query();
-		array_unshift(
-			$data,
-			array(
-				'id'            => '',
-				'text'          => _x( '(none)', 'Publisher Select in Settings Screen.', 'wordlift' ),
-				'type'          => '',
-				'thumbnail_url' => false,
-			)
-		);
-
-		// Call the select internal render.
-		$this->do_render( $params, $publisher_id, $data );
-
-	}
-
-	/**
-	 * Render the 'create publisher' form.
-	 *
-	 * @param array $params An array of parameters.
-	 *
-	 * @since 3.11.0
-	 */
+    /**
+     * The {@link Wordlift_Publisher_Service} instance.
+     *
+     * @since  3.11.0
+     * @access private
+     * @var \Wordlift_Publisher_Service $publisher_service The {@link Wordlift_Publisher_Service} instance.
+     */
+    private $publisher_service;
+
+    /**
+     * @var Wordlift_Admin_Tabs_Element
+     */
+    private $tabs_element;
+
+    /**
+     * Create a {@link Wordlift_Admin_Publisher_Element} instance.
+     *
+     * @param \Wordlift_Publisher_Service     $publisher_service The {@link Wordlift_Publisher_Service} instance.
+     * @param \Wordlift_Admin_Tabs_Element    $tabs_element The {@link Wordlift_Admin_Tabs_Element} instance.
+     * @param \Wordlift_Admin_Select2_Element $select_element The {@link Wordlift_Admin_Select_Element} instance.
+     *
+     * @since 3.11.0
+     */
+    public function __construct( $publisher_service, $tabs_element, $select_element ) {
+        parent::__construct( $publisher_service, $select_element );
+
+        $this->publisher_service = $publisher_service;
+
+        // Child elements.
+        $this->tabs_element = $tabs_element;
+
+    }
+
+    /**
+     * @inheritdoc
+     */
+    public function render( $args ) {
+
+        // Parse the arguments and merge with default values.
+        $params = wp_parse_args(
+            $args,
+            array(
+                'id'   => uniqid( 'wl-input-' ),
+                'name' => uniqid( 'wl-input-' ),
+            )
+        );
+
+        // Get the number of potential candidates as publishers.
+        $count = $this->publisher_service->count();
+
+        $this->tabs_element->render(
+            array(
+                'tabs'   => array(
+                    array(
+                        'label'    => __( 'Select an Existing Publisher', 'wordlift' ),
+                        'callback' => array( $this, 'select' ),
+                        'args'     => $params,
+                    ),
+                    array(
+                        'label'    => __( 'Create a New Publisher', 'wordlift' ),
+                        'callback' => array( $this, 'create' ),
+                        'args'     => $params,
+                    ),
+                ),
+                // Set the default tab according to the number of potential publishers
+                // configured in WP: 0 = select, 1 = create.
+                'active' => 0 === $count ? 1 : 0,
+            )
+        );
+
+        // Finally return the element instance.
+        return $this;
+    }
+
+    /**
+     * Render the publisher's select.
+     *
+     * @param array $params An array of parameters.
+     *
+     * @since 3.11.0
+     */
+    public function select( $params ) {
+
+        // Get the configured publisher id. In case a publisher id is already configured
+        // this must be pre-loaded in the options.
+        $publisher_id = Wordlift_Configuration_Service::get_instance()->get_publisher_id();
+
+        // Get the publisher data.
+        $data = $this->publisher_service->query();
+        array_unshift(
+            $data,
+            array(
+                'id'            => '',
+                'text'          => _x( '(none)', 'Publisher Select in Settings Screen.', 'wordlift' ),
+                'type'          => '',
+                'thumbnail_url' => false,
+            )
+        );
+
+        // Call the select internal render.
+        $this->do_render( $params, $publisher_id, $data );
+
+    }
+
+    /**
+     * Render the 'create publisher' form.
+     *
+     * @param array $params An array of parameters.
+     *
+     * @since 3.11.0
+     */
     // phpcs:ignore VariableAnalysis.CodeAnalysis.VariableAnalysis.UnusedVariable
-	public function create( $params ) {
-		?>
+    public function create( $params ) {
+        ?>
 		<p>
 			<strong><?php esc_html_e( 'Are you publishing as an individual or as a company?', 'wordlift' ); ?></strong>
 		</p>
@@ -197,6 +197,6 @@ 
 			</p>
 		</div>
 		<?php
-	}
+    }
 
 }

Spacing +19 added lines, -19 removed lines

@@ -42,8 +42,8 @@ 
 	 *
 	 * @since 3.11.0
 	 */
-	public function __construct( $publisher_service, $tabs_element, $select_element ) {
-		parent::__construct( $publisher_service, $select_element );
+	public function __construct($publisher_service, $tabs_element, $select_element) {
+		parent::__construct($publisher_service, $select_element);
 
 		$this->publisher_service = $publisher_service;
 
@@ -55,14 +55,14 @@ 
 	/**
 	 * @inheritdoc
 	 */
-	public function render( $args ) {
+	public function render($args) {
 
 		// Parse the arguments and merge with default values.
 		$params = wp_parse_args(
 			$args,
 			array(
-				'id'   => uniqid( 'wl-input-' ),
-				'name' => uniqid( 'wl-input-' ),
+				'id'   => uniqid('wl-input-'),
+				'name' => uniqid('wl-input-'),
 			)
 		);
 
@@ -73,13 +73,13 @@ 
 			array(
 				'tabs'   => array(
 					array(
-						'label'    => __( 'Select an Existing Publisher', 'wordlift' ),
-						'callback' => array( $this, 'select' ),
+						'label'    => __('Select an Existing Publisher', 'wordlift'),
+						'callback' => array($this, 'select'),
 						'args'     => $params,
 					),
 					array(
-						'label'    => __( 'Create a New Publisher', 'wordlift' ),
-						'callback' => array( $this, 'create' ),
+						'label'    => __('Create a New Publisher', 'wordlift'),
+						'callback' => array($this, 'create'),
 						'args'     => $params,
 					),
 				),
@@ -100,7 +100,7 @@ 
 	 *
 	 * @since 3.11.0
 	 */
-	public function select( $params ) {
+	public function select($params) {
 
 		// Get the configured publisher id. In case a publisher id is already configured
 		// this must be pre-loaded in the options.
@@ -112,14 +112,14 @@ 
 			$data,
 			array(
 				'id'            => '',
-				'text'          => _x( '(none)', 'Publisher Select in Settings Screen.', 'wordlift' ),
+				'text'          => _x('(none)', 'Publisher Select in Settings Screen.', 'wordlift'),
 				'type'          => '',
 				'thumbnail_url' => false,
 			)
 		);
 
 		// Call the select internal render.
-		$this->do_render( $params, $publisher_id, $data );
+		$this->do_render($params, $publisher_id, $data);
 
 	}
 
@@ -131,10 +131,10 @@ 
 	 * @since 3.11.0
 	 */
     // phpcs:ignore VariableAnalysis.CodeAnalysis.VariableAnalysis.UnusedVariable
-	public function create( $params ) {
+	public function create($params) {
 		?>
 		<p>
-			<strong><?php esc_html_e( 'Are you publishing as an individual or as a company?', 'wordlift' ); ?></strong>
+			<strong><?php esc_html_e('Are you publishing as an individual or as a company?', 'wordlift'); ?></strong>
 		</p>
 
 		<p id="wl-publisher-type">
@@ -148,7 +148,7 @@ 
 				>
 
 				<label for="wl-publisher-person">
-					<?php esc_html_e( 'Person', 'wordlift' ); ?>
+					<?php esc_html_e('Person', 'wordlift'); ?>
 				</label>
 			</span>
 
@@ -161,7 +161,7 @@ 
 				>
 
 				<label for="wl-publisher-company">
-					<?php esc_html_e( 'Company', 'wordlift' ); ?>
+					<?php esc_html_e('Company', 'wordlift'); ?>
 				</label>
 			</span>
 		</p>
@@ -170,7 +170,7 @@ 
 			<input
 					type="text"
 					name="wl_publisher[name]"
-					placeholder="<?php echo esc_attr__( "What's your name?", 'wordlift' ); ?>"
+					placeholder="<?php echo esc_attr__("What's your name?", 'wordlift'); ?>"
 			>
 		</p>
 
@@ -182,7 +182,7 @@ 
 			/>
 
 			<p>
-				<b><?php esc_html_e( "Choose the publisher's Logo", 'wordlift' ); ?></b>
+				<b><?php esc_html_e("Choose the publisher's Logo", 'wordlift'); ?></b>
 			</p>
 
 			<p>
@@ -192,7 +192,7 @@ 
 						type="button"
 						class="button"
 						id="wl-publisher-media-uploader">
-					<?php esc_html_e( 'Select an existing image or upload a new one', 'wordlift' ); ?>
+					<?php esc_html_e('Select an existing image or upload a new one', 'wordlift'); ?>
 				</button>
 			</p>
 		</div>

src/admin/class-wordlift-admin-settings-analytics-page-action-link.php

Indentation +44 added lines, -44 removed lines

@@ -19,55 +19,55 @@
  */
 class Wordlift_Admin_Settings_Analytics_Page_Action_Link {
 
-	/**
-	 * The {@link Wordlift_Admin_Settings_Analytics_Page} instance.
-	 *
-	 * @since  3.11.0
-	 * @access private
-	 * @var \Wordlift_Admin_Settings_Analytics_Page $settings_page The {@link Wordlift_Admin_Settings_Analytics_Page} instance.
-	 */
-	private $settings_page;
+    /**
+     * The {@link Wordlift_Admin_Settings_Analytics_Page} instance.
+     *
+     * @since  3.11.0
+     * @access private
+     * @var \Wordlift_Admin_Settings_Analytics_Page $settings_page The {@link Wordlift_Admin_Settings_Analytics_Page} instance.
+     */
+    private $settings_page;
 
-	/**
-	 * Create a {@link Wordlift_Admin_Settings_Analytics_Page_Action_Link} instance.
-	 *
-	 * @since  3.11.0
-	 *
-	 * @param \Wordlift_Admin_Settings_Analytics_Page $settings_page The {@link Wordlift_Admin_Settings_Analytics_Page} instance.
-	 */
-	public function __construct( $settings_page ) {
+    /**
+     * Create a {@link Wordlift_Admin_Settings_Analytics_Page_Action_Link} instance.
+     *
+     * @since  3.11.0
+     *
+     * @param \Wordlift_Admin_Settings_Analytics_Page $settings_page The {@link Wordlift_Admin_Settings_Analytics_Page} instance.
+     */
+    public function __construct( $settings_page ) {
 
-		$this->settings_page = $settings_page;
+        $this->settings_page = $settings_page;
 
-	}
+    }
 
-	/**
-	 * Create a link to the WordLift settings page.
-	 *
-	 * @since 3.11.0
-	 *
-	 * @param array $links An array of links.
-	 *
-	 * @return array An array of links including those added by the plugin.
-	 */
-	public function action_links( $links ) {
+    /**
+     * Create a link to the WordLift settings page.
+     *
+     * @since 3.11.0
+     *
+     * @param array $links An array of links.
+     *
+     * @return array An array of links including those added by the plugin.
+     */
+    public function action_links( $links ) {
 
-		// Get the menu slug from the page, then prepare the path, hence the url.
-		$menu_slug = $this->settings_page->get_menu_slug();
-		$path      = "admin.php?page=$menu_slug";
-		$url       = get_admin_url( null, $path );
+        // Get the menu slug from the page, then prepare the path, hence the url.
+        $menu_slug = $this->settings_page->get_menu_slug();
+        $path      = "admin.php?page=$menu_slug";
+        $url       = get_admin_url( null, $path );
 
-		// Add our own link to the list of links.
-		return array_merge(
-			$links,
-			array(
-				sprintf(
-					'<a href="%1$s">%2$s</a>',
-					esc_url( $url ),
-					esc_html__( 'Analytics Settings', 'wordlift' )
-				),
-			)
-		);
-	}
+        // Add our own link to the list of links.
+        return array_merge(
+            $links,
+            array(
+                sprintf(
+                    '<a href="%1$s">%2$s</a>',
+                    esc_url( $url ),
+                    esc_html__( 'Analytics Settings', 'wordlift' )
+                ),
+            )
+        );
+    }
 
 }

Spacing +5 added lines, -5 removed lines

@@ -35,7 +35,7 @@ 
 	 *
 	 * @param \Wordlift_Admin_Settings_Analytics_Page $settings_page The {@link Wordlift_Admin_Settings_Analytics_Page} instance.
 	 */
-	public function __construct( $settings_page ) {
+	public function __construct($settings_page) {
 
 		$this->settings_page = $settings_page;
 
@@ -50,12 +50,12 @@ 
 	 *
 	 * @return array An array of links including those added by the plugin.
 	 */
-	public function action_links( $links ) {
+	public function action_links($links) {
 
 		// Get the menu slug from the page, then prepare the path, hence the url.
 		$menu_slug = $this->settings_page->get_menu_slug();
 		$path      = "admin.php?page=$menu_slug";
-		$url       = get_admin_url( null, $path );
+		$url       = get_admin_url(null, $path);
 
 		// Add our own link to the list of links.
 		return array_merge(
@@ -63,8 +63,8 @@ 
 			array(
 				sprintf(
 					'<a href="%1$s">%2$s</a>',
-					esc_url( $url ),
-					esc_html__( 'Analytics Settings', 'wordlift' )
+					esc_url($url),
+					esc_html__('Analytics Settings', 'wordlift')
 				),
 			)
 		);

src/admin/wordlift-admin-save-post.php

Indentation +42 added lines, -42 removed lines

@@ -25,24 +25,24 @@ 
  */
 function wl_transition_post_status( $new_status, $old_status, $post ) {
 
-	// transition from *published* to any other status: delete the post.
-	if ( 'publish' === $old_status && 'publish' !== $new_status ) {
-		// Delete the post from the triple store.
-		rl_delete_post( $post );
-
-		// Remove all relation instances for the current post from `wl_relation_instances`.
-		wl_core_delete_relation_instances( $post->ID );
-	}
-
-	// when a post is published, then all the referenced entities must be published.
-	if ( 'publish' !== $old_status && 'publish' === $new_status
-	     // phpcs:ignore WordPress.NamingConventions.ValidHookName.UseUnderscores
-		 && apply_filters( 'wl_feature__enable__entity-auto-publish', true ) ) {
-
-		foreach ( wl_core_get_related_entity_ids( $post->ID ) as $entity_id ) {
-			wl_update_post_status( $entity_id, 'publish' );
-		}
-	}
+    // transition from *published* to any other status: delete the post.
+    if ( 'publish' === $old_status && 'publish' !== $new_status ) {
+        // Delete the post from the triple store.
+        rl_delete_post( $post );
+
+        // Remove all relation instances for the current post from `wl_relation_instances`.
+        wl_core_delete_relation_instances( $post->ID );
+    }
+
+    // when a post is published, then all the referenced entities must be published.
+    if ( 'publish' !== $old_status && 'publish' === $new_status
+            // phpcs:ignore WordPress.NamingConventions.ValidHookName.UseUnderscores
+         && apply_filters( 'wl_feature__enable__entity-auto-publish', true ) ) {
+
+        foreach ( wl_core_get_related_entity_ids( $post->ID ) as $entity_id ) {
+            wl_update_post_status( $entity_id, 'publish' );
+        }
+    }
 }
 
 // hook save events.
@@ -55,10 +55,10 @@ 
  */
 function rl_delete_post( $post ) {
 
-	$post_id = ( is_numeric( $post ) ? $post : $post->ID );
+    $post_id = ( is_numeric( $post ) ? $post : $post->ID );
 
-	// Remove the post.
-	do_action( 'wl_legacy_linked_data__remove', $post_id );
+    // Remove the post.
+    do_action( 'wl_legacy_linked_data__remove', $post_id );
 
 }
 
@@ -70,32 +70,32 @@ 
  */
 function wl_update_post_status( $post_id, $status ) {
 
-	global $wpdb;
+    global $wpdb;
 
-	$post = get_post( $post_id );
-	if ( ! $post ) {
-		return;
-	}
+    $post = get_post( $post_id );
+    if ( ! $post ) {
+        return;
+    }
 
-	if ( $status === $post->post_status ) {
-		return;
-	}
+    if ( $status === $post->post_status ) {
+        return;
+    }
 
-	$wpdb->update( $wpdb->posts, array( 'post_status' => $status ), array( 'ID' => $post->ID ) );
+    $wpdb->update( $wpdb->posts, array( 'post_status' => $status ), array( 'ID' => $post->ID ) );
 
-	clean_post_cache( $post->ID );
+    clean_post_cache( $post->ID );
 
-	$old_status        = $post->post_status;
-	$post->post_status = $status;
+    $old_status        = $post->post_status;
+    $post->post_status = $status;
 
-	wp_transition_post_status( $status, $old_status, $post );
+    wp_transition_post_status( $status, $old_status, $post );
 
-	/** This action is documented in wp-includes/post.php */
-	do_action( 'edit_post', $post->ID, $post );
-	/** This action is documented in wp-includes/post.php */
-	do_action( "save_post_{$post->post_type}", $post->ID, $post, true );
-	/** This action is documented in wp-includes/post.php */
-	do_action( 'wl_linked_data_save_post', $post->ID );
-	/** This action is documented in wp-includes/post.php */
-	do_action( 'wp_insert_post', $post->ID, $post, true );
+    /** This action is documented in wp-includes/post.php */
+    do_action( 'edit_post', $post->ID, $post );
+    /** This action is documented in wp-includes/post.php */
+    do_action( "save_post_{$post->post_type}", $post->ID, $post, true );
+    /** This action is documented in wp-includes/post.php */
+    do_action( 'wl_linked_data_save_post', $post->ID );
+    /** This action is documented in wp-includes/post.php */
+    do_action( 'wp_insert_post', $post->ID, $post, true );
 }

Spacing +23 added lines, -23 removed lines

@@ -23,42 +23,42 @@ 
  * @param string $old_status The old post status
  * @param array  $post An array with the post data
  */
-function wl_transition_post_status( $new_status, $old_status, $post ) {
+function wl_transition_post_status($new_status, $old_status, $post) {
 
 	// transition from *published* to any other status: delete the post.
-	if ( 'publish' === $old_status && 'publish' !== $new_status ) {
+	if ('publish' === $old_status && 'publish' !== $new_status) {
 		// Delete the post from the triple store.
-		rl_delete_post( $post );
+		rl_delete_post($post);
 
 		// Remove all relation instances for the current post from `wl_relation_instances`.
-		wl_core_delete_relation_instances( $post->ID );
+		wl_core_delete_relation_instances($post->ID);
 	}
 
 	// when a post is published, then all the referenced entities must be published.
-	if ( 'publish' !== $old_status && 'publish' === $new_status
+	if ('publish' !== $old_status && 'publish' === $new_status
 	     // phpcs:ignore WordPress.NamingConventions.ValidHookName.UseUnderscores
-		 && apply_filters( 'wl_feature__enable__entity-auto-publish', true ) ) {
+		 && apply_filters('wl_feature__enable__entity-auto-publish', true)) {
 
-		foreach ( wl_core_get_related_entity_ids( $post->ID ) as $entity_id ) {
-			wl_update_post_status( $entity_id, 'publish' );
+		foreach (wl_core_get_related_entity_ids($post->ID) as $entity_id) {
+			wl_update_post_status($entity_id, 'publish');
 		}
 	}
 }
 
 // hook save events.
-add_action( 'transition_post_status', 'wl_transition_post_status', 10, 3 );
+add_action('transition_post_status', 'wl_transition_post_status', 10, 3);
 
 /**
  * Delete the specified post from the triple store.
  *
  * @param array|int $post An array of post data
  */
-function rl_delete_post( $post ) {
+function rl_delete_post($post) {
 
-	$post_id = ( is_numeric( $post ) ? $post : $post->ID );
+	$post_id = (is_numeric($post) ? $post : $post->ID);
 
 	// Remove the post.
-	do_action( 'wl_legacy_linked_data__remove', $post_id );
+	do_action('wl_legacy_linked_data__remove', $post_id);
 
 }
 
@@ -68,34 +68,34 @@ 
  * @param int    $post_id The post ID
  * @param string $status The new status
  */
-function wl_update_post_status( $post_id, $status ) {
+function wl_update_post_status($post_id, $status) {
 
 	global $wpdb;
 
-	$post = get_post( $post_id );
-	if ( ! $post ) {
+	$post = get_post($post_id);
+	if ( ! $post) {
 		return;
 	}
 
-	if ( $status === $post->post_status ) {
+	if ($status === $post->post_status) {
 		return;
 	}
 
-	$wpdb->update( $wpdb->posts, array( 'post_status' => $status ), array( 'ID' => $post->ID ) );
+	$wpdb->update($wpdb->posts, array('post_status' => $status), array('ID' => $post->ID));
 
-	clean_post_cache( $post->ID );
+	clean_post_cache($post->ID);
 
 	$old_status        = $post->post_status;
 	$post->post_status = $status;
 
-	wp_transition_post_status( $status, $old_status, $post );
+	wp_transition_post_status($status, $old_status, $post);
 
 	/** This action is documented in wp-includes/post.php */
-	do_action( 'edit_post', $post->ID, $post );
+	do_action('edit_post', $post->ID, $post);
 	/** This action is documented in wp-includes/post.php */
-	do_action( "save_post_{$post->post_type}", $post->ID, $post, true );
+	do_action("save_post_{$post->post_type}", $post->ID, $post, true);
 	/** This action is documented in wp-includes/post.php */
-	do_action( 'wl_linked_data_save_post', $post->ID );
+	do_action('wl_linked_data_save_post', $post->ID);
 	/** This action is documented in wp-includes/post.php */
-	do_action( 'wp_insert_post', $post->ID, $post, true );
+	do_action('wp_insert_post', $post->ID, $post, true);
 }