Skip to content
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

Remove decoder interface usage #10453

Merged
merged 3 commits into from
Aug 8, 2017
Merged

Remove decoder interface usage #10453

merged 3 commits into from
Aug 8, 2017

Commits on Aug 7, 2017

  1. Replace the usage of Magento\Framework\Json\DecoderInterface in the B… 

    …ookmark model,

 - deprecate the $jsonDecoder,
 - inject the new \Magento\Framework\Serialize\Serializer\Json
 - replace the usage of $jsonDecoder with the $serializer
    @dmanners
    dmanners committed Aug 7, 2017
    Configuration menu
    Copy the full SHA
    e9fa41e View commit details
    Browse the repository at this point in the history

  2. Replace the usage of Magento\Framework\Json\DecoderInterface in the b… 

    …ookmark save controller

 - deprecate $jsonDecoder,
 - inject the new \Magento\Framework\Serialize\Serializer\Json,
 - replace usage of $jsonDecoder with $serializer
    @dmanners
    dmanners committed Aug 7, 2017
    Configuration menu
    Copy the full SHA
    075eba8 View commit details
    Browse the repository at this point in the history

  3. Update the docblock for the constructor in the bookmark ui save contr… 

    …oller
    @dmanners
    dmanners committed Aug 7, 2017
    Configuration menu
    Copy the full SHA
    0e4a1a3 View commit details
    Browse the repository at this point in the history