search

vcatalano / py-authorize

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

parse_response #4

Closed cappy123abc closed 10 years ago

cappy123abc commented 10 years ago

In the recursive function parse_response in the response_parser module I find that it only returns the last transaction in the list. It seems as tho' the function is writing over the dict_items variable when it returns from a recursive call. line 82. There doesn't seem to be allowance for multiple child elements that have the same name, such as 'transaction'. I guess an AttrDict needs unique keys?

vcatalano commented 10 years ago

I saw this issue last night. The problem appears to be that the 'transactions' value in the response is an AttrDict instead of a list of transactions. I should get a fix out for that within the next day or two.

vcatalano commented 10 years ago

I've updated the parse_response method to handle the list of transactions.