[wp-trac] Re: [WordPress Trac] #8098: trailing p's removed from query by canonical redirect

WordPress Trac wp-trac at lists.automattic.com
Sun Nov 9 22:04:21 GMT 2008


#8098: trailing p's removed from query by canonical redirect
---------------------+------------------------------------------------------
 Reporter:  DD32     |        Owner:  anonymous
     Type:  defect   |       Status:  new      
 Priority:  normal   |    Milestone:  2.7      
Component:  General  |      Version:  2.7      
 Severity:  normal   |   Resolution:           
 Keywords:           |  
---------------------+------------------------------------------------------
Comment (by stevish):

 Here's one that will grab an empty variable at the beginning, end, or
 middle. I did a lot of testing, and couldn't fool it. Maybe someone else
 can see a problem with it?

 {{{'#(^|&)(p|page_id|cat|tag)=?$|(p|page_id|cat|tag)=&#'}}}

 Notice that when looking for an empty declaration in the beginning or
 middle (without requiring an "end of string"), it requires a '=&' at the
 end, without checking for one at the beginning. This will make sure we
 only remove one '&' per declaration, and conceivably, will never touch any
 set of letters directly following an '='.

 Any objections?

-- 
Ticket URL: <http://trac.wordpress.org/ticket/8098#comment:15>
WordPress Trac <http://trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list