Refs #7560, if old list is found in DB for ArchivesToPurgeDistributedList, convert it to new list w/ only year-date to avoid error.

1 files changed, 23 insertions, 0 deletions

diff --git a/plugins/CoreAdminHome/Tasks/ArchivesToPurgeDistributedList.php b/plugins/CoreAdminHome/Tasks/ArchivesToPurgeDistributedList.php
index 6a30d6348a..ef5c7f533c 100644
--- a/plugins/CoreAdminHome/Tasks/ArchivesToPurgeDistributedList.php
+++ b/plugins/CoreAdminHome/Tasks/ArchivesToPurgeDistributedList.php
@@ -41,6 +41,13 @@ class ArchivesToPurgeDistributedList extends DistributedList
         parent::setAll($yearMonths);
     }
 
+    protected function getListOptionValue()
+    {
+        $result = parent::getListOptionValue();
+        $this->convertOldDistributedList($result);
+        return $result;
+    }
+
     public function getAllAsDates()
     {
         $dates = array();
@@ -61,4 +68,20 @@ class ArchivesToPurgeDistributedList extends DistributedList
         $yearMonth = $date->toString('Y_m');
         $this->remove($yearMonth);
     }
+
+    /**
+     * Before 2.12.0 Piwik stored this list as an array mapping year months to arrays of site IDs. If this is
+     * found in the DB, we convert the array to an array of year months to avoid errors and to make sure
+     * the correct tables are still purged.
+     */
+    private function convertOldDistributedList(&$yearMonths)
+    {
+        foreach ($yearMonths as $key => $value) {
+            if (preg_match("/^[0-9]{4}_[0-9]{2}$/", $key)) {
+                unset($yearMonths[$key]);
+
+                $yearMonths[] = $key;
+            }
+        }
+    }
 }
\ No newline at end of file