Homec4science

Check if app is installed for user before displying

Authored by Josh Cox <joshcox@uberatc.com> on Sep 8 2016, 03:49.

Description

Check if app is installed for user before displying

Summary: Fixes T11595. Previously if a user didn't have permissions to view an application it would still appear in the application typeahead in various menus. This change will prevent that by checking if the app is installed for the viewer before displaying it as an option

Test Plan: I went to the "Pin Applications" menu and typed in "Conpherence" and saw it appear as an option. I then went to the "Edit Policies" menu as an admin and removed permissions for my user to use Conpherence. Once I did that, it no longer showed up in the typeahead menu

Reviewers: epriestley, #blessed_reviewers

Reviewed By: epriestley, #blessed_reviewers

Subscribers: Korvin, epriestley, yelirekim

Maniphest Tasks: T11595

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

Details

Committed
Josh Cox <joshcox@uberatc.com>Sep 8 2016, 04:42
Pushed
aubortJan 31 2017, 17:16
Parents
rPH25bd14a9b1e4: Respect "End Date" for recurring Calendar events in detail UI
Branches
Unknown
Tags
Unknown

Event Timeline

Josh Cox <joshcox@uberatc.com> committed rPH0030bda17ea3: Check if app is installed for user before displying (authored by Josh Cox <joshcox@uberatc.com>).Sep 8 2016, 04:42