Skip to content
This repository has been archived by the owner on Jan 8, 2020. It is now read-only.

Missing dependency to zend-form in zend-mvc #6506

Closed
tux-rampage opened this issue Aug 1, 2014 · 2 comments
Closed

Missing dependency to zend-form in zend-mvc #6506

tux-rampage opened this issue Aug 1, 2014 · 2 comments
Assignees
Milestone

Comments

@tux-rampage
Copy link
Contributor

See Issue #6162 (I cannot reopen this one)

This dependency is not in the skeleton app, but hardcoded in Zend\Mvc\Service\ServiceListenerFactory.

So there are two options:

  1. Add a dependency to zend-form to the composer.json of zend-mvc
  2. Change the ServiceListenerFactory to check for the existence of Zend\Form\FormAbstractServiceFactory before adding it as abstract factory.

I can offer a PR for option 2 if this is preferred.

@tux-rampage
Copy link
Contributor Author

Dependency was introduced in commit eaa7874

@Ocramius
Copy link
Member

Ocramius commented Aug 6, 2014

@tux-rampage fixed, thanks

@Ocramius Ocramius self-assigned this Aug 6, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants