[Tickets #961] NEW: hook_spam_bounce does not work.

[bugs at bugs.horde.org]

DO NOT REPLY TO THIS MESSAGE. THIS EMAIL ADDRESS IS NOT MONITORED.

Ticket URL: http://bugs.horde.org/ticket/?id=961
-----------------------------------------------------------------------
 Ticket     | 961
 Created By | kurt.dillen at dls-belgium.com
 Summary    | hook_spam_bounce does not work.
 Queue      | IMP
 Version    | 4.0-RC3
 State      | Unconfirmed
 Priority   | 3. High
 Type       | Bug
 Owners     | 
-----------------------------------------------------------------------


kurt.dillen at dls-belgium.com (2004-12-09 04:47) wrote:

I have configured imp to use with the spam_bounce hook for dspam after
installing RC3 this did not work anymore.  After a compare I found the
problem:  

Code from RC2:  

            /* If a (not)spam bounce email address has been provided, use
             * it. */
            if (!empty($GLOBALS['conf'][$action]['bounce'])) {
                $to = $GLOBALS['conf'][$action]['bounce'];

                /* Call the bounce email generation hook, if requested. */
                if (!empty($GLOBALS['conf']['hooks']['spam_bounce'])) {
                    require_once HORDE_BASE . '/config/hooks.php';
                    if (function_exists('_imp_hook_spam_bounce')) {
                        $to = call_user_func('_imp_hook_spam_bounce',
$action);
                    }
                }

Code from RC3 Not Working!!!:

            /* If a (not)spam bounce email address has been provided, use
             * it. */
            if (!empty($GLOBALS['conf'][$action]['bounce'])) {
                $to = $GLOBALS['conf'][$action]['bounce'];
            } elseif (!empty($GLOBALS['conf']['hooks']['spam_bounce'])) {
                /* Call the bounce email generation hook, if requested. */
                require_once HORDE_BASE . '/config/hooks.php';
                if (function_exists('_imp_hook_spam_bounce')) {
                    $to = call_user_func('_imp_hook_spam_bounce', $action);
                }
            }

Please solve this because I really like the feature.