From 5d0028cb9e2a9dcf65d2a4a860a79b76290f0863 Mon Sep 17 00:00:00 2001
From: Erick Hitter <git-contrib@ethitter.com>
Date: Sun, 26 Jun 2022 13:19:36 -0700
Subject: [PATCH] PHPCS
---
automatically-paginate-posts.php | 178 +++++++++++++++++++++----------
1 file changed, 123 insertions(+), 55 deletions(-)
diff --git a/automatically-paginate-posts.php b/automatically-paginate-posts.php
index 4568cd8..3a7b653 100644
--- a/automatically-paginate-posts.php
+++ b/automatically-paginate-posts.php
@@ -1,5 +1,7 @@
-<?php
+<?php // phpcs:ignore WordPress.Files.FileName.InvalidClassFileName
/**
+ * Automatically inserts the <!--nextpage--> Quicktag into WordPress posts, pages, or custom post type content.
+ *
* Plugin Name: Automatically Paginate Posts
* Plugin URI: http://www.oomphinc.com/plugins-modules/automatically-paginate-posts/
* Description: Automatically inserts the <!--nextpage--> Quicktag into WordPress posts, pages, or custom post type content.
@@ -24,30 +26,98 @@
* Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
*/
+/**
+ * Class Automatically_Paginate_Posts.
+ */
class Automatically_Paginate_Posts {
/**
- * Class variables
+ * Supported post types.
+ *
+ * @var array
*/
private $post_types;
+
+ /**
+ * Default supported post types.
+ *
+ * @var array
+ */
private $post_types_default = array( 'post' );
+ /**
+ * Desired number of pages to split to.
+ *
+ * @var int
+ */
private $num_pages;
+
+ /**
+ * Method for splitting content, either words or desired number of pages.
+ *
+ * @var string
+ */
private $paging_type_default = 'pages';
+
+ /**
+ * Default number of pages to split to.
+ *
+ * @var int
+ */
private $num_pages_default = 2;
+
+ /**
+ * Default number of words to split on.
+ *
+ * @var string|int
+ */
private $num_words_default = '';
+ /**
+ * Allowed split types.
+ *
+ * @var array
+ */
private $paging_types_allowed = array( 'pages', 'words' );
- // Ensure option names match values in this::uninstall
+ // Ensure option names match values in `uninstall()` method.
+
+ /**
+ * Supported-post-types option name.
+ *
+ * @var string
+ */
private $option_name_post_types = 'autopaging_post_types';
+
+ /**
+ * Split-type option name.
+ *
+ * @var string
+ */
private $option_name_paging_type = 'pages';
+
+ /**
+ * Option holding number of pages to split to.
+ *
+ * @var string
+ */
private $option_name_num_pages = 'autopaging_num_pages';
+
+ /**
+ * Option holding number of words to split on.
+ *
+ * @var string
+ */
private $option_name_num_words = 'autopaging_num_words';
+ /**
+ * Meta key used to indicate that a post shouldn't be automatically split.
+ *
+ * @var string
+ */
private $meta_key_disable_autopaging = '_disable_autopaging';
/**
- * Register actions and filters
+ * Register hooks.
*
* @uses add_action, register_uninstall_hook, add_filter
* @return null
@@ -56,12 +126,12 @@ class Automatically_Paginate_Posts {
add_action( 'plugins_loaded', array( $this, 'load_textdomain' ) );
add_action( 'init', array( $this, 'action_init' ) );
- //Admin settings
+ // Admin settings.
register_uninstall_hook( __FILE__, array( 'Automatically_Paginate_Posts', 'uninstall' ) );
add_filter( 'plugin_action_links', array( $this, 'filter_plugin_action_links' ), 10, 2 );
add_action( 'admin_init', array( $this, 'action_admin_init' ) );
- //Post-type settings
+ // Post-type settings.
add_action( 'add_meta_boxes', array( $this, 'action_add_meta_boxes' ) );
add_action( 'save_post', array( $this, 'action_save_post' ) );
add_filter( 'the_posts', array( $this, 'filter_the_posts' ) );
@@ -81,23 +151,23 @@ class Automatically_Paginate_Posts {
}
/**
- * Set post types this plugin can act on, either from Reading page or via filter
- * Also sets default number of pages to break content over, either from Reading page or via filter
+ * Set post types this plugin can act on, either from Reading page or via filter.
+ * Also sets default number of pages to break content over, either from Reading page or via filter.
*
* @uses apply_filters, get_option
* @action init
* @return null
*/
public function action_init() {
- //Post types
+ // Post types.
$this->post_types = apply_filters( 'autopaging_post_types', get_option( $this->option_name_post_types, $this->post_types_default ) );
- //Number of pages to break over
+ // Number of pages to break over.
$this->num_pages = absint( apply_filters( 'autopaging_num_pages_default', get_option( $this->option_name_num_pages, $this->num_pages_default ) ) );
if ( 0 == $this->num_pages )
$this->num_pages = $this->num_pages_default;
- //Number of words to break over
+ // Number of words to break over.
$this->num_words = absint( apply_filters( 'autopaging_num_words_default', get_option( $this->option_name_num_words, $this->num_words_default ) ) );
if ( 0 == $this->num_words )
$this->num_words = $this->num_words_default;
@@ -133,8 +203,8 @@ class Automatically_Paginate_Posts {
}
/**
- * Register settings and settings sections
- * Settings appear on the Reading page
+ * Register settings and settings sections.
+ * Settings appear on the Reading page.
*
* @uses register_setting, add_settings_section, __, __return_false, add_settings_field
* @action admin_init
@@ -152,24 +222,23 @@ class Automatically_Paginate_Posts {
}
/**
- * Render post types options
+ * Render post types options.
*
* @uses get_post_types, get_option, esc_attr, checked, esc_html
- * @return string
+ * @return void
*/
public function settings_field_post_types() {
- //Get all public post types
+ // Get all public post types.
$post_types = get_post_types( array(
'public' => true
), 'objects' );
- //Remove attachments
unset( $post_types[ 'attachment' ] );
- //Current settings
+ // Current settings.
$current_types = get_option( $this->option_name_post_types, $this->post_types_default );
- //Output checkboxes
+ // Output checkboxes.
foreach ( $post_types as $post_type => $atts ) :
?>
<input type="checkbox" name="<?php echo esc_attr( $this->option_name_post_types ); ?>[]" id="post-type-<?php echo esc_attr( $post_type ); ?>" value="<?php echo esc_attr( $post_type ); ?>"<?php checked( in_array( $post_type, $current_types ) ); ?> /> <label for="post-type-<?php echo esc_attr( $post_type ); ?>"><?php echo esc_html( $atts->label ); ?></label><br />
@@ -178,7 +247,7 @@ class Automatically_Paginate_Posts {
}
/**
- * Sanitize post type inputs
+ * Sanitize post type inputs.
*
* @param array $post_types_checked
* @uses get_post_types
@@ -187,17 +256,16 @@ class Automatically_Paginate_Posts {
public function sanitize_supported_post_types( $post_types_checked ) {
$post_types_sanitized = array();
- //Ensure that only existing, public post types are submitted as valid options
+ // Ensure that only existing, public post types are submitted as valid options.
if ( is_array( $post_types_checked ) && ! empty( $post_types_checked ) ) {
- //Get all public post types
+ // Get all public post types.
$post_types = get_post_types( array(
'public' => true
) );
- //Remove attachments
unset( $post_types[ 'attachment' ] );
- //Check input post types against those registered with WordPress and made available to this plugin
+ // Check input post types against those registered with WordPress and made available to this plugin.
foreach ( $post_types_checked as $post_type ) {
if ( array_key_exists( $post_type, $post_types ) )
$post_types_sanitized[] = $post_type;
@@ -208,12 +276,12 @@ class Automatically_Paginate_Posts {
}
/**
- * Render option to choose paging type and options for that type
+ * Render option to choose paging type and options for that type.
*
* @uses get_option()
* @uses esc_attr()
* @uses checked()
- * @return string
+ * @return void
*/
public function settings_field_paging_type() {
$paging_type = get_option( $this->option_name_paging_type, $this->paging_type_default );
@@ -237,20 +305,20 @@ class Automatically_Paginate_Posts {
}
/**
- * Validate chosen paging type against allowed values
+ * Validate chosen paging type against allowed values.
*
* @param string
* @return string
*/
public function sanitize_paging_type( $type ) {
- return in_array( $type, $this->paging_types_allowed ) ? $type : $this->paging_type_default;
+ return in_array( $type, $this->paging_types_allowed, true ) ? $type : $this->paging_type_default;
}
/**
- * Render dropdown for choosing number of pages to break content over
+ * Render dropdown for choosing number of pages to break content over.
*
* @uses get_option, apply_filters, esc_attr, selected
- * @return string
+ * @return void
*/
public function settings_field_num_pages() {
$num_pages = get_option( $this->option_name_num_pages, $this->num_pages_default );
@@ -266,9 +334,9 @@ class Automatically_Paginate_Posts {
}
/**
- * Sanitize number of pages input
+ * Sanitize number of pages input.
*
- * @param int $num_pages
+ * @param int $num_pages Number of pages to split to.
* @uses apply_filters
* @return int
*/
@@ -277,7 +345,7 @@ class Automatically_Paginate_Posts {
}
/**
- * Render input field for specifying approximate number of words each page should contain
+ * Render input field for specifying approximate number of words each page should contain.
*
* @uses get_option, apply_filters, esc_attr, selected
* @return string
@@ -292,9 +360,9 @@ class Automatically_Paginate_Posts {
}
/**
- * Sanitize number of words input. No fewer than 10 by default, filterable by autopaging_max_num_words
+ * Sanitize number of words input. No fewer than 10 by default, filterable by `autopaging_max_num_words`.
*
- * @param int $num_words
+ * @param int $num_words Number of words to split on.
* @uses apply_filters
* @return int
*/
@@ -309,7 +377,7 @@ class Automatically_Paginate_Posts {
}
/**
- * Add autopaging metabox
+ * Add autopaging metabox.
*
* @uses add_metabox, __
* @action add_meta_box
@@ -322,7 +390,7 @@ class Automatically_Paginate_Posts {
}
/**
- * Render autopaging metabox
+ * Render autopaging metabox.
*
* @param object $post
* @uses esc_attr, checked, _e, __, wp_nonce_field
@@ -340,7 +408,7 @@ class Automatically_Paginate_Posts {
}
/**
- * Save autopaging metabox
+ * Save autopaging metabox.
*
* @param int $post_id
* @uses DOING_AUTOSAVE, wp_verify_nonce, update_post_meta, delete_post_meta
@@ -365,7 +433,7 @@ class Automatically_Paginate_Posts {
* Automatically page posts by injecting <!--nextpage--> Quicktag.
* Only applied if the post type matches specified options and post doesn't already contain the Quicktag.
*
- * @param array $posts
+ * @param array $posts Array of posts retrieved by WP_Query.
* @uses is_admin, get_post_meta, absint, apply_filters
* @filter the_posts
* @return array
@@ -374,33 +442,33 @@ class Automatically_Paginate_Posts {
if ( ! is_admin() ) {
foreach( $posts as $the_post ) {
if ( in_array( $the_post->post_type, $this->post_types ) && ! preg_match( '#<!--nextpage-->#i', $the_post->post_content ) && ! (bool) get_post_meta( $the_post->ID, $this->meta_key_disable_autopaging, true ) ) {
- //In-time filtering of number of pages to break over, based on post data. If value is less than 2, nothing should be done.
+ // In-time filtering of number of pages to break over, based on post data. If value is less than 2, nothing should be done.
$num_pages = absint( apply_filters( 'autopaging_num_pages', absint( $this->num_pages ), $the_post ) );
$num_words = absint( apply_filters( 'autopaging_num_words', absint( $this->num_words ), $the_post ) );
if ( $num_pages < 2 && empty( $num_words ) )
continue;
- //Start with post content, but alias to protect the raw content.
+ // Start with post content, but alias to protect the raw content.
$content = $the_post->post_content;
- //Normalize post content to simplify paragraph counting and automatic paging. Accounts for content that hasn't been cleaned up by TinyMCE.
+ // Normalize post content to simplify paragraph counting and automatic paging. Accounts for content that hasn't been cleaned up by TinyMCE.
$content = preg_replace( '#<p>(.+?)</p>#i', "$1\r\n\r\n", $content );
$content = preg_replace( '#<br(\s*/)?>#i', "\r\n", $content );
- //Count paragraphs
+ // C.ount paragraphs
$count = preg_match_all( '#\r\n\r\n#', $content, $matches );
- //Keep going, if we have something to count.
+ // Keep going, if we have something to count.
if ( is_int( $count ) && 0 < $count ) {
- //Explode content at double (or more) line breaks
+ // Explode content at double (or more) line breaks.
$content = explode( "\r\n\r\n", $content );
switch ( get_option( $this->option_name_paging_type, $this->paging_type_default ) ) {
case 'words' :
$word_counter = 0;
- // Count words per paragraph and break after the paragraph that exceeds the set threshold
+ // Count words per paragraph and break after the paragraph that exceeds the set threshold.
foreach ( $content as $index => $paragraph ) {
$paragraph_words = count( preg_split( '/\s+/', strip_tags( $paragraph ) ) );
$word_counter += $paragraph_words;
@@ -422,22 +490,22 @@ class Automatically_Paginate_Posts {
case 'pages' :
default :
- //Count number of paragraphs content was exploded to
+ // Count number of paragraphs content was exploded to.
$count = count( $content );
- //Determine when to insert Quicktag
+ // Determine when to insert Quicktag.
$insert_every = $count / $num_pages;
$insert_every_rounded = round( $insert_every );
- //If number of pages is greater than number of paragraphs, put each paragraph on its own page
+ // If number of pages is greater than number of paragraphs, put each paragraph on its own page.
if ( $num_pages > $count ) {
$insert_every_rounded = 1;
}
- //Set initial counter position.
+ // Set initial counter position.
$i = $count - 1 == $num_pages ? 2 : 1;
- //Loop through content pieces and append Quicktag as is appropriate
+ // Loop through content pieces and append Quicktag as is appropriate.
foreach ( $content as $key => $value ) {
if ( $key + 1 == $count ) {
break;
@@ -449,7 +517,7 @@ class Automatically_Paginate_Posts {
}
}
- //Clean up
+ // Clean up.
unset( $count );
unset( $insert_every );
unset( $insert_every_rounded );
@@ -459,14 +527,14 @@ class Automatically_Paginate_Posts {
break;
}
- //Reunite content
+ // Reunite content.
$content = implode( "\r\n\r\n", $content );
- //And, overwrite the original content
+ // And, overwrite the original content.
$the_post->post_content = $content;
}
- //Lastly, clean up.
+ // Lastly, clean up.
unset( $num_pages );
unset( $num_words );
unset( $content );
--
GitLab