From 1b2df8d4c39de5cbf9afd1a0ba55bea4651a2f98 Mon Sep 17 00:00:00 2001
From: Alexey Skobkin <x@skobkin.ru>
Date: Sat, 14 Jan 2017 03:58:18 +0300
Subject: [PATCH] #35 done. LoadSubscribersData are now non-random.

---
 .../DataFixtures/ORM/LoadSubscribersData.php  | 56 +++++--------------
 1 file changed, 13 insertions(+), 43 deletions(-)

diff --git a/src/Skobkin/Bundle/PointToolsBundle/DataFixtures/ORM/LoadSubscribersData.php b/src/Skobkin/Bundle/PointToolsBundle/DataFixtures/ORM/LoadSubscribersData.php
index 8838f3c..942a0f5 100644
--- a/src/Skobkin/Bundle/PointToolsBundle/DataFixtures/ORM/LoadSubscribersData.php
+++ b/src/Skobkin/Bundle/PointToolsBundle/DataFixtures/ORM/LoadSubscribersData.php
@@ -18,22 +18,22 @@ class LoadSubscribersData extends AbstractFixture implements OrderedFixtureInter
     {
         /** @var User[] $users */
         $users = [
-            $this->getReference('test_user_99999'),
-            $this->getReference('test_user_99998'),
-            $this->getReference('test_user_99997'),
-            $this->getReference('test_user_99996'),
-            $this->getReference('test_user_99995'),
+            99999 => $this->getReference('test_user_99999'),
+            99998 => $this->getReference('test_user_99998'),
+            99997 => $this->getReference('test_user_99997'),
+            99996 => $this->getReference('test_user_99996'),
+            99995 => $this->getReference('test_user_99995'),
+        ];
+
+        $subscriptions = [
+            99999 => [99998, 99997, 99996, 99995],
+            99998 => [99999, 99997],
+            99997 => [99999],
         ];
 
         foreach ($users as $key => $user) {
-            // At least 2 subscribers for first user in the list
-            if (0 === $key) {
-                $minimum = 2;
-            } else {
-                $minimum = random_int(0, count($users));
-            }
-
-            foreach ($this->getRandomSubscribers($users, $minimum) as $subscriber) {
+            foreach ($subscriptions[$key] as $userId) {
+                $subscriber = $users[$userId];
                 $subscription = new Subscription($user, $subscriber);
                 $subscriptionEvent = new SubscriptionEvent($user, $subscriber, SubscriptionEvent::ACTION_SUBSCRIBE);
                 $om->persist($subscription);
@@ -49,34 +49,4 @@ class LoadSubscribersData extends AbstractFixture implements OrderedFixtureInter
     {
         return 4;
     }
-
-    /**
-     * Returns array with random users from given users array
-     *
-     * @param User[] $users
-     * @param int $min
-     *
-     * @return User[]
-     */
-    private function getRandomSubscribers(array $users, int $min = 0): array
-    {
-        if (0 === $number = mt_rand($min, count($users))) {
-            return [];
-        }
-
-        $keys = array_rand($users, $number);
-
-        // If array_rand was called with $number = 1
-        if (!is_array($keys)) {
-            $keys = [$keys];
-        }
-
-        $result = [];
-
-        foreach ($keys as $key) {
-            $result[] = $users[$key];
-        }
-
-        return $result;
-    }
 }
\ No newline at end of file