Skip to content

Fixed producer fallback in sandbox mode#282

Closed
ramunasd wants to merge 1 commit intophp-amqplib:masterfrom
ramunasd:producer_interface
Closed

Fixed producer fallback in sandbox mode#282
ramunasd wants to merge 1 commit intophp-amqplib:masterfrom
ramunasd:producer_interface

Conversation

@ramunasd
Copy link
Copy Markdown
Member

Fallback lacks method setContentType

@videlalvaro
Copy link
Copy Markdown
Collaborator

The tests are not passing.

Also, wouldn't this break BC?

@ramunasd
Copy link
Copy Markdown
Member Author

ramunasd commented Jul 6, 2015

Travis had temporary issue, now it's passing all tests.
About BC - if someone has written it's own producer that implements ProducerInterface - then yes, it BC.

@videlalvaro
Copy link
Copy Markdown
Collaborator

Sorry to comment so late, but I think this method and setDeliveryMode should be added here as well? Why not have them in the ProducerInterface as well?

@github-actions
Copy link
Copy Markdown

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@ramunasd
Copy link
Copy Markdown
Member Author

@mihaileu this is the old one, but maybe still actual?

@mihaileu mihaileu added this to the 3.0 milestone Mar 17, 2021
@mihaileu
Copy link
Copy Markdown
Collaborator

As is a BC, it should be added in the next major version.

@ramunasd ramunasd closed this Mar 23, 2026
@ramunasd ramunasd deleted the producer_interface branch March 23, 2026 10:07
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