ownCloud should check for mbstring.func_overload

[BernhardPosselt]

Linking issue owncloud-archive/news#732

mbstring.func_overload basically tells PHP to use mb functions instead of the default ones, e.g.: stlren is turned into mb_strlen

This can cause all sorts of shenanigans, like broken pages (https://github.com/owncloud/core/blob/master/lib/private/appframework/app.php#L116) etc.

We should therefore warn on admins on the admin and setup page if they've enabled this setting

@DeepDiver1975 @MorrisJobke @LukasReschke @PVince81

[BernhardPosselt]

#shit-php-does

[BernhardPosselt]

@RealRancor why is news in the modules array? This thing applies globally

[MorrisJobke]

@RealRancor: Please add them there:

core/lib/private/util.php

Line 575 in 7a9d428

$dependencies = array(

[ghost]

Sorry, i won't start doing core development.

The checks in the linked app are easy for me to add and test on my stable7 productive system and the settings.php is also linked in a forums thread.

If this should be added to core someone else needs to do it.

[ghost]

@MorrisJobke
Btw. its now the third ini setting which should be checked in the util.php:

new: mbstring.func_overload, output_buffering
existing: default_charset

so its probably better that an more experienced dev is adding an general test for ini settings in the util.php

[LukasReschke]

Yes – fair point – will add an helper here as well. Thanks @RealRancor.