Tickets :: [#8680] quota hook parameters don't get passed to the hook function

6.0.0-beta6

4/1/26

Summary	quota hook parameters don't get passed to the hook function
Queue	IMP
Queue Version	Git master
Type	Bug
State	Resolved
Priority	1. Low
Owners
Requester	Valentin.Vidic (at) carnet (dot) hr
Created	11/6/09 (5990 days ago)
Due
Updated	1/13/10 (5922 days ago)
Assigned
Resolved	11/6/09 (5990 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch	Yes

01/13/2010 12:15:03 AM	CVS Commit	Comment #5	Reply to this comment
Changes have been made in Git for this ticket: ~~Ticket #8680~~: fix hook call; also, fix hook name Signed-off-by: Michael M Slusarz <slusarz@curecanti.org> http://git.horde.org/diff.php/imp/lib/Quota/Hook.php?rt=horde-git&r1=d58245fa6817cbbf97fc5169615794d0232ff08f&r2=3c5edf58a21d21ce988d95fcf29c5a6c964e570b

11/06/2009 10:23:00 PM	Michael Slusarz	Comment #4 State ⇒ Resolved Version ⇒ Git master	Reply to this comment
Fixed in 4.3.6.

11/06/2009 10:22:27 PM	CVS Commit	Comment #3	Reply to this comment
Changes have been made in Git for this ticket: ~~Ticket #8680~~: fix hook call; also, fix hook name Signed-off-by: Michael M Slusarz <slusarz@curecanti.org> http://git.horde.org/diff.php/imp/lib/Quota/Hook.php?rt=horde-hatchery&r1=d58245fa6817cbbf97fc5169615794d0232ff08f&r2=3c5edf58a21d21ce988d95fcf29c5a6c964e570b

11/06/2009 10:22:18 PM	CVS Commit	Comment #2	Reply to this comment
Changes have been made in CVS for this ticket: ~~Bug: 8680~~ Submitted by: Valentin.Vidic@CARNet.hr Fix hook call. http://cvs.horde.org/diff.php/imp/lib/Quota/hook.php?rt=horde&r1=1.1.2.4&r2=1.1.2.5&ty=u

11/06/2009 10:15:07 AM	Valentin (dot) Vidic (at) carnet (dot) hr	Comment #1 Priority ⇒ 1. Low New Attachment: quota_hook.diff Patch ⇒ Yes Milestone ⇒ Summary ⇒ quota hook parameters don't get passed to the hook function Type ⇒ Bug State ⇒ Unconfirmed Queue ⇒ IMP	Reply to this comment
Trying to write a custom quota hook function I realized that quota plugin parameters set in imp/config/servers.php don't get passed properly to the hook function. It turns out an array() wrap was missing in hook call for this to work. Patch attached.