Homec4science

Fix bad Phortune Subscriptions query

Authored by epriestley <git@epriestley.com> on Feb 16 2015, 20:17.

Description

Fix bad Phortune Subscriptions query

Summary:
Fixes T7285. If the user tries to view a subscription they don't have permission to view, we may filter all the subscriptions out, then still try to load related data. This can fatal because it's invalid.

Instead, bail if we filtered everything.

Test Plan: Subscritption detail page of another user's subscription is now 404 instead of fatal.

Reviewers: btrahan, chad

Reviewed By: chad

Subscribers: epriestley

Maniphest Tasks: T7285

Differential Revision: https://secure.phabricator.com/D11780

Details

Committed
epriestley <git@epriestley.com>Feb 16 2015, 20:17
Pushed
aubortJan 31 2017, 17:16
Parents
rPHf206da2dbf3d: Increase height of message box on invite workflow
Branches
Unknown
Tags
Unknown

Event Timeline

epriestley <git@epriestley.com> committed rPH5a9d70707b9e: Fix bad Phortune Subscriptions query (authored by epriestley <git@epriestley.com>).Feb 16 2015, 20:17