From: kink <kink@7612ce4b-ef26-0410-bec9-ea0150e637f0>
Date: Sun, 10 Dec 2006 19:37:11 +0000 (+0000)
Subject: must pass $args by ref to to plugin function too, if we want the plugin
X-Git-Url: https://vcs.fsf.org/?a=commitdiff_plain;h=141105fcf352bbce695f83127ae0f9bedd3e68a3;p=squirrelmail.git

must pass $args by ref to to plugin function too, if we want the plugin
to be able to modify it


git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@11992 7612ce4b-ef26-0410-bec9-ea0150e637f0
---

diff --git a/functions/plugin.php b/functions/plugin.php
index c4df4a30..20ccf4c3 100644
--- a/functions/plugin.php
+++ b/functions/plugin.php
@@ -80,7 +80,7 @@ function do_hook($name, &$args) {
         foreach ($squirrelmail_plugin_hooks[$name] as $plugin_name => $function) {
             use_plugin($plugin_name);
             if (function_exists($function)) {
-                $ret = $function($args, $ret);
+                $ret = $function(&$args, $ret);
             }
         }
     }
@@ -219,11 +219,12 @@ function boolean_hook_function($name, &$args, $priority=0, $tie=false) {
 }
 
 /**
+ * Do not use, use checkForJavascript() instead.
+ *
  * This function checks whether the user's USER_AGENT is known to
  * be broken. If so, returns true and the plugin is invisible to the
  * offending browser.
  * *** THIS IS A TEST FOR JAVASCRIPT SUPPORT ***
- * FIXME: This function needs to have its name changed!
  *
  * @return bool whether this browser properly supports JavaScript
  * @deprecated use checkForJavascript() since 1.5.1