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

Added 'mbstring.func_overload = 0' check as installation requirement #8104

Merged
merged 1 commit into from Jun 15, 2015
Merged

Added 'mbstring.func_overload = 0' check as installation requirement #8104

merged 1 commit into from Jun 15, 2015

Conversation

barbushin
Copy link
Contributor

There is a problem with installing Piwik when mbstring.func_overload in php.ini is not 0.

Piwik just throws exception like Can't install Piwik: An exception has been thrown during the rendering of a template ("parse error: failed at content: ; line: 3682"winking smiley in "@Installation/layout.twig" at line 40.. And it's not so easy to understand what is wrong.

It's kind of popular issue:

#5338
http://forum.piwik.org/read.php?2,117181,117181
https://discussion.heroku.com/t/piwik-installation-problems-push-fails-configuration-not-correct/1073

@mnapoli mnapoli added the Bug For errors / faults / flaws / inconsistencies etc. label Jun 15, 2015
@mnapoli mnapoli added this to the 2.14.0 milestone Jun 15, 2015
@mnapoli
Copy link
Contributor

mnapoli commented Jun 15, 2015

👍 for the record the installation doesn't even work if that setting is on

mnapoli added a commit that referenced this pull request Jun 15, 2015
Added 'mbstring.func_overload = 0' check as installation requirement
@mnapoli mnapoli merged commit a8c8c23 into matomo-org:master Jun 15, 2015
@@ -38,6 +38,12 @@
To enjoy Piwik, you need remove <pre>ini_set</pre> from your <pre>disable_functions</pre> directive in php.ini, and restart your webserver.</p>";
}

if (ini_get('mbstring.func_overload')) {
$piwik_errorMessage .= "<p><strong>Piwik does not works when PHP is configured with <pre>mbstring.func_overload = " . ini_get('mbstring.func_overload') . "</pre></strong></p>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Small: Piwik does not works --> Piwik does not work

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

Successfully merging this pull request may close these issues.

None yet

3 participants