Merge pull request #713 from pomaxa/rand

mt_rand instead of rand()

Author: fzaninotto

src/Faker/ORM/Doctrine/EntityPopulator.php

@@ -114,16 +114,19 @@ public function guessColumnFormatters(\Faker\Generator $generator)
 
             $index = 0;
             $formatters[$assocName] = function ($inserted) use ($relatedClass, &$index, $unique, $optional) {
-                if ($unique && isset($inserted[$relatedClass])) {
-                    $related = null;
-                    if (isset($inserted[$relatedClass][$index]) || !$optional) {
-                        $related = $inserted[$relatedClass][$index];
-                    }
 
-                    $index++;
+                if (isset($inserted[$relatedClass])) {
+                    if ($unique) {
+                        $related = null;
+                        if (isset($inserted[$relatedClass][$index]) || !$optional) {
+                            $related = $inserted[$relatedClass][$index];
+                        }
+
+                        $index++;
+
+                        return $related;
+                    }
 
-                    return $related;
-                } elseif (isset($inserted[$relatedClass])) {
                     return $inserted[$relatedClass][mt_rand(0, count($inserted[$relatedClass]) - 1)];
                 }
 
@@ -186,7 +189,7 @@ private function generateId($obj, $column, EntityManagerInterface $manager)
 
         $id = null;
         do {
-            $id = rand();
+            $id = mt_rand();
         } while (in_array($id, $ids));
 
         return $id;