Fixes #6514: advanced application SignupForm::signup() should not return User…

Fixes #6514: advanced application SignupForm::signup() should not return User model if saving failed

Fixes #6514: advanced application SignupForm::signup() should not return User…
46746dac · Alexander Makarov · e75e233d · 46746dac
Commit 46746dac authored Dec 14, 2014 by Alexander Makarov
Hide whitespace changes
Inline Side-by-side

Showing with 3 additions and 2 deletions

SignupForm.php apps/advanced/frontend/models/SignupForm.php +3 -2

No files found.
--- a/apps/advanced/frontend/models/SignupForm.php
+++ b/apps/advanced/frontend/models/SignupForm.php
@@ -48,8 +48,9 @@ class SignupForm extends Model
            $user->email = $this->email;
            $user->setPassword($this->password);
            $user->generateAuthKey();
-            $user->save();
-            return $user;
+            if ($user->save()) {
+                return $user;
+            }
        }

        return null;