6.0.0-beta1
7/6/25

[#14605] Horde not loading vhosts confs
Summary Horde not loading vhosts confs
Queue Horde Framework Packages
Type Bug
State Resolved
Priority 2. Medium
Owners Horde Developers (at) , mrubinsk (at) horde (dot) org
Requester boutilpj (at) ednet (dot) ns (dot) ca
Created 03/31/2017 (3019 days ago)
Due
Updated 04/01/2017 (3018 days ago)
Assigned 03/31/2017 (3019 days ago)
Resolved 04/01/2017 (3018 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch Yes

History
04/01/2017 02:53:21 AM Michael Rubinsky Assigned to Michael Rubinsky
State ⇒ Resolved
 
04/01/2017 02:45:24 AM Git Commit Comment #4 Reply to this comment
Changes have been made in Git (master):

commit b34fd24db43c266a4d1b7fadadc0308048a566c1
Author: Michael J Rubinsky <mrubinsk@horde.org>
Date:   Fri Mar 31 22:43:49 2017 -0400

     Bug: 14605 Fix regression in loading vhost files.

  framework/Core/lib/Horde/Registry/Loadconfig.php | 5 +++--
  1 file changed, 3 insertions(+), 2 deletions(-)

http://github.com/horde/horde/commit/b34fd24db43c266a4d1b7fadadc0308048a566c1
03/31/2017 03:59:30 PM Michael Rubinsky Comment #3
Assigned to Horde DevelopersHorde Developers
State ⇒ Assigned
Priority ⇒ 2. Medium
Reply to this comment
The problem is that foreach() operates on a copy of the array, so it 
can't be modified within the loop. However, the solution isn't to 
revert back to the deprecated php language construct. It needs to be 
changed to something like:

$k =0;
for ($v = reset($flist); $v; $v = next($flist)) {
.
   if ($k++ == $end) {
       .
   }

}

I can fix this when I'm back in front of my dev box late 
tonight/tomorrow if no one else beats me to it.
03/31/2017 03:06:29 PM boutilpj (at) ednet (dot) ns (dot) ca Comment #2
New Attachment: Loadconfig.php.patch Download
Reply to this comment
Patch

03/31/2017 03:04:12 PM boutilpj (at) ednet (dot) ns (dot) ca Comment #1
Priority ⇒ 3. High
Type ⇒ Bug
Summary ⇒ Horde not loading vhosts confs
Queue ⇒ Horde Framework Packages
Milestone ⇒
Patch ⇒ Yes
State ⇒ Unconfirmed
Reply to this comment
Since Horde_Core-2.27.7 other than config/registry-vhost.php, no other 
vhosts files are read. This can be traced back to this commit.

https://github.com/horde/horde/commit/d92682fda55e0ed156f92f17bd97a30bf84549ce#diff-f9a576be4845e5fd565c524a2133f4a6

Saved Queries