humanmade / repeatable-posts

hm-post-repeat.php

Braces +5 added lines, -2 removed lines

@@ -82,9 +82,12 @@
 
 			<strong><?php printf( esc_html__( 'Repeat of %s', 'hm-post-repeat' ), '<a href="' . esc_url( get_edit_post_link( get_post()->post_parent ) ) . '">' . esc_html( get_the_title( get_post_field( 'post_parent', get_the_id() ) ) ) . '</a>' ); ?></strong>
 
-		<?php else : ?>
+		<?php else {
+	: ?>
 
-			<?php $repeating_schedule = get_repeating_schedule( get_the_id() ); ?>
+			<?php $repeating_schedule = get_repeating_schedule( get_the_id() );
+}
+?>
 			<?php $is_repeating_post = is_repeating_post( get_the_id() ) && isset( $repeating_schedule ); ?>
 
 			<strong><?php echo ! $is_repeating_post ? esc_html__( 'No', 'hm-post-repeat' ) : esc_html( $repeating_schedule['display'] ); ?></strong>

Spacing +95 added lines, -95 removed lines

@@ -35,27 +35,27 @@ 
 /**
  * Setup the actions and filters required by this class.
  */
-add_action( 'post_submitbox_misc_actions', __NAMESPACE__ . '\publish_box_ui' );
-add_action( 'save_post', __NAMESPACE__ . '\save_post_repeating_status', 10 );
-add_action( 'save_post', __NAMESPACE__ . '\create_next_repeat_post', 11 );
-add_action( 'admin_enqueue_scripts', __NAMESPACE__ . '\enqueue_scripts' );
-add_filter( 'display_post_states', __NAMESPACE__ . '\post_states', 10, 2 );
+add_action('post_submitbox_misc_actions', __NAMESPACE__.'\publish_box_ui');
+add_action('save_post', __NAMESPACE__.'\save_post_repeating_status', 10);
+add_action('save_post', __NAMESPACE__.'\create_next_repeat_post', 11);
+add_action('admin_enqueue_scripts', __NAMESPACE__.'\enqueue_scripts');
+add_filter('display_post_states', __NAMESPACE__.'\post_states', 10, 2);
 
 /**
  * Enqueue the scripts and styles that are needed by this plugin.
  */
-function enqueue_scripts( $hook ) {
+function enqueue_scripts($hook) {
 
 	// Ensure we only load them on the edit post and add new post admin screens
-	if ( ! in_array( $hook, array( 'post.php', 'post-new.php' ) ) ) {
+	if ( ! in_array($hook, array('post.php', 'post-new.php'))) {
 		return;
 	}
 
-	$plugin_data = get_plugin_data( __FILE__ );
-	$plugin_dir_url = plugin_dir_url( __FILE__ );
+	$plugin_data = get_plugin_data(__FILE__);
+	$plugin_dir_url = plugin_dir_url(__FILE__);
 
-	wp_enqueue_script( 'hm-post-repeat', $plugin_dir_url . 'hm-post-repeat.js', 'jquery', $plugin_data['Version'], true );
-	wp_enqueue_style( 'hm-post-repeat', $plugin_dir_url . 'hm-post-repeat.css', array(), $plugin_data['Version'] );
+	wp_enqueue_script('hm-post-repeat', $plugin_dir_url.'hm-post-repeat.js', 'jquery', $plugin_data['Version'], true);
+	wp_enqueue_style('hm-post-repeat', $plugin_dir_url.'hm-post-repeat.css', array(), $plugin_data['Version']);
 
 }
 
@@ -67,7 +67,7 @@ 
  */
 function publish_box_ui() {
 
-	if ( ! in_array( get_post_type(), repeating_post_types() ) ) {
+	if ( ! in_array(get_post_type(), repeating_post_types())) {
 		return;
 	} ?>
 
@@ -75,31 +75,31 @@ 
 
 		<span class="dashicons dashicons-controls-repeat"></span>
 
-		<?php esc_html_e( 'Repeat:', 'hm-post-repeat' ); ?>
+		<?php esc_html_e('Repeat:', 'hm-post-repeat'); ?>
 
-		<?php if ( is_repeat_post( get_the_id() ) ) : ?>
+		<?php if (is_repeat_post(get_the_id())) : ?>
 
-			<strong><?php printf( esc_html__( 'Repeat of %s', 'hm-post-repeat' ), '<a href="' . esc_url( get_edit_post_link( get_post()->post_parent ) ) . '">' . esc_html( get_the_title( get_post_field( 'post_parent', get_the_id() ) ) ) . '</a>' ); ?></strong>
+			<strong><?php printf(esc_html__('Repeat of %s', 'hm-post-repeat'), '<a href="'.esc_url(get_edit_post_link(get_post()->post_parent)).'">'.esc_html(get_the_title(get_post_field('post_parent', get_the_id()))).'</a>'); ?></strong>
 
 		<?php else : ?>
 
-			<?php $repeating_schedule = get_repeating_schedule( get_the_id() ); ?>
-			<?php $is_repeating_post = is_repeating_post( get_the_id() ) && isset( $repeating_schedule ); ?>
+			<?php $repeating_schedule = get_repeating_schedule(get_the_id()); ?>
+			<?php $is_repeating_post = is_repeating_post(get_the_id()) && isset($repeating_schedule); ?>
 
-			<strong><?php echo ! $is_repeating_post ? esc_html__( 'No', 'hm-post-repeat' ) : esc_html( $repeating_schedule['display'] ); ?></strong>
+			<strong><?php echo ! $is_repeating_post ? esc_html__('No', 'hm-post-repeat') : esc_html($repeating_schedule['display']); ?></strong>
 
-			<a href="#hm-post-repeat" class="edit-hm-post-repeat hide-if-no-js"><span aria-hidden="true"><?php esc_html_e( 'Edit', 'hm-post-repeat' ); ?></span> <span class="screen-reader-text"><?php esc_html_e( 'Edit Repeat Settings', 'hm-post-repeat' ); ?></span></a>
+			<a href="#hm-post-repeat" class="edit-hm-post-repeat hide-if-no-js"><span aria-hidden="true"><?php esc_html_e('Edit', 'hm-post-repeat'); ?></span> <span class="screen-reader-text"><?php esc_html_e('Edit Repeat Settings', 'hm-post-repeat'); ?></span></a>
 
 			<span class="hide-if-js" id="hm-post-repeat">
 
 				<select name="hm-post-repeat">
-					<option<?php selected( ! $is_repeating_post ); ?> value="no"><?php esc_html_e( 'No', 'hm-post-repeat' ); ?></option>
-					<?php foreach ( get_repeating_schedules() as $schedule_slug => $schedule ) : ?>
-						<option<?php selected( $is_repeating_post && $schedule_slug === $repeating_schedule['slug'] ); ?> value="<?php echo esc_attr( $schedule_slug ); ?>"><?php echo esc_html( $schedule['display'] ); ?></option>
+					<option<?php selected( ! $is_repeating_post); ?> value="no"><?php esc_html_e('No', 'hm-post-repeat'); ?></option>
+					<?php foreach (get_repeating_schedules() as $schedule_slug => $schedule) : ?>
+						<option<?php selected($is_repeating_post && $schedule_slug === $repeating_schedule['slug']); ?> value="<?php echo esc_attr($schedule_slug); ?>"><?php echo esc_html($schedule['display']); ?></option>
 					<?php endforeach; ?>
 				</select>
 
-				<a href="#hm-post-repeat" class="save-post-hm-post-repeat hide-if-no-js button"><?php esc_html_e( 'OK', 'hm-post-repeat' ); ?></a>
+				<a href="#hm-post-repeat" class="save-post-hm-post-repeat hide-if-no-js button"><?php esc_html_e('OK', 'hm-post-repeat'); ?></a>
 
 			</span>
 
@@ -118,21 +118,21 @@ 
  * @param WP_Post $post        The post object to get / return the states.
  * @return array The array of post states with ours added.
  */
-function post_states( $post_states, $post ) {
+function post_states($post_states, $post) {
 
-	if ( is_repeating_post( $post->ID ) ) {
+	if (is_repeating_post($post->ID)) {
 
 		// If the schedule has been removed since publishing, let the user know.
-		if ( get_repeating_schedule( $post->ID ) ) {
-			$post_states['hm-post-repeat'] = __( 'Repeating', 'hm-post-repeat' );
+		if (get_repeating_schedule($post->ID)) {
+			$post_states['hm-post-repeat'] = __('Repeating', 'hm-post-repeat');
 		} else {
-			$post_states['hm-post-repeat'] = __( 'Invalid Repeating Schedule', 'hm-post-repeat' );
+			$post_states['hm-post-repeat'] = __('Invalid Repeating Schedule', 'hm-post-repeat');
 		}
 
 	}
 
-	if ( is_repeat_post( $post->ID ) ) {
-		$post_states['hm-post-repeat'] = __( 'Repeat', 'hm-post-repeat' );
+	if (is_repeat_post($post->ID)) {
+		$post_states['hm-post-repeat'] = __('Repeat', 'hm-post-repeat');
 	}
 
 	return $post_states;
@@ -147,23 +147,23 @@ 
  * @param int    $post_id             The ID of the post.
  * @param string $post_repeat_setting Used to manually set the repeating schedule from tests.
  */
-function save_post_repeating_status( $post_id = null, $post_repeat_setting = null ) {
+function save_post_repeating_status($post_id = null, $post_repeat_setting = null) {
 
-	if ( is_null( $post_repeat_setting ) ) {
-		$post_repeat_setting = isset( $_POST['hm-post-repeat'] ) ? sanitize_text_field( $_POST['hm-post-repeat'] ) : '';
+	if (is_null($post_repeat_setting)) {
+		$post_repeat_setting = isset($_POST['hm-post-repeat']) ? sanitize_text_field($_POST['hm-post-repeat']) : '';
 	}
 
-	if ( ! in_array( get_post_type( $post_id ), repeating_post_types() ) || empty( $post_repeat_setting ) ) {
+	if ( ! in_array(get_post_type($post_id), repeating_post_types()) || empty($post_repeat_setting)) {
 		return;
 	}
 
-	if ( 'no' === $post_repeat_setting ) {
-		delete_post_meta( $post_id, 'hm-post-repeat' );
+	if ('no' === $post_repeat_setting) {
+		delete_post_meta($post_id, 'hm-post-repeat');
 	}
 
 	// Make sure we have a valid schedule.
-	elseif ( in_array( $post_repeat_setting, array_keys( get_repeating_schedules() ) ) ) {
-		update_post_meta( $post_id, 'hm-post-repeat', $post_repeat_setting );
+	elseif (in_array($post_repeat_setting, array_keys(get_repeating_schedules()))) {
+		update_post_meta($post_id, 'hm-post-repeat', $post_repeat_setting);
 	}
 
 }
@@ -179,50 +179,50 @@ 
  *
  * @param int $post_id The ID of the post.
  */
-function create_next_repeat_post( $post_id ) {
+function create_next_repeat_post($post_id) {
 
-	if ( ! in_array( get_post_type( $post_id ), repeating_post_types() ) ) {
+	if ( ! in_array(get_post_type($post_id), repeating_post_types())) {
 		return false;
 	}
 
-	if ( 'publish' !== get_post_status( $post_id ) ) {
+	if ('publish' !== get_post_status($post_id)) {
 		return false;
 	}
 
-	$original_post_id = get_repeating_post( $post_id );
+	$original_post_id = get_repeating_post($post_id);
 
 	// Bail if we're not publishing a repeat(ing) post
-	if ( ! $original_post_id ) {
+	if ( ! $original_post_id) {
 		return false;
 	}
 
-	$original_post = get_post( $original_post_id, ARRAY_A );
+	$original_post = get_post($original_post_id, ARRAY_A);
 
 	// If there is already a repeat post scheduled don't create another one
-	if ( get_next_scheduled_repeat_post( $original_post['ID'] ) ) {
+	if (get_next_scheduled_repeat_post($original_post['ID'])) {
 		return false;
 	}
 
 	// Bail if the saved schedule doesn't exist
-	$repeating_schedule = get_repeating_schedule( $original_post['ID'] );
+	$repeating_schedule = get_repeating_schedule($original_post['ID']);
 
-	if ( ! $repeating_schedule ) {
+	if ( ! $repeating_schedule) {
 		return false;
 	}
 
 	// Bail if the original post isn't already published
-	if ( 'publish' !== $original_post['post_status'] ) {
+	if ('publish' !== $original_post['post_status']) {
 		return false;
 	}
 
 	$next_post = $original_post;
 
 	// Create the repeat post as a copy of the original, but ignore some fields
-	unset( $next_post['ID'] );
-	unset( $next_post['guid'] );
-	unset( $next_post['post_date_gmt'] );
-	unset( $next_post['post_modified'] );
-	unset( $next_post['post_modified_gmt'] );
+	unset($next_post['ID']);
+	unset($next_post['guid']);
+	unset($next_post['post_date_gmt']);
+	unset($next_post['post_modified']);
+	unset($next_post['post_modified_gmt']);
 
 	// We set the post_parent to the original post_id, so they're related
 	$next_post['post_parent'] = $original_post['ID'];
@@ -231,44 +231,44 @@ 
 	$next_post['post_status'] = 'future';
 
 	// Use the date of the current post being saved as the base
-	$next_post['post_date'] = date( 'Y-m-d H:i:s', strtotime( get_post_field( 'post_date', $post_id ) . ' + ' . $repeating_schedule['interval'] ) );
+	$next_post['post_date'] = date('Y-m-d H:i:s', strtotime(get_post_field('post_date', $post_id).' + '.$repeating_schedule['interval']));
 
 	// Make sure the next post will be in the future
-	if ( strtotime( $next_post['post_date'] ) <= time() ) {
+	if (strtotime($next_post['post_date']) <= time()) {
 		return false;
 	}
 
 	// All checks done, get that post scheduled!
-	$next_post_id = wp_insert_post( wp_slash( $next_post ), true );
+	$next_post_id = wp_insert_post(wp_slash($next_post), true);
 
-	if ( is_wp_error( $next_post_id ) ) {
+	if (is_wp_error($next_post_id)) {
 		return false;
 	}
 
 	// Mirror any post_meta
-	$post_meta = get_post_meta( $original_post['ID'] );
+	$post_meta = get_post_meta($original_post['ID']);
 
-	if ( $post_meta  ) {
+	if ($post_meta) {
 
 		// Ignore some internal meta fields
-		unset( $post_meta['_edit_lock'] );
-		unset( $post_meta['_edit_last'] );
+		unset($post_meta['_edit_lock']);
+		unset($post_meta['_edit_last']);
 
 		// Don't copy the post repeat meta as only the original post should have that
-		unset( $post_meta['hm-post-repeat'] );
+		unset($post_meta['hm-post-repeat']);
 
-		foreach ( $post_meta as $key => $values ) {
-			foreach ( $values as $value ) {
-				add_post_meta( $next_post_id, $key, maybe_unserialize( $value ) );
+		foreach ($post_meta as $key => $values) {
+			foreach ($values as $value) {
+				add_post_meta($next_post_id, $key, maybe_unserialize($value));
 			}
 		}
 	}
 
 	// Mirror any term relationships
-	$taxonomies = get_object_taxonomies( $original_post['post_type'] );
+	$taxonomies = get_object_taxonomies($original_post['post_type']);
 
-	foreach ( $taxonomies as $taxonomy ) {
-		wp_set_object_terms( $next_post_id, wp_list_pluck( wp_get_object_terms( $original_post['ID'], $taxonomy ), 'slug' ), $taxonomy );
+	foreach ($taxonomies as $taxonomy) {
+		wp_set_object_terms($next_post_id, wp_list_pluck(wp_get_object_terms($original_post['ID'], $taxonomy), 'slug'), $taxonomy);
 	}
 
 	return $next_post_id;
@@ -289,7 +289,7 @@ 
 	 *
 	 * @param string[] $post_types Post type slugs.
 	 */
-	return apply_filters( 'hm_post_repeat_post_types', array( 'post' ) );
+	return apply_filters('hm_post_repeat_post_types', array('post'));
 
 }
 
@@ -305,13 +305,13 @@ 
 	 *
 	 * @param array[] $schedules Schedule array items.
 	 */
-	$schedules = apply_filters( 'hm_post_repeat_schedules', array(
-		'daily'   => array( 'interval' => '1 day',   'display' => __( 'Daily',   'hm-post-repeat' ) ),
-		'weekly'  => array( 'interval' => '1 week',  'display' => __( 'Weekly',  'hm-post-repeat' ) ),
-		'monthly' => array( 'interval' => '1 month', 'display' => __( 'Monthly', 'hm-post-repeat' ) ),
-	) );
+	$schedules = apply_filters('hm_post_repeat_schedules', array(
+		'daily'   => array('interval' => '1 day', 'display' => __('Daily', 'hm-post-repeat')),
+		'weekly'  => array('interval' => '1 week', 'display' => __('Weekly', 'hm-post-repeat')),
+		'monthly' => array('interval' => '1 month', 'display' => __('Monthly', 'hm-post-repeat')),
+	));
 
-	foreach ( $schedules as $slug => &$schedule ) {
+	foreach ($schedules as $slug => &$schedule) {
 		$schedule['slug'] = $slug;
 	}
 
@@ -325,22 +325,22 @@ 
  * @param int $post_id The id of the post you want to check.
  * @return array|null The schedule to repeat by, or null if invalid.
  */
-function get_repeating_schedule( $post_id ) {
+function get_repeating_schedule($post_id) {
 
-	if ( ! is_repeating_post( $post_id ) ) {
+	if ( ! is_repeating_post($post_id)) {
 		return;
 	}
 
-	$repeating_schedule = get_post_meta( $post_id, 'hm-post-repeat', true );
+	$repeating_schedule = get_post_meta($post_id, 'hm-post-repeat', true);
 	$schedules = get_repeating_schedules();
 
 	// Backwards compatibility with 0.3 when we only supported weekly
-	if ( '1' === $repeating_schedule ) {
+	if ('1' === $repeating_schedule) {
 		$repeating_schedule = 'weekly';
 	}
 
-	if ( array_key_exists( $repeating_schedule, $schedules ) ) {
-		return $schedules[ $repeating_schedule ];
+	if (array_key_exists($repeating_schedule, $schedules)) {
+		return $schedules[$repeating_schedule];
 	}
 
 }
@@ -353,14 +353,14 @@ 
  * @param int $post_id The id of the post you want to check.
  * @return bool Whether the passed post_id is a repeating post or not.
  */
-function is_repeating_post( $post_id ) {
+function is_repeating_post($post_id) {
 
 	// We check $_POST data so that this function works inside a `save_post` hook when the post_meta hasn't yet been saved
-	if ( isset( $_POST['hm-post-repeat'] ) && isset( $_POST['ID'] ) && $_POST['ID'] === $post_id ) {
+	if (isset($_POST['hm-post-repeat']) && isset($_POST['ID']) && $_POST['ID'] === $post_id) {
 		return true;
 	}
 
-	if ( get_post_meta( $post_id, 'hm-post-repeat', true ) ) {
+	if (get_post_meta($post_id, 'hm-post-repeat', true)) {
 		return true;
 	}
 
@@ -376,11 +376,11 @@ 
  * @param int $post_id The id of the post you want to check.
  * @return bool Whether the passed post_id is a repeat post or not.
  */
-function is_repeat_post( $post_id ) {
+function is_repeat_post($post_id) {
 
-	$post_parent = get_post_field( 'post_parent', $post_id );
+	$post_parent = get_post_field('post_parent', $post_id);
 
-	if ( $post_parent && get_post_meta( $post_parent, 'hm-post-repeat', true ) ) {
+	if ($post_parent && get_post_meta($post_parent, 'hm-post-repeat', true)) {
 		return true;
 	}
 
@@ -394,13 +394,13 @@ 
  * @param int $post_id The id of a repeat or repeating post
  * @return Int|Bool Return the ID of the next repeat post_id or false if it can't find one
  */
-function get_next_scheduled_repeat_post( $post_id ) {
+function get_next_scheduled_repeat_post($post_id) {
 
-	$post = get_post( get_repeating_post( $post_id ) );
+	$post = get_post(get_repeating_post($post_id));
 
-	$repeat_posts = get_posts( array( 'post_status' => 'future', 'post_parent' => $post->ID ) );
+	$repeat_posts = get_posts(array('post_status' => 'future', 'post_parent' => $post->ID));
 
-	if ( isset( $repeat_posts[0] ) ) {
+	if (isset($repeat_posts[0])) {
 	 	return $repeat_posts[0];
 	}
 
@@ -414,17 +414,17 @@ 
  * @param int $post_id The id of a repeat or repeating post
  * @return Int|Bool Return the original repeating post_id or false if it can't find it
  */
-function get_repeating_post( $post_id ) {
+function get_repeating_post($post_id) {
 
 	$original_post_id = false;
 
 	// Are we publishing a repeat post
-	if ( is_repeat_post( $post_id ) ) {
-		$original_post_id = get_post( $post_id )->post_parent;
+	if (is_repeat_post($post_id)) {
+		$original_post_id = get_post($post_id)->post_parent;
 	}
 
 	// Or the original
-	elseif ( is_repeating_post( $post_id ) ) {
+	elseif (is_repeating_post($post_id)) {
 		$original_post_id = $post_id;
 	}