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.

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

Remove laminas-json #257

Open
arhimede opened this issue Nov 22, 2024 · 3 comments
Open

Remove laminas-json #257

arhimede opened this issue Nov 22, 2024 · 3 comments
Assignees
Labels
Bug Something isn't working
Milestone

Comments

@arhimede
Copy link
Member

Bug Report

Q A
Version(s) x.y.z

Summary

TSC decided to mark the package laminas-json as abandoned.

Current behavior

Package laminas/laminas-json is abandoned, you should avoid using it. No replacement was suggested.

Expected behavior

No laminas-json mention in composer.

@arhimede arhimede added the Bug Something isn't working label Nov 22, 2024
@gsteel
Copy link
Member

gsteel commented Nov 22, 2024

It's already removed in 3.0-dev, but it can't be killed in 2.x because it would be a BC break in the Json View Helper due to the possible, but deprecated usage of the expression finder functionality.

Potentially, it could be moved to dev deps in 2.x as an "optional" dependency, but I think that would be a bad idea, and a source of confusion for people who use the Json view helper to simply encode data

@gsteel
Copy link
Member

gsteel commented Nov 22, 2024

FYI, its usage inside of JsonModel and JsonRenderer can easily be swapped out with native json_encode - it's usage in the view helper which is problematic

@gsteel
Copy link
Member

gsteel commented Nov 22, 2024

Ref #237 and #238

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants