Page MenuHomePhorge

Fix invalid return values for void
Needs ReviewPublic

Authored by aklapper on Tue, Apr 29, 11:21.

Details

Reviewers
None
Group Reviewers
O1: Blessed Committers
Summary

Remove or correct wrong @return types in PHPDoc and/or correct wrong return values in methods.

Test Plan

Run static code analysis; read code; check method in parent class and other callers, etc.

Diff Detail

Repository
rP Phorge
Branch
returnVoid (branched from master)
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 1901
Build 1901: arc lint + arc unit

Event Timeline

src/applications/people/phid/PhabricatorPeopleUserEmailPHIDType.php
38

At this point this return can be dropped, with an evil laugh

src/applications/search/query/PhabricatorSearchDocumentQuery.php
108

At this point this return can be dropped, with an evil laugh


✅ P.S. I verified that nothing calls ->nextPage()->something()

src/infrastructure/daemon/workers/query/PhabricatorWorkerTriggerQuery.php
77

At this point this return can be dropped, with an evil laugh