Page MenuHomePhabricator

Make translation, timezone and pronoun into real settings
ClosedPublic

Authored by epriestley on Jun 1 2016, 11:57 PM.
Tags
None
Referenced Files
F15466216: D16005.id38527.diff
Thu, Apr 3, 3:02 AM
F15448686: D16005.id38518.diff
Fri, Mar 28, 6:19 AM
F15444420: D16005.id.diff
Thu, Mar 27, 9:32 AM
F15441945: D16005.diff
Wed, Mar 26, 8:42 PM
F15427867: D16005.diff
Sun, Mar 23, 4:30 PM
F15427037: D16005.id38527.diff
Sun, Mar 23, 12:17 PM
F15393932: D16005.id38518.diff
Sat, Mar 15, 11:17 PM
F15393217: D16005.id38518.diff
Sat, Mar 15, 8:28 PM
Subscribers
None

Details

Summary

Ref T4103. These are currently stored on the user, for historic/performance reasons.

Since I want administrators to be able to set defaults for translations and timezones at a minimum and there's no longer a meaningful performance penalty for moving them off the user record, turn them into real preferences and then nuke the columns.

Test Plan
  • Set settings to unusual values.
  • Ran migrations.
  • Verified my unusual settings survived.
  • Created a new user.
  • Edited all settings with old and new UIs.
  • Reconciled client/server timezone disagreement.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Make translation, timezone and pronoun into real settings.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
src/applications/metamta/storage/PhabricatorMetaMTAMail.php
167

This has been unused for a long time, it just never got cleaned up.

src/applications/settings/controller/PhabricatorSettingsTimezoneController.php
119–121

I guess "GMT" is deprecated and "UTC" is the new hotness.

chad edited edge metadata.
This revision is now accepted and ready to land.Jun 2 2016, 12:17 AM
This revision was automatically updated to reflect the committed changes.