Paths

Table of Contentst

Remove PhabricatorUser->loadPreferences()
ClosedPublic
Actions

Authored by epriestley on Jun 4 2016, 11:26 PM.

Tags

None

Referenced Files

	F19511359: D16039.diff
	Mon, Jan 12, 1:05 AM

	F19511355: D16039.diff
	Mon, Jan 12, 1:03 AM

	F19067342: D16039.id.diff
	Nov 30 2025, 8:09 AM

	F19062998: D16039.diff
	Nov 29 2025, 5:35 PM

	F19056482: D16039.id38605.diff
	Nov 28 2025, 7:50 PM

	F19030269: D16039.id38589.diff
	Nov 24 2025, 8:38 PM

	F18940466: D16039.id38590.diff
	Nov 11 2025, 8:29 AM

	F18849194: D16039.id.diff
	Oct 30 2025, 8:19 AM

View All Files

Subscribers

None

Details

Reviewers

chad

Maniphest Tasks

T4103: Implement "Role Profiles" to provide search, homepage and application defaults

Commits

rP7170b062e63c: Remove PhabricatorUser->loadPreferences()

Summary

Ref T4103. This method has no more callers.

Test Plan

grep

Diff Detail

Repository

rP Phabricator

Branch

setting28

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 12461
Build 15788: Run Core Tests
Build 15787: arc lint + arc unit

Event Timeline

Also remove cache dirty in PhabricatorUserPreferences->save(). All writes now go through the Editor.

chad accepted this revision.Jun 5 2016, 2:06 AM

chad edited edge metadata.

This revision is now accepted and ready to land.Jun 5 2016, 2:06 AM

Closed by commit rP7170b062e63c: Remove PhabricatorUser->loadPreferences() (authored by epriestley, committed by epriestley). · Explain WhyJun 5 2016, 3:50 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

people/

storage/

PhabricatorUser.php

23 lines

settings/

storage/

PhabricatorUserPreferences.php

10 lines

Diff 38590

View Options

src/applications/people/storage/PhabricatorUser.php