search

spookylukey / django-paypal

A pluggable Django application for integrating PayPal Payments Standard or Payments Pro
MIT License
729 stars 208 forks source link

does not support option_selectionX variable #129

Closed nikulk closed 8 years ago

nikulk commented 8 years ago

(Not to be confused with the Mass Pay variables option_selectionX_Y)

These are the selections corresponding to option_nameX which you already handle. I think these variables were just missed for some reason.

spookylukey commented 8 years ago

They are supported, like all 'fooX' options (i.e. fields with variable number of items), via posted_data_dict.

The option_name1 and option_name2 fields ought to be removed, and the documentation for IPN/PDT needs updating regarding this.

spookylukey commented 8 years ago

Sorry, these fields ought to be supported, since only 2 are listed, rather than a variable number like X. I'll add them shortly.

spookylukey commented 8 years ago

I've fixed this now on the master branch.