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

Laminas migration? #62

Open
demiankatz opened this issue Feb 4, 2020 · 8 comments
Open

Laminas migration? #62

demiankatz opened this issue Feb 4, 2020 · 8 comments

Comments

@demiankatz
Copy link

Just curious if there are plans to move this library to Laminas -- and if so, whether that involves forking the project, issuing a new major version, or something else.

I'm not currently using this -- it's just a component I have been considering for the future -- so please feel no pressure to act on this inquiry. However, I thought I might not be the only one wondering how Laminas will impact it, so I'm opening this issue to capture that answer for future reference. Apologies if this is already documented somewhere and I have missed it!

@ddelrio1986
Copy link

It should work as-is with the laminas-zendframework-bridge. I haven't tested it yet though.

@demiankatz
Copy link
Author

@ddelrio1986, I expect that you are correct and this should work fine with the bridge. I'm just curious about long-term plans, since the bridge is intended as a stopgap measure. (And like I say, I'm not trying to rush this along because I have no direct stake in it at the moment ... it's just something that probably needs to be addressed sooner or later).

@gburetz
Copy link

gburetz commented Mar 24, 2020

This library does not work with the laminas-zendframework-bridge. There are instanceof checks that don't pass.

@ddelrio1986
Copy link

ddelrio1986 commented Mar 31, 2020

@gburetz thanks for testing that. I think that long-term there should be a new major release that updates all references to laminas.

@visto9259
Copy link

We created the Laminas-Commons organization to host ZF-Commons packages that are migrated to Laminas like ZfcUser and ZfcRbac.
It could be the place to host Zfr packages as well.
Check out the organization at https://github.com/Laminas-Commons.

@alejandro-fiore
Copy link

I made a fork from buddyloans/zfr-cors that was forked from zf-fr/zfr-cors, bump version to 3.0.0 and upload to packagist, so you can use it with Laminas: https://github.com/alejandro-fiore/zfr-cors/

I barely tested, works fine for me!

@visto9259
Copy link

I'll check it out. I am thinking of adding zfr-cors to the Laminas-Commons organization so that it has a home for the future on the Laminas framework.

@visto9259
Copy link

I created Laminas-Commons/LmcCors under the Laminas-Commons organization and ported the master branch of the repo to Laminas.
I did not carry over all the previous versions.
I also bumped up minimum PHP to 7.2.
The namespace was changed to LmcCors and the config key was changed to lmc_cors.
Other than that, they are no changes in functionality. The versioning was reset to 1.0.0.
New issues can be submitted and they will be addressed as when I have bandwidth available.
The Laminas-Commons organization is opened to people who want to maintain LmcCors in the future. Just let us know and we can add you the Dev team for the library.

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

No branches or pull requests

5 participants