Page MenuHomePhabricator
Differential D16936

Make logged-out users use global settings, not default settings
ClosedPublic
Actions

Authored by epriestley on Nov 23 2016, 10:06 PM.

Details

Reviewers
chad
Maniphest Tasks
T11917: Global default settings only apply to logged in users
Commits
rPa1025ca52e03: Make logged-out users use global settings, not default settings
Summary

Fixes T11917. Give logged-out / omnipotent users the global settings, not the default settings.

Test Plan

Changed applications and language, logged out, saw changes as a public user.

Diff Detail

Repository
rP Phabricator
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

epriestley updated this revision to Diff 40765.Nov 23 2016, 10:06 PM
epriestley retitled this revision from to Make logged-out users use global settings, not default settings.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
epriestley added a task: T11917: Global default settings only apply to logged in users.
chad accepted this revision.Nov 23 2016, 10:14 PM
chad edited edge metadata.
This revision is now accepted and ready to land.Nov 23 2016, 10:14 PM
Closed by commit rPa1025ca52e03: Make logged-out users use global settings, not default settings (authored by epriestley). · Explain WhyNov 23 2016, 10:26 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 40767

View Options

src/applications/people/storage/PhabricatorUser.php

Loading...
Phabricator · Built by Phacility