fix bug

toplan · Dec 5, 2016 · eccd20b · eccd20b
1 parent 9464863
commit eccd20b
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 7 deletions.
diff --git a/composer.json b/composer.json
@@ -2,7 +2,7 @@
     "name": "toplan/laravel-sms",
     "description": "sms send package for laravel",
     "license": "MIT",
-    "version": "2.5.1",
+    "version": "2.5.2",
     "keywords": ["sms", "laravel sms", "laravel-sms"],
     "authors": [
         {

diff --git a/migrations/2015_12_21_111514_create_sms_table.php b/migrations/2015_12_21_111514_create_sms_table.php
@@ -41,7 +41,7 @@ public function up()
             $table->integer('sent_time')->unsigned()->default(0);
 
             //代理器使用日志，记录每个代理器的发送状态，可用于排错
-            $table->text('result_info');
+            $table->text('result_info')->nullable();
 
             $table->timestamps();
             $table->softDeletes();

diff --git a/src/Toplan/LaravelSms/validations.php b/src/Toplan/LaravelSms/validations.php
@@ -12,11 +12,13 @@
 
 Validator::extend('verify_code', function ($attribute, $value) {
     $state = SmsManager::retrieveState();
-    $maxAttempts = config('laravel-sms.verifyCode.maxAttempts', 0);
-    $attempts = $state['attempts'] + 1;
-    SmsManager::updateState('attempts', $attempts);
-    if ($maxAttempts > 0 && $attempts > $maxAttempts) {
-        return false;
+    if (isset($state['attempts'])) {
+        $maxAttempts = config('laravel-sms.verifyCode.maxAttempts', 0);
+        $attempts = $state['attempts'] + 1;
+        SmsManager::updateState('attempts', $attempts);
+        if ($maxAttempts > 0 && $attempts > $maxAttempts) {
+            return false;
+        }
     }
 
     return $state && $state['deadline'] >= time() && $state['code'] === $value;