Page MenuHomePhorge

PHP 8.2: fix deprecated use of "parent" in callables
ClosedPublic

Authored by valerio.bozzolan on Mar 26 2023, 15:42.
Tags
None
Referenced Files
F2982170: D25099.1739946762.diff
Tue, Feb 18, 06:32
F2981821: D25099.1739938111.diff
Tue, Feb 18, 04:08
F2981808: D25099.1739937067.diff
Tue, Feb 18, 03:51
F2963713: D25099.1739217613.diff
Sun, Feb 9, 20:00
F2963486: D25099.1739215169.diff
Sun, Feb 9, 19:19
F2963485: D25099.1739215162.diff
Sun, Feb 9, 19:19
F2963484: D25099.1739215155.diff
Sun, Feb 9, 19:19
F2948527: D25099.1738575890.diff
Sun, Feb 2, 09:44
Tokens
"Like" token, awarded by valerio.bozzolan.

Details

Summary

Closes T15200

Test Plan

Test 1:

I was able to run arc unit --everything without the error thrown from T15200

Test 2:

I also tried this and it still works:

php -a
require 'src/applications/metamta/exception/PhabricatorMetaMTAReceivedMailProcessingException.php';
$asd = new PhabricatorMetaMTAReceivedMailProcessingException("1", "TEST MESSAGE");
var_dump($asd->getMessage());

You get:

string(12) "TEST MESSAGE"

Diff Detail

Repository
rP Phorge
Branch
T15200-callable
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 190
Build 190: arc lint + arc unit

Event Timeline

src/applications/metamta/exception/PhabricatorMetaMTAReceivedMailProcessingException.php
17

The above line means that if you do this somewhere:

new PhabricatorMetaMTAReceivedMailProcessingException( 'asd', 'lol' );

The above lines do this:

parent::__construct( 'asd', 'lol' );

That for my case is:

Exception::__construct( 'asd', 'lol' );

Note that get_parent_class($this) returns the word Exception.

src/applications/metamta/exception/PhabricatorMetaMTAReceivedMailProcessingException.php
18

In short using the string 'parent' is deprecated.

It was replaced with get_parent_class($this) that at runtime gives the parent, and it's the exact replacement.

This revision is now accepted and ready to land.Mar 29 2023, 18:18