Skip to content

Commit 17dc131

Browse files
author
Yuri Kovsher
committed
MAGETWO-34174: CI builds stabilization
1 parent feceb6f commit 17dc131

File tree

1 file changed

+17
-9
lines changed
  • app/code/Magento/User/Controller/Adminhtml/User

1 file changed

+17
-9
lines changed

app/code/Magento/User/Controller/Adminhtml/User/Save.php

Lines changed: 17 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -66,16 +66,24 @@ public function execute()
6666
} catch (\Magento\Framework\Validator\ValidatorException $e) {
6767
$messages = $e->getMessages();
6868
$this->messageManager->addMessages($messages);
69-
70-
if (empty($messages)) {
71-
if ($e->getMessage()) {
72-
$this->messageManager->addError($e->getMessage());
73-
}
69+
$this->redirectToEdit($model, $data);
70+
} catch (\Magento\Framework\Exception\LocalizedException $e) { //
71+
if ($e->getMessage()) {
72+
$this->messageManager->addError($e->getMessage());
7473
}
75-
$this->_getSession()->setUserData($data);
76-
$arguments = $model->getId() ? ['user_id' => $model->getId()] : [];
77-
$arguments = array_merge($arguments, ['_current' => true]);
78-
$this->_redirect('adminhtml/*/edit', $arguments);
74+
$this->redirectToEdit($model, $data);
7975
}
8076
}
77+
78+
/**
79+
* @param \Magento\User\Model\User $model
80+
* @param array $data
81+
*/
82+
protected function redirectToEdit(\Magento\User\Model\User $model, array $data)
83+
{
84+
$this->_getSession()->setUserData($data);
85+
$arguments = $model->getId() ? ['user_id' => $model->getId()] : [];
86+
$arguments = array_merge($arguments, ['_current' => true]);
87+
$this->_redirect('adminhtml/*/edit', $arguments);
88+
}
8189
}

0 commit comments

Comments
 (0)