uc_authorizenet should include phone field

If you are using Auth.net CIM and have configured this in ubercart then this shouldn't happen.

As noted on the project page we still don't have full Auth.net ARB support due to recurring code in the core uc_authorizenet module and we need people like yourself who are using this gateway to push the ubercart guys to commit the patch I have supplied to fix this issue: #580938: Create Authorize.net ARB recurring fee handler

The ubercart issue: #592420: Move UC Recurring Fees code out of Authorize.Net module

I have no interest in this gateway myself so its not something I am willing to spend much time on, I have done the work, someone else need to take this issue up with the ubercart guys and convince them to commit it.

Note: This may or may not be related, but that can occur if the total is $0.00. In that case, it may actually be sending to Authorize.net, but Authorize.net rejects it as an API schema error (as if the XML were invalid). There is no admin comment or log comment to explain what happened, and I think that there would be no record of it on the Authorize.net side either (not in the merchant interface, anyway).

Related: http://drupal.org/node/691884 and http://drupal.org/node/691926

I looked into this and it turns out that a missing phone number in the authorize.net payment profile is the problem. Once that is filled in the payment goes through. But why would the phone be required for the recurring payment but not the initial one? Weird...

Here's the fix. You've got to add the phone field to the CIM account creation. The file is modules\ubercart\payment\uc_authorizenet\uc_authorizenet.module around line 1106.
Make sure the fields go in this order:

function _uc_authorize_cim_xml_billto($order) {
  $billing_country = uc_get_country_data(array('country_id' => $order->billing_country));
  return array(
    'firstName' => substr($order->billing_first_name, 0, 50),
    'lastName' => substr($order->billing_last_name, 0, 50),
    'company' => substr($order->billing_company, 0, 50),
    'address' => substr($order->billing_street1, 0, 60),
    'city' => substr($order->billing_city, 0, 40),
    'state' => substr(uc_get_zone_code($order->billing_zone), 0, 2),
    'zip' => substr($order->billing_postal_code, 0, 20),
    'country' => !$billing_country ? '' : $billing_country[0]['country_iso_code_2'],
    'phoneNumber' => substr($order->billing_phone, 0, 25)	
  );
}

If this is the issue then this is a problem that needs to be fixed in the uc_authorizenet module in ubercart.

@flamingvan: Is this documented in the Authorize.net protocol? Can you roll a patch for this fix? Also, if the phone number *must* be sent for the transaction to succeed, then you should add a hook_form_alter() to uc_authorizenet in order to make the phone number field mandatory on the checkout page.

Sorry for my slow reply. Would you still like me to roll a patch for this?

Yes please.

Here's a patch. Sorry this took so long for me to do. I have no way of testing it. I don't have an authorize.net account. I was using my client's account and their site is in production so I can't mess with it. Can anyone test this?

I didn't make the phone number field mandatory. I don't think it needs to be filled out for this to work.

Committed to both branches.

As I said in #8, if phone field is required by Authorize.net in order for transactions to succeed, then this payment gateway should alter the checkout form to ensure that data is collected.

If the phone field is *not* required by Authorize.net, then I don't see how this patch fixes the original issue.

No one who has experienced the problem has tested the patch, so it's not clear to me that it fixes anything.

At least we are sending the phone number now if one was supplied, which is an improvement from before. If this is still an issue the community can reopen it and describe any remaining problems.