Skip to content

[amqp-ext] Passing null to parameter is deprecated. #1275

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.

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

Closed
wants to merge 1 commit into from
Closed

[amqp-ext] Passing null to parameter is deprecated. #1275

wants to merge 1 commit into from

Conversation

EmilMassey
Copy link

Fixes deprecation notice Deprecated: AMQPQueue::consume(): Passing null to parameter #3 ($consumer_tag) of type string is deprecated in

@EmilMassey
Copy link
Author

I did not change the line reported by the code style check in this PR. I fixed that in a separate PR #1276

@nightlinus
Copy link
Contributor

Nice change for 8.1 compatibility, can we have this merged please @makasim

@makasim
Copy link
Member

makasim commented Dec 11, 2022

Nice change for 8.1 compatibility, can we have this merged please @makasim

Indeed, once all tests are green

@EmilMassey
Copy link
Author

Nice change for 8.1 compatibility, can we have this merged please @makasim

Indeed, once all tests are green

The failures seem to be unrelated to my changes (403 from AWS). Can you please take a look? Maybe they need to be rerun?

@makasim
Copy link
Member

makasim commented Dec 11, 2022

Sorry, I am not going to look into it. I don't use the library nor write in PHP.

I can review and merge when everything is green, and release it.

Fixes deprecation notice "Deprecated: AMQPQueue::consume(): Passing null to parameter #3 ($consumer_tag) of type string is deprecated in"
@EmilMassey EmilMassey closed this Feb 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants