[Tickets #2094] NEW: Choosing Delete action on query builder repeatedly brings up unserialize error

[bugs@bugs.horde.org]

DO NOT REPLY TO THIS MESSAGE. THIS EMAIL ADDRESS IS NOT MONITORED.

Ticket URL: http://bugs.horde.org/ticket/?id=2094
-----------------------------------------------------------------------
 Ticket             | 2094
 Created By         | jhart at webroot.com
 Summary            | Choosing Delete action on query builder repeatedly brings up unserialize error
 Queue              | Whups
 State              | Unconfirmed
 Priority           | 1. Low
 Type               | Bug
 Owners             | 
-----------------------------------------------------------------------


jhart at webroot.com (2005-06-07 12:27) wrote:

Choosing the delete action on the 'current query' repeatedly first brings up
a 'permission denied' error, then going back to the query builder, the
following messages are printed

Notice: unserialize(): Argument is not a string in
/usr/local/horde/whups/query.php on line 28

Fatal error: Call to a member function on a non-object in
/usr/local/horde/whups/query.php on line 29

Looking in the source:

// Find our current query.
if (isset($_SESSION['whups_query'])) {
    $whups_query = unserialize($_SESSION['whups_query']);
    if (!$whups_query->hasPermission(Auth::getAuth(), PERMS_READ)) {

It looks as if along with an isset, the session variable should be checked
to see if it is a unserializable string. 

Logging out (destroying the session) causes this error to go away, and the
query browser is usable again.