Restore psyduck to his proper place
ClosedPublic
Actions

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

Tags

None

Referenced Files

	F13773598: D8594.diff
	Mon, Sep 9, 9:38 PM

	Unknown Object (File)
	Mon, Sep 2, 12:02 AM

	Unknown Object (File)
	Sun, Sep 1, 4:38 PM

	Unknown Object (File)
	Wed, Aug 21, 1:40 PM

	Unknown Object (File)
	Sat, Aug 17, 11:14 PM

	Unknown Object (File)
	Sat, Aug 17, 3:48 AM

	Unknown Object (File)
	Wed, Aug 14, 9:09 AM

	Unknown Object (File)
	Aug 9 2024, 9:20 AM

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