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

Update Module.php #34

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Update Module.php #34

wants to merge 1 commit into from

Conversation

elia-c
Copy link

@elia-c elia-c commented Aug 8, 2014

I tried to use your module with zf 2.3.2dev and when I execute the import command the module delete all my data but, doesn't create anything, that because the command:
$paths = $sm->get('doctrine.configuration.fixtures');
in Module.php doesn't work.
Just change it with:
$config = $sm->get('Config'); $paths = $config['doctrine']['configuration']['fixtures'];
and please change the documentation for the reference of fixture paths.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.08%) when pulling a5b3a4e on elia-c:patch-1 into 0132973 on Hounddog:master.

@Ocramius
Copy link
Collaborator

Ocramius commented Aug 8, 2014

What does "doesn't work" mean here? Are you getting an exception? What is the return value?

@Ocramius
Copy link
Collaborator

@elia-c I'd rather say that we should find out why the service is an empty array. It should rather be filled. We provided a dedicated service especially to prevent manual traversal of the configuration nodes.

@edipoReboucas
Copy link

Follow step by step de README, I have the problem with namespace doctrine[fixture], I found de solution on stack overflow http://stackoverflow.com/questions/18022089/zend-framework-2-doctrine2-doctrinedatafixturemodule-no-loading-fixtures?answertab=votes#tab-top.

This guy said to use 'doctrine-fixture' root namespace instead of doctrine[fixture] how REAMD said.

Please, update de README and thanks by write this module, easy my life.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants