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

Bug #9992 orderBy=0 is lost in _parseHttpRequest()
Submitted: 2007-01-31 19:27 UTC
From: michael dot mauch at gmx dot de Assigned: wiesemann
Status: Closed Package: Structures_DataGrid (version 0.8.1)
PHP Version: Irrelevant OS: Linux
Roadmaps: (Not assigned)    
Subscription  
Comments Add Comment Add patch


Anyone can comment on a bug. Have a simpler test case? Does it work for you on a different platform? Let us know! Just going to say 'Me too!'? Don't clutter the database with that please !
Your email address:
MUST BE VALID
Solve the problem : 50 - 27 = ?

 
 [2007-01-31 19:27 UTC] michael dot mauch at gmx dot de (Michael Mauch)
Description: ------------ Hi, with DATAGRID_SOURCE_ARRAY, I can't sort for the first (zeroth) column, because orderBy=0 and _parseHttpRequest() has a line if ($orderBy = $this->_getRequestArgument('orderBy')) { instead of if (($orderBy = $this->_getRequestArgument('orderBy')) !== null) { Regards... Michael Test script: --------------- <?php require 'Structures/DataGrid.php'; $values = array(array("A1","Bx1","C1",), array("A2","Bc2","C2",), array("A3","Bz3","C3",), ); $datagrid =& new Structures_DataGrid(); $test = $datagrid->bind($values, null, DATAGRID_SOURCE_ARRAY); if (PEAR::isError($test)) { echo $test->getMessage(); } $test = $datagrid->render(); if (PEAR::isError($test)) { echo $test->getMessage(); } ?> Expected result: ---------------- The sort links should work for all columns. Actual result: -------------- Column 0 cannot be sorted.

Comments

 [2007-02-01 11:56 UTC] User who submitted this comment has not confirmed identity
If you submitted this note, check your email.If you do not have a message, click here to re-send
MANUAL CONFIRMATION IS NOT POSSIBLE.  Write a message to pear-dev@lists.php.net
to request the confirmation link.  All bugs/comments/patches associated with this

email address will be deleted within 48 hours if the account request is not confirmed!