From ee2a3c9beae604937f97d293355df5968cea059e Mon Sep 17 00:00:00 2001
From: Yuri Kuznetsov <yurikuzn@users.noreply.github.com>
Date: Wed, 1 Sep 2021 15:51:46 +0300
Subject: [PATCH] fix x-skip-duplicate-check header

---
 .../Espo/Core/Record/CreateParamsFetcher.php  |   4 +-
 .../Core/Record/CreateParamsFetcherTest.php   | 122 ++++++++++++++++++
 2 files changed, 124 insertions(+), 2 deletions(-)
 create mode 100644 tests/unit/Espo/Core/Record/CreateParamsFetcherTest.php

diff --git a/application/Espo/Core/Record/CreateParamsFetcher.php b/application/Espo/Core/Record/CreateParamsFetcher.php
index ea1eb782bd..ec2dc8729b 100644
--- a/application/Espo/Core/Record/CreateParamsFetcher.php
+++ b/application/Espo/Core/Record/CreateParamsFetcher.php
@@ -41,8 +41,8 @@ class CreateParamsFetcher
     {
         $data = $request->getParsedBody();
 
-        $skipDuplicateCheck =
-            $request->getHeader('X-Skip-Duplicate-Check') ??
+        $skipDuplicateCheck = $request->hasHeader('X-Skip-Duplicate-Check') ?
+            strtolower($request->getHeader('X-Skip-Duplicate-Check')) === 'true' :
             $data->_skipDuplicateCheck ?? // legacy
             $data->skipDuplicateCheck ?? // legacy
             $data->forceDuplicate ?? // legacy
diff --git a/tests/unit/Espo/Core/Record/CreateParamsFetcherTest.php b/tests/unit/Espo/Core/Record/CreateParamsFetcherTest.php
new file mode 100644
index 0000000000..396707bd7b
--- /dev/null
+++ b/tests/unit/Espo/Core/Record/CreateParamsFetcherTest.php
@@ -0,0 +1,122 @@
+<?php
+/************************************************************************
+ * This file is part of EspoCRM.
+ *
+ * EspoCRM - Open Source CRM application.
+ * Copyright (C) 2014-2021 Yurii Kuznietsov, Taras Machyshyn, Oleksii Avramenko
+ * Website: https://www.espocrm.com
+ *
+ * EspoCRM is free software: you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation, either version 3 of the License, or
+ * (at your option) any later version.
+ *
+ * EspoCRM is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with EspoCRM. If not, see http://www.gnu.org/licenses/.
+ *
+ * The interactive user interfaces in modified source and object code versions
+ * of this program must display Appropriate Legal Notices, as required under
+ * Section 5 of the GNU General Public License version 3.
+ *
+ * In accordance with Section 7(b) of the GNU General Public License version 3,
+ * these Appropriate Legal Notices must retain the display of the "EspoCRM" word.
+ ************************************************************************/
+
+namespace tests\unit\Espo\Core\Record;
+
+use Espo\Core\Record\CreateParamsFetcher;
+use Espo\Core\Api\RequestWrapper;
+
+class CreateParamsFetcherTest extends \PHPUnit\Framework\TestCase
+{
+    public function test1(): void
+    {
+        $request = $this->createMock(RequestWrapper::class);
+
+        $request
+            ->method('hasHeader')
+            ->willReturn(true);
+
+        $request
+            ->method('getHeader')
+            ->with('X-Skip-Duplicate-Check')
+            ->willReturn('true');
+
+        $params = (new CreateParamsFetcher())->fetch($request);
+
+        $this->assertTrue($params->skipDuplicateCheck());
+    }
+
+    public function test2(): void
+    {
+        $request = $this->createMock(RequestWrapper::class);
+
+        $request
+            ->method('hasHeader')
+            ->willReturn(true);
+
+        $request
+            ->method('getHeader')
+            ->with('X-Skip-Duplicate-Check')
+            ->willReturn('false');
+
+        $params = (new CreateParamsFetcher())->fetch($request);
+
+        $this->assertFalse($params->skipDuplicateCheck());
+    }
+
+    public function test3(): void
+    {
+        $request = $this->createMock(RequestWrapper::class);
+
+        $request
+            ->method('hasHeader')
+            ->willReturn(false);
+
+        $params = (new CreateParamsFetcher())->fetch($request);
+
+        $this->assertFalse($params->skipDuplicateCheck());
+    }
+
+    public function test4(): void
+    {
+        $request = $this->createMock(RequestWrapper::class);
+
+        $request
+            ->method('hasHeader')
+            ->willReturn(true);
+
+        $request
+            ->method('getHeader')
+            ->with('X-Skip-Duplicate-Check')
+            ->willReturn('TRUE');
+
+        $params = (new CreateParamsFetcher())->fetch($request);
+
+        $this->assertTrue($params->skipDuplicateCheck());
+    }
+
+    public function test5(): void
+    {
+        $request = $this->createMock(RequestWrapper::class);
+
+        $request
+            ->method('hasHeader')
+            ->willReturn(false);
+
+        $request
+            ->method('getParsedBody')
+            ->willReturn((object) [
+                '_skipDuplicateCheck' => true,
+            ]);
+
+        $params = (new CreateParamsFetcher())->fetch($request);
+
+        $this->assertTrue($params->skipDuplicateCheck());
+    }
+}