707914 - Return to store link from Paypal redirects user to error page

Status: RESOLVED DUPLICATE of bug 701777

(addons.mozilla.org Graveyard :: Public Pages, defect)

Description

[Cristian Boldan]

Steps to Reproduce:
1. Login to AMO apps page https://apps-preview-dev.allizom.org/en-US/apps/
2. Click on "Tools -> Manage my apps" link
3. Click on "Manage payments" on a Free app
4. Complete first step and click on "Visit Paypal to grant permision" link
5. On Paypal website, after login, Cancel permision and click on "Return to store" link

Expected Results:
User is redirected to the app Manage Payments page

Actual Results:
User is redirected to an error page and receives an error message "Oops! We had an error."

Notes/Issues:
Screencast for this issue http://screencast.com/t/KL4fgv2MgM

Comment 1

[krupa raj[:krupa]]

  File "/data/www/apps-preview-dev.allizom.org/zamboni/vendor/src/django/django/core/handlers/base.py", line 111, in get_response
    response = callback(request, *callback_args, **callback_kwargs)

  File "/data/www/apps-preview-dev.allizom.org/zamboni/apps/amo/decorators.py", line 102, in wrapper
    return f(*args, **kw)

  File "/data/www/apps-preview-dev.allizom.org/zamboni/apps/amo/decorators.py", line 94, in wrapper
    return f(*args, **kw)

  File "/data/www/apps-preview-dev.allizom.org/zamboni/apps/amo/decorators.py", line 102, in wrapper
    return f(*args, **kw)

  File "/data/www/apps-preview-dev.allizom.org/zamboni/apps/amo/decorators.py", line 94, in wrapper
    return f(*args, **kw)

  File "/data/www/apps-preview-dev.allizom.org/zamboni/apps/addons/decorators.py", line 29, in wrapper
    return f(request, addon, *args, **kw)

  File "/data/www/apps-preview-dev.allizom.org/zamboni/apps/amo/decorators.py", line 28, in wrapper
    return func(request, *args, **kw)

  File "/data/www/apps-preview-dev.allizom.org/zamboni/apps/devhub/decorators.py", line 42, in wrapper
    return fun()

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

  File "/data/www/apps-preview-dev.allizom.org/zamboni/apps/devhub/views.py", line 469, in acquire_refund_permission
    token = paypal.get_permissions_token(request.GET['request_token'],

  File "/data/www/apps-preview-dev.allizom.org/zamboni/vendor/src/django/django/utils/datastructures.py", line 254, in __getitem__
    raise MultiValueDictKeyError("Key %r not found in %r" % (key, self))
MultiValueDictKeyError: "Key 'request_token' not found in <QueryDict: {u'dest': [u'wizard']}>"

Comment 2

[Andy McKay]

Looks like we just need to cope with request_token not being there if people cancel.