Paths

Table of Contentst

Remove PhabricatorUser->loadPreferences()
ClosedPublic
Actions

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

Tags

None

Referenced Files

	F15441755: D16039.diff
	Wed, Mar 26, 7:44 PM

	F15424465: D16039.id38605.diff
	Sat, Mar 22, 10:33 PM

	F15413899: D16039.id38589.diff
	Wed, Mar 19, 9:54 PM

	F15409914: D16039.id38590.diff
	Wed, Mar 19, 5:55 AM

	F15393027: D16039.id.diff
	Sat, Mar 15, 7:00 PM

	F15387894: D16039.diff
	Sat, Mar 15, 3:04 AM

	F15380161: D16039.diff
	Fri, Mar 14, 1:44 AM

	F15375756: D16039.id38589.diff
	Thu, Mar 13, 12: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

Branch

setting28

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 12460
Build 15786: Run Core Tests
Build 15785: 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

Diff 38589

View Options

src/applications/people/storage/PhabricatorUser.php

Remove PhabricatorUser->loadPreferences()ClosedPublicActions