Package home | Report new bug | New search | Development Roadmap Status: Open | Feedback | All | Closed Since Version 0.9.3

Bug #15769 setURLFormat method does not return clean query string intermittently
Submitted: 2009-01-25 18:12 UTC
From: mpgjunky Assigned: doconnor
Status: Closed Package: Structures_DataGrid (version 0.9.0)
PHP Version: 5.1.6 OS: Fedora 6
Roadmaps: (Not assigned)    

 [2009-01-25 18:12 UTC] mpgjunky (Michael Mussulis)
Description: ------------ Applies to both outputting the grid and the pager links: setting $datagrid->setUrlFormat('/page/:page/:orderBy/:direction') returns a clean query string like: However, if one clicks on the next page link, the url is re-formatted as: Test script: --------------- Setup a grid with pager links and set url format to whatever you wish. You MUST click on the next page link to obtain the bad result. The default output (i.e. page 1) is always ok. Expected result: ---------------- the url query string should not contain normal style query string, i.e. ?page=[n]&orderBy.... Actual result: --------------


 [2009-01-25 18:19 UTC] mpgjunky (Michael Mussulis)
I have uploaded a patch. Not sure if it's the correct fix, but it works for me. Cheers, Michael.
 [2009-01-25 18:23 UTC] mpgjunky (Michael Mussulis)
PS, using: Pager 2.4.7 stable Structures_DataGrid 0.9.0 beta Structures_DataGrid_DataSource_Array 0.1.4 beta Structures_DataGrid_DataSource_DB 0.1.1 beta Structures_DataGrid_DataSource_DBQuery 0.1.11 beta Structures_DataGrid_DataSource_DBTable 0.1.7 beta Structures_DataGrid_DataSource_MDB2 0.1.11 beta Structures_DataGrid_DataSource_PDO 0.1.0 beta Structures_DataGrid_Renderer_HTMLSortForm 0.1.3 beta Structures_DataGrid_Renderer_HTMLTable 0.1.5 beta Structures_DataGrid_Renderer_Pager 0.1.3 beta
 [2011-12-26 21:06 UTC] doconnor (Daniel O'Connor)
-Status: Open +Status: Closed -Assigned To: +Assigned To: doconnor
This bug has been fixed in SVN. If this was a documentation problem, the fix will appear on by the end of next Sunday (CET). If this was a problem with the website, the change should be live shortly. Otherwise, the fix will appear in the package's next release. Thank you for the report and for helping us make PEAR better.