insideout10 / wordlift-plugin

src/admin/wordlift_admin_meta_box_entities.php

Indentation +84 added lines, -84 removed lines

@@ -10,15 +10,15 @@ 
  */
 function wl_register_metaboxes() {
 
-	// Load metabox classes
-	require_once( 'WL_Metabox/class-wl-metabox.php' );
+    // Load metabox classes
+    require_once( 'WL_Metabox/class-wl-metabox.php' );
 
-	$wl_metabox = new WL_Metabox();     // Everything is done inside here with the correct timing
+    $wl_metabox = 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' );
 }
 
 
@@ -29,12 +29,12 @@ 
  */
 function wl_admin_add_entities_meta_box( $post_type ) {
 
-	// wl_write_log( "wl_admin_add_entities_meta_box [ post type :: $post_type ]" );
+    // wl_write_log( "wl_admin_add_entities_meta_box [ post type :: $post_type ]" );
 
-	// Add main meta box for related entities and 4W
-	add_meta_box(
-		'wordlift_entities_box', __( 'WordLift', 'wordlift' ), 'wl_entities_box_content', $post_type, 'side', 'high'
-	);
+    // Add main meta box for related entities and 4W
+    add_meta_box(
+        'wordlift_entities_box', __( 'WordLift', 'wordlift' ), 'wl_entities_box_content', $post_type, 'side', 'high'
+    );
 }
 
 add_action( 'add_meta_boxes', 'wl_admin_add_entities_meta_box' );
@@ -46,23 +46,23 @@ 
  */
 function wl_entities_box_content( $post ) {
 
-	// wl_write_log( "wl_entities_box_content [ post id :: $post->ID ]" );
+    // wl_write_log( "wl_entities_box_content [ post id :: $post->ID ]" );
 
-	// Angularjs edit-post widget wrapper
-	echo '<div id="wordlift-edit-post-outer-wrapper"></div>';
+    // Angularjs edit-post widget wrapper
+    echo '<div id="wordlift-edit-post-outer-wrapper"></div>';
 
-	// Angularjs edit-post widget classification boxes configuration
-	$classification_boxes = unserialize( WL_CORE_POST_CLASSIFICATION_BOXES );
+    // Angularjs edit-post widget classification boxes configuration
+    $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 ) {
-		// Build the proper relation name
-		$relation_name = $box['id'];
+    // Add selected entities to classification_boxes
+    foreach ( $classification_boxes as $i => $box ) {
+        // Build the proper relation name
+        $relation_name = $box['id'];
 
-		// wl_write_log( "Going to related of $relation_name" );
+        // wl_write_log( "Going to related of $relation_name" );
 //
 //		// Get entity ids related to the current post for the given relation name (both draft and published entities)
 //		$draft_entity_ids   = wl_core_get_related_entity_ids( $post->ID, array(
@@ -82,68 +82,68 @@ 
 //			$entity_id = wl_get_entity_uri( $entity_id );
 //		} );
 //
-		// Get the entity referenced from the post content.
-		$entity_uris = Wordlift_Content_Filter_Service::get_instance()->get_entity_uris( $post->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();
-		$entity_ids     = array_map( function ( $item ) use ( $entity_service ) {
-			$post = $entity_service->get_entity_post_by_uri( $item );
-
-			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 = json_encode( $classification_boxes );
-
-	// Ensure there are no repetitions of the referenced entities
-	$all_referenced_entities_ids = array_unique( $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 );
-		$referenced_entities_obj[ $entity['id'] ] = $entity;
-	}
-
-	$referenced_entities_obj = empty( $referenced_entities_obj ) ?
-		'{}' : 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 ) ?
-		json_encode( wl_serialize_entity( $published_place_id ) ) :
-		'undefined';
-
-	$topic_id  = get_post_meta(
-		$post->ID, Wordlift_Schema_Service::FIELD_TOPIC, true
-	);
-	$topic_obj = ( $topic_id ) ?
-		json_encode( wl_serialize_entity( $topic_id ) ) :
-		'undefined';
-
-	$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       = wl_get_entity_uri( $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();
-
-	echo <<<EOF
+        // Get the entity referenced from the post content.
+        $entity_uris = Wordlift_Content_Filter_Service::get_instance()->get_entity_uris( $post->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();
+        $entity_ids     = array_map( function ( $item ) use ( $entity_service ) {
+            $post = $entity_service->get_entity_post_by_uri( $item );
+
+            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 = json_encode( $classification_boxes );
+
+    // Ensure there are no repetitions of the referenced entities
+    $all_referenced_entities_ids = array_unique( $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 );
+        $referenced_entities_obj[ $entity['id'] ] = $entity;
+    }
+
+    $referenced_entities_obj = empty( $referenced_entities_obj ) ?
+        '{}' : 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 ) ?
+        json_encode( wl_serialize_entity( $published_place_id ) ) :
+        'undefined';
+
+    $topic_id  = get_post_meta(
+        $post->ID, Wordlift_Schema_Service::FIELD_TOPIC, true
+    );
+    $topic_obj = ( $topic_id ) ?
+        json_encode( wl_serialize_entity( $topic_id ) ) :
+        'undefined';
+
+    $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       = wl_get_entity_uri( $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();
+
+    echo <<<EOF
     <script type="text/javascript">
         jQuery( function() {
 

Spacing +31 added lines, -33 removed lines

@@ -11,14 +11,14 @@ 
 function wl_register_metaboxes() {
 
 	// Load metabox classes
-	require_once( 'WL_Metabox/class-wl-metabox.php' );
+	require_once('WL_Metabox/class-wl-metabox.php');
 
-	$wl_metabox = new WL_Metabox();     // Everything is done inside here with the correct timing
+	$wl_metabox = 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');
 }
 
 
@@ -27,24 +27,24 @@ 
  *
  * @param string $post_type The type of the current open post.
  */
-function wl_admin_add_entities_meta_box( $post_type ) {
+function wl_admin_add_entities_meta_box($post_type) {
 
 	// wl_write_log( "wl_admin_add_entities_meta_box [ post type :: $post_type ]" );
 
 	// Add main meta box for related entities and 4W
 	add_meta_box(
-		'wordlift_entities_box', __( 'WordLift', 'wordlift' ), 'wl_entities_box_content', $post_type, 'side', 'high'
+		'wordlift_entities_box', __('WordLift', 'wordlift'), 'wl_entities_box_content', $post_type, 'side', 'high'
 	);
 }
 
-add_action( 'add_meta_boxes', 'wl_admin_add_entities_meta_box' );
+add_action('add_meta_boxes', 'wl_admin_add_entities_meta_box');
 
 /**
  * Displays the meta box contents (called by *add_meta_box* callback).
  *
  * @param WP_Post $post The current post.
  */
-function wl_entities_box_content( $post ) {
+function wl_entities_box_content($post) {
 
 	// wl_write_log( "wl_entities_box_content [ post id :: $post->ID ]" );
 
@@ -52,13 +52,13 @@ 
 	echo '<div id="wordlift-edit-post-outer-wrapper"></div>';
 
 	// Angularjs edit-post widget classification boxes configuration
-	$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) {
 		// Build the proper relation name
 		$relation_name = $box['id'];
 
@@ -83,63 +83,61 @@ 
 //		} );
 //
 		// Get the entity referenced from the post content.
-		$entity_uris = Wordlift_Content_Filter_Service::get_instance()->get_entity_uris( $post->post_content );
+		$entity_uris = Wordlift_Content_Filter_Service::get_instance()->get_entity_uris($post->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();
-		$entity_ids     = array_map( function ( $item ) use ( $entity_service ) {
-			$post = $entity_service->get_entity_post_by_uri( $item );
+		$entity_ids     = array_map(function($item) use ($entity_service) {
+			$post = $entity_service->get_entity_post_by_uri($item);
 
 			return $post->ID;
-		}, $entity_uris );
+		}, $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 = json_encode( $classification_boxes );
+	$classification_boxes = 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);
 
 	// Build the entity storage object
 	$referenced_entities_obj = array();
-	foreach ( $all_referenced_entities_ids as $referenced_entity ) {
-		$entity                                   = wl_serialize_entity( $referenced_entity );
-		$referenced_entities_obj[ $entity['id'] ] = $entity;
+	foreach ($all_referenced_entities_ids as $referenced_entity) {
+		$entity                                   = wl_serialize_entity($referenced_entity);
+		$referenced_entities_obj[$entity['id']] = $entity;
 	}
 
-	$referenced_entities_obj = empty( $referenced_entities_obj ) ?
-		'{}' : json_encode( $referenced_entities_obj );
+	$referenced_entities_obj = empty($referenced_entities_obj) ?
+		'{}' : 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 ) ?
-		json_encode( wl_serialize_entity( $published_place_id ) ) :
-		'undefined';
+	$published_place_obj = ($published_place_id) ?
+		json_encode(wl_serialize_entity($published_place_id)) : 'undefined';
 
 	$topic_id  = get_post_meta(
 		$post->ID, Wordlift_Schema_Service::FIELD_TOPIC, true
 	);
-	$topic_obj = ( $topic_id ) ?
-		json_encode( wl_serialize_entity( $topic_id ) ) :
-		'undefined';
+	$topic_obj = ($topic_id) ?
+		json_encode(wl_serialize_entity($topic_id)) : 'undefined';
 
 	$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       = wl_get_entity_uri( $post->ID );
+	$current_post_uri       = wl_get_entity_uri($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();
 

src/admin/partials/wordlift-admin-batch-analysis-page.php

Indentation +43 added lines, -43 removed lines

@@ -11,68 +11,68 @@
 	<h2><?php esc_html_e( 'WordLift Batch Analyze Monitor', 'wordlift' ) ?></h2>
 
 	<?php
-	$queue        = $this->batch_analysis_service->waiting_for_analysis();
-	$submit_count = count( $queue );
-	?>
+    $queue        = $this->batch_analysis_service->waiting_for_analysis();
+    $submit_count = count( $queue );
+    ?>
 	<h3><?php esc_html_e( 'Posts in queue', 'wordlift' ) ?><?php echo " ($submit_count)"; ?></h3>
 	<?php
-	if ( empty( $queue ) ) {
-		esc_html_e( 'Nothing is currently in queue', 'wordlift' );
-	} else {
-		echo '<ul>';
-		foreach ( $queue as $pid ) {
-			$cancel_link = admin_url( "admin-ajax.php?action=wl_batch_analysis_cancel&post=$pid" );
-			?>
+    if ( empty( $queue ) ) {
+        esc_html_e( 'Nothing is currently in queue', 'wordlift' );
+    } else {
+        echo '<ul>';
+        foreach ( $queue as $pid ) {
+            $cancel_link = admin_url( "admin-ajax.php?action=wl_batch_analysis_cancel&post=$pid" );
+            ?>
 			<li><a href="<?php echo get_edit_post_link( $pid ) ?>"><?php
-					echo get_the_title( $pid ) ?></a> [<a
+                    echo get_the_title( $pid ) ?></a> [<a
 					href="<?php echo esc_attr( $cancel_link ); ?> "><?php esc_html_e( 'Cancel', 'wordlift' ); ?></a>]
 			</li>
 			<?php
-		}
-		echo '</ul>';
-	}
+        }
+        echo '</ul>';
+    }
 
-	$queue         = $this->batch_analysis_service->waiting_for_response();
-	$request_count = count( $queue );
-	?>
+    $queue         = $this->batch_analysis_service->waiting_for_response();
+    $request_count = count( $queue );
+    ?>
 	<h3><?php esc_html_e( 'Posts being processed', 'wordlift' ) ?><?php echo " ($request_count)"; ?></h3>
 	<?php
-	if ( empty( $queue ) ) {
-		esc_html_e( 'Nothing is currently being processed', 'wordlift' );
-	} else {
-		echo '<ul>';
-		foreach ( $queue as $pid ) {
-			$cancel_link = admin_url( "admin-ajax.php?action=wl_batch_analysis_cancel&post=$pid" );
-			?>
+    if ( empty( $queue ) ) {
+        esc_html_e( 'Nothing is currently being processed', 'wordlift' );
+    } else {
+        echo '<ul>';
+        foreach ( $queue as $pid ) {
+            $cancel_link = admin_url( "admin-ajax.php?action=wl_batch_analysis_cancel&post=$pid" );
+            ?>
 			<li><a href="<?php echo get_edit_post_link( $pid ) ?>"><?php
-					echo get_the_title( $pid ) ?></a> [<a
+                    echo get_the_title( $pid ) ?></a> [<a
 					href="<?php echo esc_attr( $cancel_link ); ?> "><?php esc_html_e( 'Cancel', 'wordlift' ); ?></a>]
 			</li>
 			<?php
-		}
-		echo '</ul>';
-	}
+        }
+        echo '</ul>';
+    }
 
-	$queue         = $this->batch_analysis_service->get_warnings();
-	$warning_count = count( $queue );
-	?>
+    $queue         = $this->batch_analysis_service->get_warnings();
+    $warning_count = count( $queue );
+    ?>
 
 	<h3><?php esc_html_e( 'Posts with warnings', 'wordlift' ) ?><?php echo " ($warning_count)"; ?></h3>
 	<?php
-	if ( empty( $queue ) ) {
-		esc_html_e( 'No warnings :-)', 'wordlift' );
-	} else {
-		echo '<ul>';
-		foreach ( $queue as $pid ) {
-			$cancel_link = admin_url( "admin-ajax.php?action=wl_batch_analysis_clear_warning&post=$pid" );
-			?>
+    if ( empty( $queue ) ) {
+        esc_html_e( 'No warnings :-)', 'wordlift' );
+    } else {
+        echo '<ul>';
+        foreach ( $queue as $pid ) {
+            $cancel_link = admin_url( "admin-ajax.php?action=wl_batch_analysis_clear_warning&post=$pid" );
+            ?>
 			<li><a href="<?php echo get_edit_post_link( $pid ) ?>"><?php
-					echo get_the_title( $pid ) ?></a> [<a
+                    echo get_the_title( $pid ) ?></a> [<a
 					href="<?php echo esc_attr( $cancel_link ); ?> "><?php esc_html_e( 'Clear warning', 'wordlift' ); ?></a>]
 			</li>
 			<?php
-		}
-		echo '</ul>';
-	}
-	?>
+        }
+        echo '</ul>';
+    }
+    ?>
 </div>

Spacing +28 added lines, -28 removed lines

@@ -8,24 +8,24 @@ 
 ?>
 
 <div class="wrap">
-	<h2><?php esc_html_e( 'WordLift Batch Analyze Monitor', 'wordlift' ) ?></h2>
+	<h2><?php esc_html_e('WordLift Batch Analyze Monitor', 'wordlift') ?></h2>
 
 	<?php
 	$queue        = $this->batch_analysis_service->waiting_for_analysis();
-	$submit_count = count( $queue );
+	$submit_count = count($queue);
 	?>
-	<h3><?php esc_html_e( 'Posts in queue', 'wordlift' ) ?><?php echo " ($submit_count)"; ?></h3>
+	<h3><?php esc_html_e('Posts in queue', 'wordlift') ?><?php echo " ($submit_count)"; ?></h3>
 	<?php
-	if ( empty( $queue ) ) {
-		esc_html_e( 'Nothing is currently in queue', 'wordlift' );
+	if (empty($queue)) {
+		esc_html_e('Nothing is currently in queue', 'wordlift');
 	} else {
 		echo '<ul>';
-		foreach ( $queue as $pid ) {
-			$cancel_link = admin_url( "admin-ajax.php?action=wl_batch_analysis_cancel&post=$pid" );
+		foreach ($queue as $pid) {
+			$cancel_link = admin_url("admin-ajax.php?action=wl_batch_analysis_cancel&post=$pid");
 			?>
-			<li><a href="<?php echo get_edit_post_link( $pid ) ?>"><?php
-					echo get_the_title( $pid ) ?></a> [<a
-					href="<?php echo esc_attr( $cancel_link ); ?> "><?php esc_html_e( 'Cancel', 'wordlift' ); ?></a>]
+			<li><a href="<?php echo get_edit_post_link($pid) ?>"><?php
+					echo get_the_title($pid) ?></a> [<a
+					href="<?php echo esc_attr($cancel_link); ?> "><?php esc_html_e('Cancel', 'wordlift'); ?></a>]
 			</li>
 			<?php
 		}
@@ -33,20 +33,20 @@ 
 	}
 
 	$queue         = $this->batch_analysis_service->waiting_for_response();
-	$request_count = count( $queue );
+	$request_count = count($queue);
 	?>
-	<h3><?php esc_html_e( 'Posts being processed', 'wordlift' ) ?><?php echo " ($request_count)"; ?></h3>
+	<h3><?php esc_html_e('Posts being processed', 'wordlift') ?><?php echo " ($request_count)"; ?></h3>
 	<?php
-	if ( empty( $queue ) ) {
-		esc_html_e( 'Nothing is currently being processed', 'wordlift' );
+	if (empty($queue)) {
+		esc_html_e('Nothing is currently being processed', 'wordlift');
 	} else {
 		echo '<ul>';
-		foreach ( $queue as $pid ) {
-			$cancel_link = admin_url( "admin-ajax.php?action=wl_batch_analysis_cancel&post=$pid" );
+		foreach ($queue as $pid) {
+			$cancel_link = admin_url("admin-ajax.php?action=wl_batch_analysis_cancel&post=$pid");
 			?>
-			<li><a href="<?php echo get_edit_post_link( $pid ) ?>"><?php
-					echo get_the_title( $pid ) ?></a> [<a
-					href="<?php echo esc_attr( $cancel_link ); ?> "><?php esc_html_e( 'Cancel', 'wordlift' ); ?></a>]
+			<li><a href="<?php echo get_edit_post_link($pid) ?>"><?php
+					echo get_the_title($pid) ?></a> [<a
+					href="<?php echo esc_attr($cancel_link); ?> "><?php esc_html_e('Cancel', 'wordlift'); ?></a>]
 			</li>
 			<?php
 		}
@@ -54,21 +54,21 @@ 
 	}
 
 	$queue         = $this->batch_analysis_service->get_warnings();
-	$warning_count = count( $queue );
+	$warning_count = count($queue);
 	?>
 
-	<h3><?php esc_html_e( 'Posts with warnings', 'wordlift' ) ?><?php echo " ($warning_count)"; ?></h3>
+	<h3><?php esc_html_e('Posts with warnings', 'wordlift') ?><?php echo " ($warning_count)"; ?></h3>
 	<?php
-	if ( empty( $queue ) ) {
-		esc_html_e( 'No warnings :-)', 'wordlift' );
+	if (empty($queue)) {
+		esc_html_e('No warnings :-)', 'wordlift');
 	} else {
 		echo '<ul>';
-		foreach ( $queue as $pid ) {
-			$cancel_link = admin_url( "admin-ajax.php?action=wl_batch_analysis_clear_warning&post=$pid" );
+		foreach ($queue as $pid) {
+			$cancel_link = admin_url("admin-ajax.php?action=wl_batch_analysis_clear_warning&post=$pid");
 			?>
-			<li><a href="<?php echo get_edit_post_link( $pid ) ?>"><?php
-					echo get_the_title( $pid ) ?></a> [<a
-					href="<?php echo esc_attr( $cancel_link ); ?> "><?php esc_html_e( 'Clear warning', 'wordlift' ); ?></a>]
+			<li><a href="<?php echo get_edit_post_link($pid) ?>"><?php
+					echo get_the_title($pid) ?></a> [<a
+					href="<?php echo esc_attr($cancel_link); ?> "><?php esc_html_e('Clear warning', 'wordlift'); ?></a>]
 			</li>
 			<?php
 		}

src/includes/wp-async-task/class-wordlift-batch-analysis-complete-async-task.php

Indentation +45 added lines, -45 removed lines

@@ -18,59 +18,59 @@
  */
 class Wordlift_Batch_Analysis_Complete_Async_Task extends WP_Async_Task {
 
-	/**
-	 * The protected $action property should be set to the action to which you
-	 * wish to attach the asynchronous task. For example, if you want to spin
-	 * off an asynchronous task whenever a post gets saved, you would set this
-	 * to save_post.
-	 *
-	 * @since  3.14.2
-	 * @access protected
-	 * @var string $action The action to which you wish to attach the
-	 *                     asynchronous task.
-	 */
-	protected $action = 'wl_batch_analysis_complete';
+    /**
+     * The protected $action property should be set to the action to which you
+     * wish to attach the asynchronous task. For example, if you want to spin
+     * off an asynchronous task whenever a post gets saved, you would set this
+     * to save_post.
+     *
+     * @since  3.14.2
+     * @access protected
+     * @var string $action The action to which you wish to attach the
+     *                     asynchronous task.
+     */
+    protected $action = 'wl_batch_analysis_complete';
 
-	/**
-	 * A {@link Wordlift_Log_Service} instance.
-	 *
-	 * @since  3.14.2
-	 * @access private
-	 * @var \Wordlift_Log_Service $log A {@link Wordlift_Log_Service} instance.
-	 */
-	private $log;
+    /**
+     * A {@link Wordlift_Log_Service} instance.
+     *
+     * @since  3.14.2
+     * @access private
+     * @var \Wordlift_Log_Service $log A {@link Wordlift_Log_Service} instance.
+     */
+    private $log;
 
-	/**
-	 * Create a {@link Wordlift_Sparql_Query_Async_Task} instance.
-	 *
-	 * @since 3.14.2
-	 *
-	 * @param int $auth_level The authentication level to use (see above)
-	 */
-	public function __construct( $auth_level = self::BOTH ) {
-		parent::__construct( $auth_level );
+    /**
+     * Create a {@link Wordlift_Sparql_Query_Async_Task} instance.
+     *
+     * @since 3.14.2
+     *
+     * @param int $auth_level The authentication level to use (see above)
+     */
+    public function __construct( $auth_level = self::BOTH ) {
+        parent::__construct( $auth_level );
 
-		$this->log = Wordlift_Log_Service::get_logger( 'Wordlift_Batch_Analysis_Complete_Async_Task' );
+        $this->log = Wordlift_Log_Service::get_logger( 'Wordlift_Batch_Analysis_Complete_Async_Task' );
 
-	}
+    }
 
-	/**
-	 * @inheritdoc
-	 */
-	protected function prepare_data( $data ) {
+    /**
+     * @inheritdoc
+     */
+    protected function prepare_data( $data ) {
 
-		// Return the link setting.
-		return array();
-	}
+        // Return the link setting.
+        return array();
+    }
 
-	/**
-	 * @inheritdoc
-	 */
-	protected function run_action() {
+    /**
+     * @inheritdoc
+     */
+    protected function run_action() {
 
-		// Run the asynchronous action.
-		do_action( "wp_async_$this->action" );
+        // Run the asynchronous action.
+        do_action( "wp_async_$this->action" );
 
-	}
+    }
 
 }

Spacing +5 added lines, -5 removed lines

@@ -47,17 +47,17 @@ 
 	 *
 	 * @param int $auth_level The authentication level to use (see above)
 	 */
-	public function __construct( $auth_level = self::BOTH ) {
-		parent::__construct( $auth_level );
+	public function __construct($auth_level = self::BOTH) {
+		parent::__construct($auth_level);
 
-		$this->log = Wordlift_Log_Service::get_logger( 'Wordlift_Batch_Analysis_Complete_Async_Task' );
+		$this->log = Wordlift_Log_Service::get_logger('Wordlift_Batch_Analysis_Complete_Async_Task');
 
 	}
 
 	/**
 	 * @inheritdoc
 	 */
-	protected function prepare_data( $data ) {
+	protected function prepare_data($data) {
 
 		// Return the link setting.
 		return array();
@@ -69,7 +69,7 @@ 
 	protected function run_action() {
 
 		// Run the asynchronous action.
-		do_action( "wp_async_$this->action" );
+		do_action("wp_async_$this->action");
 
 	}
 

src/includes/wp-async-task/wp-async-task.php

Indentation +269 added lines, -269 removed lines

@@ -8,275 +8,275 @@
  */
 
 if ( ! class_exists( 'WP_Async_Task' ) ) {
-	abstract class WP_Async_Task {
-
-		/**
-		 * Constant identifier for a task that should be available to logged-in users
-		 *
-		 * See constructor documentation for more details.
-		 */
-		const LOGGED_IN = 1;
-
-		/**
-		 * Constant identifier for a task that should be available to logged-out users
-		 *
-		 * See constructor documentation for more details.
-		 */
-		const LOGGED_OUT = 2;
-
-		/**
-		 * Constant identifier for a task that should be available to all users regardless of auth status
-		 *
-		 * See constructor documentation for more details.
-		 */
-		const BOTH = 3;
-
-		/**
-		 * This is the argument count for the main action set in the constructor. It
-		 * is set to an arbitrarily high value of twenty, but can be overridden if
-		 * necessary
-		 *
-		 * @var int
-		 */
-		protected $argument_count = 20;
-
-		/**
-		 * Priority to fire intermediate action.
-		 *
-		 * @var int
-		 */
-		protected $priority = 10;
-
-		/**
-		 * @var string
-		 */
-		protected $action;
-
-		/**
-		 * @var array
-		 */
-		protected $_body_data;
-
-		/**
-		 * Constructor to wire up the necessary actions
-		 *
-		 * Which hooks the asynchronous postback happens on can be set by the
-		 * $auth_level parameter. There are essentially three options: logged in users
-		 * only, logged out users only, or both. Set this when you instantiate an
-		 * object by using one of the three class constants to do so:
-		 *  - LOGGED_IN
-		 *  - LOGGED_OUT
-		 *  - BOTH
-		 * $auth_level defaults to BOTH
-		 *
-		 * @throws Exception If the class' $action value hasn't been set
-		 *
-		 * @param int $auth_level The authentication level to use (see above)
-		 */
-		public function __construct( $auth_level = self::BOTH ) {
-			if ( empty( $this->action ) ) {
-				throw new Exception( 'Action not defined for class ' . __CLASS__ );
-			}
-			add_action( $this->action, array(
-				$this,
-				'launch',
-			), (int) $this->priority, (int) $this->argument_count );
-			if ( $auth_level & self::LOGGED_IN ) {
-				add_action( "admin_post_wp_async_$this->action", array(
-					$this,
-					'handle_postback',
-				) );
-			}
-			if ( $auth_level & self::LOGGED_OUT ) {
-				add_action( "admin_post_nopriv_wp_async_$this->action", array(
-					$this,
-					'handle_postback',
-				) );
-			}
-		}
-
-		/**
-		 * Add the shutdown action for launching the real postback if we don't
-		 * get an exception thrown by prepare_data().
-		 *
-		 * @uses func_get_args() To grab any arguments passed by the action
-		 */
-		public function launch() {
-			$data = func_get_args();
-			try {
-				$data = $this->prepare_data( $data );
-			} catch ( Exception $e ) {
-				return;
-			}
-
-			$data['action'] = "wp_async_$this->action";
-			$data['_nonce'] = $this->create_async_nonce();
-
-			$this->_body_data = $data;
-
-			if ( ! has_action( 'shutdown', array(
-				$this,
-				'launch_on_shutdown',
-			) )
-			) {
-				add_action( 'shutdown', array( $this, 'launch_on_shutdown' ) );
-			}
-		}
-
-		/**
-		 * Launch the request on the WordPress shutdown hook
-		 *
-		 * On VIP we got into data races due to the postback sometimes completing
-		 * faster than the data could propogate to the database server cluster.
-		 * This made WordPress get empty data sets from the database without
-		 * failing. On their advice, we're moving the actual firing of the async
-		 * postback to the shutdown hook. Supposedly that will ensure that the
-		 * data at least has time to get into the object cache.
-		 *
-		 * @uses $_COOKIE        To send a cookie header for async postback
-		 * @uses apply_filters()
-		 * @uses admin_url()
-		 * @uses wp_remote_post()
-		 */
-		public function launch_on_shutdown() {
-			if ( ! empty( $this->_body_data ) ) {
-				$cookies = array();
-				foreach ( $_COOKIE as $name => $value ) {
-					$cookies[] = "$name=" . urlencode( is_array( $value ) ? serialize( $value ) : $value );
-				}
-
-				$request_args = array(
-					'timeout'   => 0.01,
-					'blocking'  => false,
-					'sslverify' => apply_filters( 'https_local_ssl_verify', false ),
-					'body'      => $this->_body_data,
-					'headers'   => array(
-						'cookie' => implode( '; ', $cookies ),
-					),
-				);
-
-				$url = admin_url( 'admin-post.php' );
-
-				wp_remote_post( $url, $request_args );
-			}
-		}
-
-		/**
-		 * Verify the postback is valid, then fire any scheduled events.
-		 *
-		 * @uses $_POST['_nonce']
-		 * @uses is_user_logged_in()
-		 * @uses add_filter()
-		 * @uses wp_die()
-		 */
-		public function handle_postback() {
-			if ( isset( $_POST['_nonce'] ) && $this->verify_async_nonce( $_POST['_nonce'] ) ) {
-				if ( ! is_user_logged_in() ) {
-					$this->action = "nopriv_$this->action";
-				}
-				$this->run_action();
-			}
-
-			add_filter( 'wp_die_handler', function () {
-				die();
-			} );
-			wp_die();
-		}
-
-		/**
-		 * Create a random, one time use token.
-		 *
-		 * Based entirely on wp_create_nonce() but does not tie the nonce to the
-		 * current logged-in user.
-		 *
-		 * @uses wp_nonce_tick()
-		 * @uses wp_hash()
-		 *
-		 * @return string The one-time use token
-		 */
-		protected function create_async_nonce() {
-			$action = $this->get_nonce_action();
-			$i      = wp_nonce_tick();
-
-			return substr( wp_hash( $i . $action . get_class( $this ), 'nonce' ), - 12, 10 );
-		}
-
-		/**
-		 * Verify that the correct nonce was used within the time limit.
-		 *
-		 * @uses wp_nonce_tick()
-		 * @uses wp_hash()
-		 *
-		 * @param string $nonce Nonce to be verified
-		 *
-		 * @return bool Whether the nonce check passed or failed
-		 */
-		protected function verify_async_nonce( $nonce ) {
-			$action = $this->get_nonce_action();
-			$i      = wp_nonce_tick();
-
-			// Nonce generated 0-12 hours ago
-			if ( substr( wp_hash( $i . $action . get_class( $this ), 'nonce' ), - 12, 10 ) == $nonce ) {
-				return 1;
-			}
-
-			// Nonce generated 12-24 hours ago
-			if ( substr( wp_hash( ( $i - 1 ) . $action . get_class( $this ), 'nonce' ), - 12, 10 ) == $nonce ) {
-				return 2;
-			}
-
-			// Invalid nonce
-			return false;
-		}
-
-		/**
-		 * Get a nonce action based on the $action property of the class
-		 *
-		 * @return string The nonce action for the current instance
-		 */
-		protected function get_nonce_action() {
-			$action = $this->action;
-			if ( substr( $action, 0, 7 ) === 'nopriv_' ) {
-				$action = substr( $action, 7 );
-			}
-			$action = "wp_async_$action";
-
-			return $action;
-		}
-
-		/**
-		 * Prepare any data to be passed to the asynchronous postback
-		 *
-		 * The array this function receives will be a numerically keyed array from
-		 * func_get_args(). It is expected that you will return an associative array
-		 * so that the $_POST values used in the asynchronous call will make sense.
-		 *
-		 * The array you send back may or may not have anything to do with the data
-		 * passed into this method. It all depends on the implementation details and
-		 * what data is needed in the asynchronous postback.
-		 *
-		 * Do not set values for 'action' or '_nonce', as those will get overwritten
-		 * later in launch().
-		 *
-		 * @throws Exception If the postback should not occur for any reason
-		 *
-		 * @param array $data The raw data received by the launch method
-		 *
-		 * @return array The prepared data
-		 */
-		abstract protected function prepare_data( $data );
-
-		/**
-		 * Run the do_action function for the asynchronous postback.
-		 *
-		 * This method needs to fetch and sanitize any and all data from the $_POST
-		 * superglobal and provide them to the do_action call.
-		 *
-		 * The action should be constructed as "wp_async_task_$this->action"
-		 */
-		abstract protected function run_action();
-
-	}
+    abstract class WP_Async_Task {
+
+        /**
+         * Constant identifier for a task that should be available to logged-in users
+         *
+         * See constructor documentation for more details.
+         */
+        const LOGGED_IN = 1;
+
+        /**
+         * Constant identifier for a task that should be available to logged-out users
+         *
+         * See constructor documentation for more details.
+         */
+        const LOGGED_OUT = 2;
+
+        /**
+         * Constant identifier for a task that should be available to all users regardless of auth status
+         *
+         * See constructor documentation for more details.
+         */
+        const BOTH = 3;
+
+        /**
+         * This is the argument count for the main action set in the constructor. It
+         * is set to an arbitrarily high value of twenty, but can be overridden if
+         * necessary
+         *
+         * @var int
+         */
+        protected $argument_count = 20;
+
+        /**
+         * Priority to fire intermediate action.
+         *
+         * @var int
+         */
+        protected $priority = 10;
+
+        /**
+         * @var string
+         */
+        protected $action;
+
+        /**
+         * @var array
+         */
+        protected $_body_data;
+
+        /**
+         * Constructor to wire up the necessary actions
+         *
+         * Which hooks the asynchronous postback happens on can be set by the
+         * $auth_level parameter. There are essentially three options: logged in users
+         * only, logged out users only, or both. Set this when you instantiate an
+         * object by using one of the three class constants to do so:
+         *  - LOGGED_IN
+         *  - LOGGED_OUT
+         *  - BOTH
+         * $auth_level defaults to BOTH
+         *
+         * @throws Exception If the class' $action value hasn't been set
+         *
+         * @param int $auth_level The authentication level to use (see above)
+         */
+        public function __construct( $auth_level = self::BOTH ) {
+            if ( empty( $this->action ) ) {
+                throw new Exception( 'Action not defined for class ' . __CLASS__ );
+            }
+            add_action( $this->action, array(
+                $this,
+                'launch',
+            ), (int) $this->priority, (int) $this->argument_count );
+            if ( $auth_level & self::LOGGED_IN ) {
+                add_action( "admin_post_wp_async_$this->action", array(
+                    $this,
+                    'handle_postback',
+                ) );
+            }
+            if ( $auth_level & self::LOGGED_OUT ) {
+                add_action( "admin_post_nopriv_wp_async_$this->action", array(
+                    $this,
+                    'handle_postback',
+                ) );
+            }
+        }
+
+        /**
+         * Add the shutdown action for launching the real postback if we don't
+         * get an exception thrown by prepare_data().
+         *
+         * @uses func_get_args() To grab any arguments passed by the action
+         */
+        public function launch() {
+            $data = func_get_args();
+            try {
+                $data = $this->prepare_data( $data );
+            } catch ( Exception $e ) {
+                return;
+            }
+
+            $data['action'] = "wp_async_$this->action";
+            $data['_nonce'] = $this->create_async_nonce();
+
+            $this->_body_data = $data;
+
+            if ( ! has_action( 'shutdown', array(
+                $this,
+                'launch_on_shutdown',
+            ) )
+            ) {
+                add_action( 'shutdown', array( $this, 'launch_on_shutdown' ) );
+            }
+        }
+
+        /**
+         * Launch the request on the WordPress shutdown hook
+         *
+         * On VIP we got into data races due to the postback sometimes completing
+         * faster than the data could propogate to the database server cluster.
+         * This made WordPress get empty data sets from the database without
+         * failing. On their advice, we're moving the actual firing of the async
+         * postback to the shutdown hook. Supposedly that will ensure that the
+         * data at least has time to get into the object cache.
+         *
+         * @uses $_COOKIE        To send a cookie header for async postback
+         * @uses apply_filters()
+         * @uses admin_url()
+         * @uses wp_remote_post()
+         */
+        public function launch_on_shutdown() {
+            if ( ! empty( $this->_body_data ) ) {
+                $cookies = array();
+                foreach ( $_COOKIE as $name => $value ) {
+                    $cookies[] = "$name=" . urlencode( is_array( $value ) ? serialize( $value ) : $value );
+                }
+
+                $request_args = array(
+                    'timeout'   => 0.01,
+                    'blocking'  => false,
+                    'sslverify' => apply_filters( 'https_local_ssl_verify', false ),
+                    'body'      => $this->_body_data,
+                    'headers'   => array(
+                        'cookie' => implode( '; ', $cookies ),
+                    ),
+                );
+
+                $url = admin_url( 'admin-post.php' );
+
+                wp_remote_post( $url, $request_args );
+            }
+        }
+
+        /**
+         * Verify the postback is valid, then fire any scheduled events.
+         *
+         * @uses $_POST['_nonce']
+         * @uses is_user_logged_in()
+         * @uses add_filter()
+         * @uses wp_die()
+         */
+        public function handle_postback() {
+            if ( isset( $_POST['_nonce'] ) && $this->verify_async_nonce( $_POST['_nonce'] ) ) {
+                if ( ! is_user_logged_in() ) {
+                    $this->action = "nopriv_$this->action";
+                }
+                $this->run_action();
+            }
+
+            add_filter( 'wp_die_handler', function () {
+                die();
+            } );
+            wp_die();
+        }
+
+        /**
+         * Create a random, one time use token.
+         *
+         * Based entirely on wp_create_nonce() but does not tie the nonce to the
+         * current logged-in user.
+         *
+         * @uses wp_nonce_tick()
+         * @uses wp_hash()
+         *
+         * @return string The one-time use token
+         */
+        protected function create_async_nonce() {
+            $action = $this->get_nonce_action();
+            $i      = wp_nonce_tick();
+
+            return substr( wp_hash( $i . $action . get_class( $this ), 'nonce' ), - 12, 10 );
+        }
+
+        /**
+         * Verify that the correct nonce was used within the time limit.
+         *
+         * @uses wp_nonce_tick()
+         * @uses wp_hash()
+         *
+         * @param string $nonce Nonce to be verified
+         *
+         * @return bool Whether the nonce check passed or failed
+         */
+        protected function verify_async_nonce( $nonce ) {
+            $action = $this->get_nonce_action();
+            $i      = wp_nonce_tick();
+
+            // Nonce generated 0-12 hours ago
+            if ( substr( wp_hash( $i . $action . get_class( $this ), 'nonce' ), - 12, 10 ) == $nonce ) {
+                return 1;
+            }
+
+            // Nonce generated 12-24 hours ago
+            if ( substr( wp_hash( ( $i - 1 ) . $action . get_class( $this ), 'nonce' ), - 12, 10 ) == $nonce ) {
+                return 2;
+            }
+
+            // Invalid nonce
+            return false;
+        }
+
+        /**
+         * Get a nonce action based on the $action property of the class
+         *
+         * @return string The nonce action for the current instance
+         */
+        protected function get_nonce_action() {
+            $action = $this->action;
+            if ( substr( $action, 0, 7 ) === 'nopriv_' ) {
+                $action = substr( $action, 7 );
+            }
+            $action = "wp_async_$action";
+
+            return $action;
+        }
+
+        /**
+         * Prepare any data to be passed to the asynchronous postback
+         *
+         * The array this function receives will be a numerically keyed array from
+         * func_get_args(). It is expected that you will return an associative array
+         * so that the $_POST values used in the asynchronous call will make sense.
+         *
+         * The array you send back may or may not have anything to do with the data
+         * passed into this method. It all depends on the implementation details and
+         * what data is needed in the asynchronous postback.
+         *
+         * Do not set values for 'action' or '_nonce', as those will get overwritten
+         * later in launch().
+         *
+         * @throws Exception If the postback should not occur for any reason
+         *
+         * @param array $data The raw data received by the launch method
+         *
+         * @return array The prepared data
+         */
+        abstract protected function prepare_data( $data );
+
+        /**
+         * Run the do_action function for the asynchronous postback.
+         *
+         * This method needs to fetch and sanitize any and all data from the $_POST
+         * superglobal and provide them to the do_action call.
+         *
+         * The action should be constructed as "wp_async_task_$this->action"
+         */
+        abstract protected function run_action();
+
+    }
 
 }
 

Spacing +34 added lines, -34 removed lines

@@ -7,7 +7,7 @@ 
  * License: MIT
  */
 
-if ( ! class_exists( 'WP_Async_Task' ) ) {
+if ( ! class_exists('WP_Async_Task')) {
 	abstract class WP_Async_Task {
 
 		/**
@@ -73,25 +73,25 @@ 
 		 *
 		 * @param int $auth_level The authentication level to use (see above)
 		 */
-		public function __construct( $auth_level = self::BOTH ) {
-			if ( empty( $this->action ) ) {
-				throw new Exception( 'Action not defined for class ' . __CLASS__ );
+		public function __construct($auth_level = self::BOTH) {
+			if (empty($this->action)) {
+				throw new Exception('Action not defined for class '.__CLASS__);
 			}
-			add_action( $this->action, array(
+			add_action($this->action, array(
 				$this,
 				'launch',
-			), (int) $this->priority, (int) $this->argument_count );
-			if ( $auth_level & self::LOGGED_IN ) {
-				add_action( "admin_post_wp_async_$this->action", array(
+			), (int) $this->priority, (int) $this->argument_count);
+			if ($auth_level & self::LOGGED_IN) {
+				add_action("admin_post_wp_async_$this->action", array(
 					$this,
 					'handle_postback',
-				) );
+				));
 			}
-			if ( $auth_level & self::LOGGED_OUT ) {
-				add_action( "admin_post_nopriv_wp_async_$this->action", array(
+			if ($auth_level & self::LOGGED_OUT) {
+				add_action("admin_post_nopriv_wp_async_$this->action", array(
 					$this,
 					'handle_postback',
-				) );
+				));
 			}
 		}
 
@@ -104,8 +104,8 @@ 
 		public function launch() {
 			$data = func_get_args();
 			try {
-				$data = $this->prepare_data( $data );
-			} catch ( Exception $e ) {
+				$data = $this->prepare_data($data);
+			} catch (Exception $e) {
 				return;
 			}
 
@@ -114,12 +114,12 @@ 
 
 			$this->_body_data = $data;
 
-			if ( ! has_action( 'shutdown', array(
+			if ( ! has_action('shutdown', array(
 				$this,
 				'launch_on_shutdown',
-			) )
+			))
 			) {
-				add_action( 'shutdown', array( $this, 'launch_on_shutdown' ) );
+				add_action('shutdown', array($this, 'launch_on_shutdown'));
 			}
 		}
 
@@ -139,25 +139,25 @@ 
 		 * @uses wp_remote_post()
 		 */
 		public function launch_on_shutdown() {
-			if ( ! empty( $this->_body_data ) ) {
+			if ( ! empty($this->_body_data)) {
 				$cookies = array();
-				foreach ( $_COOKIE as $name => $value ) {
-					$cookies[] = "$name=" . urlencode( is_array( $value ) ? serialize( $value ) : $value );
+				foreach ($_COOKIE as $name => $value) {
+					$cookies[] = "$name=".urlencode(is_array($value) ? serialize($value) : $value);
 				}
 
 				$request_args = array(
 					'timeout'   => 0.01,
 					'blocking'  => false,
-					'sslverify' => apply_filters( 'https_local_ssl_verify', false ),
+					'sslverify' => apply_filters('https_local_ssl_verify', false),
 					'body'      => $this->_body_data,
 					'headers'   => array(
-						'cookie' => implode( '; ', $cookies ),
+						'cookie' => implode('; ', $cookies),
 					),
 				);
 
-				$url = admin_url( 'admin-post.php' );
+				$url = admin_url('admin-post.php');
 
-				wp_remote_post( $url, $request_args );
+				wp_remote_post($url, $request_args);
 			}
 		}
 
@@ -170,14 +170,14 @@ 
 		 * @uses wp_die()
 		 */
 		public function handle_postback() {
-			if ( isset( $_POST['_nonce'] ) && $this->verify_async_nonce( $_POST['_nonce'] ) ) {
-				if ( ! is_user_logged_in() ) {
+			if (isset($_POST['_nonce']) && $this->verify_async_nonce($_POST['_nonce'])) {
+				if ( ! is_user_logged_in()) {
 					$this->action = "nopriv_$this->action";
 				}
 				$this->run_action();
 			}
 
-			add_filter( 'wp_die_handler', function () {
+			add_filter('wp_die_handler', function() {
 				die();
 			} );
 			wp_die();
@@ -198,7 +198,7 @@ 
 			$action = $this->get_nonce_action();
 			$i      = wp_nonce_tick();
 
-			return substr( wp_hash( $i . $action . get_class( $this ), 'nonce' ), - 12, 10 );
+			return substr(wp_hash($i.$action.get_class($this), 'nonce'), - 12, 10);
 		}
 
 		/**
@@ -211,17 +211,17 @@ 
 		 *
 		 * @return bool Whether the nonce check passed or failed
 		 */
-		protected function verify_async_nonce( $nonce ) {
+		protected function verify_async_nonce($nonce) {
 			$action = $this->get_nonce_action();
 			$i      = wp_nonce_tick();
 
 			// Nonce generated 0-12 hours ago
-			if ( substr( wp_hash( $i . $action . get_class( $this ), 'nonce' ), - 12, 10 ) == $nonce ) {
+			if (substr(wp_hash($i.$action.get_class($this), 'nonce'), - 12, 10) == $nonce) {
 				return 1;
 			}
 
 			// Nonce generated 12-24 hours ago
-			if ( substr( wp_hash( ( $i - 1 ) . $action . get_class( $this ), 'nonce' ), - 12, 10 ) == $nonce ) {
+			if (substr(wp_hash(($i - 1).$action.get_class($this), 'nonce'), - 12, 10) == $nonce) {
 				return 2;
 			}
 
@@ -236,8 +236,8 @@ 
 		 */
 		protected function get_nonce_action() {
 			$action = $this->action;
-			if ( substr( $action, 0, 7 ) === 'nopriv_' ) {
-				$action = substr( $action, 7 );
+			if (substr($action, 0, 7) === 'nopriv_') {
+				$action = substr($action, 7);
 			}
 			$action = "wp_async_$action";
 
@@ -264,7 +264,7 @@ 
 		 *
 		 * @return array The prepared data
 		 */
-		abstract protected function prepare_data( $data );
+		abstract protected function prepare_data($data);
 
 		/**
 		 * Run the do_action function for the asynchronous postback.

src/includes/wp-async-task/class-wordlift-batch-analysis-request-async-task.php

Indentation +45 added lines, -45 removed lines

@@ -18,59 +18,59 @@
  */
 class Wordlift_Batch_Analysis_Request_Async_Task extends WP_Async_Task {
 
-	/**
-	 * The protected $action property should be set to the action to which you
-	 * wish to attach the asynchronous task. For example, if you want to spin
-	 * off an asynchronous task whenever a post gets saved, you would set this
-	 * to save_post.
-	 *
-	 * @since  3.14.2
-	 * @access protected
-	 * @var string $action The action to which you wish to attach the
-	 *                     asynchronous task.
-	 */
-	protected $action = 'wl_batch_analysis_request';
+    /**
+     * The protected $action property should be set to the action to which you
+     * wish to attach the asynchronous task. For example, if you want to spin
+     * off an asynchronous task whenever a post gets saved, you would set this
+     * to save_post.
+     *
+     * @since  3.14.2
+     * @access protected
+     * @var string $action The action to which you wish to attach the
+     *                     asynchronous task.
+     */
+    protected $action = 'wl_batch_analysis_request';
 
-	/**
-	 * A {@link Wordlift_Log_Service} instance.
-	 *
-	 * @since  3.14.2
-	 * @access private
-	 * @var \Wordlift_Log_Service $log A {@link Wordlift_Log_Service} instance.
-	 */
-	private $log;
+    /**
+     * A {@link Wordlift_Log_Service} instance.
+     *
+     * @since  3.14.2
+     * @access private
+     * @var \Wordlift_Log_Service $log A {@link Wordlift_Log_Service} instance.
+     */
+    private $log;
 
-	/**
-	 * Create a {@link Wordlift_Sparql_Query_Async_Task} instance.
-	 *
-	 * @since 3.14.2
-	 *
-	 * @param int $auth_level The authentication level to use (see above)
-	 */
-	public function __construct( $auth_level = self::BOTH ) {
-		parent::__construct( $auth_level );
+    /**
+     * Create a {@link Wordlift_Sparql_Query_Async_Task} instance.
+     *
+     * @since 3.14.2
+     *
+     * @param int $auth_level The authentication level to use (see above)
+     */
+    public function __construct( $auth_level = self::BOTH ) {
+        parent::__construct( $auth_level );
 
-		$this->log = Wordlift_Log_Service::get_logger( 'Wordlift_Batch_Analysis_Request_Async_Task' );
+        $this->log = Wordlift_Log_Service::get_logger( 'Wordlift_Batch_Analysis_Request_Async_Task' );
 
-	}
+    }
 
-	/**
-	 * @inheritdoc
-	 */
-	protected function prepare_data( $data ) {
+    /**
+     * @inheritdoc
+     */
+    protected function prepare_data( $data ) {
 
-		// Return the link setting.
-		return array();
-	}
+        // Return the link setting.
+        return array();
+    }
 
-	/**
-	 * @inheritdoc
-	 */
-	protected function run_action() {
+    /**
+     * @inheritdoc
+     */
+    protected function run_action() {
 
-		// Run the asynchronous action.
-		do_action( "wp_async_$this->action" );
+        // Run the asynchronous action.
+        do_action( "wp_async_$this->action" );
 
-	}
+    }
 
 }

Spacing +5 added lines, -5 removed lines

@@ -47,17 +47,17 @@ 
 	 *
 	 * @param int $auth_level The authentication level to use (see above)
 	 */
-	public function __construct( $auth_level = self::BOTH ) {
-		parent::__construct( $auth_level );
+	public function __construct($auth_level = self::BOTH) {
+		parent::__construct($auth_level);
 
-		$this->log = Wordlift_Log_Service::get_logger( 'Wordlift_Batch_Analysis_Request_Async_Task' );
+		$this->log = Wordlift_Log_Service::get_logger('Wordlift_Batch_Analysis_Request_Async_Task');
 
 	}
 
 	/**
 	 * @inheritdoc
 	 */
-	protected function prepare_data( $data ) {
+	protected function prepare_data($data) {
 
 		// Return the link setting.
 		return array();
@@ -69,7 +69,7 @@ 
 	protected function run_action() {
 
 		// Run the asynchronous action.
-		do_action( "wp_async_$this->action" );
+		do_action("wp_async_$this->action");
 
 	}
 

src/includes/class-wordlift-content-filter-service.php

Indentation +156 added lines, -156 removed lines

@@ -18,161 +18,161 @@
  */
 class Wordlift_Content_Filter_Service {
 
-	/**
-	 * The pattern to find entities in text.
-	 *
-	 * @since 3.8.0
-	 */
-	const PATTERN = '/<(\\w+)[^<]*class="([^"]*)"\\sitemid=\"([^"]+)\"[^>]*>([^<]*)<\\/\\1>/i';
-
-	/**
-	 * A {@link Wordlift_Entity_Service} instance.
-	 *
-	 * @since  3.8.0
-	 * @access private
-	 * @var \Wordlift_Entity_Service $entity_service A {@link Wordlift_Entity_Service} instance.
-	 */
-	private $entity_service;
-
-	/**
-	 * The {@link Wordlift_Configuration_Service} instance.
-	 *
-	 * @since  3.13.0
-	 * @access private
-	 * @var \Wordlift_Configuration_Service $configuration_service The {@link Wordlift_Configuration_Service} instance.
-	 */
-	private $configuration_service;
-
-	/**
-	 * The `link by default` setting.
-	 *
-	 * @since  3.13.0
-	 * @access private
-	 * @var bool True if link by default is enabled otherwise false.
-	 */
-	private $is_link_by_default;
-
-	/**
-	 * The {@link Wordlift_Content_Filter_Service} singleton instance.
-	 *
-	 * @since  3.14.2
-	 * @access private
-	 * @var \Wordlift_Content_Filter_Service $instance The {@link Wordlift_Content_Filter_Service} singleton instance.
-	 */
-	private static $instance;
-
-	/**
-	 * Create a {@link Wordlift_Content_Filter_Service} instance.
-	 *
-	 * @since 3.8.0
-	 *
-	 * @param \Wordlift_Entity_Service        $entity_service        The {@link Wordlift_Entity_Service} instance.
-	 * @param \Wordlift_Configuration_Service $configuration_service The {@link Wordlift_Configuration_Service} instance.
-	 */
-	public function __construct( $entity_service, $configuration_service ) {
-
-		$this->entity_service        = $entity_service;
-		$this->configuration_service = $configuration_service;
-
-		self::$instance = $this;
-
-	}
-
-	/**
-	 * Get the {@link Wordlift_Content_Filter_Service} singleton instance.
-	 *
-	 * @since 3.14.2
-	 * @return \Wordlift_Content_Filter_Service The {@link Wordlift_Content_Filter_Service} singleton instance.
-	 */
-	public static function get_instance() {
-
-		return self::$instance;
-	}
-
-	/**
-	 * Mangle the content by adding links to the entity pages. This function is
-	 * hooked to the 'the_content' WP's filter.
-	 *
-	 * @since 3.8.0
-	 *
-	 * @param string $content The content being filtered.
-	 *
-	 * @return string The filtered content.
-	 */
-	public function the_content( $content ) {
-
-		// Preload the `link by default` setting.
-		$this->is_link_by_default = $this->configuration_service->is_link_by_default();
-
-		// Replace each match of the entity tag with the entity link. If an error
-		// occurs fail silently returning the original content.
-		return preg_replace_callback( self::PATTERN, array(
-			$this,
-			'link',
-		), $content ) ?: $content;
-	}
-
-	/**
-	 * Get the entity match and replace it with a page link.
-	 *
-	 * @since 3.8.0
-	 *
-	 * @param array $matches An array of matches.
-	 *
-	 * @return string The replaced text with the link to the entity page.
-	 */
-	private function link( $matches ) {
-
-		// Get the entity itemid URI and label.
-		$css_class = $matches[2];
-		$uri       = $matches[3];
-		$label     = $matches[4];
-
-		// Get the entity post by URI.
-		if ( null === ( $post = $this->entity_service->get_entity_post_by_uri( $uri ) ) ) {
-
-			// If the entity post is not found return the label w/o the markup
-			// around it.
-			//
-			// See https://github.com/insideout10/wordlift-plugin/issues/461
-			return $label;
-		}
-
-		$no_link = - 1 < strpos( $css_class, 'wl-no-link' );
-		$link    = - 1 < strpos( $css_class, 'wl-link' );
-
-		// Don't link if links are disabled and the entity is not link or the
-		// entity is do not link.
-		$dont_link = ( ! $this->is_link_by_default && ! $link ) || $no_link;
-
-		// Return the label if it's don't link.
-		if ( $dont_link ) {
-			return $label;
-		}
-
-		// Get the link.
-		$href = get_permalink( $post );
-
-		// Return the link.
-		return "<a class='wl-entity-page-link' href='$href'>$label</a>";
-	}
-
-	/**
-	 * Get the entity URIs (configured in the `itemid` attribute) contained in
-	 * the provided content.
-	 *
-	 * @since 3.14.2
-	 *
-	 * @param string $content The content.
-	 *
-	 * @return array An array of URIs.
-	 */
-	public function get_entity_uris( $content ) {
-
-		$matches = array();
-		preg_match_all( Wordlift_Content_Filter_Service::PATTERN, $content, $matches );
-
-		return array_unique( $matches[3] );
-	}
+    /**
+     * The pattern to find entities in text.
+     *
+     * @since 3.8.0
+     */
+    const PATTERN = '/<(\\w+)[^<]*class="([^"]*)"\\sitemid=\"([^"]+)\"[^>]*>([^<]*)<\\/\\1>/i';
+
+    /**
+     * A {@link Wordlift_Entity_Service} instance.
+     *
+     * @since  3.8.0
+     * @access private
+     * @var \Wordlift_Entity_Service $entity_service A {@link Wordlift_Entity_Service} instance.
+     */
+    private $entity_service;
+
+    /**
+     * The {@link Wordlift_Configuration_Service} instance.
+     *
+     * @since  3.13.0
+     * @access private
+     * @var \Wordlift_Configuration_Service $configuration_service The {@link Wordlift_Configuration_Service} instance.
+     */
+    private $configuration_service;
+
+    /**
+     * The `link by default` setting.
+     *
+     * @since  3.13.0
+     * @access private
+     * @var bool True if link by default is enabled otherwise false.
+     */
+    private $is_link_by_default;
+
+    /**
+     * The {@link Wordlift_Content_Filter_Service} singleton instance.
+     *
+     * @since  3.14.2
+     * @access private
+     * @var \Wordlift_Content_Filter_Service $instance The {@link Wordlift_Content_Filter_Service} singleton instance.
+     */
+    private static $instance;
+
+    /**
+     * Create a {@link Wordlift_Content_Filter_Service} instance.
+     *
+     * @since 3.8.0
+     *
+     * @param \Wordlift_Entity_Service        $entity_service        The {@link Wordlift_Entity_Service} instance.
+     * @param \Wordlift_Configuration_Service $configuration_service The {@link Wordlift_Configuration_Service} instance.
+     */
+    public function __construct( $entity_service, $configuration_service ) {
+
+        $this->entity_service        = $entity_service;
+        $this->configuration_service = $configuration_service;
+
+        self::$instance = $this;
+
+    }
+
+    /**
+     * Get the {@link Wordlift_Content_Filter_Service} singleton instance.
+     *
+     * @since 3.14.2
+     * @return \Wordlift_Content_Filter_Service The {@link Wordlift_Content_Filter_Service} singleton instance.
+     */
+    public static function get_instance() {
+
+        return self::$instance;
+    }
+
+    /**
+     * Mangle the content by adding links to the entity pages. This function is
+     * hooked to the 'the_content' WP's filter.
+     *
+     * @since 3.8.0
+     *
+     * @param string $content The content being filtered.
+     *
+     * @return string The filtered content.
+     */
+    public function the_content( $content ) {
+
+        // Preload the `link by default` setting.
+        $this->is_link_by_default = $this->configuration_service->is_link_by_default();
+
+        // Replace each match of the entity tag with the entity link. If an error
+        // occurs fail silently returning the original content.
+        return preg_replace_callback( self::PATTERN, array(
+            $this,
+            'link',
+        ), $content ) ?: $content;
+    }
+
+    /**
+     * Get the entity match and replace it with a page link.
+     *
+     * @since 3.8.0
+     *
+     * @param array $matches An array of matches.
+     *
+     * @return string The replaced text with the link to the entity page.
+     */
+    private function link( $matches ) {
+
+        // Get the entity itemid URI and label.
+        $css_class = $matches[2];
+        $uri       = $matches[3];
+        $label     = $matches[4];
+
+        // Get the entity post by URI.
+        if ( null === ( $post = $this->entity_service->get_entity_post_by_uri( $uri ) ) ) {
+
+            // If the entity post is not found return the label w/o the markup
+            // around it.
+            //
+            // See https://github.com/insideout10/wordlift-plugin/issues/461
+            return $label;
+        }
+
+        $no_link = - 1 < strpos( $css_class, 'wl-no-link' );
+        $link    = - 1 < strpos( $css_class, 'wl-link' );
+
+        // Don't link if links are disabled and the entity is not link or the
+        // entity is do not link.
+        $dont_link = ( ! $this->is_link_by_default && ! $link ) || $no_link;
+
+        // Return the label if it's don't link.
+        if ( $dont_link ) {
+            return $label;
+        }
+
+        // Get the link.
+        $href = get_permalink( $post );
+
+        // Return the link.
+        return "<a class='wl-entity-page-link' href='$href'>$label</a>";
+    }
+
+    /**
+     * Get the entity URIs (configured in the `itemid` attribute) contained in
+     * the provided content.
+     *
+     * @since 3.14.2
+     *
+     * @param string $content The content.
+     *
+     * @return array An array of URIs.
+     */
+    public function get_entity_uris( $content ) {
+
+        $matches = array();
+        preg_match_all( Wordlift_Content_Filter_Service::PATTERN, $content, $matches );
+
+        return array_unique( $matches[3] );
+    }
 
 }

Spacing +14 added lines, -14 removed lines

@@ -69,7 +69,7 @@ 
 	 * @param \Wordlift_Entity_Service        $entity_service        The {@link Wordlift_Entity_Service} instance.
 	 * @param \Wordlift_Configuration_Service $configuration_service The {@link Wordlift_Configuration_Service} instance.
 	 */
-	public function __construct( $entity_service, $configuration_service ) {
+	public function __construct($entity_service, $configuration_service) {
 
 		$this->entity_service        = $entity_service;
 		$this->configuration_service = $configuration_service;
@@ -99,17 +99,17 @@ 
 	 *
 	 * @return string The filtered content.
 	 */
-	public function the_content( $content ) {
+	public function the_content($content) {
 
 		// Preload the `link by default` setting.
 		$this->is_link_by_default = $this->configuration_service->is_link_by_default();
 
 		// Replace each match of the entity tag with the entity link. If an error
 		// occurs fail silently returning the original content.
-		return preg_replace_callback( self::PATTERN, array(
+		return preg_replace_callback(self::PATTERN, array(
 			$this,
 			'link',
-		), $content ) ?: $content;
+		), $content) ?: $content;
 	}
 
 	/**
@@ -121,7 +121,7 @@ 
 	 *
 	 * @return string The replaced text with the link to the entity page.
 	 */
-	private function link( $matches ) {
+	private function link($matches) {
 
 		// Get the entity itemid URI and label.
 		$css_class = $matches[2];
@@ -129,7 +129,7 @@ 
 		$label     = $matches[4];
 
 		// Get the entity post by URI.
-		if ( null === ( $post = $this->entity_service->get_entity_post_by_uri( $uri ) ) ) {
+		if (null === ($post = $this->entity_service->get_entity_post_by_uri($uri))) {
 
 			// If the entity post is not found return the label w/o the markup
 			// around it.
@@ -138,20 +138,20 @@ 
 			return $label;
 		}
 
-		$no_link = - 1 < strpos( $css_class, 'wl-no-link' );
-		$link    = - 1 < strpos( $css_class, 'wl-link' );
+		$no_link = - 1 < strpos($css_class, 'wl-no-link');
+		$link    = - 1 < strpos($css_class, 'wl-link');
 
 		// Don't link if links are disabled and the entity is not link or the
 		// entity is do not link.
-		$dont_link = ( ! $this->is_link_by_default && ! $link ) || $no_link;
+		$dont_link = ( ! $this->is_link_by_default && ! $link) || $no_link;
 
 		// Return the label if it's don't link.
-		if ( $dont_link ) {
+		if ($dont_link) {
 			return $label;
 		}
 
 		// Get the link.
-		$href = get_permalink( $post );
+		$href = get_permalink($post);
 
 		// Return the link.
 		return "<a class='wl-entity-page-link' href='$href'>$label</a>";
@@ -167,12 +167,12 @@ 
 	 *
 	 * @return array An array of URIs.
 	 */
-	public function get_entity_uris( $content ) {
+	public function get_entity_uris($content) {
 
 		$matches = array();
-		preg_match_all( Wordlift_Content_Filter_Service::PATTERN, $content, $matches );
+		preg_match_all(Wordlift_Content_Filter_Service::PATTERN, $content, $matches);
 
-		return array_unique( $matches[3] );
+		return array_unique($matches[3]);
 	}
 
 }

src/includes/class-wordlift-batch-analysis-adapter.php

Indentation +54 added lines, -54 removed lines

@@ -16,87 +16,87 @@
  */
 class Wordlift_Batch_Analysis_Adapter {
 
-	/**
-	 * @var Wordlift_Batch_Analysis_Service
-	 */
-	private $batch_analysis_service;
+    /**
+     * @var Wordlift_Batch_Analysis_Service
+     */
+    private $batch_analysis_service;
 
 
-	/**
-	 * Wordlift_Batch_Analysis_Adapter constructor.
-	 *
-	 * @since 3.14.2
-	 *
-	 * @param \Wordlift_Batch_Analysis_Service $batch_analysis_service
-	 */
-	public function __construct( $batch_analysis_service ) {
+    /**
+     * Wordlift_Batch_Analysis_Adapter constructor.
+     *
+     * @since 3.14.2
+     *
+     * @param \Wordlift_Batch_Analysis_Service $batch_analysis_service
+     */
+    public function __construct( $batch_analysis_service ) {
 
-		$this->batch_analysis_service = $batch_analysis_service;
+        $this->batch_analysis_service = $batch_analysis_service;
 
-	}
+    }
 
-	public function submit_auto_selected_posts() {
+    public function submit_auto_selected_posts() {
 
-		if ( ! isset( $_REQUEST['link'] ) ) {
-			wp_die( 'The `link` parameter is required.' );
-		}
+        if ( ! isset( $_REQUEST['link'] ) ) {
+            wp_die( 'The `link` parameter is required.' );
+        }
 
-		$count = $this->batch_analysis_service->submit_auto_selected_posts( $_REQUEST['link'] );
+        $count = $this->batch_analysis_service->submit_auto_selected_posts( $_REQUEST['link'] );
 
-		// Clear any buffer.
-		ob_clean();
+        // Clear any buffer.
+        ob_clean();
 
-		// Send the response.
-		wp_send_json_success( array( 'count' => $count ) );
+        // Send the response.
+        wp_send_json_success( array( 'count' => $count ) );
 
-	}
+    }
 
-	public function submit() {
+    public function submit() {
 
-		if ( ! isset( $_REQUEST['link'] ) || ! isset( $_REQUEST['post'] ) ) {
-			wp_die( 'The `link` and `post` parameters are required.' );
-		}
+        if ( ! isset( $_REQUEST['link'] ) || ! isset( $_REQUEST['post'] ) ) {
+            wp_die( 'The `link` and `post` parameters are required.' );
+        }
 
-		$count = $this->batch_analysis_service->submit( (array) $_REQUEST['post'], $_REQUEST['link'] );
+        $count = $this->batch_analysis_service->submit( (array) $_REQUEST['post'], $_REQUEST['link'] );
 
-		// Clear any buffer.
-		ob_clean();
+        // Clear any buffer.
+        ob_clean();
 
-		// Send the response.
-		wp_send_json_success( array( 'count' => $count ) );
+        // Send the response.
+        wp_send_json_success( array( 'count' => $count ) );
 
-	}
+    }
 
-	public function cancel() {
+    public function cancel() {
 
-		if ( ! isset( $_REQUEST['post'] ) ) {
-			wp_die( 'The `post` parameter is required.' );
-		}
+        if ( ! isset( $_REQUEST['post'] ) ) {
+            wp_die( 'The `post` parameter is required.' );
+        }
 
-		$count = $this->batch_analysis_service->cancel( (array) $_REQUEST['post'] );
+        $count = $this->batch_analysis_service->cancel( (array) $_REQUEST['post'] );
 
-		// Clear any buffer.
-		ob_clean();
+        // Clear any buffer.
+        ob_clean();
 
-		// Send the response.
-		wp_send_json_success( array( 'count' => $count ) );
+        // Send the response.
+        wp_send_json_success( array( 'count' => $count ) );
 
-	}
+    }
 
-	public function clear_warning() {
+    public function clear_warning() {
 
-		if ( ! isset( $_REQUEST['post'] ) ) {
-			wp_die( 'The `post` parameter is required.' );
-		}
+        if ( ! isset( $_REQUEST['post'] ) ) {
+            wp_die( 'The `post` parameter is required.' );
+        }
 
-		$this->batch_analysis_service->clear_warning( (array) $_REQUEST['post'] );
+        $this->batch_analysis_service->clear_warning( (array) $_REQUEST['post'] );
 
-		// Clear any buffer.
-		ob_clean();
+        // Clear any buffer.
+        ob_clean();
 
-		// Send the response.
-		wp_send_json_success();
+        // Send the response.
+        wp_send_json_success();
 
-	}
+    }
 
 }

Spacing +16 added lines, -16 removed lines

@@ -29,7 +29,7 @@ 
 	 *
 	 * @param \Wordlift_Batch_Analysis_Service $batch_analysis_service
 	 */
-	public function __construct( $batch_analysis_service ) {
+	public function __construct($batch_analysis_service) {
 
 		$this->batch_analysis_service = $batch_analysis_service;
 
@@ -37,59 +37,59 @@ 
 
 	public function submit_auto_selected_posts() {
 
-		if ( ! isset( $_REQUEST['link'] ) ) {
-			wp_die( 'The `link` parameter is required.' );
+		if ( ! isset($_REQUEST['link'])) {
+			wp_die('The `link` parameter is required.');
 		}
 
-		$count = $this->batch_analysis_service->submit_auto_selected_posts( $_REQUEST['link'] );
+		$count = $this->batch_analysis_service->submit_auto_selected_posts($_REQUEST['link']);
 
 		// Clear any buffer.
 		ob_clean();
 
 		// Send the response.
-		wp_send_json_success( array( 'count' => $count ) );
+		wp_send_json_success(array('count' => $count));
 
 	}
 
 	public function submit() {
 
-		if ( ! isset( $_REQUEST['link'] ) || ! isset( $_REQUEST['post'] ) ) {
-			wp_die( 'The `link` and `post` parameters are required.' );
+		if ( ! isset($_REQUEST['link']) || ! isset($_REQUEST['post'])) {
+			wp_die('The `link` and `post` parameters are required.');
 		}
 
-		$count = $this->batch_analysis_service->submit( (array) $_REQUEST['post'], $_REQUEST['link'] );
+		$count = $this->batch_analysis_service->submit((array) $_REQUEST['post'], $_REQUEST['link']);
 
 		// Clear any buffer.
 		ob_clean();
 
 		// Send the response.
-		wp_send_json_success( array( 'count' => $count ) );
+		wp_send_json_success(array('count' => $count));
 
 	}
 
 	public function cancel() {
 
-		if ( ! isset( $_REQUEST['post'] ) ) {
-			wp_die( 'The `post` parameter is required.' );
+		if ( ! isset($_REQUEST['post'])) {
+			wp_die('The `post` parameter is required.');
 		}
 
-		$count = $this->batch_analysis_service->cancel( (array) $_REQUEST['post'] );
+		$count = $this->batch_analysis_service->cancel((array) $_REQUEST['post']);
 
 		// Clear any buffer.
 		ob_clean();
 
 		// Send the response.
-		wp_send_json_success( array( 'count' => $count ) );
+		wp_send_json_success(array('count' => $count));
 
 	}
 
 	public function clear_warning() {
 
-		if ( ! isset( $_REQUEST['post'] ) ) {
-			wp_die( 'The `post` parameter is required.' );
+		if ( ! isset($_REQUEST['post'])) {
+			wp_die('The `post` parameter is required.');
 		}
 
-		$this->batch_analysis_service->clear_warning( (array) $_REQUEST['post'] );
+		$this->batch_analysis_service->clear_warning((array) $_REQUEST['post']);
 
 		// Clear any buffer.
 		ob_clean();

src/includes/class-wordlift-batch-analysis-service.php

Indentation +603 added lines, -603 removed lines

@@ -51,158 +51,158 @@ 
  */
 class Wordlift_Batch_Analysis_Service {
 
-	/**
-	 * The list of states for the Batch Analysis:
-	 *  - STATE_META_KEY: the batch analysis state meta key,
-	 *  - STATE_SUBMIT: a post/page has been submitted for analysis,
-	 *  - STATE_REQUEST: the plugin requested an analysis for the submitted
-	 *      post/page,
-	 *  - STATE_SUCCESS: the analysis has completed successfully,
-	 *  - STATE_ERROR: the analysis returned an error.
-	 *
-	 * @since 3.14.2
-	 */
-	const STATE_META_KEY = '_wl_batch_analysis_state';
-	const STATE_SUBMIT = 0;
-	const STATE_REQUEST = 1;
-	//### COMPLETE states.
-	const STATE_SUCCESS = 2;
-	const STATE_ERROR = 2;
-
-	/**
-	 * The submit timestamp meta key. A post may have more than one timestamp.
-	 *
-	 * @since 3.14.2
-	 */
-	const SUBMIT_TIMESTAMP_META_KEY = '_wl_batch_analysis_submit_timestamp';
-
-	/**
-	 * The request timestamp meta key. A post may have more than one timestamp.
-	 *
-	 * @since 3.14.2
-	 */
-	const REQUEST_TIMESTAMP_META_KEY = '_wl_batch_analysis_request_timestamp';
-
-	/**
-	 * The complete (success or error) timestamp meta key. A post may have more
-	 * than one timestamp.
-	 *
-	 * @since 3.14.2
-	 */
-	const COMPLETE_TIMESTAMP_META_KEY = '_wl_batch_analysis_complete_timestamp';
-
-	/**
-	 * The link setting meta key. A post may have more than one setting.
-	 *
-	 * @since 3.14.2
-	 */
-	const LINK_META_KEY = '_wl_batch_analysis_link';
-
-	/**
-	 * The warning timestamp meta key. A post has only zero/one value.
-	 *
-	 * @since 3.14.2
-	 */
-	const WARNING_META_KEY = '_wl_batch_analysis_warning';
-
-	/**
-	 * Option name.
-	 *
-	 * @since  3.14.0
-	 */
-	const OPTION_NAME = 'wl_analyze_batch';
-
-	/**
-	 * Name of waiting to be processed queue array inside the option.
-	 *
-	 * @since  3.14.0
-	 */
-	const ANALYZE_QUEUE = 'queue';
-
-	/**
-	 * Name of waiting for response queue array inside the option.
-	 *
-	 * @since  3.14.0
-	 */
-	const RESPONSE_QUEUE = 'processing';
-
-	/**
-	 * The {@link Wordlift} plugin instance.
-	 *
-	 * @since  3.14.0
-	 * @access private
-	 * @var \Wordlift $plugin The {@link Wordlift} plugin instance.
-	 */
-	private $plugin;
-
-	/**
-	 * The {@link Wordlift_Configuration_Service} instance.
-	 *
-	 * @since  3.14.0
-	 * @access private
-	 * @var \Wordlift_Configuration_Service $configuration_service The {@link Wordlift_Configuration_Service} instance.
-	 */
-	private $configuration_service;
-
-	/**
-	 * A {@link Wordlift_Log_Service} instance.
-	 *
-	 * @since  3.14.2
-	 * @access private
-	 * @var \Wordlift_Log_Service $log A {@link Wordlift_Log_Service} instance.
-	 */
-	private $log;
-
-	/**
-	 * The {@link Class_Wordlift_Batch_Analys_Service} instance.
-	 *
-	 * @since 3.14.0
-	 *
-	 * @param \Wordlift                       $plugin                The {@link Wordlift} plugin instance.
-	 * @param \Wordlift_Configuration_Service $configuration_service The {@link Wordlift_Configuration_Service} instance.
-	 */
-	public function __construct( $plugin, $configuration_service ) {
-
-		$this->plugin                = $plugin;
-		$this->configuration_service = $configuration_service;
-		$this->log                   = Wordlift_Log_Service::get_logger( 'Wordlift_Batch_Analysis_Service' );
-
-		add_action( 'wp_async_wl_batch_analysis_request', array(
-			$this,
-			'request',
-		) );
-		add_action( 'wp_async_wl_batch_analysis_complete', array(
-			$this,
-			'complete',
-		) );
-
-	}
-
-	/**
-	 * Get the base SQL statement to submit a post for Batch Analysis.
-	 *
-	 * Functions may use this base SQL and add their own filters.
-	 *
-	 * @since 3.14.2
-	 *
-	 * @param string $link The link setting ('yes'/'no').
-	 *
-	 * @return string The base SQL.
-	 */
-	private function get_sql( $link ) {
-		global $wpdb;
-
-		// Prepare the statement:
-		//  1. Insert into `postmeta` the meta keys and values:
-		//    a) state meta, with value of SUBMIT (0),
-		//    b) submit timestamp, with value of UTC timestamp,
-		//    c) link meta, with the provided value.
-		//  2. Join the current state value, can be used for filters by other
-		//     functions.
-		//  3. Filter by `post`/`page` types.
-		//  4. Filter by `publish` status.
-		return $wpdb->prepare(
-			"
+    /**
+     * The list of states for the Batch Analysis:
+     *  - STATE_META_KEY: the batch analysis state meta key,
+     *  - STATE_SUBMIT: a post/page has been submitted for analysis,
+     *  - STATE_REQUEST: the plugin requested an analysis for the submitted
+     *      post/page,
+     *  - STATE_SUCCESS: the analysis has completed successfully,
+     *  - STATE_ERROR: the analysis returned an error.
+     *
+     * @since 3.14.2
+     */
+    const STATE_META_KEY = '_wl_batch_analysis_state';
+    const STATE_SUBMIT = 0;
+    const STATE_REQUEST = 1;
+    //### COMPLETE states.
+    const STATE_SUCCESS = 2;
+    const STATE_ERROR = 2;
+
+    /**
+     * The submit timestamp meta key. A post may have more than one timestamp.
+     *
+     * @since 3.14.2
+     */
+    const SUBMIT_TIMESTAMP_META_KEY = '_wl_batch_analysis_submit_timestamp';
+
+    /**
+     * The request timestamp meta key. A post may have more than one timestamp.
+     *
+     * @since 3.14.2
+     */
+    const REQUEST_TIMESTAMP_META_KEY = '_wl_batch_analysis_request_timestamp';
+
+    /**
+     * The complete (success or error) timestamp meta key. A post may have more
+     * than one timestamp.
+     *
+     * @since 3.14.2
+     */
+    const COMPLETE_TIMESTAMP_META_KEY = '_wl_batch_analysis_complete_timestamp';
+
+    /**
+     * The link setting meta key. A post may have more than one setting.
+     *
+     * @since 3.14.2
+     */
+    const LINK_META_KEY = '_wl_batch_analysis_link';
+
+    /**
+     * The warning timestamp meta key. A post has only zero/one value.
+     *
+     * @since 3.14.2
+     */
+    const WARNING_META_KEY = '_wl_batch_analysis_warning';
+
+    /**
+     * Option name.
+     *
+     * @since  3.14.0
+     */
+    const OPTION_NAME = 'wl_analyze_batch';
+
+    /**
+     * Name of waiting to be processed queue array inside the option.
+     *
+     * @since  3.14.0
+     */
+    const ANALYZE_QUEUE = 'queue';
+
+    /**
+     * Name of waiting for response queue array inside the option.
+     *
+     * @since  3.14.0
+     */
+    const RESPONSE_QUEUE = 'processing';
+
+    /**
+     * The {@link Wordlift} plugin instance.
+     *
+     * @since  3.14.0
+     * @access private
+     * @var \Wordlift $plugin The {@link Wordlift} plugin instance.
+     */
+    private $plugin;
+
+    /**
+     * The {@link Wordlift_Configuration_Service} instance.
+     *
+     * @since  3.14.0
+     * @access private
+     * @var \Wordlift_Configuration_Service $configuration_service The {@link Wordlift_Configuration_Service} instance.
+     */
+    private $configuration_service;
+
+    /**
+     * A {@link Wordlift_Log_Service} instance.
+     *
+     * @since  3.14.2
+     * @access private
+     * @var \Wordlift_Log_Service $log A {@link Wordlift_Log_Service} instance.
+     */
+    private $log;
+
+    /**
+     * The {@link Class_Wordlift_Batch_Analys_Service} instance.
+     *
+     * @since 3.14.0
+     *
+     * @param \Wordlift                       $plugin                The {@link Wordlift} plugin instance.
+     * @param \Wordlift_Configuration_Service $configuration_service The {@link Wordlift_Configuration_Service} instance.
+     */
+    public function __construct( $plugin, $configuration_service ) {
+
+        $this->plugin                = $plugin;
+        $this->configuration_service = $configuration_service;
+        $this->log                   = Wordlift_Log_Service::get_logger( 'Wordlift_Batch_Analysis_Service' );
+
+        add_action( 'wp_async_wl_batch_analysis_request', array(
+            $this,
+            'request',
+        ) );
+        add_action( 'wp_async_wl_batch_analysis_complete', array(
+            $this,
+            'complete',
+        ) );
+
+    }
+
+    /**
+     * Get the base SQL statement to submit a post for Batch Analysis.
+     *
+     * Functions may use this base SQL and add their own filters.
+     *
+     * @since 3.14.2
+     *
+     * @param string $link The link setting ('yes'/'no').
+     *
+     * @return string The base SQL.
+     */
+    private function get_sql( $link ) {
+        global $wpdb;
+
+        // Prepare the statement:
+        //  1. Insert into `postmeta` the meta keys and values:
+        //    a) state meta, with value of SUBMIT (0),
+        //    b) submit timestamp, with value of UTC timestamp,
+        //    c) link meta, with the provided value.
+        //  2. Join the current state value, can be used for filters by other
+        //     functions.
+        //  3. Filter by `post`/`page` types.
+        //  4. Filter by `publish` status.
+        return $wpdb->prepare(
+            "
 			INSERT INTO $wpdb->postmeta ( post_id, meta_key, meta_value )
 			SELECT p.ID, metas.*
 			FROM (
@@ -218,492 +218,492 @@ 
 			WHERE p.post_type IN ('post', 'page')
 				AND p.post_status = 'publish'
 			",
-			self::STATE_META_KEY,
-			self::SUBMIT_TIMESTAMP_META_KEY,
-			self::LINK_META_KEY,
-			$link,
-			self::STATE_META_KEY
-		);
-	}
-
-	/**
-	 * Submit for analysis all the posts/pages which do not have annotations
-	 * and haven't been analyzed before.
-	 *
-	 * @since 3.14.2
-	 *
-	 * @param string $link The link setting.
-	 *
-	 * @return false|int The number of submitted {@link WP_Post}s or false on
-	 *                   error.
-	 */
-	public function submit_auto_selected_posts( $link ) {
-		global $wpdb;
-
-		// Submit the posts/pages and return the number of affected results.
-		// We're using a SQL query here because we could have potentially
-		// thousands of rows.
-		$count = $wpdb->query( $wpdb->prepare(
-			$this->get_sql( $link ) .
-			"
+            self::STATE_META_KEY,
+            self::SUBMIT_TIMESTAMP_META_KEY,
+            self::LINK_META_KEY,
+            $link,
+            self::STATE_META_KEY
+        );
+    }
+
+    /**
+     * Submit for analysis all the posts/pages which do not have annotations
+     * and haven't been analyzed before.
+     *
+     * @since 3.14.2
+     *
+     * @param string $link The link setting.
+     *
+     * @return false|int The number of submitted {@link WP_Post}s or false on
+     *                   error.
+     */
+    public function submit_auto_selected_posts( $link ) {
+        global $wpdb;
+
+        // Submit the posts/pages and return the number of affected results.
+        // We're using a SQL query here because we could have potentially
+        // thousands of rows.
+        $count = $wpdb->query( $wpdb->prepare(
+            $this->get_sql( $link ) .
+            "
 				AND batch_analysis_state.meta_value IS NULL
 				AND p.post_content NOT REGEXP %s;
 			",
-			'<[a-z]+ id="urn:[^"]+" class="[^"]+" itemid="[^"]+">'
-		) );
-
-		// Request Batch Analysis (the operation is handled asynchronously).
-		do_action( 'wl_batch_analysis_request' );
-
-		// Divide the count by 3 to get the number of posts/pages queued.
-		return $count / 3;
-	}
-
-	/**
-	 * Submit the provided list of {@link WP_Post}s' ids for Batch Analysis.
-	 *
-	 * @since 3.14.2
-	 *
-	 * @param int|array $post_ids A single {@link WP_Post}'s id or an array of
-	 *                            {@link WP_Post}s' ids.
-	 * @param string    $link     The link setting.
-	 *
-	 * @return int The number of submitted {@link WP_Post}s or false on error.
-	 */
-	public function submit( $post_ids, $link ) {
-		global $wpdb;
-
-		// Submit the posts/pages and return the number of affected results.
-		// We're using a SQL query here because we could have potentially
-		// thousands of rows.
-		$count = $wpdb->query(
-			$this->get_sql( $link ) .
-			' AND p.ID IN ( ' . implode( ',', wp_parse_id_list( $post_ids ) ) . ' )'
-		);
-
-		// Request Batch Analysis (the operation is handled asynchronously).
-		do_action( 'wl_batch_analysis_request' );
-
-		// Divide the count by 3 to get the number of posts/pages queued.
-		return $count / 3;
-	}
-
-	/**
-	 * Cancel the Batch Analysis request for the specified {@link WP_Post}s.
-	 *
-	 * @since 3.14.2
-	 *
-	 * @param int|array $post_ids A single {@link WP_Post}'s id or an array of
-	 *                            {@link WP_Post}s' ids.
-	 *
-	 * @return false|int The number of cancelled {@link WP_Post}s or false on
-	 *                   error.
-	 */
-	public function cancel( $post_ids ) {
-		global $wpdb;
-
-		return $wpdb->query( $wpdb->prepare(
-			"
+            '<[a-z]+ id="urn:[^"]+" class="[^"]+" itemid="[^"]+">'
+        ) );
+
+        // Request Batch Analysis (the operation is handled asynchronously).
+        do_action( 'wl_batch_analysis_request' );
+
+        // Divide the count by 3 to get the number of posts/pages queued.
+        return $count / 3;
+    }
+
+    /**
+     * Submit the provided list of {@link WP_Post}s' ids for Batch Analysis.
+     *
+     * @since 3.14.2
+     *
+     * @param int|array $post_ids A single {@link WP_Post}'s id or an array of
+     *                            {@link WP_Post}s' ids.
+     * @param string    $link     The link setting.
+     *
+     * @return int The number of submitted {@link WP_Post}s or false on error.
+     */
+    public function submit( $post_ids, $link ) {
+        global $wpdb;
+
+        // Submit the posts/pages and return the number of affected results.
+        // We're using a SQL query here because we could have potentially
+        // thousands of rows.
+        $count = $wpdb->query(
+            $this->get_sql( $link ) .
+            ' AND p.ID IN ( ' . implode( ',', wp_parse_id_list( $post_ids ) ) . ' )'
+        );
+
+        // Request Batch Analysis (the operation is handled asynchronously).
+        do_action( 'wl_batch_analysis_request' );
+
+        // Divide the count by 3 to get the number of posts/pages queued.
+        return $count / 3;
+    }
+
+    /**
+     * Cancel the Batch Analysis request for the specified {@link WP_Post}s.
+     *
+     * @since 3.14.2
+     *
+     * @param int|array $post_ids A single {@link WP_Post}'s id or an array of
+     *                            {@link WP_Post}s' ids.
+     *
+     * @return false|int The number of cancelled {@link WP_Post}s or false on
+     *                   error.
+     */
+    public function cancel( $post_ids ) {
+        global $wpdb;
+
+        return $wpdb->query( $wpdb->prepare(
+            "
 			DELETE FROM $wpdb->postmeta
 			WHERE meta_key = %s
 				AND meta_value = %s
 				AND post_id IN ( " . implode( ',', wp_parse_id_list( $post_ids ) ) . " )
 			",
-			self::STATE_META_KEY,
-			self::STATE_REQUEST
-		) );
+            self::STATE_META_KEY,
+            self::STATE_REQUEST
+        ) );
 
-	}
+    }
 
-	/**
-	 * Request the batch analysis for submitted posts.
-	 *
-	 * @since 3.14.2
-	 */
-	public function request() {
+    /**
+     * Request the batch analysis for submitted posts.
+     *
+     * @since 3.14.2
+     */
+    public function request() {
 
-		$this->log->debug( "Requesting analysis..." );
+        $this->log->debug( "Requesting analysis..." );
 
-		// By default 5 posts of any post type are returned.
-		$posts = get_posts( array(
-			'fields'     => 'ids',
-			'meta_key'   => self::STATE_META_KEY,
-			'meta_value' => self::STATE_SUBMIT,
-			'orderby'    => 'ID',
-		) );
+        // By default 5 posts of any post type are returned.
+        $posts = get_posts( array(
+            'fields'     => 'ids',
+            'meta_key'   => self::STATE_META_KEY,
+            'meta_value' => self::STATE_SUBMIT,
+            'orderby'    => 'ID',
+        ) );
 
-		// Bail out if there are no submitted posts.
-		if ( empty( $posts ) ) {
-			$this->log->debug( 'No posts to submit found, checking for completed requests...' );
+        // Bail out if there are no submitted posts.
+        if ( empty( $posts ) ) {
+            $this->log->debug( 'No posts to submit found, checking for completed requests...' );
 
-			do_action( 'wl_batch_analysis_complete' );
+            do_action( 'wl_batch_analysis_complete' );
 
-			return;
-		}
+            return;
+        }
 
-		// Send a request for each post.
-		foreach ( $posts as $id ) {
-			$this->log->debug( "Requesting analysis for post $id..." );
+        // Send a request for each post.
+        foreach ( $posts as $id ) {
+            $this->log->debug( "Requesting analysis for post $id..." );
 
-			// Change the state to `REQUEST`.
-			$this->set_state( $id, self::STATE_REQUEST );
+            // Change the state to `REQUEST`.
+            $this->set_state( $id, self::STATE_REQUEST );
 
-			// Send the actual request to the remote service.
-			$result = $this->do_request( $id );
+            // Send the actual request to the remote service.
+            $result = $this->do_request( $id );
 
-			$this->log->debug( "Analysis requested for post $id." );
+            $this->log->debug( "Analysis requested for post $id." );
 
-			// Set an error if we received an error.
-			if ( is_wp_error( $result ) ) {
-				$this->log->error( "Analysis request for post $id returned {$result->get_error_message()}." );
+            // Set an error if we received an error.
+            if ( is_wp_error( $result ) ) {
+                $this->log->error( "Analysis request for post $id returned {$result->get_error_message()}." );
 
-				$this->set_state( $id, self::STATE_ERROR );
-			}
+                $this->set_state( $id, self::STATE_ERROR );
+            }
 
-		}
+        }
 
-		// Call the `wl_batch_analysis_request` action again. This is going
-		// to be handled by the async task.
-		do_action( 'wl_batch_analysis_request' );
+        // Call the `wl_batch_analysis_request` action again. This is going
+        // to be handled by the async task.
+        do_action( 'wl_batch_analysis_request' );
 
-	}
+    }
 
-	/**
-	 * Get the results for the Batch Analysis.
-	 *
-	 * @since 3.14.2
-	 */
-	public function complete() {
+    /**
+     * Get the results for the Batch Analysis.
+     *
+     * @since 3.14.2
+     */
+    public function complete() {
 
-		$this->log->debug( "Requesting results..." );
+        $this->log->debug( "Requesting results..." );
 
-		// By default 5 posts of any post type are returned.
-		$posts = get_posts( array(
-			'fields'     => 'ids',
-			'meta_key'   => self::STATE_META_KEY,
-			'meta_value' => self::STATE_REQUEST,
-			'orderby'    => 'ID',
-		) );
+        // By default 5 posts of any post type are returned.
+        $posts = get_posts( array(
+            'fields'     => 'ids',
+            'meta_key'   => self::STATE_META_KEY,
+            'meta_value' => self::STATE_REQUEST,
+            'orderby'    => 'ID',
+        ) );
 
-		// Bail out if there are no submitted posts.
-		if ( empty( $posts ) ) {
-			$this->log->debug( 'No posts in request state found.' );
+        // Bail out if there are no submitted posts.
+        if ( empty( $posts ) ) {
+            $this->log->debug( 'No posts in request state found.' );
 
-			return;
-		}
+            return;
+        }
 
-		// Send a request for each post.
-		foreach ( $posts as $id ) {
-			$this->log->debug( "Requesting results for post $id..." );
+        // Send a request for each post.
+        foreach ( $posts as $id ) {
+            $this->log->debug( "Requesting results for post $id..." );
 
-			// Send the actual request to the remote service.
-			$response = $this->do_complete( $id );
+            // Send the actual request to the remote service.
+            $response = $this->do_complete( $id );
 
-			$this->log->debug( "Results requested for post $id." );
+            $this->log->debug( "Results requested for post $id." );
 
-			// Set an error if we received an error.
-			if ( ! is_wp_error( $response ) && isset( $response['body'] ) ) {
+            // Set an error if we received an error.
+            if ( ! is_wp_error( $response ) && isset( $response['body'] ) ) {
 
-				$this->log->debug( "Results received for post $id." );
+                $this->log->debug( "Results received for post $id." );
 
-				// Save the returned content as new revision.
-				$json = json_decode( $response['body'] );
-
-				// Continue if the content isn't set.
-				if ( ! isset( $json->content ) || empty( $json->content ) ) {
-					continue;
-				}
+                // Save the returned content as new revision.
+                $json = json_decode( $response['body'] );
+
+                // Continue if the content isn't set.
+                if ( ! isset( $json->content ) || empty( $json->content ) ) {
+                    continue;
+                }
 
-				$this->set_warning_based_on_content( $id, $json->content );
-
-				$content = wp_slash( $json->content );
-
-				wp_update_post( array(
-					'ID'           => $id,
-					'post_content' => $content,
-				) );
-
-				// Update the status.
-				$this->set_state( $id, self::STATE_SUCCESS );
-
-				$this->log->debug( "Post $id updated with batch analysis results." );
-
-				continue;
-			}
-
-			// @todo: implement a kind of timeout that sets an error if the
-			// results haven't been received after a long time.
-
-		}
-
-		// Call the `wl_batch_analysis_request` action again. This is going
-		// to be handled by the async task.
-		do_action( 'wl_batch_analysis_complete' );
-
-	}
-
-	/**
-	 * Set a warning flag on the {@link WP_Post} if its content has suspicious
-	 * interpolations.
-	 *
-	 * @since 3.14.2
-	 *
-	 * @param int    $post_id The {@link WP_Post}'s id.
-	 * @param string $content The {@link WP_Post}'s content.
-	 *
-	 * @return string The content (for chaining operations).
-	 */
-	private function set_warning_based_on_content( $post_id, $content ) {
-
-		$matches = array();
-
-		// Check for suspicious interpolations.
-		$warning = 0 < preg_match_all( '/\w<[a-z]+ id="urn:enhancement-[^"]+" class="[^"]+" itemid="[^"]+">/', $content, $matches )
-		           || 0 < preg_match_all( '/<[a-z]+ id="urn:enhancement-[^"]+" class="[^"]+" itemid="[^"]+">\s/', $content, $matches );
-
-		// Set the warning flag accordingly.
-		$this->set_warning( $post_id, $warning );
-
-		return $content;
-	}
-
-	/**
-	 * Clear the warning flag for the specified {@link WP_Post}s.
-	 *
-	 * @since 3.14.2
-	 *
-	 * @param int|array $post_ids A single {@link WP_Post}'s id or an array of
-	 *                            {@link WP_Post}s' ids.
-	 */
-	public function clear_warning( $post_ids ) {
-
-		foreach ( (array) $post_ids as $post_id ) {
-			delete_post_meta( $post_id, self::WARNING_META_KEY );
-		}
-
-	}
-
-	/**
-	 * Set the warning flag for the specified {@link WP_Post}.
-	 *
-	 * @since 3.14.2
-	 *
-	 * @param int  $post_id The {@link WP_Post}'s id.
-	 * @param bool $value   The flag's value.
-	 *
-	 * @return int|bool Meta ID if the key didn't exist, true on successful update,
-	 *                  false on failure.
-	 */
-	private function set_warning( $post_id, $value ) {
-
-		return update_post_meta( $post_id, self::WARNING_META_KEY, ( true === $value ? 'yes' : 'no' ) );
-	}
-
-	/**
-	 * Get the post/page batch analysis state.
-	 *
-	 * @since 3.14.2
-	 *
-	 * @param int $post_id The {@link WP_Post}'s id.
-	 *
-	 * @return int|string The post state or an empty string if not set.
-	 */
-	public function get_state( $post_id ) {
-
-		return get_post_meta( $post_id, self::STATE_META_KEY, true );
-	}
-
-	/**
-	 * Set the post/page batch analysis state.
-	 *
-	 * @since 3.14.2
-	 *
-	 * @param int $post_id The {@link WP_Post}'s id.
-	 * @param int $value   The new state.
-	 *
-	 * @return int|bool Meta ID if the key didn't exist, true on successful update,
-	 *                  false on failure.
-	 */
-	public function set_state( $post_id, $value ) {
-
-		// Update the state.
-		$result = update_post_meta( $post_id, self::STATE_META_KEY, $value );
-
-		// Update timestamps as required.
-		switch ( $value ) {
-
-			//### REQUEST state.
-			case self::STATE_REQUEST:
-				add_post_meta( $post_id, self::REQUEST_TIMESTAMP_META_KEY, current_time( 'mysql', true ) );
-				break;
-
-			//### SUCCESS/ERROR state.
-			case self::STATE_SUCCESS:
-			case self::STATE_ERROR:
-				add_post_meta( $post_id, self::COMPLETE_TIMESTAMP_META_KEY, current_time( 'mysql', true ) );
-				break;
-		}
-
-		// Finally return the result.
-		return $result;
-	}
-
-	/**
-	 * Get the link setting for a {@link WP_Post}.
-	 *
-	 * If there are multiple link settings, only the last one is returned.
-	 *
-	 * @since 3.14.2
-	 *
-	 * @param int $post_id The {@link WP_Post}'s id.
-	 *
-	 * @return string The link setting or `default` if not set.
-	 */
-	public function get_link( $post_id ) {
-
-		$values = get_post_meta( $post_id, self::LINK_META_KEY );
-
-		return end( $values ) ?: 'default';
-	}
-
-	/**
-	 * Get the array of post IDS waiting in the queue to start processing.
-	 *
-	 * @since 3.14.0
-	 *
-	 * @return array The waiting to be processed post ids queue.
-	 */
-	public function waiting_for_analysis() {
-
-		return get_posts( array(
-			'posts_per_page' => - 1,
-			'fields'         => 'ids',
-			'post_status'    => 'any',
-			'meta_key'       => self::STATE_META_KEY,
-			'meta_value'     => self::STATE_SUBMIT,
-			'orderby'        => 'ID',
-		) );
-	}
-
-	/**
-	 * Get the array of post IDS waiting for response.
-	 *
-	 * @deprecated
-	 * @since 3.14.0
-	 *
-	 * @return array The waiting for response post ids queue.
-	 */
-	public function waiting_for_response() {
-
-		return get_posts( array(
-			'posts_per_page' => - 1,
-			'fields'         => 'ids',
-			'post_status'    => 'any',
-			'meta_key'       => self::STATE_META_KEY,
-			'meta_value'     => self::STATE_REQUEST,
-			'orderby'        => 'ID',
-		) );
-	}
-
-	/**
-	 * Request the analysis for the specified {@link WP_Post}.
-	 *
-	 * @since 3.14.2
-	 *
-	 * @param int $post_id The {@link WP_Post}'s id.
-	 *
-	 * @return WP_Error|array The response or WP_Error on failure.
-	 */
-	private function do_request( $post_id ) {
-
-		// Get the post.
-		$post = get_post( $post_id );
-
-		// Bail out if the post isn't found.
-		if ( null === $post ) {
-			$this->log->warn( "Post $post_id not found." );
-
-			return new WP_Error( 0, "Cannot find post $post_id." );
-		}
-
-		// Get the link setting.
-		$link = $this->get_link( $post_id );
-
-		$this->log->debug( "Sending analysis request for post $post_id [ link :: $link ]..." );
-
-		// Get the batch analysis URL.
-		$url = $this->configuration_service->get_batch_analysis_url();
-
-		// Prepare the POST parameters.
-		$param = array(
-			'id'              => $post->ID,
-			'key'             => $this->configuration_service->get_key(),
-			'content'         => $post->post_content,
-			'contentLanguage' => $this->configuration_service->get_language_code(),
-			'version'         => $this->plugin->get_version(),
-			'links'           => $link,
-			'scope'           => 'local',
-		);
-
-		// Get the HTTP options.
-		$args = array_merge_recursive( unserialize( WL_REDLINK_API_HTTP_OPTIONS ), array(
-			'method'      => 'POST',
-			'headers'     => array(
-				'Accept'       => 'application/json',
-				'Content-type' => 'application/json; charset=UTF-8',
-			),
-			// we need to downgrade the HTTP version in this case since chunked encoding is dumping numbers in the response.
-			'httpversion' => '1.0',
-			'body'        => wp_json_encode( $param ),
-		) );
-
-		$this->log->debug( "Posting analysis request for post $post_id to $url..." );
-
-		// Post the parameter.
-		return wp_remote_post( $url, $args );
-	}
-
-	/**
-	 * Get the Batch Analysis results for the specified {@link WP_Post}.
-	 *
-	 * @since 3.14.2
-	 *
-	 * @param int $post_id The {@link WP_Post}'s id.
-	 *
-	 * @return WP_Error|array The response or WP_Error on failure.
-	 */
-	private function do_complete( $post_id ) {
-
-		$post = get_post( $post_id );
-
-		if ( null === $post ) {
-			// Post was possibly deleted, just bailout.
-			return new WP_Error( 0, "Post $post_id not found." );
-		}
-
-		$url = $this->configuration_service->get_batch_analysis_url();
-		$key = $this->configuration_service->get_key();
-		$url = $url . '/' . $post->ID . '?key=' . $key;
-
-		return wp_remote_get( $url, unserialize( WL_REDLINK_API_HTTP_OPTIONS ) );
-	}
-
-	/**
-	 * Get the {@link WP_Post}s' ids flagged with warnings.
-	 *
-	 * @since 3.14.2
-	 *
-	 * @return array An array of {@link WP_Post}s' ids.
-	 */
-	public function get_warnings() {
-
-		return get_posts( array(
-			'fields'      => 'ids',
-			'numberposts' => - 1,
-			'post_status' => 'any',
-			'meta_key'    => self::WARNING_META_KEY,
-			'meta_value'  => 'yes',
-		) );
-	}
+                $this->set_warning_based_on_content( $id, $json->content );
+
+                $content = wp_slash( $json->content );
+
+                wp_update_post( array(
+                    'ID'           => $id,
+                    'post_content' => $content,
+                ) );
+
+                // Update the status.
+                $this->set_state( $id, self::STATE_SUCCESS );
+
+                $this->log->debug( "Post $id updated with batch analysis results." );
+
+                continue;
+            }
+
+            // @todo: implement a kind of timeout that sets an error if the
+            // results haven't been received after a long time.
+
+        }
+
+        // Call the `wl_batch_analysis_request` action again. This is going
+        // to be handled by the async task.
+        do_action( 'wl_batch_analysis_complete' );
+
+    }
+
+    /**
+     * Set a warning flag on the {@link WP_Post} if its content has suspicious
+     * interpolations.
+     *
+     * @since 3.14.2
+     *
+     * @param int    $post_id The {@link WP_Post}'s id.
+     * @param string $content The {@link WP_Post}'s content.
+     *
+     * @return string The content (for chaining operations).
+     */
+    private function set_warning_based_on_content( $post_id, $content ) {
+
+        $matches = array();
+
+        // Check for suspicious interpolations.
+        $warning = 0 < preg_match_all( '/\w<[a-z]+ id="urn:enhancement-[^"]+" class="[^"]+" itemid="[^"]+">/', $content, $matches )
+                   || 0 < preg_match_all( '/<[a-z]+ id="urn:enhancement-[^"]+" class="[^"]+" itemid="[^"]+">\s/', $content, $matches );
+
+        // Set the warning flag accordingly.
+        $this->set_warning( $post_id, $warning );
+
+        return $content;
+    }
+
+    /**
+     * Clear the warning flag for the specified {@link WP_Post}s.
+     *
+     * @since 3.14.2
+     *
+     * @param int|array $post_ids A single {@link WP_Post}'s id or an array of
+     *                            {@link WP_Post}s' ids.
+     */
+    public function clear_warning( $post_ids ) {
+
+        foreach ( (array) $post_ids as $post_id ) {
+            delete_post_meta( $post_id, self::WARNING_META_KEY );
+        }
+
+    }
+
+    /**
+     * Set the warning flag for the specified {@link WP_Post}.
+     *
+     * @since 3.14.2
+     *
+     * @param int  $post_id The {@link WP_Post}'s id.
+     * @param bool $value   The flag's value.
+     *
+     * @return int|bool Meta ID if the key didn't exist, true on successful update,
+     *                  false on failure.
+     */
+    private function set_warning( $post_id, $value ) {
+
+        return update_post_meta( $post_id, self::WARNING_META_KEY, ( true === $value ? 'yes' : 'no' ) );
+    }
+
+    /**
+     * Get the post/page batch analysis state.
+     *
+     * @since 3.14.2
+     *
+     * @param int $post_id The {@link WP_Post}'s id.
+     *
+     * @return int|string The post state or an empty string if not set.
+     */
+    public function get_state( $post_id ) {
+
+        return get_post_meta( $post_id, self::STATE_META_KEY, true );
+    }
+
+    /**
+     * Set the post/page batch analysis state.
+     *
+     * @since 3.14.2
+     *
+     * @param int $post_id The {@link WP_Post}'s id.
+     * @param int $value   The new state.
+     *
+     * @return int|bool Meta ID if the key didn't exist, true on successful update,
+     *                  false on failure.
+     */
+    public function set_state( $post_id, $value ) {
+
+        // Update the state.
+        $result = update_post_meta( $post_id, self::STATE_META_KEY, $value );
+
+        // Update timestamps as required.
+        switch ( $value ) {
+
+            //### REQUEST state.
+            case self::STATE_REQUEST:
+                add_post_meta( $post_id, self::REQUEST_TIMESTAMP_META_KEY, current_time( 'mysql', true ) );
+                break;
+
+            //### SUCCESS/ERROR state.
+            case self::STATE_SUCCESS:
+            case self::STATE_ERROR:
+                add_post_meta( $post_id, self::COMPLETE_TIMESTAMP_META_KEY, current_time( 'mysql', true ) );
+                break;
+        }
+
+        // Finally return the result.
+        return $result;
+    }
+
+    /**
+     * Get the link setting for a {@link WP_Post}.
+     *
+     * If there are multiple link settings, only the last one is returned.
+     *
+     * @since 3.14.2
+     *
+     * @param int $post_id The {@link WP_Post}'s id.
+     *
+     * @return string The link setting or `default` if not set.
+     */
+    public function get_link( $post_id ) {
+
+        $values = get_post_meta( $post_id, self::LINK_META_KEY );
+
+        return end( $values ) ?: 'default';
+    }
+
+    /**
+     * Get the array of post IDS waiting in the queue to start processing.
+     *
+     * @since 3.14.0
+     *
+     * @return array The waiting to be processed post ids queue.
+     */
+    public function waiting_for_analysis() {
+
+        return get_posts( array(
+            'posts_per_page' => - 1,
+            'fields'         => 'ids',
+            'post_status'    => 'any',
+            'meta_key'       => self::STATE_META_KEY,
+            'meta_value'     => self::STATE_SUBMIT,
+            'orderby'        => 'ID',
+        ) );
+    }
+
+    /**
+     * Get the array of post IDS waiting for response.
+     *
+     * @deprecated
+     * @since 3.14.0
+     *
+     * @return array The waiting for response post ids queue.
+     */
+    public function waiting_for_response() {
+
+        return get_posts( array(
+            'posts_per_page' => - 1,
+            'fields'         => 'ids',
+            'post_status'    => 'any',
+            'meta_key'       => self::STATE_META_KEY,
+            'meta_value'     => self::STATE_REQUEST,
+            'orderby'        => 'ID',
+        ) );
+    }
+
+    /**
+     * Request the analysis for the specified {@link WP_Post}.
+     *
+     * @since 3.14.2
+     *
+     * @param int $post_id The {@link WP_Post}'s id.
+     *
+     * @return WP_Error|array The response or WP_Error on failure.
+     */
+    private function do_request( $post_id ) {
+
+        // Get the post.
+        $post = get_post( $post_id );
+
+        // Bail out if the post isn't found.
+        if ( null === $post ) {
+            $this->log->warn( "Post $post_id not found." );
+
+            return new WP_Error( 0, "Cannot find post $post_id." );
+        }
+
+        // Get the link setting.
+        $link = $this->get_link( $post_id );
+
+        $this->log->debug( "Sending analysis request for post $post_id [ link :: $link ]..." );
+
+        // Get the batch analysis URL.
+        $url = $this->configuration_service->get_batch_analysis_url();
+
+        // Prepare the POST parameters.
+        $param = array(
+            'id'              => $post->ID,
+            'key'             => $this->configuration_service->get_key(),
+            'content'         => $post->post_content,
+            'contentLanguage' => $this->configuration_service->get_language_code(),
+            'version'         => $this->plugin->get_version(),
+            'links'           => $link,
+            'scope'           => 'local',
+        );
+
+        // Get the HTTP options.
+        $args = array_merge_recursive( unserialize( WL_REDLINK_API_HTTP_OPTIONS ), array(
+            'method'      => 'POST',
+            'headers'     => array(
+                'Accept'       => 'application/json',
+                'Content-type' => 'application/json; charset=UTF-8',
+            ),
+            // we need to downgrade the HTTP version in this case since chunked encoding is dumping numbers in the response.
+            'httpversion' => '1.0',
+            'body'        => wp_json_encode( $param ),
+        ) );
+
+        $this->log->debug( "Posting analysis request for post $post_id to $url..." );
+
+        // Post the parameter.
+        return wp_remote_post( $url, $args );
+    }
+
+    /**
+     * Get the Batch Analysis results for the specified {@link WP_Post}.
+     *
+     * @since 3.14.2
+     *
+     * @param int $post_id The {@link WP_Post}'s id.
+     *
+     * @return WP_Error|array The response or WP_Error on failure.
+     */
+    private function do_complete( $post_id ) {
+
+        $post = get_post( $post_id );
+
+        if ( null === $post ) {
+            // Post was possibly deleted, just bailout.
+            return new WP_Error( 0, "Post $post_id not found." );
+        }
+
+        $url = $this->configuration_service->get_batch_analysis_url();
+        $key = $this->configuration_service->get_key();
+        $url = $url . '/' . $post->ID . '?key=' . $key;
+
+        return wp_remote_get( $url, unserialize( WL_REDLINK_API_HTTP_OPTIONS ) );
+    }
+
+    /**
+     * Get the {@link WP_Post}s' ids flagged with warnings.
+     *
+     * @since 3.14.2
+     *
+     * @return array An array of {@link WP_Post}s' ids.
+     */
+    public function get_warnings() {
+
+        return get_posts( array(
+            'fields'      => 'ids',
+            'numberposts' => - 1,
+            'post_status' => 'any',
+            'meta_key'    => self::WARNING_META_KEY,
+            'meta_value'  => 'yes',
+        ) );
+    }
 
 }

Spacing +101 added lines, -101 removed lines

@@ -161,20 +161,20 @@ 
 	 * @param \Wordlift                       $plugin                The {@link Wordlift} plugin instance.
 	 * @param \Wordlift_Configuration_Service $configuration_service The {@link Wordlift_Configuration_Service} instance.
 	 */
-	public function __construct( $plugin, $configuration_service ) {
+	public function __construct($plugin, $configuration_service) {
 
 		$this->plugin                = $plugin;
 		$this->configuration_service = $configuration_service;
-		$this->log                   = Wordlift_Log_Service::get_logger( 'Wordlift_Batch_Analysis_Service' );
+		$this->log                   = Wordlift_Log_Service::get_logger('Wordlift_Batch_Analysis_Service');
 
-		add_action( 'wp_async_wl_batch_analysis_request', array(
+		add_action('wp_async_wl_batch_analysis_request', array(
 			$this,
 			'request',
-		) );
-		add_action( 'wp_async_wl_batch_analysis_complete', array(
+		));
+		add_action('wp_async_wl_batch_analysis_complete', array(
 			$this,
 			'complete',
-		) );
+		));
 
 	}
 
@@ -189,7 +189,7 @@ 
 	 *
 	 * @return string The base SQL.
 	 */
-	private function get_sql( $link ) {
+	private function get_sql($link) {
 		global $wpdb;
 
 		// Prepare the statement:
@@ -237,23 +237,23 @@ 
 	 * @return false|int The number of submitted {@link WP_Post}s or false on
 	 *                   error.
 	 */
-	public function submit_auto_selected_posts( $link ) {
+	public function submit_auto_selected_posts($link) {
 		global $wpdb;
 
 		// Submit the posts/pages and return the number of affected results.
 		// We're using a SQL query here because we could have potentially
 		// thousands of rows.
-		$count = $wpdb->query( $wpdb->prepare(
-			$this->get_sql( $link ) .
+		$count = $wpdb->query($wpdb->prepare(
+			$this->get_sql($link).
 			"
 				AND batch_analysis_state.meta_value IS NULL
 				AND p.post_content NOT REGEXP %s;
 			",
 			'<[a-z]+ id="urn:[^"]+" class="[^"]+" itemid="[^"]+">'
-		) );
+		));
 
 		// Request Batch Analysis (the operation is handled asynchronously).
-		do_action( 'wl_batch_analysis_request' );
+		do_action('wl_batch_analysis_request');
 
 		// Divide the count by 3 to get the number of posts/pages queued.
 		return $count / 3;
@@ -270,19 +270,19 @@ 
 	 *
 	 * @return int The number of submitted {@link WP_Post}s or false on error.
 	 */
-	public function submit( $post_ids, $link ) {
+	public function submit($post_ids, $link) {
 		global $wpdb;
 
 		// Submit the posts/pages and return the number of affected results.
 		// We're using a SQL query here because we could have potentially
 		// thousands of rows.
 		$count = $wpdb->query(
-			$this->get_sql( $link ) .
-			' AND p.ID IN ( ' . implode( ',', wp_parse_id_list( $post_ids ) ) . ' )'
+			$this->get_sql($link).
+			' AND p.ID IN ( '.implode(',', wp_parse_id_list($post_ids)).' )'
 		);
 
 		// Request Batch Analysis (the operation is handled asynchronously).
-		do_action( 'wl_batch_analysis_request' );
+		do_action('wl_batch_analysis_request');
 
 		// Divide the count by 3 to get the number of posts/pages queued.
 		return $count / 3;
@@ -299,19 +299,19 @@ 
 	 * @return false|int The number of cancelled {@link WP_Post}s or false on
 	 *                   error.
 	 */
-	public function cancel( $post_ids ) {
+	public function cancel($post_ids) {
 		global $wpdb;
 
-		return $wpdb->query( $wpdb->prepare(
+		return $wpdb->query($wpdb->prepare(
 			"
 			DELETE FROM $wpdb->postmeta
 			WHERE meta_key = %s
 				AND meta_value = %s
-				AND post_id IN ( " . implode( ',', wp_parse_id_list( $post_ids ) ) . " )
+				AND post_id IN ( ".implode(',', wp_parse_id_list($post_ids))." )
 			",
 			self::STATE_META_KEY,
 			self::STATE_REQUEST
-		) );
+		));
 
 	}
 
@@ -322,49 +322,49 @@ 
 	 */
 	public function request() {
 
-		$this->log->debug( "Requesting analysis..." );
+		$this->log->debug("Requesting analysis...");
 
 		// By default 5 posts of any post type are returned.
-		$posts = get_posts( array(
+		$posts = get_posts(array(
 			'fields'     => 'ids',
 			'meta_key'   => self::STATE_META_KEY,
 			'meta_value' => self::STATE_SUBMIT,
 			'orderby'    => 'ID',
-		) );
+		));
 
 		// Bail out if there are no submitted posts.
-		if ( empty( $posts ) ) {
-			$this->log->debug( 'No posts to submit found, checking for completed requests...' );
+		if (empty($posts)) {
+			$this->log->debug('No posts to submit found, checking for completed requests...');
 
-			do_action( 'wl_batch_analysis_complete' );
+			do_action('wl_batch_analysis_complete');
 
 			return;
 		}
 
 		// Send a request for each post.
-		foreach ( $posts as $id ) {
-			$this->log->debug( "Requesting analysis for post $id..." );
+		foreach ($posts as $id) {
+			$this->log->debug("Requesting analysis for post $id...");
 
 			// Change the state to `REQUEST`.
-			$this->set_state( $id, self::STATE_REQUEST );
+			$this->set_state($id, self::STATE_REQUEST);
 
 			// Send the actual request to the remote service.
-			$result = $this->do_request( $id );
+			$result = $this->do_request($id);
 
-			$this->log->debug( "Analysis requested for post $id." );
+			$this->log->debug("Analysis requested for post $id.");
 
 			// Set an error if we received an error.
-			if ( is_wp_error( $result ) ) {
-				$this->log->error( "Analysis request for post $id returned {$result->get_error_message()}." );
+			if (is_wp_error($result)) {
+				$this->log->error("Analysis request for post $id returned {$result->get_error_message()}.");
 
-				$this->set_state( $id, self::STATE_ERROR );
+				$this->set_state($id, self::STATE_ERROR);
 			}
 
 		}
 
 		// Call the `wl_batch_analysis_request` action again. This is going
 		// to be handled by the async task.
-		do_action( 'wl_batch_analysis_request' );
+		do_action('wl_batch_analysis_request');
 
 	}
 
@@ -375,58 +375,58 @@ 
 	 */
 	public function complete() {
 
-		$this->log->debug( "Requesting results..." );
+		$this->log->debug("Requesting results...");
 
 		// By default 5 posts of any post type are returned.
-		$posts = get_posts( array(
+		$posts = get_posts(array(
 			'fields'     => 'ids',
 			'meta_key'   => self::STATE_META_KEY,
 			'meta_value' => self::STATE_REQUEST,
 			'orderby'    => 'ID',
-		) );
+		));
 
 		// Bail out if there are no submitted posts.
-		if ( empty( $posts ) ) {
-			$this->log->debug( 'No posts in request state found.' );
+		if (empty($posts)) {
+			$this->log->debug('No posts in request state found.');
 
 			return;
 		}
 
 		// Send a request for each post.
-		foreach ( $posts as $id ) {
-			$this->log->debug( "Requesting results for post $id..." );
+		foreach ($posts as $id) {
+			$this->log->debug("Requesting results for post $id...");
 
 			// Send the actual request to the remote service.
-			$response = $this->do_complete( $id );
+			$response = $this->do_complete($id);
 
-			$this->log->debug( "Results requested for post $id." );
+			$this->log->debug("Results requested for post $id.");
 
 			// Set an error if we received an error.
-			if ( ! is_wp_error( $response ) && isset( $response['body'] ) ) {
+			if ( ! is_wp_error($response) && isset($response['body'])) {
 
-				$this->log->debug( "Results received for post $id." );
+				$this->log->debug("Results received for post $id.");
 
 				// Save the returned content as new revision.
-				$json = json_decode( $response['body'] );
+				$json = json_decode($response['body']);
 
 				// Continue if the content isn't set.
-				if ( ! isset( $json->content ) || empty( $json->content ) ) {
+				if ( ! isset($json->content) || empty($json->content)) {
 					continue;
 				}
 
-				$this->set_warning_based_on_content( $id, $json->content );
+				$this->set_warning_based_on_content($id, $json->content);
 
-				$content = wp_slash( $json->content );
+				$content = wp_slash($json->content);
 
-				wp_update_post( array(
+				wp_update_post(array(
 					'ID'           => $id,
 					'post_content' => $content,
-				) );
+				));
 
 				// Update the status.
-				$this->set_state( $id, self::STATE_SUCCESS );
+				$this->set_state($id, self::STATE_SUCCESS);
 
-				$this->log->debug( "Post $id updated with batch analysis results." );
+				$this->log->debug("Post $id updated with batch analysis results.");
 
 				continue;
 			}
@@ -438,7 +438,7 @@ 
 
 		// Call the `wl_batch_analysis_request` action again. This is going
 		// to be handled by the async task.
-		do_action( 'wl_batch_analysis_complete' );
+		do_action('wl_batch_analysis_complete');
 
 	}
 
@@ -453,16 +453,16 @@ 
 	 *
 	 * @return string The content (for chaining operations).
 	 */
-	private function set_warning_based_on_content( $post_id, $content ) {
+	private function set_warning_based_on_content($post_id, $content) {
 
 		$matches = array();
 
 		// Check for suspicious interpolations.
-		$warning = 0 < preg_match_all( '/\w<[a-z]+ id="urn:enhancement-[^"]+" class="[^"]+" itemid="[^"]+">/', $content, $matches )
-		           || 0 < preg_match_all( '/<[a-z]+ id="urn:enhancement-[^"]+" class="[^"]+" itemid="[^"]+">\s/', $content, $matches );
+		$warning = 0 < preg_match_all('/\w<[a-z]+ id="urn:enhancement-[^"]+" class="[^"]+" itemid="[^"]+">/', $content, $matches)
+		           || 0 < preg_match_all('/<[a-z]+ id="urn:enhancement-[^"]+" class="[^"]+" itemid="[^"]+">\s/', $content, $matches);
 
 		// Set the warning flag accordingly.
-		$this->set_warning( $post_id, $warning );
+		$this->set_warning($post_id, $warning);
 
 		return $content;
 	}
@@ -475,10 +475,10 @@ 
 	 * @param int|array $post_ids A single {@link WP_Post}'s id or an array of
 	 *                            {@link WP_Post}s' ids.
 	 */
-	public function clear_warning( $post_ids ) {
+	public function clear_warning($post_ids) {
 
-		foreach ( (array) $post_ids as $post_id ) {
-			delete_post_meta( $post_id, self::WARNING_META_KEY );
+		foreach ((array) $post_ids as $post_id) {
+			delete_post_meta($post_id, self::WARNING_META_KEY);
 		}
 
 	}
@@ -494,9 +494,9 @@ 
 	 * @return int|bool Meta ID if the key didn't exist, true on successful update,
 	 *                  false on failure.
 	 */
-	private function set_warning( $post_id, $value ) {
+	private function set_warning($post_id, $value) {
 
-		return update_post_meta( $post_id, self::WARNING_META_KEY, ( true === $value ? 'yes' : 'no' ) );
+		return update_post_meta($post_id, self::WARNING_META_KEY, (true === $value ? 'yes' : 'no'));
 	}
 
 	/**
@@ -508,9 +508,9 @@ 
 	 *
 	 * @return int|string The post state or an empty string if not set.
 	 */
-	public function get_state( $post_id ) {
+	public function get_state($post_id) {
 
-		return get_post_meta( $post_id, self::STATE_META_KEY, true );
+		return get_post_meta($post_id, self::STATE_META_KEY, true);
 	}
 
 	/**
@@ -524,23 +524,23 @@ 
 	 * @return int|bool Meta ID if the key didn't exist, true on successful update,
 	 *                  false on failure.
 	 */
-	public function set_state( $post_id, $value ) {
+	public function set_state($post_id, $value) {
 
 		// Update the state.
-		$result = update_post_meta( $post_id, self::STATE_META_KEY, $value );
+		$result = update_post_meta($post_id, self::STATE_META_KEY, $value);
 
 		// Update timestamps as required.
-		switch ( $value ) {
+		switch ($value) {
 
 			//### REQUEST state.
 			case self::STATE_REQUEST:
-				add_post_meta( $post_id, self::REQUEST_TIMESTAMP_META_KEY, current_time( 'mysql', true ) );
+				add_post_meta($post_id, self::REQUEST_TIMESTAMP_META_KEY, current_time('mysql', true));
 				break;
 
 			//### SUCCESS/ERROR state.
 			case self::STATE_SUCCESS:
 			case self::STATE_ERROR:
-				add_post_meta( $post_id, self::COMPLETE_TIMESTAMP_META_KEY, current_time( 'mysql', true ) );
+				add_post_meta($post_id, self::COMPLETE_TIMESTAMP_META_KEY, current_time('mysql', true));
 				break;
 		}
 
@@ -559,11 +559,11 @@ 
 	 *
 	 * @return string The link setting or `default` if not set.
 	 */
-	public function get_link( $post_id ) {
+	public function get_link($post_id) {
 
-		$values = get_post_meta( $post_id, self::LINK_META_KEY );
+		$values = get_post_meta($post_id, self::LINK_META_KEY);
 
-		return end( $values ) ?: 'default';
+		return end($values) ?: 'default';
 	}
 
 	/**
@@ -575,14 +575,14 @@ 
 	 */
 	public function waiting_for_analysis() {
 
-		return get_posts( array(
-			'posts_per_page' => - 1,
+		return get_posts(array(
+			'posts_per_page' => -1,
 			'fields'         => 'ids',
 			'post_status'    => 'any',
 			'meta_key'       => self::STATE_META_KEY,
 			'meta_value'     => self::STATE_SUBMIT,
 			'orderby'        => 'ID',
-		) );
+		));
 	}
 
 	/**
@@ -595,14 +595,14 @@ 
 	 */
 	public function waiting_for_response() {
 
-		return get_posts( array(
-			'posts_per_page' => - 1,
+		return get_posts(array(
+			'posts_per_page' => -1,
 			'fields'         => 'ids',
 			'post_status'    => 'any',
 			'meta_key'       => self::STATE_META_KEY,
 			'meta_value'     => self::STATE_REQUEST,
 			'orderby'        => 'ID',
-		) );
+		));
 	}
 
 	/**
@@ -614,22 +614,22 @@ 
 	 *
 	 * @return WP_Error|array The response or WP_Error on failure.
 	 */
-	private function do_request( $post_id ) {
+	private function do_request($post_id) {
 
 		// Get the post.
-		$post = get_post( $post_id );
+		$post = get_post($post_id);
 
 		// Bail out if the post isn't found.
-		if ( null === $post ) {
-			$this->log->warn( "Post $post_id not found." );
+		if (null === $post) {
+			$this->log->warn("Post $post_id not found.");
 
-			return new WP_Error( 0, "Cannot find post $post_id." );
+			return new WP_Error(0, "Cannot find post $post_id.");
 		}
 
 		// Get the link setting.
-		$link = $this->get_link( $post_id );
+		$link = $this->get_link($post_id);
 
-		$this->log->debug( "Sending analysis request for post $post_id [ link :: $link ]..." );
+		$this->log->debug("Sending analysis request for post $post_id [ link :: $link ]...");
 
 		// Get the batch analysis URL.
 		$url = $this->configuration_service->get_batch_analysis_url();
@@ -646,7 +646,7 @@ 
 		);
 
 		// Get the HTTP options.
-		$args = array_merge_recursive( unserialize( WL_REDLINK_API_HTTP_OPTIONS ), array(
+		$args = array_merge_recursive(unserialize(WL_REDLINK_API_HTTP_OPTIONS), array(
 			'method'      => 'POST',
 			'headers'     => array(
 				'Accept'       => 'application/json',
@@ -654,13 +654,13 @@ 
 			),
 			// we need to downgrade the HTTP version in this case since chunked encoding is dumping numbers in the response.
 			'httpversion' => '1.0',
-			'body'        => wp_json_encode( $param ),
-		) );
+			'body'        => wp_json_encode($param),
+		));
 
-		$this->log->debug( "Posting analysis request for post $post_id to $url..." );
+		$this->log->debug("Posting analysis request for post $post_id to $url...");
 
 		// Post the parameter.
-		return wp_remote_post( $url, $args );
+		return wp_remote_post($url, $args);
 	}
 
 	/**
@@ -672,20 +672,20 @@ 
 	 *
 	 * @return WP_Error|array The response or WP_Error on failure.
 	 */
-	private function do_complete( $post_id ) {
+	private function do_complete($post_id) {
 
-		$post = get_post( $post_id );
+		$post = get_post($post_id);
 
-		if ( null === $post ) {
+		if (null === $post) {
 			// Post was possibly deleted, just bailout.
-			return new WP_Error( 0, "Post $post_id not found." );
+			return new WP_Error(0, "Post $post_id not found.");
 		}
 
 		$url = $this->configuration_service->get_batch_analysis_url();
 		$key = $this->configuration_service->get_key();
-		$url = $url . '/' . $post->ID . '?key=' . $key;
+		$url = $url.'/'.$post->ID.'?key='.$key;
 
-		return wp_remote_get( $url, unserialize( WL_REDLINK_API_HTTP_OPTIONS ) );
+		return wp_remote_get($url, unserialize(WL_REDLINK_API_HTTP_OPTIONS));
 	}
 
 	/**
@@ -697,13 +697,13 @@ 
 	 */
 	public function get_warnings() {
 
-		return get_posts( array(
+		return get_posts(array(
 			'fields'      => 'ids',
-			'numberposts' => - 1,
+			'numberposts' => -1,
 			'post_status' => 'any',
 			'meta_key'    => self::WARNING_META_KEY,
 			'meta_value'  => 'yes',
-		) );
+		));
 	}
 
 }

src/includes/class-wordlift-relation-rebuild-adapter.php

Indentation +19 added lines, -19 removed lines

@@ -1,31 +1,31 @@
 <?php
 
 class Wordlift_Relation_Rebuild_Adapter {
-	/**
-	 * @var Wordlift_Relation_Rebuild_Service
-	 */
-	private $relation_rebuild_service;
+    /**
+     * @var Wordlift_Relation_Rebuild_Service
+     */
+    private $relation_rebuild_service;
 
 
-	/**
-	 * Wordlift_Relation_Rebuild_Adapter constructor.
-	 *
-	 * @param \Wordlift_Relation_Rebuild_Service $relation_rebuild_service
-	 */
-	public function __construct( $relation_rebuild_service ) {
-		$this->relation_rebuild_service = $relation_rebuild_service;
-	}
+    /**
+     * Wordlift_Relation_Rebuild_Adapter constructor.
+     *
+     * @param \Wordlift_Relation_Rebuild_Service $relation_rebuild_service
+     */
+    public function __construct( $relation_rebuild_service ) {
+        $this->relation_rebuild_service = $relation_rebuild_service;
+    }
 
-	public function process_all() {
+    public function process_all() {
 
-		$this->relation_rebuild_service->process_all();
+        $this->relation_rebuild_service->process_all();
 
-		ob_clean();
+        ob_clean();
 
-		wp_send_json_success( array(
-			'count' => $this->relation_rebuild_service->get_count(),
-		) );
+        wp_send_json_success( array(
+            'count' => $this->relation_rebuild_service->get_count(),
+        ) );
 
-	}
+    }
 
 }

Spacing +3 added lines, -3 removed lines

@@ -12,7 +12,7 @@ 
 	 *
 	 * @param \Wordlift_Relation_Rebuild_Service $relation_rebuild_service
 	 */
-	public function __construct( $relation_rebuild_service ) {
+	public function __construct($relation_rebuild_service) {
 		$this->relation_rebuild_service = $relation_rebuild_service;
 	}
 
@@ -22,9 +22,9 @@ 
 
 		ob_clean();
 
-		wp_send_json_success( array(
+		wp_send_json_success(array(
 			'count' => $this->relation_rebuild_service->get_count(),
-		) );
+		));
 
 	}