HomePhorge

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, 9:25 AM
themackabuPushed on Mar 25 2025, 8:07 PM
Parents
rPa13864179540: Add a 7th column to AphrontMultiColumnView
Branches
Unknown
Tags
Unknown

Event Timeline