Apparently the segment needs to be double encoded when sent in the climulti:request command. (#16947)

* Apparently the segment needs to be double encoded when sent in the climulti:request command.

* add test

* add expected

* fix test

1 files changed, 2 insertions, 2 deletions

diff --git a/core/CronArchive.php b/core/CronArchive.php
index 477720a892..790a0cb552 100644
--- a/core/CronArchive.php
+++ b/core/CronArchive.php
@@ -561,7 +561,7 @@ class CronArchive
         $visits = (int) $visits;
 
         $this->logger->info("Archived website id {$params['idSite']}, period = {$params['period']}, date = "
-            . "{$params['date']}, segment = '" . (isset($params['segment']) ? $params['segment'] : '') . "', "
+            . "{$params['date']}, segment = '" . (isset($params['segment']) ? urldecode($params['segment']) : '') . "', "
             . ($plugin ? "plugin = $plugin, " : "") . ($report ? "report = $report, " : "") . "$visits visits found. $timer");
     }
 
@@ -654,7 +654,7 @@ class CronArchive
     {
         $request = "?module=API&method=CoreAdminHome.archiveReports&idSite=$idSite&period=$period&date=" . $date . "&format=json";
         if ($segment) {
-            $request .= '&segment=' . urlencode($segment);
+            $request .= '&segment=' . urlencode(urlencode($segment));
         }
         if (!empty($plugin)) {
             $request .= "&plugin=" . $plugin;