Reuse common getUser() / setUser() in PHUICalendarListView
ClosedPublic
Actions

Authored by epriestley on Feb 25 2014, 2:29 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Mon, Feb 24, 11:32 AM

	Unknown Object (File)
	Mon, Feb 17, 11:48 AM

	Unknown Object (File)
	Mon, Feb 17, 10:44 AM

	Unknown Object (File)
	Mon, Feb 17, 10:44 AM

	Unknown Object (File)
	Sun, Feb 16, 12:25 AM

	Unknown Object (File)
	Thu, Feb 13, 4:02 AM

	Unknown Object (File)
	Wed, Feb 5, 3:35 PM

	Unknown Object (File)
	Sun, Feb 2, 11:26 PM

View All Files

Subscribers

aran

Details

Reviewers

btrahan
chad

Commits

Restricted Diffusion Commit
rPd19e2155be0b: Reuse common getUser() / setUser() in PHUICalendarListView

Summary

These methods already exist in AphrontView. The redefinition can sometimes cause this warning:

[2014-02-24 18:27:48] ERROR 2048: Declaration of PHUICalendarListView::setUser() should be compatible with AphrontView::setUser(PhabricatorUser $user) at [/INSECURE/devtools/phabricator/src/view/phui/calendar/PHUICalendarListView.php:138]

Test Plan

Viewed calendar on profile.

Diff Detail

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

thx

Closed by commit rPd19e2155be0b (authored by @epriestley).

Revision Contents
Changeset List

Path

Size

src/

view/

phui/

calendar/

PHUICalendarListView.php

15 lines

Diff 19809

View Options

src/view/phui/calendar/PHUICalendarListView.php

Reuse common getUser() / setUser() in PHUICalendarListViewClosedPublicActions