fix: Fixed an issue where a "Integrity constraint violation" error co…

…uld occurr if malformed `redirectSrcUrl`s was introduced into the statistics table ([#305](#305))
nystudio107 · Jun 19, 2024 · 2f67754 · 2f67754
1 parent 8a2f744
commit 2f67754
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/src/services/Statistics.php b/src/services/Statistics.php
@@ -189,13 +189,13 @@ public function incrementStatistics(string $url, bool $handled = false, $siteId
         }
         // Normalize the $url via the validator
         $stats = new StatsModel([
-            'redirectSrcUrl' => $url,
+            'redirectSrcUrl' => TextHelper::cleanupText($url),
         ]);
         $stats->validate();
         // Find any existing retour_stats record
         $statsConfig = (new Query())
             ->from(['{{%retour_stats}}'])
-            ->where(['redirectSrcUrl' => TextHelper::cleanupText($stats->redirectSrcUrl)])
+            ->where(['redirectSrcUrl' => $stats->redirectSrcUrl])
             ->one();
         // If no record is found, initialize some values
         if ($statsConfig === null) {