Couple assorted changes (#13935)

* Allow annotations API to accept multiple periods, so evolution graphs that use multiple periods work.

* Remove warning when rows_to_display viewdatatable config property is left at its default value.

* Allow individual cells in an html visualization to be styled (if the visualization is extended).

* Remove unneeded TODO.

* In series picker encode picked rows in case the labels have commas.

* Must decode the rows value as well (as it is not handled by API, must be done in plugin).

* Allow joins to specified through LogAggregator::queryConversionsByDimension().

* Add safety check to _idts processing: if visitor is unknown, ignore _idts value, since it is their first visit.

* In the tracker when searching by visitor ID, search through entire log_visit table instead of just in the last 30 mins.

* When tracking visitor days since first, do not round since this can result in inaccurate data when rounding up. Which can cause trouble when finding the start visit for a log.

* Allow HtmlTable descendants to add any html attributes to cells.

* Allow derived Visualizations to add custom parameters to API requests via a new RequestConfig method.

* Tweak to TODO.

* Add test for annotations API change & get to pass.

* Apply more review feedback

* Update INI config docs for window_look_back_for_visitor.

* Only copy visitor properties if action is part of an existing visit.

* Some more properties that should be copied over from known visitor even if new visit.

* Fixing some tests.

* update test

* Fix CustomEventsTest test failures.

* Fixing more tests.

* update rest of tests

* Fixing tests.

* Update some test files.

* Fix log statements.

* To better handle out of order actions, add part of last_action_time check to visitor ID search.

* Update tests.

* Updating expected screenshots.

* Fix ArchiveCronTest.

* Throw exception if idorder not unique.

* Only throw exception if idorder specified.

* Fixing a couple tests.

* Fix another test.

5 files changed, 5 insertions, 4 deletions

diff --git a/plugins/PrivacyManager/tests/Fixtures/MultipleSitesMultipleVisitsFixture.php b/plugins/PrivacyManager/tests/Fixtures/MultipleSitesMultipleVisitsFixture.php
index bea167e3b2..e858b0cefe 100644
--- a/plugins/PrivacyManager/tests/Fixtures/MultipleSitesMultipleVisitsFixture.php
+++ b/plugins/PrivacyManager/tests/Fixtures/MultipleSitesMultipleVisitsFixture.php
@@ -322,6 +322,7 @@ class MultipleSitesMultipleVisitsFixture extends Fixture
     {
         for ($day = 0; $day < $numIterations; $day++) {
             // we track over several days to make sure we have some data to aggregate in week reports
+            // NOTE: some action times are out of order in visits on purpose
 
             if ($day > 0) {
                 $this->trackingTime = Date::factory($this->dateTime)->addDay($day * 3)->getDatetime();
diff --git a/plugins/PrivacyManager/tests/System/expected/anonymizeVisitInformation_allSitesAllDates.json b/plugins/PrivacyManager/tests/System/expected/anonymizeVisitInformation_allSitesAllDates.json
index b72d6b4b03..a2d3735a74 100644
--- a/plugins/PrivacyManager/tests/System/expected/anonymizeVisitInformation_allSitesAllDates.json
+++ b/plugins/PrivacyManager/tests/System/expected/anonymizeVisitInformation_allSitesAllDates.json
@@ -11949,4 +11949,4 @@
             "name": "My product 211"
         }
     ]
-}
+}
\ No newline at end of file
diff --git a/plugins/PrivacyManager/tests/System/expected/anonymizeVisitInformation_anonymizeUserId.json b/plugins/PrivacyManager/tests/System/expected/anonymizeVisitInformation_anonymizeUserId.json
index 11f251897d..60da4c0cf4 100644
--- a/plugins/PrivacyManager/tests/System/expected/anonymizeVisitInformation_anonymizeUserId.json
+++ b/plugins/PrivacyManager/tests/System/expected/anonymizeVisitInformation_anonymizeUserId.json
@@ -4526,4 +4526,4 @@
             "name": "My product 211"
         }
     ]
-}
+}
\ No newline at end of file
diff --git a/plugins/PrivacyManager/tests/System/expected/anonymizeVisitInformation_restrictDate.json b/plugins/PrivacyManager/tests/System/expected/anonymizeVisitInformation_restrictDate.json
index 499ae92c76..9280745a96 100644
--- a/plugins/PrivacyManager/tests/System/expected/anonymizeVisitInformation_restrictDate.json
+++ b/plugins/PrivacyManager/tests/System/expected/anonymizeVisitInformation_restrictDate.json
@@ -12398,4 +12398,4 @@
             "name": "My product 211"
         }
     ]
-}
+}
\ No newline at end of file
diff --git a/plugins/PrivacyManager/tests/System/expected/anonymizeVisitInformation_restrictSites.json b/plugins/PrivacyManager/tests/System/expected/anonymizeVisitInformation_restrictSites.json
index 640d1ad173..fee651760c 100644
--- a/plugins/PrivacyManager/tests/System/expected/anonymizeVisitInformation_restrictSites.json
+++ b/plugins/PrivacyManager/tests/System/expected/anonymizeVisitInformation_restrictSites.json
@@ -12174,4 +12174,4 @@
             "name": "My product 211"
         }
     ]
-}
+}
\ No newline at end of file