[buddypress-trac] [BuddyPress] #1050: Fix activity filter links

buddypress-trac at lists.automattic.com buddypress-trac at lists.automattic.com
Sun Sep 20 21:43:50 UTC 2009


#1050: Fix activity filter links
----------------------------+-----------------------------------------------
Reporter:  johnjamesjacoby  |       Owner:  johnjamesjacoby
    Type:  defect           |      Status:  new            
Priority:  major            |   Milestone:  1.1            
Keywords:  has-patch        |  
----------------------------+-----------------------------------------------
 Because add_query_arg() uses $_SERVER['REQUEST_URI'] directly and not from
 a passed link or variable, the previous method was removing arguments that
 would get immediately added again inside the loop.

 This method rearranges the order a bit and relies on add_query_arg() to
 return the URI, and also removes the acpage from the filter links.

 Previously acpage would still get passed, causing an unintended jump to
 filtered pages (i.e. - if status updates have 6 pages and groups have 2,
 changing filters causes a page-not-found.)

-- 
Ticket URL: <http://trac.buddypress.org/ticket/1050>
BuddyPress <http://buddypress.org/>
BuddyPress


More information about the buddypress-trac mailing list