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

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

Tags

None

Referenced Files

	F14027404: D8329.diff
	Fri, Nov 8, 7:05 AM

	F14007258: D8329.id.diff
	Tue, Oct 29, 2:39 AM

	F14003530: D8329.diff
	Sat, Oct 26, 7:10 AM

	Unknown Object (File)
	Sep 23 2024, 1:05 PM

	Unknown Object (File)
	Sep 23 2024, 1:05 PM

	Unknown Object (File)
	Sep 23 2024, 1:05 PM

	Unknown Object (File)
	Sep 23 2024, 1:02 PM

	Unknown Object (File)
	Sep 15 2024, 9:00 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