[Tickets #7664] Re: fatal error clicking 'file manager' button: no class VFS_

[bugs at horde.org]

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

Ticket URL: http://bugs.horde.org/ticket/7664
------------------------------------------------------------------------------
  Ticket             | 7664
  Updated By         | horde at smartsector.hu
  Summary            | fatal error clicking 'file manager' button: no class
                     | VFS_
  Queue              | Gollem
  Version            | HEAD
  Type               | Bug
  State              | No Feedback
  Priority           | 1. Low
  Milestone          |
  Patch              |
  Owners             | Michael Slusarz
------------------------------------------------------------------------------


horde at smartsector.hu (2009-01-26 09:43) wrote:

I found the problem as i think:

It's an IE bug again.(i test it under IE7). under opera and FF this  
function works well.

I do not follow back the error but find some things:

1. in IE we have a $SESSION['gollem'] array always:
Array
(
     [backends] => Array
         (
         )

     [selectlist] => Array
         (
         )

     [backend_key] =>
)
under opera and ff we have no $SESSION['gollem'] array in this case.

2. This cause the error in the redirect.php:
this if is always true if we go with IE:

if (isset($_SESSION['gollem']) &&
     is_array($_SESSION['gollem']) &&
     ($_SESSION['gollem']['backend_key'] == $backend_key)) {

and this try redirect us to the gollem/manager.php and this cause the error.

3. we set up the SESSION in the lib/base.php in this line:
     $registry = &Registry::singleton();

4. i do not follow back the error more - sorry limited time :-/

my solution is in redirect.php - line 64-66
version:
* $Horde: gollem/redirect.php,v 1.74 2009-01-06 17:50:05 jan Exp $


old one:
if (isset($_SESSION['gollem']) &&
     is_array($_SESSION['gollem']) &&
     ($_SESSION['gollem']['backend_key'] == $backend_key)) {

new one:
if (isset($_SESSION['gollem']) &&
     is_array($_SESSION['gollem']) &&
     ($_SESSION['gollem']['backend_key'] == $backend_key) &&
     !empty($_SESSION['gollem']['backends'])
     ) {

i'm not sure this patch is good in all case.