Skip to content

PHP 8.4 | RemovedFunctionParameters: account for deprecation of overloaded session_set_save_handler() signature (RFC) #1677

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
wimg merged 1 commit into from
Feb 11, 2024
Merged

PHP 8.4 | RemovedFunctionParameters: account for deprecation of overloaded session_set_save_handler() signature (RFC) #1677

wimg merged 1 commit into from
Feb 11, 2024

Conversation

@jrfnl
Copy link
Member

@jrfnl jrfnl commented Feb 11, 2024

@jrfnl
PHP 8.4 | RemovedFunctionParameters: account for deprecation of overl…
fe59c28 
…oaded session_set_save_handler() signature (RFC)

> - Session:
>   . Calling session_set_save_handler() with more than 2 arguments is
>     deprecated. Use the 2-parameter signature instead.

Refs:
* https://wiki.php.net/rfc/deprecate_functions_with_overloaded_signatures#session_set_save_handler
* php/php-src 12769
* php/php-src@b36eac9
* https://www.php.net/ldap_exop

Related to 1589
@jrfnl jrfnl added Type: enhancement PR: quick merge PR only contains relatively simple changes PR: ready for review labels Feb 11, 2024
@jrfnl jrfnl added this to the 10.0.0 milestone Feb 11, 2024
@jrfnl jrfnl requested a review from wimg February 11, 2024 20:38
@wimg wimg enabled auto-merge February 11, 2024 20:40
@jrfnl jrfnl mentioned this pull request Feb 11, 2024
Open
35 tasks
@wimg wimg merged commit 8a46895 into develop Feb 11, 2024
@wimg wimg deleted the php-8.4/removedfunctionparams-session-set-save-handler branch February 11, 2024 20:50
@github-actions github-actions bot removed PR: quick merge PR only contains relatively simple changes PR: ready for review labels Feb 11, 2024
@jrfnl jrfnl added the PHP: 8.4 label Nov 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@wimg wimg wimg approved these changes

Assignees

No one assigned

Labels

PHP: 8.4 Type: enhancement

Projects

None yet

Milestone

10.0.0-alpha1

Development

Successfully merging this pull request may close these issues.

3 participants

@jrfnl @wimg