[Tickets #7881] Driver.php,v 1.46 - realm and conf.php

[bugs at horde.org]

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

Ticket URL: http://bugs.horde.org/ticket/7881
------------------------------------------------------------------------------
  Ticket             | 7881
  Created By         | horde at smartsector.hu
  Summary            | Driver.php,v 1.46 - realm and conf.php
  Queue              | Vacation
  Version            | 3.1-RC1
  Type               | Bug
  State              | Unconfirmed
  Priority           | 2. Medium
  Milestone          |
  Patch              |
  Owners             |
------------------------------------------------------------------------------


horde at smartsector.hu (2009-01-23 12:24) wrote:

There si a little bug when you do not set up any realm and use full  
user names like:
info at example.com

you must set up the $params correctly before you check the hordeauth.
here is a patch, just need change the lines:

The original one:
  function Vacation_Driver($user, $realm, $params = array())
     {
         $this->_params = $params;

         // Check if hordeauth is set to 'full'
         $hordeauth = $this->getParam('hordeauth');
         if ($hordeauth !== 'full') {
             @list($user,) = explode('@', $user, 2);
         }
         $this->_user = $user;

         if (!isset($params[$realm])) {
             $realm = 'default';
         }
         $this->_realm = $realm;
     }

The fixed one:
  function Vacation_Driver($user, $realm, $params = array())
     {
         $this->_params = $params;

         if (!isset($params[$realm])) {
             $realm = 'default';
         }
         $this->_realm = $realm;

         // Check if hordeauth is set to 'full'
         $hordeauth = $this->getParam('hordeauth');
         if ($hordeauth !== 'full') {
             @list($user,) = explode('@', $user, 2);
         }
         $this->_user = $user;
      }