Description:

Was wondering a few times lately why the file /plugins/VisitFrequency/Archiver.php was removed locally after running tests.
Seems the reason is, that some updater tests are trying to update from a very old version, causing all update scripts to be executed.

I guess the same might actually happen if someone tries updating from Piwik before 2.2 to Matomo after 4.1.2. The update would first create the Archiver file and the update script would then remove it again when it is executed.
Should be better to remove that one.

ping @diosmosis You had removed that file in one of your PRs by accident as well. guess this is the reason.

Review

• Functional review done
• Potential edge cases thought about (behavior of the code with strange input, with strange internal state or possible interactions with other Matomo subsystems)
• Usability review done (is anything maybe unclear or think about anything that would cause people to reach out to support)
• Security review done
• Wording review done
• Code review done
• Tests were added if useful/possible
• Reviewed for breaking changes
• Developer changelog updated if needed
• Documentation added if needed
• Existing documentation updated if needed