Homestyx hydra
Diffusion hydra 97ff7fe259d4

Make "isInstalled()" respect beta apps

Description

Make "isInstalled()" respect beta apps

Summary:
Currently, isInstalled() and getAllInstalledApplications() are inconsistent:

  • isInstalled() returns true for beta apps, even if phabricator.show-beta-applications is false.
  • getAllInstalledApplications() omits beta apps if phabricator.show-beta-applications is false.

Making the beta config control installs (not just homepage visibility) makes far more sense as we roll out more thorough application integrations.

Make isInstalled() respect beta, and clean up some callsites.

D5602 builds on this.

Test Plan: Installed/uninstalled beta apps, verified Conpherence menu/panel and other application integrations dropped out of the UI.

Reviewers: vrana, btrahan

Reviewed By: vrana

CC: aran

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

Details

Provenance
epriestleyAuthored on Apr 6 2013, 12:25 PM
sirocylPushed on Oct 16 2024, 5:49 AM
Parents
R1:a13864179540: Add a 7th column to AphrontMultiColumnView
Branches
Unknown
Tags
Unknown

Event Timeline