Page MenuHomeDevCentral

Upgrade code to PHP 7.1
ClosedPublic

Authored by dereckson on Dec 19 2016, 00:35.
Referenced Files
F3779059: D753.id1936.diff
Mon, Nov 25, 07:09
Unknown Object (File)
Fri, Nov 22, 18:13
Unknown Object (File)
Fri, Nov 22, 17:51
Unknown Object (File)
Fri, Nov 22, 12:39
Unknown Object (File)
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
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 Passed
Unit
Tests Passed
Branch
php7 (branched from master)

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.