vcatalano / py-authorize

A full-featured Python API for the Authorize.net payment gateway.
MIT License
42 stars 35 forks source link

Transaction.list issue in 1.2.1.0 #20

Closed clearcode closed 10 years ago

clearcode commented 10 years ago

Hi, we found bug in the new version. In response on request:

Transaction.list(batch_id)

we always get only one transaction, even if should be more.

Responsible for it is function that parsing response, because all transactions are in XML data which come from authorize.

Possible solution: if you change in response_parser.py in LIST_FIELDS "transactions" to "transaction", transactions list will be available under keys:

result = authorize.Transaction.list('3346371') transaction_list = result.transactions['transaction']

but we are not sure the way you want to parse it.

vcatalano commented 10 years ago

I think you're right. I want to make sure that this doesn't affect any other transaction fields that may be part of a request. I will look into this.