Skip to content
This repository has been archived by the owner on Mar 14, 2023. It is now read-only.

Configuration: php settings moved to local config file #44

Closed
wants to merge 1 commit into from
Closed

Configuration: php settings moved to local config file #44

wants to merge 1 commit into from

Conversation

rostenkowski
Copy link

Notes:

  • The timezone option has been simply moved.
  • The disabled zlib.output_compression yes has been changed to enabled and probably safer no and has been moved too.
  • It would be also cool to add information about php configuring to the http://doc.nette.org/cs/configuring page. Currently only in Session section is mentioned the ability to configure php with a link to the PHP site. The directives set in the sandbox should be mentioned on that page.

@rostenkowski
Copy link
Author

Because it is more intuitive as the default value in PHP is 0 and not #1 ;)

@milo
Copy link
Member

milo commented Oct 3, 2013

And when somebody has zlib.output_compression: yes in php.ini?

@rostenkowski
Copy link
Author

ok - so what? revert this change?

@milo
Copy link
Member

milo commented Oct 5, 2013

Probably yes. Personally, I do not need this PR at all, I wrote only the remark.

@milo
Copy link
Member

milo commented Oct 5, 2013

@hosiplan How? Which warning?

@fprochazka
Copy link
Contributor

Oh, wrong pull :D Sorry :P

@dg
Copy link
Member

dg commented Jan 10, 2014

Time zone should be set always and disabling compression is really not good idea.

@dg dg closed this Jan 10, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
4 participants