[Tickets #13111] hooks not working due to loacConfiguration() flaw

[noreply at bugs.horde.org]

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

Ticket URL: http://bugs.horde.org/ticket/13111
------------------------------------------------------------------------------
  Ticket             | 13111
  Created By         | maciej.uhlig at us.edu.pl
  Summary            | hooks not working due to loacConfiguration() flaw
  Queue              | Horde Base
  Version            | FRAMEWORK_5_1
  Type               | Bug
  State              | Unconfirmed
  Priority           | 3. High
  Milestone          |
  Patch              | 1
  Owners             |
------------------------------------------------------------------------------


maciej.uhlig at us.edu.pl (2014-04-12 12:40) wrote:

Trying to use Horde hooks funcionality I encountered problems due to  
impossibility of loading configuration file containing Horde_Hooks  
class. xdebug shown exception in line 372 of Horde.php  
loadConfiguration():

     0.0351    1621528                           -> sprintf('Failed to  
import configuration file "%s": ',  
'/data/var/www/webmail_test/lib/../config/hooks.php')  
/usr/share/pear/Horde.php:372
                                                  >=> 'Failed to  
import configuration file  
"/data/var/www/webmail_test/lib/../config/hooks.php": '
     0.0352    1621720                           -> strip_tags('\t\t')  
/usr/share/pear/Horde.php:372
                                                  >=> '\t\t'
     0.0352    1621576                           ->  
Exception->__construct('Failed to import configuration file  
"/data/var/www/webmail_test/lib/../config/hooks.php": \t\t')  
/usr/share/pear/Horde.php:372

Please note in this case $output is expected to be '' however it's  
equal to '\t\t' (God knows why) and line 363 condition is evaluated  
while it shouldn't be.

The simple solution is to add after line 357 line:

$output = trim($output);

This fix removes the problem.