New "generate" command #13

Open
wants to merge 3 commits into
from

Projects

None yet

4 participants

@weaverryan

Hey guys!

This is a basic generate command for generating a new fixture class. It's basic and has really just one option: whether or not it should be container aware.

Also, small updates for the other task.

Thanks!

@fspillner

Here, I'd expect, you call a static method, but renderFile is an instance method, hm?

@fspillner

I'd rename the filename as "fixturesClass.php.twig".

@fspillner

Well done, Ryan!

@weaverryan

Thanks Fabian! I've made your suggested changes and re-tested the command.

@stof
Doctrine member

I'm going through the pending PRs of this repo (and I'm amazed about how old they can be). As I'm considering dropping the bundle for 2.1 (see my comment on #26), I think this command should probably be added in SensioGeneratorBundle instead

@lavoiesl
Doctrine member

Wow, it is indeed very old… Moving it to SensioGeneratorBundle would be nice indeed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment