X-Git-Url: https://scripts.mit.edu/gitweb/autoinstalls/wordpress.git/blobdiff_plain/41578db67d72562346e4dbb2a14889b23d522813..53f4633144ed68c8b8fb5861f992b5489894a940:/wp-includes/rewrite.php
diff --git a/wp-includes/rewrite.php b/wp-includes/rewrite.php
index 0de3b3e3..a2ddc590 100644
--- a/wp-includes/rewrite.php
+++ b/wp-includes/rewrite.php
@@ -9,12 +9,13 @@
/**
* Add a straight rewrite rule.
*
- * @see WP_Rewrite::add_rule() for long description.
* @since 2.1.0
*
- * @param string $regex Regular Expression to match request against.
+ * @global WP_Rewrite $wp_rewrite
+ *
+ * @param string $regex Regular Expression to match request against.
* @param string $redirect Page to redirect to.
- * @param string $after Optional, default is 'bottom'. Where to add rule, can also be 'top'.
+ * @param string $after Optional, default is 'bottom'. Where to add rule, can also be 'top'.
*/
function add_rewrite_rule($regex, $redirect, $after = 'bottom') {
global $wp_rewrite;
@@ -28,10 +29,12 @@ function add_rewrite_rule($regex, $redirect, $after = 'bottom') {
* you call this on, or before, the 'init' hook. This is because $query defaults
* to "$tag=", and for this to work a new query var has to be added.
*
- * @see WP_Rewrite::add_rewrite_tag()
* @since 2.1.0
*
- * @param string $tag Name of the new rewrite tag.
+ * @global WP_Rewrite $wp_rewrite
+ * @global WP $wp
+ *
+ * @param string $tag Name of the new rewrite tag.
* @param string $regex Regular expression to substitute the tag for in rewrite rules.
* @param string $query String to append to the rewritten query. Must end in '='. Optional.
*/
@@ -54,13 +57,14 @@ function add_rewrite_tag( $tag, $regex, $query = '' ) {
/**
* Add permalink structure.
*
- * @see WP_Rewrite::add_permastruct()
* @since 3.0.0
*
- * @param string $name Name for permalink structure.
+ * @global WP_Rewrite $wp_rewrite
+ *
+ * @param string $name Name for permalink structure.
* @param string $struct Permalink structure.
- * @param array $args Optional configuration for building the rules from the permalink structure,
- * see {@link WP_Rewrite::add_permastruct()} for full details.
+ * @param array $args Optional configuration for building the rules from the permalink structure,
+ * see {@link WP_Rewrite::add_permastruct()} for full details.
*/
function add_permastruct( $name, $struct, $args = array() ) {
global $wp_rewrite;
@@ -71,7 +75,7 @@ function add_permastruct( $name, $struct, $args = array() ) {
if ( func_num_args() == 4 )
$args['ep_mask'] = func_get_arg( 3 );
- return $wp_rewrite->add_permastruct( $name, $struct, $args );
+ $wp_rewrite->add_permastruct( $name, $struct, $args );
}
/**
@@ -79,7 +83,9 @@ function add_permastruct( $name, $struct, $args = array() ) {
*
* @since 2.1.0
*
- * @param string $feedname
+ * @global WP_Rewrite $wp_rewrite
+ *
+ * @param string $feedname
* @param callback $function Callback to run on feed display.
* @return string Feed action name.
*/
@@ -97,11 +103,12 @@ function add_feed($feedname, $function) {
/**
* Remove rewrite rules and then recreate rewrite rules.
*
- * @see WP_Rewrite::flush_rules()
* @since 3.0.0
*
+ * @global WP_Rewrite $wp_rewrite
+ *
* @param bool $hard Whether to update .htaccess (hard flush) or just update
- * rewrite_rules transient (soft flush). Default is true (hard).
+ * rewrite_rules transient (soft flush). Default is true (hard).
*/
function flush_rewrite_rules( $hard = true ) {
global $wp_rewrite;
@@ -226,33 +233,33 @@ define( 'EP_ALL', EP_PERMALINK | EP_ATTACHMENT | EP_ROOT | EP_COMMENTS | EP_SEAR
* Adding an endpoint creates extra rewrite rules for each of the matching
* places specified by the provided bitmask. For example:
*
- *
- * add_rewrite_endpoint( 'json', EP_PERMALINK | EP_PAGES );
- *
+ * add_rewrite_endpoint( 'json', EP_PERMALINK | EP_PAGES );
*
* will add a new rewrite rule ending with "json(/(.*))?/?$" for every permastruct
* that describes a permalink (post) or page. This is rewritten to "json=$match"
* where $match is the part of the URL matched by the endpoint regex (e.g. "foo" in
- * "/json/foo/").
+ * "[permalink]/json/foo/").
*
* A new query var with the same name as the endpoint will also be created.
*
* When specifying $places ensure that you are using the EP_* constants (or a
* combination of them using the bitwise OR operator) as their values are not
- * guaranteed to remain static (especially EP_ALL).
+ * guaranteed to remain static (especially `EP_ALL`).
*
- * Be sure to flush the rewrite rules - flush_rewrite_rules() - when your plugin gets
+ * Be sure to flush the rewrite rules - see flush_rewrite_rules() - when your plugin gets
* activated and deactivated.
*
* @since 2.1.0
- * @see WP_Rewrite::add_endpoint()
- * @global object $wp_rewrite
+ * @since 4.3.0 Added support for skipping query var registration by passing `false` to `$query_var`.
+ *
+ * @global WP_Rewrite $wp_rewrite
*
- * @param string $name Name of the endpoint.
- * @param int $places Endpoint mask describing the places the endpoint should be added.
- * @param string $query_var Name of the corresponding query variable. Defaults to $name.
+ * @param string $name Name of the endpoint.
+ * @param int $places Endpoint mask describing the places the endpoint should be added.
+ * @param string|bool $query_var Name of the corresponding query variable. Pass `false` to skip registering a query_var
+ * for this endpoint. Defaults to the value of `$name`.
*/
-function add_rewrite_endpoint( $name, $places, $query_var = null ) {
+function add_rewrite_endpoint( $name, $places, $query_var = true ) {
global $wp_rewrite;
$wp_rewrite->add_endpoint( $name, $places, $query_var );
}
@@ -276,6 +283,121 @@ function _wp_filter_taxonomy_base( $base ) {
return $base;
}
+
+/**
+ * Resolve numeric slugs that collide with date permalinks.
+ *
+ * Permalinks of posts with numeric slugs can sometimes look to WP_Query::parse_query()
+ * like a date archive, as when your permalink structure is `/%year%/%postname%/` and
+ * a post with post_name '05' has the URL `/2015/05/`.
+ *
+ * This function detects conflicts of this type and resolves them in favor of the
+ * post permalink.
+ *
+ * Note that, since 4.3.0, wp_unique_post_slug() prevents the creation of post slugs
+ * that would result in a date archive conflict. The resolution performed in this
+ * function is primarily for legacy content, as well as cases when the admin has changed
+ * the site's permalink structure in a way that introduces URL conflicts.
+ *
+ * @since 4.3.0
+ *
+ * @param array $query_vars Optional. Query variables for setting up the loop, as determined in
+ * WP::parse_request(). Default empty array.
+ * @return array Returns the original array of query vars, with date/post conflicts resolved.
+ */
+function wp_resolve_numeric_slug_conflicts( $query_vars = array() ) {
+ if ( ! isset( $query_vars['year'] ) && ! isset( $query_vars['monthnum'] ) && ! isset( $query_vars['day'] ) ) {
+ return $query_vars;
+ }
+
+ // Identify the 'postname' position in the permastruct array.
+ $permastructs = array_values( array_filter( explode( '/', get_option( 'permalink_structure' ) ) ) );
+ $postname_index = array_search( '%postname%', $permastructs );
+
+ if ( false === $postname_index ) {
+ return $query_vars;
+ }
+
+ /*
+ * A numeric slug could be confused with a year, month, or day, depending on position. To account for
+ * the possibility of post pagination (eg 2015/2 for the second page of a post called '2015'), our
+ * `is_*` checks are generous: check for year-slug clashes when `is_year` *or* `is_month`, and check
+ * for month-slug clashes when `is_month` *or* `is_day`.
+ */
+ $compare = '';
+ if ( 0 === $postname_index && ( isset( $query_vars['year'] ) || isset( $query_vars['monthnum'] ) ) ) {
+ $compare = 'year';
+ } elseif ( '%year%' === $permastructs[ $postname_index - 1 ] && ( isset( $query_vars['monthnum'] ) || isset( $query_vars['day'] ) ) ) {
+ $compare = 'monthnum';
+ } elseif ( '%monthnum%' === $permastructs[ $postname_index - 1 ] && isset( $query_vars['day'] ) ) {
+ $compare = 'day';
+ }
+
+ if ( ! $compare ) {
+ return $query_vars;
+ }
+
+ // This is the potentially clashing slug.
+ $value = $query_vars[ $compare ];
+
+ $post = get_page_by_path( $value, OBJECT, 'post' );
+ if ( ! ( $post instanceof WP_Post ) ) {
+ return $query_vars;
+ }
+
+ // If the date of the post doesn't match the date specified in the URL, resolve to the date archive.
+ if ( preg_match( '/^([0-9]{4})\-([0-9]{2})/', $post->post_date, $matches ) && isset( $query_vars['year'] ) && ( 'monthnum' === $compare || 'day' === $compare ) ) {
+ // $matches[1] is the year the post was published.
+ if ( intval( $query_vars['year'] ) !== intval( $matches[1] ) ) {
+ return $query_vars;
+ }
+
+ // $matches[2] is the month the post was published.
+ if ( 'day' === $compare && isset( $query_vars['monthnum'] ) && intval( $query_vars['monthnum'] ) !== intval( $matches[2] ) ) {
+ return $query_vars;
+ }
+ }
+
+ /*
+ * If the located post contains nextpage pagination, then the URL chunk following postname may be
+ * intended as the page number. Verify that it's a valid page before resolving to it.
+ */
+ $maybe_page = '';
+ if ( 'year' === $compare && isset( $query_vars['monthnum'] ) ) {
+ $maybe_page = $query_vars['monthnum'];
+ } elseif ( 'monthnum' === $compare && isset( $query_vars['day'] ) ) {
+ $maybe_page = $query_vars['day'];
+ }
+
+ $post_page_count = substr_count( $post->post_content, '' ) + 1;
+
+ // If the post doesn't have multiple pages, but a 'page' candidate is found, resolve to the date archive.
+ if ( 1 === $post_page_count && $maybe_page ) {
+ return $query_vars;
+ }
+
+ // If the post has multiple pages and the 'page' number isn't valid, resolve to the date archive.
+ if ( $post_page_count > 1 && $maybe_page > $post_page_count ) {
+ return $query_vars;
+ }
+
+ // If we've gotten to this point, we have a slug/date clash. First, adjust for nextpage.
+ if ( '' !== $maybe_page ) {
+ $query_vars['page'] = intval( $maybe_page );
+ }
+
+ // Next, unset autodetected date-related query vars.
+ unset( $query_vars['year'] );
+ unset( $query_vars['monthnum'] );
+ unset( $query_vars['day'] );
+
+ // Then, set the identified post.
+ $query_vars['name'] = $post->post_name;
+
+ // Finally, return the modified query vars.
+ return $query_vars;
+}
+
/**
* Examine a url and try to determine the post ID it represents.
*
@@ -283,10 +405,13 @@ function _wp_filter_taxonomy_base( $base ) {
*
* @since 1.0.0
*
+ * @global WP_Rewrite $wp_rewrite
+ * @global WP $wp
+ *
* @param string $url Permalink to check.
* @return int Post ID, or 0 on failure.
*/
-function url_to_postid($url) {
+function url_to_postid( $url ) {
global $wp_rewrite;
/**
@@ -365,7 +490,7 @@ function url_to_postid($url) {
if ( preg_match("#^$match#", $request_match, $matches) ) {
if ( $wp_rewrite->use_verbose_page_rules && preg_match( '/pagename=\$matches\[([0-9]+)\]/', $query, $varmatch ) ) {
- // this is a verbose page match, lets check to be sure about it
+ // This is a verbose page match, let's check to be sure about it.
if ( ! get_page_by_path( $matches[ $varmatch[1] ] ) )
continue;
}
@@ -391,6 +516,9 @@ function url_to_postid($url) {
}
}
+ // Resolve conflicts between posts with numeric slugs and date archive queries.
+ $query = wp_resolve_numeric_slug_conflicts( $query );
+
// Do the query
$query = new WP_Query( $query );
if ( ! empty( $query->posts ) && $query->is_singular )
@@ -422,19 +550,17 @@ class WP_Rewrite {
* Permalink structure for posts.
*
* @since 1.5.0
- * @access private
* @var string
*/
- var $permalink_structure;
+ public $permalink_structure;
/**
* Whether to add trailing slashes.
*
* @since 2.2.0
- * @access private
* @var bool
*/
- var $use_trailing_slashes;
+ public $use_trailing_slashes;
/**
* Base for the author permalink structure (example.com/$author_base/authorname).
@@ -503,10 +629,18 @@ class WP_Rewrite {
* Pagination permalink base.
*
* @since 3.1.0
+ * @var string
+ */
+ public $pagination_base = 'page';
+
+ /**
+ * Comments pagination permalink base.
+ *
+ * @since 4.2.0
* @access private
* @var string
*/
- var $pagination_base = 'page';
+ var $comments_pagination_base = 'comment-page';
/**
* Feed permalink base.
@@ -524,7 +658,7 @@ class WP_Rewrite {
* @access private
* @var string
*/
- var $comments_feed_structure;
+ var $comment_feed_structure;
/**
* Feed request permalink structure.
@@ -544,10 +678,9 @@ class WP_Rewrite {
*
* @see WP_Rewrite::init()
* @since 1.5.0
- * @access private
* @var string
*/
- var $front;
+ public $front;
/**
* The prefix for all permalink structures.
@@ -559,10 +692,9 @@ class WP_Rewrite {
* @see WP_Rewrite::init()
* @see WP_Rewrite::using_index_permalinks()
* @since 1.5.0
- * @access private
* @var string
*/
- var $root = '';
+ public $root = '';
/**
* The name of the index file which is the entry point to all requests.
@@ -571,7 +703,7 @@ class WP_Rewrite {
* @access public
* @var string
*/
- var $index = 'index.php';
+ public $index = 'index.php';
/**
* Variable name to use for regex matches in the rewritten query.
@@ -654,7 +786,7 @@ class WP_Rewrite {
* @access public
* @var bool
*/
- var $use_verbose_rules = false;
+ public $use_verbose_rules = false;
/**
* Could post permalinks be confused with those of pages?
@@ -666,13 +798,13 @@ class WP_Rewrite {
* Now it tells {@link WP::parse_request()} to check if a URL matching the
* page permastruct is actually a page before accepting it.
*
- * @link http://core.trac.wordpress.org/ticket/16687
+ * @link https://core.trac.wordpress.org/ticket/16687
* @see WP_Rewrite::init()
* @since 2.5.0
* @access public
* @var bool
*/
- var $use_verbose_page_rules = true;
+ public $use_verbose_page_rules = true;
/**
* Rewrite tags that can be used in permalink structures.
@@ -748,10 +880,9 @@ class WP_Rewrite {
* Supported default feeds.
*
* @since 1.5.0
- * @access private
* @var array
*/
- var $feeds = array( 'feed', 'rdf', 'rss', 'rss2', 'atom' );
+ public $feeds = array( 'feed', 'rdf', 'rss', 'rss2', 'atom' );
/**
* Whether permalinks are being used.
@@ -763,7 +894,7 @@ class WP_Rewrite {
*
* @return bool True, if permalinks are enabled.
*/
- function using_permalinks() {
+ public function using_permalinks() {
return ! empty($this->permalink_structure);
}
@@ -777,15 +908,12 @@ class WP_Rewrite {
*
* @return bool
*/
- function using_index_permalinks() {
- if ( empty($this->permalink_structure) )
+ public function using_index_permalinks() {
+ if ( empty( $this->permalink_structure ) ) {
return false;
-
+ }
// If the index is not in the permalink, we're using mod_rewrite.
- if ( preg_match('#^/*' . $this->index . '#', $this->permalink_structure) )
- return true;
-
- return false;
+ return preg_match( '#^/*' . $this->index . '#', $this->permalink_structure );
}
/**
@@ -798,11 +926,8 @@ class WP_Rewrite {
*
* @return bool
*/
- function using_mod_rewrite_permalinks() {
- if ( $this->using_permalinks() && ! $this->using_index_permalinks() )
- return true;
- else
- return false;
+ public function using_mod_rewrite_permalinks() {
+ return $this->using_permalinks() && ! $this->using_index_permalinks();
}
/**
@@ -821,7 +946,7 @@ class WP_Rewrite {
* @param int $number Index number.
* @return string
*/
- function preg_index($number) {
+ public function preg_index($number) {
$match_prefix = '$';
$match_suffix = '';
@@ -842,9 +967,11 @@ class WP_Rewrite {
* @since 2.5.0
* @access public
*
+ * @global wpdb $wpdb
+ *
* @return array Array of page URIs as first element and attachment URIs as second element.
*/
- function page_uri_index() {
+ public function page_uri_index() {
global $wpdb;
//get pages in order of hierarchy, i.e. children after parents
@@ -886,7 +1013,7 @@ class WP_Rewrite {
*
* @return array
*/
- function page_rewrite_rules() {
+ public function page_rewrite_rules() {
// the extra .? at the beginning prevents clashes with other regular expressions in the rules array
$this->add_rewrite_tag( '%pagename%', '(.?.+?)', 'pagename=' );
@@ -911,9 +1038,9 @@ class WP_Rewrite {
* @since 1.5.0
* @access public
*
- * @return bool|string False on no permalink structure. Date permalink structure.
+ * @return string|false False on no permalink structure. Date permalink structure.
*/
- function get_date_permastruct() {
+ public function get_date_permastruct() {
if ( isset($this->date_structure) )
return $this->date_structure;
@@ -965,9 +1092,9 @@ class WP_Rewrite {
* @since 1.5.0
* @access public
*
- * @return bool|string False on failure. Year structure on success.
+ * @return false|string False on failure. Year structure on success.
*/
- function get_year_permastruct() {
+ public function get_year_permastruct() {
$structure = $this->get_date_permastruct();
if ( empty($structure) )
@@ -975,7 +1102,6 @@ class WP_Rewrite {
$structure = str_replace('%monthnum%', '', $structure);
$structure = str_replace('%day%', '', $structure);
-
$structure = preg_replace('#/+#', '/', $structure);
return $structure;
@@ -990,16 +1116,15 @@ class WP_Rewrite {
* @since 1.5.0
* @access public
*
- * @return bool|string False on failure. Year/Month structure on success.
+ * @return false|string False on failure. Year/Month structure on success.
*/
- function get_month_permastruct() {
+ public function get_month_permastruct() {
$structure = $this->get_date_permastruct();
if ( empty($structure) )
return false;
$structure = str_replace('%day%', '', $structure);
-
$structure = preg_replace('#/+#', '/', $structure);
return $structure;
@@ -1013,9 +1138,9 @@ class WP_Rewrite {
* @since 1.5.0
* @access public
*
- * @return bool|string False on failure. Year/Month/Day structure on success.
+ * @return string|false False on failure. Year/Month/Day structure on success.
*/
- function get_day_permastruct() {
+ public function get_day_permastruct() {
return $this->get_date_permastruct();
}
@@ -1030,9 +1155,9 @@ class WP_Rewrite {
* @since 1.5.0
* @access public
*
- * @return bool|string False on failure. Category permalink structure.
+ * @return string|false False on failure. Category permalink structure.
*/
- function get_category_permastruct() {
+ public function get_category_permastruct() {
return $this->get_extra_permastruct('category');
}
@@ -1047,9 +1172,9 @@ class WP_Rewrite {
* @since 2.3.0
* @access public
*
- * @return bool|string False on failure. Tag permalink structure.
+ * @return string|false False on failure. Tag permalink structure.
*/
- function get_tag_permastruct() {
+ public function get_tag_permastruct() {
return $this->get_extra_permastruct('post_tag');
}
@@ -1060,9 +1185,9 @@ class WP_Rewrite {
* @access public
*
* @param string $name Permalink structure name.
- * @return string|bool False if not found. Permalink structure string.
+ * @return string|false False if not found. Permalink structure string.
*/
- function get_extra_permastruct($name) {
+ public function get_extra_permastruct($name) {
if ( empty($this->permalink_structure) )
return false;
@@ -1082,9 +1207,9 @@ class WP_Rewrite {
* @since 1.5.0
* @access public
*
- * @return string|bool False if not found. Permalink structure string.
+ * @return string|false False if not found. Permalink structure string.
*/
- function get_author_permastruct() {
+ public function get_author_permastruct() {
if ( isset($this->author_structure) )
return $this->author_structure;
@@ -1108,9 +1233,9 @@ class WP_Rewrite {
* @since 1.5.0
* @access public
*
- * @return string|bool False if not found. Permalink structure string.
+ * @return string|false False if not found. Permalink structure string.
*/
- function get_search_permastruct() {
+ public function get_search_permastruct() {
if ( isset($this->search_structure) )
return $this->search_structure;
@@ -1134,9 +1259,9 @@ class WP_Rewrite {
* @since 1.5.0
* @access public
*
- * @return string|bool False if not found. Permalink structure string.
+ * @return string|false False if not found. Permalink structure string.
*/
- function get_page_permastruct() {
+ public function get_page_permastruct() {
if ( isset($this->page_structure) )
return $this->page_structure;
@@ -1160,9 +1285,9 @@ class WP_Rewrite {
* @since 1.5.0
* @access public
*
- * @return string|bool False if not found. Permalink structure string.
+ * @return string|false False if not found. Permalink structure string.
*/
- function get_feed_permastruct() {
+ public function get_feed_permastruct() {
if ( isset($this->feed_structure) )
return $this->feed_structure;
@@ -1186,9 +1311,9 @@ class WP_Rewrite {
* @since 1.5.0
* @access public
*
- * @return string|bool False if not found. Permalink structure string.
+ * @return string|false False if not found. Permalink structure string.
*/
- function get_comment_feed_permastruct() {
+ public function get_comment_feed_permastruct() {
if ( isset($this->comment_feed_structure) )
return $this->comment_feed_structure;
@@ -1214,11 +1339,11 @@ class WP_Rewrite {
* @since 1.5.0
* @access public
*
- * @param string $tag Name of the rewrite tag to add or update.
+ * @param string $tag Name of the rewrite tag to add or update.
* @param string $regex Regular expression to substitute the tag for in rewrite rules.
* @param string $query String to append to the rewritten query. Must end in '='.
*/
- function add_rewrite_tag( $tag, $regex, $query ) {
+ public function add_rewrite_tag( $tag, $regex, $query ) {
$position = array_search( $tag, $this->rewritecode );
if ( false !== $position && null !== $position ) {
$this->rewritereplace[ $position ] = $regex;
@@ -1241,16 +1366,16 @@ class WP_Rewrite {
* @access public
*
* @param string $permalink_structure The permalink structure.
- * @param int $ep_mask Endpoint mask defining what endpoints are added to the structure. Default is EP_NONE.
- * @param bool $paged Should archive pagination rules be added for the structure? Default is true.
- * @param bool $feed Should feed rewrite rules be added for the structure? Default is true.
- * @param bool $forcomments Should the feed rules be a query for a comments feed? Default is false.
- * @param bool $walk_dirs Should the 'directories' making up the structure be walked over and rewrite rules
- * built for each in turn? Default is true.
- * @param bool $endpoints Should endpoints be applied to the generated rewrite rules? Default is true.
+ * @param int $ep_mask Endpoint mask defining what endpoints are added to the structure. Default is EP_NONE.
+ * @param bool $paged Should archive pagination rules be added for the structure? Default is true.
+ * @param bool $feed Should feed rewrite rules be added for the structure? Default is true.
+ * @param bool $forcomments Should the feed rules be a query for a comments feed? Default is false.
+ * @param bool $walk_dirs Should the 'directories' making up the structure be walked over and rewrite rules
+ * built for each in turn? Default is true.
+ * @param bool $endpoints Should endpoints be applied to the generated rewrite rules? Default is true.
* @return array Rewrite rule list.
*/
- function generate_rewrite_rules($permalink_structure, $ep_mask = EP_NONE, $paged = true, $feed = true, $forcomments = false, $walk_dirs = true, $endpoints = true) {
+ public function generate_rewrite_rules($permalink_structure, $ep_mask = EP_NONE, $paged = true, $feed = true, $forcomments = false, $walk_dirs = true, $endpoints = true) {
//build a regex to match the feed section of URLs, something like (feed|atom|rss|rss2)/?
$feedregex2 = '';
foreach ( (array) $this->feeds as $feed_name)
@@ -1264,7 +1389,7 @@ class WP_Rewrite {
//build a regex to match the trackback and page/xx parts of URLs
$trackbackregex = 'trackback/?$';
$pageregex = $this->pagination_base . '/?([0-9]{1,})/?$';
- $commentregex = 'comment-page-([0-9]{1,})/?$';
+ $commentregex = $this->comments_pagination_base . '-([0-9]{1,})/?$';
//build up an array of endpoint regexes to append => queries to append
if ( $endpoints ) {
@@ -1290,6 +1415,7 @@ class WP_Rewrite {
$trackbackindex = $index;
//build a list from the rewritecode and queryreplace arrays, that will look something like
//tagname=$matches[i] where i is the current $i
+ $queries = array();
for ( $i = 0; $i < $num_tokens; ++$i ) {
if ( 0 < $i )
$queries[$i] = $queries[$i - 1] . '&';
@@ -1330,7 +1456,7 @@ class WP_Rewrite {
$num_toks = preg_match_all('/%.+?%/', $struct, $toks);
//get the 'tagname=$matches[i]'
- $query = ( isset($queries) && is_array($queries) && !empty($num_toks) ) ? $queries[$num_toks - 1] : '';
+ $query = ( ! empty( $num_toks ) && isset( $queries[$num_toks - 1] ) ) ? $queries[$num_toks - 1] : '';
//set up $ep_mask_specific which is used to match more specific URL types
switch ( $dirs[$j] ) {
@@ -1383,11 +1509,11 @@ class WP_Rewrite {
$rewrite = array_merge($rewrite, array($pagematch => $pagequery));
//only on pages with comments add ../comment-page-xx/
- if ( EP_PAGES & $ep_mask || EP_PERMALINK & $ep_mask )
+ if ( EP_PAGES & $ep_mask || EP_PERMALINK & $ep_mask ) {
$rewrite = array_merge($rewrite, array($commentmatch => $commentquery));
- else if ( EP_ROOT & $ep_mask && get_option('page_on_front') )
+ } elseif ( EP_ROOT & $ep_mask && get_option('page_on_front') ) {
$rewrite = array_merge($rewrite, array($rootcommentmatch => $rootcommentquery));
-
+ }
//do endpoints
if ( $endpoints ) {
foreach ( (array) $ep_query_append as $regex => $ep) {
@@ -1517,10 +1643,10 @@ class WP_Rewrite {
* @access public
*
* @param string $permalink_structure The permalink structure to generate rules.
- * @param bool $walk_dirs Optional, default is false. Whether to create list of directories to walk over.
+ * @param bool $walk_dirs Optional, default is false. Whether to create list of directories to walk over.
* @return array
*/
- function generate_rewrite_rule($permalink_structure, $walk_dirs = false) {
+ public function generate_rewrite_rule($permalink_structure, $walk_dirs = false) {
return $this->generate_rewrite_rules($permalink_structure, EP_NONE, false, false, false, $walk_dirs);
}
@@ -1542,7 +1668,7 @@ class WP_Rewrite {
*
* @return array An associate array of matches and queries.
*/
- function rewrite_rules() {
+ public function rewrite_rules() {
$rewrite = array();
if ( empty($this->permalink_structure) )
@@ -1678,7 +1804,7 @@ class WP_Rewrite {
/**
* Filter rewrite rules used for individual permastructs.
*
- * The dynamic portion of the hook name, $permastructname, refers
+ * The dynamic portion of the hook name, `$permastructname`, refers
* to the name of the registered permastruct, e.g. 'post_tag' (tags),
* 'category' (categories), etc.
*
@@ -1744,7 +1870,7 @@ class WP_Rewrite {
*
* @return array Rewrite rules.
*/
- function wp_rewrite_rules() {
+ public function wp_rewrite_rules() {
$this->rules = get_option('rewrite_rules');
if ( empty($this->rules) ) {
$this->matches = 'matches';
@@ -1769,7 +1895,7 @@ class WP_Rewrite {
*
* @return string
*/
- function mod_rewrite_rules() {
+ public function mod_rewrite_rules() {
if ( ! $this->using_permalinks() )
return '';
@@ -1851,9 +1977,7 @@ class WP_Rewrite {
*
* @param string $rules mod_rewrite Rewrite rules formatted for .htaccess.
*/
- $rules = apply_filters( 'rewrite_rules', $rules ); // Deprecated
-
- return $rules;
+ return apply_filters( 'rewrite_rules', $rules );
}
/**
@@ -1867,8 +1991,7 @@ class WP_Rewrite {
*
* @return string
*/
- function iis7_url_rewrite_rules( $add_parent_tags = false ) {
-
+ public function iis7_url_rewrite_rules( $add_parent_tags = false ) {
if ( ! $this->using_permalinks() )
return '';
$rules = '';
@@ -1904,9 +2027,7 @@ class WP_Rewrite {
*
* @param string $rules Rewrite rules formatted for IIS web.config.
*/
- $rules = apply_filters( 'iis7_url_rewrite_rules', $rules );
-
- return $rules;
+ return apply_filters( 'iis7_url_rewrite_rules', $rules );
}
/**
@@ -1918,13 +2039,13 @@ class WP_Rewrite {
* @since 2.1.0
* @access public
*
- * @param string $regex Regular expression to match against request.
+ * @param string $regex Regular expression to match against request.
* @param string $redirect URL regex redirects to when regex matches request.
- * @param string $after Optional, default is bottom. Location to place rule.
+ * @param string $after Optional, default is bottom. Location to place rule.
*/
- function add_rule($regex, $redirect, $after = 'bottom') {
+ public function add_rule($regex, $redirect, $after = 'bottom') {
//get everything up to the first ?
- $index = (strpos($redirect, '?') == false ? strlen($redirect) : strpos($redirect, '?'));
+ $index = (strpos($redirect, '?') === false ? strlen($redirect) : strpos($redirect, '?'));
$front = substr($redirect, 0, $index);
if ( $front != $this->index ) { //it doesn't redirect to WP's index.php
$this->add_external_rule($regex, $redirect);
@@ -1945,10 +2066,10 @@ class WP_Rewrite {
* @since 2.1.0
* @access public
*
- * @param string $regex Regular expression to match against request.
+ * @param string $regex Regular expression to match against request.
* @param string $redirect URL regex redirects to when regex matches request.
*/
- function add_external_rule($regex, $redirect) {
+ public function add_external_rule($regex, $redirect) {
$this->non_wp_rules[$regex] = $redirect;
}
@@ -1957,22 +2078,31 @@ class WP_Rewrite {
*
* @since 2.1.0
* @since 3.9.0 $query_var parameter added.
+ * @since 4.3.0 Added support for skipping query var registration by passing `false` to `$query_var`.
* @access public
*
* @see add_rewrite_endpoint() for full documentation.
- * @uses WP::add_query_var()
*
- * @param string $name Name of the endpoint.
- * @param int $places Endpoint mask describing the places the endpoint should be added.
- * @param string $query_var Name of the corresponding query variable. Default is value of $name.
+ * @global WP $wp
+ *
+ * @param string $name Name of the endpoint.
+ * @param int $places Endpoint mask describing the places the endpoint should be added.
+ * @param string|bool $query_var Optional. Name of the corresponding query variable. Pass `false` to
+ * skip registering a query_var for this endpoint. Defaults to the
+ * value of `$name`.
*/
- function add_endpoint( $name, $places, $query_var = null ) {
+ public function add_endpoint( $name, $places, $query_var = true ) {
global $wp;
- if ( null === $query_var ) {
+
+ // For backward compatibility, if `null` has explicitly been passed as `$query_var`, assume `true`.
+ if ( true === $query_var || null === func_get_arg( 2 ) ) {
$query_var = $name;
}
$this->endpoints[] = array( $places, $name, $query_var );
- $wp->add_query_var( $query_var );
+
+ if ( $query_var ) {
+ $wp->add_query_var( $query_var );
+ }
}
/**
@@ -1991,9 +2121,9 @@ class WP_Rewrite {
* @since 2.5.0
* @access public
*
- * @param string $name Name for permalink structure.
+ * @param string $name Name for permalink structure.
* @param string $struct Permalink structure (e.g. category/%category%)
- * @param array $args Optional configuration for building the rules from the permalink structure:
+ * @param array $args Optional configuration for building the rules from the permalink structure:
* - with_front (bool) - Should the structure be prepended with WP_Rewrite::$front? Default is true.
* - ep_mask (int) - Endpoint mask defining what endpoints are added to the structure. Default is EP_NONE.
* - paged (bool) - Should archive pagination rules be added for the structure? Default is true.
@@ -2003,7 +2133,7 @@ class WP_Rewrite {
* rules built for each in turn? Default is true.
* - endpoints (bool) - Should endpoints be applied to the generated rewrite rules? Default is true.
*/
- function add_permastruct( $name, $struct, $args = array() ) {
+ public function add_permastruct( $name, $struct, $args = array() ) {
// backwards compatibility for the old parameters: $with_front and $ep_mask
if ( ! is_array( $args ) )
$args = array( 'with_front' => $args );
@@ -2040,9 +2170,26 @@ class WP_Rewrite {
*
* @since 2.0.1
* @access public
+ *
+ * @staticvar bool $do_hard_later
+ *
* @param bool $hard Whether to update .htaccess (hard flush) or just update rewrite_rules option (soft flush). Default is true (hard).
*/
- function flush_rules($hard = true) {
+ public function flush_rules( $hard = true ) {
+ static $do_hard_later = null;
+
+ // Prevent this action from running before everyone has registered their rewrites
+ if ( ! did_action( 'wp_loaded' ) ) {
+ add_action( 'wp_loaded', array( $this, 'flush_rules' ) );
+ $do_hard_later = ( isset( $do_hard_later ) ) ? $do_hard_later || $hard : $hard;
+ return;
+ }
+
+ if ( isset( $do_hard_later ) ) {
+ $hard = $do_hard_later;
+ unset( $do_hard_later );
+ }
+
delete_option('rewrite_rules');
$this->wp_rewrite_rules();
/**
@@ -2073,7 +2220,7 @@ class WP_Rewrite {
* @since 1.5.0
* @access public
*/
- function init() {
+ public function init() {
$this->extra_rules = $this->non_wp_rules = $this->endpoints = array();
$this->permalink_structure = get_option('permalink_structure');
$this->front = substr($this->permalink_structure, 0, strpos($this->permalink_structure, '%'));
@@ -2110,7 +2257,7 @@ class WP_Rewrite {
*
* @param string $permalink_structure Permalink structure.
*/
- function set_permalink_structure($permalink_structure) {
+ public function set_permalink_structure($permalink_structure) {
if ( $permalink_structure != $this->permalink_structure ) {
$old_permalink_structure = $this->permalink_structure;
update_option('permalink_structure', $permalink_structure);
@@ -2140,7 +2287,7 @@ class WP_Rewrite {
*
* @param string $category_base Category permalink structure base.
*/
- function set_category_base($category_base) {
+ public function set_category_base($category_base) {
if ( $category_base != get_option('category_base') ) {
update_option('category_base', $category_base);
$this->init();
@@ -2159,7 +2306,7 @@ class WP_Rewrite {
*
* @param string $tag_base Tag permalink structure base.
*/
- function set_tag_base( $tag_base ) {
+ public function set_tag_base( $tag_base ) {
if ( $tag_base != get_option( 'tag_base') ) {
update_option( 'tag_base', $tag_base );
$this->init();
@@ -2172,9 +2319,8 @@ class WP_Rewrite {
* @since 1.5.0
* @access public
*
- * @return WP_Rewrite
*/
- function __construct() {
+ public function __construct() {
$this->init();
}
}