Merge pull request #204 from viktorprogger/v0.9.x

Fix automatic vertex ID generation when using vertex IDs with strings

Author: clue

src/Graph.php

@@ -289,7 +289,7 @@ private function getNextId()
         }
 
         // auto ID
-        return max(array_keys($this->verticesStorage))+1;
+        return max(array_map('intval', array_keys($this->verticesStorage)))+1;
     }
 
     /**

tests/GraphTest.php

@@ -9,6 +9,9 @@
 
 class GraphTest extends AbstractAttributeAwareTest
 {
+    /** @var Graph */
+    private $graph;
+
     /**
      * @before
      */
@@ -17,6 +20,17 @@ public function setUpGraph()
         $this->graph = new Graph();
     }
 
+    public function testMixedIds()
+    {
+        $vertexString = $this->graph->createVertex('test');
+        $vertex1 = $this->graph->createVertex();
+        $vertex2 = $this->graph->createVertex();
+
+        self::assertEquals('test', $vertexString->getId());
+        self::assertEquals(1, $vertex1->getId());
+        self::assertEquals(2, $vertex2->getId());
+    }
+
     public function testVertexClone()
     {
         $graph = new Graph();