insideout10 / wordlift-plugin

src/admin/wordlift-admin-meta-box-entities.php

Indentation +168 added lines, -168 removed lines

@@ -14,13 +14,13 @@ 
  */
 function wl_register_metaboxes() {
 
-	new Wl_Metabox();     // Everything is done inside here with the correct timing.
+    new Wl_Metabox();     // Everything is done inside here with the correct timing.
 
 }
 
 if ( is_admin() ) {
-	add_action( 'load-post.php', 'wl_register_metaboxes' );
-	add_action( 'load-post-new.php', 'wl_register_metaboxes' );
+    add_action( 'load-post.php', 'wl_register_metaboxes' );
+    add_action( 'load-post-new.php', 'wl_register_metaboxes' );
 }
 
 /**
@@ -31,41 +31,41 @@ 
  */
 function wl_admin_add_entities_meta_box( $post_type, $post ) {
 
-	/*
+    /*
 	 * Call the `wl_can_see_classification_box` filter to determine whether we can display the classification box.
 	 *
 	 * @since 3.20.3
 	 *
 	 * @see https://github.com/insideout10/wordlift-plugin/issues/914
 	 */
-	if ( ! apply_filters( 'wl_can_see_classification_box', true ) ) {
-		return;
-	}
-
-	// phpcs:ignore WordPress.NamingConventions.ValidHookName.UseUnderscores
-	if ( ! apply_filters( 'wl_feature__enable__classification-sidebar', true ) ) {
-		return;
-	}
-
-	// Bail out if the post type doesn't support a TinyMCE editor.
-	if ( ! wl_post_type_supports_editor( $post_type ) ) {
-		return;
-	}
-
-	// If the editor is not gutenberg and not any other custom editor then we use the sidebar.
-	if ( ! Wordlift_Admin::is_gutenberg() && ! No_Editor_Analysis_Feature::can_no_editor_analysis_be_used(
-		$post->ID
-	) ) {
-		// Add main meta box for related entities and 4W only if not Gutenberg
-		add_meta_box(
-			'wordlift_entities_box',
-			__( 'WordLift', 'wordlift' ),
-			'wl_entities_box_content',
-			$post_type,
-			'side',
-			'high'
-		);
-	}
+    if ( ! apply_filters( 'wl_can_see_classification_box', true ) ) {
+        return;
+    }
+
+    // phpcs:ignore WordPress.NamingConventions.ValidHookName.UseUnderscores
+    if ( ! apply_filters( 'wl_feature__enable__classification-sidebar', true ) ) {
+        return;
+    }
+
+    // Bail out if the post type doesn't support a TinyMCE editor.
+    if ( ! wl_post_type_supports_editor( $post_type ) ) {
+        return;
+    }
+
+    // If the editor is not gutenberg and not any other custom editor then we use the sidebar.
+    if ( ! Wordlift_Admin::is_gutenberg() && ! No_Editor_Analysis_Feature::can_no_editor_analysis_be_used(
+        $post->ID
+    ) ) {
+        // Add main meta box for related entities and 4W only if not Gutenberg
+        add_meta_box(
+            'wordlift_entities_box',
+            __( 'WordLift', 'wordlift' ),
+            'wl_entities_box_content',
+            $post_type,
+            'side',
+            'high'
+        );
+    }
 
 }
 
@@ -82,18 +82,18 @@ 
  */
 function wl_post_type_supports_editor( $post_type ) {
 
-	$default = post_type_supports( $post_type, 'editor' );
-
-	/**
-	 * Allow 3rd parties to force the classification to load.
-	 *
-	 * @param bool $default The preset value as gathered by the `post_type_supports` call.
-	 *
-	 * @see https://github.com/insideout10/wordlift-plugin/issues/847.
-	 *
-	 * @since 3.19.4
-	 */
-	return apply_filters( 'wl_post_type_supports_editor', $default, $post_type );
+    $default = post_type_supports( $post_type, 'editor' );
+
+    /**
+     * Allow 3rd parties to force the classification to load.
+     *
+     * @param bool $default The preset value as gathered by the `post_type_supports` call.
+     *
+     * @see https://github.com/insideout10/wordlift-plugin/issues/847.
+     *
+     * @since 3.19.4
+     */
+    return apply_filters( 'wl_post_type_supports_editor', $default, $post_type );
 }
 
 /**
@@ -103,141 +103,141 @@ 
  */
 function wl_entities_box_content( $post, $wrapper = true ) {
 
-	// Angularjs edit-post widget wrapper.
-	if ( $wrapper ) {
-		echo '<div id="wordlift-edit-post-outer-wrapper"></div>';
-	}
+    // Angularjs edit-post widget wrapper.
+    if ( $wrapper ) {
+        echo '<div id="wordlift-edit-post-outer-wrapper"></div>';
+    }
 }
 
 function wl_entities_box_content_scripts() {
-	$post = get_post();
+    $post = get_post();
 
-	// Angularjs edit-post widget classification boxes configuration.
-	// phpcs:ignore WordPress.PHP.DiscouragedPHPFunctions.serialize_unserialize
-	$classification_boxes = unserialize( WL_CORE_POST_CLASSIFICATION_BOXES );
+    // Angularjs edit-post widget classification boxes configuration.
+    // phpcs:ignore WordPress.PHP.DiscouragedPHPFunctions.serialize_unserialize
+    $classification_boxes = unserialize( WL_CORE_POST_CLASSIFICATION_BOXES );
 
-	// Array to store all related entities ids.
-	$all_referenced_entities_ids = array();
+    // Array to store all related entities ids.
+    $all_referenced_entities_ids = array();
 
-	// Add selected entities to classification_boxes.
-	foreach ( $classification_boxes as $i => $box ) {
+    // Add selected entities to classification_boxes.
+    foreach ( $classification_boxes as $i => $box ) {
 
-		// Get the entity referenced from the post content.
-		/*
+        // Get the entity referenced from the post content.
+        /*
 		 * Allow 3rd parties to provide another post content.
 		 *
 		 * @since 3.20.0
 		 */
-		$post_content = apply_filters( 'wl_post_content', $post->post_content, $post );
-		$entity_uris  = Wordlift_Content_Filter_Service::get_instance()->get_entity_uris( $post_content );
-
-		// Enhance current box selected entities.
-		$classification_boxes[ $i ]['selectedEntities'] = $entity_uris;
-
-		// Maps the URIs to entity posts.
-		$entity_service = Wordlift_Entity_Service::get_instance();
-
-		// Replace all entity URI's with post ID's if found or null if there is no related post.
-		$entity_ids = array_map(
-			function ( $item ) use ( $entity_service ) {
-				// Return entity post by the entity URI or null.
-				$post = $entity_service->get_entity_post_by_uri( $item );
-
-				// Check that the post object is not null.
-				if ( ! empty( $post ) ) {
-					  return $post->ID;
-				}
-			},
-			$entity_uris
-		);
-		// Store the entity ids for all the 4W.
-		$all_referenced_entities_ids = array_merge( $all_referenced_entities_ids, $entity_ids );
-
-	}
-
-	// Json encoding for classification boxes structure.
-	$classification_boxes = wp_json_encode( $classification_boxes );
-
-	// Ensure there are no repetitions of the referenced entities.
-	$all_referenced_entities_ids = array_unique( $all_referenced_entities_ids );
-
-	// Remove all null, false and empty strings.
-	// NULL is being returned in some cases, when there is not related post, so we need to remove it.
-	$all_referenced_entities_ids = array_filter( $all_referenced_entities_ids );
-
-	// Build the entity storage object.
-	$referenced_entities_obj = array();
-	foreach ( $all_referenced_entities_ids as $referenced_entity ) {
-		$entity = wl_serialize_entity( $referenced_entity );
-		// Set a default confidence of `PHP_INT_MAX` for already annotated entities.
-		$referenced_entities_obj[ $entity['id'] ] = $entity
-													+ array( 'confidence' => PHP_INT_MAX );
-	}
-
-	$referenced_entities_obj = empty( $referenced_entities_obj ) ?
-		'{}' : wp_json_encode( $referenced_entities_obj );
-
-	$published_place_id  = get_post_meta(
-		$post->ID,
-		Wordlift_Schema_Service::FIELD_LOCATION_CREATED,
-		true
-	);
-	$published_place_obj = ( $published_place_id ) ?
-		wp_json_encode( wl_serialize_entity( $published_place_id ) ) :
-		null;
-
-	$topic_id  = get_post_meta(
-		$post->ID,
-		Wordlift_Schema_Service::FIELD_TOPIC,
-		true
-	);
-	$topic_obj = ( $topic_id ) ?
-		wp_json_encode( wl_serialize_entity( $topic_id ) ) :
-		null;
-
-	$configuration_service = Wordlift_Configuration_Service::get_instance();
-
-	$default_thumbnail_path = WL_DEFAULT_THUMBNAIL_PATH;
-	$default_path           = WL_DEFAULT_PATH;
-	$dataset_uri            = $configuration_service->get_dataset_uri();
-	$current_post_uri       = Wordlift_Entity_Service::get_instance()->get_uri( $post->ID );
-	$is_entity              = Wordlift_Entity_Service::get_instance()->is_entity( $post->ID );
-
-	// Retrieve the current post author.
-	$post_author = get_userdata( $post->post_author )->display_name;
-	// Retrive the published date.
-	$published_date = get_the_time( 'Y-m-d', $post->ID );
-	// Current language.
-	$current_language            = $configuration_service->get_language_code();
-	$wordlift_timeline_shortcode = new Wordlift_Timeline_Shortcode();
-	$timelinejs_default_options  = wp_json_encode( $wordlift_timeline_shortcode->get_timelinejs_default_options(), JSON_PRETTY_PRINT );
-	$addslashes_post_author      = addslashes( $post_author );
-
-	$metabox_settings = array(
-		'classificationBoxes'      => json_decode( $classification_boxes ),
-		'entities'                 => json_decode( $referenced_entities_obj ),
-		'currentPostId'            => intval( $post->ID ),
-		'currentPostUri'           => $current_post_uri,
-		'currentPostType'          => $post->post_type,
-		'isEntity'                 => ! empty( $is_entity ),
-		'defaultThumbnailPath'     => $default_thumbnail_path,
-		'defaultWordLiftPath'      => $default_path,
-		'datasetUri'               => $dataset_uri,
-		'currentUser'              => $addslashes_post_author,
-		'publishedDate'            => $published_date,
-		'publishedPlace'           => $published_place_obj,
-		'topic'                    => json_decode( $topic_obj ),
-		'currentLanguage'          => $current_language,
-		'timelinejsDefaultOptions' => json_decode( $timelinejs_default_options ),
-		'ajax_url'                 => admin_url( 'admin-ajax.php' ),
-	);
-
-	// Allow Classic and Block Editor scripts to register first.
-	// Hook to the Block Editor script.
-	wp_localize_script( 'wl-block-editor', '_wlMetaBoxSettings', array( 'settings' => $metabox_settings ) );
-
-	// Hook to the Classic Editor script, see Wordlift_Admin_Post_Edit_Page.
-	wp_localize_script( 'wl-classic-editor', '_wlMetaBoxSettings', array( 'settings' => $metabox_settings ) );
+        $post_content = apply_filters( 'wl_post_content', $post->post_content, $post );
+        $entity_uris  = Wordlift_Content_Filter_Service::get_instance()->get_entity_uris( $post_content );
+
+        // Enhance current box selected entities.
+        $classification_boxes[ $i ]['selectedEntities'] = $entity_uris;
+
+        // Maps the URIs to entity posts.
+        $entity_service = Wordlift_Entity_Service::get_instance();
+
+        // Replace all entity URI's with post ID's if found or null if there is no related post.
+        $entity_ids = array_map(
+            function ( $item ) use ( $entity_service ) {
+                // Return entity post by the entity URI or null.
+                $post = $entity_service->get_entity_post_by_uri( $item );
+
+                // Check that the post object is not null.
+                if ( ! empty( $post ) ) {
+                        return $post->ID;
+                }
+            },
+            $entity_uris
+        );
+        // Store the entity ids for all the 4W.
+        $all_referenced_entities_ids = array_merge( $all_referenced_entities_ids, $entity_ids );
+
+    }
+
+    // Json encoding for classification boxes structure.
+    $classification_boxes = wp_json_encode( $classification_boxes );
+
+    // Ensure there are no repetitions of the referenced entities.
+    $all_referenced_entities_ids = array_unique( $all_referenced_entities_ids );
+
+    // Remove all null, false and empty strings.
+    // NULL is being returned in some cases, when there is not related post, so we need to remove it.
+    $all_referenced_entities_ids = array_filter( $all_referenced_entities_ids );
+
+    // Build the entity storage object.
+    $referenced_entities_obj = array();
+    foreach ( $all_referenced_entities_ids as $referenced_entity ) {
+        $entity = wl_serialize_entity( $referenced_entity );
+        // Set a default confidence of `PHP_INT_MAX` for already annotated entities.
+        $referenced_entities_obj[ $entity['id'] ] = $entity
+                                                    + array( 'confidence' => PHP_INT_MAX );
+    }
+
+    $referenced_entities_obj = empty( $referenced_entities_obj ) ?
+        '{}' : wp_json_encode( $referenced_entities_obj );
+
+    $published_place_id  = get_post_meta(
+        $post->ID,
+        Wordlift_Schema_Service::FIELD_LOCATION_CREATED,
+        true
+    );
+    $published_place_obj = ( $published_place_id ) ?
+        wp_json_encode( wl_serialize_entity( $published_place_id ) ) :
+        null;
+
+    $topic_id  = get_post_meta(
+        $post->ID,
+        Wordlift_Schema_Service::FIELD_TOPIC,
+        true
+    );
+    $topic_obj = ( $topic_id ) ?
+        wp_json_encode( wl_serialize_entity( $topic_id ) ) :
+        null;
+
+    $configuration_service = Wordlift_Configuration_Service::get_instance();
+
+    $default_thumbnail_path = WL_DEFAULT_THUMBNAIL_PATH;
+    $default_path           = WL_DEFAULT_PATH;
+    $dataset_uri            = $configuration_service->get_dataset_uri();
+    $current_post_uri       = Wordlift_Entity_Service::get_instance()->get_uri( $post->ID );
+    $is_entity              = Wordlift_Entity_Service::get_instance()->is_entity( $post->ID );
+
+    // Retrieve the current post author.
+    $post_author = get_userdata( $post->post_author )->display_name;
+    // Retrive the published date.
+    $published_date = get_the_time( 'Y-m-d', $post->ID );
+    // Current language.
+    $current_language            = $configuration_service->get_language_code();
+    $wordlift_timeline_shortcode = new Wordlift_Timeline_Shortcode();
+    $timelinejs_default_options  = wp_json_encode( $wordlift_timeline_shortcode->get_timelinejs_default_options(), JSON_PRETTY_PRINT );
+    $addslashes_post_author      = addslashes( $post_author );
+
+    $metabox_settings = array(
+        'classificationBoxes'      => json_decode( $classification_boxes ),
+        'entities'                 => json_decode( $referenced_entities_obj ),
+        'currentPostId'            => intval( $post->ID ),
+        'currentPostUri'           => $current_post_uri,
+        'currentPostType'          => $post->post_type,
+        'isEntity'                 => ! empty( $is_entity ),
+        'defaultThumbnailPath'     => $default_thumbnail_path,
+        'defaultWordLiftPath'      => $default_path,
+        'datasetUri'               => $dataset_uri,
+        'currentUser'              => $addslashes_post_author,
+        'publishedDate'            => $published_date,
+        'publishedPlace'           => $published_place_obj,
+        'topic'                    => json_decode( $topic_obj ),
+        'currentLanguage'          => $current_language,
+        'timelinejsDefaultOptions' => json_decode( $timelinejs_default_options ),
+        'ajax_url'                 => admin_url( 'admin-ajax.php' ),
+    );
+
+    // Allow Classic and Block Editor scripts to register first.
+    // Hook to the Block Editor script.
+    wp_localize_script( 'wl-block-editor', '_wlMetaBoxSettings', array( 'settings' => $metabox_settings ) );
+
+    // Hook to the Classic Editor script, see Wordlift_Admin_Post_Edit_Page.
+    wp_localize_script( 'wl-classic-editor', '_wlMetaBoxSettings', array( 'settings' => $metabox_settings ) );
 
 }
 

Spacing +57 added lines, -59 removed lines

@@ -14,13 +14,13 @@ 
  */
 function wl_register_metaboxes() {
 
-	new Wl_Metabox();     // Everything is done inside here with the correct timing.
+	new Wl_Metabox(); // Everything is done inside here with the correct timing.
 
 }
 
-if ( is_admin() ) {
-	add_action( 'load-post.php', 'wl_register_metaboxes' );
-	add_action( 'load-post-new.php', 'wl_register_metaboxes' );
+if (is_admin()) {
+	add_action('load-post.php', 'wl_register_metaboxes');
+	add_action('load-post-new.php', 'wl_register_metaboxes');
 }
 
 /**
@@ -29,7 +29,7 @@ 
  * @param string  $post_type The type of the current open post.
  * @param WP_Post $post WordPress post.
  */
-function wl_admin_add_entities_meta_box( $post_type, $post ) {
+function wl_admin_add_entities_meta_box($post_type, $post) {
 
 	/*
 	 * Call the `wl_can_see_classification_box` filter to determine whether we can display the classification box.
@@ -38,28 +38,28 @@ 
 	 *
 	 * @see https://github.com/insideout10/wordlift-plugin/issues/914
 	 */
-	if ( ! apply_filters( 'wl_can_see_classification_box', true ) ) {
+	if ( ! apply_filters('wl_can_see_classification_box', true)) {
 		return;
 	}
 
 	// phpcs:ignore WordPress.NamingConventions.ValidHookName.UseUnderscores
-	if ( ! apply_filters( 'wl_feature__enable__classification-sidebar', true ) ) {
+	if ( ! apply_filters('wl_feature__enable__classification-sidebar', true)) {
 		return;
 	}
 
 	// Bail out if the post type doesn't support a TinyMCE editor.
-	if ( ! wl_post_type_supports_editor( $post_type ) ) {
+	if ( ! wl_post_type_supports_editor($post_type)) {
 		return;
 	}
 
 	// If the editor is not gutenberg and not any other custom editor then we use the sidebar.
 	if ( ! Wordlift_Admin::is_gutenberg() && ! No_Editor_Analysis_Feature::can_no_editor_analysis_be_used(
 		$post->ID
-	) ) {
+	)) {
 		// Add main meta box for related entities and 4W only if not Gutenberg
 		add_meta_box(
 			'wordlift_entities_box',
-			__( 'WordLift', 'wordlift' ),
+			__('WordLift', 'wordlift'),
 			'wl_entities_box_content',
 			$post_type,
 			'side',
@@ -69,7 +69,7 @@ 
 
 }
 
-add_action( 'add_meta_boxes', 'wl_admin_add_entities_meta_box', 10, 2 );
+add_action('add_meta_boxes', 'wl_admin_add_entities_meta_box', 10, 2);
 
 /**
  * Whether the post type supports the editor UI.
@@ -80,9 +80,9 @@ 
  *
  * @return bool True if the editor UI is supported otherwise false.
  */
-function wl_post_type_supports_editor( $post_type ) {
+function wl_post_type_supports_editor($post_type) {
 
-	$default = post_type_supports( $post_type, 'editor' );
+	$default = post_type_supports($post_type, 'editor');
 
 	/**
 	 * Allow 3rd parties to force the classification to load.
@@ -93,7 +93,7 @@ 
 	 *
 	 * @since 3.19.4
 	 */
-	return apply_filters( 'wl_post_type_supports_editor', $default, $post_type );
+	return apply_filters('wl_post_type_supports_editor', $default, $post_type);
 }
 
 /**
@@ -101,10 +101,10 @@ 
  *
  * @param WP_Post $post The current post.
  */
-function wl_entities_box_content( $post, $wrapper = true ) {
+function wl_entities_box_content($post, $wrapper = true) {
 
 	// Angularjs edit-post widget wrapper.
-	if ( $wrapper ) {
+	if ($wrapper) {
 		echo '<div id="wordlift-edit-post-outer-wrapper"></div>';
 	}
 }
@@ -114,13 +114,13 @@ 
 
 	// Angularjs edit-post widget classification boxes configuration.
 	// phpcs:ignore WordPress.PHP.DiscouragedPHPFunctions.serialize_unserialize
-	$classification_boxes = unserialize( WL_CORE_POST_CLASSIFICATION_BOXES );
+	$classification_boxes = unserialize(WL_CORE_POST_CLASSIFICATION_BOXES);
 
 	// Array to store all related entities ids.
 	$all_referenced_entities_ids = array();
 
 	// Add selected entities to classification_boxes.
-	foreach ( $classification_boxes as $i => $box ) {
+	foreach ($classification_boxes as $i => $box) {
 
 		// Get the entity referenced from the post content.
 		/*
@@ -128,118 +128,116 @@ 
 		 *
 		 * @since 3.20.0
 		 */
-		$post_content = apply_filters( 'wl_post_content', $post->post_content, $post );
-		$entity_uris  = Wordlift_Content_Filter_Service::get_instance()->get_entity_uris( $post_content );
+		$post_content = apply_filters('wl_post_content', $post->post_content, $post);
+		$entity_uris  = Wordlift_Content_Filter_Service::get_instance()->get_entity_uris($post_content);
 
 		// Enhance current box selected entities.
-		$classification_boxes[ $i ]['selectedEntities'] = $entity_uris;
+		$classification_boxes[$i]['selectedEntities'] = $entity_uris;
 
 		// Maps the URIs to entity posts.
 		$entity_service = Wordlift_Entity_Service::get_instance();
 
 		// Replace all entity URI's with post ID's if found or null if there is no related post.
 		$entity_ids = array_map(
-			function ( $item ) use ( $entity_service ) {
+			function($item) use ($entity_service) {
 				// Return entity post by the entity URI or null.
-				$post = $entity_service->get_entity_post_by_uri( $item );
+				$post = $entity_service->get_entity_post_by_uri($item);
 
 				// Check that the post object is not null.
-				if ( ! empty( $post ) ) {
+				if ( ! empty($post)) {
 					  return $post->ID;
 				}
 			},
 			$entity_uris
 		);
 		// Store the entity ids for all the 4W.
-		$all_referenced_entities_ids = array_merge( $all_referenced_entities_ids, $entity_ids );
+		$all_referenced_entities_ids = array_merge($all_referenced_entities_ids, $entity_ids);
 
 	}
 
 	// Json encoding for classification boxes structure.
-	$classification_boxes = wp_json_encode( $classification_boxes );
+	$classification_boxes = wp_json_encode($classification_boxes);
 
 	// Ensure there are no repetitions of the referenced entities.
-	$all_referenced_entities_ids = array_unique( $all_referenced_entities_ids );
+	$all_referenced_entities_ids = array_unique($all_referenced_entities_ids);
 
 	// Remove all null, false and empty strings.
 	// NULL is being returned in some cases, when there is not related post, so we need to remove it.
-	$all_referenced_entities_ids = array_filter( $all_referenced_entities_ids );
+	$all_referenced_entities_ids = array_filter($all_referenced_entities_ids);
 
 	// Build the entity storage object.
 	$referenced_entities_obj = array();
-	foreach ( $all_referenced_entities_ids as $referenced_entity ) {
-		$entity = wl_serialize_entity( $referenced_entity );
+	foreach ($all_referenced_entities_ids as $referenced_entity) {
+		$entity = wl_serialize_entity($referenced_entity);
 		// Set a default confidence of `PHP_INT_MAX` for already annotated entities.
-		$referenced_entities_obj[ $entity['id'] ] = $entity
-													+ array( 'confidence' => PHP_INT_MAX );
+		$referenced_entities_obj[$entity['id']] = $entity
+													+ array('confidence' => PHP_INT_MAX);
 	}
 
-	$referenced_entities_obj = empty( $referenced_entities_obj ) ?
-		'{}' : wp_json_encode( $referenced_entities_obj );
+	$referenced_entities_obj = empty($referenced_entities_obj) ?
+		'{}' : wp_json_encode($referenced_entities_obj);
 
-	$published_place_id  = get_post_meta(
+	$published_place_id = get_post_meta(
 		$post->ID,
 		Wordlift_Schema_Service::FIELD_LOCATION_CREATED,
 		true
 	);
-	$published_place_obj = ( $published_place_id ) ?
-		wp_json_encode( wl_serialize_entity( $published_place_id ) ) :
-		null;
+	$published_place_obj = ($published_place_id) ?
+		wp_json_encode(wl_serialize_entity($published_place_id)) : null;
 
-	$topic_id  = get_post_meta(
+	$topic_id = get_post_meta(
 		$post->ID,
 		Wordlift_Schema_Service::FIELD_TOPIC,
 		true
 	);
-	$topic_obj = ( $topic_id ) ?
-		wp_json_encode( wl_serialize_entity( $topic_id ) ) :
-		null;
+	$topic_obj = ($topic_id) ?
+		wp_json_encode(wl_serialize_entity($topic_id)) : null;
 
 	$configuration_service = Wordlift_Configuration_Service::get_instance();
 
 	$default_thumbnail_path = WL_DEFAULT_THUMBNAIL_PATH;
 	$default_path           = WL_DEFAULT_PATH;
 	$dataset_uri            = $configuration_service->get_dataset_uri();
-	$current_post_uri       = Wordlift_Entity_Service::get_instance()->get_uri( $post->ID );
-	$is_entity              = Wordlift_Entity_Service::get_instance()->is_entity( $post->ID );
+	$current_post_uri       = Wordlift_Entity_Service::get_instance()->get_uri($post->ID);
+	$is_entity              = Wordlift_Entity_Service::get_instance()->is_entity($post->ID);
 
 	// Retrieve the current post author.
-	$post_author = get_userdata( $post->post_author )->display_name;
+	$post_author = get_userdata($post->post_author)->display_name;
 	// Retrive the published date.
-	$published_date = get_the_time( 'Y-m-d', $post->ID );
+	$published_date = get_the_time('Y-m-d', $post->ID);
 	// Current language.
 	$current_language            = $configuration_service->get_language_code();
 	$wordlift_timeline_shortcode = new Wordlift_Timeline_Shortcode();
-	$timelinejs_default_options  = wp_json_encode( $wordlift_timeline_shortcode->get_timelinejs_default_options(), JSON_PRETTY_PRINT );
-	$addslashes_post_author      = addslashes( $post_author );
+	$timelinejs_default_options  = wp_json_encode($wordlift_timeline_shortcode->get_timelinejs_default_options(), JSON_PRETTY_PRINT);
+	$addslashes_post_author      = addslashes($post_author);
 
 	$metabox_settings = array(
-		'classificationBoxes'      => json_decode( $classification_boxes ),
-		'entities'                 => json_decode( $referenced_entities_obj ),
-		'currentPostId'            => intval( $post->ID ),
+		'classificationBoxes'      => json_decode($classification_boxes),
+		'entities'                 => json_decode($referenced_entities_obj),
+		'currentPostId'            => intval($post->ID),
 		'currentPostUri'           => $current_post_uri,
 		'currentPostType'          => $post->post_type,
-		'isEntity'                 => ! empty( $is_entity ),
+		'isEntity'                 => ! empty($is_entity),
 		'defaultThumbnailPath'     => $default_thumbnail_path,
 		'defaultWordLiftPath'      => $default_path,
 		'datasetUri'               => $dataset_uri,
 		'currentUser'              => $addslashes_post_author,
 		'publishedDate'            => $published_date,
 		'publishedPlace'           => $published_place_obj,
-		'topic'                    => json_decode( $topic_obj ),
+		'topic'                    => json_decode($topic_obj),
 		'currentLanguage'          => $current_language,
-		'timelinejsDefaultOptions' => json_decode( $timelinejs_default_options ),
-		'ajax_url'                 => admin_url( 'admin-ajax.php' ),
+		'timelinejsDefaultOptions' => json_decode($timelinejs_default_options),
+		'ajax_url'                 => admin_url('admin-ajax.php'),
 	);
 
 	// Allow Classic and Block Editor scripts to register first.
 	// Hook to the Block Editor script.
-	wp_localize_script( 'wl-block-editor', '_wlMetaBoxSettings', array( 'settings' => $metabox_settings ) );
+	wp_localize_script('wl-block-editor', '_wlMetaBoxSettings', array('settings' => $metabox_settings));
 
 	// Hook to the Classic Editor script, see Wordlift_Admin_Post_Edit_Page.
-	wp_localize_script( 'wl-classic-editor', '_wlMetaBoxSettings', array( 'settings' => $metabox_settings ) );
+	wp_localize_script('wl-classic-editor', '_wlMetaBoxSettings', array('settings' => $metabox_settings));
 
 }
 
-add_action( 'admin_print_scripts-post.php', 'wl_entities_box_content_scripts', 11 );
-add_action( 'admin_print_scripts-post-new.php', 'wl_entities_box_content_scripts', 11 );
+add_action('admin_print_scripts-post.php', 'wl_entities_box_content_scripts', 11);
+add_action('admin_print_scripts-post-new.php', 'wl_entities_box_content_scripts', 11);

src/admin/class-wordlift-admin-setup.php

Indentation +238 added lines, -238 removed lines

@@ -13,7 +13,7 @@ 
  */
 
 if ( ! defined( 'ABSPATH' ) ) {
-	exit;
+    exit;
 }
 
 /**
@@ -28,126 +28,126 @@ 
  */
 class Wordlift_Admin_Setup {
 
-	/**
-	 * A {@link Wordlift_Key_Validation_Service} instance.
-	 *
-	 * @since  3.9.0
-	 * @access private
-	 * @var Wordlift_Key_Validation_Service A {@link Wordlift_Key_Validation_Service} instance.
-	 */
-	private $key_validation_service;
-
-	/**
-	 * A {@link Wordlift_Entity_Service} instance.
-	 *
-	 * @since  3.9.0
-	 * @access private
-	 * @var Wordlift_Entity_Service $entity_service A {@link Wordlift_Entity_Service} instance.
-	 */
-	private $entity_service;
-
-	/**
-	 * A {@link Wordlift_Admin_Language_Select_Element} element renderer.
-	 *
-	 * @since  3.20.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.20.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;
-
-	/**
-	 * Initialize the class and set its properties.
-	 *
-	 * @param \Wordlift_Key_Validation_Service        $key_validation_service A {@link Wordlift_Key_Validation_Service} instance.
-	 * @param \Wordlift_Entity_Service                $entity_service A {@link Wordlift_Entity_Service} instance.
-	 * @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.
-	 *
-	 * @since    3.9.0
-	 */
-	public function __construct( $key_validation_service, $entity_service, $language_select_element, $country_select_element ) {
-
-		// Set a reference to the key validation service.
-		$this->key_validation_service = $key_validation_service;
-
-		// Set a reference to the entity service.
-		$this->entity_service = $entity_service;
-
-		// Set a reference to the UI elements language and country.
-		$this->language_select_element = $language_select_element;
-		$this->country_select_element  = $country_select_element;
-
-		// Hook to some WP's events:
-		// When WP is loaded check whether the user decided to skip the set-up, i.e. don't show us even if WL is not set up.
-		add_action( 'wp_loaded', array( $this, 'hide_notices' ) );
-
-		// Hook to `admin_menu` in order to add our own setup wizard page.
-		add_action( 'admin_menu', array( $this, 'admin_menu' ) );
-
-		// Triggered when the user accesses the admin area, we decide whether to show our own wizard.
-		add_action( 'admin_init', array( $this, 'show_page' ) );
-
-		/**
-		 * 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 ) ) {
-			// Hook to `admin_notices` to display our notices.
-			add_action( 'admin_notices', array( $this, 'admin_notices' ) );
-		}
-	}
-
-	/**
-	 * Hook to `admin_init` and redirect to WordLift's setup page if the `_wl_activation_redirect` transient flag is set.
-	 *
-	 * @since 3.9.0
-	 */
-	public function admin_init() {
-
-		// If the `_wl_activation_redirect` is set, the redirect to the setup page.
-		if ( get_transient( '_wl_activation_redirect' ) ) {
-			delete_transient( '_wl_activation_redirect' );
-
-			// If the user asked to skip the wizard then comply.
-			if ( Wordlift_Configuration_Service::get_instance()->is_skip_wizard() ) {
-				return;
-			}
-
-			// If we're already on the page or the user doesn't have permissions, return.
-			if ( ( ! empty( $_GET['page'] ) && 'wl-setup' === $_GET['page'] ) || is_network_admin() || isset( $_GET['activate-multi'] ) || ! current_user_can( 'manage_options' ) ) { //phpcs:ignore WordPress.Security.NonceVerification.Recommended
-				return;
-			}
-
-			// Finally redirect to the setup page.
-			wp_safe_redirect( admin_url( 'index.php?page=wl-setup' ) );
-
-			exit;
-		}
-
-	}
-
-	/**
-	 * Generate an admin notice suggesting to start the wizard if there is no configuration.
-	 *
-	 * @since    3.9.0
-	 */
-	public function admin_notices() {
-
-		// Use `wl_configuration_get_key` to check whether WL's key is set and that the user didn't disable the wizard.
-		if ( '' === Wordlift_Configuration_Service::get_instance()->get_key() && ! Wordlift_Configuration_Service::get_instance()->is_skip_wizard() ) { ?>
+    /**
+     * A {@link Wordlift_Key_Validation_Service} instance.
+     *
+     * @since  3.9.0
+     * @access private
+     * @var Wordlift_Key_Validation_Service A {@link Wordlift_Key_Validation_Service} instance.
+     */
+    private $key_validation_service;
+
+    /**
+     * A {@link Wordlift_Entity_Service} instance.
+     *
+     * @since  3.9.0
+     * @access private
+     * @var Wordlift_Entity_Service $entity_service A {@link Wordlift_Entity_Service} instance.
+     */
+    private $entity_service;
+
+    /**
+     * A {@link Wordlift_Admin_Language_Select_Element} element renderer.
+     *
+     * @since  3.20.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.20.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;
+
+    /**
+     * Initialize the class and set its properties.
+     *
+     * @param \Wordlift_Key_Validation_Service        $key_validation_service A {@link Wordlift_Key_Validation_Service} instance.
+     * @param \Wordlift_Entity_Service                $entity_service A {@link Wordlift_Entity_Service} instance.
+     * @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.
+     *
+     * @since    3.9.0
+     */
+    public function __construct( $key_validation_service, $entity_service, $language_select_element, $country_select_element ) {
+
+        // Set a reference to the key validation service.
+        $this->key_validation_service = $key_validation_service;
+
+        // Set a reference to the entity service.
+        $this->entity_service = $entity_service;
+
+        // Set a reference to the UI elements language and country.
+        $this->language_select_element = $language_select_element;
+        $this->country_select_element  = $country_select_element;
+
+        // Hook to some WP's events:
+        // When WP is loaded check whether the user decided to skip the set-up, i.e. don't show us even if WL is not set up.
+        add_action( 'wp_loaded', array( $this, 'hide_notices' ) );
+
+        // Hook to `admin_menu` in order to add our own setup wizard page.
+        add_action( 'admin_menu', array( $this, 'admin_menu' ) );
+
+        // Triggered when the user accesses the admin area, we decide whether to show our own wizard.
+        add_action( 'admin_init', array( $this, 'show_page' ) );
+
+        /**
+         * 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 ) ) {
+            // Hook to `admin_notices` to display our notices.
+            add_action( 'admin_notices', array( $this, 'admin_notices' ) );
+        }
+    }
+
+    /**
+     * Hook to `admin_init` and redirect to WordLift's setup page if the `_wl_activation_redirect` transient flag is set.
+     *
+     * @since 3.9.0
+     */
+    public function admin_init() {
+
+        // If the `_wl_activation_redirect` is set, the redirect to the setup page.
+        if ( get_transient( '_wl_activation_redirect' ) ) {
+            delete_transient( '_wl_activation_redirect' );
+
+            // If the user asked to skip the wizard then comply.
+            if ( Wordlift_Configuration_Service::get_instance()->is_skip_wizard() ) {
+                return;
+            }
+
+            // If we're already on the page or the user doesn't have permissions, return.
+            if ( ( ! empty( $_GET['page'] ) && 'wl-setup' === $_GET['page'] ) || is_network_admin() || isset( $_GET['activate-multi'] ) || ! current_user_can( 'manage_options' ) ) { //phpcs:ignore WordPress.Security.NonceVerification.Recommended
+                return;
+            }
+
+            // Finally redirect to the setup page.
+            wp_safe_redirect( admin_url( 'index.php?page=wl-setup' ) );
+
+            exit;
+        }
+
+    }
+
+    /**
+     * Generate an admin notice suggesting to start the wizard if there is no configuration.
+     *
+     * @since    3.9.0
+     */
+    public function admin_notices() {
+
+        // Use `wl_configuration_get_key` to check whether WL's key is set and that the user didn't disable the wizard.
+        if ( '' === Wordlift_Configuration_Service::get_instance()->get_key() && ! Wordlift_Configuration_Service::get_instance()->is_skip_wizard() ) { ?>
 			<div id="wl-message" class="updated">
 				<p><?php esc_html_e( 'Welcome to WordLift &#8211; You&lsquo;re almost ready to start', 'wordlift' ); ?></p>
 				<p class="submit">
@@ -158,138 +158,138 @@ 
 				</p>
 			</div>
 			<?php
-		}
-
-	}
-
-	/**
-	 * Handle hiding the wizard notices by user request.
-	 *
-	 * @since    3.9.0
-	 */
-	public function hide_notices() {
-
-		// If it's not a `wl-hide-notice` or the nonce is not set, return.
-		if ( ! isset( $_GET['wl-hide-notice'], $_GET['_wl_notice_nonce'] ) ) {
-			return;
-		}
-
-		// If the nonce is invalid, return an error.
-		if ( ! wp_verify_nonce( sanitize_text_field( wp_unslash( $_GET['_wl_notice_nonce'] ) ), 'wordlift_hide_notices_nonce' ) ) {
-			wp_die( esc_html__( 'Action failed. Please refresh the page and retry.', 'wordlift' ) );
-		}
-
-		// If the user doesn't have the right privileges, return an error.
-		if ( ! current_user_can( 'manage_options' ) ) {
-			wp_die( esc_html__( 'Cheatin&#8217; huh?', 'wordlift' ) );
-		}
-
-		// Store a flag telling to skip the wizard.
-		Wordlift_Configuration_Service::get_instance()->set_skip_wizard( true );
-
-	}
-
-	/**
-	 * Register the wizard page to be able to access it.
-	 *
-	 * @since    3.9.0
-	 */
-	public function admin_menu() {
-		/**
-		 *
-		 * @see https://github.com/insideout10/wordlift-plugin/issues/1209
-		 * @since 3.27.7
-		 */
-		// @todo: find another way to do this, since this is adding an empty space in WP's dashboard menu.
-		add_dashboard_page( '', '', 'manage_options', 'wl-setup', '' );
-	}
-
-	/**
-	 * Displays the wizard page.
-	 *
-	 * @since    3.9.0
-	 */
-	public function show_page() {
-
-		// First check if we are in the wizard page at all, if not do nothing.
-		if ( empty( $_GET['page'] ) || 'wl-setup' !== $_GET['page'] ) {
-			return;
-		}
-
-		// If it's a POST and the `wl-save-configuration` action is set, save the configuration.
-		if ( isset( $_POST['action'] ) && 'wl-save-configuration' === $_POST['action'] ) {
-
-			// Check the nonce and the user capabilities.
-			check_admin_referer( 'wl-save-configuration' );
-
-			// Check if the user has the right privileges.
-			if ( ! current_user_can( 'manage_options' ) ) {
-				wp_die( esc_html__( 'Sorry, you do not have a permission to save the settings', 'wordlift' ) );
-			}
-
-			// Save the configuration.
-			$this->save_configuration( $_POST );
-
-			// Redirect to the admin's page.
-			wp_safe_redirect( admin_url() );
-			exit();
-		}
+        }
+
+    }
+
+    /**
+     * Handle hiding the wizard notices by user request.
+     *
+     * @since    3.9.0
+     */
+    public function hide_notices() {
+
+        // If it's not a `wl-hide-notice` or the nonce is not set, return.
+        if ( ! isset( $_GET['wl-hide-notice'], $_GET['_wl_notice_nonce'] ) ) {
+            return;
+        }
+
+        // If the nonce is invalid, return an error.
+        if ( ! wp_verify_nonce( sanitize_text_field( wp_unslash( $_GET['_wl_notice_nonce'] ) ), 'wordlift_hide_notices_nonce' ) ) {
+            wp_die( esc_html__( 'Action failed. Please refresh the page and retry.', 'wordlift' ) );
+        }
+
+        // If the user doesn't have the right privileges, return an error.
+        if ( ! current_user_can( 'manage_options' ) ) {
+            wp_die( esc_html__( 'Cheatin&#8217; huh?', 'wordlift' ) );
+        }
+
+        // Store a flag telling to skip the wizard.
+        Wordlift_Configuration_Service::get_instance()->set_skip_wizard( true );
+
+    }
+
+    /**
+     * Register the wizard page to be able to access it.
+     *
+     * @since    3.9.0
+     */
+    public function admin_menu() {
+        /**
+         *
+         * @see https://github.com/insideout10/wordlift-plugin/issues/1209
+         * @since 3.27.7
+         */
+        // @todo: find another way to do this, since this is adding an empty space in WP's dashboard menu.
+        add_dashboard_page( '', '', 'manage_options', 'wl-setup', '' );
+    }
+
+    /**
+     * Displays the wizard page.
+     *
+     * @since    3.9.0
+     */
+    public function show_page() {
+
+        // First check if we are in the wizard page at all, if not do nothing.
+        if ( empty( $_GET['page'] ) || 'wl-setup' !== $_GET['page'] ) {
+            return;
+        }
+
+        // If it's a POST and the `wl-save-configuration` action is set, save the configuration.
+        if ( isset( $_POST['action'] ) && 'wl-save-configuration' === $_POST['action'] ) {
+
+            // Check the nonce and the user capabilities.
+            check_admin_referer( 'wl-save-configuration' );
+
+            // Check if the user has the right privileges.
+            if ( ! current_user_can( 'manage_options' ) ) {
+                wp_die( esc_html__( 'Sorry, you do not have a permission to save the settings', 'wordlift' ) );
+            }
+
+            // Save the configuration.
+            $this->save_configuration( $_POST );
+
+            // Redirect to the admin's page.
+            wp_safe_redirect( admin_url() );
+            exit();
+        }
 
         // phpcs:ignore VariableAnalysis.CodeAnalysis.VariableAnalysis.UnusedVariable
-		$language_select = $this->language_select_element;
-		// phpcs:ignore VariableAnalysis.CodeAnalysis.VariableAnalysis.UnusedVariable
-		$country_select = $this->country_select_element;
+        $language_select = $this->language_select_element;
+        // phpcs:ignore VariableAnalysis.CodeAnalysis.VariableAnalysis.UnusedVariable
+        $country_select = $this->country_select_element;
 
-		include plugin_dir_path( __DIR__ ) . 'admin/partials/wordlift-admin-setup.php';
+        include plugin_dir_path( __DIR__ ) . 'admin/partials/wordlift-admin-setup.php';
 
-		exit;
-	}
+        exit;
+    }
 
-	/**
-	 * Save WordLift's configuration using the provided parameters.
-	 *
-	 * @param array $params An array of configuration parameters.
-	 *
-	 * @since 3.9.0
-	 */
-	public function save_configuration( $params ) {
+    /**
+     * Save WordLift's configuration using the provided parameters.
+     *
+     * @param array $params An array of configuration parameters.
+     *
+     * @since 3.9.0
+     */
+    public function save_configuration( $params ) {
 
-		// We have the following parameters:
-		// `key`, holding WL's key,
-		// `vocabulary`, holding the vocabulary path,
-		// `wl-country-code` with the country code (e.g. `us`),
-		// `share-diagnostic`, the user preferences about sharing data with us.
-		// `user_type`, the user type either `personal` or `company`,
-		// `name`, with the `personal` or `company`'s name,
-		// `logo`, the attachment id for the `personal` or `company` entity.
+        // We have the following parameters:
+        // `key`, holding WL's key,
+        // `vocabulary`, holding the vocabulary path,
+        // `wl-country-code` with the country code (e.g. `us`),
+        // `share-diagnostic`, the user preferences about sharing data with us.
+        // `user_type`, the user type either `personal` or `company`,
+        // `name`, with the `personal` or `company`'s name,
+        // `logo`, the attachment id for the `personal` or `company` entity.
 
-		// Store the key:
-		Wordlift_Configuration_Service::get_instance()->set_key( $params['key'] );
+        // Store the key:
+        Wordlift_Configuration_Service::get_instance()->set_key( $params['key'] );
 
-		// Store the vocabulary path:
-		Wordlift_Configuration_Service::get_instance()->set_entity_base_path( $params['vocabulary'] );
+        // Store the vocabulary path:
+        Wordlift_Configuration_Service::get_instance()->set_entity_base_path( $params['vocabulary'] );
 
-		// Store the site's country:
-		Wordlift_Configuration_Service::get_instance()->set_country_code( $params['wl-country-code'] );
+        // Store the site's country:
+        Wordlift_Configuration_Service::get_instance()->set_country_code( $params['wl-country-code'] );
 
-		// Store the preferences in variable, because if the checkbox is not checked
-		// the `share-diagnostic` will not exists in `$params` array.
-		$share_diagnostic_preferences = empty( $params['share-diagnostic'] ) ? 'no' : 'yes';
+        // Store the preferences in variable, because if the checkbox is not checked
+        // the `share-diagnostic` will not exists in `$params` array.
+        $share_diagnostic_preferences = empty( $params['share-diagnostic'] ) ? 'no' : 'yes';
 
-		// Store the diagnostic preferences:
-		Wordlift_Configuration_Service::get_instance()->set_diagnostic_preferences( $share_diagnostic_preferences );
+        // Store the diagnostic preferences:
+        Wordlift_Configuration_Service::get_instance()->set_diagnostic_preferences( $share_diagnostic_preferences );
 
-		// Set the type URI, either http://schema.org/Person or http://schema.org/Organization.
-		$type_uri = sprintf( 'http://schema.org/%s', 'organization' === $params['user_type'] ? 'Organization' : 'Person' );
+        // Set the type URI, either http://schema.org/Person or http://schema.org/Organization.
+        $type_uri = sprintf( 'http://schema.org/%s', 'organization' === $params['user_type'] ? 'Organization' : 'Person' );
 
-		// Create an entity for the publisher.
-		$publisher_post_id = $this->entity_service->create( $params['name'], $type_uri, $params['logo'], 'publish' );
+        // Create an entity for the publisher.
+        $publisher_post_id = $this->entity_service->create( $params['name'], $type_uri, $params['logo'], 'publish' );
 
-		// Store the publisher entity post id in the configuration.
-		Wordlift_Configuration_Service::get_instance()->set_publisher_id( $publisher_post_id );
+        // Store the publisher entity post id in the configuration.
+        Wordlift_Configuration_Service::get_instance()->set_publisher_id( $publisher_post_id );
 
-		flush_rewrite_rules(); // Needed because of possible change to the entity base path.
+        flush_rewrite_rules(); // Needed because of possible change to the entity base path.
 
-	}
+    }
 
 }

Spacing +41 added lines, -41 removed lines

@@ -12,7 +12,7 @@ 
  * @since      3.9.0
  */
 
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -74,7 +74,7 @@ 
 	 *
 	 * @since    3.9.0
 	 */
-	public function __construct( $key_validation_service, $entity_service, $language_select_element, $country_select_element ) {
+	public function __construct($key_validation_service, $entity_service, $language_select_element, $country_select_element) {
 
 		// Set a reference to the key validation service.
 		$this->key_validation_service = $key_validation_service;
@@ -88,13 +88,13 @@ 
 
 		// Hook to some WP's events:
 		// When WP is loaded check whether the user decided to skip the set-up, i.e. don't show us even if WL is not set up.
-		add_action( 'wp_loaded', array( $this, 'hide_notices' ) );
+		add_action('wp_loaded', array($this, 'hide_notices'));
 
 		// Hook to `admin_menu` in order to add our own setup wizard page.
-		add_action( 'admin_menu', array( $this, 'admin_menu' ) );
+		add_action('admin_menu', array($this, 'admin_menu'));
 
 		// Triggered when the user accesses the admin area, we decide whether to show our own wizard.
-		add_action( 'admin_init', array( $this, 'show_page' ) );
+		add_action('admin_init', array($this, 'show_page'));
 
 		/**
 		 * Filter: wl_feature__enable__notices.
@@ -104,9 +104,9 @@ 
 		 * @return bool
 		 * @since 3.27.6
 		 */
-		if ( apply_filters( 'wl_feature__enable__notices', true ) ) {
+		if (apply_filters('wl_feature__enable__notices', true)) {
 			// Hook to `admin_notices` to display our notices.
-			add_action( 'admin_notices', array( $this, 'admin_notices' ) );
+			add_action('admin_notices', array($this, 'admin_notices'));
 		}
 	}
 
@@ -118,21 +118,21 @@ 
 	public function admin_init() {
 
 		// If the `_wl_activation_redirect` is set, the redirect to the setup page.
-		if ( get_transient( '_wl_activation_redirect' ) ) {
-			delete_transient( '_wl_activation_redirect' );
+		if (get_transient('_wl_activation_redirect')) {
+			delete_transient('_wl_activation_redirect');
 
 			// If the user asked to skip the wizard then comply.
-			if ( Wordlift_Configuration_Service::get_instance()->is_skip_wizard() ) {
+			if (Wordlift_Configuration_Service::get_instance()->is_skip_wizard()) {
 				return;
 			}
 
 			// If we're already on the page or the user doesn't have permissions, return.
-			if ( ( ! empty( $_GET['page'] ) && 'wl-setup' === $_GET['page'] ) || is_network_admin() || isset( $_GET['activate-multi'] ) || ! current_user_can( 'manage_options' ) ) { //phpcs:ignore WordPress.Security.NonceVerification.Recommended
+			if (( ! empty($_GET['page']) && 'wl-setup' === $_GET['page']) || is_network_admin() || isset($_GET['activate-multi']) || ! current_user_can('manage_options')) { //phpcs:ignore WordPress.Security.NonceVerification.Recommended
 				return;
 			}
 
 			// Finally redirect to the setup page.
-			wp_safe_redirect( admin_url( 'index.php?page=wl-setup' ) );
+			wp_safe_redirect(admin_url('index.php?page=wl-setup'));
 
 			exit;
 		}
@@ -147,14 +147,14 @@ 
 	public function admin_notices() {
 
 		// Use `wl_configuration_get_key` to check whether WL's key is set and that the user didn't disable the wizard.
-		if ( '' === Wordlift_Configuration_Service::get_instance()->get_key() && ! Wordlift_Configuration_Service::get_instance()->is_skip_wizard() ) { ?>
+		if ('' === Wordlift_Configuration_Service::get_instance()->get_key() && ! Wordlift_Configuration_Service::get_instance()->is_skip_wizard()) { ?>
 			<div id="wl-message" class="updated">
-				<p><?php esc_html_e( 'Welcome to WordLift &#8211; You&lsquo;re almost ready to start', 'wordlift' ); ?></p>
+				<p><?php esc_html_e('Welcome to WordLift &#8211; You&lsquo;re almost ready to start', 'wordlift'); ?></p>
 				<p class="submit">
-					<a href="<?php echo esc_url( admin_url( 'admin.php?page=wl-setup' ) ); ?>"
-					   class="button-primary"><?php esc_html_e( 'Run the Setup Wizard', 'wordlift' ); ?></a>
+					<a href="<?php echo esc_url(admin_url('admin.php?page=wl-setup')); ?>"
+					   class="button-primary"><?php esc_html_e('Run the Setup Wizard', 'wordlift'); ?></a>
 					<a class="button-secondary skip"
-					   href="<?php echo esc_url( wp_nonce_url( add_query_arg( 'wl-hide-notice', 'install' ), 'wordlift_hide_notices_nonce', '_wl_notice_nonce' ) ); ?>"><?php esc_html_e( 'Skip Setup', 'wordlift' ); ?></a>
+					   href="<?php echo esc_url(wp_nonce_url(add_query_arg('wl-hide-notice', 'install'), 'wordlift_hide_notices_nonce', '_wl_notice_nonce')); ?>"><?php esc_html_e('Skip Setup', 'wordlift'); ?></a>
 				</p>
 			</div>
 			<?php
@@ -170,22 +170,22 @@ 
 	public function hide_notices() {
 
 		// If it's not a `wl-hide-notice` or the nonce is not set, return.
-		if ( ! isset( $_GET['wl-hide-notice'], $_GET['_wl_notice_nonce'] ) ) {
+		if ( ! isset($_GET['wl-hide-notice'], $_GET['_wl_notice_nonce'])) {
 			return;
 		}
 
 		// If the nonce is invalid, return an error.
-		if ( ! wp_verify_nonce( sanitize_text_field( wp_unslash( $_GET['_wl_notice_nonce'] ) ), 'wordlift_hide_notices_nonce' ) ) {
-			wp_die( esc_html__( 'Action failed. Please refresh the page and retry.', 'wordlift' ) );
+		if ( ! wp_verify_nonce(sanitize_text_field(wp_unslash($_GET['_wl_notice_nonce'])), 'wordlift_hide_notices_nonce')) {
+			wp_die(esc_html__('Action failed. Please refresh the page and retry.', 'wordlift'));
 		}
 
 		// If the user doesn't have the right privileges, return an error.
-		if ( ! current_user_can( 'manage_options' ) ) {
-			wp_die( esc_html__( 'Cheatin&#8217; huh?', 'wordlift' ) );
+		if ( ! current_user_can('manage_options')) {
+			wp_die(esc_html__('Cheatin&#8217; huh?', 'wordlift'));
 		}
 
 		// Store a flag telling to skip the wizard.
-		Wordlift_Configuration_Service::get_instance()->set_skip_wizard( true );
+		Wordlift_Configuration_Service::get_instance()->set_skip_wizard(true);
 
 	}
 
@@ -201,7 +201,7 @@ 
 		 * @since 3.27.7
 		 */
 		// @todo: find another way to do this, since this is adding an empty space in WP's dashboard menu.
-		add_dashboard_page( '', '', 'manage_options', 'wl-setup', '' );
+		add_dashboard_page('', '', 'manage_options', 'wl-setup', '');
 	}
 
 	/**
@@ -212,26 +212,26 @@ 
 	public function show_page() {
 
 		// First check if we are in the wizard page at all, if not do nothing.
-		if ( empty( $_GET['page'] ) || 'wl-setup' !== $_GET['page'] ) {
+		if (empty($_GET['page']) || 'wl-setup' !== $_GET['page']) {
 			return;
 		}
 
 		// If it's a POST and the `wl-save-configuration` action is set, save the configuration.
-		if ( isset( $_POST['action'] ) && 'wl-save-configuration' === $_POST['action'] ) {
+		if (isset($_POST['action']) && 'wl-save-configuration' === $_POST['action']) {
 
 			// Check the nonce and the user capabilities.
-			check_admin_referer( 'wl-save-configuration' );
+			check_admin_referer('wl-save-configuration');
 
 			// Check if the user has the right privileges.
-			if ( ! current_user_can( 'manage_options' ) ) {
-				wp_die( esc_html__( 'Sorry, you do not have a permission to save the settings', 'wordlift' ) );
+			if ( ! current_user_can('manage_options')) {
+				wp_die(esc_html__('Sorry, you do not have a permission to save the settings', 'wordlift'));
 			}
 
 			// Save the configuration.
-			$this->save_configuration( $_POST );
+			$this->save_configuration($_POST);
 
 			// Redirect to the admin's page.
-			wp_safe_redirect( admin_url() );
+			wp_safe_redirect(admin_url());
 			exit();
 		}
 
@@ -240,7 +240,7 @@ 
 		// phpcs:ignore VariableAnalysis.CodeAnalysis.VariableAnalysis.UnusedVariable
 		$country_select = $this->country_select_element;
 
-		include plugin_dir_path( __DIR__ ) . 'admin/partials/wordlift-admin-setup.php';
+		include plugin_dir_path(__DIR__).'admin/partials/wordlift-admin-setup.php';
 
 		exit;
 	}
@@ -252,7 +252,7 @@ 
 	 *
 	 * @since 3.9.0
 	 */
-	public function save_configuration( $params ) {
+	public function save_configuration($params) {
 
 		// We have the following parameters:
 		// `key`, holding WL's key,
@@ -264,29 +264,29 @@ 
 		// `logo`, the attachment id for the `personal` or `company` entity.
 
 		// Store the key:
-		Wordlift_Configuration_Service::get_instance()->set_key( $params['key'] );
+		Wordlift_Configuration_Service::get_instance()->set_key($params['key']);
 
 		// Store the vocabulary path:
-		Wordlift_Configuration_Service::get_instance()->set_entity_base_path( $params['vocabulary'] );
+		Wordlift_Configuration_Service::get_instance()->set_entity_base_path($params['vocabulary']);
 
 		// Store the site's country:
-		Wordlift_Configuration_Service::get_instance()->set_country_code( $params['wl-country-code'] );
+		Wordlift_Configuration_Service::get_instance()->set_country_code($params['wl-country-code']);
 
 		// Store the preferences in variable, because if the checkbox is not checked
 		// the `share-diagnostic` will not exists in `$params` array.
-		$share_diagnostic_preferences = empty( $params['share-diagnostic'] ) ? 'no' : 'yes';
+		$share_diagnostic_preferences = empty($params['share-diagnostic']) ? 'no' : 'yes';
 
 		// Store the diagnostic preferences:
-		Wordlift_Configuration_Service::get_instance()->set_diagnostic_preferences( $share_diagnostic_preferences );
+		Wordlift_Configuration_Service::get_instance()->set_diagnostic_preferences($share_diagnostic_preferences);
 
 		// Set the type URI, either http://schema.org/Person or http://schema.org/Organization.
-		$type_uri = sprintf( 'http://schema.org/%s', 'organization' === $params['user_type'] ? 'Organization' : 'Person' );
+		$type_uri = sprintf('http://schema.org/%s', 'organization' === $params['user_type'] ? 'Organization' : 'Person');
 
 		// Create an entity for the publisher.
-		$publisher_post_id = $this->entity_service->create( $params['name'], $type_uri, $params['logo'], 'publish' );
+		$publisher_post_id = $this->entity_service->create($params['name'], $type_uri, $params['logo'], 'publish');
 
 		// Store the publisher entity post id in the configuration.
-		Wordlift_Configuration_Service::get_instance()->set_publisher_id( $publisher_post_id );
+		Wordlift_Configuration_Service::get_instance()->set_publisher_id($publisher_post_id);
 
 		flush_rewrite_rules(); // Needed because of possible change to the entity base path.
 

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

Indentation +189 added lines, -189 removed lines

@@ -7,194 +7,194 @@
  */
 class Wordlift_Notice_Service {
 
-	/**
-	 * The template used to display notices. The <em>notice dismissible</em> style classes make this notice dismissible
-	 * on the WordPress UI (via a small X button on the right side of the notice).
-	 *
-	 * @since 3.2.0
-	 */
-	const TEMPLATE = '<div class="wl-notice notice is-dismissible %s"><p>%s</p></div>';
-
-	/**
-	 * The standard WordPress <em>update</em> style class.
-	 *
-	 * @since 3.2.0
-	 */
-	const UPDATE = 'update';
-
-	/**
-	 * The standard WordPress <em>update-nag</em> style class.
-	 *
-	 * @since 3.2.0
-	 */
-	const UPDATE_NAG = 'update-nag';
-
-	/**
-	 * The standard WordPress <em>error</em> style class.
-	 *
-	 * @since 3.2.0
-	 */
-	const ERROR = 'error';
-
-	/**
-	 * A custom WordLift css style class used for WordLift suggestions.
-	 *
-	 * @since 3.3.0
-	 */
-	const SUGGESTION = 'wl-suggestion';
-
-	/**
-	 * The array of notices.
-	 *
-	 * @since 3.2.0
-	 * @access private
-	 * @var array $notices The array of notices.
-	 */
-	private $notices = array();
-
-	/**
-	 * A singleton instance of the Notice service.
-	 *
-	 * @since 3.2.0
-	 * @access private
-	 * @var \Wordlift_Notice_Service $instance A singleton instance of the Notice service.
-	 */
-	private static $instance;
-
-	/**
-	 * Create an instance of the Notice service.
-	 *
-	 * @since 3.2.0
-	 */
-	public function __construct() {
-		/**
-		 * 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 ) ) {
-			// Hook to be called when to display notices.
-			add_action( 'admin_notices', array( $this, 'admin_notices' ) );
-		}
-		self::$instance = $this;
-
-	}
-
-	/**
-	 * Get the singleton instance of the Notice service.
-	 *
-	 * @return \Wordlift_Notice_Service The singleton instance of the Notice service.
-	 * @since 3.2.0
-	 */
-	public static function get_instance() {
-
-		return self::$instance;
-	}
-
-	/**
-	 * Add a notice.
-	 *
-	 * @param string $class The css class.
-	 * @param string $message The message.
-	 *
-	 * @since 3.2.0
-	 */
-	public function add( $class, $message ) {
-
-		$this->notices[] = sprintf( self::TEMPLATE, $class, $this->transform( $message ) );
-
-	}
-
-	/**
-	 * Add an update notice (message with a white background and a green left border).
-	 *
-	 * @param string $message The message to display.
-	 *
-	 * @since 3.2.0
-	 */
-	public function add_update( $message ) {
-
-		$this->add( self::UPDATE, $message );
-
-	}
-
-	/**
-	 * Add an update nag notice (message with a white background and a yellow left border).
-	 *
-	 * @param string $message The message to display.
-	 *
-	 * @since 3.2.0
-	 */
-	public function add_update_nag( $message ) {
-
-		$this->add( self::UPDATE_NAG, $message );
-
-	}
-
-	/**
-	 * Add an error notice (message with a white background and a red left border).
-	 *
-	 * @param string $message The message to display.
-	 *
-	 * @since 3.2.0
-	 */
-	public function add_error( $message ) {
-
-		$this->add( self::ERROR, $message );
-
-	}
-
-	/**
-	 * Add a suggestion notice (message with a white background and a WordLift brand colored left border).
-	 *
-	 * @param string $message The message to display.
-	 *
-	 * @since 3.3.0
-	 */
-	public function add_suggestion( $message ) {
-
-		$this->add( self::SUGGESTION, $message );
-
-	}
-
-	/**
-	 * Print out the notices when the admin_notices action is called.
-	 *
-	 * @since 3.2.0
-	 */
-	public function admin_notices() {
-
-		foreach ( $this->notices as $notice ) {
-			echo wp_kses(
-				$notice,
-				array(
-					'div' => array( 'class' => array() ),
-					'p'   => array(),
-				)
-			);
-		}
-
-	}
-
-	/**
-	 * Transform message depending on message type. Return a string
-	 *
-	 * @param string $message The message.
-	 *
-	 * @since 3.3.0
-	 */
-	private function transform( $message ) {
-
-		switch ( gettype( $message ) ) {
-			case 'array':
-				return implode( '<br />', $message );
-			default:
-				return $message;
-		}
-
-	}
+    /**
+     * The template used to display notices. The <em>notice dismissible</em> style classes make this notice dismissible
+     * on the WordPress UI (via a small X button on the right side of the notice).
+     *
+     * @since 3.2.0
+     */
+    const TEMPLATE = '<div class="wl-notice notice is-dismissible %s"><p>%s</p></div>';
+
+    /**
+     * The standard WordPress <em>update</em> style class.
+     *
+     * @since 3.2.0
+     */
+    const UPDATE = 'update';
+
+    /**
+     * The standard WordPress <em>update-nag</em> style class.
+     *
+     * @since 3.2.0
+     */
+    const UPDATE_NAG = 'update-nag';
+
+    /**
+     * The standard WordPress <em>error</em> style class.
+     *
+     * @since 3.2.0
+     */
+    const ERROR = 'error';
+
+    /**
+     * A custom WordLift css style class used for WordLift suggestions.
+     *
+     * @since 3.3.0
+     */
+    const SUGGESTION = 'wl-suggestion';
+
+    /**
+     * The array of notices.
+     *
+     * @since 3.2.0
+     * @access private
+     * @var array $notices The array of notices.
+     */
+    private $notices = array();
+
+    /**
+     * A singleton instance of the Notice service.
+     *
+     * @since 3.2.0
+     * @access private
+     * @var \Wordlift_Notice_Service $instance A singleton instance of the Notice service.
+     */
+    private static $instance;
+
+    /**
+     * Create an instance of the Notice service.
+     *
+     * @since 3.2.0
+     */
+    public function __construct() {
+        /**
+         * 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 ) ) {
+            // Hook to be called when to display notices.
+            add_action( 'admin_notices', array( $this, 'admin_notices' ) );
+        }
+        self::$instance = $this;
+
+    }
+
+    /**
+     * Get the singleton instance of the Notice service.
+     *
+     * @return \Wordlift_Notice_Service The singleton instance of the Notice service.
+     * @since 3.2.0
+     */
+    public static function get_instance() {
+
+        return self::$instance;
+    }
+
+    /**
+     * Add a notice.
+     *
+     * @param string $class The css class.
+     * @param string $message The message.
+     *
+     * @since 3.2.0
+     */
+    public function add( $class, $message ) {
+
+        $this->notices[] = sprintf( self::TEMPLATE, $class, $this->transform( $message ) );
+
+    }
+
+    /**
+     * Add an update notice (message with a white background and a green left border).
+     *
+     * @param string $message The message to display.
+     *
+     * @since 3.2.0
+     */
+    public function add_update( $message ) {
+
+        $this->add( self::UPDATE, $message );
+
+    }
+
+    /**
+     * Add an update nag notice (message with a white background and a yellow left border).
+     *
+     * @param string $message The message to display.
+     *
+     * @since 3.2.0
+     */
+    public function add_update_nag( $message ) {
+
+        $this->add( self::UPDATE_NAG, $message );
+
+    }
+
+    /**
+     * Add an error notice (message with a white background and a red left border).
+     *
+     * @param string $message The message to display.
+     *
+     * @since 3.2.0
+     */
+    public function add_error( $message ) {
+
+        $this->add( self::ERROR, $message );
+
+    }
+
+    /**
+     * Add a suggestion notice (message with a white background and a WordLift brand colored left border).
+     *
+     * @param string $message The message to display.
+     *
+     * @since 3.3.0
+     */
+    public function add_suggestion( $message ) {
+
+        $this->add( self::SUGGESTION, $message );
+
+    }
+
+    /**
+     * Print out the notices when the admin_notices action is called.
+     *
+     * @since 3.2.0
+     */
+    public function admin_notices() {
+
+        foreach ( $this->notices as $notice ) {
+            echo wp_kses(
+                $notice,
+                array(
+                    'div' => array( 'class' => array() ),
+                    'p'   => array(),
+                )
+            );
+        }
+
+    }
+
+    /**
+     * Transform message depending on message type. Return a string
+     *
+     * @param string $message The message.
+     *
+     * @since 3.3.0
+     */
+    private function transform( $message ) {
+
+        switch ( gettype( $message ) ) {
+            case 'array':
+                return implode( '<br />', $message );
+            default:
+                return $message;
+        }
+
+    }
 
 }

Spacing +17 added lines, -17 removed lines

@@ -75,9 +75,9 @@ 
 		 * @return bool
 		 * @since 3.27.6
 		 */
-		if ( apply_filters( 'wl_feature__enable__notices', true ) ) {
+		if (apply_filters('wl_feature__enable__notices', true)) {
 			// Hook to be called when to display notices.
-			add_action( 'admin_notices', array( $this, 'admin_notices' ) );
+			add_action('admin_notices', array($this, 'admin_notices'));
 		}
 		self::$instance = $this;
 
@@ -102,9 +102,9 @@ 
 	 *
 	 * @since 3.2.0
 	 */
-	public function add( $class, $message ) {
+	public function add($class, $message) {
 
-		$this->notices[] = sprintf( self::TEMPLATE, $class, $this->transform( $message ) );
+		$this->notices[] = sprintf(self::TEMPLATE, $class, $this->transform($message));
 
 	}
 
@@ -115,9 +115,9 @@ 
 	 *
 	 * @since 3.2.0
 	 */
-	public function add_update( $message ) {
+	public function add_update($message) {
 
-		$this->add( self::UPDATE, $message );
+		$this->add(self::UPDATE, $message);
 
 	}
 
@@ -128,9 +128,9 @@ 
 	 *
 	 * @since 3.2.0
 	 */
-	public function add_update_nag( $message ) {
+	public function add_update_nag($message) {
 
-		$this->add( self::UPDATE_NAG, $message );
+		$this->add(self::UPDATE_NAG, $message);
 
 	}
 
@@ -141,9 +141,9 @@ 
 	 *
 	 * @since 3.2.0
 	 */
-	public function add_error( $message ) {
+	public function add_error($message) {
 
-		$this->add( self::ERROR, $message );
+		$this->add(self::ERROR, $message);
 
 	}
 
@@ -154,9 +154,9 @@ 
 	 *
 	 * @since 3.3.0
 	 */
-	public function add_suggestion( $message ) {
+	public function add_suggestion($message) {
 
-		$this->add( self::SUGGESTION, $message );
+		$this->add(self::SUGGESTION, $message);
 
 	}
 
@@ -167,11 +167,11 @@ 
 	 */
 	public function admin_notices() {
 
-		foreach ( $this->notices as $notice ) {
+		foreach ($this->notices as $notice) {
 			echo wp_kses(
 				$notice,
 				array(
-					'div' => array( 'class' => array() ),
+					'div' => array('class' => array()),
 					'p'   => array(),
 				)
 			);
@@ -186,11 +186,11 @@ 
 	 *
 	 * @since 3.3.0
 	 */
-	private function transform( $message ) {
+	private function transform($message) {
 
-		switch ( gettype( $message ) ) {
+		switch (gettype($message)) {
 			case 'array':
-				return implode( '<br />', $message );
+				return implode('<br />', $message);
 			default:
 				return $message;
 		}

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

Indentation +128 added lines, -128 removed lines

@@ -18,133 +18,133 @@
  */
 abstract class Wordlift_Admin_Page {
 
-	/**
-	 * Define the {@link Wordlift_Admin_Page} constructor.
-	 *
-	 * @since 3.20.0
-	 */
-	public function __construct() {
-		add_action( 'admin_menu', array( $this, 'admin_menu' ), 10, 0 );
-	}
-
-	/**
-	 * Get the parent slug.
-	 *
-	 * @return string The parent slug (default 'wl_admin_menu').
-	 * @since 3.11.0
-	 */
-	protected function get_parent_slug() {
-
-		return 'wl_admin_menu';
-	}
-
-	/**
-	 * Get the required capability.
-	 *
-	 * @return string The capability (default 'manage_options').
-	 * @since 3.11.0
-	 */
-	protected function get_capability() {
-
-		return 'manage_options';
-	}
-
-	/**
-	 * Get the page title. Will be translated.
-	 *
-	 * @return string The page title.
-	 * @since 3.11.0
-	 */
-	abstract public function get_page_title();
-
-	/**
-	 * Get the menu title. Will be translated.
-	 *
-	 * @return string The menu title.
-	 * @since 3.11.0
-	 */
-	abstract public function get_menu_title();
-
-	/**
-	 * Get the menu slug.
-	 *
-	 * @return string The menu slug.
-	 * @since 3.11.0
-	 */
-	abstract public function get_menu_slug();
-
-	/**
-	 * Get the page url.
-	 *
-	 * @return string The escaped url of the admin page
-	 * @since 3.14.0
-	 */
-	public function get_url() {
-
-		// ideally should have used menu_page_url, but it is loaded later than some usages.
-		$url = admin_url( 'admin.php?page=' . $this->get_menu_slug() );
-
-		return esc_url( $url );
-	}
-
-	/**
-	 * Get the partial file name, used in the {@link render} function.
-	 *
-	 * @return string The partial file name.
-	 * @since 3.11.0
-	 */
-	abstract public function get_partial_name();
-
-	/**
-	 * The `admin_menu` callback. Will call {@link add_submenu_page} to add the
-	 * page to the admin menu.
-	 *
-	 * @return false|string The resulting page's hook_suffix, or false if the user does not have the capability required.
-	 * @since 3.11.0
-	 */
-	public function admin_menu() {
-		// Add the sub-menu page.
-		//
-		// See http://codex.wordpress.org/Function_Reference/add_submenu_page
-		$page = add_submenu_page(
-			$this->get_parent_slug(),
-			$this->get_page_title(),
-			$this->get_menu_title(),
-			$this->get_capability(),                   // The required capability, provided by the calling hook.
-			$this->get_menu_slug(),
-			array( $this, 'render' )
-		);
-
-		// Set a hook to enqueue scripts only when the settings page is displayed.
-		add_action(
-			'admin_print_scripts-' . $page,
-			array(
-				$this,
-				'enqueue_scripts',
-			)
-		);
-
-		// Finally return the page hook_suffix.
-		return $page;
-	}
-
-	/**
-	 * Enqueue scripts for the specific page. Subclasses can override this function
-	 * to provide their own styles/scripts.
-	 *
-	 * @since 3.11.0
-	 */
-	public function enqueue_scripts() {
-	}
-
-	/**
-	 * Render the page.
-	 */
-	public function render() {
-
-		// Include the partial.
-		include plugin_dir_path( __FILE__ ) . 'partials/' . $this->get_partial_name();
-
-	}
+    /**
+     * Define the {@link Wordlift_Admin_Page} constructor.
+     *
+     * @since 3.20.0
+     */
+    public function __construct() {
+        add_action( 'admin_menu', array( $this, 'admin_menu' ), 10, 0 );
+    }
+
+    /**
+     * Get the parent slug.
+     *
+     * @return string The parent slug (default 'wl_admin_menu').
+     * @since 3.11.0
+     */
+    protected function get_parent_slug() {
+
+        return 'wl_admin_menu';
+    }
+
+    /**
+     * Get the required capability.
+     *
+     * @return string The capability (default 'manage_options').
+     * @since 3.11.0
+     */
+    protected function get_capability() {
+
+        return 'manage_options';
+    }
+
+    /**
+     * Get the page title. Will be translated.
+     *
+     * @return string The page title.
+     * @since 3.11.0
+     */
+    abstract public function get_page_title();
+
+    /**
+     * Get the menu title. Will be translated.
+     *
+     * @return string The menu title.
+     * @since 3.11.0
+     */
+    abstract public function get_menu_title();
+
+    /**
+     * Get the menu slug.
+     *
+     * @return string The menu slug.
+     * @since 3.11.0
+     */
+    abstract public function get_menu_slug();
+
+    /**
+     * Get the page url.
+     *
+     * @return string The escaped url of the admin page
+     * @since 3.14.0
+     */
+    public function get_url() {
+
+        // ideally should have used menu_page_url, but it is loaded later than some usages.
+        $url = admin_url( 'admin.php?page=' . $this->get_menu_slug() );
+
+        return esc_url( $url );
+    }
+
+    /**
+     * Get the partial file name, used in the {@link render} function.
+     *
+     * @return string The partial file name.
+     * @since 3.11.0
+     */
+    abstract public function get_partial_name();
+
+    /**
+     * The `admin_menu` callback. Will call {@link add_submenu_page} to add the
+     * page to the admin menu.
+     *
+     * @return false|string The resulting page's hook_suffix, or false if the user does not have the capability required.
+     * @since 3.11.0
+     */
+    public function admin_menu() {
+        // Add the sub-menu page.
+        //
+        // See http://codex.wordpress.org/Function_Reference/add_submenu_page
+        $page = add_submenu_page(
+            $this->get_parent_slug(),
+            $this->get_page_title(),
+            $this->get_menu_title(),
+            $this->get_capability(),                   // The required capability, provided by the calling hook.
+            $this->get_menu_slug(),
+            array( $this, 'render' )
+        );
+
+        // Set a hook to enqueue scripts only when the settings page is displayed.
+        add_action(
+            'admin_print_scripts-' . $page,
+            array(
+                $this,
+                'enqueue_scripts',
+            )
+        );
+
+        // Finally return the page hook_suffix.
+        return $page;
+    }
+
+    /**
+     * Enqueue scripts for the specific page. Subclasses can override this function
+     * to provide their own styles/scripts.
+     *
+     * @since 3.11.0
+     */
+    public function enqueue_scripts() {
+    }
+
+    /**
+     * Render the page.
+     */
+    public function render() {
+
+        // Include the partial.
+        include plugin_dir_path( __FILE__ ) . 'partials/' . $this->get_partial_name();
+
+    }
 
 }

Spacing +7 added lines, -7 removed lines

@@ -24,7 +24,7 @@ 
 	 * @since 3.20.0
 	 */
 	public function __construct() {
-		add_action( 'admin_menu', array( $this, 'admin_menu' ), 10, 0 );
+		add_action('admin_menu', array($this, 'admin_menu'), 10, 0);
 	}
 
 	/**
@@ -82,9 +82,9 @@ 
 	public function get_url() {
 
 		// ideally should have used menu_page_url, but it is loaded later than some usages.
-		$url = admin_url( 'admin.php?page=' . $this->get_menu_slug() );
+		$url = admin_url('admin.php?page='.$this->get_menu_slug());
 
-		return esc_url( $url );
+		return esc_url($url);
 	}
 
 	/**
@@ -110,14 +110,14 @@ 
 			$this->get_parent_slug(),
 			$this->get_page_title(),
 			$this->get_menu_title(),
-			$this->get_capability(),                   // The required capability, provided by the calling hook.
+			$this->get_capability(), // The required capability, provided by the calling hook.
 			$this->get_menu_slug(),
-			array( $this, 'render' )
+			array($this, 'render')
 		);
 
 		// Set a hook to enqueue scripts only when the settings page is displayed.
 		add_action(
-			'admin_print_scripts-' . $page,
+			'admin_print_scripts-'.$page,
 			array(
 				$this,
 				'enqueue_scripts',
@@ -143,7 +143,7 @@ 
 	public function render() {
 
 		// Include the partial.
-		include plugin_dir_path( __FILE__ ) . 'partials/' . $this->get_partial_name();
+		include plugin_dir_path(__FILE__).'partials/'.$this->get_partial_name();
 
 	}
 

src/admin/class-wordlift-admin.php

Indentation +338 added lines, -338 removed lines

@@ -30,393 +30,393 @@
  */
 class Wordlift_Admin {
 
-	/**
-	 * The singleton instance.
-	 *
-	 * @since 3.19.4
-	 * @access private
-	 * @var Wordlift_Admin $instance The singleton instance.
-	 */
-	private static $instance;
-	/**
-	 * The ID of this plugin.
-	 *
-	 * @since    1.0.0
-	 * @access   private
-	 * @var      string $plugin_name The ID of this plugin.
-	 */
-	private $plugin_name;
-	/**
-	 * The version of this plugin.
-	 *
-	 * @since    1.0.0
-	 * @access   private
-	 * @var      string $version The current version of this plugin.
-	 */
-	private $version;
-	/**
-	 * The {@link Wordlift_User_Service} instance.
-	 *
-	 * @since  3.14.0
-	 * @access private
-	 * @var Wordlift_User_Service $user_service The {@link Wordlift_User_Service} instance.
-	 */
-	private $user_service;
-	/**
-	 * The {@link Wordlift_Batch_Operation_Ajax_Adapter} instance.
-	 *
-	 * @since 3.20.0
-	 * @access private
-	 * @var Wordlift_Batch_Operation_Ajax_Adapter $sync_batch_operation_ajax_adapter The {@link Wordlift_Batch_Operation_Ajax_Adapter} instance.
-	 */
-	private $sync_batch_operation_ajax_adapter;
-
-	/**
-	 * Initialize the class and set its properties.
-	 *
-	 * @param string                  $plugin_name The name of this plugin.
-	 * @param string                  $version The version of this plugin.
-	 * @param Wordlift_Notice_Service $notice_service The notice service.
-	 * @param Wordlift_User_Service   $user_service The {@link Wordlift_User_Service} instance.
-	 *
-	 * @since  1.0.0
-	 */
-	public function __construct( $plugin_name, $version, $notice_service, $user_service ) {
-
-		$this->plugin_name = $plugin_name;
-		$this->version     = $version;
-
-		$this->user_service = $user_service;
-
-		$configuration_service = Wordlift_Configuration_Service::get_instance();
-		$dataset_uri           = $configuration_service->get_dataset_uri();
-		$key                   = $configuration_service->get_key();
-		$features_registry     = Features_Registry::get_instance();
-		if ( empty( $dataset_uri ) ) {
-			$settings_page = Wordlift_Admin_Settings_Page::get_instance();
-			if ( empty( $key ) ) {
-				/* translators: %s: The link to the settings page. */
-				$error = sprintf( esc_html__( "WordLift's key isn't set, please open the %s to set WordLift's key.", 'wordlift' ), '<a href="' . $settings_page->get_url() . '">' . esc_html__( 'settings page', 'wordlift' ) . '</a>' );
-			} else {
-				/* translators: %s: The link to the settings page. */
-				$error = sprintf( esc_html__( "WordLift's dataset URI is not configured: please open the %s to set WordLift's key again.", 'wordlift' ), '<a href="' . $settings_page->get_url() . '">' . esc_html__( 'settings page', 'wordlift' ) . '</a>' );
-			}
-			$notice_service->add_error( $error );
-		}
-
-		// Load additional code if we're in the admin UI.
-		if ( is_admin() ) {
-
-			// Require the PHP files for the next code fragment.
-			self::require_files();
-
-			/*
+    /**
+     * The singleton instance.
+     *
+     * @since 3.19.4
+     * @access private
+     * @var Wordlift_Admin $instance The singleton instance.
+     */
+    private static $instance;
+    /**
+     * The ID of this plugin.
+     *
+     * @since    1.0.0
+     * @access   private
+     * @var      string $plugin_name The ID of this plugin.
+     */
+    private $plugin_name;
+    /**
+     * The version of this plugin.
+     *
+     * @since    1.0.0
+     * @access   private
+     * @var      string $version The current version of this plugin.
+     */
+    private $version;
+    /**
+     * The {@link Wordlift_User_Service} instance.
+     *
+     * @since  3.14.0
+     * @access private
+     * @var Wordlift_User_Service $user_service The {@link Wordlift_User_Service} instance.
+     */
+    private $user_service;
+    /**
+     * The {@link Wordlift_Batch_Operation_Ajax_Adapter} instance.
+     *
+     * @since 3.20.0
+     * @access private
+     * @var Wordlift_Batch_Operation_Ajax_Adapter $sync_batch_operation_ajax_adapter The {@link Wordlift_Batch_Operation_Ajax_Adapter} instance.
+     */
+    private $sync_batch_operation_ajax_adapter;
+
+    /**
+     * Initialize the class and set its properties.
+     *
+     * @param string                  $plugin_name The name of this plugin.
+     * @param string                  $version The version of this plugin.
+     * @param Wordlift_Notice_Service $notice_service The notice service.
+     * @param Wordlift_User_Service   $user_service The {@link Wordlift_User_Service} instance.
+     *
+     * @since  1.0.0
+     */
+    public function __construct( $plugin_name, $version, $notice_service, $user_service ) {
+
+        $this->plugin_name = $plugin_name;
+        $this->version     = $version;
+
+        $this->user_service = $user_service;
+
+        $configuration_service = Wordlift_Configuration_Service::get_instance();
+        $dataset_uri           = $configuration_service->get_dataset_uri();
+        $key                   = $configuration_service->get_key();
+        $features_registry     = Features_Registry::get_instance();
+        if ( empty( $dataset_uri ) ) {
+            $settings_page = Wordlift_Admin_Settings_Page::get_instance();
+            if ( empty( $key ) ) {
+                /* translators: %s: The link to the settings page. */
+                $error = sprintf( esc_html__( "WordLift's key isn't set, please open the %s to set WordLift's key.", 'wordlift' ), '<a href="' . $settings_page->get_url() . '">' . esc_html__( 'settings page', 'wordlift' ) . '</a>' );
+            } else {
+                /* translators: %s: The link to the settings page. */
+                $error = sprintf( esc_html__( "WordLift's dataset URI is not configured: please open the %s to set WordLift's key again.", 'wordlift' ), '<a href="' . $settings_page->get_url() . '">' . esc_html__( 'settings page', 'wordlift' ) . '</a>' );
+            }
+            $notice_service->add_error( $error );
+        }
+
+        // Load additional code if we're in the admin UI.
+        if ( is_admin() ) {
+
+            // Require the PHP files for the next code fragment.
+            self::require_files();
+
+            /*
 			 * @since 3.24.2 This function isn't called anymore because it was causing the Block Category to
 			 * multiply in Block Editor.
 			 *
 			 * @see https://github.com/insideout10/wordlift-plugin/issues/1004
 			 */
-			// Add Wordlift custom block category.
-			// self::add_block_category();
+            // Add Wordlift custom block category.
+            // self::add_block_category();
 
-			new Wordlift_Dashboard_Latest_News();
+            new Wordlift_Dashboard_Latest_News();
 
-			/*
+            /*
 			 * Add support for `All Entity Types`.
 			 *
 			 * @since 3.20.0
 			 *
 			 * @see https://github.com/insideout10/wordlift-plugin/issues/835
 			 */
-			// phpcs:ignore WordPress.NamingConventions.ValidHookName.UseUnderscores
-			if ( apply_filters( 'wl_feature__enable__all-entity-types', WL_ALL_ENTITY_TYPES ) ) {
+            // phpcs:ignore WordPress.NamingConventions.ValidHookName.UseUnderscores
+            if ( apply_filters( 'wl_feature__enable__all-entity-types', WL_ALL_ENTITY_TYPES ) ) {
 
-				require_once plugin_dir_path( __DIR__ ) . 'admin/class-wordlift-admin-schemaorg-taxonomy-metabox.php';
-				/*
+                require_once plugin_dir_path( __DIR__ ) . 'admin/class-wordlift-admin-schemaorg-taxonomy-metabox.php';
+                /*
 				 * The `Mappings` admin page.
 				 */
-				require_once plugin_dir_path( __DIR__ ) . 'admin/class-wordlift-admin-mappings-page.php';
+                require_once plugin_dir_path( __DIR__ ) . 'admin/class-wordlift-admin-mappings-page.php';
 
-				new Wordlift_Admin_Mappings_Page();
+                new Wordlift_Admin_Mappings_Page();
 
-				/*
+                /*
 				 * Allow sync'ing the schema.org taxonomy with the schema.org json file.
 				 *
 				 * @since 3.20.0
 				 */
-				require_once plugin_dir_path( __DIR__ ) . 'includes/schemaorg/class-wordlift-schemaorg-sync-batch-operation.php';
+                require_once plugin_dir_path( __DIR__ ) . 'includes/schemaorg/class-wordlift-schemaorg-sync-batch-operation.php';
 
-				$this->sync_batch_operation_ajax_adapter = new Wordlift_Batch_Operation_Ajax_Adapter( new Wordlift_Schemaorg_Sync_Batch_Operation(), 'wl_schemaorg_sync' );
+                $this->sync_batch_operation_ajax_adapter = new Wordlift_Batch_Operation_Ajax_Adapter( new Wordlift_Schemaorg_Sync_Batch_Operation(), 'wl_schemaorg_sync' );
 
-			}
+            }
 
-			/*
+            /*
 			 * Add the {@link Wordlift_Admin_Term_Adapter}.
 			 *
 			 * @since 3.20.0
 			 */
-			require_once plugin_dir_path( __DIR__ ) . 'admin/class-wordlift-admin-term-adapter.php';
-			new Wordlift_Admin_Term_Adapter();
+            require_once plugin_dir_path( __DIR__ ) . 'admin/class-wordlift-admin-term-adapter.php';
+            new Wordlift_Admin_Term_Adapter();
 
-			/*
+            /*
 			 * The new dashboard.
 			 *
 			 * @since 3.20.0
 			 *
 			 * @see https://github.com/insideout10/wordlift-plugin/issues/879
 			 */
-			new Wordlift_Admin_Dashboard_V2(
-				Wordlift_Entity_Service::get_instance()
-			);
-			new Wordlift_Admin_Not_Enriched_Filter();
-
-		}
-
-		// @@todo only load this class if ACF is available.
-		// Add support for ACF mappings, so that the admin edit mappings page can pick up ACF support when ACF is available.
-		new Acf_Mappings();
-
-		// Add the Mappings' REST Controller.
-		new Mappings_REST_Controller();
-
-		$features_registry->register_feature_from_slug(
-			'mappings',
-			( defined( 'WL_ENABLE_MAPPINGS' ) && WL_ENABLE_MAPPINGS ),
-			array( $this, 'init_mappings' )
-		);
-
-		// Set the singleton instance.
-		self::$instance = $this;
-
-	}
-
-	/**
-	 * Require files needed for the Admin UI.
-	 *
-	 * @since 3.20.0
-	 */
-	private static function require_files() {
-
-		require_once plugin_dir_path( __DIR__ ) . 'admin/class-wordlift-dashboard-latest-news.php';
-		require_once plugin_dir_path( __DIR__ ) . 'admin/class-wordlift-admin-dashboard-v2.php';
-		require_once plugin_dir_path( __DIR__ ) . 'admin/class-wordlift-admin-not-enriched-filter.php';
-
-	}
-
-	/**
-	 * Get the singleton instance.
-	 *
-	 * @return Wordlift_Admin The singleton instance.
-	 * @since 3.19.4
-	 */
-	public static function get_instance() {
-
-		return self::$instance;
-	}
-
-	public static function is_gutenberg() {
-		if ( function_exists( 'is_gutenberg_page' ) &&
-			 is_gutenberg_page()
-		) {
-			// The Gutenberg plugin is on.
-			return true;
-		}
-		$current_screen = get_current_screen();
-		if ( method_exists( $current_screen, 'is_block_editor' ) &&
-			 $current_screen->is_block_editor()
-		) {
-			// Gutenberg page on 5+.
-			return true;
-		}
-
-		return false;
-	}
-
-	/**
-	 * Register the stylesheets for the admin area.
-	 *
-	 * @since    1.0.0
-	 */
-	public function enqueue_styles() {
-
-		/**
-		 * This function is provided for demonstration purposes only.
-		 *
-		 * An instance of this class should be passed to the run() function
-		 * defined in Wordlift_Loader as all of the hooks are defined
-		 * in that particular class.
-		 *
-		 * The Wordlift_Loader will then create the relationship
-		 * between the defined hooks and the functions defined in this
-		 * class.
-		 */
-
-		wp_enqueue_style( $this->plugin_name, plugin_dir_url( __FILE__ ) . 'css/wordlift-admin.css', array(), $this->version, 'all' );
-
-	}
-
-	/**
-	 * Register the JavaScript for the admin area.
-	 *
-	 * @since    1.0.0
-	 */
-	public function enqueue_scripts() {
-
-		/*
+            new Wordlift_Admin_Dashboard_V2(
+                Wordlift_Entity_Service::get_instance()
+            );
+            new Wordlift_Admin_Not_Enriched_Filter();
+
+        }
+
+        // @@todo only load this class if ACF is available.
+        // Add support for ACF mappings, so that the admin edit mappings page can pick up ACF support when ACF is available.
+        new Acf_Mappings();
+
+        // Add the Mappings' REST Controller.
+        new Mappings_REST_Controller();
+
+        $features_registry->register_feature_from_slug(
+            'mappings',
+            ( defined( 'WL_ENABLE_MAPPINGS' ) && WL_ENABLE_MAPPINGS ),
+            array( $this, 'init_mappings' )
+        );
+
+        // Set the singleton instance.
+        self::$instance = $this;
+
+    }
+
+    /**
+     * Require files needed for the Admin UI.
+     *
+     * @since 3.20.0
+     */
+    private static function require_files() {
+
+        require_once plugin_dir_path( __DIR__ ) . 'admin/class-wordlift-dashboard-latest-news.php';
+        require_once plugin_dir_path( __DIR__ ) . 'admin/class-wordlift-admin-dashboard-v2.php';
+        require_once plugin_dir_path( __DIR__ ) . 'admin/class-wordlift-admin-not-enriched-filter.php';
+
+    }
+
+    /**
+     * Get the singleton instance.
+     *
+     * @return Wordlift_Admin The singleton instance.
+     * @since 3.19.4
+     */
+    public static function get_instance() {
+
+        return self::$instance;
+    }
+
+    public static function is_gutenberg() {
+        if ( function_exists( 'is_gutenberg_page' ) &&
+             is_gutenberg_page()
+        ) {
+            // The Gutenberg plugin is on.
+            return true;
+        }
+        $current_screen = get_current_screen();
+        if ( method_exists( $current_screen, 'is_block_editor' ) &&
+             $current_screen->is_block_editor()
+        ) {
+            // Gutenberg page on 5+.
+            return true;
+        }
+
+        return false;
+    }
+
+    /**
+     * Register the stylesheets for the admin area.
+     *
+     * @since    1.0.0
+     */
+    public function enqueue_styles() {
+
+        /**
+         * This function is provided for demonstration purposes only.
+         *
+         * An instance of this class should be passed to the run() function
+         * defined in Wordlift_Loader as all of the hooks are defined
+         * in that particular class.
+         *
+         * The Wordlift_Loader will then create the relationship
+         * between the defined hooks and the functions defined in this
+         * class.
+         */
+
+        wp_enqueue_style( $this->plugin_name, plugin_dir_url( __FILE__ ) . 'css/wordlift-admin.css', array(), $this->version, 'all' );
+
+    }
+
+    /**
+     * Register the JavaScript for the admin area.
+     *
+     * @since    1.0.0
+     */
+    public function enqueue_scripts() {
+
+        /*
 		 * Do not load our scripts on the Filter Urls plugin admin pages.
 		 *
 		 * @see https://github.com/insideout10/wordlift-plugin/issues/901
 		 * @since 3.20.0
 		 */
-		$screen = get_current_screen();
-		if ( is_a( $screen, 'WP_Screen' ) && 'filter-urls_page_filter_urls_form' === $screen->id ) {
-			return;
-		}
-
-		// Enqueue the admin scripts.
-		wp_enqueue_script(
-			$this->plugin_name,
-			plugin_dir_url( __FILE__ ) . 'js/1/admin.js',
-			array(
-				'jquery',
-				'underscore',
-				'backbone',
-			),
-			$this->version,
-			false
-		);
-
-		$params = $this->get_params();
-
-		// Finally output the params as `wlSettings` for JavaScript code.
-		wp_localize_script( $this->plugin_name, 'wlSettings', apply_filters( 'wl_admin_settings', $params ) );
-
-	}
-
-	/**
-	 * Return the settings array by applying filters.
-	 *
-	 * @return array
-	 */
-	public function get_params() {
-		$can_edit_wordlift_entities = current_user_can( 'edit_wordlift_entities' );
-		/*
+        $screen = get_current_screen();
+        if ( is_a( $screen, 'WP_Screen' ) && 'filter-urls_page_filter_urls_form' === $screen->id ) {
+            return;
+        }
+
+        // Enqueue the admin scripts.
+        wp_enqueue_script(
+            $this->plugin_name,
+            plugin_dir_url( __FILE__ ) . 'js/1/admin.js',
+            array(
+                'jquery',
+                'underscore',
+                'backbone',
+            ),
+            $this->version,
+            false
+        );
+
+        $params = $this->get_params();
+
+        // Finally output the params as `wlSettings` for JavaScript code.
+        wp_localize_script( $this->plugin_name, 'wlSettings', apply_filters( 'wl_admin_settings', $params ) );
+
+    }
+
+    /**
+     * Return the settings array by applying filters.
+     *
+     * @return array
+     */
+    public function get_params() {
+        $can_edit_wordlift_entities = current_user_can( 'edit_wordlift_entities' );
+        /*
 		 * People that can create entities will see the scope set in the wp-config.php file (by default `cloud`). People
 		 * that cannot edit create entities will always see the local entities.
 		 *
 		 * @see https://github.com/insideout10/wordlift-plugin/issues/839
 		 */
-		$autocomplete_scope = $can_edit_wordlift_entities ? WL_AUTOCOMPLETE_SCOPE : 'local';
-
-		// Set the basic params.
-		$params = array(
-			// @todo scripts in admin should use wp.post.
-			'ajax_url'                     => admin_url( 'admin-ajax.php' ),
-			// @todo remove specific actions from settings.
-			'action'                       => 'entity_by_title',
-			'datasetUri'                   => Wordlift_Configuration_Service::get_instance()->get_dataset_uri(),
-			'language'                     => Wordlift_Configuration_Service::get_instance()->get_language_code(),
-			'link_by_default'              => Wordlift_Configuration_Service::get_instance()->is_link_by_default(),
-			// Whether the current user is allowed to create new entities.
-			//
-			// @see https://github.com/insideout10/wordlift-plugin/issues/561
-			// @see https://github.com/insideout10/wordlift-plugin/issues/1267
-			'can_create_entities'          => apply_filters( 'wl_feature__enable__dataset', true ) ? ( $can_edit_wordlift_entities ? 'yes' : 'no' ) : 'no',
-			'l10n'                         => array(
-				'You already published an entity with the same name' => __( 'You already published an entity with the same name: ', 'wordlift' ),
-				'logo_selection_title'                    => __( 'WordLift Choose Logo', 'wordlift' ),
-				'logo_selection_button'                   => array( 'text' => __( 'Choose Logo', 'wordlift' ) ),
-				'Type at least 3 characters to search...' => _x( 'Type at least 3 characters to search...', 'Autocomplete Select', 'wordlift' ),
-				'No results found for your search.'       => _x( 'No results found: try changing or removing some words.', 'Autocomplete Select', 'wordlift' ),
-				'Please wait while we look for entities in the linked data cloud...' => _x( 'Please wait while we look for entities in the linked data cloud...', 'Autocomplete Select', 'wordlift' ),
-				'Add keywords to track'                   => __( 'Add Keywords to track', 'wordlift' ),
-			),
-			'wl_autocomplete_nonce'        => wp_create_nonce( 'wl_autocomplete' ),
-			'autocomplete_scope'           => $autocomplete_scope,
-			/**
-			 * Allow 3rd parties to define the default editor id. This turns useful if 3rd parties load
-			 * or change the TinyMCE id.
-			 *
-			 * The editor id is currently referenced by `src/coffee/editpost-widget/app.services.EditorAdapter.coffee`.
-			 *
-			 * @param string $editor The default editor id, by default `content`.
-			 *
-			 * @see https://github.com/insideout10/wordlift-plugin/issues/848
-			 *
-			 * @since 3.19.4
-			 */
-			'default_editor_id'            => apply_filters( 'wl_default_editor_id', 'content' ),
-
-			'analysis'                     => array( '_wpnonce' => wp_create_nonce( 'wl_analyze' ) ),
-			/**
-			 * Faceted search default limit
-			 *
-			 * @since 3.26.1
-			 */
-			'faceted_search_default_limit' => apply_filters( 'wl_faceted_search_default_limit', 10 ),
-			/**
-			 * WL Root path, to access in JS
-			 *
-			 * @since 3.27.3
-			 */
-			'wl_root'                      => plugin_dir_url( __DIR__ ),
-			/**
-			 * Enable synonyms, to access in JS
-			 * Show classification sidebar, to access in JS
-			 *
-			 * @since 3.30.0
-			 */
-			// phpcs:ignore WordPress.NamingConventions.ValidHookName.UseUnderscores
-			'can_add_synonyms'             => apply_filters( 'wl_feature__enable__add-synonyms', true ),
-			// phpcs:ignore WordPress.NamingConventions.ValidHookName.UseUnderscores
-			'show_classification_sidebar'  => apply_filters( 'wl_feature__enable__classification-sidebar', true ),
-			// By default the videoobject should not show.
-			'show_videoobject'             => apply_filters( 'wl_feature__enable__videoobject', false ),
-		);
-
-		// Set post-related values if there's a current post.
-		$entity_being_edited = get_post();
-		$post                = $entity_being_edited;
-		if ( null !== $post ) {
-
-			$params['post_id']           = $entity_being_edited->ID;
-			$entity_service              = Wordlift_Entity_Service::get_instance();
-			$params['entityBeingEdited'] = isset( $entity_being_edited->post_type ) && $entity_service->is_entity( $post->ID ) && is_numeric( get_the_ID() );
-			// We add the `itemId` here to give a chance to the analysis to use it in order to tell WLS to exclude it
-			// from the results, since we don't want the current entity to be discovered by the analysis.
-			//
-			// See https://github.com/insideout10/wordlift-plugin/issues/345
-			$params['itemId']                      = $entity_service->get_uri( $entity_being_edited->ID );
-			$params['wl_schemaorg_property_nonce'] = wp_create_nonce( 'wl_schemaorg_property' );
-
-			/*
+        $autocomplete_scope = $can_edit_wordlift_entities ? WL_AUTOCOMPLETE_SCOPE : 'local';
+
+        // Set the basic params.
+        $params = array(
+            // @todo scripts in admin should use wp.post.
+            'ajax_url'                     => admin_url( 'admin-ajax.php' ),
+            // @todo remove specific actions from settings.
+            'action'                       => 'entity_by_title',
+            'datasetUri'                   => Wordlift_Configuration_Service::get_instance()->get_dataset_uri(),
+            'language'                     => Wordlift_Configuration_Service::get_instance()->get_language_code(),
+            'link_by_default'              => Wordlift_Configuration_Service::get_instance()->is_link_by_default(),
+            // Whether the current user is allowed to create new entities.
+            //
+            // @see https://github.com/insideout10/wordlift-plugin/issues/561
+            // @see https://github.com/insideout10/wordlift-plugin/issues/1267
+            'can_create_entities'          => apply_filters( 'wl_feature__enable__dataset', true ) ? ( $can_edit_wordlift_entities ? 'yes' : 'no' ) : 'no',
+            'l10n'                         => array(
+                'You already published an entity with the same name' => __( 'You already published an entity with the same name: ', 'wordlift' ),
+                'logo_selection_title'                    => __( 'WordLift Choose Logo', 'wordlift' ),
+                'logo_selection_button'                   => array( 'text' => __( 'Choose Logo', 'wordlift' ) ),
+                'Type at least 3 characters to search...' => _x( 'Type at least 3 characters to search...', 'Autocomplete Select', 'wordlift' ),
+                'No results found for your search.'       => _x( 'No results found: try changing or removing some words.', 'Autocomplete Select', 'wordlift' ),
+                'Please wait while we look for entities in the linked data cloud...' => _x( 'Please wait while we look for entities in the linked data cloud...', 'Autocomplete Select', 'wordlift' ),
+                'Add keywords to track'                   => __( 'Add Keywords to track', 'wordlift' ),
+            ),
+            'wl_autocomplete_nonce'        => wp_create_nonce( 'wl_autocomplete' ),
+            'autocomplete_scope'           => $autocomplete_scope,
+            /**
+             * Allow 3rd parties to define the default editor id. This turns useful if 3rd parties load
+             * or change the TinyMCE id.
+             *
+             * The editor id is currently referenced by `src/coffee/editpost-widget/app.services.EditorAdapter.coffee`.
+             *
+             * @param string $editor The default editor id, by default `content`.
+             *
+             * @see https://github.com/insideout10/wordlift-plugin/issues/848
+             *
+             * @since 3.19.4
+             */
+            'default_editor_id'            => apply_filters( 'wl_default_editor_id', 'content' ),
+
+            'analysis'                     => array( '_wpnonce' => wp_create_nonce( 'wl_analyze' ) ),
+            /**
+             * Faceted search default limit
+             *
+             * @since 3.26.1
+             */
+            'faceted_search_default_limit' => apply_filters( 'wl_faceted_search_default_limit', 10 ),
+            /**
+             * WL Root path, to access in JS
+             *
+             * @since 3.27.3
+             */
+            'wl_root'                      => plugin_dir_url( __DIR__ ),
+            /**
+             * Enable synonyms, to access in JS
+             * Show classification sidebar, to access in JS
+             *
+             * @since 3.30.0
+             */
+            // phpcs:ignore WordPress.NamingConventions.ValidHookName.UseUnderscores
+            'can_add_synonyms'             => apply_filters( 'wl_feature__enable__add-synonyms', true ),
+            // phpcs:ignore WordPress.NamingConventions.ValidHookName.UseUnderscores
+            'show_classification_sidebar'  => apply_filters( 'wl_feature__enable__classification-sidebar', true ),
+            // By default the videoobject should not show.
+            'show_videoobject'             => apply_filters( 'wl_feature__enable__videoobject', false ),
+        );
+
+        // Set post-related values if there's a current post.
+        $entity_being_edited = get_post();
+        $post                = $entity_being_edited;
+        if ( null !== $post ) {
+
+            $params['post_id']           = $entity_being_edited->ID;
+            $entity_service              = Wordlift_Entity_Service::get_instance();
+            $params['entityBeingEdited'] = isset( $entity_being_edited->post_type ) && $entity_service->is_entity( $post->ID ) && is_numeric( get_the_ID() );
+            // We add the `itemId` here to give a chance to the analysis to use it in order to tell WLS to exclude it
+            // from the results, since we don't want the current entity to be discovered by the analysis.
+            //
+            // See https://github.com/insideout10/wordlift-plugin/issues/345
+            $params['itemId']                      = $entity_service->get_uri( $entity_being_edited->ID );
+            $params['wl_schemaorg_property_nonce'] = wp_create_nonce( 'wl_schemaorg_property' );
+
+            /*
 			 * Add the `properties` if `WL_ALL_ENTITY_TYPES` is enabled.
 			 *
 			 * @see https://github.com/insideout10/wordlift-plugin/issues/835
 			 */
-			// phpcs:ignore WordPress.NamingConventions.ValidHookName.UseUnderscores
-			if ( apply_filters( 'wl_feature__enable__all-entity-types', WL_ALL_ENTITY_TYPES ) ) {
-				$params['properties'] = Wordlift_Schemaorg_Property_Service::get_instance()->get_all( $post->ID );
-			}
-		}
-
-		return $params;
-	}
-
-	/**
-	 * Initialize the mappings.
-	 *
-	 * @return void
-	 */
-	public function init_mappings() {
-		new Admin_Mappings_Page();
-		/**
-		 * @since 3.27.0
-		 * Hooks in to ui of edit mapping screen, add taxonomy as a option.
-		 */
-		$taxonomy_option = new Taxonomy_Option();
-		$taxonomy_option->add_taxonomy_option();
-		new Edit_Mappings_Page( new Mappings_Transform_Functions_Registry() );
-	}
+            // phpcs:ignore WordPress.NamingConventions.ValidHookName.UseUnderscores
+            if ( apply_filters( 'wl_feature__enable__all-entity-types', WL_ALL_ENTITY_TYPES ) ) {
+                $params['properties'] = Wordlift_Schemaorg_Property_Service::get_instance()->get_all( $post->ID );
+            }
+        }
+
+        return $params;
+    }
+
+    /**
+     * Initialize the mappings.
+     *
+     * @return void
+     */
+    public function init_mappings() {
+        new Admin_Mappings_Page();
+        /**
+         * @since 3.27.0
+         * Hooks in to ui of edit mapping screen, add taxonomy as a option.
+         */
+        $taxonomy_option = new Taxonomy_Option();
+        $taxonomy_option->add_taxonomy_option();
+        new Edit_Mappings_Page( new Mappings_Transform_Functions_Registry() );
+    }
 
 }

Spacing +49 added lines, -49 removed lines

@@ -81,7 +81,7 @@ 
 	 *
 	 * @since  1.0.0
 	 */
-	public function __construct( $plugin_name, $version, $notice_service, $user_service ) {
+	public function __construct($plugin_name, $version, $notice_service, $user_service) {
 
 		$this->plugin_name = $plugin_name;
 		$this->version     = $version;
@@ -92,20 +92,20 @@ 
 		$dataset_uri           = $configuration_service->get_dataset_uri();
 		$key                   = $configuration_service->get_key();
 		$features_registry     = Features_Registry::get_instance();
-		if ( empty( $dataset_uri ) ) {
+		if (empty($dataset_uri)) {
 			$settings_page = Wordlift_Admin_Settings_Page::get_instance();
-			if ( empty( $key ) ) {
+			if (empty($key)) {
 				/* translators: %s: The link to the settings page. */
-				$error = sprintf( esc_html__( "WordLift's key isn't set, please open the %s to set WordLift's key.", 'wordlift' ), '<a href="' . $settings_page->get_url() . '">' . esc_html__( 'settings page', 'wordlift' ) . '</a>' );
+				$error = sprintf(esc_html__("WordLift's key isn't set, please open the %s to set WordLift's key.", 'wordlift'), '<a href="'.$settings_page->get_url().'">'.esc_html__('settings page', 'wordlift').'</a>');
 			} else {
 				/* translators: %s: The link to the settings page. */
-				$error = sprintf( esc_html__( "WordLift's dataset URI is not configured: please open the %s to set WordLift's key again.", 'wordlift' ), '<a href="' . $settings_page->get_url() . '">' . esc_html__( 'settings page', 'wordlift' ) . '</a>' );
+				$error = sprintf(esc_html__("WordLift's dataset URI is not configured: please open the %s to set WordLift's key again.", 'wordlift'), '<a href="'.$settings_page->get_url().'">'.esc_html__('settings page', 'wordlift').'</a>');
 			}
-			$notice_service->add_error( $error );
+			$notice_service->add_error($error);
 		}
 
 		// Load additional code if we're in the admin UI.
-		if ( is_admin() ) {
+		if (is_admin()) {
 
 			// Require the PHP files for the next code fragment.
 			self::require_files();
@@ -129,13 +129,13 @@ 
 			 * @see https://github.com/insideout10/wordlift-plugin/issues/835
 			 */
 			// phpcs:ignore WordPress.NamingConventions.ValidHookName.UseUnderscores
-			if ( apply_filters( 'wl_feature__enable__all-entity-types', WL_ALL_ENTITY_TYPES ) ) {
+			if (apply_filters('wl_feature__enable__all-entity-types', WL_ALL_ENTITY_TYPES)) {
 
-				require_once plugin_dir_path( __DIR__ ) . 'admin/class-wordlift-admin-schemaorg-taxonomy-metabox.php';
+				require_once plugin_dir_path(__DIR__).'admin/class-wordlift-admin-schemaorg-taxonomy-metabox.php';
 				/*
 				 * The `Mappings` admin page.
 				 */
-				require_once plugin_dir_path( __DIR__ ) . 'admin/class-wordlift-admin-mappings-page.php';
+				require_once plugin_dir_path(__DIR__).'admin/class-wordlift-admin-mappings-page.php';
 
 				new Wordlift_Admin_Mappings_Page();
 
@@ -144,9 +144,9 @@ 
 				 *
 				 * @since 3.20.0
 				 */
-				require_once plugin_dir_path( __DIR__ ) . 'includes/schemaorg/class-wordlift-schemaorg-sync-batch-operation.php';
+				require_once plugin_dir_path(__DIR__).'includes/schemaorg/class-wordlift-schemaorg-sync-batch-operation.php';
 
-				$this->sync_batch_operation_ajax_adapter = new Wordlift_Batch_Operation_Ajax_Adapter( new Wordlift_Schemaorg_Sync_Batch_Operation(), 'wl_schemaorg_sync' );
+				$this->sync_batch_operation_ajax_adapter = new Wordlift_Batch_Operation_Ajax_Adapter(new Wordlift_Schemaorg_Sync_Batch_Operation(), 'wl_schemaorg_sync');
 
 			}
 
@@ -155,7 +155,7 @@ 
 			 *
 			 * @since 3.20.0
 			 */
-			require_once plugin_dir_path( __DIR__ ) . 'admin/class-wordlift-admin-term-adapter.php';
+			require_once plugin_dir_path(__DIR__).'admin/class-wordlift-admin-term-adapter.php';
 			new Wordlift_Admin_Term_Adapter();
 
 			/*
@@ -181,8 +181,8 @@ 
 
 		$features_registry->register_feature_from_slug(
 			'mappings',
-			( defined( 'WL_ENABLE_MAPPINGS' ) && WL_ENABLE_MAPPINGS ),
-			array( $this, 'init_mappings' )
+			(defined('WL_ENABLE_MAPPINGS') && WL_ENABLE_MAPPINGS),
+			array($this, 'init_mappings')
 		);
 
 		// Set the singleton instance.
@@ -197,9 +197,9 @@ 
 	 */
 	private static function require_files() {
 
-		require_once plugin_dir_path( __DIR__ ) . 'admin/class-wordlift-dashboard-latest-news.php';
-		require_once plugin_dir_path( __DIR__ ) . 'admin/class-wordlift-admin-dashboard-v2.php';
-		require_once plugin_dir_path( __DIR__ ) . 'admin/class-wordlift-admin-not-enriched-filter.php';
+		require_once plugin_dir_path(__DIR__).'admin/class-wordlift-dashboard-latest-news.php';
+		require_once plugin_dir_path(__DIR__).'admin/class-wordlift-admin-dashboard-v2.php';
+		require_once plugin_dir_path(__DIR__).'admin/class-wordlift-admin-not-enriched-filter.php';
 
 	}
 
@@ -215,14 +215,14 @@ 
 	}
 
 	public static function is_gutenberg() {
-		if ( function_exists( 'is_gutenberg_page' ) &&
+		if (function_exists('is_gutenberg_page') &&
 			 is_gutenberg_page()
 		) {
 			// The Gutenberg plugin is on.
 			return true;
 		}
 		$current_screen = get_current_screen();
-		if ( method_exists( $current_screen, 'is_block_editor' ) &&
+		if (method_exists($current_screen, 'is_block_editor') &&
 			 $current_screen->is_block_editor()
 		) {
 			// Gutenberg page on 5+.
@@ -251,7 +251,7 @@ 
 		 * class.
 		 */
 
-		wp_enqueue_style( $this->plugin_name, plugin_dir_url( __FILE__ ) . 'css/wordlift-admin.css', array(), $this->version, 'all' );
+		wp_enqueue_style($this->plugin_name, plugin_dir_url(__FILE__).'css/wordlift-admin.css', array(), $this->version, 'all');
 
 	}
 
@@ -269,14 +269,14 @@ 
 		 * @since 3.20.0
 		 */
 		$screen = get_current_screen();
-		if ( is_a( $screen, 'WP_Screen' ) && 'filter-urls_page_filter_urls_form' === $screen->id ) {
+		if (is_a($screen, 'WP_Screen') && 'filter-urls_page_filter_urls_form' === $screen->id) {
 			return;
 		}
 
 		// Enqueue the admin scripts.
 		wp_enqueue_script(
 			$this->plugin_name,
-			plugin_dir_url( __FILE__ ) . 'js/1/admin.js',
+			plugin_dir_url(__FILE__).'js/1/admin.js',
 			array(
 				'jquery',
 				'underscore',
@@ -289,7 +289,7 @@ 
 		$params = $this->get_params();
 
 		// Finally output the params as `wlSettings` for JavaScript code.
-		wp_localize_script( $this->plugin_name, 'wlSettings', apply_filters( 'wl_admin_settings', $params ) );
+		wp_localize_script($this->plugin_name, 'wlSettings', apply_filters('wl_admin_settings', $params));
 
 	}
 
@@ -299,7 +299,7 @@ 
 	 * @return array
 	 */
 	public function get_params() {
-		$can_edit_wordlift_entities = current_user_can( 'edit_wordlift_entities' );
+		$can_edit_wordlift_entities = current_user_can('edit_wordlift_entities');
 		/*
 		 * People that can create entities will see the scope set in the wp-config.php file (by default `cloud`). People
 		 * that cannot edit create entities will always see the local entities.
@@ -311,7 +311,7 @@ 
 		// Set the basic params.
 		$params = array(
 			// @todo scripts in admin should use wp.post.
-			'ajax_url'                     => admin_url( 'admin-ajax.php' ),
+			'ajax_url'                     => admin_url('admin-ajax.php'),
 			// @todo remove specific actions from settings.
 			'action'                       => 'entity_by_title',
 			'datasetUri'                   => Wordlift_Configuration_Service::get_instance()->get_dataset_uri(),
@@ -321,17 +321,17 @@ 
 			//
 			// @see https://github.com/insideout10/wordlift-plugin/issues/561
 			// @see https://github.com/insideout10/wordlift-plugin/issues/1267
-			'can_create_entities'          => apply_filters( 'wl_feature__enable__dataset', true ) ? ( $can_edit_wordlift_entities ? 'yes' : 'no' ) : 'no',
+			'can_create_entities'          => apply_filters('wl_feature__enable__dataset', true) ? ($can_edit_wordlift_entities ? 'yes' : 'no') : 'no',
 			'l10n'                         => array(
-				'You already published an entity with the same name' => __( 'You already published an entity with the same name: ', 'wordlift' ),
-				'logo_selection_title'                    => __( 'WordLift Choose Logo', 'wordlift' ),
-				'logo_selection_button'                   => array( 'text' => __( 'Choose Logo', 'wordlift' ) ),
-				'Type at least 3 characters to search...' => _x( 'Type at least 3 characters to search...', 'Autocomplete Select', 'wordlift' ),
-				'No results found for your search.'       => _x( 'No results found: try changing or removing some words.', 'Autocomplete Select', 'wordlift' ),
-				'Please wait while we look for entities in the linked data cloud...' => _x( 'Please wait while we look for entities in the linked data cloud...', 'Autocomplete Select', 'wordlift' ),
-				'Add keywords to track'                   => __( 'Add Keywords to track', 'wordlift' ),
+				'You already published an entity with the same name' => __('You already published an entity with the same name: ', 'wordlift'),
+				'logo_selection_title'                    => __('WordLift Choose Logo', 'wordlift'),
+				'logo_selection_button'                   => array('text' => __('Choose Logo', 'wordlift')),
+				'Type at least 3 characters to search...' => _x('Type at least 3 characters to search...', 'Autocomplete Select', 'wordlift'),
+				'No results found for your search.'       => _x('No results found: try changing or removing some words.', 'Autocomplete Select', 'wordlift'),
+				'Please wait while we look for entities in the linked data cloud...' => _x('Please wait while we look for entities in the linked data cloud...', 'Autocomplete Select', 'wordlift'),
+				'Add keywords to track'                   => __('Add Keywords to track', 'wordlift'),
 			),
-			'wl_autocomplete_nonce'        => wp_create_nonce( 'wl_autocomplete' ),
+			'wl_autocomplete_nonce'        => wp_create_nonce('wl_autocomplete'),
 			'autocomplete_scope'           => $autocomplete_scope,
 			/**
 			 * Allow 3rd parties to define the default editor id. This turns useful if 3rd parties load
@@ -345,21 +345,21 @@ 
 			 *
 			 * @since 3.19.4
 			 */
-			'default_editor_id'            => apply_filters( 'wl_default_editor_id', 'content' ),
+			'default_editor_id'            => apply_filters('wl_default_editor_id', 'content'),
 
-			'analysis'                     => array( '_wpnonce' => wp_create_nonce( 'wl_analyze' ) ),
+			'analysis'                     => array('_wpnonce' => wp_create_nonce('wl_analyze')),
 			/**
 			 * Faceted search default limit
 			 *
 			 * @since 3.26.1
 			 */
-			'faceted_search_default_limit' => apply_filters( 'wl_faceted_search_default_limit', 10 ),
+			'faceted_search_default_limit' => apply_filters('wl_faceted_search_default_limit', 10),
 			/**
 			 * WL Root path, to access in JS
 			 *
 			 * @since 3.27.3
 			 */
-			'wl_root'                      => plugin_dir_url( __DIR__ ),
+			'wl_root'                      => plugin_dir_url(__DIR__),
 			/**
 			 * Enable synonyms, to access in JS
 			 * Show classification sidebar, to access in JS
@@ -367,27 +367,27 @@ 
 			 * @since 3.30.0
 			 */
 			// phpcs:ignore WordPress.NamingConventions.ValidHookName.UseUnderscores
-			'can_add_synonyms'             => apply_filters( 'wl_feature__enable__add-synonyms', true ),
+			'can_add_synonyms'             => apply_filters('wl_feature__enable__add-synonyms', true),
 			// phpcs:ignore WordPress.NamingConventions.ValidHookName.UseUnderscores
-			'show_classification_sidebar'  => apply_filters( 'wl_feature__enable__classification-sidebar', true ),
+			'show_classification_sidebar'  => apply_filters('wl_feature__enable__classification-sidebar', true),
 			// By default the videoobject should not show.
-			'show_videoobject'             => apply_filters( 'wl_feature__enable__videoobject', false ),
+			'show_videoobject'             => apply_filters('wl_feature__enable__videoobject', false),
 		);
 
 		// Set post-related values if there's a current post.
 		$entity_being_edited = get_post();
 		$post                = $entity_being_edited;
-		if ( null !== $post ) {
+		if (null !== $post) {
 
 			$params['post_id']           = $entity_being_edited->ID;
 			$entity_service              = Wordlift_Entity_Service::get_instance();
-			$params['entityBeingEdited'] = isset( $entity_being_edited->post_type ) && $entity_service->is_entity( $post->ID ) && is_numeric( get_the_ID() );
+			$params['entityBeingEdited'] = isset($entity_being_edited->post_type) && $entity_service->is_entity($post->ID) && is_numeric(get_the_ID());
 			// We add the `itemId` here to give a chance to the analysis to use it in order to tell WLS to exclude it
 			// from the results, since we don't want the current entity to be discovered by the analysis.
 			//
 			// See https://github.com/insideout10/wordlift-plugin/issues/345
-			$params['itemId']                      = $entity_service->get_uri( $entity_being_edited->ID );
-			$params['wl_schemaorg_property_nonce'] = wp_create_nonce( 'wl_schemaorg_property' );
+			$params['itemId']                      = $entity_service->get_uri($entity_being_edited->ID);
+			$params['wl_schemaorg_property_nonce'] = wp_create_nonce('wl_schemaorg_property');
 
 			/*
 			 * Add the `properties` if `WL_ALL_ENTITY_TYPES` is enabled.
@@ -395,8 +395,8 @@ 
 			 * @see https://github.com/insideout10/wordlift-plugin/issues/835
 			 */
 			// phpcs:ignore WordPress.NamingConventions.ValidHookName.UseUnderscores
-			if ( apply_filters( 'wl_feature__enable__all-entity-types', WL_ALL_ENTITY_TYPES ) ) {
-				$params['properties'] = Wordlift_Schemaorg_Property_Service::get_instance()->get_all( $post->ID );
+			if (apply_filters('wl_feature__enable__all-entity-types', WL_ALL_ENTITY_TYPES)) {
+				$params['properties'] = Wordlift_Schemaorg_Property_Service::get_instance()->get_all($post->ID);
 			}
 		}
 
@@ -416,7 +416,7 @@ 
 		 */
 		$taxonomy_option = new Taxonomy_Option();
 		$taxonomy_option->add_taxonomy_option();
-		new Edit_Mappings_Page( new Mappings_Transform_Functions_Registry() );
+		new Edit_Mappings_Page(new Mappings_Transform_Functions_Registry());
 	}
 
 }

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

Indentation +120 added lines, -120 removed lines

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

Spacing +15 added lines, -15 removed lines

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

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

Indentation +42 added lines, -42 removed lines

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

Spacing +14 added lines, -14 removed lines

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

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

Indentation +64 added lines, -64 removed lines

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

Spacing +14 added lines, -14 removed lines

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

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

Indentation +50 added lines, -50 removed lines

@@ -18,75 +18,75 @@
  */
 class Wordlift_Admin_Input_Element implements Wordlift_Admin_Element {
 
-	/**
-	 * Output the HTML for an input box type settings_page
-	 *
-	 * @param array $args An array with the following keys:
-	 *                    Parameters controlling the result.
-	 *
-	 * @type string name The name attribute of the input element. Mandatory.
-	 *
-	 * @type string id    The id attribute of the input element. Optional.
-	 * @type string id    The id attribute of the input element.
-	 *                            Optional, randomly generated one is used if not supplied.
-	 * @type string value    The value of the input element.
-	 *                            Optional, defaults to empty string.
-	 * @type bool readonly    Indicates whether the input is read only.
-	 *                            Optional, defaults to read-write
-	 * @type string css_class    The class attribute for the input element.
-	 *                            If empty string no class attribute will be added.
-	 *                            Optional, defaults to empty string.
-	 * @type string description    The descriptio text to be displayed below the element.
-	 *                            Can include some HTML element.
-	 *                            If empty string no description will be displayed.
-	 *                            Optional, defaults to empty string.
-	 * @since 3.21.0 added the ability to use a $type arg.
-	 *
-	 * @return $this|Wordlift_Admin_Element
-	 */
-	public function render( $args ) {
+    /**
+     * Output the HTML for an input box type settings_page
+     *
+     * @param array $args An array with the following keys:
+     *                    Parameters controlling the result.
+     *
+     * @type string name The name attribute of the input element. Mandatory.
+     *
+     * @type string id    The id attribute of the input element. Optional.
+     * @type string id    The id attribute of the input element.
+     *                            Optional, randomly generated one is used if not supplied.
+     * @type string value    The value of the input element.
+     *                            Optional, defaults to empty string.
+     * @type bool readonly    Indicates whether the input is read only.
+     *                            Optional, defaults to read-write
+     * @type string css_class    The class attribute for the input element.
+     *                            If empty string no class attribute will be added.
+     *                            Optional, defaults to empty string.
+     * @type string description    The descriptio text to be displayed below the element.
+     *                            Can include some HTML element.
+     *                            If empty string no description will be displayed.
+     *                            Optional, defaults to empty string.
+     * @since 3.21.0 added the ability to use a $type arg.
+     *
+     * @return $this|Wordlift_Admin_Element
+     */
+    public function render( $args ) {
 
-		/*
+        /*
 		 * Parse the arguments and merge with default values.
 		 * Name intentionally do not have a default as it has to be in SyncEvent
 		 * with form handling code
 		 */
-		$pre_params = wp_parse_args(
-			$args,
-			array(
-				'id'          => uniqid( 'wl-input-' ),
-				'value'       => '',
-				'readonly'    => false,
-				'css_class'   => '',
-				'description' => '',
-			)
-		);
-		$params     = apply_filters( 'wl_admin_input_element_params', $pre_params );
-		// allow different types of input - default to 'text'.
-		$input_type = ! empty( $params['type'] ) ? $params['type'] : 'text';
-		?>
+        $pre_params = wp_parse_args(
+            $args,
+            array(
+                'id'          => uniqid( 'wl-input-' ),
+                'value'       => '',
+                'readonly'    => false,
+                'css_class'   => '',
+                'description' => '',
+            )
+        );
+        $params     = apply_filters( 'wl_admin_input_element_params', $pre_params );
+        // allow different types of input - default to 'text'.
+        $input_type = ! empty( $params['type'] ) ? $params['type'] : 'text';
+        ?>
 
 		<input type="<?php echo esc_attr( $input_type ); ?>"
 			   id="<?php echo esc_attr( $params['id'] ); ?>"
 			   name="<?php echo esc_attr( $params['name'] ); ?>"
 			   value="<?php echo esc_attr( $params['value'] ); ?>"
 			<?php
-			if ( ! empty( $params['readonly'] ) ) {
-				?>
+            if ( ! empty( $params['readonly'] ) ) {
+                ?>
 				 readonly="readonly" <?php } ?>
 			<?php
-			if ( ! empty( $params['css_class'] ) ) {
-				?>
+            if ( ! empty( $params['css_class'] ) ) {
+                ?>
 				 class="<?php echo esc_attr( $params['css_class'] ); ?>" <?php } ?>
 		/>
 		<?php
-		if ( ! empty( $params['description'] ) ) {
-			?>
+        if ( ! empty( $params['description'] ) ) {
+            ?>
 			<p><?php echo wp_kses( $params['description'], array( 'a' => array( 'href' => array() ) ) ); ?></p><?php } ?>
 
 		<?php
 
-		return $this;
-	}
+        return $this;
+    }
 
 }

Spacing +13 added lines, -13 removed lines

@@ -44,7 +44,7 @@ 
 	 *
 	 * @return $this|Wordlift_Admin_Element
 	 */
-	public function render( $args ) {
+	public function render($args) {
 
 		/*
 		 * Parse the arguments and merge with default values.
@@ -54,35 +54,35 @@ 
 		$pre_params = wp_parse_args(
 			$args,
 			array(
-				'id'          => uniqid( 'wl-input-' ),
+				'id'          => uniqid('wl-input-'),
 				'value'       => '',
 				'readonly'    => false,
 				'css_class'   => '',
 				'description' => '',
 			)
 		);
-		$params     = apply_filters( 'wl_admin_input_element_params', $pre_params );
+		$params     = apply_filters('wl_admin_input_element_params', $pre_params);
 		// allow different types of input - default to 'text'.
-		$input_type = ! empty( $params['type'] ) ? $params['type'] : 'text';
+		$input_type = ! empty($params['type']) ? $params['type'] : 'text';
 		?>
 
-		<input type="<?php echo esc_attr( $input_type ); ?>"
-			   id="<?php echo esc_attr( $params['id'] ); ?>"
-			   name="<?php echo esc_attr( $params['name'] ); ?>"
-			   value="<?php echo esc_attr( $params['value'] ); ?>"
+		<input type="<?php echo esc_attr($input_type); ?>"
+			   id="<?php echo esc_attr($params['id']); ?>"
+			   name="<?php echo esc_attr($params['name']); ?>"
+			   value="<?php echo esc_attr($params['value']); ?>"
 			<?php
-			if ( ! empty( $params['readonly'] ) ) {
+			if ( ! empty($params['readonly'])) {
 				?>
 				 readonly="readonly" <?php } ?>
 			<?php
-			if ( ! empty( $params['css_class'] ) ) {
+			if ( ! empty($params['css_class'])) {
 				?>
-				 class="<?php echo esc_attr( $params['css_class'] ); ?>" <?php } ?>
+				 class="<?php echo esc_attr($params['css_class']); ?>" <?php } ?>
 		/>
 		<?php
-		if ( ! empty( $params['description'] ) ) {
+		if ( ! empty($params['description'])) {
 			?>
-			<p><?php echo wp_kses( $params['description'], array( 'a' => array( 'href' => array() ) ) ); ?></p><?php } ?>
+			<p><?php echo wp_kses($params['description'], array('a' => array('href' => array()))); ?></p><?php } ?>
 
 		<?php