PSFS\controller\ConfigController

Complexity

Total Complexity

7

Size/Duplication

Total Lines	80
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	8

Test Coverage

Coverage

22.86%

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	getConfigParams()	0	11	2
B	saveConfig()	0	28	4
A	config()	0	12	1

<?php
namespace PSFS\controller;

use PSFS\base\config\Config;
use PSFS\base\config\ConfigForm;
use PSFS\base\Logger;
use PSFS\base\Router;
use PSFS\base\Security;
use PSFS\base\types\helpers\GeneratorHelper;
use PSFS\controller\base\Admin;

/**
 * Class ConfigController
 * @package PSFS\controller
 */
class ConfigController extends Admin
{

    /**
     * Servicio que devuelve los parámetros disponibles
     * @GET
     * @route /admin/config/params
     * @label Parámetros de configuración de PSGS
     * @visible false
     * @return mixed

Consider making the return type a bit more specific; maybe use string|null.

     */
    public function getConfigParams()
    {
        $response = array_merge(Config::$required, Config::$optional);
        $domains = Router::getInstance()->getDomains();
        foreach ($domains as $domain => $routes) {
            $pDomain = str_replace('@', '', $domain);
            $pDomain = str_replace('/', '', $pDomain);
            $response[] = strtolower($pDomain) . '.api.secret';
        }
        return $this->json($response);
    }

    /**
     * Método que gestiona la configuración de las variables
     * @GET
     * @Route /admin/config
     * @return string|null
     * @throws \HttpException
     */
    public function config()
    {
        Logger::log("Config loaded executed by " . $this->getRequest()->getRequestUri());
        /* @var $form \PSFS\base\config\ConfigForm */
        $form = new ConfigForm(Router::getInstance()->getRoute('admin-config'), Config::$required, Config::$optional, Config::getInstance()->dumpConfig());

This line exceeds maximum limit of 120 characters; contains 155 characters

        $form->build();
        return $this->render('welcome.html.twig', array(
            'text' => _("Bienvenido a PSFS"),
            'config' => $form,
            'typeahead_data' => array_merge(Config::$required, Config::$optional),
        ));
    }

    /**
     * Servicio que guarda la configuración de la plataforma
     * @POST
     * @route /admin/config
     * @visible false
     * @return string

Should the return type not be string|null?

     * @throws \HttpException
     */
    public function saveConfig()
    {
        Logger::getInstance()->infoLog(_("Guardando configuración"));
        /* @var $form \PSFS\base\config\ConfigForm */
        $form = new ConfigForm(Router::getInstance()->getRoute('admin-config'), Config::$required, Config::$optional, Config::getInstance()->dumpConfig());

This line exceeds maximum limit of 120 characters; contains 155 characters

        $form->build();
        $form->hydrate();
        if ($form->isValid()) {
            $debug = Config::getInstance()->getDebugMode();
            $newDebug = $form->getFieldValue("debug");
            if (Config::save($form->getData(), $form->getExtraData())) {
                Logger::log(_('Configuración guardada correctamente'));
                //Verificamos si tenemos que limpiar la cache del DocumentRoot
                if (boolval($debug) !== boolval($newDebug)) {
                    GeneratorHelper::clearDocumentRoot();
                }
                Security::getInstance()->setFlash("callback_message", _("Configuración actualizada correctamente"));
                Security::getInstance()->setFlash("callback_route", $this->getRoute("admin-config", true));
            } else {
                throw new \HttpException(_('Error al guardar la configuración, prueba a cambiar los permisos'), 403);
            }
        }
        return $this->render('welcome.html.twig', array(
            'text' => _("Bienvenido a PSFS"),
            'config' => $form,
            'typeahead_data' => array_merge(Config::$required, Config::$optional),
        ));
    }
}