From 76142d50cd022cfb72a4ee5110b73bea9a8ca0bb Mon Sep 17 00:00:00 2001 From: shlurbee Date: Tue, 29 May 2012 10:06:36 -0700 Subject: [PATCH] Remove user id from authorize.net invoice number Previously, invoice number was [user_id]T[link_id]C[campaign_id]. Since campaigns are represented as Things now, their ids are longer, putting us over the 20 char limit for invoice numbers defined in the authorize.net xml api. (see https://api.authorize.net/xml/v1/schema/AnetApiSchema.xsd ) Now invoice number will be T[link_id]C[campaign_id], which is still unique across campaigns since a link is always associated with only one user --- r2/r2/lib/authorize/interaction.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/r2/r2/lib/authorize/interaction.py b/r2/r2/lib/authorize/interaction.py index 74cdc9746..d38e6031c 100644 --- a/r2/r2/lib/authorize/interaction.py +++ b/r2/r2/lib/authorize/interaction.py @@ -116,7 +116,7 @@ def auth_transaction(amount, user, payid, thing, campaign, test = None): return bid.transaction, "" elif int(payid) in PayID.get_ids(user): - order = Order(invoiceNumber = "%dT%dC%d" % (user._id, thing._id, campaign)) + order = Order(invoiceNumber = "T%dC%d" % (thing._id, campaign)) success, res = _make_transaction(ProfileTransAuthOnly, amount, user, payid, order = order, test = test)