make sure migrations are not published twice

Author: freekmurze

.travis.yml

@@ -6,10 +6,6 @@ php:
   - 5.6
   - hhvm
   - 7
-  
-matrix:
-  allow_failures:
-    - php: 7  
 
 before_script:
   - travis_retry composer self-update

CHANGELOG.md

@@ -2,6 +2,9 @@
 
 All Notable changes to `activitylog` will be documented in this file
 
+### 2.4.2 - 2016-03-23
+- Make sure the migrations are not published twice
+
 ### 2.4.1
 - Added `userModel` configuration
 

src/Spatie/Activitylog/ActivitylogServiceProvider.php

@@ -20,17 +20,15 @@ public function boot()
     {
         // Publish a config file
         $this->publishes([
-            __DIR__.'/../../config/activitylog.php' => config_path('activitylog.php'),
+            __DIR__ . '/../../config/activitylog.php' => config_path('activitylog.php'),
         ], 'config');
 
-        $files = glob(database_path('/migrations/*_create_activity_log_table.php'));
-
-        if (! $files) {
-            // Publish your migrations
+        if (!$this->migrationHasAlreadyBeenPublished()) {
+            // Publish migration
             $timestamp = date('Y_m_d_His', time());
 
             $this->publishes([
-                __DIR__.'/../../migrations/create_activity_log_table.stub' => database_path('/migrations/'.$timestamp.'_create_activity_log_table.php'),
+                __DIR__ . "/../../migrations/create_activity_log_table.stub" => database_path("/migrations/{$timestamp}_create_activity_log_table.php"),
             ], 'migrations');
         }
     }
@@ -60,4 +58,14 @@ public function provides()
     {
         return [];
     }
+
+    /**
+     * @return bool
+     */
+    protected function migrationHasAlreadyBeenPublished()
+    {
+        $files = glob(database_path('/migrations/*_create_activity_log_table.php'));
+
+        return count($files) > 0;
+    }
 }