From 20385fe70c3d480d11be168efd6c674d3c11e304 Mon Sep 17 00:00:00 2001
From: Artem Bondarenko <abonda@rainpos.com>
Date: Mon, 12 Oct 2020 20:44:37 +0300
Subject: [PATCH] fix cs

---
 src/Api/Domain.php                          | 11 +++++------
 src/Model/Domain/ClickTracking.php          |  4 ++--
 src/Model/Domain/OpenTracking.php           |  2 +-
 src/Model/Domain/UnsubscribeTracking.php    |  2 +-
 tests/Model/Domain/TrackingResponseTest.php |  1 +
 5 files changed, 10 insertions(+), 10 deletions(-)

diff --git a/src/Api/Domain.php b/src/Api/Domain.php
index 2847113..dbb3d76 100644
--- a/src/Api/Domain.php
+++ b/src/Api/Domain.php
@@ -325,8 +325,8 @@ class Domain extends HttpApi
      *
      * @param string $domain name of the domain
      *
-     * @param string $active
      * @return UpdateClickTrackingResponse|array|ResponseInterface
+     *
      * @throws \Exception
      */
     public function updateClickTracking(string $domain, string $active)
@@ -348,7 +348,7 @@ class Domain extends HttpApi
      * Updates a domain open tracking settings.
      *
      * @param string $domain name of the domain
-     * @param bool $active
+     * @param bool   $active
      *
      * @return UpdateOpenTrackingResponse|array|ResponseInterface
      */
@@ -370,10 +370,9 @@ class Domain extends HttpApi
     /**
      * Updates a domain unsubscribe tracking settings.
      *
-     * @param string $domain name of the domain
-     * @param string $active
-     * @param string $htmlFooter
+     * @param string      $domain     name of the domain
      * @param string|null $textFooter
+     *
      * @return UpdateUnsubscribeTrackingResponse|array|ResponseInterface
      */
     public function updateUnsubscribeTracking(string $domain, string $active, string $htmlFooter, string $textFooter)
@@ -385,7 +384,7 @@ class Domain extends HttpApi
         Assert::nullOrString($textFooter);
 
         $params = [
-            'active' => (in_array($active, ['yes', 'true'])) ? 'true' : 'false',
+            'active' => (in_array($active, ['yes', 'true'], true)) ? 'true' : 'false',
             'html_footer' => $htmlFooter,
             'text_footer' => $textFooter,
         ];
diff --git a/src/Model/Domain/ClickTracking.php b/src/Model/Domain/ClickTracking.php
index 97825ad..c5ae3a6 100644
--- a/src/Model/Domain/ClickTracking.php
+++ b/src/Model/Domain/ClickTracking.php
@@ -47,11 +47,11 @@ final class ClickTracking
 
     public function isActive(): bool
     {
-        return $this->getActive() === 'yes';
+        return 'yes' === $this->getActive();
     }
 
     public function isHtmlOnly(): bool
     {
-        return $this->getActive() === 'htmlonly';
+        return 'htmlonly' === $this->getActive();
     }
 }
diff --git a/src/Model/Domain/OpenTracking.php b/src/Model/Domain/OpenTracking.php
index 768863c..9c67e6e 100644
--- a/src/Model/Domain/OpenTracking.php
+++ b/src/Model/Domain/OpenTracking.php
@@ -50,6 +50,6 @@ final class OpenTracking
 
     public function isActive(): bool
     {
-        return $this->getActive() === 'yes';
+        return 'yes' === $this->getActive();
     }
 }
diff --git a/src/Model/Domain/UnsubscribeTracking.php b/src/Model/Domain/UnsubscribeTracking.php
index e2d510a..1c64bef 100644
--- a/src/Model/Domain/UnsubscribeTracking.php
+++ b/src/Model/Domain/UnsubscribeTracking.php
@@ -38,7 +38,7 @@ final class UnsubscribeTracking
 
     public function isActive(): bool
     {
-        return $this->active === 'true';
+        return 'yes' === $this->getActive();
     }
 
     public function getActive(): string
diff --git a/tests/Model/Domain/TrackingResponseTest.php b/tests/Model/Domain/TrackingResponseTest.php
index 29a01bb..5b916d8 100644
--- a/tests/Model/Domain/TrackingResponseTest.php
+++ b/tests/Model/Domain/TrackingResponseTest.php
@@ -44,6 +44,7 @@ JSON;
         $this->assertInstanceOf(ClickTracking::class, $model->getClick());
         $this->assertEquals('htmlonly', $model->getClick()->getActive());
         $this->assertFalse($model->getClick()->isActive());
+        $this->assertTrue($model->getClick()->isHtmlOnly());
 
         $this->assertNotEmpty($model->getOpen());
         $this->assertInstanceOf(OpenTracking::class, $model->getOpen());