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

ZF3, EventManager 3 support #146

Merged
merged 4 commits into from
Jun 29, 2016
Merged

ZF3, EventManager 3 support #146

merged 4 commits into from
Jun 29, 2016

Conversation

Saeven
Copy link
Contributor

@Saeven Saeven commented Jun 29, 2016

This PR solves the event manager crisis caused by the changes to ZF3's SM. :)

Hooray! ZF3!

@tylkomat
Copy link

There are more open problems to make it running under ZF3.

The Initializers and Factories have not been updated to the new signatures. Also to make the console working one would need the package zendframework/zend-mvc-console, but with that package it is not compatible with ZF2. Another option would be to migrate to zfcampus/zf-console. That would make the console independent from zend-mvc package and backward compatible.

@widmogrod
Copy link
Owner

IMHO adapting signatures to ZF3 would be the best option. Then version of this project could be bumped to 2.0 and backward compatibility wouldn't be an issue, since anyone who needs previous version would use different version

@Saeven @tylkomat would you like/mind to continue work on ZF3?

@tylkomat
Copy link

tylkomat commented Jun 30, 2016

@widmogrod I don't mind. I anyway have a work in progress already, but there I was thinking to port to zf-console, since I noticed that the mvc-console has some issues see here and here, which have the same reason I suppose.

Also one would need to implement updates for both versions.

@widmogrod
Copy link
Owner

@tylkomat sounds reasonable to use zf-console.

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.

None yet

3 participants