From 3eafb108e4c5fc0467a934cbb7ecf3d9d8a2d0ba Mon Sep 17 00:00:00 2001
From: sgiehl <stefan@matomo.org>
Date: Fri, 7 Feb 2020 08:57:53 +0100
Subject: [PATCH] remove old events

---
 CHANGELOG.md                                           |  6 ++----
 plugins/CustomJsTracker/TrackerUpdater.php             | 10 ----------
 .../TrackingCode/PluginTrackerFiles.php                |  5 -----
 3 files changed, 2 insertions(+), 19 deletions(-)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index 7f8abc988113..bffcc3c19123 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -33,10 +33,8 @@ The Product Changelog at **[matomo.org/changelog](https://matomo.org/changelog)*
 * The JavaScript tracker now uses `sendBeacon` by default if supported by the browser. You can disable this by calling the tracker method `disableAlwaysUseSendBeacon`. As a result, callback parameters won't work anymore and a tracking request might not appear in the developer tools.
 * The console option `--piwik-domain` has been removed. Use `--matomo-domain` instead
 * The core plugin `CustomPiwikJs` was renamed `CustomJsTracker`
-
-### Deprecations
-* The event `CustomPiwikJs.piwikJsChanged` has been deprecated. Use `CustomJsTracker.trackerJsChanged` instead
-* The event `CustomPiwikJs.shouldAddTrackerFile` has been deprecated. Use `CustomJsTracker.shouldAddTrackerFile` instead
+* The event `CustomPiwikJs.piwikJsChanged` has been renamed. Use `CustomJsTracker.trackerJsChanged` instead
+* The event `CustomPiwikJs.shouldAddTrackerFile` has been renamed. Use `CustomJsTracker.shouldAddTrackerFile` instead
 
 ## Matomo 3.13.1
 
diff --git a/plugins/CustomJsTracker/TrackerUpdater.php b/plugins/CustomJsTracker/TrackerUpdater.php
index 19b63f50e676..c301862b90fc 100644
--- a/plugins/CustomJsTracker/TrackerUpdater.php
+++ b/plugins/CustomJsTracker/TrackerUpdater.php
@@ -140,11 +140,6 @@ public function update()
                  * @param string $absolutePath The path to the new piwik.js file.
                  */
                 Piwik::postEvent('CustomJsTracker.trackerJsChanged', [$savedFile]);
-
-                /**
-                 * @todo remove in Matomo 5. For BC only
-                 */
-                Piwik::postEvent('CustomPiwikJs.piwikJsChanged', [$savedFile]);
             }
 
         }
@@ -163,11 +158,6 @@ private function updateAlternative($fromFile, $toFile, $newContent)
                 $savedFiles = $file->save($newContent);
                 foreach ($savedFiles as $savedFile) {
                     Piwik::postEvent('CustomJsTracker.trackerJsChanged', [$savedFile]);
-
-                    /**
-                     * @todo remove in Matomo 5. For BC only
-                     */
-                    Piwik::postEvent('CustomPiwikJs.piwikJsChanged', [$savedFile]);
                 }
             }
         }
diff --git a/plugins/CustomJsTracker/TrackingCode/PluginTrackerFiles.php b/plugins/CustomJsTracker/TrackingCode/PluginTrackerFiles.php
index 7e403b39a23b..e062a7351674 100644
--- a/plugins/CustomJsTracker/TrackingCode/PluginTrackerFiles.php
+++ b/plugins/CustomJsTracker/TrackingCode/PluginTrackerFiles.php
@@ -94,11 +94,6 @@ protected function shouldIncludeFile($pluginName)
          */
         Piwik::postEvent('CustomJsTracker.shouldAddTrackerFile', array(&$shouldAddFile, $pluginName));
 
-        /**
-         * @todo remove in Matomo 5. For BC only
-         */
-        Piwik::postEvent('CustomPiwikJs.shouldAddTrackerFile', array(&$shouldAddFile, $pluginName));
-
         return $shouldAddFile;
     }