fixes PSR error

API.php

@@ -158,7 +158,6 @@ private function filterAdditionalEmails($additionalEmails)
         }
 
         foreach ($additionalEmails as &$email) {
-
             $email = trim($email);
             if (empty($email)) {
                 $email = false;
@@ -173,7 +172,6 @@ private function filterPhoneNumbers($phoneNumbers)
         $availablePhoneNumbers = (new \Piwik\Plugins\MobileMessaging\Model())->getActivatedPhoneNumbers(Piwik::getCurrentUserLogin());
 
         foreach ($phoneNumbers as $key => &$phoneNumber) {
-
             $phoneNumber = trim($phoneNumber);
 
             if (!in_array($phoneNumber, $availablePhoneNumbers)) {
@@ -278,5 +276,4 @@ public function getTriggeredAlerts($idSites)
 
         return $this->getModel()->getTriggeredAlerts($idSites, $login);
     }
-
 }

Controller.php

@@ -1,4 +1,5 @@
 <?php
+
 /**
  * Matomo - free/libre analytics platform
  *
@@ -105,7 +106,6 @@ private function findSiteName($alert)
         $idSite = $this->findSiteId($alert);
 
         if (!empty($idSite)) {
-
             return Site::getNameFor($idSite);
         }
     }
@@ -157,14 +157,12 @@ public function formatAlerts($triggeredAlerts, $format)
                 return $view->render();
 
             case 'sms':
-
                 $view                  = new View('@CustomAlerts/smsTriggeredAlerts');
                 $view->triggeredAlerts = $this->enrichTriggeredAlerts($triggeredAlerts);
 
                 return $view->render();
 
             case 'text':
-
                 $view                  = new View('@CustomAlerts/textTriggeredAlerts');
                 $view->triggeredAlerts = $this->enrichTriggeredAlerts($triggeredAlerts);
 

CustomAlerts.php

@@ -11,7 +11,6 @@
 
 use Piwik\Common;
 use Piwik\Container\StaticContainer;
-use Piwik\Option;
 use Piwik\Piwik;
 use Piwik\Plugins\SitesManager\API as SitesManagerApi;
 use Piwik\Scheduler\Task;

Menu.php

@@ -1,4 +1,5 @@
 <?php
+
 /**
  * Matomo - free/libre analytics platform
  *

Model.php

@@ -20,7 +20,6 @@
  */
 class Model
 {
-
     public static function install()
     {
         $tableAlert = "`idalert` INT NOT NULL PRIMARY KEY ,
@@ -434,5 +433,4 @@ public function deleteTriggeredAlertsForSite($idSite)
     {
         $this->getDb()->query("DELETE FROM " . Common::prefixTable("alert_triggered") . " WHERE idsite = ?", $idSite);
     }
-
 }

Notifier.php

@@ -72,7 +72,6 @@ private function groupAlertsPerEmailRecipient($triggeredAlerts)
         $alertsPerEmail = array();
 
         foreach ($triggeredAlerts as $triggeredAlert) {
-
             $emails = $this->getEmailRecipientsForAlert($triggeredAlert);
 
             foreach ($emails as $mail) {
@@ -189,7 +188,6 @@ private function groupAlertsPerSmsRecipient($triggeredAlerts)
         $recipients = array();
 
         foreach ($triggeredAlerts as $triggeredAlert) {
-
             $phoneNumbers = $triggeredAlert['phone_numbers'];
 
             if (empty($phoneNumbers)) {

Processor.php

@@ -15,7 +15,6 @@
 use Piwik\Context;
 use Piwik\DataTable;
 use Piwik\Date;
-use Piwik\Option;
 use Piwik\Piwik;
 use Piwik\Plugins\API\ProcessedReport;
 use Piwik\Scheduler\RetryableException;
@@ -453,5 +452,4 @@ protected function triggerAlert($alert, $idSite, $valueNew, $valueOld)
     {
         $this->getModel()->triggerAlert($alert['idalert'], $idSite, $valueNew, $valueOld, Date::now()->getDatetime());
     }
-
 }

Tasks.php

@@ -1,4 +1,5 @@
 <?php
+
 /**
  * Matomo - free/libre analytics platform
  *
@@ -67,4 +68,4 @@ private function runAlerts($period, $idSite)
         $this->processor->processAlerts($period, (int) $idSite);
         $this->notifier->sendNewAlerts($period, (int) $idSite);
     }
-}
+}

Updates/0.0.2.php

@@ -1,4 +1,5 @@
 <?php
+
 /**
  * Matomo - free/libre analytics platform
  *

Updates/0.0.3.php

@@ -1,4 +1,5 @@
 <?php
+
 /**
  * Matomo - free/libre analytics platform
  *

Updates/0.0.4.php

@@ -1,4 +1,5 @@
 <?php
+
 /**
  * Matomo - free/libre analytics platform
  *

Updates/0.0.5.php

@@ -1,4 +1,5 @@
 <?php
+
 /**
  * Matomo - free/libre analytics platform
  *

Updates/0.0.6.php

@@ -1,4 +1,5 @@
 <?php
+
 /**
  * Matomo - free/libre analytics platform
  *

Updates/0.0.7.php

@@ -1,4 +1,5 @@
 <?php
+
 /**
  * Matomo - free/libre analytics platform
  *

Updates/0.0.8.php

@@ -1,4 +1,5 @@
 <?php
+
 /**
  * Matomo - free/libre analytics platform
  *
@@ -39,8 +40,10 @@ public function getMigrations(Updater $updater)
         $triggeredTable = Common::prefixTable('alert_triggered');
 
         return array(
-            $this->migration->db->sql("RENAME TABLE `" . Common::prefixTable('alert_log') . "` TO `" . Common::prefixTable('alert_triggered') . "`",
-                array(DbMigration::ERROR_CODE_DUPLICATE_COLUMN, DbMigration::ERROR_CODE_TABLE_NOT_EXISTS, DbMigration::ERROR_CODE_TABLE_EXISTS)),
+            $this->migration->db->sql(
+                "RENAME TABLE `" . Common::prefixTable('alert_log') . "` TO `" . Common::prefixTable('alert_triggered') . "`",
+                array(DbMigration::ERROR_CODE_DUPLICATE_COLUMN, DbMigration::ERROR_CODE_TABLE_NOT_EXISTS, DbMigration::ERROR_CODE_TABLE_EXISTS)
+            ),
             $this->migration->db->addColumns('alert_triggered', array(
                 'name'              => 'VARCHAR(100) NOT NULL',
                 'login'             => 'VARCHAR(100) NOT NULL',

Updates/0.1.10.php

@@ -1,4 +1,5 @@
 <?php
+
 /**
  * Matomo - free/libre analytics platform
  *

Updates/0.1.17.php

@@ -1,4 +1,5 @@
 <?php
+
 /**
  * Matomo - free/libre analytics platform
  *

Updates/0.1.8.php

@@ -1,4 +1,5 @@
 <?php
+
 /**
  * Matomo - free/libre analytics platform
  *

Validator.php

@@ -44,7 +44,6 @@ public function filterPhoneNumbers($phoneNumbers)
         $availablePhoneNumbers = (new \Piwik\Plugins\MobileMessaging\Model())->getActivatedPhoneNumbers(Piwik::getCurrentUserLogin());
 
         foreach ($phoneNumbers as $key => &$phoneNumber) {
-
             $phoneNumber = trim($phoneNumber);
 
             if (!in_array($phoneNumber, $availablePhoneNumbers)) {

tests/Fixtures/CustomAlerts.php

@@ -1,4 +1,5 @@
 <?php
+
 /**
  * Matomo - free/libre analytics platform
  *

tests/Integration/ApiTest.php

@@ -1,4 +1,5 @@
 <?php
+
 /**
  * Matomo - free/libre analytics platform
  *
@@ -18,7 +19,6 @@
  */
 class ApiTest extends BaseTest
 {
-
     public function setUp(): void
     {
         parent::setUp();
@@ -39,17 +39,29 @@ protected function createAlert(
         $reportCondition = 'matches_exactly',
         $emails = array('[email protected]', '[email protected]'),
         $comparedTo = 1
-    )
-    {
+    ) {
         if (is_null($idSites)) {
             $idSites = $this->idSite;
         }
 
         // those should be dropped by the api as they do not exist in Piwik
         $phoneNumbers = array('+1234567890', '1234567890');
 
-        $id = $this->api->addAlert($name, $idSites, $period, 0, $emails, $phoneNumbers, $metric, $metricCondition,
-            $metricMatched = 5, $comparedTo, $report, $reportCondition, 'Piwik');
+        $id = $this->api->addAlert(
+            $name,
+            $idSites,
+            $period,
+            0,
+            $emails,
+            $phoneNumbers,
+            $metric,
+            $metricCondition,
+            $metricMatched = 5,
+            $comparedTo,
+            $report,
+            $reportCondition,
+            'Piwik'
+        );
         return $id;
     }
 
@@ -164,8 +176,7 @@ protected function assertIsAlert(
         $report = 'MultiSites_getOne',
         $reportCondition = 'matches_exactly',
         $reportMatched = 'Piwik'
-    )
-    {
+    ) {
         if (is_null($idSites)) {
             $idSites = array($this->idSite);
         }
@@ -211,8 +222,7 @@ protected function editAlert(
         $metricCondition = 'less_than',
         $reportCondition = 'matches_exactly',
         $emails = array('[email protected]', '[email protected]')
-    )
-    {
+    ) {
         if (is_null($idSites)) {
             $idSites = $this->idSite;
         }
@@ -221,8 +231,22 @@ protected function editAlert(
         $phoneNumbers = array('+1234567890', '1234567890');
         $comparedTo   = 1;
 
-        $id = $this->api->editAlert($id, $name, $idSites, $period, 0, $emails, $phoneNumbers, $metric, $metricCondition,
-            $metricMatched = 5, $comparedTo, $report, $reportCondition, 'Piwik');
+        $id = $this->api->editAlert(
+            $id,
+            $name,
+            $idSites,
+            $period,
+            0,
+            $emails,
+            $phoneNumbers,
+            $metric,
+            $metricCondition,
+            $metricMatched = 5,
+            $comparedTo,
+            $report,
+            $reportCondition,
+            'Piwik'
+        );
         return $id;
     }
 
@@ -571,4 +595,4 @@ public function test_getTriggeredAlerts_ShouldReturnAllThatMatchesLoginAndIdSite
         $triggeredAlerts         = $this->api->getTriggeredAlerts(array(1));
         $this->assertCount(0, $triggeredAlerts);
     }
-}
+}

tests/Integration/BaseTest.php

@@ -1,4 +1,5 @@
 <?php
+
 /**
  * Matomo - free/libre analytics platform
  *
@@ -75,4 +76,4 @@ protected function setUser()
         FakeAccess::$idSitesView  = array(99);
         FakeAccess::$identity     = 'aUser';
     }
-}
+}

tests/Integration/ControllerTest.php

@@ -1,4 +1,5 @@
 <?php
+
 /**
  * Matomo - free/libre analytics platform
  *
@@ -108,8 +109,7 @@ private function buildAlert(
         $report = 'MultiSites_getOne',
         $reportCondition = 'matches_exactly',
         $reportMatched = 'Piwik'
-    )
-    {
+    ) {
         return array(
             'idalert'           => $id,
             'idsite'            => $idSite,
@@ -192,7 +192,6 @@ public function test_formatAlerts_asHtml()
 FORMATTED;
 
         $this->assertEquals($expected, $rendered, "Got following HTML response: " . var_export($rendered, true));
-
     }
 
     public function test_formatAlertsNoConditions_asHtml()
@@ -227,7 +226,6 @@ public function test_formatAlertsNoConditions_asHtml()
 FORMATTED;
 
         $this->assertEquals($expected, $rendered, "Got following HTML response: " . var_export($rendered, true));
-
     }
 
     public function test_enrichTriggeredAlerts_shouldEnrichAlerts_IfReportExistsAndMetricIsValid()
@@ -341,5 +339,4 @@ public function test_enrichTriggeredAlerts_shouldEnrichAlerts_IfReportExistsAndM
 
         $this->assertEquals($alerts, $enriched);
     }
-
 }

tests/Integration/CustomAlertsTest.php

@@ -1,4 +1,5 @@
 <?php
+
 /**
  * Matomo - free/libre analytics platform
  *
@@ -10,7 +11,6 @@
 
 use Piwik\Container\StaticContainer;
 use Piwik\Date;
-use Piwik\Option;
 use Piwik\Piwik;
 use Piwik\Plugins\CustomAlerts\CustomAlerts;
 use Piwik\Scheduler\Task;
@@ -89,8 +89,22 @@ private function createAlert($name, $phoneNumbers, $idSites = array(1), $login =
             $login = Piwik::getCurrentUserLogin();
         }
 
-        $id = $this->model->createAlert($name, $idSites, $login, 'week', 0, $emails, $phoneNumbers, 'nb_visits',
-            'less_than', 5, $comparedTo = 7, $report, 'matches_exactly', 'Piwik');
+        $id = $this->model->createAlert(
+            $name,
+            $idSites,
+            $login,
+            'week',
+            0,
+            $emails,
+            $phoneNumbers,
+            'nb_visits',
+            'less_than',
+            5,
+            $comparedTo = 7,
+            $report,
+            'matches_exactly',
+            'Piwik'
+        );
 
         return $this->model->getAlert($id);
     }