Restore psyduck to his proper place
ClosedPublic
Actions

Authored by epriestley on Mar 22 2014, 6:09 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Mon, Aug 5, 9:33 AM

	Unknown Object (File)
	Mon, Aug 5, 1:49 AM

	Unknown Object (File)
	Mon, Aug 5, 1:47 AM

	Unknown Object (File)
	Mon, Aug 5, 1:46 AM

	Unknown Object (File)
	Mon, Aug 5, 1:44 AM

	Unknown Object (File)
	Sat, Jul 27, 9:32 AM

	Unknown Object (File)
	Fri, Jul 26, 12:57 PM

	Unknown Object (File)
	Sun, Jul 21, 2:41 PM

View All Files

Subscribers

epriestley

Details

Reviewers

btrahan
chad

Maniphest Tasks

T4665: Default Pysduck in the main header is missing

Commits

Restricted Diffusion Commit
rP9c872749a05e: Restore psyduck to his proper place

Summary

Fixes T4665. The "attachable" logic was a little off after a recent change.

Test Plan

With and without a profile image, viewed a page.

Diff Detail

Repository

rP Phabricator

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

epriestley updated this revision to Diff 20379.Mar 22 2014, 6:09 PM

epriestley retitled this revision from to Restore psyduck to his proper place.

epriestley updated this object.

epriestley edited the test plan for this revision. (Show Details)

epriestley added reviewers: chad, btrahan.

epriestley added a task: T4665: Default Pysduck in the main header is missing.

Herald added a subscriber: epriestley.Mar 22 2014, 6:09 PM

chad accepted this revision.Mar 22 2014, 6:28 PM

chad edited edge metadata.

This revision is now accepted and ready to land.Mar 22 2014, 6:28 PM

Closed by commit rP9c872749a05e (authored by @epriestley).

Revision Contents
Changeset List

Path

Size

src/

applications/

people/

storage/

PhabricatorUser.php

8 lines

Diff 20380

View Options

src/applications/people/storage/PhabricatorUser.php

Restore psyduck to his proper placeClosedPublicActions