HomePhorge

Improve some settings-related performance

Description

Improve some settings-related performance

Summary:
Ref T4103. Two small improvements:

  • Don't work as hard to validate translations. We just need to know if a translation exists, we don't need to count how many strings it has and build the entire menu.
  • Allow getUserSetting() to work on any setting without doing all the application/visibility checks. It's OK for code to look at, say, your "Conpherence Notifications" setting even if that application is not installed for you.

Test Plan: Used XHProf and saw 404 page drop from ~60ms to ~40ms locally.

Reviewers: chad

Reviewed By: chad

Maniphest Tasks: T4103

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

Details

Provenance
epriestleyAuthored on Jun 5 2016, 7:47 AM
themackabuPushed on Mar 25 2025, 8:07 PM
Parents
rP7969f66dfe07: Fully modularize the "Quick Actions" menu
Branches
Unknown
Tags
Unknown

Event Timeline