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)
	Sat, Dec 28, 1:56 AM

	Unknown Object (File)
	Sat, Dec 28, 12:54 AM

	Unknown Object (File)
	Sun, Dec 15, 3:47 AM

	Unknown Object (File)
	Tue, Dec 3, 12:06 PM

	Unknown Object (File)
	Tue, Dec 3, 12:06 PM

	Unknown Object (File)
	Tue, Dec 3, 12:06 PM

	Unknown Object (File)
	Tue, Dec 3, 11:47 AM

	Unknown Object (File)
	Nov 29 2024, 1:56 AM

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