Fix bug when creating slug from null value

CHANGELOG.md

@@ -2,6 +2,10 @@
 
 All Notable changes to `laravel-sluggable` will be documented in this file
 
+## 1.0.2 - 2016-01-12
+
+- Fix bug when creating slugs from null values
+
 ## 1.0.1 - 2015-12-27
 
 - Fix Postgres bug

src/HasSlug.php

@@ -56,9 +56,10 @@ protected function addSlug()
     protected function getSlugSourceString() : string
     {
         $slugSourceString = collect($this->slugOptions->generateSlugFrom)
-            ->map(function (string $fieldName) : string {
-                return $this->$fieldName;
-            })
+            ->map(function (string $fieldName) : string
+{
+    return $this->$fieldName ?? '';
+})
             ->implode('-');
 
         return substr($slugSourceString, 0, $this->slugOptions->maximumLength);

tests/Integration/HasSlugTest.php

@@ -16,6 +16,16 @@ public function it_will_save_a_slug_when_saving_a_model()
         $this->assertEquals('this-is-a-test', $model->url);
     }
 
+    /**
+     * @test
+     */
+    public function it_can_handle_null_values_when_creating_slugs()
+    {
+        $model = TestModel::create(['name' => null]);
+
+        $this->assertEquals('-1', $model->url);
+    }
+
     /**
      * @test
      */
@@ -73,12 +83,12 @@ public function it_can_generate_duplicate_slugs()
     {
         foreach (range(1, 10) as $i) {
             $model = new class extends TestModel
-            {
-                public function getSlugOptions() : SlugOptions
-                {
-                    return parent::getSlugOptions()->allowDuplicateSlugs();
-                }
-            };
+ {
+     public function getSlugOptions() : SlugOptions
+     {
+         return parent::getSlugOptions()->allowDuplicateSlugs();
+     }
+ };
 
             $model->name = 'this is a test';
             $model->save();
@@ -93,12 +103,12 @@ public function getSlugOptions() : SlugOptions
     public function it_can_generate_slugs_with_a_maximum_length()
     {
         $model = new class extends TestModel
-        {
-            public function getSlugOptions() : SlugOptions
-            {
-                return parent::getSlugOptions()->slugsShouldBeNoLongerThan(5);
-            }
-        };
+ {
+     public function getSlugOptions() : SlugOptions
+     {
+         return parent::getSlugOptions()->slugsShouldBeNoLongerThan(5);
+     }
+ };
 
         $model->name = '123456789';
         $model->save();

tests/Integration/TestCase.php

@@ -45,7 +45,7 @@ protected function setUpDatabase(Application $app)
 
         $app['db']->connection()->getSchemaBuilder()->create('test_models', function (Blueprint $table) {
             $table->increments('id');
-            $table->string('name');
+            $table->string('name')->nullable();
             $table->string('other_field')->nullable();
             $table->string('url')->nullable();
         });