Skip to content

Adding Path option to make:Controller Command #294

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

Closed

Conversation

m0ns1f
Copy link

@m0ns1f m0ns1f commented Oct 14, 2018

f.g : php bin/console make:Controller --path=Foo/bon
and adding template option fg : php bin/console make:Controller --path=Foo/bon --template

@ckrack
Copy link
Contributor

ckrack commented Oct 27, 2018

In part this is a duplicate of #185 and #265
Changing the templates is something that won't be merged, mainly due to having to support them in the future. For example, the maker could not change the variables in the templates without breaking things.

@weaverryan
Copy link
Member

I DID just merge a --no-template option, to skip the template generation. But @ckrack is right that we won't merge an option to change the controller path. Actually, this is already possible - you can pass whatever fully-qualified class name you want as the controller argument, and the bundle will use the autoloading rules to place this in the correct location.

Cheers!

@weaverryan weaverryan closed this Dec 13, 2018
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.

3 participants