Restore psyduck to his proper place
ClosedPublic
Actions

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

Tags

None

Referenced Files

	F18837016: D8594.id20380.diff
	Mon, Oct 27, 4:10 AM

	F18825303: D8594.diff
	Thu, Oct 23, 8:55 PM

	F18771919: D8594.id.diff
	Wed, Oct 8, 10:53 PM

	F18769826: D8594.id20379.diff
	Wed, Oct 8, 9:26 AM

	F18760130: D8594.id20380.diff
	Mon, Oct 6, 8:09 AM

	F18742564: D8594.id.diff
	Thu, Oct 2, 8:16 PM

	F18736711: D8594.diff
	Wed, Oct 1, 7:20 AM

	F18625380: D8594.diff
	Sep 15 2025, 10:39 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