Skip to content

Removed zend framework bridge.#2270

Open
Daniel-KM wants to merge 4 commits intoomeka:developfrom
Daniel-KM:perf/remove_zend_bridge
Open

Removed zend framework bridge.#2270
Daniel-KM wants to merge 4 commits intoomeka:developfrom
Daniel-KM:perf/remove_zend_bridge

Conversation

@Daniel-KM
Copy link
Contributor

This library is useless now, no module or dependencies uses it.

@Daniel-KM
Copy link
Contributor Author

There is an issue but "it works on my computer", so some investigation to do.

@Daniel-KM
Copy link
Contributor Author

So it requires to updated doctrine orm config and to replace deprecated doctrine cache by symfony cache.

And in fact, i see that there are some other abandoned packages that should be replaced by symfony or psr equivalent:

  • Package laminas/laminas-dom is abandoned, you should avoid using it. Use symfony/dom-crawler instead.
  • Package laminas/laminas-json is abandoned, you should avoid using it. No replacement was suggested.
  • Package laminas/laminas-loader is abandoned, you should avoid using it. No replacement was suggested.
  • Package laminas/laminas-log is abandoned, you should avoid using it. Use monolog/monolog instead.
  • Package laminas/laminas-mail is abandoned, you should avoid using it. Use symfony/mailer instead.
  • Package laminas/laminas-math is abandoned, you should avoid using it. No replacement was suggested.
  • Package laminas/laminas-mime is abandoned, you should avoid using it. Use symfony/mime instead.
  • Package lsolesen/pel is abandoned, you should avoid using it. No replacement was suggested.

@Daniel-KM
Copy link
Contributor Author

More changes are needed to use entities annotations.

@Daniel-KM
Copy link
Contributor Author

Daniel-KM commented Mar 10, 2025

So to use the symfony cache requires to update all entities to add a use statement and @ORM or php 8 attributes (annotations are deprecated). But it breaks modules with entities. So this pr can be used for next version and for a more complete upgrade (see #2272).

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 this pull request may close these issues.

1 participant