Page MenuHomeDevCentral

Upgrade code to PHP 7.1
ClosedPublic

Authored by dereckson on Dec 19 2016, 00:35.
Referenced Files
F7662474: D753.id1936.diff
Wed, Apr 30, 15:00
F7643370: D753.diff
Wed, Apr 30, 07:15
F7639674: D753.id1914.diff
Wed, Apr 30, 05:12
F7620647: D753.id.diff
Tue, Apr 29, 21:06
Unknown Object (File)
Tue, Apr 29, 13:15
Unknown Object (File)
Mon, Apr 28, 23:50
Unknown Object (File)
Sat, Apr 26, 00:41
Unknown Object (File)
Fri, Apr 25, 03:59
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.