Skip to content

Conversation

@jrfnl
Copy link
Member

@jrfnl jrfnl commented Jul 11, 2020

The XMLRPC extension will be removed without deprecation in PHP 8.0.

This commit adds the functions and class from the extension to the respective RemovedFunctions and RemovedClasses sniffs.

The class is not listed in the PHP manual, but is in the stub file (as a final class), so, while I'm not 100% sure, I do think we should account for it.

Includes unit tests.

Refs:

Related to #809

The [XMLRPC](https://www.php.net/manual/en/book.xmlrpc.php) extension will be removed without deprecation in PHP 8.0.

This commit adds the functions and class from the extension to the respective `RemovedFunctions` and `RemovedClasses` sniffs.

The class is not listed in the PHP manual, but is in the stub file (as a final class), so, while I'm not 100% sure, I do think we should account for it.

Includes unit tests.

Refs:
* https://wiki.php.net/rfc/unbundle_xmlprc
* php/php-src@4fa3687
* https://www.php.net/manual/en/book.xmlrpc.php
* https://github.com/php/php-src/blob/444aa2636e81053e439623f2dbf981eaa48e6161/ext/xmlrpc/xmlrpc.stub.php
@jrfnl jrfnl added Type: enhancement PR: quick merge PR only contains relatively simple changes PR: ready for review labels Jul 11, 2020
@jrfnl jrfnl added this to the 10.0.0 milestone Jul 11, 2020
@jrfnl jrfnl requested a review from wimg July 11, 2020 19:31
@wimg wimg merged commit f739365 into develop Jul 18, 2020
@wimg wimg deleted the php-8.0/removed-xmlrpc-extension branch July 18, 2020 22:26
@jrfnl jrfnl removed PR: quick merge PR only contains relatively simple changes PR: ready for review labels Jul 19, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants