humanmade / repeatable-posts

hm-post-repeat.php

Spacing +125 added lines, -125 removed lines

@@ -35,37 +35,37 @@ 
 /**
  * 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__ . '\admin_table_row_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__.'\admin_table_row_post_states', 10, 2);
 
 // Add repeat type table view links to admin screen for each CPT that supports Repeatable Posts.
-add_action( 'init', function() {
-	foreach ( repeating_post_types() as $post_type ) {
-		add_filter( "views_edit-{$post_type}", __NAMESPACE__ . '\admin_table_views_links' );
+add_action('init', function() {
+	foreach (repeating_post_types() as $post_type) {
+		add_filter("views_edit-{$post_type}", __NAMESPACE__.'\admin_table_views_links');
 	}
 } );
 
 // Display only Repeatable Posts in admin table view for registered view links.
-add_filter( 'pre_get_posts', __NAMESPACE__ . '\admin_table_repeat_type_posts_query' );
+add_filter('pre_get_posts', __NAMESPACE__.'\admin_table_repeat_type_posts_query');
 
 /**
  * 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']);
 
 }
 
@@ -77,7 +77,7 @@ 
  */
 function publish_box_ui() {
 
-	if ( ! in_array( get_post_type(), repeating_post_types() ) ) {
+	if ( ! in_array(get_post_type(), repeating_post_types())) {
 		return;
 	} ?>
 
@@ -85,31 +85,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>
 
@@ -128,21 +128,21 @@ 
  * @param WP_Post $post        The post object to get / return the states.
  * @return array The array of post states with ours added.
  */
-function admin_table_row_post_states( $post_states, $post ) {
+function admin_table_row_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;
@@ -161,29 +161,29 @@ 
  * @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) {
 	global $is_creating_repeat;
 
 	// Stop - Repeat post is being saved, it doesn't need meta data.
 	// N.B. when inserting a Repeat post via `wp_insert_post` the `save_post` filters are firing again.
-	if ( $is_creating_repeat ) {
+	if ($is_creating_repeat) {
 		return;
 	}
 
-	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);
 	}
 }
 
@@ -206,57 +206,57 @@ 
  * @return int|\WP_Error A Repeat post ID on success, WP_Error if Repeat post could not be created.
  *                       Terminate function execution if conditions are not met.
  */
-function create_next_repeat_post( $post_id ) {
+function create_next_repeat_post($post_id) {
 	global $is_creating_repeat;
 
 	// Stop - Repeat post is being saved, we need to run this code just once.
 	// N.B. when inserting a Repeat post via `wp_insert_post` the `save_post` filters are firing again.
-	if ( $is_creating_repeat ) {
+	if ($is_creating_repeat) {
 		return;
 	}
 
-	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'];
@@ -265,10 +265,10 @@ 
 	$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 from the current time.
-	if ( strtotime( $next_post['post_date'] ) <= time() ) {
+	if (strtotime($next_post['post_date']) <= time()) {
 		return false;
 	}
 
@@ -279,41 +279,41 @@ 
 	 * @param array $repeating_schedule Repeating schedule info.
 	 * @param array $original_post      The original repeating post.
 	 */
-	$next_post = apply_filters( 'hm_post_repeat_edit_repeat_post', $next_post, $repeating_schedule, $original_post );
+	$next_post = apply_filters('hm_post_repeat_edit_repeat_post', $next_post, $repeating_schedule, $original_post);
 
 	// All checks done, get that post scheduled!
 	$is_creating_repeat = true;
-	$next_post_id       = wp_insert_post( wp_slash( $next_post ), true );
+	$next_post_id       = wp_insert_post(wp_slash($next_post), true);
 	$is_creating_repeat = false;
 
-	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;
@@ -333,7 +333,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'));
 
 }
 
@@ -349,13 +349,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;
 	}
 
@@ -369,22 +369,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];
 	}
 
 }
@@ -397,14 +397,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;
 	}
 
@@ -420,11 +420,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;
 	}
 
@@ -438,13 +438,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];
 	}
 
@@ -458,17 +458,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;
 	}
 
@@ -486,10 +486,10 @@ 
  *
  * @return array Available list of table views with custom added views per repeat type.
  */
-function admin_table_views_links( $views ) {
+function admin_table_views_links($views) {
 
 	// Add status link for each repeat type.
-	foreach ( get_available_repeat_types() as $repeat_type => $repeat_desc ) {
+	foreach (get_available_repeat_types() as $repeat_type => $repeat_desc) {
 
 		$url_args = array(
 			'post_type'      => get_current_screen()->post_type,
@@ -497,25 +497,25 @@ 
 		);
 
 		// Custom WP_Query to get posts count of repeat type.
-		$repeat_type_query = new \WP_Query( get_repeat_type_query_params( $repeat_type ) );
+		$repeat_type_query = new \WP_Query(get_repeat_type_query_params($repeat_type));
 
 		$link_label = sprintf(
 				'%s <span class="count">(%s)</span>',
-			esc_html( $repeat_desc ),
-			esc_html( number_format_i18n( $repeat_type_query->post_count ) )
+			esc_html($repeat_desc),
+			esc_html(number_format_i18n($repeat_type_query->post_count))
 		);
 
 		// Add current class to the link to highlight it when it's selected.
-		$class_html = ( get_repeat_type_url_param() === $repeat_type ) ? ' class="current"' : '';
+		$class_html = (get_repeat_type_url_param() === $repeat_type) ? ' class="current"' : '';
 
 		$link_html = sprintf(
 			'<a href="%s"%s>%s</a>',
-			esc_url( add_query_arg( $url_args, 'edit.php' ) ),
+			esc_url(add_query_arg($url_args, 'edit.php')),
 			$class_html, // html - hardcoded, no need to escape.
 			$link_label  // html - escaped earlier in the code.
 		);
 
-		$views[ $repeat_type ] = $link_html;
+		$views[$repeat_type] = $link_html;
 	}
 
 	return $views;
@@ -529,23 +529,23 @@ 
  *
  * @return mixed Main admin query with edited params to get posts of specified repeat type.
  */
-function admin_table_repeat_type_posts_query( $wp_query ) {
+function admin_table_repeat_type_posts_query($wp_query) {
 
 	// Stop - if not admin or not main query (there are secondary WP_Query for counting posts for view links, etc).
-	if ( ! is_admin() || ! $wp_query->is_main_query() ) {
+	if ( ! is_admin() || ! $wp_query->is_main_query()) {
 		return $wp_query;
 	}
 
 	// Get URL query param for repeat type and check it's valid.
 	$repeat_type = get_repeat_type_url_param();
 
-	if ( ! $repeat_type || ! is_allowed_repeat_type( $repeat_type ) ) {
+	if ( ! $repeat_type || ! is_allowed_repeat_type($repeat_type)) {
 		return $wp_query;
 	}
 
 	// Add a table view link per each repeat type.
-	foreach ( get_repeat_type_query_params( $repeat_type ) as $key => $value ) {
-		$wp_query->set( $key, $value );
+	foreach (get_repeat_type_query_params($repeat_type) as $key => $value) {
+		$wp_query->set($key, $value);
 	}
 
 	return $wp_query;
@@ -559,7 +559,7 @@ 
  *
  * @return array Array of custom WP_Query params.
  */
-function get_repeat_type_query_params( $repeat_type ) {
+function get_repeat_type_query_params($repeat_type) {
 
 	$query['post_type'] = get_current_screen()->post_type;
 
@@ -571,11 +571,11 @@ 
 			),
 	);
 
-	if ( $repeat_type === 'repeat' ) {
-		$query['post_parent__not_in'] = array( 0 );
+	if ($repeat_type === 'repeat') {
+		$query['post_parent__not_in'] = array(0);
 
-	} elseif ( $repeat_type === 'repeating' ) {
-		$query['post_parent__in'] = array( 0 );
+	} elseif ($repeat_type === 'repeating') {
+		$query['post_parent__in'] = array(0);
 	}
 
 	return $query;
@@ -588,7 +588,7 @@ 
  * @return string Sanitized string of repeat type being displayed.
  */
 function get_repeat_type_url_param() {
-	return isset( $_GET['hm-post-repeat'] ) ? sanitize_text_field( $_GET['hm-post-repeat'] ) : '';
+	return isset($_GET['hm-post-repeat']) ? sanitize_text_field($_GET['hm-post-repeat']) : '';
 }
 
 /**
@@ -598,8 +598,8 @@ 
  */
 function get_available_repeat_types() {
 	return array(
-		'repeating' => __( 'Repeating', 'hm-post-repeat' ),
-		'repeat'    => __( 'Repeat', 'hm-post-repeat' ),
+		'repeating' => __('Repeating', 'hm-post-repeat'),
+		'repeat'    => __('Repeat', 'hm-post-repeat'),
 	);
 }
 
@@ -611,6 +611,6 @@ 
  * @return bool True if repeat type is valid,
  *              False otherwise.
  */
-function is_allowed_repeat_type( $repeat_type ) {
-	return in_array( $repeat_type, array_keys( get_available_repeat_types() ) );
+function is_allowed_repeat_type($repeat_type) {
+	return in_array($repeat_type, array_keys(get_available_repeat_types()));
 }