[9.x] Fix incrementing string keys

src/Illuminate/Database/Eloquent/Concerns/HasUlids.php

@@ -55,4 +55,18 @@ public function getKeyType()
 
         return $this->keyType;
     }
+
+    /**
+     * Get the value indicating whether the IDs are incrementing.
+     *
+     * @return bool
+     */
+    public function getIncrementing()
+    {
+        if (in_array($this->getKeyName(), $this->uniqueIds())) {
+            return false;
+        }
+
+        return $this->incrementing;
+    }
 }

src/Illuminate/Database/Eloquent/Concerns/HasUuids.php

@@ -55,4 +55,18 @@ public function getKeyType()
 
         return $this->keyType;
     }
+
+    /**
+     * Get the value indicating whether the IDs are incrementing.
+     *
+     * @return bool
+     */
+    public function getIncrementing()
+    {
+        if (in_array($this->getKeyName(), $this->uniqueIds())) {
+            return false;
+        }
+
+        return $this->incrementing;
+    }
 }

src/Illuminate/Database/Eloquent/Model.php

@@ -1851,10 +1851,6 @@ public function setKeyType($type)
      */
     public function getIncrementing()
     {
-        if ($this->getKeyType() === 'string') {
-            return false;
-        }
-
         return $this->incrementing;
     }
 

tests/Database/DatabaseEloquentModelTest.php

@@ -2216,10 +2216,9 @@ public function testIntKeyTypePreserved()
     public function testStringKeyTypePreserved()
     {
         $model = $this->getMockBuilder(EloquentKeyTypeModelStub::class)->onlyMethods(['newModelQuery', 'updateTimestamps', 'refresh'])->getMock();
-        $model->id = 'string id';
 
         $query = m::mock(Builder::class);
-        $query->shouldReceive('insert')->once()->with(['id' => 'string id']);
+        $query->shouldReceive('insertGetId')->once()->with([], 'id')->andReturn('string id');
         $query->shouldReceive('getConnection')->once();
         $model->expects($this->once())->method('newModelQuery')->willReturn($query);