Paths

Table of Contentst

Remove PhabricatorUser->loadPreferences()
ClosedPublic
Actions

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

Tags

None

Referenced Files

	F13089137: D16039.diff
	Thu, Apr 25, 1:41 AM

	Unknown Object (File)
	Fri, Apr 19, 4:49 PM

	Unknown Object (File)
	Thu, Apr 11, 9:01 AM

	Unknown Object (File)
	Tue, Apr 9, 3:51 PM

	Unknown Object (File)
	Wed, Apr 3, 5:35 PM

	Unknown Object (File)
	Mon, Apr 1, 5:25 PM

	Unknown Object (File)
	Sat, Mar 30, 2:14 AM

	Unknown Object (File)
	Mar 21 2024, 10:29 PM

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