enupal\socializer\controllers\SettingsController

Complexity

Total Complexity

2

Size/Duplication

Total Lines	35
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
A	actionSaveSettings()	0	26	2

<?php
/**
 * Socializer plugin for Craft CMS 3.x
 *
 * @link      https://enupal.com/
 * @copyright Copyright (c) 2019 Enupal LLC
 */

namespace enupal\socializer\controllers;

use Craft;
use craft\web\Controller as BaseController;
use enupal\socializer\Socializer;

class SettingsController extends BaseController
{
    /**
     * Save Plugin Settings
     *
     * @return \yii\web\Response
     * @throws \craft\errors\MissingComponentException
     * @throws \yii\web\BadRequestHttpException
     */
    public function actionSaveSettings()
    {
        $this->requirePostRequest();
        $settings = Craft::$app->getRequest()->getBodyParam('settings');
        $message = Craft::t('enupal-socializer','Settings saved.');

        $plugin = Socializer::$app->settings->getPlugin();
        $settingsModel = $plugin->getSettings();

        $settingsModel->setAttributes($settings, false);

        if (!Socializer::$app->settings->saveSettings($settingsModel)) {

It seems like $settingsModel can also be of type null; however, parameter $settings of enupal\socializer\services\Settings::saveSettings() does only seem to accept enupal\socializer\models\Settings, maybe add an additional type check?

            Craft::$app->getSession()->setError(Craft::t('enupal-socializer','Couldn’t save settings.'));

            // Send the settings back to the template
            Craft::$app->getUrlManager()->setRouteParams([
                'settings' => $settingsModel
            ]);

            return null;
        }

        Craft::$app->getSession()->setNotice($message);

        return $this->redirectToPostedUrl();
    }
}