Revert secondsInQueue to use updated_at (#459)

* add more logs and update secondsInQueue (#456)

Co-authored-by: Sneer-ra2 <[email protected]>

* set AutoSaveInterval to 0

* update secondsInQueue to use updated_at

---------

Co-authored-by: Sneer-ra2 <[email protected]>

Author: alexp8

cncnet-api/app/Extensions/Qm/Matchup/PlayerMatchupHandler.php

@@ -54,7 +54,6 @@ public function matchup(): void
         if ($matchableOpponents->count() < $numberOfOpponentsNeeded)
         {
             Log::debug("FindOpponent ** Not enough players for match yet, ladder: $ladder->abbreviation, player: $playerName");
-            $this->qmPlayer->touch();
             return;
         }
 
@@ -78,7 +77,6 @@ public function matchup(): void
         if (count($commonQmMaps) < 1)
         {
             Log::info("FindOpponent ** No common maps available for ladder: $ladder->abbreviation, player: $playerName");
-            $this->qmPlayer->touch();
             return;
         }
 

cncnet-api/app/Extensions/Qm/Matchup/TeamMatchupHandler.php

@@ -78,7 +78,6 @@ public function matchup(): void
         {
             Log::info("  ❌ MATCH FAILED: Not enough players ({$matchableOpponentsCount}/{$numberOfOpponentsNeeded} needed)");
             Log::info("=== TeamMatchup END for {$playerInQueue} - NO MATCH ===\n");
-            $this->qmPlayer->touch();
             return;
         }
 
@@ -120,7 +119,6 @@ public function matchup(): void
         {
             Log::info("  ❌ MATCH FAILED: No common maps available between all players");
             Log::info("=== TeamMatchup END for {$playerInQueue} - NO MATCH ===\n");
-            $this->qmPlayer->touch();
             return;
         }
 

cncnet-api/app/Http/Services/QuickMatchService.php

@@ -355,8 +355,8 @@ public function getMatchableOpponents(QmQueueEntry $currentQmQueueEntry, Collect
                 continue;
             }
 
-            // (updated_at - created_at) / 60 = seconds duration player has been waiting in queue
-            $pointsTime = ((strtotime($currentQmQueueEntry->updated_at) - strtotime($currentQmQueueEntry->created_at))) * $ladder->qmLadderRules->points_per_second;
+            // Calculate wait time bonus using secondsinQueue()
+            $pointsTime = $currentQmQueueEntry->secondsinQueue() * $ladder->qmLadderRules->points_per_second;
 
             // is the opponent within the point filter
             if ($pointsTime + $ladder->qmLadderRules->max_points_difference > abs($currentQmQueueEntry->points - $opponent->points))
@@ -460,8 +460,8 @@ public function getEntriesInPointRange(QmQueueEntry $currentQmQueueEntry, Collec
                 continue;
             }
 
-            // (updated_at - created_at) / 60 = seconds duration player has been waiting in queue
-            $pointsTime = ((strtotime($currentQmQueueEntry->updated_at) - strtotime($currentQmQueueEntry->created_at))) * $pointsPerSecond;
+            // Calculate wait time bonus using secondsinQueue()
+            $pointsTime = $currentQmQueueEntry->secondsinQueue() * $pointsPerSecond;
 
             // is the opponent within the point filter
             if ($pointsTime + $maxPointsDifference > abs($currentQmQueueEntry->points - $opponent->points))

cncnet-api/app/Models/QmQueueEntry.php

@@ -19,14 +19,8 @@ public function ladderHistory()
 
     public function secondsinQueue()
     {
-        $createdAt = $this->created_at;
-
-        // Convert timestamp to Carbon instance
-        $createdAtCarbon = Carbon::parse($createdAt);
-
-        // Calculate the difference in seconds from created_at to NOW
-        $secondsDifference = now()->diffInSeconds($createdAtCarbon);
-
-        return $secondsDifference;
+        // Calculate the difference between updated_at and created_at
+        // updated_at is touched by matchup handlers when no match is found
+        return $this->updated_at->diffInSeconds($this->created_at);
     }
 }