6.0.0-RC7
6/21/26

[#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 3/31/17 (3369 days ago)
Due
Updated 4/1/17 (3368 days ago)
Assigned 3/31/17 (3369 days ago)
Resolved 4/1/17 (3368 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch Yes

History
212 Michael Rubinsky Assigned to Michael Rubinsky
State ⇒ Resolved
 
242 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
303 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.
293 boutilpj (at) ednet (dot) ns (dot) ca Comment #2
New Attachment: Loadconfig.php.patch Download
Reply to this comment
Patch

123 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