[dev] Commit: 7e1fbe - Cache config after merge breaks loading sql config

Michael M Slusarz slusarz at horde.org
Mon Feb 3 04:00:00 UTC 2014


Quoting Michael M Slusarz <slusarz at horde.org>:

> Quoting Michael J Rubinsky <mrubinsk at horde.org>:
>
>> The following commit:
>>
>> https://github.com/horde/horde/commit/7e1fbe7c56c0fc6a677b49dd5b9912d2f6519a47#diff-756457dc85f13450b3dfba2cbc1465e5
>>
>> Causes other application's sql configuration to be leaked into  
>> other applications on pages, like the portal, where multiple  
>> applications are pushed/popped. E.g., The Nag Summary block  
>> attempts to query Wicked's tables in my install.
>
> Sounds like a preexisting bug that the recent changes have  
> triggered.  My guess is that the Sql factory is not properly  
> accessing the config array.  I will take a look.

Or never mind.  The importConfig() method wasn't setting the global  
$conf variable.  Whoops.

michael

___________________________________
Michael Slusarz [slusarz at horde.org]



More information about the dev mailing list