Page MenuHomeDevCentral

Upgrade code to PHP 7.1
ClosedPublic

Authored by dereckson on Dec 19 2016, 00:35.
Referenced Files
F3763806: D753.id1936.diff
Fri, Nov 22, 02:39
Unknown Object (File)
Sun, Nov 17, 03:20
Unknown Object (File)
Wed, Nov 13, 08:35
Unknown Object (File)
Sat, Nov 9, 23:29
Unknown Object (File)
Sat, Nov 9, 23:28
Unknown Object (File)
Wed, Nov 6, 17:39
Unknown Object (File)
Tue, Nov 5, 05:21
Unknown Object (File)
Mon, Oct 28, 09:37
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.