Page MenuHomePhabricator

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

Authored by epriestley on Feb 25 2014, 2:29 AM.
Tags
None
Referenced Files
F18474033: D8329.id.diff
Tue, Sep 2, 10:07 PM
F18455493: D8329.diff
Mon, Sep 1, 9:06 AM
F18344751: D8329.diff
Tue, Aug 26, 10:10 AM
F18009437: D8329.id19809.diff
Aug 2 2025, 3:28 PM
F17883917: D8329.id.diff
Jul 28 2025, 11:38 PM
F17850336: D8329.diff
Jul 27 2025, 9:51 AM
Unknown Object (File)
Jun 25 2025, 5:22 PM
Unknown Object (File)
Jun 19 2025, 10:30 PM
Subscribers

Details

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