search

laminas / laminas-view

Flexible view layer supporting and providing multiple view layers, helpers, and more
https://docs.laminas.dev/laminas-view/
BSD 3-Clause "New" or "Revised" License
74 stars 46 forks source link

Remove laminas-json #257

Open arhimede opened 4 days ago

arhimede commented 4 days ago

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.

gsteel commented 4 days ago

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 commented 4 days ago

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 commented 4 days ago

Ref #237 and #238