Skip to content
This repository was archived by the owner on Nov 27, 2020. It is now read-only.

added parameter doctrine.orm.auto_generate_proxy_classes (in comment)...... #289

Closed
wants to merge 1 commit into from

Conversation

brikou
Copy link
Contributor

@brikou brikou commented Mar 7, 2012

... makes it easier to configure when running ./app/console --env=prod doctrine:ensure-production-settings

…... makes it easier to configure when running `./app/console --env=prod doctrine:ensure-production-settings`
@fabpot
Copy link
Member

fabpot commented Mar 7, 2012

When would you want to change the default value?

@brikou
Copy link
Contributor Author

brikou commented Mar 7, 2012

I think I miss something,... Maybe I forgot to set debug to false when checking "ensure production settings", i'll check again tomorrow, cheer @fabpot

@brikou
Copy link
Contributor Author

brikou commented Mar 8, 2012

I made a mistake when "ensuring production settings" ... forgot the no-debug switch... ./app/console --env=prod --no-debug doctrine:ensure-production-settings
Everything is fine... anyway this emphasizes the fact that "doctrine:ensure-production-settings" should return a better message...

Environment "prod" (with debug set to off) is correctly configured for production.

instead of:

Environment is correctly configured for production.

and:

Proxy classes are always regenerating in "prod" environment (with debug set to on).

instead of

Proxy Classes are always regenerating.

@fabpot anyway thanks

@brikou brikou closed this Mar 8, 2012
@mmenozzi
Copy link

I think that documentation at http://symfony.com/doc/2.0/book/doctrine.html should be updated with the --no-debug switch. Currently the no debug switch isn't present.

Thank you, anyway!

@stof
Copy link
Member

stof commented Oct 15, 2012

hmm, it is true that the console file was not switching to no-debug automatically when using the prod environment. Can you open a ticket (or a PR updating it) on the symfony-docs repo ?

@mmenozzi
Copy link

Done.

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

Successfully merging this pull request may close these issues.

4 participants