Skip to content
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

New system check diagnostic for max_execution_time #11050

Closed
mattab opened this issue Dec 20, 2016 · 0 comments
Closed

New system check diagnostic for max_execution_time #11050

mattab opened this issue Dec 20, 2016 · 0 comments
Labels
c: Usability For issues that let users achieve a defined goal more effectively or efficiently. Help wanted Beginner friendly issues or issues where we'd highly appreciate community's help and involvement. not-in-changelog For issues or pull requests that should not be included in our release changelog on matomo.org.
Milestone

Comments

@mattab
Copy link
Member

mattab commented Dec 20, 2016

Per email feedback:

I have tried to install piwik for many hours.
I have read your manual and forums several times.
I had many messages about missing database, settings etc.
My solution was found on http://stackoverflow.com.

I would recommend you to put the php.ini setting "max_execution_time = 0" in red at the beginning of your installation manual.
This will help many noobs like me for sure. :-)

-> We could add a new diagnostic that checks that max_execution_time is set at least to 30 or to 0 (unlimited).

@mattab mattab added this to the Backlog (Help wanted) milestone Feb 20, 2017
@mattab mattab added the c: Usability For issues that let users achieve a defined goal more effectively or efficiently. label Feb 20, 2017
@mattab mattab added the Help wanted Beginner friendly issues or issues where we'd highly appreciate community's help and involvement. label Oct 23, 2017
diosmosis pushed a commit that referenced this issue Aug 6, 2018
* New system check diagnostic for max_execution_time (#11050)

* PhpSettingsCheck: getRequiredSettings() returns array instead building it in property

* Rename PhpSettingsCheckService to RequiredPhpSetting and swap || with OR in string

* PhpSettingsCheck: method to set whether setting should raise an error or warning
@diosmosis diosmosis modified the milestones: Backlog (Help wanted), 3.6.0 Aug 6, 2018
@diosmosis diosmosis added the not-in-changelog For issues or pull requests that should not be included in our release changelog on matomo.org. label Aug 6, 2018
InfinityVoid pushed a commit to InfinityVoid/matomo that referenced this issue Oct 11, 2018
…matomo-org#12576)

* New system check diagnostic for max_execution_time (matomo-org#11050)

* PhpSettingsCheck: getRequiredSettings() returns array instead building it in property

* Rename PhpSettingsCheckService to RequiredPhpSetting and swap || with OR in string

* PhpSettingsCheck: method to set whether setting should raise an error or warning
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
c: Usability For issues that let users achieve a defined goal more effectively or efficiently. Help wanted Beginner friendly issues or issues where we'd highly appreciate community's help and involvement. not-in-changelog For issues or pull requests that should not be included in our release changelog on matomo.org.
Projects
None yet
Development

No branches or pull requests

2 participants