Skip to content

Error on executing - fastly:format:serializetojson #520

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
novykov opened this issue Mar 18, 2022 · 3 comments · Fixed by #521
Closed

Error on executing - fastly:format:serializetojson #520

novykov opened this issue Mar 18, 2022 · 3 comments · Fixed by #521

Comments

@novykov
Copy link

novykov commented Mar 18, 2022

STEPS TO REPRODUCE:

Setup Magento with Fastly module https://github.com/fastly/fastly-magento2
Run - bin/magento fastly:format:serializetojson

ACTUAL RESULTS:
Throws below error
bin/magento fastly:format:serializetojson PHP Fatal error: Uncaught Error: Call to undefined method Magento\Enterprise\Model\ProductMetadata::unserialize() in vendor/fastly/magento2/Console/Command/SerializeToJson.php:115

@dpotkoc
Copy link
Contributor

dpotkoc commented Mar 18, 2022

@novykov Hello, please could you provide us exactly php / Magento version?

Thnx

@novykov
Copy link
Author

novykov commented Mar 18, 2022

@dpotkoc tested on 2.4.3-p1 with php 7.4 and on 2.3.5 with php 7.3
The issue is reproducible on both

dpotkoc added a commit to favicode/fastly-magento2 that referenced this issue Mar 21, 2022
@dpotkoc dpotkoc mentioned this issue Mar 21, 2022
vvuksan added a commit that referenced this issue Mar 21, 2022
@vvuksan
Copy link
Contributor

vvuksan commented Mar 21, 2022

Fixed in 1.2.181

MickaelDatadome pushed a commit to DataDome/fastly-magento2 that referenced this issue Oct 5, 2023
MickaelDatadome added a commit to DataDome/fastly-magento2 that referenced this issue Oct 5, 2023
Co-authored-by: Shohei Maeda <[email protected]>
Co-authored-by: Domagoj Potkoc <[email protected]>
Co-authored-by: kirgiv4oja <[email protected]>
Co-authored-by: Vladimir Vuksan <[email protected]>
Co-authored-by: Pawel Siejba <[email protected]>
Co-authored-by: Andrew Kett <[email protected]>
Co-authored-by: Peter Dohogne <[email protected]>
Co-authored-by: Lex <[email protected]>
Co-authored-by: Dan Wallis <[email protected]>
Co-authored-by: Sean <[email protected]>
Co-authored-by: Domagoj Potkoc <[email protected]>
Co-authored-by: Jack Scotson <[email protected]>
Co-authored-by: Bohdan Korablov <[email protected]>
Co-authored-by: github-actions <smaeda-ks [email protected]>
Co-authored-by: Alejandro Marroni <[email protected]>
Co-authored-by: github-actions <vvuksan [email protected]>
Co-authored-by: Vladimir Vuksan <[email protected]>
Co-authored-by: Joao Oliveira Pereira <[email protected]>
Co-authored-by: Borna Butkovic <[email protected]>
Co-authored-by: Frangolacci Sebastien <[email protected]>
Co-authored-by: Borzas Laszlo <[email protected]>
Co-authored-by: mizdebski-netacea <[email protected]>
Co-authored-by: Pawel Siejba <[email protected]>
Co-authored-by: ivanviduka <[email protected]>
Co-authored-by: Lauredg <[email protected]>
Co-authored-by: Lauredg <[email protected]>
Fix unset req.http.graphql
fix typo (fastly#464)
fix Fastly const value ti int value fastly#466
fix plugin annotation fastly#466
fix fastly#466
fixing old value in core_config_data fastly#466
fix type check in Image (fastly#472)
fix config save - image optimization fastly#476
Fix ambiguous behavior in Blocking toggle (fastly#479)
Fix type check (fastly#483)
Fix bugs in upadte backend dialog (fastly#487)
Fix for: GetUpdateFlag call flushes all configuration fastly#499
Fix for fastly#504
fix for fastly#520
fixing duplicate line reported under issue#518
Fix fastly#393
fix fastly#393
Fix javascript fastly#544
fix fastly#551
fix getImageOptimization - Deprecated Functionality: explode(): Passing null to parameter #2
fix for fastly#549 as submitted by customer
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 a pull request may close this issue.

3 participants