Skip to content
Snippets Groups Projects
Commit 364f9e00 authored by Cristian Partica's avatar Cristian Partica
Browse files

MAGETWO-58924: SQL error wait timeout error when saving categories

- fix static issues
parent f1e86dfb
Branches
No related merge requests found
...@@ -16,8 +16,8 @@ use Magento\Framework\DB\Select; ...@@ -16,8 +16,8 @@ use Magento\Framework\DB\Select;
*/ */
class TemporaryTableService class TemporaryTableService
{ {
CONST HASH = 'HASH'; const INDEX_METHOD_HASH = 'HASH';
CONST INNODB = 'INNODB'; const INDEX_METHOD_INNODB = 'INNODB';
/** /**
* @var string[] * @var string[]
...@@ -46,8 +46,8 @@ class TemporaryTableService ...@@ -46,8 +46,8 @@ class TemporaryTableService
*/ */
public function __construct( public function __construct(
\Magento\Framework\Math\Random $random, \Magento\Framework\Math\Random $random,
$allowedIndexMethods = [self::HASH], $allowedIndexMethods = [],
$allowedEngines = [self::INNODB] $allowedEngines = []
) { ) {
$this->random = $random; $this->random = $random;
$this->allowedIndexMethods = $allowedIndexMethods; $this->allowedIndexMethods = $allowedIndexMethods;
...@@ -83,18 +83,18 @@ class TemporaryTableService ...@@ -83,18 +83,18 @@ class TemporaryTableService
Select $select, Select $select,
AdapterInterface $adapter, AdapterInterface $adapter,
array $indexes = [], array $indexes = [],
$indexMethod = self::HASH, $indexMethod = self::INDEX_METHOD_HASH,
$dbEngine = self::INNODB $dbEngine = self::INDEX_METHOD_INNODB
) { ) {
if (!in_array($indexMethod, $this->allowedIndexMethods)) { if (!in_array($indexMethod, $this->allowedIndexMethods)) {
throw new \InvalidArgumentException( throw new \InvalidArgumentException(
sprintf('indexMethod must be of type %s', implode(',', $this->allowedIndexMethods)) sprintf('indexMethod must be one of %s', implode(',', $this->allowedIndexMethods))
); );
} }
if (!in_array($dbEngine, $this->allowedEngines)) { if (!in_array($dbEngine, $this->allowedEngines)) {
throw new \InvalidArgumentException( throw new \InvalidArgumentException(
sprintf('dbEngine must be of type %s', implode(',', $this->allowedEngines)) sprintf('dbEngine must be one of %s', implode(',', $this->allowedEngines))
); );
} }
......
...@@ -46,7 +46,9 @@ class TemporaryTableServiceTest extends \PHPUnit_Framework_TestCase ...@@ -46,7 +46,9 @@ class TemporaryTableServiceTest extends \PHPUnit_Framework_TestCase
$this->temporaryTableService = (new ObjectManager($this))->getObject( $this->temporaryTableService = (new ObjectManager($this))->getObject(
TemporaryTableService::class, TemporaryTableService::class,
[ [
'random' => $this->randomMock 'random' => $this->randomMock,
'allowedIndexMethods' => ['HASH'],
'allowedEngines' => ['INNODB']
] ]
); );
} }
...@@ -72,8 +74,8 @@ class TemporaryTableServiceTest extends \PHPUnit_Framework_TestCase ...@@ -72,8 +74,8 @@ class TemporaryTableServiceTest extends \PHPUnit_Framework_TestCase
$this->selectMock, $this->selectMock,
$this->adapterMock, $this->adapterMock,
$indexes, $indexes,
TemporaryTableService::HASH . "Other", TemporaryTableService::INDEX_METHOD_HASH . "Other",
TemporaryTableService::INNODB . "Other" TemporaryTableService::INDEX_METHOD_INNODB . "Other"
) )
); );
} }
......
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment