Skip to content

[fs] Do not throw error in jsonUnserialize on deprecation notice #671

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

Merged
merged 1 commit into from
Dec 6, 2018
Merged

[fs] Do not throw error in jsonUnserialize on deprecation notice #671

merged 1 commit into from
Dec 6, 2018

Conversation

ssiergl
Copy link
Contributor

@ssiergl ssiergl commented Dec 5, 2018

Caused by queue-interop package

[ErrorException]
The "Enqueue\Fs\FsMessage" class implements "Interop\Queue\PsrMessage" that is deprecated will be removed in later versions. use one without Psr prefix.

  • The Message interface is the root interface of all transport messages. Most message-oriented middleware (MOM) products treat messages as lightweight entities that consist of a header and a payload. The header contains fields used for message routing an
    d identification; the payload contains the application data being sent.
  • Within this general form, the definition of a message varies significantly across products.

thrown here: https://github.com/ssiergl/enqueue-dev/blob/master/pkg/fs/FsConsumer.php#L131

As discussed in gitter. Without this patch fs transporter is not usable.

Caused by queue-interop package

[ErrorException]
  The "Enqueue\Fs\FsMessage" class implements "Interop\Queue\PsrMessage" that is deprecated will be removed in later versions. use one without Psr prefix.
   * The Message interface is the root interface of all transport messages. Most message-oriented middleware (MOM) products treat messages as lightweight entities that consist of a header and a payload. The header contains fields used for message routing an
  d identification; the payload contains the application data being sent.
   * Within this general form, the definition of a message varies significantly across products.

thrown here: https://github.com/ssiergl/enqueue-dev/blob/master/pkg/fs/FsConsumer.php#L131
@ssiergl
Copy link
Contributor Author

ssiergl commented Dec 5, 2018

@makasim as discussed ignoring the deprecation. Works as expected now

@ssiergl
Copy link
Contributor Author

ssiergl commented Dec 5, 2018

well failed functional testing is not caused by this patch :D

@makasim makasim merged commit dfea18e into php-enqueue:master Dec 6, 2018
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.

2 participants