fix bug in --force-periods logic

refs #319
refs #5396

core/CronArchive.php

@@ -377,10 +377,6 @@ private function archiveSingleSite($idSite, $requestsBefore)
             $skipDayArchive = false;
         }
 
-        if (!$this->shouldProcessPeriod("day")) {
-            $skipDayArchive = true;
-        }
-
         if ($skipDayArchive) {
             $this->log("Skipped website id $idSite, already done "
                 . \Piwik\MetricsFormatter::getPrettyTimeFromSeconds($elapsedSinceLastArchiving, true, $isHtml = false)
@@ -483,6 +479,11 @@ private function initSegmentsToArchive()
      */
     protected function processArchiveDays($idSite, $lastTimestampWebsiteProcessedDay, $shouldArchivePeriods, Timer $timerWebsite)
     {
+        if (!$this->shouldProcessPeriod("day")) {
+            // skip day archiving and proceed to period processing
+            return true;
+        }
+
         // Fake that the request is already done, so that other core:archive commands
         // running do not grab the same website from the queue
         Option::set($this->lastRunKey($idSite, "day"), time());