Paths

Table of Contentst

Remove PhabricatorUser->loadPreferences()
ClosedPublic
Actions

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

Tags

None

Referenced Files

	F13176530: D16039.id38605.diff
	Wed, May 8, 2:48 PM

	Unknown Object (File)
	Tue, May 7, 5:40 AM

	Unknown Object (File)
	Mon, May 6, 6:45 AM

	Unknown Object (File)
	Wed, May 1, 12:08 AM

	Unknown Object (File)
	Sun, Apr 28, 9:19 AM

	Unknown Object (File)
	Thu, Apr 25, 1:41 AM

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

	Unknown Object (File)
	Thu, Apr 11, 9:01 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

Lint

Lint Not Applicable

Unit

Tests Not Applicable

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 38605

View Options

src/applications/people/storage/PhabricatorUser.php