Restore psyduck to his proper place
ClosedPublic
Actions

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

Tags

None

Referenced Files

	F14759588: D8594.id20380.diff
	Wed, Jan 22, 1:16 PM

	Unknown Object (File)
	Sat, Jan 11, 5:20 PM

	Unknown Object (File)
	Sat, Jan 11, 5:20 PM

	Unknown Object (File)
	Sat, Jan 11, 5:20 PM

	Unknown Object (File)
	Dec 10 2024, 7:02 PM

	Unknown Object (File)
	Nov 19 2024, 11:24 PM

	Unknown Object (File)
	Nov 16 2024, 2:58 AM

	Unknown Object (File)
	Nov 11 2024, 7:10 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