Page MenuHomeDevCentral

Upgrade code to PHP 7.1
ClosedPublic

Authored by dereckson on Dec 19 2016, 00:35.
Referenced Files
Unknown Object (File)
Sun, Mar 30, 17:06
Unknown Object (File)
Sun, Mar 30, 09:53
Unknown Object (File)
Sun, Mar 30, 04:55
Unknown Object (File)
Sat, Mar 29, 22:56
Unknown Object (File)
Sat, Mar 29, 12:52
Unknown Object (File)
Sat, Mar 29, 02:00
Unknown Object (File)
Mon, Mar 24, 04:36
Unknown Object (File)
Mon, Mar 24, 04:31
Subscribers

Details

Summary

The current code works virtually without issue on PHP 7,
as it was prepared with PHP 7 in mind as target version.

In order to work nicely with PHP 7.1:

  • use nullable parameters type
  • fix ProjectsMapTest tests mock type, using an anonymous class

We also bump the version in Composer.

References:

Fixes T1064. Ref T1062.

Test Plan

All unit tests pass.

Diff Detail

Repository
rNOTIF Notifications center
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

tests/Phabricator/ProjectsMapTest.php
191

?

dereckson edited edge metadata.

Whitespace issue, use APIClient

Rebased against master. Jenkins will still try this on a PHP 5.2 container (see T625), so tests are expected to fail.

dereckson edited edge metadata.
This revision is now accepted and ready to land.Dec 25 2016, 16:40
This revision was automatically updated to reflect the committed changes.