更新

server/app/adminapi/logic/stats/ConversionLogic.php

@@ -966,6 +966,7 @@ class ConversionLogic
      * - 加粉之后、统计结束前须有 msg_audit_approved（排除未完成链路的幽灵事件）；
      * - 加粉之后、统计结束前若出现 del_external_contact（客户从企微侧删除）则不计。
      *   不扣减 del_follow_user：企微客户列表中改名带「删」的客户往往仍在列表中；
+     * - 剔除非投放加粉：跟进人 add_way∈{1 扫一扫, 2 搜索手机号, 3 名片分享}；
      * - 剔除继承客户：跟进人 add_way∈{201 内部成员共享, 202 管理员/负责人分配}（含在职/离职继承）。
      *
      * @param array<string, mixed>|null $mediaChannel
@@ -989,7 +990,7 @@ class ConversionLogic
             }
         }
 
-        $baseKey = self::requestRowsCacheKey('fans-effective-v4', [
+        $baseKey = self::requestRowsCacheKey('fans-effective-v6', [
             $startTimestamp,
             $endTimestamp,
             self::mediaChannelCacheKey($mediaChannel),
@@ -1070,7 +1071,7 @@ class ConversionLogic
         }
 
         $pairs = $query->select()->toArray();
-        $pairs = self::excludeInheritedFanPairs($pairs);
+        $pairs = self::excludeUncountedFanPairs($pairs);
 
         $countsByUser = [];
         foreach ($pairs as $pair) {
@@ -1095,13 +1096,17 @@ class ConversionLogic
     }
 
     /**
-     * 剔除企微「继承/分配」客户：跟进人 add_way 为 201（内部成员共享）或 202（管理员/负责人分配，含在职/离职继承）。
+     * 剔除不应计入加粉的跟进来源：
+     * - add_way=1 扫一扫（客户通过扫一扫添加）；
+     * - add_way=2 搜索手机号（成员通过搜索手机号添加）；
+     * - add_way=3 名片分享（客户通过名片分享添加）；
+     * - add_way=201/202 继承/分配（内部成员共享、管理员/负责人分配，含在职/离职继承）。
      * 无本地客户档案或跟进信息不含该员工时保守保留（无法判定则仍计加粉）。
      *
      * @param array<int, array<string, mixed>> $pairs
      * @return array<int, array<string, mixed>>
      */
-    private static function excludeInheritedFanPairs(array $pairs): array
+    private static function excludeUncountedFanPairs(array $pairs): array
     {
         if ($pairs === []) {
             return [];
@@ -1119,8 +1124,8 @@ class ConversionLogic
             return $pairs;
         }
 
-        /** @var array<string, true> $inheritedKeys user_id\0external_userid */
-        $inheritedKeys = [];
+        /** @var array<string, true> $excludedKeys user_id\0external_userid */
+        $excludedKeys = [];
         foreach (array_chunk($externalIdList, 500) as $chunk) {
             $contactRows = Db::name('qywx_external_contact')
                 ->whereIn('external_userid', $chunk)
@@ -1145,19 +1150,19 @@ class ConversionLogic
                         continue;
                     }
                     $addWay = (int) ($fu['add_way'] ?? $fu['AddWay'] ?? 0);
-                    if ($addWay !== 201 && $addWay !== 202) {
+                    if (!in_array($addWay, [1, 2, 3, 201, 202], true)) {
                         continue;
                     }
                     $followUserId = trim((string) ($fu['userid'] ?? $fu['UserId'] ?? ''));
                     if ($followUserId === '') {
                         continue;
                     }
-                    $inheritedKeys[$followUserId . "\0" . $extId] = true;
+                    $excludedKeys[$followUserId . "\0" . $extId] = true;
                 }
             }
         }
 
-        if ($inheritedKeys === []) {
+        if ($excludedKeys === []) {
             return $pairs;
         }
 
@@ -1168,7 +1173,7 @@ class ConversionLogic
             if ($userId === '' || $extId === '') {
                 continue;
             }
-            if (isset($inheritedKeys[$userId . "\0" . $extId])) {
+            if (isset($excludedKeys[$userId . "\0" . $extId])) {
                 continue;
             }
             $kept[] = $pair;