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

Wrong Memory value #1136

Closed
dugwood opened this issue Feb 1, 2010 · 3 comments
Closed

Wrong Memory value #1136

dugwood opened this issue Feb 1, 2010 · 3 comments
Labels
Bug For errors / faults / flaws / inconsistencies etc.
Milestone

Comments

@dugwood
Copy link

dugwood commented Feb 1, 2010

In the 2nd page of installation, I can read:
Limite mmoire 5242880M
(memory limit)

My php.ini is as follow:
memory_limit = 52428800

It's another way to fill the data, by specifing bytes only. So the formula isn't working here.

Page was: /piwik/index.php?action=systemCheck
Keywords: memory

@robocoder
Copy link
Contributor

(In [1820]) fixes #1136 - get memory limit expects setting in megabytes

@dugwood
Copy link
Author

dugwood commented Feb 1, 2010

if(substr($memory, -1) == 'K') <====== shouldn't it be k not K? Or the use of strtolower() on the value? Perhaps a switch() statement would be better here (even if performance on this script isn't really an issue).

Thanks for fixing the bug.

@mattab
Copy link
Member

mattab commented Feb 11, 2010

see [1821]

@dugwood dugwood added this to the Piwik 0.5.5 milestone Jul 8, 2014
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug For errors / faults / flaws / inconsistencies etc.
Projects
None yet
Development

No branches or pull requests

3 participants