fix: require phone number when verify sms code

leancloud · May 10, 2024 · 6342c29 · 6342c29
1 parent 7143b44
commit 6342c29
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 8 deletions.
diff --git a/src/LeanCloud/User.php b/src/LeanCloud/User.php
@@ -441,9 +441,11 @@ public static function requestPasswordResetBySmsCode($phoneNumber) {
      * @param string $smsCode
      * @param string $newPassword
      */
-    public static function resetPasswordBySmsCode($smsCode, $newPassword) {
-        Client::put("/resetPasswordBySmsCode/{$smsCode}",
-                        array("password" => $newPassword));
+    public static function resetPasswordBySmsCode($smsCode, $newPassword, $mobilePhoneNumber) {
+        Client::put("/resetPasswordBySmsCode/{$smsCode}", array(
+          "password" => $newPassword,
+          "mobilePhoneNumber" => $mobilePhoneNumber
+        ));
     }
 
     /**
@@ -463,8 +465,10 @@ public static function requestMobilePhoneVerify($phoneNumber) {
      *
      * @param string $smsCode
      */
-    public static function verifyMobilePhone($smsCode) {
-        Client::post("/verifyMobilePhone/{$smsCode}", null);
+    public static function verifyMobilePhone($smsCode, $mobilePhoneNumber) {
+        Client::post("/verifyMobilePhone/{$smsCode}", array(
+          "mobilePhoneNumber" => $mobilePhoneNumber
+        ));
     }
 
     /**
@@ -591,4 +595,3 @@ public function unlinkWith($provider) {
     }
 
 }
-
diff --git a/test/UserTest.php b/test/UserTest.php
@@ -172,7 +172,7 @@ public function testUpdatePassword() {
     public function testVerifyMobilePhone() {
         // Ensure the post format is correct
         $this->setExpectedException("LeanCloud\CloudException", null, 603);
-        User::verifyMobilePhone("000000");
+        User::verifyMobilePhone("000000", "18612340000");
     }
 
     public function testSignUpOrLoginByMobilePhone() {
@@ -308,4 +308,3 @@ public function testFindUserWithSession() {
     }
 
 }
-