Merge pull request #3 from dharple/master

Fix issues encountered during implementation

Author: infinitypaul

Diff for: README.md

@@ -34,7 +34,7 @@ This package will observe the created and updated event of the models (check the
 In Your Form Request or Inline Validation, All You Need To Do Is Instantiate The `NotFromPasswordHistory` class passing the current user as an argument
 ``` php
 <?php
-use Infinitypaul\LaravelPasswordHistoryValidation\Models\PasswordHistoryRepo;
+use Infinitypaul\LaravelPasswordHistoryValidation\Rules\NotFromPasswordHistory;
 
 $this->validate($request, [
             'password' => [

Diff for: src/LaravelPasswordHistoryValidationServiceProvider.php

@@ -15,7 +15,7 @@ public function boot()
     {
         if ($this->app->runningInConsole()) {
             $this->publishes([
-                __DIR__.'/../config/passwordHistory.php' => config_path('password-history-validation.php'),
+                __DIR__.'/../config/passwordHistory.php' => config_path('password-history.php'),
             ], 'password-config');
             $this->loadMigrationsFrom(__DIR__.'/../database/migrations');
 

Diff for: src/Traits/PasswordHistoryTrait.php

@@ -17,8 +17,20 @@ public function passwordHistory()
 
     public function deletePasswordHistory()
     {
+        $keep = config('password-history.keep');
+        $ids = $this->passwordHistory()
+            ->pluck('id')
+            ->sort()
+            ->reverse();
+
+        if ($ids->count() < $keep) {
+            return;
+        }
+
+        $delete = $ids->splice($keep);
+
         $this->passwordHistory()
-            ->where('id', '<=', $this->passwordHistory()->first()->id - config('password-history.keep'))
+            ->whereIn('id', $delete)
             ->delete();
     }
 }