Making a function null?

[microguy]

This is my first attempt at modifying code, and I need a little help.

Our billing system currently send notices to subscribers upon each successful rebill. We want to turn this feature off. I have found the function in the code that performs this action. There are two files involved. One that processes the credit card and calls the function and the other that contains the sub routine.

cc_core.inc.php

case CC_RESULT_SUCCESS:
            $err = $db->finish_waiting_payment(
                $payment['payment_id'], $payment['paysys_id'], 
                    $receipt_id, $payment['amount'], '', cc_core_get_payer_id($vars, $member));
            if ($err) {
                $db->log_error($err . ": payment_id = $payment[payment_id] (rebilling)");
            }
            mail_rebill_success_member($member, $payment_id, $product);
        break;
        case CC_RESULT_INTERNAL_ERROR:
        case CC_RESULT_DECLIME_TEMP:
            if ($pl->config['reattempt'] != '')
                $new_expire = cc_core_prorate_subscription($p['payment_id'], $pl->config['reattempt'], $dat);
            mail_rebill_failed_member($member, $payment_id, $product, "$err_msg", $new_expire);
        break;

common.inc.php

function mail_rebill_success_member($user, $payment_id, $product){
    global $db;
    $t = &new_smarty();
    $t->assign('user',    $user);
    $t->assign('payment', $db->get_payment($payment_id));
    $t->assign('product', $product->config);
    ///
    $et = & new aMemberEmailTemplate();
    $et->name = "cc_rebill_success";
    mail_template_user($t, $et, $user);
}

I don't want to send customer email after rebill so how do I null mail_rebill_success_member?

[Kudose]

I think that function also generates a template for the user. I would look deeper (into the mail_template_user function) and comment (with either a # or //) out the actual call to mail().

[stolzyboy]

what is included in the mail_template_user function, is there a mail() call there?

[microguy]

I tried commenting out mail_rebill_success_member and it appears that the script dies after attempting one rebill. Is it ok to post both complete files here?

[stolzyboy]

you need to figure out where the mail() call is, look in the mail_template_user function for starters...

[microguy]

Here is that code.

function mail_template_user($t, $et, $u, $add_unsubscribe=false){
    global $db, $config, $_AMEMBER_TEMPLATE;
    $t->assign('config', $config);
    $et->lang = $u['data']['selected_lang'] ? $u['data']['selected_lang'] : get_default_lang();
    // load and find templated
    if (!$et->find_applicable()){
        trigger_error("Cannot find applicable e-mail template for [{$et->name},{$et->lang},{$et->product_id},{$et->day}]", E_USER_WARNING);
        return false;
    }
    $_AMEMBER_TEMPLATE['text'] = $et->get_smarty_template();
    $parsed_mail = $t->fetch('memory:text');
    unset($_AMEMBER_TEMPLATE['text']);
    mail_customer($u['email'], $parsed_mail,
    null, null, null, $add_unsubscribe,
    $u['name_f'] . ' ' . $u['name_l']);

////////////////////////////////////////////////////////
// The following line was added below to enable sending 
// admin carbon copies of all emails sent to subscribers
// The instructions for this modification can be located 
// at http://amember.com/forum/showthread.php?t=6979
///////////////////////////////////////////////////////

mail_admin($parsed_mail,null);
}

[stolzyboy]

you need to just keep following function trails... i'd look for mail_customer() next

have you tried commenting out the mail_template_user() function that is inside the mail_rebill_success_member() function?

[leatherback]

stolzyboy wrote:

you need to just keep following function trails... i'd look for mail_customer() next

have you tried commenting out the mail_template_user() function that is inside the mail_rebill_success_member() function?

Yes. And / or try what happens when you comment:

mail_customer($u['email'], $parsed_mail,
null, null, null, $add_unsubscribe,
$u['name_f'] . ' ' . $u['name_l']);

Which seems to just sent the email, if the naming is usefull.,.

[microguy]

I don't want to turn off all emails, just the ones associated with rebills.

case CC_RESULT_SUCCESS:
            $err = $db->finish_waiting_payment(
                $payment['payment_id'], $payment['paysys_id'], 
                    $receipt_id, $payment['amount'], '', cc_core_get_payer_id($vars, $member));
            if ($err) {
                $db->log_error($err . ": payment_id = $payment[payment_id] (rebilling)");
            }
             // Turned off Rebill Emails
             // mail_rebill_success_member($member, $payment_id, $product);
        break;
        case CC_RESULT_INTERNAL_ERROR:
        case CC_RESULT_DECLIME_TEMP:
            if ($pl->config['reattempt'] != '')
                $new_expire = cc_core_prorate_subscription($p['payment_id'], $pl->config['reattempt'], $dat);
             // Turned off Failed Emails
             // mail_rebill_failed_member($member, $payment_id, $product, "$err_msg", $new_expire);
        break;
        case CC_RESULT_DECLINE_PERM:
            if ($pl->config['reattempt'] != '')
                $new_expire = cc_core_prorate_subscription($p['payment_id'], $pl->config['reattempt'], $dat);
            // Turned off Failed Emails
            // mail_rebill_failed_member($member, $payment_id, $product, "$err_msg", $new_expire);
//            clean_cc_info($member);
        break;
		case CC_RESULT_IGNORE:
		break;
        default:
            $db->log_error("Unknown return from plugin_bill: $res");
        };

This is what I've got at the moment. But since rebills only occur once per day at midnight, its hard to test.

[microguy]

I advanced the system clock 24 hours and ran a cron. The code in my last post appears to be working fine. Thanks everyone.

[laserlight]

Remember to mark this thread as resolved using the thread tools.