diff --git a/datamodels/2.x/itop-portal-base/portal/src/Form/ObjectFormManager.php b/datamodels/2.x/itop-portal-base/portal/src/Form/ObjectFormManager.php index 88d434496..d93f3debd 100644 --- a/datamodels/2.x/itop-portal-base/portal/src/Form/ObjectFormManager.php +++ b/datamodels/2.x/itop-portal-base/portal/src/Form/ObjectFormManager.php @@ -1051,7 +1051,7 @@ class ObjectFormManager extends FormManager /** * @inheritDoc */ - public function CheckTransaction($aData) + public function CheckTransaction(&$aData) { $isTransactionValid = \utils::IsTransactionValid($this->oForm->GetTransactionId(), false); //The transaction token is kept in order to preserve BC with ajax forms (the second call would fail if the token is deleted). (The GC will take care of cleaning the token for us later on) if (!$isTransactionValid) { @@ -1066,8 +1066,6 @@ class ObjectFormManager extends FormManager ]; $aData['valid'] = false; } - - return $aData; } /** diff --git a/dictionaries/en.dictionary.itop.ui.php b/dictionaries/en.dictionary.itop.ui.php index 66c9c78bb..5d4a4215f 100644 --- a/dictionaries/en.dictionary.itop.ui.php +++ b/dictionaries/en.dictionary.itop.ui.php @@ -465,7 +465,7 @@ Dict::Add('EN US', 'English', 'English', array( 'UI:Error:InvalidDashboard' => 'Error: invalid dashboard', 'UI:Error:MaintenanceMode' => 'Application is currently in maintenance', 'UI:Error:MaintenanceTitle' => 'Maintenance', - 'UI:Error:InvalidToken' => 'Error: the requested operation have already been performed (CSRF token not found)', + 'UI:Error:InvalidToken' => 'Error: the requested operation has already been performed (CSRF token not found)', 'UI:GroupBy:Count' => 'Count', 'UI:GroupBy:Count+' => 'Number of elements', diff --git a/sources/form/formmanager.class.inc.php b/sources/form/formmanager.class.inc.php index cb6e15f2d..4d87b6a23 100644 --- a/sources/form/formmanager.class.inc.php +++ b/sources/form/formmanager.class.inc.php @@ -175,7 +175,7 @@ abstract class FormManager ), ); - $aData = $this->CheckTransaction($aData); + $this->CheckTransaction($aData); return $aData; } @@ -187,7 +187,7 @@ abstract class FormManager * * @since 2.7.4 3.0.0 N°3430 */ - public function CheckTransaction($aData) + public function CheckTransaction(&$aData) { $isTransactionValid = \utils::IsTransactionValid($this->oForm->GetTransactionId(), false); //The transaction token is kept in order to preserve BC with ajax forms (the second call would fail if the token is deleted). (The GC will take care of cleaning the token for us later on) if (!$isTransactionValid) { @@ -196,8 +196,6 @@ abstract class FormManager ]; $aData['valid'] = false; } - - return $aData; } /**