insideout10 / wordlift-plugin

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

Indentation +373 added lines, -373 removed lines

@@ -18,378 +18,378 @@
  */
 class Wordlift_Admin_Settings_Page extends Wordlift_Admin_Page {
 
-	/**
-	 * A {@link Wordlift_Entity_Service} instance.
-	 *
-	 * @since  3.11.0
-	 * @access private
-	 * @var \Wordlift_Entity_Service $entity_service A {@link Wordlift_Entity_Service} instance.
-	 */
-	private $entity_service;
-
-	/**
-	 * A {@link Wordlift_Admin_Input_Element} element renderer.
-	 *
-	 * @since  3.11.0
-	 * @access private
-	 * @var \Wordlift_Admin_Input_Element $input_element An {@link Wordlift_Admin_Input_Element} element renderer.
-	 */
-	private $input_element;
-
-	/**
-	 * A {@link Wordlift_Admin_Radio_Input_Element} element renderer.
-	 *
-	 * @since  3.13.0
-	 * @access protected
-	 * @var \Wordlift_Admin_Radio_Input_Element $radio_input_element A {@link Wordlift_Admin_Radio_Input_Element} element renderer.
-	 */
-	private $radio_input_element;
-
-	/**
-	 * A {@link Wordlift_Admin_Language_Select_Element} element renderer.
-	 *
-	 * @since  3.11.0
-	 * @access private
-	 * @var \Wordlift_Admin_Language_Select_Element $language_select_element A {@link Wordlift_Admin_Language_Select_Element} element renderer.
-	 */
-	private $language_select_element;
-
-	/**
-	 * A {@link Wordlift_Admin_Country_Select_Element} element renderer.
-	 *
-	 * @since  3.18.0
-	 * @access private
-	 * @var \Wordlift_Admin_Country_Select_Element $country_select_element A {@link Wordlift_Admin_Country_Select_Element} element renderer.
-	 */
-	private $country_select_element;
-
-	/**
-	 * A {@link Wordlift_Admin_Publisher_Element} element renderer.
-	 *
-	 * @since  3.11.0
-	 * @access private
-	 * @var \Wordlift_Admin_Publisher_Element $publisher_element A {@link Wordlift_Admin_Publisher_Element} element renderer.
-	 */
-	private $publisher_element;
-
-	/**
-	 * Create a {@link Wordlift_Admin_Settings_Page} instance.
-	 *
-	 * @param \Wordlift_Entity_Service $entity_service A {@link Wordlift_Entity_Service} instance.
-	 * @param \Wordlift_Admin_Input_Element $input_element A {@link Wordlift_Admin_Input_Element} element renderer.
-	 * @param \Wordlift_Admin_Language_Select_Element $language_select_element A {@link Wordlift_Admin_Language_Select_Element} element renderer.
-	 * @param \Wordlift_Admin_Country_Select_Element $country_select_element A {@link Wordlift_Admin_Country_Select_Element} element renderer.
-	 * @param \Wordlift_Admin_Publisher_Element $publisher_element A {@link Wordlift_Admin_Publisher_Element} element renderer.
-	 * @param \Wordlift_Admin_Radio_Input_Element $radio_input_element A {@link Wordlift_Admin_Radio_Input_Element} element renderer.
-	 *
-	 * @since 3.11.0
-	 */
-	public function __construct( $entity_service, $input_element, $language_select_element, $country_select_element, $publisher_element, $radio_input_element ) {
-
-		$this->entity_service = $entity_service;
-
-		// Set a reference to the UI elements.
-		$this->input_element           = $input_element;
-		$this->radio_input_element     = $radio_input_element;
-		$this->language_select_element = $language_select_element;
-		$this->country_select_element  = $country_select_element;
-		$this->publisher_element       = $publisher_element;
-
-	}
-
-	private static $instance;
-
-	/**
-	 * Get the singleton instance of the Notice service.
-	 *
-	 * @return \Wordlift_Admin_Settings_Page The singleton instance of the settings page service.
-	 * @since 3.14.0
-	 */
-	public static function get_instance() {
-
-		if ( ! isset( self::$instance ) ) {
-			$publisher_element = new Wordlift_Admin_Publisher_Element(
-				Wordlift_Publisher_Service::get_instance(),
-				new Wordlift_Admin_Tabs_Element(),
-				new Wordlift_Admin_Select2_Element()
-			);
-
-			self::$instance = new self(
-				Wordlift_Entity_Service::get_instance(),
-				new Wordlift_Admin_Input_Element(),
-				new Wordlift_Admin_Language_Select_Element(),
-				new Wordlift_Admin_Country_Select_Element(),
-				$publisher_element,
-				new Wordlift_Admin_Radio_Input_Element()
-			);
-		}
-
-		return self::$instance;
-	}
-
-	/**
-	 * @inheritdoc
-	 */
-	public function get_parent_slug() {
-
-		return 'wl_admin_menu';
-	}
-
-	/**
-	 * @inheritdoc
-	 */
-	public function get_capability() {
-
-		return 'manage_options';
-	}
-
-	/**
-	 * @inheritdoc
-	 */
-	public function get_page_title() {
-
-		return 'WordLift Settings';
-	}
-
-	/**
-	 * @inheritdoc
-	 */
-	public function get_menu_title() {
-
-		return 'Settings';
-	}
-
-	/**
-	 * @inheritdoc
-	 */
-	public function get_menu_slug() {
-
-		return 'wl_configuration_admin_menu';
-	}
-
-	/**
-	 * @inheritdoc
-	 */
-	public function get_partial_name() {
-
-		return 'wordlift-admin-settings-page.php';
-	}
-
-	/**
-	 * @inheritdoc
-	 */
-	public function enqueue_scripts() {
-
-		// Enqueue the media scripts to be used for the publisher's logo selection.
-		wp_enqueue_media();
-
-		// JavaScript required for the settings page.
-		// @todo: try to move to the `wordlift-admin.bundle.js`.
-		wp_enqueue_script( 'wordlift-admin-settings-page', plugin_dir_url( __DIR__ ) . 'admin/js/1/settings.js', array( 'wp-util' ), WORDLIFT_VERSION, false );
-		wp_enqueue_style( 'wordlift-admin-settings-page', plugin_dir_url( __DIR__ ) . 'admin/js/1/settings.css', array(), WORDLIFT_VERSION );
-
-	}
-
-	/**
-	 * Configure all the configuration parameters.
-	 *
-	 * Called by the *admin_init* hook.
-	 *
-	 * @since 3.11.0
-	 */
-	public function admin_init() {
-		// Register WordLift's general settings, providing our own sanitize callback
-		// which will also check whether the user filled the WL Publisher form.
-		register_setting(
-			'wl_general_settings',
-			'wl_general_settings',
-			array( $this, 'sanitize_callback' )
-		);
-
-		// Add the general settings section.
-		add_settings_section(
-			'wl_general_settings_section', // ID used to identify this section and with which to register options.
-			'',                            // Section header.
-			'',                            // Callback used to render the description of the section.
-			'wl_general_settings'          // Page on which to add this section of options.
-		);
-
-		$key_args = array(
-			'id'          => 'wl-key',
-			'name'        => 'wl_general_settings[' . Wordlift_Configuration_Service::KEY . ']',
-			'value'       => Wordlift_Configuration_Service::get_instance()->get_key(),
-			'description' => __( 'Insert the <a href="https://www.wordlift.io/blogger">WordLift Key</a> you received via email.', 'wordlift' )
-			                 . ' [' . get_option( 'home' ) . ']',
-		);
-
-		// Before we were used to validate the key beforehand, but this means
-		// an http call whenever a page is opened in the admin area. Therefore
-		// we now leave the input `untouched`, leaving to the client to update
-		// the `css_class`.
-		//
-		// See https://github.com/insideout10/wordlift-plugin/issues/669.
-		$key_args['css_class'] = 'untouched';
-
-		// Add the `key` field.
-		add_settings_field(
-			'wl-key',                                       // Element id used to identify the field throughout the theme.
-			__( 'WordLift Key', 'wordlift' ),               // The label to the left of the option interface element.
-			// The name of the function responsible for rendering the option interface.
-			array( $this->input_element, 'render' ),
-			'wl_general_settings',                          // The page on which this option will be displayed.
-			'wl_general_settings_section',                  // The name of the section to which this field belongs.
-			$key_args                                       // The array of arguments to pass to the callback. In this case, just a description.
-		);
-
-		// Entity Base Path input.
-		$entity_base_path_args = array(
-			// The array of arguments to pass to the callback. In this case, just a description.
-			'id'          => 'wl-entity-base-path',
-			'name'        => 'wl_general_settings[' . Wordlift_Configuration_Service::ENTITY_BASE_PATH_KEY . ']',
-			'value'       => Wordlift_Configuration_Service::get_instance()->get_entity_base_path(),
-			/* translators: Placeholders: %s - a link to FAQ's page. */
-			'description' => sprintf( __( 'All new pages created with WordLift, will be stored inside your internal vocabulary. You can customize the url pattern of these pages in the field above. Check our <a href="%s">FAQs</a> if you need more info.', 'wordlift' ), 'https://wordlift.io/wordlift-user-faqs/#10-why-and-how-should-i-customize-the-url-of-the-entity-pages-created-in-my-vocabulary' ),
-		);
-
-		// The following call is very heavy on large web sites and is always run
-		// also when not needed:
-		// $entity_base_path_args['readonly'] = 0 < $this->entity_service->count();
-		//
-		// It is now replaced by a filter to add the `readonly` flag to the
-		// input element when this is actually rendered.
-		add_filter(
-			'wl_admin_input_element_params',
-			array(
-				$this,
-				'entity_path_input_element_params',
-			)
-		);
-
-		// Add the `wl_entity_base_path` field.
-		add_settings_field(
-			'wl-entity-base-path',                                // ID used to identify the field throughout the theme
-			__( 'Entity Base Path', 'wordlift' ),                 // The label to the left of the option interface element
-			// The name of the function responsible for rendering the option interface
-			array( $this->input_element, 'render' ),
-			'wl_general_settings',                                // The page on which this option will be displayed
-			'wl_general_settings_section',                        // The name of the section to which this field belongs
-			$entity_base_path_args
-		);
-
-		// Add the `country_code` field.
-		add_settings_field(
-			'wl-country-code',
-			__( 'Country', 'wordlift' ),
-			array( $this->country_select_element, 'render' ),
-			'wl_general_settings',
-			'wl_general_settings_section',
-			array(
-				'id'          => 'wl-country-code',
-				'name'        => 'wl_general_settings[' . Wordlift_Configuration_Service::COUNTRY_CODE . ']',
-				'value'       => Wordlift_Configuration_Service::get_instance()->get_country_code(),
-				'description' => __( 'Please select a country.', 'wordlift' ),
-				'notice'      => __( 'The selected language is not supported in this country.</br>Please choose another country or language.', 'wordlift' ),
-			)
-		);
-
-		// Add the `publisher` field.
-		add_settings_field(
-			'wl-publisher-id',
-			__( 'Publisher', 'wordlift' ),
-			array( $this->publisher_element, 'render' ),
-			'wl_general_settings',
-			'wl_general_settings_section',
-			array(
-				'id'   => 'wl-publisher-id',
-				'name' => 'wl_general_settings[' . Wordlift_Configuration_Service::PUBLISHER_ID . ']',
-			)
-		);
-
-		// Add the `link by default` field.
-		add_settings_field(
-			'wl-link-by-default',
-			__( 'Link by Default', 'wordlift' ),
-			array( $this->radio_input_element, 'render' ),
-			'wl_general_settings',
-			'wl_general_settings_section',
-			array(
-				'id'          => 'wl-link-by-default',
-				'name'        => 'wl_general_settings[' . Wordlift_Configuration_Service::LINK_BY_DEFAULT . ']',
-				'value'       => Wordlift_Configuration_Service::get_instance()->is_link_by_default() ? 'yes' : 'no',
-				'description' => __( 'Whether to link entities by default or not. This setting applies to all the entities.', 'wordlift' ),
-			)
-		);
-
-		// Add the `diagnostic data` field.
-		add_settings_field(
-			'wl-send-diagnostic',
-			__( 'Send Diagnostic Data', 'wordlift' ),
-			array( $this->radio_input_element, 'render' ),
-			'wl_general_settings',
-			'wl_general_settings_section',
-			array(
-				'id'          => 'wl-send-diagnostic',
-				'name'        => 'wl_general_settings[' . Wordlift_Configuration_Service::SEND_DIAGNOSTIC . ']',
-				'value'       => 'yes' === Wordlift_Configuration_Service::get_instance()->get_diagnostic_preferences() ? 'yes' : 'no',
-				'description' => __( 'Whether to send diagnostic data or not.', 'wordlift' ),
-			)
-		);
-
-	}
-
-	/**
-	 * Filter the {@link Wordlift_Admin_Input_Element} in order to add the
-	 * `readonly` flag to the `wl-entity-base-path` input.
-	 *
-	 * @param array $args An array of {@link Wordlift_Admin_Input_Element} parameters.
-	 *
-	 * @return array The updated array.
-	 * @since 3.17.0
-	 */
-	public function entity_path_input_element_params( $args ) {
-
-		// Bail out if it's not the `wl-entity-base-path`).
-		if ( 'wl-entity-base-path' !== $args['id'] ) {
-			return $args;
-		}
-
-		// Set the readonly flag according to the entities count.
-		$args['readonly'] = 0 < $this->entity_service->count();
-
-		// Return the updated args.
-		return $args;
-	}
-
-	/**
-	 * Sanitize the configuration settings to be stored.
-	 *
-	 * If a new entity is being created for the publisher, create it and set The
-	 * publisher setting.
-	 *
-	 * @param array $input The configuration settings array.
-	 *
-	 * @return array The sanitized input array.
-	 * @since 3.11.0
-	 */
-	public function sanitize_callback( $input ) {
-		// No nonce verification since this callback is handled by settings api.
-		// Check whether a publisher name has been set.
-		// phpcs:ignore Standard.Category.SniffName.ErrorCode
-		if ( isset( $_POST['wl_publisher'] ) && ! empty( $_POST['wl_publisher']['name'] ) ) { //phpcs:ignore WordPress.Security.NonceVerification.Missing
-			$name = isset( $_POST['wl_publisher']['name'] ) ? sanitize_text_field( wp_unslash( (string) $_POST['wl_publisher']['name'] ) ) : ''; //phpcs:ignore WordPress.Security.NonceVerification.Missing
-			$type = isset( $_POST['wl_publisher']['type'] ) ? sanitize_text_field( wp_unslash( (string) $_POST['wl_publisher']['type'] ) ) : ''; //phpcs:ignore WordPress.Security.NonceVerification.Missing
-			// phpcs:ignore Standard.Category.SniffName.ErrorCode
-			$thumbnail_id = isset( $_POST['wl_publisher']['thumbnail_id'] ) ? sanitize_text_field( wp_unslash( $_POST['wl_publisher']['thumbnail_id'] ) ) : null; //phpcs:ignore WordPress.Security.NonceVerification.Missing
-
-			// Set the type URI, either http://schema.org/Person or http://schema.org/Organization.
-			$type_uri = sprintf( 'http://schema.org/%s', 'organization' === $type ? 'Organization' : 'Person' );
-
-			// Create an entity for the publisher and assign it to the input
-			// parameter which WordPress automatically saves into the settings.
-			$input['publisher_id'] = $this->entity_service->create( $name, $type_uri, $thumbnail_id, 'publish' );
-		}
-
-		return $input;
-	}
+    /**
+     * A {@link Wordlift_Entity_Service} instance.
+     *
+     * @since  3.11.0
+     * @access private
+     * @var \Wordlift_Entity_Service $entity_service A {@link Wordlift_Entity_Service} instance.
+     */
+    private $entity_service;
+
+    /**
+     * A {@link Wordlift_Admin_Input_Element} element renderer.
+     *
+     * @since  3.11.0
+     * @access private
+     * @var \Wordlift_Admin_Input_Element $input_element An {@link Wordlift_Admin_Input_Element} element renderer.
+     */
+    private $input_element;
+
+    /**
+     * A {@link Wordlift_Admin_Radio_Input_Element} element renderer.
+     *
+     * @since  3.13.0
+     * @access protected
+     * @var \Wordlift_Admin_Radio_Input_Element $radio_input_element A {@link Wordlift_Admin_Radio_Input_Element} element renderer.
+     */
+    private $radio_input_element;
+
+    /**
+     * A {@link Wordlift_Admin_Language_Select_Element} element renderer.
+     *
+     * @since  3.11.0
+     * @access private
+     * @var \Wordlift_Admin_Language_Select_Element $language_select_element A {@link Wordlift_Admin_Language_Select_Element} element renderer.
+     */
+    private $language_select_element;
+
+    /**
+     * A {@link Wordlift_Admin_Country_Select_Element} element renderer.
+     *
+     * @since  3.18.0
+     * @access private
+     * @var \Wordlift_Admin_Country_Select_Element $country_select_element A {@link Wordlift_Admin_Country_Select_Element} element renderer.
+     */
+    private $country_select_element;
+
+    /**
+     * A {@link Wordlift_Admin_Publisher_Element} element renderer.
+     *
+     * @since  3.11.0
+     * @access private
+     * @var \Wordlift_Admin_Publisher_Element $publisher_element A {@link Wordlift_Admin_Publisher_Element} element renderer.
+     */
+    private $publisher_element;
+
+    /**
+     * Create a {@link Wordlift_Admin_Settings_Page} instance.
+     *
+     * @param \Wordlift_Entity_Service $entity_service A {@link Wordlift_Entity_Service} instance.
+     * @param \Wordlift_Admin_Input_Element $input_element A {@link Wordlift_Admin_Input_Element} element renderer.
+     * @param \Wordlift_Admin_Language_Select_Element $language_select_element A {@link Wordlift_Admin_Language_Select_Element} element renderer.
+     * @param \Wordlift_Admin_Country_Select_Element $country_select_element A {@link Wordlift_Admin_Country_Select_Element} element renderer.
+     * @param \Wordlift_Admin_Publisher_Element $publisher_element A {@link Wordlift_Admin_Publisher_Element} element renderer.
+     * @param \Wordlift_Admin_Radio_Input_Element $radio_input_element A {@link Wordlift_Admin_Radio_Input_Element} element renderer.
+     *
+     * @since 3.11.0
+     */
+    public function __construct( $entity_service, $input_element, $language_select_element, $country_select_element, $publisher_element, $radio_input_element ) {
+
+        $this->entity_service = $entity_service;
+
+        // Set a reference to the UI elements.
+        $this->input_element           = $input_element;
+        $this->radio_input_element     = $radio_input_element;
+        $this->language_select_element = $language_select_element;
+        $this->country_select_element  = $country_select_element;
+        $this->publisher_element       = $publisher_element;
+
+    }
+
+    private static $instance;
+
+    /**
+     * Get the singleton instance of the Notice service.
+     *
+     * @return \Wordlift_Admin_Settings_Page The singleton instance of the settings page service.
+     * @since 3.14.0
+     */
+    public static function get_instance() {
+
+        if ( ! isset( self::$instance ) ) {
+            $publisher_element = new Wordlift_Admin_Publisher_Element(
+                Wordlift_Publisher_Service::get_instance(),
+                new Wordlift_Admin_Tabs_Element(),
+                new Wordlift_Admin_Select2_Element()
+            );
+
+            self::$instance = new self(
+                Wordlift_Entity_Service::get_instance(),
+                new Wordlift_Admin_Input_Element(),
+                new Wordlift_Admin_Language_Select_Element(),
+                new Wordlift_Admin_Country_Select_Element(),
+                $publisher_element,
+                new Wordlift_Admin_Radio_Input_Element()
+            );
+        }
+
+        return self::$instance;
+    }
+
+    /**
+     * @inheritdoc
+     */
+    public function get_parent_slug() {
+
+        return 'wl_admin_menu';
+    }
+
+    /**
+     * @inheritdoc
+     */
+    public function get_capability() {
+
+        return 'manage_options';
+    }
+
+    /**
+     * @inheritdoc
+     */
+    public function get_page_title() {
+
+        return 'WordLift Settings';
+    }
+
+    /**
+     * @inheritdoc
+     */
+    public function get_menu_title() {
+
+        return 'Settings';
+    }
+
+    /**
+     * @inheritdoc
+     */
+    public function get_menu_slug() {
+
+        return 'wl_configuration_admin_menu';
+    }
+
+    /**
+     * @inheritdoc
+     */
+    public function get_partial_name() {
+
+        return 'wordlift-admin-settings-page.php';
+    }
+
+    /**
+     * @inheritdoc
+     */
+    public function enqueue_scripts() {
+
+        // Enqueue the media scripts to be used for the publisher's logo selection.
+        wp_enqueue_media();
+
+        // JavaScript required for the settings page.
+        // @todo: try to move to the `wordlift-admin.bundle.js`.
+        wp_enqueue_script( 'wordlift-admin-settings-page', plugin_dir_url( __DIR__ ) . 'admin/js/1/settings.js', array( 'wp-util' ), WORDLIFT_VERSION, false );
+        wp_enqueue_style( 'wordlift-admin-settings-page', plugin_dir_url( __DIR__ ) . 'admin/js/1/settings.css', array(), WORDLIFT_VERSION );
+
+    }
+
+    /**
+     * Configure all the configuration parameters.
+     *
+     * Called by the *admin_init* hook.
+     *
+     * @since 3.11.0
+     */
+    public function admin_init() {
+        // Register WordLift's general settings, providing our own sanitize callback
+        // which will also check whether the user filled the WL Publisher form.
+        register_setting(
+            'wl_general_settings',
+            'wl_general_settings',
+            array( $this, 'sanitize_callback' )
+        );
+
+        // Add the general settings section.
+        add_settings_section(
+            'wl_general_settings_section', // ID used to identify this section and with which to register options.
+            '',                            // Section header.
+            '',                            // Callback used to render the description of the section.
+            'wl_general_settings'          // Page on which to add this section of options.
+        );
+
+        $key_args = array(
+            'id'          => 'wl-key',
+            'name'        => 'wl_general_settings[' . Wordlift_Configuration_Service::KEY . ']',
+            'value'       => Wordlift_Configuration_Service::get_instance()->get_key(),
+            'description' => __( 'Insert the <a href="https://www.wordlift.io/blogger">WordLift Key</a> you received via email.', 'wordlift' )
+                                . ' [' . get_option( 'home' ) . ']',
+        );
+
+        // Before we were used to validate the key beforehand, but this means
+        // an http call whenever a page is opened in the admin area. Therefore
+        // we now leave the input `untouched`, leaving to the client to update
+        // the `css_class`.
+        //
+        // See https://github.com/insideout10/wordlift-plugin/issues/669.
+        $key_args['css_class'] = 'untouched';
+
+        // Add the `key` field.
+        add_settings_field(
+            'wl-key',                                       // Element id used to identify the field throughout the theme.
+            __( 'WordLift Key', 'wordlift' ),               // The label to the left of the option interface element.
+            // The name of the function responsible for rendering the option interface.
+            array( $this->input_element, 'render' ),
+            'wl_general_settings',                          // The page on which this option will be displayed.
+            'wl_general_settings_section',                  // The name of the section to which this field belongs.
+            $key_args                                       // The array of arguments to pass to the callback. In this case, just a description.
+        );
+
+        // Entity Base Path input.
+        $entity_base_path_args = array(
+            // The array of arguments to pass to the callback. In this case, just a description.
+            'id'          => 'wl-entity-base-path',
+            'name'        => 'wl_general_settings[' . Wordlift_Configuration_Service::ENTITY_BASE_PATH_KEY . ']',
+            'value'       => Wordlift_Configuration_Service::get_instance()->get_entity_base_path(),
+            /* translators: Placeholders: %s - a link to FAQ's page. */
+            'description' => sprintf( __( 'All new pages created with WordLift, will be stored inside your internal vocabulary. You can customize the url pattern of these pages in the field above. Check our <a href="%s">FAQs</a> if you need more info.', 'wordlift' ), 'https://wordlift.io/wordlift-user-faqs/#10-why-and-how-should-i-customize-the-url-of-the-entity-pages-created-in-my-vocabulary' ),
+        );
+
+        // The following call is very heavy on large web sites and is always run
+        // also when not needed:
+        // $entity_base_path_args['readonly'] = 0 < $this->entity_service->count();
+        //
+        // It is now replaced by a filter to add the `readonly` flag to the
+        // input element when this is actually rendered.
+        add_filter(
+            'wl_admin_input_element_params',
+            array(
+                $this,
+                'entity_path_input_element_params',
+            )
+        );
+
+        // Add the `wl_entity_base_path` field.
+        add_settings_field(
+            'wl-entity-base-path',                                // ID used to identify the field throughout the theme
+            __( 'Entity Base Path', 'wordlift' ),                 // The label to the left of the option interface element
+            // The name of the function responsible for rendering the option interface
+            array( $this->input_element, 'render' ),
+            'wl_general_settings',                                // The page on which this option will be displayed
+            'wl_general_settings_section',                        // The name of the section to which this field belongs
+            $entity_base_path_args
+        );
+
+        // Add the `country_code` field.
+        add_settings_field(
+            'wl-country-code',
+            __( 'Country', 'wordlift' ),
+            array( $this->country_select_element, 'render' ),
+            'wl_general_settings',
+            'wl_general_settings_section',
+            array(
+                'id'          => 'wl-country-code',
+                'name'        => 'wl_general_settings[' . Wordlift_Configuration_Service::COUNTRY_CODE . ']',
+                'value'       => Wordlift_Configuration_Service::get_instance()->get_country_code(),
+                'description' => __( 'Please select a country.', 'wordlift' ),
+                'notice'      => __( 'The selected language is not supported in this country.</br>Please choose another country or language.', 'wordlift' ),
+            )
+        );
+
+        // Add the `publisher` field.
+        add_settings_field(
+            'wl-publisher-id',
+            __( 'Publisher', 'wordlift' ),
+            array( $this->publisher_element, 'render' ),
+            'wl_general_settings',
+            'wl_general_settings_section',
+            array(
+                'id'   => 'wl-publisher-id',
+                'name' => 'wl_general_settings[' . Wordlift_Configuration_Service::PUBLISHER_ID . ']',
+            )
+        );
+
+        // Add the `link by default` field.
+        add_settings_field(
+            'wl-link-by-default',
+            __( 'Link by Default', 'wordlift' ),
+            array( $this->radio_input_element, 'render' ),
+            'wl_general_settings',
+            'wl_general_settings_section',
+            array(
+                'id'          => 'wl-link-by-default',
+                'name'        => 'wl_general_settings[' . Wordlift_Configuration_Service::LINK_BY_DEFAULT . ']',
+                'value'       => Wordlift_Configuration_Service::get_instance()->is_link_by_default() ? 'yes' : 'no',
+                'description' => __( 'Whether to link entities by default or not. This setting applies to all the entities.', 'wordlift' ),
+            )
+        );
+
+        // Add the `diagnostic data` field.
+        add_settings_field(
+            'wl-send-diagnostic',
+            __( 'Send Diagnostic Data', 'wordlift' ),
+            array( $this->radio_input_element, 'render' ),
+            'wl_general_settings',
+            'wl_general_settings_section',
+            array(
+                'id'          => 'wl-send-diagnostic',
+                'name'        => 'wl_general_settings[' . Wordlift_Configuration_Service::SEND_DIAGNOSTIC . ']',
+                'value'       => 'yes' === Wordlift_Configuration_Service::get_instance()->get_diagnostic_preferences() ? 'yes' : 'no',
+                'description' => __( 'Whether to send diagnostic data or not.', 'wordlift' ),
+            )
+        );
+
+    }
+
+    /**
+     * Filter the {@link Wordlift_Admin_Input_Element} in order to add the
+     * `readonly` flag to the `wl-entity-base-path` input.
+     *
+     * @param array $args An array of {@link Wordlift_Admin_Input_Element} parameters.
+     *
+     * @return array The updated array.
+     * @since 3.17.0
+     */
+    public function entity_path_input_element_params( $args ) {
+
+        // Bail out if it's not the `wl-entity-base-path`).
+        if ( 'wl-entity-base-path' !== $args['id'] ) {
+            return $args;
+        }
+
+        // Set the readonly flag according to the entities count.
+        $args['readonly'] = 0 < $this->entity_service->count();
+
+        // Return the updated args.
+        return $args;
+    }
+
+    /**
+     * Sanitize the configuration settings to be stored.
+     *
+     * If a new entity is being created for the publisher, create it and set The
+     * publisher setting.
+     *
+     * @param array $input The configuration settings array.
+     *
+     * @return array The sanitized input array.
+     * @since 3.11.0
+     */
+    public function sanitize_callback( $input ) {
+        // No nonce verification since this callback is handled by settings api.
+        // Check whether a publisher name has been set.
+        // phpcs:ignore Standard.Category.SniffName.ErrorCode
+        if ( isset( $_POST['wl_publisher'] ) && ! empty( $_POST['wl_publisher']['name'] ) ) { //phpcs:ignore WordPress.Security.NonceVerification.Missing
+            $name = isset( $_POST['wl_publisher']['name'] ) ? sanitize_text_field( wp_unslash( (string) $_POST['wl_publisher']['name'] ) ) : ''; //phpcs:ignore WordPress.Security.NonceVerification.Missing
+            $type = isset( $_POST['wl_publisher']['type'] ) ? sanitize_text_field( wp_unslash( (string) $_POST['wl_publisher']['type'] ) ) : ''; //phpcs:ignore WordPress.Security.NonceVerification.Missing
+            // phpcs:ignore Standard.Category.SniffName.ErrorCode
+            $thumbnail_id = isset( $_POST['wl_publisher']['thumbnail_id'] ) ? sanitize_text_field( wp_unslash( $_POST['wl_publisher']['thumbnail_id'] ) ) : null; //phpcs:ignore WordPress.Security.NonceVerification.Missing
+
+            // Set the type URI, either http://schema.org/Person or http://schema.org/Organization.
+            $type_uri = sprintf( 'http://schema.org/%s', 'organization' === $type ? 'Organization' : 'Person' );
+
+            // Create an entity for the publisher and assign it to the input
+            // parameter which WordPress automatically saves into the settings.
+            $input['publisher_id'] = $this->entity_service->create( $name, $type_uri, $thumbnail_id, 'publish' );
+        }
+
+        return $input;
+    }
 
 }

Spacing +47 added lines, -47 removed lines

@@ -84,7 +84,7 @@ 
 	 *
 	 * @since 3.11.0
 	 */
-	public function __construct( $entity_service, $input_element, $language_select_element, $country_select_element, $publisher_element, $radio_input_element ) {
+	public function __construct($entity_service, $input_element, $language_select_element, $country_select_element, $publisher_element, $radio_input_element) {
 
 		$this->entity_service = $entity_service;
 
@@ -107,7 +107,7 @@ 
 	 */
 	public static function get_instance() {
 
-		if ( ! isset( self::$instance ) ) {
+		if ( ! isset(self::$instance)) {
 			$publisher_element = new Wordlift_Admin_Publisher_Element(
 				Wordlift_Publisher_Service::get_instance(),
 				new Wordlift_Admin_Tabs_Element(),
@@ -185,8 +185,8 @@ 
 
 		// JavaScript required for the settings page.
 		// @todo: try to move to the `wordlift-admin.bundle.js`.
-		wp_enqueue_script( 'wordlift-admin-settings-page', plugin_dir_url( __DIR__ ) . 'admin/js/1/settings.js', array( 'wp-util' ), WORDLIFT_VERSION, false );
-		wp_enqueue_style( 'wordlift-admin-settings-page', plugin_dir_url( __DIR__ ) . 'admin/js/1/settings.css', array(), WORDLIFT_VERSION );
+		wp_enqueue_script('wordlift-admin-settings-page', plugin_dir_url(__DIR__).'admin/js/1/settings.js', array('wp-util'), WORDLIFT_VERSION, false);
+		wp_enqueue_style('wordlift-admin-settings-page', plugin_dir_url(__DIR__).'admin/js/1/settings.css', array(), WORDLIFT_VERSION);
 
 	}
 
@@ -203,23 +203,23 @@ 
 		register_setting(
 			'wl_general_settings',
 			'wl_general_settings',
-			array( $this, 'sanitize_callback' )
+			array($this, 'sanitize_callback')
 		);
 
 		// Add the general settings section.
 		add_settings_section(
 			'wl_general_settings_section', // ID used to identify this section and with which to register options.
-			'',                            // Section header.
-			'',                            // Callback used to render the description of the section.
+			'', // Section header.
+			'', // Callback used to render the description of the section.
 			'wl_general_settings'          // Page on which to add this section of options.
 		);
 
 		$key_args = array(
 			'id'          => 'wl-key',
-			'name'        => 'wl_general_settings[' . Wordlift_Configuration_Service::KEY . ']',
+			'name'        => 'wl_general_settings['.Wordlift_Configuration_Service::KEY.']',
 			'value'       => Wordlift_Configuration_Service::get_instance()->get_key(),
-			'description' => __( 'Insert the <a href="https://www.wordlift.io/blogger">WordLift Key</a> you received via email.', 'wordlift' )
-			                 . ' [' . get_option( 'home' ) . ']',
+			'description' => __('Insert the <a href="https://www.wordlift.io/blogger">WordLift Key</a> you received via email.', 'wordlift')
+			                 . ' ['.get_option('home').']',
 		);
 
 		// Before we were used to validate the key beforehand, but this means
@@ -232,12 +232,12 @@ 
 
 		// Add the `key` field.
 		add_settings_field(
-			'wl-key',                                       // Element id used to identify the field throughout the theme.
-			__( 'WordLift Key', 'wordlift' ),               // The label to the left of the option interface element.
+			'wl-key', // Element id used to identify the field throughout the theme.
+			__('WordLift Key', 'wordlift'), // The label to the left of the option interface element.
 			// The name of the function responsible for rendering the option interface.
-			array( $this->input_element, 'render' ),
-			'wl_general_settings',                          // The page on which this option will be displayed.
-			'wl_general_settings_section',                  // The name of the section to which this field belongs.
+			array($this->input_element, 'render'),
+			'wl_general_settings', // The page on which this option will be displayed.
+			'wl_general_settings_section', // The name of the section to which this field belongs.
 			$key_args                                       // The array of arguments to pass to the callback. In this case, just a description.
 		);
 
@@ -245,10 +245,10 @@ 
 		$entity_base_path_args = array(
 			// The array of arguments to pass to the callback. In this case, just a description.
 			'id'          => 'wl-entity-base-path',
-			'name'        => 'wl_general_settings[' . Wordlift_Configuration_Service::ENTITY_BASE_PATH_KEY . ']',
+			'name'        => 'wl_general_settings['.Wordlift_Configuration_Service::ENTITY_BASE_PATH_KEY.']',
 			'value'       => Wordlift_Configuration_Service::get_instance()->get_entity_base_path(),
 			/* translators: Placeholders: %s - a link to FAQ's page. */
-			'description' => sprintf( __( 'All new pages created with WordLift, will be stored inside your internal vocabulary. You can customize the url pattern of these pages in the field above. Check our <a href="%s">FAQs</a> if you need more info.', 'wordlift' ), 'https://wordlift.io/wordlift-user-faqs/#10-why-and-how-should-i-customize-the-url-of-the-entity-pages-created-in-my-vocabulary' ),
+			'description' => sprintf(__('All new pages created with WordLift, will be stored inside your internal vocabulary. You can customize the url pattern of these pages in the field above. Check our <a href="%s">FAQs</a> if you need more info.', 'wordlift'), 'https://wordlift.io/wordlift-user-faqs/#10-why-and-how-should-i-customize-the-url-of-the-entity-pages-created-in-my-vocabulary'),
 		);
 
 		// The following call is very heavy on large web sites and is always run
@@ -267,71 +267,71 @@ 
 
 		// Add the `wl_entity_base_path` field.
 		add_settings_field(
-			'wl-entity-base-path',                                // ID used to identify the field throughout the theme
-			__( 'Entity Base Path', 'wordlift' ),                 // The label to the left of the option interface element
+			'wl-entity-base-path', // ID used to identify the field throughout the theme
+			__('Entity Base Path', 'wordlift'), // The label to the left of the option interface element
 			// The name of the function responsible for rendering the option interface
-			array( $this->input_element, 'render' ),
-			'wl_general_settings',                                // The page on which this option will be displayed
-			'wl_general_settings_section',                        // The name of the section to which this field belongs
+			array($this->input_element, 'render'),
+			'wl_general_settings', // The page on which this option will be displayed
+			'wl_general_settings_section', // The name of the section to which this field belongs
 			$entity_base_path_args
 		);
 
 		// Add the `country_code` field.
 		add_settings_field(
 			'wl-country-code',
-			__( 'Country', 'wordlift' ),
-			array( $this->country_select_element, 'render' ),
+			__('Country', 'wordlift'),
+			array($this->country_select_element, 'render'),
 			'wl_general_settings',
 			'wl_general_settings_section',
 			array(
 				'id'          => 'wl-country-code',
-				'name'        => 'wl_general_settings[' . Wordlift_Configuration_Service::COUNTRY_CODE . ']',
+				'name'        => 'wl_general_settings['.Wordlift_Configuration_Service::COUNTRY_CODE.']',
 				'value'       => Wordlift_Configuration_Service::get_instance()->get_country_code(),
-				'description' => __( 'Please select a country.', 'wordlift' ),
-				'notice'      => __( 'The selected language is not supported in this country.</br>Please choose another country or language.', 'wordlift' ),
+				'description' => __('Please select a country.', 'wordlift'),
+				'notice'      => __('The selected language is not supported in this country.</br>Please choose another country or language.', 'wordlift'),
 			)
 		);
 
 		// Add the `publisher` field.
 		add_settings_field(
 			'wl-publisher-id',
-			__( 'Publisher', 'wordlift' ),
-			array( $this->publisher_element, 'render' ),
+			__('Publisher', 'wordlift'),
+			array($this->publisher_element, 'render'),
 			'wl_general_settings',
 			'wl_general_settings_section',
 			array(
 				'id'   => 'wl-publisher-id',
-				'name' => 'wl_general_settings[' . Wordlift_Configuration_Service::PUBLISHER_ID . ']',
+				'name' => 'wl_general_settings['.Wordlift_Configuration_Service::PUBLISHER_ID.']',
 			)
 		);
 
 		// Add the `link by default` field.
 		add_settings_field(
 			'wl-link-by-default',
-			__( 'Link by Default', 'wordlift' ),
-			array( $this->radio_input_element, 'render' ),
+			__('Link by Default', 'wordlift'),
+			array($this->radio_input_element, 'render'),
 			'wl_general_settings',
 			'wl_general_settings_section',
 			array(
 				'id'          => 'wl-link-by-default',
-				'name'        => 'wl_general_settings[' . Wordlift_Configuration_Service::LINK_BY_DEFAULT . ']',
+				'name'        => 'wl_general_settings['.Wordlift_Configuration_Service::LINK_BY_DEFAULT.']',
 				'value'       => Wordlift_Configuration_Service::get_instance()->is_link_by_default() ? 'yes' : 'no',
-				'description' => __( 'Whether to link entities by default or not. This setting applies to all the entities.', 'wordlift' ),
+				'description' => __('Whether to link entities by default or not. This setting applies to all the entities.', 'wordlift'),
 			)
 		);
 
 		// Add the `diagnostic data` field.
 		add_settings_field(
 			'wl-send-diagnostic',
-			__( 'Send Diagnostic Data', 'wordlift' ),
-			array( $this->radio_input_element, 'render' ),
+			__('Send Diagnostic Data', 'wordlift'),
+			array($this->radio_input_element, 'render'),
 			'wl_general_settings',
 			'wl_general_settings_section',
 			array(
 				'id'          => 'wl-send-diagnostic',
-				'name'        => 'wl_general_settings[' . Wordlift_Configuration_Service::SEND_DIAGNOSTIC . ']',
+				'name'        => 'wl_general_settings['.Wordlift_Configuration_Service::SEND_DIAGNOSTIC.']',
 				'value'       => 'yes' === Wordlift_Configuration_Service::get_instance()->get_diagnostic_preferences() ? 'yes' : 'no',
-				'description' => __( 'Whether to send diagnostic data or not.', 'wordlift' ),
+				'description' => __('Whether to send diagnostic data or not.', 'wordlift'),
 			)
 		);
 
@@ -346,10 +346,10 @@ 
 	 * @return array The updated array.
 	 * @since 3.17.0
 	 */
-	public function entity_path_input_element_params( $args ) {
+	public function entity_path_input_element_params($args) {
 
 		// Bail out if it's not the `wl-entity-base-path`).
-		if ( 'wl-entity-base-path' !== $args['id'] ) {
+		if ('wl-entity-base-path' !== $args['id']) {
 			return $args;
 		}
 
@@ -371,22 +371,22 @@ 
 	 * @return array The sanitized input array.
 	 * @since 3.11.0
 	 */
-	public function sanitize_callback( $input ) {
+	public function sanitize_callback($input) {
 		// No nonce verification since this callback is handled by settings api.
 		// Check whether a publisher name has been set.
 		// phpcs:ignore Standard.Category.SniffName.ErrorCode
-		if ( isset( $_POST['wl_publisher'] ) && ! empty( $_POST['wl_publisher']['name'] ) ) { //phpcs:ignore WordPress.Security.NonceVerification.Missing
-			$name = isset( $_POST['wl_publisher']['name'] ) ? sanitize_text_field( wp_unslash( (string) $_POST['wl_publisher']['name'] ) ) : ''; //phpcs:ignore WordPress.Security.NonceVerification.Missing
-			$type = isset( $_POST['wl_publisher']['type'] ) ? sanitize_text_field( wp_unslash( (string) $_POST['wl_publisher']['type'] ) ) : ''; //phpcs:ignore WordPress.Security.NonceVerification.Missing
+		if (isset($_POST['wl_publisher']) && ! empty($_POST['wl_publisher']['name'])) { //phpcs:ignore WordPress.Security.NonceVerification.Missing
+			$name = isset($_POST['wl_publisher']['name']) ? sanitize_text_field(wp_unslash((string) $_POST['wl_publisher']['name'])) : ''; //phpcs:ignore WordPress.Security.NonceVerification.Missing
+			$type = isset($_POST['wl_publisher']['type']) ? sanitize_text_field(wp_unslash((string) $_POST['wl_publisher']['type'])) : ''; //phpcs:ignore WordPress.Security.NonceVerification.Missing
 			// phpcs:ignore Standard.Category.SniffName.ErrorCode
-			$thumbnail_id = isset( $_POST['wl_publisher']['thumbnail_id'] ) ? sanitize_text_field( wp_unslash( $_POST['wl_publisher']['thumbnail_id'] ) ) : null; //phpcs:ignore WordPress.Security.NonceVerification.Missing
+			$thumbnail_id = isset($_POST['wl_publisher']['thumbnail_id']) ? sanitize_text_field(wp_unslash($_POST['wl_publisher']['thumbnail_id'])) : null; //phpcs:ignore WordPress.Security.NonceVerification.Missing
 
 			// Set the type URI, either http://schema.org/Person or http://schema.org/Organization.
-			$type_uri = sprintf( 'http://schema.org/%s', 'organization' === $type ? 'Organization' : 'Person' );
+			$type_uri = sprintf('http://schema.org/%s', 'organization' === $type ? 'Organization' : 'Person');
 
 			// Create an entity for the publisher and assign it to the input
 			// parameter which WordPress automatically saves into the settings.
-			$input['publisher_id'] = $this->entity_service->create( $name, $type_uri, $thumbnail_id, 'publish' );
+			$input['publisher_id'] = $this->entity_service->create($name, $type_uri, $thumbnail_id, 'publish');
 		}
 
 		return $input;

src/admin/class-wordlift-admin-entity-taxonomy-list-page.php

Indentation +103 added lines, -103 removed lines

@@ -13,7 +13,7 @@ 
  */
 
 if ( ! defined( 'ABSPATH' ) ) {
-	exit;
+    exit;
 }
 
 /**
@@ -28,107 +28,107 @@ 
  */
 class Wordlift_Admin_Entity_Taxonomy_List_Page {
 
-	/**
-	 * Hook to `wl_entity_type_row_actions` to add an "action" link to Thread
-	 * SEO related settings for the term.
-	 *
-	 * @see   https://developer.wordpress.org/reference/hooks/taxonomy_row_actions/
-	 *
-	 * @since 3.11.0
-	 *
-	 * @param array  $actions An array of action links to be displayed. Default
-	 *                        'Edit', 'Quick Edit', 'Delete', and 'View'.
-	 * @param object $term    Term object.
-	 *
-	 * @return array  $actions An array of action links to be displayed. Default
-	 *                        'Edit', 'Quick Edit', 'Delete', and 'View'.
-	 */
-	public function wl_entity_type_row_actions( $actions, $term ) {
-
-		$url               = admin_url( "admin.php?page=wl_entity_type_settings&tag_ID=$term->term_id" );
-		$actions['wl-seo'] = '<a href="' . esc_url( $url ) . '">' . __( 'SEO Settings', 'wordlift' ) . '</a>';
-
-		// Hacks for pre 4.7 compatibility:
-		// * remove the quick edit.
-		unset( $actions['inline hide-if-no-js'] );
-		// * remove the edit link.
-		unset( $actions['edit'] );
-
-		return $actions;
-	}
-
-	/**
-	 * Override the capabilities related to managing the entity type terms
-	 * for multisite super admin to prevent it from manipulating it in any
-	 * way.
-	 *
-	 * @since 3.12.0
-	 *
-	 * @param array  $caps    The user's current capabilities.
-	 * @param string $cap     Capability name.
-	 *
-	 * @return array    Array containing the do_not_allow capability for super admin
-	 *                  when editing and deleting entity type terms capabilities
-	 *                    are being "approved"
-	 */
-	public function restrict_super_admin( $caps, $cap ) {
-
-		switch ( $cap ) {
-			case 'wl_entity_type_edit_term':
-			case 'wl_entity_type_delete_term':
-				$caps[] = 'do_not_allow';
-		}
-
-		return $caps;
-	}
-
-	/**
-	 * Override the capabilities related to managing the entity type terms
-	 * required for WordPress < 4.7 to get access to the admin page.
-	 *
-	 * Before 4.7, WordPress checks whether the user has permission to access
-	 * the `edit-tags.php` page. So we need to provide temporary the permission
-	 * otherwise the user will get an `access not allowed` when trying to access
-	 * the page.
-	 *
-	 * @see   https://github.com/insideout10/wordlift-plugin/issues/512
-	 *
-	 * @since 3.12.0
-	 *
-	 * @param array  $caps    The user's current capabilities.
-	 * @param string $cap     Capability name.
-	 *
-	 * @return array    Array containing the manage_options capability
-	 */
-	public function enable_admin_access_pre_47( $caps, $cap ) {
-
-		// Bail out if we're not dealing with the `wl_entity_type_edit_term`
-		// capability.
-		if ( 'wl_entity_type_edit_term' !== $cap || ! isset( $_SERVER['REQUEST_URI'] ) ) {
-			return $caps;
-		}
-
-		$path = wp_parse_url( esc_url_raw( wp_unslash( $_SERVER['REQUEST_URI'], PHP_URL_PATH ) ) );
-
-		// Bail out if the `path` cannot be determined.
-		if ( null === $path ) {
-			return $caps;
-		}
-
-		// Bail out if a specific term is being handled.
-		if ( isset( $_REQUEST['action'] ) ) { //phpcs:ignore WordPress.Security.NonceVerification.Recommended
-			return $caps;
-		}
-
-		// Split the path and check if the page is the `edit-tags.php`. We can't
-		// use WP's own function because this is called very early.
-		$components = explode( '/', $path );
-		if ( 'edit-tags.php' !== $components[ count( $components ) - 1 ] ) {
-			return $caps;
-		}
-
-		// Give permission to the user.
-		return array( 'manage_options' );
-	}
+    /**
+     * Hook to `wl_entity_type_row_actions` to add an "action" link to Thread
+     * SEO related settings for the term.
+     *
+     * @see   https://developer.wordpress.org/reference/hooks/taxonomy_row_actions/
+     *
+     * @since 3.11.0
+     *
+     * @param array  $actions An array of action links to be displayed. Default
+     *                        'Edit', 'Quick Edit', 'Delete', and 'View'.
+     * @param object $term    Term object.
+     *
+     * @return array  $actions An array of action links to be displayed. Default
+     *                        'Edit', 'Quick Edit', 'Delete', and 'View'.
+     */
+    public function wl_entity_type_row_actions( $actions, $term ) {
+
+        $url               = admin_url( "admin.php?page=wl_entity_type_settings&tag_ID=$term->term_id" );
+        $actions['wl-seo'] = '<a href="' . esc_url( $url ) . '">' . __( 'SEO Settings', 'wordlift' ) . '</a>';
+
+        // Hacks for pre 4.7 compatibility:
+        // * remove the quick edit.
+        unset( $actions['inline hide-if-no-js'] );
+        // * remove the edit link.
+        unset( $actions['edit'] );
+
+        return $actions;
+    }
+
+    /**
+     * Override the capabilities related to managing the entity type terms
+     * for multisite super admin to prevent it from manipulating it in any
+     * way.
+     *
+     * @since 3.12.0
+     *
+     * @param array  $caps    The user's current capabilities.
+     * @param string $cap     Capability name.
+     *
+     * @return array    Array containing the do_not_allow capability for super admin
+     *                  when editing and deleting entity type terms capabilities
+     *                    are being "approved"
+     */
+    public function restrict_super_admin( $caps, $cap ) {
+
+        switch ( $cap ) {
+            case 'wl_entity_type_edit_term':
+            case 'wl_entity_type_delete_term':
+                $caps[] = 'do_not_allow';
+        }
+
+        return $caps;
+    }
+
+    /**
+     * Override the capabilities related to managing the entity type terms
+     * required for WordPress < 4.7 to get access to the admin page.
+     *
+     * Before 4.7, WordPress checks whether the user has permission to access
+     * the `edit-tags.php` page. So we need to provide temporary the permission
+     * otherwise the user will get an `access not allowed` when trying to access
+     * the page.
+     *
+     * @see   https://github.com/insideout10/wordlift-plugin/issues/512
+     *
+     * @since 3.12.0
+     *
+     * @param array  $caps    The user's current capabilities.
+     * @param string $cap     Capability name.
+     *
+     * @return array    Array containing the manage_options capability
+     */
+    public function enable_admin_access_pre_47( $caps, $cap ) {
+
+        // Bail out if we're not dealing with the `wl_entity_type_edit_term`
+        // capability.
+        if ( 'wl_entity_type_edit_term' !== $cap || ! isset( $_SERVER['REQUEST_URI'] ) ) {
+            return $caps;
+        }
+
+        $path = wp_parse_url( esc_url_raw( wp_unslash( $_SERVER['REQUEST_URI'], PHP_URL_PATH ) ) );
+
+        // Bail out if the `path` cannot be determined.
+        if ( null === $path ) {
+            return $caps;
+        }
+
+        // Bail out if a specific term is being handled.
+        if ( isset( $_REQUEST['action'] ) ) { //phpcs:ignore WordPress.Security.NonceVerification.Recommended
+            return $caps;
+        }
+
+        // Split the path and check if the page is the `edit-tags.php`. We can't
+        // use WP's own function because this is called very early.
+        $components = explode( '/', $path );
+        if ( 'edit-tags.php' !== $components[ count( $components ) - 1 ] ) {
+            return $caps;
+        }
+
+        // Give permission to the user.
+        return array( 'manage_options' );
+    }
 
 }

Spacing +16 added lines, -16 removed lines

@@ -12,7 +12,7 @@ 
  * @since      3.11.0
  */
 
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -43,16 +43,16 @@ 
 	 * @return array  $actions An array of action links to be displayed. Default
 	 *                        'Edit', 'Quick Edit', 'Delete', and 'View'.
 	 */
-	public function wl_entity_type_row_actions( $actions, $term ) {
+	public function wl_entity_type_row_actions($actions, $term) {
 
-		$url               = admin_url( "admin.php?page=wl_entity_type_settings&tag_ID=$term->term_id" );
-		$actions['wl-seo'] = '<a href="' . esc_url( $url ) . '">' . __( 'SEO Settings', 'wordlift' ) . '</a>';
+		$url               = admin_url("admin.php?page=wl_entity_type_settings&tag_ID=$term->term_id");
+		$actions['wl-seo'] = '<a href="'.esc_url($url).'">'.__('SEO Settings', 'wordlift').'</a>';
 
 		// Hacks for pre 4.7 compatibility:
 		// * remove the quick edit.
-		unset( $actions['inline hide-if-no-js'] );
+		unset($actions['inline hide-if-no-js']);
 		// * remove the edit link.
-		unset( $actions['edit'] );
+		unset($actions['edit']);
 
 		return $actions;
 	}
@@ -71,9 +71,9 @@ 
 	 *                  when editing and deleting entity type terms capabilities
 	 *                    are being "approved"
 	 */
-	public function restrict_super_admin( $caps, $cap ) {
+	public function restrict_super_admin($caps, $cap) {
 
-		switch ( $cap ) {
+		switch ($cap) {
 			case 'wl_entity_type_edit_term':
 			case 'wl_entity_type_delete_term':
 				$caps[] = 'do_not_allow';
@@ -100,35 +100,35 @@ 
 	 *
 	 * @return array    Array containing the manage_options capability
 	 */
-	public function enable_admin_access_pre_47( $caps, $cap ) {
+	public function enable_admin_access_pre_47($caps, $cap) {
 
 		// Bail out if we're not dealing with the `wl_entity_type_edit_term`
 		// capability.
-		if ( 'wl_entity_type_edit_term' !== $cap || ! isset( $_SERVER['REQUEST_URI'] ) ) {
+		if ('wl_entity_type_edit_term' !== $cap || ! isset($_SERVER['REQUEST_URI'])) {
 			return $caps;
 		}
 
-		$path = wp_parse_url( esc_url_raw( wp_unslash( $_SERVER['REQUEST_URI'], PHP_URL_PATH ) ) );
+		$path = wp_parse_url(esc_url_raw(wp_unslash($_SERVER['REQUEST_URI'], PHP_URL_PATH)));
 
 		// Bail out if the `path` cannot be determined.
-		if ( null === $path ) {
+		if (null === $path) {
 			return $caps;
 		}
 
 		// Bail out if a specific term is being handled.
-		if ( isset( $_REQUEST['action'] ) ) { //phpcs:ignore WordPress.Security.NonceVerification.Recommended
+		if (isset($_REQUEST['action'])) { //phpcs:ignore WordPress.Security.NonceVerification.Recommended
 			return $caps;
 		}
 
 		// Split the path and check if the page is the `edit-tags.php`. We can't
 		// use WP's own function because this is called very early.
-		$components = explode( '/', $path );
-		if ( 'edit-tags.php' !== $components[ count( $components ) - 1 ] ) {
+		$components = explode('/', $path);
+		if ('edit-tags.php' !== $components[count($components) - 1]) {
 			return $caps;
 		}
 
 		// Give permission to the user.
-		return array( 'manage_options' );
+		return array('manage_options');
 	}
 
 }

src/admin/class-wordlift-admin-entity-type-settings.php

Indentation +201 added lines, -201 removed lines

@@ -13,7 +13,7 @@ 
  */
 
 if ( ! defined( 'ABSPATH' ) ) {
-	exit;
+    exit;
 }
 
 /**
@@ -30,217 +30,217 @@ 
  */
 class Wordlift_Admin_Entity_Type_Settings {
 
-	/**
-	 * A {@link Wordlift_Log_Service} instance.
-	 *
-	 * @since  3.14.0
-	 * @access private
-	 * @var \Wordlift_Log_Service $log A {@link Wordlift_Log_Service} instance.
-	 */
-	private $log;
-
-	/**
-	 * Create a {@link Wordlift_Admin_Entity_Type_Settings} instance.
-	 *
-	 * @since 3.14.0
-	 */
-	public function __construct() {
-
-		$this->log = Wordlift_Log_Service::get_logger( 'Wordlift_Admin_Entity_Type_Settings' );
-
-	}
-
-	/**
-	 * Handle menu registration.
-	 *
-	 * The registration is required, although we do not want to actually to add
-	 * an item to the menu, in order to "whitelist" the access to the settings page in
-	 * the admin.
-	 *
-	 * @since 3.11.0
-	 */
-	public function admin_menu() {
-
-		/*
+    /**
+     * A {@link Wordlift_Log_Service} instance.
+     *
+     * @since  3.14.0
+     * @access private
+     * @var \Wordlift_Log_Service $log A {@link Wordlift_Log_Service} instance.
+     */
+    private $log;
+
+    /**
+     * Create a {@link Wordlift_Admin_Entity_Type_Settings} instance.
+     *
+     * @since 3.14.0
+     */
+    public function __construct() {
+
+        $this->log = Wordlift_Log_Service::get_logger( 'Wordlift_Admin_Entity_Type_Settings' );
+
+    }
+
+    /**
+     * Handle menu registration.
+     *
+     * The registration is required, although we do not want to actually to add
+     * an item to the menu, in order to "whitelist" the access to the settings page in
+     * the admin.
+     *
+     * @since 3.11.0
+     */
+    public function admin_menu() {
+
+        /*
 		 * Before anything else check if an settings form was submitted.
 		 * This has to be done before any output happens in order to be able to
 		 * display proper "die" error messages and redirect.
 		 */
-		if ( isset( $_GET['page'] ) && ( 'wl_entity_type_settings' === $_GET['page'] ) ) { //phpcs:ignore WordPress.Security.NonceVerification.Recommended
-
-			// Validate inputs. Do not return on invalid parameters or capabilities.
-			$this->validate_proper_term();
-
-			// If proper form submission, handle it and redirect back to the settings page.
-			if ( isset( $_POST['action'] ) && ( 'wl_edit_entity_type_term' === $_POST['action'] ) ) { //phpcs:ignore WordPress.Security.NonceVerification.Missing
-				$this->handle_form_submission();
-			}
-			/**
-			 * Filter: wl_feature__enable__notices.
-			 *
-			 * @param bool whether the notices needs to be enabled or not.
-			 *
-			 * @return bool
-			 * @since 3.27.6
-			 */
-			if ( apply_filters( 'wl_feature__enable__notices', true ) ) {
-				// Register admin notices handler.
-				add_action( 'admin_notices', array( $this, 'admin_notice' ) );
-			}
-		}
-
-		/*
+        if ( isset( $_GET['page'] ) && ( 'wl_entity_type_settings' === $_GET['page'] ) ) { //phpcs:ignore WordPress.Security.NonceVerification.Recommended
+
+            // Validate inputs. Do not return on invalid parameters or capabilities.
+            $this->validate_proper_term();
+
+            // If proper form submission, handle it and redirect back to the settings page.
+            if ( isset( $_POST['action'] ) && ( 'wl_edit_entity_type_term' === $_POST['action'] ) ) { //phpcs:ignore WordPress.Security.NonceVerification.Missing
+                $this->handle_form_submission();
+            }
+            /**
+             * Filter: wl_feature__enable__notices.
+             *
+             * @param bool whether the notices needs to be enabled or not.
+             *
+             * @return bool
+             * @since 3.27.6
+             */
+            if ( apply_filters( 'wl_feature__enable__notices', true ) ) {
+                // Register admin notices handler.
+                add_action( 'admin_notices', array( $this, 'admin_notice' ) );
+            }
+        }
+
+        /*
 		 * Use a null parent slug to prevent the menu from actually appearing
 		 * in the admin menu.
 		 */
-		// @todo: use the new {@link Wordlift_Admin_Page}.
-		add_submenu_page(
-			null,
-			__( 'Edit Entity term', 'wordlift' ),
-			__( 'Edit Entity term', 'wordlift' ),
-			'manage_options',
-			'wl_entity_type_settings',
-			array( $this, 'render' )
-		);
-	}
-
-	/**
-	 * Output admin notices if needed, based on the message url parameter.
-	 * A value of 1 indicates that a successful save was done.
-	 *
-	 * @since 3.11.0
-	 */
-	public function admin_notice() {
-		if ( isset( $_GET['message'] ) && ( '1' === $_GET['message'] ) ) { //phpcs:ignore WordPress.Security.NonceVerification.Recommended
-			?>
+        // @todo: use the new {@link Wordlift_Admin_Page}.
+        add_submenu_page(
+            null,
+            __( 'Edit Entity term', 'wordlift' ),
+            __( 'Edit Entity term', 'wordlift' ),
+            'manage_options',
+            'wl_entity_type_settings',
+            array( $this, 'render' )
+        );
+    }
+
+    /**
+     * Output admin notices if needed, based on the message url parameter.
+     * A value of 1 indicates that a successful save was done.
+     *
+     * @since 3.11.0
+     */
+    public function admin_notice() {
+        if ( isset( $_GET['message'] ) && ( '1' === $_GET['message'] ) ) { //phpcs:ignore WordPress.Security.NonceVerification.Recommended
+            ?>
 			<div class="notice notice-success is-dismissible">
 				<p><?php esc_html_e( 'Settings saved', 'wordlift' ); ?></p>
 			</div>
 			<?php
-		}
-	}
-
-	/**
-	 * Validate the existence of the entity type indicated by the tag_ID url
-	 * parameter before doing any processing. Done before any output to mimic
-	 * the way WordPress handles same situation with "normal" term editing screens.
-	 *
-	 * @since 3.11.0
-	 */
-	public function validate_proper_term() {
-
-		// Validate capabilities.
-		if ( ! current_user_can( 'manage_options' ) ) {
-			wp_die(
-				'<h1>' . esc_html__( 'Cheatin&#8217; uh?', 'default' ) . '</h1>' .
-				'<p>' . esc_html__( 'Sorry, you are not allowed to edit this item.', 'default' ) . '</p>',
-				403
-			);
-		}
-
-		// Get the term id and the actual term.
-		$term_id = isset( $_REQUEST['tag_ID'] ) ? (int) $_REQUEST['tag_ID'] : 0; //phpcs:ignore WordPress.Security.NonceVerification.Recommended
-
-		if ( ! term_exists( $term_id, Wordlift_Entity_Type_Taxonomy_Service::TAXONOMY_NAME ) ) {
-			wp_die( esc_html__( 'You attempted to edit an entity type term that doesn&#8217;t exist.', 'wordlift' ) );
-		}
-
-	}
-
-	/**
-	 * Handle the form submission of the settings form. On successful
-	 * handling redirect tp the setting edit page.
-	 *
-	 * @since 3.11.0
-	 */
-	public function handle_form_submission() {
-
-		$term_id = isset( $_POST['tag_ID'] ) ? (int) $_POST['tag_ID'] : 0;
-
-		// Check the nonce.
-		check_admin_referer( 'update-entity_type_term_' . $term_id );
-
-		$term = get_term( $term_id, 'wl_entity_type' );
-
-		$title       = isset( $_POST['title'] ) ? sanitize_text_field( wp_unslash( (string) $_POST['title'] ) ) : '';
-		$description = isset( $_POST['description'] ) ? sanitize_text_field( wp_unslash( (string) $_POST['description'] ) ) : '';
-		$this->set_setting(
-			$term_id,
-			trim( wp_unslash( $title ) ),
-			wp_unslash( $description )
-		);
-
-		// Redirect back to the term settings page and indicate a save was done.
-		$url = admin_url( "admin.php?page=wl_entity_type_settings&tag_ID=$term->term_id&message=1" );
-
-		wp_safe_redirect( $url );
-		exit();
-
-	}
-
-	/**
-	 * Render the settings page for the term.
-	 *
-	 * Access and parameter validity is assumed to be done earlier.
-	 *
-	 * @since 3.11.0
-	 */
-	public function render() {
-
-		// Set variables used by the partial
-		$term_id = isset( $_REQUEST['tag_ID'] ) ? absint( $_REQUEST['tag_ID'] ) : 0; //phpcs:ignore WordPress.Security.NonceVerification.Recommended
+        }
+    }
+
+    /**
+     * Validate the existence of the entity type indicated by the tag_ID url
+     * parameter before doing any processing. Done before any output to mimic
+     * the way WordPress handles same situation with "normal" term editing screens.
+     *
+     * @since 3.11.0
+     */
+    public function validate_proper_term() {
+
+        // Validate capabilities.
+        if ( ! current_user_can( 'manage_options' ) ) {
+            wp_die(
+                '<h1>' . esc_html__( 'Cheatin&#8217; uh?', 'default' ) . '</h1>' .
+                '<p>' . esc_html__( 'Sorry, you are not allowed to edit this item.', 'default' ) . '</p>',
+                403
+            );
+        }
+
+        // Get the term id and the actual term.
+        $term_id = isset( $_REQUEST['tag_ID'] ) ? (int) $_REQUEST['tag_ID'] : 0; //phpcs:ignore WordPress.Security.NonceVerification.Recommended
+
+        if ( ! term_exists( $term_id, Wordlift_Entity_Type_Taxonomy_Service::TAXONOMY_NAME ) ) {
+            wp_die( esc_html__( 'You attempted to edit an entity type term that doesn&#8217;t exist.', 'wordlift' ) );
+        }
+
+    }
+
+    /**
+     * Handle the form submission of the settings form. On successful
+     * handling redirect tp the setting edit page.
+     *
+     * @since 3.11.0
+     */
+    public function handle_form_submission() {
+
+        $term_id = isset( $_POST['tag_ID'] ) ? (int) $_POST['tag_ID'] : 0;
+
+        // Check the nonce.
+        check_admin_referer( 'update-entity_type_term_' . $term_id );
+
+        $term = get_term( $term_id, 'wl_entity_type' );
+
+        $title       = isset( $_POST['title'] ) ? sanitize_text_field( wp_unslash( (string) $_POST['title'] ) ) : '';
+        $description = isset( $_POST['description'] ) ? sanitize_text_field( wp_unslash( (string) $_POST['description'] ) ) : '';
+        $this->set_setting(
+            $term_id,
+            trim( wp_unslash( $title ) ),
+            wp_unslash( $description )
+        );
+
+        // Redirect back to the term settings page and indicate a save was done.
+        $url = admin_url( "admin.php?page=wl_entity_type_settings&tag_ID=$term->term_id&message=1" );
+
+        wp_safe_redirect( $url );
+        exit();
+
+    }
+
+    /**
+     * Render the settings page for the term.
+     *
+     * Access and parameter validity is assumed to be done earlier.
+     *
+     * @since 3.11.0
+     */
+    public function render() {
+
+        // Set variables used by the partial
+        $term_id = isset( $_REQUEST['tag_ID'] ) ? absint( $_REQUEST['tag_ID'] ) : 0; //phpcs:ignore WordPress.Security.NonceVerification.Recommended
         // phpcs:ignore VariableAnalysis.CodeAnalysis.VariableAnalysis.UnusedVariable
-		$settings = $this->get_setting( $term_id );
-
-		include plugin_dir_path( __DIR__ ) . 'admin/partials/wordlift-admin-entity-type-settings.php';
-
-	}
-
-	/**
-	 * Store the entity type term settings in the DB
-	 *
-	 * @param integer $term_id The ID of the entity type term
-	 * @param string  $title The override for the terms title.
-	 * @param string  $description The override for the terms description.
-	 *
-	 * @since 3.11.0
-	 */
-	public function set_setting( $term_id, $title, $description ) {
-
-		$settings             = get_option( 'wl_entity_type_settings', array() );
-		$settings[ $term_id ] = array(
-			'title'       => $title,
-			'description' => $description,
-		);
-		update_option( 'wl_entity_type_settings', $settings );
-
-	}
-
-	/**
-	 * Retrieve the entity type term settings from the DB
-	 *
-	 * @param integer $term_id The ID of the entity type term
-	 *
-	 * @return    null|array {
-	 *                null is returned when there are no settings otherwise
-	 *                an array is returned with following fields
-	 *
-	 * @type    string    title    The overriding title for the term
-	 * @type    string    description    The overriding description for the term
-	 *            }
-	 * @since 3.11.0
-	 */
-	public function get_setting( $term_id ) {
-
-		$settings = get_option( 'wl_entity_type_settings', array() );
-
-		if ( isset( $settings[ $term_id ] ) ) {
-			return $settings[ $term_id ];
-		}
-
-		$this->log->warn( "No settings found for term id $term_id." );
-
-		return null;
-	}
+        $settings = $this->get_setting( $term_id );
+
+        include plugin_dir_path( __DIR__ ) . 'admin/partials/wordlift-admin-entity-type-settings.php';
+
+    }
+
+    /**
+     * Store the entity type term settings in the DB
+     *
+     * @param integer $term_id The ID of the entity type term
+     * @param string  $title The override for the terms title.
+     * @param string  $description The override for the terms description.
+     *
+     * @since 3.11.0
+     */
+    public function set_setting( $term_id, $title, $description ) {
+
+        $settings             = get_option( 'wl_entity_type_settings', array() );
+        $settings[ $term_id ] = array(
+            'title'       => $title,
+            'description' => $description,
+        );
+        update_option( 'wl_entity_type_settings', $settings );
+
+    }
+
+    /**
+     * Retrieve the entity type term settings from the DB
+     *
+     * @param integer $term_id The ID of the entity type term
+     *
+     * @return    null|array {
+     *                null is returned when there are no settings otherwise
+     *                an array is returned with following fields
+     *
+     * @type    string    title    The overriding title for the term
+     * @type    string    description    The overriding description for the term
+     *            }
+     * @since 3.11.0
+     */
+    public function get_setting( $term_id ) {
+
+        $settings = get_option( 'wl_entity_type_settings', array() );
+
+        if ( isset( $settings[ $term_id ] ) ) {
+            return $settings[ $term_id ];
+        }
+
+        $this->log->warn( "No settings found for term id $term_id." );
+
+        return null;
+    }
 
 }

Spacing +38 added lines, -38 removed lines

@@ -12,7 +12,7 @@ 
  * @since      3.11.0
  */
 
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -46,7 +46,7 @@ 
 	 */
 	public function __construct() {
 
-		$this->log = Wordlift_Log_Service::get_logger( 'Wordlift_Admin_Entity_Type_Settings' );
+		$this->log = Wordlift_Log_Service::get_logger('Wordlift_Admin_Entity_Type_Settings');
 
 	}
 
@@ -66,13 +66,13 @@ 
 		 * This has to be done before any output happens in order to be able to
 		 * display proper "die" error messages and redirect.
 		 */
-		if ( isset( $_GET['page'] ) && ( 'wl_entity_type_settings' === $_GET['page'] ) ) { //phpcs:ignore WordPress.Security.NonceVerification.Recommended
+		if (isset($_GET['page']) && ('wl_entity_type_settings' === $_GET['page'])) { //phpcs:ignore WordPress.Security.NonceVerification.Recommended
 
 			// Validate inputs. Do not return on invalid parameters or capabilities.
 			$this->validate_proper_term();
 
 			// If proper form submission, handle it and redirect back to the settings page.
-			if ( isset( $_POST['action'] ) && ( 'wl_edit_entity_type_term' === $_POST['action'] ) ) { //phpcs:ignore WordPress.Security.NonceVerification.Missing
+			if (isset($_POST['action']) && ('wl_edit_entity_type_term' === $_POST['action'])) { //phpcs:ignore WordPress.Security.NonceVerification.Missing
 				$this->handle_form_submission();
 			}
 			/**
@@ -83,9 +83,9 @@ 
 			 * @return bool
 			 * @since 3.27.6
 			 */
-			if ( apply_filters( 'wl_feature__enable__notices', true ) ) {
+			if (apply_filters('wl_feature__enable__notices', true)) {
 				// Register admin notices handler.
-				add_action( 'admin_notices', array( $this, 'admin_notice' ) );
+				add_action('admin_notices', array($this, 'admin_notice'));
 			}
 		}
 
@@ -96,11 +96,11 @@ 
 		// @todo: use the new {@link Wordlift_Admin_Page}.
 		add_submenu_page(
 			null,
-			__( 'Edit Entity term', 'wordlift' ),
-			__( 'Edit Entity term', 'wordlift' ),
+			__('Edit Entity term', 'wordlift'),
+			__('Edit Entity term', 'wordlift'),
 			'manage_options',
 			'wl_entity_type_settings',
-			array( $this, 'render' )
+			array($this, 'render')
 		);
 	}
 
@@ -111,10 +111,10 @@ 
 	 * @since 3.11.0
 	 */
 	public function admin_notice() {
-		if ( isset( $_GET['message'] ) && ( '1' === $_GET['message'] ) ) { //phpcs:ignore WordPress.Security.NonceVerification.Recommended
+		if (isset($_GET['message']) && ('1' === $_GET['message'])) { //phpcs:ignore WordPress.Security.NonceVerification.Recommended
 			?>
 			<div class="notice notice-success is-dismissible">
-				<p><?php esc_html_e( 'Settings saved', 'wordlift' ); ?></p>
+				<p><?php esc_html_e('Settings saved', 'wordlift'); ?></p>
 			</div>
 			<?php
 		}
@@ -130,19 +130,19 @@ 
 	public function validate_proper_term() {
 
 		// Validate capabilities.
-		if ( ! current_user_can( 'manage_options' ) ) {
+		if ( ! current_user_can('manage_options')) {
 			wp_die(
-				'<h1>' . esc_html__( 'Cheatin&#8217; uh?', 'default' ) . '</h1>' .
-				'<p>' . esc_html__( 'Sorry, you are not allowed to edit this item.', 'default' ) . '</p>',
+				'<h1>'.esc_html__('Cheatin&#8217; uh?', 'default').'</h1>'.
+				'<p>'.esc_html__('Sorry, you are not allowed to edit this item.', 'default').'</p>',
 				403
 			);
 		}
 
 		// Get the term id and the actual term.
-		$term_id = isset( $_REQUEST['tag_ID'] ) ? (int) $_REQUEST['tag_ID'] : 0; //phpcs:ignore WordPress.Security.NonceVerification.Recommended
+		$term_id = isset($_REQUEST['tag_ID']) ? (int) $_REQUEST['tag_ID'] : 0; //phpcs:ignore WordPress.Security.NonceVerification.Recommended
 
-		if ( ! term_exists( $term_id, Wordlift_Entity_Type_Taxonomy_Service::TAXONOMY_NAME ) ) {
-			wp_die( esc_html__( 'You attempted to edit an entity type term that doesn&#8217;t exist.', 'wordlift' ) );
+		if ( ! term_exists($term_id, Wordlift_Entity_Type_Taxonomy_Service::TAXONOMY_NAME)) {
+			wp_die(esc_html__('You attempted to edit an entity type term that doesn&#8217;t exist.', 'wordlift'));
 		}
 
 	}
@@ -155,25 +155,25 @@ 
 	 */
 	public function handle_form_submission() {
 
-		$term_id = isset( $_POST['tag_ID'] ) ? (int) $_POST['tag_ID'] : 0;
+		$term_id = isset($_POST['tag_ID']) ? (int) $_POST['tag_ID'] : 0;
 
 		// Check the nonce.
-		check_admin_referer( 'update-entity_type_term_' . $term_id );
+		check_admin_referer('update-entity_type_term_'.$term_id);
 
-		$term = get_term( $term_id, 'wl_entity_type' );
+		$term = get_term($term_id, 'wl_entity_type');
 
-		$title       = isset( $_POST['title'] ) ? sanitize_text_field( wp_unslash( (string) $_POST['title'] ) ) : '';
-		$description = isset( $_POST['description'] ) ? sanitize_text_field( wp_unslash( (string) $_POST['description'] ) ) : '';
+		$title       = isset($_POST['title']) ? sanitize_text_field(wp_unslash((string) $_POST['title'])) : '';
+		$description = isset($_POST['description']) ? sanitize_text_field(wp_unslash((string) $_POST['description'])) : '';
 		$this->set_setting(
 			$term_id,
-			trim( wp_unslash( $title ) ),
-			wp_unslash( $description )
+			trim(wp_unslash($title)),
+			wp_unslash($description)
 		);
 
 		// Redirect back to the term settings page and indicate a save was done.
-		$url = admin_url( "admin.php?page=wl_entity_type_settings&tag_ID=$term->term_id&message=1" );
+		$url = admin_url("admin.php?page=wl_entity_type_settings&tag_ID=$term->term_id&message=1");
 
-		wp_safe_redirect( $url );
+		wp_safe_redirect($url);
 		exit();
 
 	}
@@ -188,11 +188,11 @@ 
 	public function render() {
 
 		// Set variables used by the partial
-		$term_id = isset( $_REQUEST['tag_ID'] ) ? absint( $_REQUEST['tag_ID'] ) : 0; //phpcs:ignore WordPress.Security.NonceVerification.Recommended
+		$term_id = isset($_REQUEST['tag_ID']) ? absint($_REQUEST['tag_ID']) : 0; //phpcs:ignore WordPress.Security.NonceVerification.Recommended
         // phpcs:ignore VariableAnalysis.CodeAnalysis.VariableAnalysis.UnusedVariable
-		$settings = $this->get_setting( $term_id );
+		$settings = $this->get_setting($term_id);
 
-		include plugin_dir_path( __DIR__ ) . 'admin/partials/wordlift-admin-entity-type-settings.php';
+		include plugin_dir_path(__DIR__).'admin/partials/wordlift-admin-entity-type-settings.php';
 
 	}
 
@@ -205,14 +205,14 @@ 
 	 *
 	 * @since 3.11.0
 	 */
-	public function set_setting( $term_id, $title, $description ) {
+	public function set_setting($term_id, $title, $description) {
 
-		$settings             = get_option( 'wl_entity_type_settings', array() );
-		$settings[ $term_id ] = array(
+		$settings             = get_option('wl_entity_type_settings', array());
+		$settings[$term_id] = array(
 			'title'       => $title,
 			'description' => $description,
 		);
-		update_option( 'wl_entity_type_settings', $settings );
+		update_option('wl_entity_type_settings', $settings);
 
 	}
 
@@ -230,15 +230,15 @@ 
 	 *            }
 	 * @since 3.11.0
 	 */
-	public function get_setting( $term_id ) {
+	public function get_setting($term_id) {
 
-		$settings = get_option( 'wl_entity_type_settings', array() );
+		$settings = get_option('wl_entity_type_settings', array());
 
-		if ( isset( $settings[ $term_id ] ) ) {
-			return $settings[ $term_id ];
+		if (isset($settings[$term_id])) {
+			return $settings[$term_id];
 		}
 
-		$this->log->warn( "No settings found for term id $term_id." );
+		$this->log->warn("No settings found for term id $term_id.");
 
 		return null;
 	}

src/admin/class-wordlift-admin-not-enriched-filter.php

Indentation +36 added lines, -36 removed lines

@@ -2,60 +2,60 @@
 
 class Wordlift_Admin_Not_Enriched_Filter {
 
-	const PARAMETER_NAME = 'wl_enriched';
+    const PARAMETER_NAME = 'wl_enriched';
 
-	public function __construct() {
+    public function __construct() {
 
-		// Hook to `views_edit-post`.
-		add_filter( 'views_edit-post', array( $this, 'view_edit' ) );
+        // Hook to `views_edit-post`.
+        add_filter( 'views_edit-post', array( $this, 'view_edit' ) );
 
-		// Add the `posts_where` filter if the filter is active.
-		if ( $this->is_filter_active() ) {
-			add_filter( 'posts_where', array( $this, 'posts_where' ) );
-		}
+        // Add the `posts_where` filter if the filter is active.
+        if ( $this->is_filter_active() ) {
+            add_filter( 'posts_where', array( $this, 'posts_where' ) );
+        }
 
-	}
+    }
 
-	public function posts_where( $where ) {
-		global $wpdb;
+    public function posts_where( $where ) {
+        global $wpdb;
 
-		return $where .
-			   " AND {$wpdb->posts}.ID NOT IN ( SELECT DISTINCT subject_id FROM {$wpdb->prefix}wl_relation_instances )";
-	}
+        return $where .
+                " AND {$wpdb->posts}.ID NOT IN ( SELECT DISTINCT subject_id FROM {$wpdb->prefix}wl_relation_instances )";
+    }
 
-	public function view_edit( $views ) {
-		global $wpdb;
+    public function view_edit( $views ) {
+        global $wpdb;
 
-		$url = add_query_arg(
-			array(
-				self::PARAMETER_NAME => 'no',
-				'post_type'          => 'post',
-			),
-			'edit.php'
-		);
+        $url = add_query_arg(
+            array(
+                self::PARAMETER_NAME => 'no',
+                'post_type'          => 'post',
+            ),
+            'edit.php'
+        );
 
-		$not_enriched_count = $wpdb->get_var(
-			"
+        $not_enriched_count = $wpdb->get_var(
+            "
 SELECT COUNT( 1 ) FROM $wpdb->posts p
  WHERE p.post_type = 'post'
    AND p.post_status <> 'trash' 
    AND p.post_status <> 'auto-draft'
    AND p.ID NOT IN ( SELECT DISTINCT subject_id FROM {$wpdb->prefix}wl_relation_instances )
 "
-		);
+        );
 
-		$link = '<a href="'
-				. esc_url( $url ) . '"'
-				. ( $this->is_filter_active() ? ' class="current" aria-current="page"' : '' )
-				. '>' . esc_html( __( 'Not enriched', 'wordlift' ) ) . '</a> (' . $not_enriched_count . ')';
+        $link = '<a href="'
+                . esc_url( $url ) . '"'
+                . ( $this->is_filter_active() ? ' class="current" aria-current="page"' : '' )
+                . '>' . esc_html( __( 'Not enriched', 'wordlift' ) ) . '</a> (' . $not_enriched_count . ')';
 
-		$views['wl_not_enriched'] = $link;
+        $views['wl_not_enriched'] = $link;
 
-		return $views;
-	}
+        return $views;
+    }
 
-	public function is_filter_active() {
-		return 'no' === filter_input( INPUT_GET, self::PARAMETER_NAME );
-	}
+    public function is_filter_active() {
+        return 'no' === filter_input( INPUT_GET, self::PARAMETER_NAME );
+    }
 
 }

Spacing +10 added lines, -10 removed lines

@@ -7,23 +7,23 @@ 
 	public function __construct() {
 
 		// Hook to `views_edit-post`.
-		add_filter( 'views_edit-post', array( $this, 'view_edit' ) );
+		add_filter('views_edit-post', array($this, 'view_edit'));
 
 		// Add the `posts_where` filter if the filter is active.
-		if ( $this->is_filter_active() ) {
-			add_filter( 'posts_where', array( $this, 'posts_where' ) );
+		if ($this->is_filter_active()) {
+			add_filter('posts_where', array($this, 'posts_where'));
 		}
 
 	}
 
-	public function posts_where( $where ) {
+	public function posts_where($where) {
 		global $wpdb;
 
-		return $where .
+		return $where.
 			   " AND {$wpdb->posts}.ID NOT IN ( SELECT DISTINCT subject_id FROM {$wpdb->prefix}wl_relation_instances )";
 	}
 
-	public function view_edit( $views ) {
+	public function view_edit($views) {
 		global $wpdb;
 
 		$url = add_query_arg(
@@ -45,9 +45,9 @@ 
 		);
 
 		$link = '<a href="'
-				. esc_url( $url ) . '"'
-				. ( $this->is_filter_active() ? ' class="current" aria-current="page"' : '' )
-				. '>' . esc_html( __( 'Not enriched', 'wordlift' ) ) . '</a> (' . $not_enriched_count . ')';
+				. esc_url($url).'"'
+				. ($this->is_filter_active() ? ' class="current" aria-current="page"' : '')
+				. '>'.esc_html(__('Not enriched', 'wordlift')).'</a> ('.$not_enriched_count.')';
 
 		$views['wl_not_enriched'] = $link;
 
@@ -55,7 +55,7 @@ 
 	}
 
 	public function is_filter_active() {
-		return 'no' === filter_input( INPUT_GET, self::PARAMETER_NAME );
+		return 'no' === filter_input(INPUT_GET, self::PARAMETER_NAME);
 	}
 
 }

src/admin/class-wordlift-admin-term-adapter.php

Indentation +151 added lines, -151 removed lines

@@ -16,82 +16,82 @@ 
  */
 class Wordlift_Admin_Term_Adapter {
 
-	/**
-	 * The meta key holding the entity id.
-	 *
-	 * @since 3.20.0
-	 */
-	const META_KEY = '_wl_entity_id';
-
-	/**
-	 * Create a Wordlift_Admin_Term_Adapter instance.
-	 *
-	 * @since 3.20.0
-	 */
-	public function __construct() {
-
-		add_action( 'registered_taxonomy', array( $this, 'add_action' ) );
-		add_action( 'edit_term', array( $this, 'edit_term' ), 10 );
-		$this->add_settings();
-
-	}
-
-	/**
-	 * Hook in to WordLift admin settings and add the term page specific
-	 * settings.
-	 *
-	 * @since 3.26.1
-	 */
-	public function add_settings() {
-		add_filter(
-			'wl_admin_settings',
-			function ( $params ) {
-				$params['show_local_entities'] = true;
-
-				return $params;
-			}
-		);
-	}
-
-	/**
-	 * Add the form fields to the entity edit screen.
-	 *
-	 * @param object $tag Current taxonomy term object.
-	 *
-	 * @since 3.20.0
-	 */
-	public function edit_form_fields( $tag ) {
-
-		// If disabled via filter, return;
-		// phpcs:ignore WordPress.NamingConventions.ValidHookName.UseUnderscores
-		if ( ! apply_filters( 'wl_feature__enable__term-entity', true ) ) {
-			return;
-		}
-
-		global $wp_version;
-
-		// Enqueue the JavaScript app.
-		if ( version_compare( $wp_version, '5.0', '>=' ) ) {
-			$term_asset = include plugin_dir_path( __DIR__ ) . 'js/dist/term.asset.php';
-			wp_enqueue_script( 'wl-term', plugin_dir_url( __DIR__ ) . 'js/dist/term.js', array_merge( array( 'wp-util' ), $term_asset['dependencies'] ), Wordlift::get_instance()->get_version(), true );
-		} else {
-			wp_enqueue_script( 'wl-term', plugin_dir_url( __DIR__ ) . 'js/dist/term.full.js', array( 'wp-util' ), Wordlift::get_instance()->get_version(), true );
-		}
-
-		wp_enqueue_style( 'wl-term', plugin_dir_url( __DIR__ ) . 'js/dist/term.css', array(), Wordlift::get_instance()->get_version() );
-
-		$values = get_term_meta( $tag->term_id, self::META_KEY );
-
-		/**
-		 * @since 3.31.3
-		 * @see https://github.com/insideout10/wordlift-plugin/issues/1446
-		 * This field should be hidden by default
-		 */
-		if ( ! $values ) {
-			return;
-		}
-
-		?>
+    /**
+     * The meta key holding the entity id.
+     *
+     * @since 3.20.0
+     */
+    const META_KEY = '_wl_entity_id';
+
+    /**
+     * Create a Wordlift_Admin_Term_Adapter instance.
+     *
+     * @since 3.20.0
+     */
+    public function __construct() {
+
+        add_action( 'registered_taxonomy', array( $this, 'add_action' ) );
+        add_action( 'edit_term', array( $this, 'edit_term' ), 10 );
+        $this->add_settings();
+
+    }
+
+    /**
+     * Hook in to WordLift admin settings and add the term page specific
+     * settings.
+     *
+     * @since 3.26.1
+     */
+    public function add_settings() {
+        add_filter(
+            'wl_admin_settings',
+            function ( $params ) {
+                $params['show_local_entities'] = true;
+
+                return $params;
+            }
+        );
+    }
+
+    /**
+     * Add the form fields to the entity edit screen.
+     *
+     * @param object $tag Current taxonomy term object.
+     *
+     * @since 3.20.0
+     */
+    public function edit_form_fields( $tag ) {
+
+        // If disabled via filter, return;
+        // phpcs:ignore WordPress.NamingConventions.ValidHookName.UseUnderscores
+        if ( ! apply_filters( 'wl_feature__enable__term-entity', true ) ) {
+            return;
+        }
+
+        global $wp_version;
+
+        // Enqueue the JavaScript app.
+        if ( version_compare( $wp_version, '5.0', '>=' ) ) {
+            $term_asset = include plugin_dir_path( __DIR__ ) . 'js/dist/term.asset.php';
+            wp_enqueue_script( 'wl-term', plugin_dir_url( __DIR__ ) . 'js/dist/term.js', array_merge( array( 'wp-util' ), $term_asset['dependencies'] ), Wordlift::get_instance()->get_version(), true );
+        } else {
+            wp_enqueue_script( 'wl-term', plugin_dir_url( __DIR__ ) . 'js/dist/term.full.js', array( 'wp-util' ), Wordlift::get_instance()->get_version(), true );
+        }
+
+        wp_enqueue_style( 'wl-term', plugin_dir_url( __DIR__ ) . 'js/dist/term.css', array(), Wordlift::get_instance()->get_version() );
+
+        $values = get_term_meta( $tag->term_id, self::META_KEY );
+
+        /**
+         * @since 3.31.3
+         * @see https://github.com/insideout10/wordlift-plugin/issues/1446
+         * This field should be hidden by default
+         */
+        if ( ! $values ) {
+            return;
+        }
+
+        ?>
         <tr class="form-field term-name-wrap">
             <th scope="row"><label
                         for="wl-entity-id"><?php echo esc_html_x( 'Entity', 'term entity', 'wordlift' ); ?></label></th>
@@ -105,80 +105,80 @@ 
         </tr>
 		<?php wp_nonce_field( 'wordlift_term_entity_edit', 'wordlift_term_entity_edit_nonce' ); ?>
 		<?php
-	}
-
-	/**
-	 * Bind the new fields to the edit term screen.
-	 *
-	 * @param string $taxonomy The taxonomy name.
-	 *
-	 * @since 3.20.0
-	 *
-	 */
-	public function add_action( $taxonomy ) {
-		/**
-		 * Filter wl_feature__enable__taxonomy_term_entity_mapping renamed to wl_feature__enable__term-entity.
-		 */
-
-		add_action( "{$taxonomy}_edit_form_fields", array( $this, 'edit_form_fields' ), 10 );
-
-	}
-
-	/**
-	 * Hook to the edit term to handle our own custom fields.
-	 *
-	 * @param int $term_id The term id.
-	 *
-	 * @since 3.20.0
-	 *
-	 */
-	public function edit_term( $term_id ) {
-
-		check_admin_referer( 'wordlift_term_entity_edit', 'wordlift_term_entity_edit_nonce' );
-
-		// Bail if the action isn't related to the term currently being edited.
-		if ( ! isset( $_POST['tag_ID'] ) || $term_id !== (int) $_POST['tag_ID'] ) {
-			return;
-		}
-
-		// Delete.
-		if ( ! isset( $_POST['wl_entity_id'] ) || ! is_array( $_POST['wl_entity_id'] ) || empty( $_POST['wl_entity_id'] ) ) {
-			delete_term_meta( $term_id, self::META_KEY );
-
-			return;
-		}
-
-		$entity_ids = array_map( 'esc_url_raw', wp_unslash( $_POST['wl_entity_id'] ) );
-		// Update.
-		//
-		// Only use mb_* functions when mbstring is available.
-		//
-		// See https://github.com/insideout10/wordlift-plugin/issues/693.
-		if ( extension_loaded( 'mbstring' ) ) {
-			mb_regex_encoding( 'UTF-8' );
-
-			$merged = array_reduce(
-				$entity_ids,
-				function ( $carry, $item ) {
-					return array_merge( $carry, mb_split( "\x{2063}", wp_unslash( $item ) ) );
-				},
-				array()
-			);
-		} else {
-			$merged = array_reduce(
-				$entity_ids,
-				function ( $carry, $item ) {
-					return array_merge( $carry, preg_split( "/\x{2063}/u", wp_unslash( $item ) ) );
-				},
-				array()
-			);
-		}
-
-		delete_term_meta( $term_id, self::META_KEY );
-		foreach ( array_unique( array_filter( $merged ) ) as $single ) {
-			add_term_meta( $term_id, self::META_KEY, $single );
-		}
-
-	}
+    }
+
+    /**
+     * Bind the new fields to the edit term screen.
+     *
+     * @param string $taxonomy The taxonomy name.
+     *
+     * @since 3.20.0
+     *
+     */
+    public function add_action( $taxonomy ) {
+        /**
+         * Filter wl_feature__enable__taxonomy_term_entity_mapping renamed to wl_feature__enable__term-entity.
+         */
+
+        add_action( "{$taxonomy}_edit_form_fields", array( $this, 'edit_form_fields' ), 10 );
+
+    }
+
+    /**
+     * Hook to the edit term to handle our own custom fields.
+     *
+     * @param int $term_id The term id.
+     *
+     * @since 3.20.0
+     *
+     */
+    public function edit_term( $term_id ) {
+
+        check_admin_referer( 'wordlift_term_entity_edit', 'wordlift_term_entity_edit_nonce' );
+
+        // Bail if the action isn't related to the term currently being edited.
+        if ( ! isset( $_POST['tag_ID'] ) || $term_id !== (int) $_POST['tag_ID'] ) {
+            return;
+        }
+
+        // Delete.
+        if ( ! isset( $_POST['wl_entity_id'] ) || ! is_array( $_POST['wl_entity_id'] ) || empty( $_POST['wl_entity_id'] ) ) {
+            delete_term_meta( $term_id, self::META_KEY );
+
+            return;
+        }
+
+        $entity_ids = array_map( 'esc_url_raw', wp_unslash( $_POST['wl_entity_id'] ) );
+        // Update.
+        //
+        // Only use mb_* functions when mbstring is available.
+        //
+        // See https://github.com/insideout10/wordlift-plugin/issues/693.
+        if ( extension_loaded( 'mbstring' ) ) {
+            mb_regex_encoding( 'UTF-8' );
+
+            $merged = array_reduce(
+                $entity_ids,
+                function ( $carry, $item ) {
+                    return array_merge( $carry, mb_split( "\x{2063}", wp_unslash( $item ) ) );
+                },
+                array()
+            );
+        } else {
+            $merged = array_reduce(
+                $entity_ids,
+                function ( $carry, $item ) {
+                    return array_merge( $carry, preg_split( "/\x{2063}/u", wp_unslash( $item ) ) );
+                },
+                array()
+            );
+        }
+
+        delete_term_meta( $term_id, self::META_KEY );
+        foreach ( array_unique( array_filter( $merged ) ) as $single ) {
+            add_term_meta( $term_id, self::META_KEY, $single );
+        }
+
+    }
 
 }

Spacing +34 added lines, -34 removed lines

@@ -30,8 +30,8 @@ 
 	 */
 	public function __construct() {
 
-		add_action( 'registered_taxonomy', array( $this, 'add_action' ) );
-		add_action( 'edit_term', array( $this, 'edit_term' ), 10 );
+		add_action('registered_taxonomy', array($this, 'add_action'));
+		add_action('edit_term', array($this, 'edit_term'), 10);
 		$this->add_settings();
 
 	}
@@ -45,7 +45,7 @@ 
 	public function add_settings() {
 		add_filter(
 			'wl_admin_settings',
-			function ( $params ) {
+			function($params) {
 				$params['show_local_entities'] = true;
 
 				return $params;
@@ -60,50 +60,50 @@ 
 	 *
 	 * @since 3.20.0
 	 */
-	public function edit_form_fields( $tag ) {
+	public function edit_form_fields($tag) {
 
 		// If disabled via filter, return;
 		// phpcs:ignore WordPress.NamingConventions.ValidHookName.UseUnderscores
-		if ( ! apply_filters( 'wl_feature__enable__term-entity', true ) ) {
+		if ( ! apply_filters('wl_feature__enable__term-entity', true)) {
 			return;
 		}
 
 		global $wp_version;
 
 		// Enqueue the JavaScript app.
-		if ( version_compare( $wp_version, '5.0', '>=' ) ) {
-			$term_asset = include plugin_dir_path( __DIR__ ) . 'js/dist/term.asset.php';
-			wp_enqueue_script( 'wl-term', plugin_dir_url( __DIR__ ) . 'js/dist/term.js', array_merge( array( 'wp-util' ), $term_asset['dependencies'] ), Wordlift::get_instance()->get_version(), true );
+		if (version_compare($wp_version, '5.0', '>=')) {
+			$term_asset = include plugin_dir_path(__DIR__).'js/dist/term.asset.php';
+			wp_enqueue_script('wl-term', plugin_dir_url(__DIR__).'js/dist/term.js', array_merge(array('wp-util'), $term_asset['dependencies']), Wordlift::get_instance()->get_version(), true);
 		} else {
-			wp_enqueue_script( 'wl-term', plugin_dir_url( __DIR__ ) . 'js/dist/term.full.js', array( 'wp-util' ), Wordlift::get_instance()->get_version(), true );
+			wp_enqueue_script('wl-term', plugin_dir_url(__DIR__).'js/dist/term.full.js', array('wp-util'), Wordlift::get_instance()->get_version(), true);
 		}
 
-		wp_enqueue_style( 'wl-term', plugin_dir_url( __DIR__ ) . 'js/dist/term.css', array(), Wordlift::get_instance()->get_version() );
+		wp_enqueue_style('wl-term', plugin_dir_url(__DIR__).'js/dist/term.css', array(), Wordlift::get_instance()->get_version());
 
-		$values = get_term_meta( $tag->term_id, self::META_KEY );
+		$values = get_term_meta($tag->term_id, self::META_KEY);
 
 		/**
 		 * @since 3.31.3
 		 * @see https://github.com/insideout10/wordlift-plugin/issues/1446
 		 * This field should be hidden by default
 		 */
-		if ( ! $values ) {
+		if ( ! $values) {
 			return;
 		}
 
 		?>
         <tr class="form-field term-name-wrap">
             <th scope="row"><label
-                        for="wl-entity-id"><?php echo esc_html_x( 'Entity', 'term entity', 'wordlift' ); ?></label></th>
+                        for="wl-entity-id"><?php echo esc_html_x('Entity', 'term entity', 'wordlift'); ?></label></th>
             <td>
-				<?php foreach ( $values as $value ) { ?>
-                    <input type="text" name="wl_entity_id[]" value="<?php echo esc_attr( $value ); ?>"/>
+				<?php foreach ($values as $value) { ?>
+                    <input type="text" name="wl_entity_id[]" value="<?php echo esc_attr($value); ?>"/>
 				<?php } ?>
                 <div id="wl-term-entity-id"></div>
-                <p class="description"><?php esc_html_e( 'The entity bound to the term.', 'wordlift' ); ?></p>
+                <p class="description"><?php esc_html_e('The entity bound to the term.', 'wordlift'); ?></p>
             </td>
         </tr>
-		<?php wp_nonce_field( 'wordlift_term_entity_edit', 'wordlift_term_entity_edit_nonce' ); ?>
+		<?php wp_nonce_field('wordlift_term_entity_edit', 'wordlift_term_entity_edit_nonce'); ?>
 		<?php
 	}
 
@@ -115,12 +115,12 @@ 
 	 * @since 3.20.0
 	 *
 	 */
-	public function add_action( $taxonomy ) {
+	public function add_action($taxonomy) {
 		/**
 		 * Filter wl_feature__enable__taxonomy_term_entity_mapping renamed to wl_feature__enable__term-entity.
 		 */
 
-		add_action( "{$taxonomy}_edit_form_fields", array( $this, 'edit_form_fields' ), 10 );
+		add_action("{$taxonomy}_edit_form_fields", array($this, 'edit_form_fields'), 10);
 
 	}
 
@@ -132,51 +132,51 @@ 
 	 * @since 3.20.0
 	 *
 	 */
-	public function edit_term( $term_id ) {
+	public function edit_term($term_id) {
 
-		check_admin_referer( 'wordlift_term_entity_edit', 'wordlift_term_entity_edit_nonce' );
+		check_admin_referer('wordlift_term_entity_edit', 'wordlift_term_entity_edit_nonce');
 
 		// Bail if the action isn't related to the term currently being edited.
-		if ( ! isset( $_POST['tag_ID'] ) || $term_id !== (int) $_POST['tag_ID'] ) {
+		if ( ! isset($_POST['tag_ID']) || $term_id !== (int) $_POST['tag_ID']) {
 			return;
 		}
 
 		// Delete.
-		if ( ! isset( $_POST['wl_entity_id'] ) || ! is_array( $_POST['wl_entity_id'] ) || empty( $_POST['wl_entity_id'] ) ) {
-			delete_term_meta( $term_id, self::META_KEY );
+		if ( ! isset($_POST['wl_entity_id']) || ! is_array($_POST['wl_entity_id']) || empty($_POST['wl_entity_id'])) {
+			delete_term_meta($term_id, self::META_KEY);
 
 			return;
 		}
 
-		$entity_ids = array_map( 'esc_url_raw', wp_unslash( $_POST['wl_entity_id'] ) );
+		$entity_ids = array_map('esc_url_raw', wp_unslash($_POST['wl_entity_id']));
 		// Update.
 		//
 		// Only use mb_* functions when mbstring is available.
 		//
 		// See https://github.com/insideout10/wordlift-plugin/issues/693.
-		if ( extension_loaded( 'mbstring' ) ) {
-			mb_regex_encoding( 'UTF-8' );
+		if (extension_loaded('mbstring')) {
+			mb_regex_encoding('UTF-8');
 
 			$merged = array_reduce(
 				$entity_ids,
-				function ( $carry, $item ) {
-					return array_merge( $carry, mb_split( "\x{2063}", wp_unslash( $item ) ) );
+				function($carry, $item) {
+					return array_merge($carry, mb_split("\x{2063}", wp_unslash($item)));
 				},
 				array()
 			);
 		} else {
 			$merged = array_reduce(
 				$entity_ids,
-				function ( $carry, $item ) {
-					return array_merge( $carry, preg_split( "/\x{2063}/u", wp_unslash( $item ) ) );
+				function($carry, $item) {
+					return array_merge($carry, preg_split("/\x{2063}/u", wp_unslash($item)));
 				},
 				array()
 			);
 		}
 
-		delete_term_meta( $term_id, self::META_KEY );
-		foreach ( array_unique( array_filter( $merged ) ) as $single ) {
-			add_term_meta( $term_id, self::META_KEY, $single );
+		delete_term_meta($term_id, self::META_KEY);
+		foreach (array_unique(array_filter($merged)) as $single) {
+			add_term_meta($term_id, self::META_KEY, $single);
 		}
 
 	}

src/admin/wordlift-admin-ajax-related-posts.php

Indentation +65 added lines, -65 removed lines

@@ -17,71 +17,71 @@
  */
 function wordlift_ajax_related_posts( $http_raw_data = null ) {
 
-	// Extract filtering conditions
-	if ( ! isset( $_GET['post_id'] ) || ! is_numeric( $_GET['post_id'] ) ) { //phpcs:ignore WordPress.Security.NonceVerification.Recommended
-		wp_die( 'Post id missing or invalid!' );
-
-		return;
-	}
-
-	// Get the current post
-	$post_id = (int) $_GET['post_id']; //phpcs:ignore WordPress.Security.NonceVerification.Recommended
-	$post    = get_post( $post_id );
-
-	Wordlift_Log_Service::get_logger( 'wordlift_ajax_related_posts' )->trace( "Going to find posts related to current with post id: $post_id ..." );
-
-	// Extract filtering conditions
-	$filtering_entity_uris = empty( $http_raw_data ) ? file_get_contents( 'php://input' ) : $http_raw_data;
-	$filtering_entity_uris = json_decode( $filtering_entity_uris );
-
-	$content_service      = Wordpress_Content_Service::get_instance();
-	$filtering_entity_ids = array_filter(
-		array_map(
-			function ( $uri ) use ( $content_service ) {
-				$content = $content_service->get_by_entity_id( $uri );
-				if ( isset( $content ) && is_a( $content->get_bag(), '\WP_Post' ) ) {
-					return $content->get_bag()->ID;
-				} else {
-					return null;
-				}
-			},
-			$filtering_entity_uris
-		)
-	);
-
-	$related_posts = array();
-
-	// If the current post is an entity, related posts to the current entity are
-	// returned.
-	$entity_service = Wordlift_Entity_Service::get_instance();
-
-	if ( $entity_service->is_entity( $post->ID ) ) {
-		$filtering_entity_ids = array( $post_id );
-	}
-
-	if ( ! empty( $filtering_entity_ids ) ) {
-
-		$related_posts = Wordlift_Relation_Service::get_instance()
-		                                          ->get_article_subjects( $filtering_entity_ids, '*', null, 'publish', array( $post_id ), 5 );
-
-		foreach ( $related_posts as $post_obj ) {
-
-			/**
-			 * Use the thumbnail.
-			 *
-			 * @see https://github.com/insideout10/wordlift-plugin/issues/825 related issue.
-			 * @see https://github.com/insideout10/wordlift-plugin/issues/837
-			 *
-			 * @since 3.19.3 We're using the medium size image.
-			 */
-			$thumbnail           = get_the_post_thumbnail_url( $post_obj, 'medium' );
-			$post_obj->thumbnail = ( $thumbnail ) ? $thumbnail : WL_DEFAULT_THUMBNAIL_PATH;
-			$post_obj->link      = get_edit_post_link( $post_obj->ID, 'none' );
-			$post_obj->permalink = get_post_permalink( $post_obj->ID );
-		}
-	}
-
-	wl_core_send_json( $related_posts );
+    // Extract filtering conditions
+    if ( ! isset( $_GET['post_id'] ) || ! is_numeric( $_GET['post_id'] ) ) { //phpcs:ignore WordPress.Security.NonceVerification.Recommended
+        wp_die( 'Post id missing or invalid!' );
+
+        return;
+    }
+
+    // Get the current post
+    $post_id = (int) $_GET['post_id']; //phpcs:ignore WordPress.Security.NonceVerification.Recommended
+    $post    = get_post( $post_id );
+
+    Wordlift_Log_Service::get_logger( 'wordlift_ajax_related_posts' )->trace( "Going to find posts related to current with post id: $post_id ..." );
+
+    // Extract filtering conditions
+    $filtering_entity_uris = empty( $http_raw_data ) ? file_get_contents( 'php://input' ) : $http_raw_data;
+    $filtering_entity_uris = json_decode( $filtering_entity_uris );
+
+    $content_service      = Wordpress_Content_Service::get_instance();
+    $filtering_entity_ids = array_filter(
+        array_map(
+            function ( $uri ) use ( $content_service ) {
+                $content = $content_service->get_by_entity_id( $uri );
+                if ( isset( $content ) && is_a( $content->get_bag(), '\WP_Post' ) ) {
+                    return $content->get_bag()->ID;
+                } else {
+                    return null;
+                }
+            },
+            $filtering_entity_uris
+        )
+    );
+
+    $related_posts = array();
+
+    // If the current post is an entity, related posts to the current entity are
+    // returned.
+    $entity_service = Wordlift_Entity_Service::get_instance();
+
+    if ( $entity_service->is_entity( $post->ID ) ) {
+        $filtering_entity_ids = array( $post_id );
+    }
+
+    if ( ! empty( $filtering_entity_ids ) ) {
+
+        $related_posts = Wordlift_Relation_Service::get_instance()
+                                                    ->get_article_subjects( $filtering_entity_ids, '*', null, 'publish', array( $post_id ), 5 );
+
+        foreach ( $related_posts as $post_obj ) {
+
+            /**
+             * Use the thumbnail.
+             *
+             * @see https://github.com/insideout10/wordlift-plugin/issues/825 related issue.
+             * @see https://github.com/insideout10/wordlift-plugin/issues/837
+             *
+             * @since 3.19.3 We're using the medium size image.
+             */
+            $thumbnail           = get_the_post_thumbnail_url( $post_obj, 'medium' );
+            $post_obj->thumbnail = ( $thumbnail ) ? $thumbnail : WL_DEFAULT_THUMBNAIL_PATH;
+            $post_obj->link      = get_edit_post_link( $post_obj->ID, 'none' );
+            $post_obj->permalink = get_post_permalink( $post_obj->ID );
+        }
+    }
+
+    wl_core_send_json( $related_posts );
 
 }
 

Spacing +21 added lines, -21 removed lines

@@ -15,31 +15,31 @@ 
  *
  * @since 3.0.0
  */
-function wordlift_ajax_related_posts( $http_raw_data = null ) {
+function wordlift_ajax_related_posts($http_raw_data = null) {
 
 	// Extract filtering conditions
-	if ( ! isset( $_GET['post_id'] ) || ! is_numeric( $_GET['post_id'] ) ) { //phpcs:ignore WordPress.Security.NonceVerification.Recommended
-		wp_die( 'Post id missing or invalid!' );
+	if ( ! isset($_GET['post_id']) || ! is_numeric($_GET['post_id'])) { //phpcs:ignore WordPress.Security.NonceVerification.Recommended
+		wp_die('Post id missing or invalid!');
 
 		return;
 	}
 
 	// Get the current post
 	$post_id = (int) $_GET['post_id']; //phpcs:ignore WordPress.Security.NonceVerification.Recommended
-	$post    = get_post( $post_id );
+	$post    = get_post($post_id);
 
-	Wordlift_Log_Service::get_logger( 'wordlift_ajax_related_posts' )->trace( "Going to find posts related to current with post id: $post_id ..." );
+	Wordlift_Log_Service::get_logger('wordlift_ajax_related_posts')->trace("Going to find posts related to current with post id: $post_id ...");
 
 	// Extract filtering conditions
-	$filtering_entity_uris = empty( $http_raw_data ) ? file_get_contents( 'php://input' ) : $http_raw_data;
-	$filtering_entity_uris = json_decode( $filtering_entity_uris );
+	$filtering_entity_uris = empty($http_raw_data) ? file_get_contents('php://input') : $http_raw_data;
+	$filtering_entity_uris = json_decode($filtering_entity_uris);
 
 	$content_service      = Wordpress_Content_Service::get_instance();
 	$filtering_entity_ids = array_filter(
 		array_map(
-			function ( $uri ) use ( $content_service ) {
-				$content = $content_service->get_by_entity_id( $uri );
-				if ( isset( $content ) && is_a( $content->get_bag(), '\WP_Post' ) ) {
+			function($uri) use ($content_service) {
+				$content = $content_service->get_by_entity_id($uri);
+				if (isset($content) && is_a($content->get_bag(), '\WP_Post')) {
 					return $content->get_bag()->ID;
 				} else {
 					return null;
@@ -55,16 +55,16 @@ 
 	// returned.
 	$entity_service = Wordlift_Entity_Service::get_instance();
 
-	if ( $entity_service->is_entity( $post->ID ) ) {
-		$filtering_entity_ids = array( $post_id );
+	if ($entity_service->is_entity($post->ID)) {
+		$filtering_entity_ids = array($post_id);
 	}
 
-	if ( ! empty( $filtering_entity_ids ) ) {
+	if ( ! empty($filtering_entity_ids)) {
 
 		$related_posts = Wordlift_Relation_Service::get_instance()
-		                                          ->get_article_subjects( $filtering_entity_ids, '*', null, 'publish', array( $post_id ), 5 );
+		                                          ->get_article_subjects($filtering_entity_ids, '*', null, 'publish', array($post_id), 5);
 
-		foreach ( $related_posts as $post_obj ) {
+		foreach ($related_posts as $post_obj) {
 
 			/**
 			 * Use the thumbnail.
@@ -74,15 +74,15 @@ 
 			 *
 			 * @since 3.19.3 We're using the medium size image.
 			 */
-			$thumbnail           = get_the_post_thumbnail_url( $post_obj, 'medium' );
-			$post_obj->thumbnail = ( $thumbnail ) ? $thumbnail : WL_DEFAULT_THUMBNAIL_PATH;
-			$post_obj->link      = get_edit_post_link( $post_obj->ID, 'none' );
-			$post_obj->permalink = get_post_permalink( $post_obj->ID );
+			$thumbnail           = get_the_post_thumbnail_url($post_obj, 'medium');
+			$post_obj->thumbnail = ($thumbnail) ? $thumbnail : WL_DEFAULT_THUMBNAIL_PATH;
+			$post_obj->link      = get_edit_post_link($post_obj->ID, 'none');
+			$post_obj->permalink = get_post_permalink($post_obj->ID);
 		}
 	}
 
-	wl_core_send_json( $related_posts );
+	wl_core_send_json($related_posts);
 
 }
 
-add_action( 'wp_ajax_wordlift_related_posts', 'wordlift_ajax_related_posts' );
+add_action('wp_ajax_wordlift_related_posts', 'wordlift_ajax_related_posts');

src/admin/class-wordlift-admin-mappings-page.php

Indentation +24 added lines, -24 removed lines

@@ -16,36 +16,36 @@
  */
 class Wordlift_Admin_Mappings_Page extends Wordlift_Admin_Page {
 
-	/**
-	 * {@inheritdoc}
-	 */
-	public function get_page_title() {
+    /**
+     * {@inheritdoc}
+     */
+    public function get_page_title() {
 
-		return __( 'Schema.org Types', 'wordlift' );
-	}
+        return __( 'Schema.org Types', 'wordlift' );
+    }
 
-	/**
-	 * {@inheritdoc}
-	 */
-	public function get_menu_title() {
+    /**
+     * {@inheritdoc}
+     */
+    public function get_menu_title() {
 
-		return __( 'Schema.org Types', 'wordlift' );
-	}
+        return __( 'Schema.org Types', 'wordlift' );
+    }
 
-	/**
-	 * {@inheritdoc}
-	 */
-	public function get_menu_slug() {
+    /**
+     * {@inheritdoc}
+     */
+    public function get_menu_slug() {
 
-		return 'wl_mappings';
-	}
+        return 'wl_mappings';
+    }
 
-	/**
-	 * {@inheritdoc}
-	 */
-	public function get_partial_name() {
+    /**
+     * {@inheritdoc}
+     */
+    public function get_partial_name() {
 
-		return 'wordlift-admin-mappings-page.php';
-	}
+        return 'wordlift-admin-mappings-page.php';
+    }
 
 }

Spacing +2 added lines, -2 removed lines

@@ -21,7 +21,7 @@ 
 	 */
 	public function get_page_title() {
 
-		return __( 'Schema.org Types', 'wordlift' );
+		return __('Schema.org Types', 'wordlift');
 	}
 
 	/**
@@ -29,7 +29,7 @@ 
 	 */
 	public function get_menu_title() {
 
-		return __( 'Schema.org Types', 'wordlift' );
+		return __('Schema.org Types', 'wordlift');
 	}
 
 	/**

src/admin/class-wordlift-dashboard-service.php

Indentation +209 added lines, -209 removed lines

@@ -18,107 +18,107 @@ 
  */
 class Wordlift_Dashboard_Service {
 
-	/**
-	 * Transient Name
-	 *
-	 * @since  3.4.0
-	 */
-	const TRANSIENT_NAME = 'wl_dashboard_stats';
-
-	/**
-	 * Transient Expiration (in seconds)
-	 *
-	 * @since  3.4.0
-	 */
-	const TRANSIENT_EXPIRATION = 86400;
-
-	/**
-	 * A {@link Wordlift_Rating_Service} instance.
-	 *
-	 * @since  3.10.0
-	 * @access private
-	 * @var \Wordlift_Rating_Service $rating_service A {@link Wordlift_Rating_Service} instance.
-	 */
-	private $rating_service;
-
-	/**
-	 * The {@link Wordlift_Entity_Service} instance.
-	 *
-	 * @since  3.15.0
-	 * @access private
-	 * @var \Wordlift_Entity_Service $entity_service The {@link Wordlift_Entity_Service} instance.
-	 */
-	private $entity_service;
-
-	/**
-	 * Create a Wordlift_Entity_List_Service.
-	 *
-	 * @since 3.4.0
-	 */
-	protected function __construct() {
-
-		$this->rating_service = Wordlift_Rating_Service::get_instance();
-		$this->entity_service = Wordlift_Entity_Service::get_instance();
-
-	}
-
-	private static $instance;
-
-	public static function get_instance() {
-		if ( ! isset( self::$instance ) ) {
-			self::$instance = new self();
-		}
-
-		return self::$instance;
-	}
-
-	/**
-	 * Return stats layout
-	 *
-	 * @since 3.4.0
-	 */
+    /**
+     * Transient Name
+     *
+     * @since  3.4.0
+     */
+    const TRANSIENT_NAME = 'wl_dashboard_stats';
+
+    /**
+     * Transient Expiration (in seconds)
+     *
+     * @since  3.4.0
+     */
+    const TRANSIENT_EXPIRATION = 86400;
+
+    /**
+     * A {@link Wordlift_Rating_Service} instance.
+     *
+     * @since  3.10.0
+     * @access private
+     * @var \Wordlift_Rating_Service $rating_service A {@link Wordlift_Rating_Service} instance.
+     */
+    private $rating_service;
+
+    /**
+     * The {@link Wordlift_Entity_Service} instance.
+     *
+     * @since  3.15.0
+     * @access private
+     * @var \Wordlift_Entity_Service $entity_service The {@link Wordlift_Entity_Service} instance.
+     */
+    private $entity_service;
+
+    /**
+     * Create a Wordlift_Entity_List_Service.
+     *
+     * @since 3.4.0
+     */
+    protected function __construct() {
+
+        $this->rating_service = Wordlift_Rating_Service::get_instance();
+        $this->entity_service = Wordlift_Entity_Service::get_instance();
+
+    }
+
+    private static $instance;
+
+    public static function get_instance() {
+        if ( ! isset( self::$instance ) ) {
+            self::$instance = new self();
+        }
+
+        return self::$instance;
+    }
+
+    /**
+     * Return stats layout
+     *
+     * @since 3.4.0
+     */
     // phpcs:ignore VariableAnalysis.CodeAnalysis.VariableAnalysis.UnusedVariable
-	public function dashboard_widget_callback( $post ) {
-		$caption_kses           = array( 'a' => array( 'href' => array() ) );
-		$enriched_posts_title   = __( 'enriched posts', 'wordlift' );
-		$enriched_posts_caption = sprintf(
-			wp_kses(
-				/* translators: 1: Percentage of annotated posts, 2: Link to the edit screen, 3: Number of annotated posts, 4: Total posts. */
-				__( '%1$s, of your <a href="%2$s">posts</a> are annotated. This means %3$s annotated posts on %4$s.', 'wordlift' ),
-				array( 'a' => array( 'href' => array() ) )
-			),
-			$this->render_stat_param( 'annotatedPostsPercentage' ),
-			esc_url( admin_url( 'edit.php' ) ),
-			$this->render_stat_param( 'annotated_posts' ),
-			$this->render_stat_param( 'posts' )
-		);
-
-		$rating_title   = __( 'average entity rating', 'wordlift' );
-		$rating_caption = sprintf(
-			wp_kses(
-					/* translators: 1: The entities count, 2: The link to the vocabulary, 3: The average rating. */
-				__( 'You have %1$s entities in your <a href="%2$s">vocabulary</a> with an average rating of %3$s.', 'wordlift' ),
-				array( 'a' => array( 'href' => array() ) )
-			),
-			$this->render_stat_param( 'entities' ),
-			esc_url( admin_url( 'edit.php?post_type=entity' ) ),
-			$this->render_stat_param( 'rating' )
-		);
-
-		$graph_title   = __( 'triples in your graph', 'wordlift' );
-		$graph_caption = sprintf(
-			wp_kses(
-					/* translators: 1: The percentage of the graph size compared to Wikidata, 2: The link to Wikidata. */
-				__( 'Your graphs size corresponds to %1$s of <a href="%2$s">Wikidata</a>.', 'wordlift' ),
-				array( 'a' => array( 'href' => array() ) )
-			),
-			$this->render_stat_param( 'wikidata' ),
-			esc_url( 'https://www.wikidata.org/' )
-		);
-
-		$triples_label = __( 'triples', 'wordlift' );
-
-		?>
+    public function dashboard_widget_callback( $post ) {
+        $caption_kses           = array( 'a' => array( 'href' => array() ) );
+        $enriched_posts_title   = __( 'enriched posts', 'wordlift' );
+        $enriched_posts_caption = sprintf(
+            wp_kses(
+                /* translators: 1: Percentage of annotated posts, 2: Link to the edit screen, 3: Number of annotated posts, 4: Total posts. */
+                __( '%1$s, of your <a href="%2$s">posts</a> are annotated. This means %3$s annotated posts on %4$s.', 'wordlift' ),
+                array( 'a' => array( 'href' => array() ) )
+            ),
+            $this->render_stat_param( 'annotatedPostsPercentage' ),
+            esc_url( admin_url( 'edit.php' ) ),
+            $this->render_stat_param( 'annotated_posts' ),
+            $this->render_stat_param( 'posts' )
+        );
+
+        $rating_title   = __( 'average entity rating', 'wordlift' );
+        $rating_caption = sprintf(
+            wp_kses(
+                    /* translators: 1: The entities count, 2: The link to the vocabulary, 3: The average rating. */
+                __( 'You have %1$s entities in your <a href="%2$s">vocabulary</a> with an average rating of %3$s.', 'wordlift' ),
+                array( 'a' => array( 'href' => array() ) )
+            ),
+            $this->render_stat_param( 'entities' ),
+            esc_url( admin_url( 'edit.php?post_type=entity' ) ),
+            $this->render_stat_param( 'rating' )
+        );
+
+        $graph_title   = __( 'triples in your graph', 'wordlift' );
+        $graph_caption = sprintf(
+            wp_kses(
+                    /* translators: 1: The percentage of the graph size compared to Wikidata, 2: The link to Wikidata. */
+                __( 'Your graphs size corresponds to %1$s of <a href="%2$s">Wikidata</a>.', 'wordlift' ),
+                array( 'a' => array( 'href' => array() ) )
+            ),
+            $this->render_stat_param( 'wikidata' ),
+            esc_url( 'https://www.wikidata.org/' )
+        );
+
+        $triples_label = __( 'triples', 'wordlift' );
+
+        ?>
 		<div id="wl-dashboard-widget-inner-wrapper">
 			<div class="wl-stat-card">
 				<div class="wl-stat-graph-wrapper">
@@ -157,114 +157,114 @@ 
 		</div>
 		<?php
 
-	}
-
-	/**
-	 * Return stats
-	 *
-	 * @since 3.4.0
-	 */
-	public function add_dashboard_widgets() {
-		wp_add_dashboard_widget(
-			'wordlift-dashboard-widget',
-			'WordLift Dashboard',
-			array(
-				$this,
-				'dashboard_widget_callback',
-			)
-		);
-	}
-
-	/**
-	 * Return stats
-	 *
-	 * @uses  https://codex.wordpress.org/Function_Reference/set_transient
-	 *
-	 * @since 3.4.0
-	 */
-	public function ajax_get_stats() {
-
-		// If needed, the transient is force to reloaded.
-		if ( isset( $_GET['force_reload'] ) ) { //phpcs:ignore WordPress.Security.NonceVerification.Recommended
-			delete_transient( self::TRANSIENT_NAME );
-		}
-
-		// Try to retrieve the transient
-		$stats = get_transient( self::TRANSIENT_NAME );
-
-		if ( ! $stats ) {
-			// Calculate stats
-			$count_triples = $this->count_triples();
-			$stats         = array(
-				'entities'        => $this->entity_service->count(),
-				'posts'           => $this->count_posts(),
-				'annotated_posts' => $this->count_annotated_posts(),
-				'triples'         => $count_triples ? $count_triples : '-',
-				'rating'          => $this->average_entities_rating(),
-			);
-			// Cache stats results trough transient
-			set_transient( self::TRANSIENT_NAME, $stats, self::TRANSIENT_EXPIRATION );
-		}
-		// Return stats as json object
-		wl_core_send_json( $stats );
-	}
-
-	/**
-	 * Calculate total number of published posts
-	 *
-	 * @uses  https://codex.wordpress.org/it:Riferimento_funzioni/wp_count_posts
-	 * @since 3.4.0
-	 *
-	 * @return int Total number of published posts.
-	 */
-	public function count_posts() {
-
-		return (int) wp_count_posts()->publish;
-	}
-
-	/**
-	 * Calculate total number of annotated published posts
-	 *
-	 * @return int Total number of annotated published posts.
-	 * @since 3.4.0
-	 */
-	public function count_annotated_posts() {
-
-		// Prepare interaction with db
-		global $wpdb;
-		// Retrieve Wordlift relation instances table name
-		// $table_name = wl_core_get_relation_instances_table_name();
-		// Calculate sql statement
-
-		// Perform the query
-		return (int) $wpdb->get_var(
-			"SELECT COUNT(distinct(p.id)) FROM $wpdb->posts as p JOIN {$wpdb->prefix}wl_relation_instances as r ON p.id = r.subject_id AND p.post_type = 'post' AND p.post_status = 'publish'"
-		);
-
-	}
-
-	/**
-	 * Calculate the average entities rating.
-	 *
-	 * @return int Average entities rating.
-	 * @since 3.4.0
-	 *
-	 * @since 3.20.0 this method is public.
-	 */
-	public function average_entities_rating() {
-
-		// Prepare interaction with db
-		global $wpdb;
-
-		// Perform the query.
-		return $this->rating_service->convert_raw_score_to_percentage(
-			$wpdb->get_var(
-				$wpdb->prepare(
-					"SELECT AVG(meta_value) FROM $wpdb->postmeta where meta_key = %s",
-					Wordlift_Rating_Service::RATING_RAW_SCORE_META_KEY
-				)
-			)
-		);
-	}
+    }
+
+    /**
+     * Return stats
+     *
+     * @since 3.4.0
+     */
+    public function add_dashboard_widgets() {
+        wp_add_dashboard_widget(
+            'wordlift-dashboard-widget',
+            'WordLift Dashboard',
+            array(
+                $this,
+                'dashboard_widget_callback',
+            )
+        );
+    }
+
+    /**
+     * Return stats
+     *
+     * @uses  https://codex.wordpress.org/Function_Reference/set_transient
+     *
+     * @since 3.4.0
+     */
+    public function ajax_get_stats() {
+
+        // If needed, the transient is force to reloaded.
+        if ( isset( $_GET['force_reload'] ) ) { //phpcs:ignore WordPress.Security.NonceVerification.Recommended
+            delete_transient( self::TRANSIENT_NAME );
+        }
+
+        // Try to retrieve the transient
+        $stats = get_transient( self::TRANSIENT_NAME );
+
+        if ( ! $stats ) {
+            // Calculate stats
+            $count_triples = $this->count_triples();
+            $stats         = array(
+                'entities'        => $this->entity_service->count(),
+                'posts'           => $this->count_posts(),
+                'annotated_posts' => $this->count_annotated_posts(),
+                'triples'         => $count_triples ? $count_triples : '-',
+                'rating'          => $this->average_entities_rating(),
+            );
+            // Cache stats results trough transient
+            set_transient( self::TRANSIENT_NAME, $stats, self::TRANSIENT_EXPIRATION );
+        }
+        // Return stats as json object
+        wl_core_send_json( $stats );
+    }
+
+    /**
+     * Calculate total number of published posts
+     *
+     * @uses  https://codex.wordpress.org/it:Riferimento_funzioni/wp_count_posts
+     * @since 3.4.0
+     *
+     * @return int Total number of published posts.
+     */
+    public function count_posts() {
+
+        return (int) wp_count_posts()->publish;
+    }
+
+    /**
+     * Calculate total number of annotated published posts
+     *
+     * @return int Total number of annotated published posts.
+     * @since 3.4.0
+     */
+    public function count_annotated_posts() {
+
+        // Prepare interaction with db
+        global $wpdb;
+        // Retrieve Wordlift relation instances table name
+        // $table_name = wl_core_get_relation_instances_table_name();
+        // Calculate sql statement
+
+        // Perform the query
+        return (int) $wpdb->get_var(
+            "SELECT COUNT(distinct(p.id)) FROM $wpdb->posts as p JOIN {$wpdb->prefix}wl_relation_instances as r ON p.id = r.subject_id AND p.post_type = 'post' AND p.post_status = 'publish'"
+        );
+
+    }
+
+    /**
+     * Calculate the average entities rating.
+     *
+     * @return int Average entities rating.
+     * @since 3.4.0
+     *
+     * @since 3.20.0 this method is public.
+     */
+    public function average_entities_rating() {
+
+        // Prepare interaction with db
+        global $wpdb;
+
+        // Perform the query.
+        return $this->rating_service->convert_raw_score_to_percentage(
+            $wpdb->get_var(
+                $wpdb->prepare(
+                    "SELECT AVG(meta_value) FROM $wpdb->postmeta where meta_key = %s",
+                    Wordlift_Rating_Service::RATING_RAW_SCORE_META_KEY
+                )
+            )
+        );
+    }
 
 }

Spacing +35 added lines, -35 removed lines

@@ -65,7 +65,7 @@ 
 	private static $instance;
 
 	public static function get_instance() {
-		if ( ! isset( self::$instance ) ) {
+		if ( ! isset(self::$instance)) {
 			self::$instance = new self();
 		}
 
@@ -78,62 +78,62 @@ 
 	 * @since 3.4.0
 	 */
     // phpcs:ignore VariableAnalysis.CodeAnalysis.VariableAnalysis.UnusedVariable
-	public function dashboard_widget_callback( $post ) {
-		$caption_kses           = array( 'a' => array( 'href' => array() ) );
-		$enriched_posts_title   = __( 'enriched posts', 'wordlift' );
+	public function dashboard_widget_callback($post) {
+		$caption_kses           = array('a' => array('href' => array()));
+		$enriched_posts_title   = __('enriched posts', 'wordlift');
 		$enriched_posts_caption = sprintf(
 			wp_kses(
 				/* translators: 1: Percentage of annotated posts, 2: Link to the edit screen, 3: Number of annotated posts, 4: Total posts. */
-				__( '%1$s, of your <a href="%2$s">posts</a> are annotated. This means %3$s annotated posts on %4$s.', 'wordlift' ),
-				array( 'a' => array( 'href' => array() ) )
+				__('%1$s, of your <a href="%2$s">posts</a> are annotated. This means %3$s annotated posts on %4$s.', 'wordlift'),
+				array('a' => array('href' => array()))
 			),
-			$this->render_stat_param( 'annotatedPostsPercentage' ),
-			esc_url( admin_url( 'edit.php' ) ),
-			$this->render_stat_param( 'annotated_posts' ),
-			$this->render_stat_param( 'posts' )
+			$this->render_stat_param('annotatedPostsPercentage'),
+			esc_url(admin_url('edit.php')),
+			$this->render_stat_param('annotated_posts'),
+			$this->render_stat_param('posts')
 		);
 
-		$rating_title   = __( 'average entity rating', 'wordlift' );
+		$rating_title   = __('average entity rating', 'wordlift');
 		$rating_caption = sprintf(
 			wp_kses(
 					/* translators: 1: The entities count, 2: The link to the vocabulary, 3: The average rating. */
-				__( 'You have %1$s entities in your <a href="%2$s">vocabulary</a> with an average rating of %3$s.', 'wordlift' ),
-				array( 'a' => array( 'href' => array() ) )
+				__('You have %1$s entities in your <a href="%2$s">vocabulary</a> with an average rating of %3$s.', 'wordlift'),
+				array('a' => array('href' => array()))
 			),
-			$this->render_stat_param( 'entities' ),
-			esc_url( admin_url( 'edit.php?post_type=entity' ) ),
-			$this->render_stat_param( 'rating' )
+			$this->render_stat_param('entities'),
+			esc_url(admin_url('edit.php?post_type=entity')),
+			$this->render_stat_param('rating')
 		);
 
-		$graph_title   = __( 'triples in your graph', 'wordlift' );
+		$graph_title   = __('triples in your graph', 'wordlift');
 		$graph_caption = sprintf(
 			wp_kses(
 					/* translators: 1: The percentage of the graph size compared to Wikidata, 2: The link to Wikidata. */
-				__( 'Your graphs size corresponds to %1$s of <a href="%2$s">Wikidata</a>.', 'wordlift' ),
-				array( 'a' => array( 'href' => array() ) )
+				__('Your graphs size corresponds to %1$s of <a href="%2$s">Wikidata</a>.', 'wordlift'),
+				array('a' => array('href' => array()))
 			),
-			$this->render_stat_param( 'wikidata' ),
-			esc_url( 'https://www.wikidata.org/' )
+			$this->render_stat_param('wikidata'),
+			esc_url('https://www.wikidata.org/')
 		);
 
-		$triples_label = __( 'triples', 'wordlift' );
+		$triples_label = __('triples', 'wordlift');
 
 		?>
 		<div id="wl-dashboard-widget-inner-wrapper">
 			<div class="wl-stat-card">
 				<div class="wl-stat-graph-wrapper">
-					<h4><?php echo esc_html( $enriched_posts_title ); ?> <a
+					<h4><?php echo esc_html($enriched_posts_title); ?> <a
 								href="http://docs.wordlift.it/en/latest/faq.html#what-is-content-enrichment"
 								target="_blank"><i class="wl-info"></i></a></h4>
 					<svg id="wl-posts-pie-chart" viewBox="0 0 32 32">
 						<circle r="16" cx="16" cy="16"/>
 					</svg>
 				</div>
-				<p><?php echo wp_kses( $enriched_posts_caption, $caption_kses ); ?> </p>
+				<p><?php echo wp_kses($enriched_posts_caption, $caption_kses); ?> </p>
 			</div>
 			<div class="wl-stat-card">
 				<div class="wl-stat-graph-wrapper">
-					<h4><?php echo esc_html( $rating_title ); ?> <a
+					<h4><?php echo esc_html($rating_title); ?> <a
 								href="http://docs.wordlift.it/en/latest/faq.html#what-factors-determine-the-rating-of-an-entity"
 								target="_blank"><i class="wl-info"></i></a></h4>
 					<svg id="wl-entities-gauge-chart" viewBox="0 0 32 32">
@@ -141,18 +141,18 @@ 
 						<circle r="16" cx="16" cy="16" class="stat"/>
 					</svg>
 				</div>
-				<p><?php echo wp_kses( $rating_caption, $caption_kses ); ?></p>
+				<p><?php echo wp_kses($rating_caption, $caption_kses); ?></p>
 			</div>
 			<div class="wl-stat-card">
 				<div class="wl-stat-graph-wrapper">
-					<h4><?php echo esc_html( $graph_title ); ?><a href="http://docs.wordlift.it/en/latest/faq.html#what-is-a-triple"
+					<h4><?php echo esc_html($graph_title); ?><a href="http://docs.wordlift.it/en/latest/faq.html#what-is-a-triple"
 										target="_blank"><i class="wl-info"></i></a></h4>
 					<div class="wl-triples">
 						<span id="wl-dashboard-widget-triples"></span>
-						<span class="wl-triples-label"><?php echo esc_html( $triples_label ); ?></span>
+						<span class="wl-triples-label"><?php echo esc_html($triples_label); ?></span>
 					</div>
 				</div>
-				<p><?php echo wp_kses( $graph_caption, $caption_kses ); ?></p>
+				<p><?php echo wp_kses($graph_caption, $caption_kses); ?></p>
 			</div>
 		</div>
 		<?php
@@ -185,14 +185,14 @@ 
 	public function ajax_get_stats() {
 
 		// If needed, the transient is force to reloaded.
-		if ( isset( $_GET['force_reload'] ) ) { //phpcs:ignore WordPress.Security.NonceVerification.Recommended
-			delete_transient( self::TRANSIENT_NAME );
+		if (isset($_GET['force_reload'])) { //phpcs:ignore WordPress.Security.NonceVerification.Recommended
+			delete_transient(self::TRANSIENT_NAME);
 		}
 
 		// Try to retrieve the transient
-		$stats = get_transient( self::TRANSIENT_NAME );
+		$stats = get_transient(self::TRANSIENT_NAME);
 
-		if ( ! $stats ) {
+		if ( ! $stats) {
 			// Calculate stats
 			$count_triples = $this->count_triples();
 			$stats         = array(
@@ -203,10 +203,10 @@ 
 				'rating'          => $this->average_entities_rating(),
 			);
 			// Cache stats results trough transient
-			set_transient( self::TRANSIENT_NAME, $stats, self::TRANSIENT_EXPIRATION );
+			set_transient(self::TRANSIENT_NAME, $stats, self::TRANSIENT_EXPIRATION);
 		}
 		// Return stats as json object
-		wl_core_send_json( $stats );
+		wl_core_send_json($stats);
 	}
 
 	/**

src/admin/class-wordlift-primashop-adapter.php

Indentation +23 added lines, -23 removed lines

@@ -7,32 +7,32 @@
  */
 class Wordlift_PrimaShop_Adapter {
 
-	/**
-	 * Create a Wordlift_PrimaShop_Adapter instance.
-	 *
-	 * @since 3.2.3
-	 */
-	public function __construct() {
+    /**
+     * Create a Wordlift_PrimaShop_Adapter instance.
+     *
+     * @since 3.2.3
+     */
+    public function __construct() {
 
-		// Tell WP (and PrimaShop) that we support the *prima-layout-settings*. This will display the Content Settings
-		// in the entity edit page.
-		add_post_type_support( Wordlift_Entity_Service::TYPE_NAME, 'prima-layout-settings' );
+        // Tell WP (and PrimaShop) that we support the *prima-layout-settings*. This will display the Content Settings
+        // in the entity edit page.
+        add_post_type_support( Wordlift_Entity_Service::TYPE_NAME, 'prima-layout-settings' );
 
-	}
+    }
 
-	/**
-	 * Intercept the <em>prima_metabox_entity_header_args</em> filter and return what a call to the related <em>post</em>
-	 * would have returned.
-	 *
-	 * @since 3.2.3
-	 *
-	 * @param array $meta The meta array.
-	 *
-	 * @return array A meta array.
-	 */
-	public function prima_metabox_entity_header_args( $meta ) {
+    /**
+     * Intercept the <em>prima_metabox_entity_header_args</em> filter and return what a call to the related <em>post</em>
+     * would have returned.
+     *
+     * @since 3.2.3
+     *
+     * @param array $meta The meta array.
+     *
+     * @return array A meta array.
+     */
+    public function prima_metabox_entity_header_args( $meta ) {
 
-		return apply_filters( 'prima_metabox_post_header_args', $meta, 'post' );
-	}
+        return apply_filters( 'prima_metabox_post_header_args', $meta, 'post' );
+    }
 
 }

Spacing +3 added lines, -3 removed lines

@@ -16,7 +16,7 @@ 
 
 		// Tell WP (and PrimaShop) that we support the *prima-layout-settings*. This will display the Content Settings
 		// in the entity edit page.
-		add_post_type_support( Wordlift_Entity_Service::TYPE_NAME, 'prima-layout-settings' );
+		add_post_type_support(Wordlift_Entity_Service::TYPE_NAME, 'prima-layout-settings');
 
 	}
 
@@ -30,9 +30,9 @@ 
 	 *
 	 * @return array A meta array.
 	 */
-	public function prima_metabox_entity_header_args( $meta ) {
+	public function prima_metabox_entity_header_args($meta) {
 
-		return apply_filters( 'prima_metabox_post_header_args', $meta, 'post' );
+		return apply_filters('prima_metabox_post_header_args', $meta, 'post');
 	}
 
 }