Private GIT

Skip to content
Snippets Groups Projects
Commit 76cb4479 authored by Lionel Assepo's avatar Lionel Assepo
Browse files

* Fix issue wit Options Form

parent 9b05da7c
No related branches found
No related tags found
No related merge requests found
...@@ -83,19 +83,16 @@ class OptionsController extends Controller ...@@ -83,19 +83,16 @@ class OptionsController extends Controller
{ {
$givenParameters = clone $this->getParams('post'); $givenParameters = clone $this->getParams('post');
$validationResult = Form::validate("form", $this->getUri(), self::$moduleName, $givenParameters); try {
if ($validationResult['success']) { OptionRepository::update($givenParameters, "default", "form", $this->getUri());
if (isset($givenParameters['token'])) { if (isset($givenParameters['token'])) {
unset($givenParameters['token']); unset($givenParameters['token']);
} }
OptionRepository::update($givenParameters);
unset($_SESSION['form_token']);
unset($_SESSION['form_token_time']);
$this->router->response()->json(array('success' => true)); $this->router->response()->json(array('success' => true));
} else { } catch (Exception $ex) {
$this->router->response()->json(array('success' => false,'error' => $validationResult['error'])); $this->router->response()->json(array('success' => false,'error' => $ex->getMessage()));
} }
} }
} }
...@@ -36,7 +36,7 @@ ...@@ -36,7 +36,7 @@
namespace CentreonAdministration\Repository; namespace CentreonAdministration\Repository;
use CentreonAdministration\Models\Options; use CentreonAdministration\Models\Options;
use Centreon\Internal\Form; use Centreon\Internal\Form\Validators\Validator;
/** /**
* @author Lionel Assepo <lassepo@centreon.com> * @author Lionel Assepo <lassepo@centreon.com>
...@@ -45,13 +45,30 @@ use Centreon\Internal\Form; ...@@ -45,13 +45,30 @@ use Centreon\Internal\Form;
*/ */
class OptionRepository class OptionRepository
{ {
/**
*
* @param type $givenParameters
* @param type $origin
* @param type $route
*/
protected static function validateForm($givenParameters, $origin = "", $route = "")
{
$formValidator = new Validator($origin, array('route' => $route, 'params' => array(), 'version' => '3.0.0'));
if ($origin == 'wizard' || $origin == 'form') {
$formValidator->csrf($givenParameters['token']);
unset($givenParameters['token']);
}
$formValidator->validate($givenParameters->all());
}
/** /**
* *
* @param type $submittedValues * @param type $submittedValues
* @param type $group * @param type $group
*/ */
public static function update($submittedValues, $group = "default") public static function update($submittedValues, $group = "default", $origin = "", $route = "")
{ {
self::validateForm($submittedValues, $origin, $route);
$currentOptionsList = Options::getOptionsKeysList(); $currentOptionsList = Options::getOptionsKeysList();
$optionsToSave = array(); $optionsToSave = array();
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please to comment