686699 - Paypal 3rd party permission error

Status: VERIFIED FIXED

(addons.mozilla.org Graveyard :: Developer Pages, defect, P2)

Description

[Andy McKay]

When fligtar hit:

https://addons-dev.allizom.org/z/en-US/developers/addon/password-exporter-pro/payments/permission/refund...

He got a traceback:

 File "/data/www/addons-dev.allizom.org/zamboni/apps/devhub/decorators.py", line 24, in <lambda>
   **kw)

 File "/data/www/addons-dev.allizom.org/zamboni/apps/devhub/views.py", line 492, in acquire_refund_permission
   paypal_permissions_token=token)

[...]

IntegrityError: (1062, "Duplicate entry '335400' for key 'addon_id'")

Comment 1

[Andy McKay]

Tentative fix in. 

https://github.com/jbalogh/zamboni/commit/572a6e

Comment 2

[krupa raj[:krupa]]

reopening since this still happens.

Comment 3

[Andy McKay]

https://github.com/jbalogh/zamboni/commit/a0e4ae

I feel dirty.

Comment 4

[krupa raj[:krupa]]

Just this traceback again while setting up payments for an app. Reopening.

Comment 5

[krupa raj[:krupa]]

STR:
1. Enroll an app/add-on into Marketplace. Set up the refund token for the add-on using a PayPal account. Click "return to Wil Clouser's test store"
2. Complete the enrollment.
3. Start enrolling a different/new app/add-on.
4. Use the same PayPal account to set up the refund token for this second add-on.
5. Click "return to Wil Clouser's test store"

observed behavior:
traceback

Comment 6

[Wil Clouser [:clouserw]]

 File "/data/www/addons-dev.allizom.org/zamboni/vendor/src/django/django/db/backends/mysql/base.py", line 86, in execute
   return self.cursor.execute(query, args)

 File "/usr/lib64/python2.6/site-packages/MySQLdb/cursors.py", line 173, in execute
   self.errorhandler(self, exc, value)

 File "/usr/lib64/python2.6/site-packages/MySQLdb/connections.py", line 36, in defaulterrorhandler
   raise errorclass, errorvalue

IntegrityError: (1062, "Duplicate entry '337557' for key 'addon_id'")


-------------------------------------

That one?  If not, please paste it in

Comment 7

[Andy McKay]

https://github.com/mozilla/zamboni/commit/fe2dd7

Comment 8

[krupa raj[:krupa]]

I haven't seen this traceback in a while. Marking this as verified at https://addons-dev.allizom.org/en-US/developers/addon/app-345078/payments