insideout10 / wordlift-plugin

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);
 }

src/admin/wordlift-admin-menu.php

Indentation +32 added lines, -32 removed lines

@@ -15,25 +15,25 @@ 
  */
 function wl_admin_menu() {
 
-	$menu_slug  = 'wl_admin_menu';
-	$capability = 'manage_options';
+    $menu_slug  = 'wl_admin_menu';
+    $capability = 'manage_options';
 
-	// see http://codex.wordpress.org/Function_Reference/add_menu_page
-	add_menu_page(
-		__( 'WordLift', 'wordlift' ), // page title
-		__( 'WordLift', 'wordlift' ), // menu title
-		$capability,                 // capabilities
-		$menu_slug,                  // menu slug
-		// 'wl_admin_menu_callback',  // TODO: function callback to draw the coming dashboard
-		'wl_configuration_admin_menu_callback',
-		WP_CONTENT_URL . '/plugins/wordlift/images/svg/wl-logo-icon.svg'
-	);  // icon URL 20x20 px
+    // see http://codex.wordpress.org/Function_Reference/add_menu_page
+    add_menu_page(
+        __( 'WordLift', 'wordlift' ), // page title
+        __( 'WordLift', 'wordlift' ), // menu title
+        $capability,                 // capabilities
+        $menu_slug,                  // menu slug
+        // 'wl_admin_menu_callback',  // TODO: function callback to draw the coming dashboard
+        'wl_configuration_admin_menu_callback',
+        WP_CONTENT_URL . '/plugins/wordlift/images/svg/wl-logo-icon.svg'
+    );  // icon URL 20x20 px
 
-	// Call hooked functions.
-	do_action( 'wl_admin_menu', $menu_slug, $capability );
+    // Call hooked functions.
+    do_action( 'wl_admin_menu', $menu_slug, $capability );
 
-	// Remove duplicate 'WordLift' subpage created by WordPress.
-	remove_submenu_page( $menu_slug, $menu_slug );
+    // Remove duplicate 'WordLift' subpage created by WordPress.
+    remove_submenu_page( $menu_slug, $menu_slug );
 
 }
 
@@ -47,20 +47,20 @@ 
  * @since 3.15.0
  */
 function wl_remove_entity_type_menu() {
-	/*
+    /*
 	 * Remove from the menu links to the entity type admin page when
 	 * under non entity hierarchy.
 	 */
-	foreach ( Wordlift_Entity_Service::valid_entity_post_types() as $post_type ) {
-		// In the context of admin menues post has no explicit indication of post type in the urls.
-		if ( 'post' !== $post_type ) {
-			remove_submenu_page( 'edit.php', 'edit-tags.php?taxonomy=' . Wordlift_Entity_Type_Taxonomy_Service::TAXONOMY_NAME );
-		}
+    foreach ( Wordlift_Entity_Service::valid_entity_post_types() as $post_type ) {
+        // In the context of admin menues post has no explicit indication of post type in the urls.
+        if ( 'post' !== $post_type ) {
+            remove_submenu_page( 'edit.php', 'edit-tags.php?taxonomy=' . Wordlift_Entity_Type_Taxonomy_Service::TAXONOMY_NAME );
+        }
 
-		if ( Wordlift_Entity_Service::TYPE_NAME !== $post_type ) {
-			remove_submenu_page( 'edit.php?post_type=' . $post_type, 'edit-tags.php?taxonomy=' . Wordlift_Entity_Type_Taxonomy_Service::TAXONOMY_NAME . '&post_type=' . $post_type );
-		}
-	}
+        if ( Wordlift_Entity_Service::TYPE_NAME !== $post_type ) {
+            remove_submenu_page( 'edit.php?post_type=' . $post_type, 'edit-tags.php?taxonomy=' . Wordlift_Entity_Type_Taxonomy_Service::TAXONOMY_NAME . '&post_type=' . $post_type );
+        }
+    }
 }
 
 add_action( 'admin_menu', 'wl_remove_entity_type_menu', 100 );
@@ -72,12 +72,12 @@ 
  */
 function wl_admin_menu_callback() {
 
-	if ( ! current_user_can( 'manage_options' ) ) {
-		wp_die( esc_html__( 'You do not have sufficient permissions to access this page.', 'default' ) );
-	}
+    if ( ! current_user_can( 'manage_options' ) ) {
+        wp_die( esc_html__( 'You do not have sufficient permissions to access this page.', 'default' ) );
+    }
 
-	echo '<div class="wrap">';
-	echo '<p>Here is where the form would go if I actually had options.</p>';
-	echo '</div>';
+    echo '<div class="wrap">';
+    echo '<p>Here is where the form would go if I actually had options.</p>';
+    echo '</div>';
 
 }

Spacing +17 added lines, -17 removed lines

@@ -20,24 +20,24 @@ 
 
 	// see http://codex.wordpress.org/Function_Reference/add_menu_page
 	add_menu_page(
-		__( 'WordLift', 'wordlift' ), // page title
-		__( 'WordLift', 'wordlift' ), // menu title
-		$capability,                 // capabilities
-		$menu_slug,                  // menu slug
+		__('WordLift', 'wordlift'), // page title
+		__('WordLift', 'wordlift'), // menu title
+		$capability, // capabilities
+		$menu_slug, // menu slug
 		// 'wl_admin_menu_callback',  // TODO: function callback to draw the coming dashboard
 		'wl_configuration_admin_menu_callback',
-		WP_CONTENT_URL . '/plugins/wordlift/images/svg/wl-logo-icon.svg'
-	);  // icon URL 20x20 px
+		WP_CONTENT_URL.'/plugins/wordlift/images/svg/wl-logo-icon.svg'
+	); // icon URL 20x20 px
 
 	// Call hooked functions.
-	do_action( 'wl_admin_menu', $menu_slug, $capability );
+	do_action('wl_admin_menu', $menu_slug, $capability);
 
 	// Remove duplicate 'WordLift' subpage created by WordPress.
-	remove_submenu_page( $menu_slug, $menu_slug );
+	remove_submenu_page($menu_slug, $menu_slug);
 
 }
 
-add_action( 'admin_menu', 'wl_admin_menu' );
+add_action('admin_menu', 'wl_admin_menu');
 
 /**
  * This function is called by the *admin_menu* hook to remove for the admin menu
@@ -51,19 +51,19 @@ 
 	 * Remove from the menu links to the entity type admin page when
 	 * under non entity hierarchy.
 	 */
-	foreach ( Wordlift_Entity_Service::valid_entity_post_types() as $post_type ) {
+	foreach (Wordlift_Entity_Service::valid_entity_post_types() as $post_type) {
 		// In the context of admin menues post has no explicit indication of post type in the urls.
-		if ( 'post' !== $post_type ) {
-			remove_submenu_page( 'edit.php', 'edit-tags.php?taxonomy=' . Wordlift_Entity_Type_Taxonomy_Service::TAXONOMY_NAME );
+		if ('post' !== $post_type) {
+			remove_submenu_page('edit.php', 'edit-tags.php?taxonomy='.Wordlift_Entity_Type_Taxonomy_Service::TAXONOMY_NAME);
 		}
 
-		if ( Wordlift_Entity_Service::TYPE_NAME !== $post_type ) {
-			remove_submenu_page( 'edit.php?post_type=' . $post_type, 'edit-tags.php?taxonomy=' . Wordlift_Entity_Type_Taxonomy_Service::TAXONOMY_NAME . '&post_type=' . $post_type );
+		if (Wordlift_Entity_Service::TYPE_NAME !== $post_type) {
+			remove_submenu_page('edit.php?post_type='.$post_type, 'edit-tags.php?taxonomy='.Wordlift_Entity_Type_Taxonomy_Service::TAXONOMY_NAME.'&post_type='.$post_type);
 		}
 	}
 }
 
-add_action( 'admin_menu', 'wl_remove_entity_type_menu', 100 );
+add_action('admin_menu', 'wl_remove_entity_type_menu', 100);
 
 /**
  * This function is called as a callback by the *wl_admin_menu* to display the actual page.
@@ -72,8 +72,8 @@ 
  */
 function wl_admin_menu_callback() {
 
-	if ( ! current_user_can( 'manage_options' ) ) {
-		wp_die( esc_html__( 'You do not have sufficient permissions to access this page.', 'default' ) );
+	if ( ! current_user_can('manage_options')) {
+		wp_die(esc_html__('You do not have sufficient permissions to access this page.', 'default'));
 	}
 
 	echo '<div class="wrap">';

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

Indentation +39 added lines, -39 removed lines

@@ -18,50 +18,50 @@
  */
 class Wordlift_Admin_Settings_Page_Action_Link {
 
-	/**
-	 * The {@link Wordlift_Admin_Settings_Page} instance.
-	 *
-	 * @since  3.11.0
-	 * @access private
-	 * @var Wordlift_Admin_Settings_Page $settings_page The {@link Wordlift_Admin_Settings_Page} instance.
-	 */
-	private $settings_page;
+    /**
+     * The {@link Wordlift_Admin_Settings_Page} instance.
+     *
+     * @since  3.11.0
+     * @access private
+     * @var Wordlift_Admin_Settings_Page $settings_page The {@link Wordlift_Admin_Settings_Page} instance.
+     */
+    private $settings_page;
 
-	/**
-	 * Create a {@link Wordlift_Admin_Settings_Page_Action_Link} instance.
-	 *
-	 * @param Wordlift_Admin_Settings_Page $settings_page The {@link Wordlift_Admin_Settings_Page} instance.
-	 *
-	 * @since  3.11.0
-	 */
-	public function __construct( $settings_page ) {
+    /**
+     * Create a {@link Wordlift_Admin_Settings_Page_Action_Link} instance.
+     *
+     * @param Wordlift_Admin_Settings_Page $settings_page The {@link Wordlift_Admin_Settings_Page} instance.
+     *
+     * @since  3.11.0
+     */
+    public function __construct( $settings_page ) {
 
-		$this->settings_page = $settings_page;
+        $this->settings_page = $settings_page;
 
-	}
+    }
 
-	/**
-	 * Create a link to the WordLift settings page.
-	 *
-	 * @param array $links An array of links.
-	 *
-	 * @return array An array of links including those added by the plugin.
-	 * @since 3.11.0
-	 */
-	public function action_links( $links ) {
+    /**
+     * Create a link to the WordLift settings page.
+     *
+     * @param array $links An array of links.
+     *
+     * @return array An array of links including those added by the plugin.
+     * @since 3.11.0
+     */
+    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='$url'>%s</a>", __( 'Settings', 'wordlift' ) ),
-			)
-		);
-	}
+        // Add our own link to the list of links.
+        return array_merge(
+            $links,
+            array(
+                sprintf( "<a href='$url'>%s</a>", __( 'Settings', 'wordlift' ) ),
+            )
+        );
+    }
 
 }

Spacing +4 added lines, -4 removed lines

@@ -34,7 +34,7 @@ 
 	 *
 	 * @since  3.11.0
 	 */
-	public function __construct( $settings_page ) {
+	public function __construct($settings_page) {
 
 		$this->settings_page = $settings_page;
 
@@ -48,18 +48,18 @@ 
 	 * @return array An array of links including those added by the plugin.
 	 * @since 3.11.0
 	 */
-	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(
 			$links,
 			array(
-				sprintf( "<a href='$url'>%s</a>", __( 'Settings', 'wordlift' ) ),
+				sprintf("<a href='$url'>%s</a>", __('Settings', 'wordlift')),
 			)
 		);
 	}

src/admin/class-wordlift-dashboard-latest-news.php

Indentation +158 added lines, -158 removed lines

@@ -20,163 +20,163 @@
  */
 class Wordlift_Dashboard_Latest_News {
 
-	/**
-	 * Add needed hooks for the latest news widget.
-	 */
-	public function __construct() {
-		add_action(
-			'wp_ajax_wordlift_get_latest_news',
-			array(
-				$this,
-				'ajax_get_latest_news',
-			)
-		);
-		add_action(
-			'wp_dashboard_setup',
-			array(
-				$this,
-				'add_dashboard_latest_news_widget',
-			)
-		);
-
-	}
-
-	/**
-	 * Return latest news html.
-	 */
-	public function render() {
-
-		wp_enqueue_script( 'wl-admin-dashboard', plugin_dir_url( __DIR__ ) . 'admin/js/wordlift-admin-dashboard.js', array( 'jquery' ), '3.22.0', true );
-
-		include plugin_dir_path( __FILE__ ) . 'partials/wordlift-admin-news-widget.php';
-	}
-
-	/**
-	 * Returns latest news data filtered by $start_position.
-	 *
-	 * @param int $start_position (news array key start position).
-	 *
-	 * @return array Latest posts data.
-	 */
-	public function get_last_wordlift_articles( $start_position = 0 ) {
-		$feed_articles = $this->get_wordlift_articles_data();
-
-		// Filter articles by $start_position
-		if ( ! empty( $feed_articles ) ) {
-			return array(
-				'posts_data'     => array_slice( $feed_articles, $start_position, 3 ),
-				'count'          => count( $feed_articles ),
-				'start_position' => $start_position,
-			);
-		}
-
-		return false;
-	}
-
-	/**
-	 * Returns latest news array data.
-	 *
-	 * @uses  https://codex.wordpress.org/Function_Reference/fetch_feed
-	 * @uses  https://codex.wordpress.org/Function_Reference/get_locale
-	 *
-	 * @param int $articles_number (articles total number).
-	 *
-	 * @return array Latest $articles_number feed posts.
-	 */
-	public function get_wordlift_articles_data( $articles_number = 10 ) {
-		// Init cache class
-		$cache_sistem_lib = new Wordlift_File_Cache_Service( WL_TEMP_DIR . 'articles/' );
-		$locale           = get_locale();
-		$cache_id         = 'news_' . gmdate( 'Y_m_d' ) . '_' . $locale;
-		$posts_data       = array();
-
-		// Get latest articles from cache
-		$feed_articles = $cache_sistem_lib->get_cache( $cache_id );
-		if ( false === $feed_articles ) {
-			// Check WordPress installation language to define articles rss url
-			$feed_uri = ( 'it_IT' === $locale ) ? 'https://wordlift.io/blog/it/feed/' : 'https://wordlift.io/blog/en/feed/';
-
-			// Get rss feed data, the response is cached by default for 12 hours
-			$feed = fetch_feed( $feed_uri );
-			if ( ! is_wp_error( $feed ) ) {
-				// Build an array of requested $articles_number, starting with element 0
-				$feed_articles = $feed->get_items( 0, $articles_number );
-				// phpcs:ignore VariableAnalysis.CodeAnalysis.VariableAnalysis.UnusedVariable
-				foreach ( $feed_articles as $key => $item ) {
-					$data = array(
-						// fetch_feed will return the title html decoded.
-						'post_title'       => $item->get_title(),
-						'post_date'        => $item->get_date( 'j F Y | g:i a' ),
-						'post_url'         => self::add_utm_parameter( $item->get_permalink() ),
-						// fetch_feed will return the description html (not decoded).
-						'post_description' => $item->get_description(),
-					);
-					array_push( $posts_data, $data );
-				}
-				// Set articles in cache.
-				$cache_sistem_lib->set_cache( $cache_id, $posts_data );
-				$feed_articles = $posts_data;
-			}
-		}
-
-		return $feed_articles;
-	}
-
-	/**
-	 * Add the `utm` parameter for GA.
-	 *
-	 * @param string $url The URL.
-	 *
-	 * @return string The URL with the `utm` parameter prepended by `&` or by `?`.
-	 * @since 3.19.0
-	 */
-	private static function add_utm_parameter( $url ) {
-
-		if ( false === strpos( $url, '?' ) ) {
-			return $url . '?utm=wl_dash';
-		}
-
-		return $url . '&utm=wl_dash';
-	}
-
-	/**
-	 * Ajax call for more latest news.
-	 *
-	 * @uses  https://codex.wordpress.org/Function_Reference/wp_send_json_success
-	 */
-	public function ajax_get_latest_news() {
-		// Get wordlift articles
-		$more_posts_link_id = isset( $_POST['more_posts_link_id'] ) ? sanitize_text_field( wp_unslash( (string) $_POST['more_posts_link_id'] ) ) : '';//phpcs:ignore WordPress.Security.NonceVerification.Missing
-		$start_position     = explode( '_', $more_posts_link_id );
-		$data               = $this->get_last_wordlift_articles( $start_position[ count( $start_position ) - 1 ] );
-
-		// Return response as json object
-		wp_send_json_success( $data );
-	}
-
-	/**
-	 * Add latest news widget to the administration dashboard.
-	 */
-	public function add_dashboard_latest_news_widget() {
-
-		/**
-		 * Filter name: wl_feature__enable__wordlift-news
-		 * Feature flag to enable / disable news widget.
-		 *
-		 * @since 3.30.0
-		 */
-		// phpcs:ignore WordPress.NamingConventions.ValidHookName.UseUnderscores
-		if ( apply_filters( 'wl_feature__enable__wordlift-news', true ) ) {
-
-			wp_add_dashboard_widget(
-				'wordlift-dashboard-latest-news-widget',
-				'Latest WordLift News',
-				array(
-					$this,
-					'render',
-				)
-			);
-		}
-	}
+    /**
+     * Add needed hooks for the latest news widget.
+     */
+    public function __construct() {
+        add_action(
+            'wp_ajax_wordlift_get_latest_news',
+            array(
+                $this,
+                'ajax_get_latest_news',
+            )
+        );
+        add_action(
+            'wp_dashboard_setup',
+            array(
+                $this,
+                'add_dashboard_latest_news_widget',
+            )
+        );
+
+    }
+
+    /**
+     * Return latest news html.
+     */
+    public function render() {
+
+        wp_enqueue_script( 'wl-admin-dashboard', plugin_dir_url( __DIR__ ) . 'admin/js/wordlift-admin-dashboard.js', array( 'jquery' ), '3.22.0', true );
+
+        include plugin_dir_path( __FILE__ ) . 'partials/wordlift-admin-news-widget.php';
+    }
+
+    /**
+     * Returns latest news data filtered by $start_position.
+     *
+     * @param int $start_position (news array key start position).
+     *
+     * @return array Latest posts data.
+     */
+    public function get_last_wordlift_articles( $start_position = 0 ) {
+        $feed_articles = $this->get_wordlift_articles_data();
+
+        // Filter articles by $start_position
+        if ( ! empty( $feed_articles ) ) {
+            return array(
+                'posts_data'     => array_slice( $feed_articles, $start_position, 3 ),
+                'count'          => count( $feed_articles ),
+                'start_position' => $start_position,
+            );
+        }
+
+        return false;
+    }
+
+    /**
+     * Returns latest news array data.
+     *
+     * @uses  https://codex.wordpress.org/Function_Reference/fetch_feed
+     * @uses  https://codex.wordpress.org/Function_Reference/get_locale
+     *
+     * @param int $articles_number (articles total number).
+     *
+     * @return array Latest $articles_number feed posts.
+     */
+    public function get_wordlift_articles_data( $articles_number = 10 ) {
+        // Init cache class
+        $cache_sistem_lib = new Wordlift_File_Cache_Service( WL_TEMP_DIR . 'articles/' );
+        $locale           = get_locale();
+        $cache_id         = 'news_' . gmdate( 'Y_m_d' ) . '_' . $locale;
+        $posts_data       = array();
+
+        // Get latest articles from cache
+        $feed_articles = $cache_sistem_lib->get_cache( $cache_id );
+        if ( false === $feed_articles ) {
+            // Check WordPress installation language to define articles rss url
+            $feed_uri = ( 'it_IT' === $locale ) ? 'https://wordlift.io/blog/it/feed/' : 'https://wordlift.io/blog/en/feed/';
+
+            // Get rss feed data, the response is cached by default for 12 hours
+            $feed = fetch_feed( $feed_uri );
+            if ( ! is_wp_error( $feed ) ) {
+                // Build an array of requested $articles_number, starting with element 0
+                $feed_articles = $feed->get_items( 0, $articles_number );
+                // phpcs:ignore VariableAnalysis.CodeAnalysis.VariableAnalysis.UnusedVariable
+                foreach ( $feed_articles as $key => $item ) {
+                    $data = array(
+                        // fetch_feed will return the title html decoded.
+                        'post_title'       => $item->get_title(),
+                        'post_date'        => $item->get_date( 'j F Y | g:i a' ),
+                        'post_url'         => self::add_utm_parameter( $item->get_permalink() ),
+                        // fetch_feed will return the description html (not decoded).
+                        'post_description' => $item->get_description(),
+                    );
+                    array_push( $posts_data, $data );
+                }
+                // Set articles in cache.
+                $cache_sistem_lib->set_cache( $cache_id, $posts_data );
+                $feed_articles = $posts_data;
+            }
+        }
+
+        return $feed_articles;
+    }
+
+    /**
+     * Add the `utm` parameter for GA.
+     *
+     * @param string $url The URL.
+     *
+     * @return string The URL with the `utm` parameter prepended by `&` or by `?`.
+     * @since 3.19.0
+     */
+    private static function add_utm_parameter( $url ) {
+
+        if ( false === strpos( $url, '?' ) ) {
+            return $url . '?utm=wl_dash';
+        }
+
+        return $url . '&utm=wl_dash';
+    }
+
+    /**
+     * Ajax call for more latest news.
+     *
+     * @uses  https://codex.wordpress.org/Function_Reference/wp_send_json_success
+     */
+    public function ajax_get_latest_news() {
+        // Get wordlift articles
+        $more_posts_link_id = isset( $_POST['more_posts_link_id'] ) ? sanitize_text_field( wp_unslash( (string) $_POST['more_posts_link_id'] ) ) : '';//phpcs:ignore WordPress.Security.NonceVerification.Missing
+        $start_position     = explode( '_', $more_posts_link_id );
+        $data               = $this->get_last_wordlift_articles( $start_position[ count( $start_position ) - 1 ] );
+
+        // Return response as json object
+        wp_send_json_success( $data );
+    }
+
+    /**
+     * Add latest news widget to the administration dashboard.
+     */
+    public function add_dashboard_latest_news_widget() {
+
+        /**
+         * Filter name: wl_feature__enable__wordlift-news
+         * Feature flag to enable / disable news widget.
+         *
+         * @since 3.30.0
+         */
+        // phpcs:ignore WordPress.NamingConventions.ValidHookName.UseUnderscores
+        if ( apply_filters( 'wl_feature__enable__wordlift-news', true ) ) {
+
+            wp_add_dashboard_widget(
+                'wordlift-dashboard-latest-news-widget',
+                'Latest WordLift News',
+                array(
+                    $this,
+                    'render',
+                )
+            );
+        }
+    }
 
 }

Spacing +29 added lines, -29 removed lines

@@ -46,9 +46,9 @@ 
 	 */
 	public function render() {
 
-		wp_enqueue_script( 'wl-admin-dashboard', plugin_dir_url( __DIR__ ) . 'admin/js/wordlift-admin-dashboard.js', array( 'jquery' ), '3.22.0', true );
+		wp_enqueue_script('wl-admin-dashboard', plugin_dir_url(__DIR__).'admin/js/wordlift-admin-dashboard.js', array('jquery'), '3.22.0', true);
 
-		include plugin_dir_path( __FILE__ ) . 'partials/wordlift-admin-news-widget.php';
+		include plugin_dir_path(__FILE__).'partials/wordlift-admin-news-widget.php';
 	}
 
 	/**
@@ -58,14 +58,14 @@ 
 	 *
 	 * @return array Latest posts data.
 	 */
-	public function get_last_wordlift_articles( $start_position = 0 ) {
+	public function get_last_wordlift_articles($start_position = 0) {
 		$feed_articles = $this->get_wordlift_articles_data();
 
 		// Filter articles by $start_position
-		if ( ! empty( $feed_articles ) ) {
+		if ( ! empty($feed_articles)) {
 			return array(
-				'posts_data'     => array_slice( $feed_articles, $start_position, 3 ),
-				'count'          => count( $feed_articles ),
+				'posts_data'     => array_slice($feed_articles, $start_position, 3),
+				'count'          => count($feed_articles),
 				'start_position' => $start_position,
 			);
 		}
@@ -83,38 +83,38 @@ 
 	 *
 	 * @return array Latest $articles_number feed posts.
 	 */
-	public function get_wordlift_articles_data( $articles_number = 10 ) {
+	public function get_wordlift_articles_data($articles_number = 10) {
 		// Init cache class
-		$cache_sistem_lib = new Wordlift_File_Cache_Service( WL_TEMP_DIR . 'articles/' );
+		$cache_sistem_lib = new Wordlift_File_Cache_Service(WL_TEMP_DIR.'articles/');
 		$locale           = get_locale();
-		$cache_id         = 'news_' . gmdate( 'Y_m_d' ) . '_' . $locale;
+		$cache_id         = 'news_'.gmdate('Y_m_d').'_'.$locale;
 		$posts_data       = array();
 
 		// Get latest articles from cache
-		$feed_articles = $cache_sistem_lib->get_cache( $cache_id );
-		if ( false === $feed_articles ) {
+		$feed_articles = $cache_sistem_lib->get_cache($cache_id);
+		if (false === $feed_articles) {
 			// Check WordPress installation language to define articles rss url
-			$feed_uri = ( 'it_IT' === $locale ) ? 'https://wordlift.io/blog/it/feed/' : 'https://wordlift.io/blog/en/feed/';
+			$feed_uri = ('it_IT' === $locale) ? 'https://wordlift.io/blog/it/feed/' : 'https://wordlift.io/blog/en/feed/';
 
 			// Get rss feed data, the response is cached by default for 12 hours
-			$feed = fetch_feed( $feed_uri );
-			if ( ! is_wp_error( $feed ) ) {
+			$feed = fetch_feed($feed_uri);
+			if ( ! is_wp_error($feed)) {
 				// Build an array of requested $articles_number, starting with element 0
-				$feed_articles = $feed->get_items( 0, $articles_number );
+				$feed_articles = $feed->get_items(0, $articles_number);
 				// phpcs:ignore VariableAnalysis.CodeAnalysis.VariableAnalysis.UnusedVariable
-				foreach ( $feed_articles as $key => $item ) {
+				foreach ($feed_articles as $key => $item) {
 					$data = array(
 						// fetch_feed will return the title html decoded.
 						'post_title'       => $item->get_title(),
-						'post_date'        => $item->get_date( 'j F Y | g:i a' ),
-						'post_url'         => self::add_utm_parameter( $item->get_permalink() ),
+						'post_date'        => $item->get_date('j F Y | g:i a'),
+						'post_url'         => self::add_utm_parameter($item->get_permalink()),
 						// fetch_feed will return the description html (not decoded).
 						'post_description' => $item->get_description(),
 					);
-					array_push( $posts_data, $data );
+					array_push($posts_data, $data);
 				}
 				// Set articles in cache.
-				$cache_sistem_lib->set_cache( $cache_id, $posts_data );
+				$cache_sistem_lib->set_cache($cache_id, $posts_data);
 				$feed_articles = $posts_data;
 			}
 		}
@@ -130,13 +130,13 @@ 
 	 * @return string The URL with the `utm` parameter prepended by `&` or by `?`.
 	 * @since 3.19.0
 	 */
-	private static function add_utm_parameter( $url ) {
+	private static function add_utm_parameter($url) {
 
-		if ( false === strpos( $url, '?' ) ) {
-			return $url . '?utm=wl_dash';
+		if (false === strpos($url, '?')) {
+			return $url.'?utm=wl_dash';
 		}
 
-		return $url . '&utm=wl_dash';
+		return $url.'&utm=wl_dash';
 	}
 
 	/**
@@ -146,12 +146,12 @@ 
 	 */
 	public function ajax_get_latest_news() {
 		// Get wordlift articles
-		$more_posts_link_id = isset( $_POST['more_posts_link_id'] ) ? sanitize_text_field( wp_unslash( (string) $_POST['more_posts_link_id'] ) ) : '';//phpcs:ignore WordPress.Security.NonceVerification.Missing
-		$start_position     = explode( '_', $more_posts_link_id );
-		$data               = $this->get_last_wordlift_articles( $start_position[ count( $start_position ) - 1 ] );
+		$more_posts_link_id = isset($_POST['more_posts_link_id']) ? sanitize_text_field(wp_unslash((string) $_POST['more_posts_link_id'])) : ''; //phpcs:ignore WordPress.Security.NonceVerification.Missing
+		$start_position     = explode('_', $more_posts_link_id);
+		$data               = $this->get_last_wordlift_articles($start_position[count($start_position) - 1]);
 
 		// Return response as json object
-		wp_send_json_success( $data );
+		wp_send_json_success($data);
 	}
 
 	/**
@@ -166,7 +166,7 @@ 
 		 * @since 3.30.0
 		 */
 		// phpcs:ignore WordPress.NamingConventions.ValidHookName.UseUnderscores
-		if ( apply_filters( 'wl_feature__enable__wordlift-news', true ) ) {
+		if (apply_filters('wl_feature__enable__wordlift-news', true)) {
 
 			wp_add_dashboard_widget(
 				'wordlift-dashboard-latest-news-widget',

src/admin/wordlift-admin-shortcodes.php

Indentation +7 added lines, -7 removed lines

@@ -9,11 +9,11 @@ 
  * Loads the buttons in TinyMCE.
  */
 function wl_admin_shortcode_buttons() {
-	// Only add hooks when the current user has permissions AND is in Rich Text editor mode
-	if ( ( current_user_can( 'edit_posts' ) || current_user_can( 'edit_pages' ) ) && get_user_option( 'rich_editing' ) ) {
-		// add_filter( 'mce_external_plugins', 'wl_admin_shortcode_buttons_register_tinymce_javascript' );
-		add_filter( 'mce_buttons', 'wl_admin_shortcode_register_buttons' );
-	}
+    // Only add hooks when the current user has permissions AND is in Rich Text editor mode
+    if ( ( current_user_can( 'edit_posts' ) || current_user_can( 'edit_pages' ) ) && get_user_option( 'rich_editing' ) ) {
+        // add_filter( 'mce_external_plugins', 'wl_admin_shortcode_buttons_register_tinymce_javascript' );
+        add_filter( 'mce_buttons', 'wl_admin_shortcode_register_buttons' );
+    }
 }
 
 /**
@@ -24,9 +24,9 @@ 
  * @return array The buttons array including the *wl_shortcodes_menu*.
  */
 function wl_admin_shortcode_register_buttons( $buttons ) {
-	array_push( $buttons, 'wl_shortcodes_menu' );
+    array_push( $buttons, 'wl_shortcodes_menu' );
 
-	return $buttons;
+    return $buttons;
 }
 
 // init process for button control

Spacing +5 added lines, -5 removed lines

@@ -10,9 +10,9 @@ 
  */
 function wl_admin_shortcode_buttons() {
 	// Only add hooks when the current user has permissions AND is in Rich Text editor mode
-	if ( ( current_user_can( 'edit_posts' ) || current_user_can( 'edit_pages' ) ) && get_user_option( 'rich_editing' ) ) {
+	if ((current_user_can('edit_posts') || current_user_can('edit_pages')) && get_user_option('rich_editing')) {
 		// add_filter( 'mce_external_plugins', 'wl_admin_shortcode_buttons_register_tinymce_javascript' );
-		add_filter( 'mce_buttons', 'wl_admin_shortcode_register_buttons' );
+		add_filter('mce_buttons', 'wl_admin_shortcode_register_buttons');
 	}
 }
 
@@ -23,11 +23,11 @@ 
  *
  * @return array The buttons array including the *wl_shortcodes_menu*.
  */
-function wl_admin_shortcode_register_buttons( $buttons ) {
-	array_push( $buttons, 'wl_shortcodes_menu' );
+function wl_admin_shortcode_register_buttons($buttons) {
+	array_push($buttons, 'wl_shortcodes_menu');
 
 	return $buttons;
 }
 
 // init process for button control
-add_action( 'admin_init', 'wl_admin_shortcode_buttons' );
+add_action('admin_init', 'wl_admin_shortcode_buttons');