ElyBy-Mirror/accounts
1
0
Fork 0
mirror of https://github.com/elyby/accounts.git synced 2024-12-26 15:10:22 +05:30
Code Issues Packages Projects Releases Wiki Activity

Фикс для формы смены ника (поломалась логика, когда ник не менялся)

Browse Source
This commit is contained in:
ErickSkrauch 2016-10-27 01:22:39 +03:00
parent 5a88229c2d
commit 8049d3c774
1 changed files with 4 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
api/models/profile/ChangeUsernameForm.php
View File

@ -30,7 +30,10 @@ class ChangeUsernameForm extends ApiForm {
public function validateUsername($attribute) {
$account = new Account();
$account->id = $this->getAccount()->id;
$account->username = $this->$attribute;
// Это чтобы unique validator учёл, что ник может быть забит на текущий аккаунт
$account->setIsNewRecord(false);
if (!$account->validate(['username'])) {
$this->addErrors($account->getErrors());
}
@ -46,7 +49,7 @@ class ChangeUsernameForm extends ApiForm {
$oldNickname = $account->username;
try {
$account->username = $this->username;
if (!$account->save()) {
if (!$account->save(false)) {
throw new ErrorException('Cannot save account model with new username');
}