Home GnuPG

Give AphrontTagView a `getViewer()`, deprecate `getUser()`
aaab1011e5a4Unpublished

Unpublished Commit ยท Learn More

Repository Importing: This repository is still importing.

Description

Give AphrontTagView a getViewer(), deprecate getUser()

Summary:
Two minor changes here:

  • Replace get/setUser() with get/setViewer() for consistency with everything else.
  • getViewer() now throws if no viewer is set. We had a lot of code that either "should" check this but didn't, or did check it in an identical way, duplicating work. In contrast, very little code checks for a viewer but works if one is not present.

Test Plan:

  • Grepped for ->user.
  • Attempted to fix all callsites inside *View classes.
  • Browsed around a bunch of applications, particularly Calendar, Differential and Diffusion, which seemed most heavily affected.

Reviewers: chad

Reviewed By: chad

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

Details

Provenance
epriestley <git@epriestley.com>Authored on Mar 6 2016, 3:26 PM
Parents
rPHABabb4c03b476d: Remove shouldShowSubscribersProperty() from SubscribableInterface
Branches
Unknown
Tags
Unknown

Event Timeline

epriestley <git@epriestley.com> committed rPHABaaab1011e5a4: Give AphrontTagView a `getViewer()`, deprecate `getUser()` (authored by epriestley <git@epriestley.com>).Mar 6 2016, 6:27 PM