[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