Page MenuHomePhabricator

Fix construction of default settings for users with no settings at all
ClosedPublic

Authored by epriestley on Jun 13 2016, 10:23 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Dec 13, 9:53 AM
Unknown Object (File)
Thu, Nov 28, 1:37 AM
Unknown Object (File)
Tue, Nov 19, 9:01 AM
Unknown Object (File)
Oct 23 2024, 2:18 PM
Unknown Object (File)
Oct 21 2024, 7:50 PM
Unknown Object (File)
Oct 21 2024, 5:07 AM
Unknown Object (File)
Oct 19 2024, 2:39 AM
Unknown Object (File)
Oct 19 2024, 1:33 AM
Subscribers
None

Details

Summary

Ref T11098. Users with at least one setting set correctly fall back to the defaults, but users with no settings at all currently do not.

Make them fall back to global defaults properly.

Test Plan
  • Set global defaults to some non-default setting.
  • Completely delete a user's settings.
  • bin/cache purge --purge-all or --purge-user.
  • View settings as the user.
    • Before change: showed hard-coded defaults instead of global defaults until you save anything.
    • After change: properly shows global defaults from the start.

Diff Detail

Repository
rP Phabricator
Branch
prefs1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 12642
Build 16058: Run Core Tests
Build 16057: arc lint + arc unit

Event Timeline

epriestley retitled this revision from to Fix construction of default settings for users with no settings at all.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
  • Very slight simplification.
chad edited edge metadata.
This revision is now accepted and ready to land.Jun 13 2016, 10:35 PM
epriestley edited edge metadata.
  • Fix a unit test issue for installs with no global settings.
This revision was automatically updated to reflect the committed changes.