GH-230 Move triggering referral task to an util

mdziekon · Jul 3, 2022 · b35b26c · b35b26c
1 parent e3ea5a9
commit b35b26c
Show file tree

Hide file tree

Showing 3 changed files with 39 additions and 19 deletions.
diff --git a/modules/overview/_includes.php b/modules/overview/_includes.php
@@ -9,6 +9,7 @@
     include($includePath . './screens/FirstLogin/FirstLogin.screen.php');
     include($includePath . './screens/FirstLogin/utils/effects/handleProxyDetection.effect.php');
     include($includePath . './screens/FirstLogin/utils/effects/handleReferralMultiAccountDetection.effect.php');
+    include($includePath . './screens/FirstLogin/utils/effects/triggerUserReferralTask.effect.php');
     include($includePath . './screens/FirstLogin/utils/helpers/getReferrerTasksData.helper.php');
 
 });

diff --git a/modules/overview/screens/FirstLogin/utils/effects/triggerUserReferralTask.effect.php b/modules/overview/screens/FirstLogin/utils/effects/triggerUserReferralTask.effect.php
@@ -0,0 +1,35 @@
+<?php
+
+namespace UniEngine\Engine\Modules\Overview\Screens\FirstLogin\Utils\Effects;
+
+/**
+ * @param array $params
+ * @param arrayRef $params['referringUserWithTasksData']
+ */
+function triggerUserReferralTask($props) {
+    $referringUserWithTasksData = &$props['referringUserWithTasksData'];
+
+    if (empty($referringUserWithTasksData)) {
+        return;
+    }
+
+    Tasks_TriggerTask(
+        $referringUserWithTasksData,
+        'NEWUSER_REGISTER',
+        [
+            'mainCheck' => function ($JobArray, $ThisCat, $TaskID, $JobID) use (&$referringUserWithTasksData) {
+                $taskStatusCheckResult = Tasks_TriggerTask_MainCheck_Progressive($JobArray, $ThisCat, $TaskID, $JobID, $referringUserWithTasksData, 1);
+
+                $referringUserWithTasksData['TaskData'][] = [
+                    'TaskID' => $TaskID,
+                    'TaskStatus' => $referringUserWithTasksData['tasks_done_parsed']['status'][$ThisCat][$TaskID][$JobID],
+                    'TaskLimit' => $JobArray[$JobArray['statusField']]
+                ];
+
+                return $taskStatusCheckResult;
+            }
+        ]
+    );
+}
+
+?>
diff --git a/overview.php b/overview.php
@@ -66,25 +66,9 @@
             'referredById' => $_User['referred'],
         ]);
 
-        if(!empty($referringUserWithTasksData))
-        {
-            Tasks_TriggerTask($referringUserWithTasksData, 'NEWUSER_REGISTER', array
-            (
-                'mainCheck' => function($JobArray, $ThisCat, $TaskID, $JobID) use (&$referringUserWithTasksData)
-                {
-                    $Return = Tasks_TriggerTask_MainCheck_Progressive($JobArray, $ThisCat, $TaskID, $JobID, $referringUserWithTasksData, 1);
-                    $referringUserWithTasksData['TaskData'][] = array
-                    (
-                        'TaskID' => $TaskID,
-                        'TaskStatus' => $referringUserWithTasksData['tasks_done_parsed']['status'][$ThisCat][$TaskID][$JobID],
-                        'TaskLimit' => $JobArray[$JobArray['statusField']]
-                    );
-                    return $Return;
-                }
-            ));
-        }
-
-        // Check IP Intersection
+        Overview\Screens\FirstLogin\Utils\Effects\triggerUserReferralTask([
+            'referringUserWithTasksData' => &$referringUserWithTasksData,
+        ]);
         Overview\Screens\FirstLogin\Utils\Effects\handleReferralMultiAccountDetection([
             'user' => &$_User,
             'referredById' => $_User['referred'],