-
Notifications
You must be signed in to change notification settings - Fork 155
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
Init Deserialize provider #780
Conversation
0e0d1fa
to
8ac206b
Compare
/** @var HttpOperationInitiatorInterface|ObjectProphecy */ | ||
private $operationInitiator; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/** @var HttpOperationInitiatorInterface|ObjectProphecy */ | |
private $operationInitiator; | |
private HttpOperationInitiatorInterface|ObjectProphecy $operationInitiator; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yeah it was on previous PR, and this is changed
/** @var SerializerInterface|ObjectProphecy */ | ||
private $serializer; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/** @var SerializerInterface|ObjectProphecy */ | |
private $serializer; | |
private SerializerInterface|ObjectProphecy $serializer; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yeah it was on previous PR, and this is changed
/** @var DeserializeListener */ | ||
private $deserializableListener; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/** @var DeserializeListener */ | |
private $deserializableListener; | |
private DeserializeListener $deserializableListener; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
see comments above
2f4a9ed
to
cf6b133
Compare
cf6b133
to
031c58a
Compare
031c58a
to
ed0300e
Compare
Co-authored-by: Dmitri Perunov <diimpp@gmail.com>
Thank you, @loic425! |
Needs #781 first
Not used and no DI for now, cause we need the ReadProvider first.