[wp-trac] [WordPress Trac] #3308: add_query_arg() removes blank query vars

WordPress Trac wp-trac at lists.automattic.com
Tue Oct 31 05:45:33 GMT 2006


#3308: add_query_arg() removes blank query vars
-------------------------+--------------------------------------------------
 Reporter:  markjaquith  |       Owner:  anonymous
     Type:  defect       |      Status:  new      
 Priority:  normal       |   Milestone:  2.1      
Component:  General      |     Version:  2.1      
 Severity:  normal       |    Keywords:           
-------------------------+--------------------------------------------------
 {{{
 skeltoac: can anybody explain why add_query_arg removes blank query vars?
 }}}

 This results in a destructive query var addition, in which existing (but
 blank) args disappear.

 {{{remove_query_arg()}}} was removing by passing a blank string.  Upcoming
 patch moves that to FALSE, which allows blank query vars to be added.

 It also preserves {{{?foo&bar=test}}} vs {{{?foo=&bar=test}}}

 In all, it just preserves the existing QS a whole lot better.

-- 
Ticket URL: <http://trac.wordpress.org/ticket/3308>
WordPress Trac <http://trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list