Switch to PSR-4 and move classes accordingly

doctrine / DoctrineModule

Doctrine Module for Laminas

http://www.doctrine-project.org/projects/doctrine-module.html

MIT License

398 stars 269 forks source link

Switch to PSR-4 and move classes accordingly #737

Closed driehle closed 3 years ago

driehle commented 3 years ago

Since doctrine-laminas-hydrator is and has always been using PSR-4, this project should follow. This PR switches from PSR-0 to PSR-4 and moves all classes in the right places, effectively droping one level of folder hierarchy in both src and test.

I'd like to target this at the next minor release, i.e. 4.2.0.

TomHAnderson commented 3 years ago

@Ocramius I wanted to do this years ago but was told then that the Doctrine standard is PSR-0. Has that changed? Can we go ahead with PSR-4 (this has my recommendation)

greg0ire commented 3 years ago

We've had a discussion about this BTW: https://github.com/orgs/doctrine/teams/doctrinecore/discussions/27

driehle commented 3 years ago

Plus, doctrine-laminas-hydrator has been using PSR-4 ever since and I strongly dislike having different standards across the Laminas-related projects. Porting doctrine-laminas-hydrator from PSR-4 to PSR-0, something which is essentially dead, does not make any sense to me.