[wp-trac] [WordPress Trac] #16687: %postname% permalinks should not trigger verbose rewrite rules
WordPress Trac
wp-trac at lists.automattic.com
Thu Jul 14 18:33:37 UTC 2011
#16687: %postname% permalinks should not trigger verbose rewrite rules
-----------------------------+------------------
Reporter: nacin | Owner:
Type: feature request | Status: new
Priority: normal | Milestone: 3.3
Component: Rewrite Rules | Version:
Severity: normal | Resolution:
Keywords: |
-----------------------------+------------------
Comment (by Otto42):
Basically, this patch makes page rules not really "verbose" anymore, and
so the verbose flag is now just used to determine if pages should take
precedence or not and whether extra queries need to be done for that case.
So also be sure to check for unexpected side effects. Make sure that Pages
and Attachments all show up properly and aren't throwing 404's or errors
or what have you.
Also be aware than in the event of a URL conflict (such as if you use
%postname% for the custom string and a page and post both have the same
slug), then the page is going to take precedence and get shown instead of
the post. Me and Nacin talked about that Monday and decided that that made
the most sense. So be sure that works properly too on your sites.
With the current patch, it's doing unnecessary extra data retrieval and
then discarding it. This can be optimized away, although I don't think it
would result in noticeable speed improvement. One thing that would be
interesting to see is the results on long hierarchical page URLs. The
speed of the lookup is dependent on the depth of the URL. So a page with
/page would be found faster than a page with
/page/url/is/really/really/deep/here/oh/noes. It might be interesting to
see how deep you need to go before it becomes a problem.
--
Ticket URL: <http://core.trac.wordpress.org/ticket/16687#comment:32>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software
More information about the wp-trac
mailing list