Page MenuHomePhabricator

Fix errors found by PHPStan
ClosedPublic

Authored by vrana on Feb 16 2017, 3:09 PM.
Tags
None
Referenced Files
F14169094: D17371.id41769.diff
Sat, Dec 7, 3:20 PM
Unknown Object (File)
Thu, Dec 5, 6:25 AM
Unknown Object (File)
Tue, Dec 3, 8:20 PM
Unknown Object (File)
Sun, Dec 1, 6:37 AM
Unknown Object (File)
Fri, Nov 29, 10:00 PM
Unknown Object (File)
Tue, Nov 26, 9:38 PM
Unknown Object (File)
Fri, Nov 22, 8:07 PM
Unknown Object (File)
Mon, Nov 18, 7:19 PM
Subscribers

Details

Test Plan

Ran phpstan analyze -a autoload.php phabricator/src.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley added inline comments.
src/applications/calendar/__tests__/CalendarTimeUtilTestCase.php
10

This test case is no longer doing anything with $day so I think it can't be right, but I can try to figure that out and follow up.

src/applications/calendar/storage/PhabricatorCalendarExternalInvitee.php
13–19

We should just remove this: it has no callsites and was improperly copy-pasted. This object has no status, inviterPHID, etc., properties, so this code will fatal even with this fix.

src/applications/conpherence/conduit/ConpherenceCreateThreadConduitAPIMethod.php
52–53

I think this case can just be removed instead. Conpherence threads are no longer required to have a title (if they don't, we'll create a default one by listing the participants) and I think this just never got cleaned up.

ERROR_EMPTY_MESSAGE should be impossible when creating a thread and I think it only refs to sending an empty message to an existing thread.

This revision is now accepted and ready to land.Feb 16 2017, 4:12 PM

Address feedback

src/applications/calendar/__tests__/CalendarTimeUtilTestCase.php
10

I've changed it to getTimestamps($u, $day, 1) and the test now passes.

src/applications/calendar/storage/PhabricatorCalendarExternalInvitee.php
13–19

Note that it returns PhabricatorCalendarEventInvitee, not PhabricatorCalendarExternalInvitee. I'll submit it like this and let you follow up.

src/applications/conpherence/conduit/ConpherenceCreateThreadConduitAPIMethod.php
52–53

Removed.

This revision was automatically updated to reflect the committed changes.