Skip to content

Conversation

@jrfnl
Copy link
Member

@jrfnl jrfnl commented Oct 27, 2025

This handles the deprecation of a number of functions which became no-op after an earlier (PHP 8.0/8.1) resource to object migration.

This commit accounts for the function deprecation.

Refs:

Curl:

Fileinfo:

GD:

XML:

Related to #1849

…(RFC)

This handles the deprecation of a number of functions which became no-op after an earlier (PHP 8.0/8.1) resource to object migration.

> - Curl:
>   . The curl_close() function has been deprecated, as CurlHandle objects are
>     freed automatically.
>     RFC: https://wiki.php.net/rfc/deprecations_php_8_5#deprecate_curl_close
>   . The curl_share_close() function has been deprecated, as CurlShareHandle
>     objects are freed automatically.
>     RFC: https://wiki.php.net/rfc/deprecations_php_8_5#deprecate_curl_share_close
> - FileInfo:
>   . The finfo_close() function has been deprecated.
>     As finfo objects are freed automatically.
>     RFC: https://wiki.php.net/rfc/deprecations_php_8_5#deprecate_finfo_close
> - GD:
>   . The imagedestroy() function has been deprecated, as GdImage objects are
>     freed automatically.
>     RFC: https://wiki.php.net/rfc/deprecations_php_8_5#deprecate_imagedestroy
> - XML:
>   . The xml_parser_free() function has been deprecated, as XMLParser objects
>     are freed automatically.
>     RFC: https://wiki.php.net/rfc/deprecations_php_8_5#deprecate_xml_parser_free

This commit accounts for the function deprecation.

Refs:
* https://wiki.php.net/rfc/deprecations_php_8_5#deprecate_no-op_functions_from_the_resource_to_object_conversion

Curl:
* https://github.com/php/php-src/blob/1570ce9f551f7dff3f51690e5c03b0f5ae0e159f/UPGRADING#L411-L416
* php/php-src 19451
* php/php-src@9b13bb1
* https://www.php.net/manual/en/function.curl-close.php
* php/php-src 19452
* php/php-src@699e563
* https://www.php.net/manual/en/function.curl-share-close.php

Fileinfo:
* https://github.com/php/php-src/blob/1570ce9f551f7dff3f51690e5c03b0f5ae0e159f/UPGRADING#L429-L431
* php/php-src 18396
* php/php-src@ccb716d
* https://www.php.net/manual/en/function.finfo-close.php

GD:
* https://github.com/php/php-src/blob/1570ce9f551f7dff3f51690e5c03b0f5ae0e159f/UPGRADING#L437-L439
* php/php-src 19454
* php/php-src@a68f3d6
* https://www.php.net/manual/en/function.imagedestroy.php

XML:
* https://github.com/php/php-src/blob/1570ce9f551f7dff3f51690e5c03b0f5ae0e159f/UPGRADING#L588-L590
* php/php-src 19449
* php/php-src@4de2ec3
* https://www.php.net/manual/en/function.xml-parser-free.php

Related to 1849
@jrfnl jrfnl added this to the 10.0.0-alpha2 milestone Oct 27, 2025
@jrfnl jrfnl requested a review from wimg October 27, 2025 20:23
@jrfnl jrfnl added Type: enhancement PR: quick merge PR only contains relatively simple changes PR: ready for review labels Oct 27, 2025
@wimg wimg enabled auto-merge October 27, 2025 20:30
@wimg wimg merged commit bc0c4ad into develop Oct 27, 2025
76 checks passed
@wimg wimg deleted the php-8.5/removedfunctions-noop-functions branch October 27, 2025 20:47
@github-actions github-actions bot removed PR: quick merge PR only contains relatively simple changes PR: ready for review labels Oct 27, 2025
@jrfnl jrfnl added the PHP: 8.5 label Nov 4, 2025
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