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

Error in \humhub\libs\DynamicConfig::merge Beta3 #1457

Closed
re2bit opened this issue Feb 3, 2016 · 1 comment
Closed

Error in \humhub\libs\DynamicConfig::merge Beta3 #1457

re2bit opened this issue Feb 3, 2016 · 1 comment

Comments

@re2bit
Copy link

re2bit commented Feb 3, 2016

Function Body:

    /**
     * Add an array to the dynamic configuration
     * 
     * @param array $new
     */
    public static function merge($new)
    {
        $config = self::load();
        \yii\helpers\ArrayHelper::merge($new);
        self::save($config);
    }

throws Error:

PHP Warning – yii\base\ErrorException

Missing argument 2 for yii\helpers\BaseArrayHelper::merge(), called in /var/www/humhub/protected/humhub/libs/DynamicConfig.php on line 32 and defined

Should be:

       /**
     * Add an array to the dynamic configuration
     * 
     * @param array $new
     */
    public static function merge($new)
    {
        $config = \yii\helpers\ArrayHelper::merge(self::load(), $new);
        self::save($config);
    }
@luke- luke- closed this as completed in 0c3765b Feb 16, 2016
@luke-
Copy link
Contributor

luke- commented Feb 16, 2016

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants