Call AFTER_CONFIRMATION event only if confirmation actually happens

andmemasin · Nov 20, 2017 · bf3b170 · bf3b170
1 parent 4978d33
commit bf3b170
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/controllers/RegistrationController.php b/controllers/RegistrationController.php
@@ -211,9 +211,10 @@ public function actionConfirm($id, $code)
 
         $this->trigger(self::EVENT_BEFORE_CONFIRM, $event);
 
-        $user->attemptConfirmation($code);
+        if($user->attemptConfirmation($code)){
+            $this->trigger(self::EVENT_AFTER_CONFIRM, $event);
+        }
 
-        $this->trigger(self::EVENT_AFTER_CONFIRM, $event);
 
         return $this->render('/message', [
             'title'  => \Yii::t('user', 'Account confirmation'),