Page MenuHomePhorge

Fix some invalid PhpDoc
ClosedPublic

Authored by aklapper on Sat, Jun 14, 18:06.

Details

Summary

Fix invalid syntax etc.

Test Plan

Compare PHPDoc and signature; run static code analysis.

Diff Detail

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

Event Timeline

mainframe98 added inline comments.
src/applications/auth/engine/PhabricatorAuthSessionEngine.php
923

Do as mainframe98 rightfully says, plus fix two further issues.
Cannot avoid going above the 80 chars per line limit as PHPStan otherwise complains about a * being parsed

Some of these array type hints feel sketchy, but they already were unusual. I suppose this is fine. The rest certainly is.

This revision is now accepted and ready to land.Sun, Jun 15, 08:02
This revision was automatically updated to reflect the committed changes.