Add dependent fixtures automatically about data-fixtures HOT 8 CLOSED

your class name let me think that you are using Symfony2. In this case, you should use https://github.com/symfony/DoctrineFixturesBundle which autoloads the fixtures (in MyBundle\DataFixtures\ORM) and integrates the command in the Symfony2 CLI.

from data-fixtures.

I'm using LiipFunctionalTestBundle to load fixtures for functional test. And, it uses DoctrineFixturesBundle to load fixtures.
I found it will throw exception when a fixture return by getDependencies() but not add by addFixtures().

My suggestion is why not just add dependent fixtures automatically. For example, when we use apt-get to install package, the dependent packages will also install automatically.

Thanks for your reply in advance.

from data-fixtures.

the bundles adds fixtures automatically when you put them in the good namespace: MyBundle\DataFixtures\ORM (note it is DataFixtures, not Fixtures).
And adding fixtures automatically when they appear in the dependencies can cause some issues: what if the fixture class needs some arguments ?

from data-fixtures.

Maybe you are right. Don't you think that the exception message should be Fixture "MyBundle\DataFixtures\ORM\LoadUserData" was declared as a dependency, but it should be added in fixture loader first. ?

from data-fixtures.

yeah, the exception message can probably be improved

from data-fixtures.

@sliver please make a PR for this exception update. =)

from data-fixtures.

Improve exception message #32.

@stof I don't have idea why fixture class needs some arguments.
Since the exception message got improved, I think it's good enough.

from data-fixtures.

@sliver to inject some dependencies of the class.

And for instance, in the Symfony bundle, an extended loader is used to allow injecting the container in the class if the fixture class implement the needed interface. Someone using a different way to handle the dependencies could choose to use arguments when registering the fixtures in the loader

from data-fixtures.