krupa raj[:krupa] Reporter
	Description • 13 years ago

steps to reproduce:
1. Pick one of your public apps
2. Change PayPal id without granting refund status as in https://marketplace-dev.allizom.org/en-US/developers/app/seavans-undersea-adventure-3/paypal/
3. Notice that the app is still available in our gallery/search results


expected behavior:
App moves to incomplete/disabled status and is not listed publicly.

observed behavior:
App is listed publicly. User can purchase the app. However, upo requesting refund we get a traceback- PaypalError: Bad refund status for seller_1322765665_per@gmail.com: NOT_PROCESSED

	Christopher Van Wiemeersch [:cvan]
	Comment 1 • 13 years ago

This is probably a matter of modifying the currency checker cronjob for bug 719201.

	Andy McKay
	Comment 2 • 13 years ago

The cronjob checker will catch it, but we shouldn't have to wait for the cronjob to catch it. If an app is public:

- we should put a note saying that doing this will disable the app until the refund permission is set up
- if they continue we should change it to disabled immediately
- when refund status is set up, move it back into public? or back into the review queue?, check highest status maybe?)

	Wil Clouser [:clouserw]
	Comment 3 • 13 years ago

> - when refund status is set up, move it back into public? or back into the
> review queue?, check highest status maybe?)

highest status sounds good so we can put it back where it was

	Wil Clouser [:clouserw]
	Comment 4 • 12 years ago

(In reply to Andy McKay [:andym] from comment #2)
> The cronjob checker will catch it, but we shouldn't have to wait for the
> cronjob to catch it. If an app is public:
> 
> - we should put a note saying that doing this will disable the app until the
> refund permission is set up
> - if they continue we should change it to disabled immediately
> - when refund status is set up, move it back into public? or back into the
> review queue?, check highest status maybe?)

It should move back to highest status.  Also, it doesn't need to be disabled for this, but it should be marked incomplete.