Merge pull request magento#755 from magento/MCLOUD-6381

Yevhen Miroshnychenko · web-flow · commit b0a8802182b9 · 2020-07-08T13:45:23.000-05:00
MCLOUD-6381: Update error descriptions
diff --git a/config/schema.error.yaml b/config/schema.error.yaml
@@ -439,7 +439,7 @@
     step: validate-config
     type: warning
 !php/const Magento\MagentoCloud\App\Error::WARN_NOT_CORRECT_LOCAL_XML_FILE:
-    title: 'Invalid file config ./pub/errors/local.xml'
+    title: 'Invalid configuration in the ./pub/errors/local.xml file.'
     stage: deploy
     step: validate-config
     type: warning
@@ -544,7 +544,7 @@
     step: 'install-update:split-db'
     type: warning
 !php/const Magento\MagentoCloud\App\Error::WARN_NOT_ENOUGH_DATA_SPLIT_DB_VAR:
-    title: 'The SPLIT_DB variable does not have data which were already split.'
+    title: 'The SPLIT_DB variable is missing the configuration for split connection types.'
     stage: deploy
     step: 'install-update:split-db'
     type: warning
diff --git a/dist/error-codes.md b/dist/error-codes.md
@@ -121,7 +121,7 @@
 | 2001 | pre-deploy:cache | Cache is configured for a Redis service that is not available. Configuration will be ignored. |  |
 | 2002 | validate-config | The configured state is not ideal |  |
 | 2003 | validate-config | The directory nesting level value for error reporting has not been configured |  |
-| 2004 | validate-config | Invalid file config ./pub/errors/local.xml |  |
+| 2004 | validate-config | Invalid configuration in the ./pub/errors/local.xml file. |  |
 | 2005 | validate-config | Admin data is used to create an admin user during initial installation only. Any changes to Admin data are ignored during the upgrade process. | After the initial installation, you can remove admin data from the configuration. |
 | 2006 | validate-config | Admin user was not created as admin email was not set | After installation, you can create an admin user manually:  Use ssh to connect to your environment. Then, run the `bin/magento admin:user:create` command. |
 | 2007 | validate-config | Update php version to recommended version |  |
@@ -141,7 +141,7 @@
 | 2021 | install-update:db-connection | For split databases used custom connections |  |
 | 2022 | install-update:db-connection | You have changed to a database configuration that is not compatible with the slave connection. |  |
 | 2023 | install-update:split-db | Enabling a split database will be skipped. |  |
-| 2024 | install-update:split-db | The SPLIT_DB variable does not have data which were already split. |  |
+| 2024 | install-update:split-db | The SPLIT_DB variable is missing the configuration for split connection types. |  |
 | 2025 | install-update:split-db | Slave connection not set. |  |
 
 ### Post-deploy stage
diff --git a/src/Command/ErrorShow.php b/src/Command/ErrorShow.php
@@ -87,7 +87,7 @@ protected function execute(InputInterface $input, OutputInterface $output)
         } else {
             $errors = $this->reader->read();
             if (empty($errors)) {
-                $output->writeln('The error log is empty or doesn\'t exist');
+                $output->writeln('The error log is empty or does not exist');
 
                 return 1;
             }
diff --git a/src/Config/Schema/Validator/SplitDb.php b/src/Config/Schema/Validator/SplitDb.php
@@ -37,7 +37,7 @@ public function validate(string $key, $value): ResultInterface
         if (array_diff($value, DeployInterface::SPLIT_DB_VALUES)) {
             return $this->resultFactory->error(sprintf(
                 'The %s variable contains the invalid value. '
-                . 'It should be array with next available values: [%s].',
+                . 'It should be an array with following values: [%s].',
                 $key,
                 implode(', ', DeployInterface::SPLIT_DB_VALUES)
             ));
diff --git a/src/Config/Validator/Deploy/DebugLogging.php b/src/Config/Validator/Deploy/DebugLogging.php
@@ -66,8 +66,7 @@ public function validate(): ResultInterface
 
         return $this->resultFactory->error(
             'Debug logging is enabled in Magento',
-            'To save space in Magento Cloud, '
-            . 'debug logging should not be enabled for your production environments.',
+            'To save space in Magento Cloud, disable debug logging for your production environments.',
             Error::WARN_DEBUG_LOG_ENABLED
         );
     }
diff --git a/src/Config/Validator/Deploy/ElasticSearchUsage.php b/src/Config/Validator/Deploy/ElasticSearchUsage.php
@@ -74,7 +74,7 @@ public function validate(): Validator\ResultInterface
 
         return $this->resultFactory->error(
             'Elasticsearch service is installed at infrastructure layer but is not used as a search engine.',
-            'Consider removing elasticsearch service from infrastructure layer for optimized resource usage.',
+            'Consider removing the Elasticsearch service from the infrastructure layer for optimized resource usage.',
             Error::WARN_ES_INSTALLED_BUT_NOT_USED
         );
     }
diff --git a/src/Config/Validator/Deploy/PhpVersion.php b/src/Config/Validator/Deploy/PhpVersion.php
@@ -102,7 +102,7 @@ public function validate(): Validator\ResultInterface
                         $currentPhpConstraint->getPrettyString()
                     ),
                     "Change the version of PHP to the version that satisfies the restriction conditions.\n"
-                    . 'Change the PHP version in the .magento.app.yaml file or create a support ticket',
+                    . 'Change the PHP version in the .magento.app.yaml file, or create a support ticket',
                     Error::WARN_UPDATE_PHP_VERSION
                 );
             }
diff --git a/src/Config/Validator/Deploy/ReportDirNestingLevel.php b/src/Config/Validator/Deploy/ReportDirNestingLevel.php
@@ -85,15 +85,19 @@ public function validate(): Validator\ResultInterface
 
             return $this->resultFactory->error(
                 'The directory nesting level value for error reporting has not been configured.',
-                'You can configure the setting using the `config.report.dir_nesting_level`'
+                'You can configure the setting using the `config.report.dir_nesting_level` variable'
                 . ' in the file ' . $reportConfigFile,
                 Error::WARN_DIR_NESTING_LEVEL_NOT_CONFIGURED
             );
         } catch (FileSystemException $exception) {
             return $this->resultFactory->error($exception->getMessage());
         } catch (NotEncodableValueException $exception) {
-            $message = sprintf('Config of the file %s is invalid. ', $reportConfigFile);
-            $message .= $exception->getMessage();
+            $message = sprintf(
+                'Invalid configuration in the %s file. %s',
+                $reportConfigFile,
+                $exception->getMessage()
+            );
+
             return $this->resultFactory->error(
                 $message,
                 'Fix the directory nesting level configuration for error reporting in the file '
diff --git a/src/Config/Validator/Deploy/SolrIntegrity.php b/src/Config/Validator/Deploy/SolrIntegrity.php
@@ -63,7 +63,7 @@ public function validate(): ResultInterface
 
             if ($this->magentoVersion->satisfies('2.1.*')) {
                 $args['suggestion'] = 'Solr support has been deprecated in Magento 2.1. ' .
-                    'Update your search engine to Elasticsearch and remove this relationship.';
+                    'Remove this relationship and use Elasticsearch.';
                 $args['errorCode'] = Error::WARN_SOLR_DEPRECATED;
             }
 
diff --git a/src/DB/Data/ConnectionFactory.php b/src/DB/Data/ConnectionFactory.php
@@ -68,7 +68,7 @@ public function create(string $connectionType): ConnectionInterface
                 return $this->getConnectionData(DbConfig::CONNECTION_SALES, false);
             default:
                 throw new RuntimeException(
-                    sprintf('Connection with type %s doesn\'t exist', $connectionType)
+                    sprintf('Connection with type %s does not exist', $connectionType)
                 );
         }
     }
diff --git a/src/DB/Data/RelationshipConnectionFactory.php b/src/DB/Data/RelationshipConnectionFactory.php
@@ -66,7 +66,7 @@ public function create(string $connectionType): ConnectionInterface
                 break;
             default:
                 throw new \RuntimeException(
-                    sprintf('Connection with type %s doesn\'t exist', $connectionType)
+                    sprintf('Connection with type %s does not exist', $connectionType)
                 );
         }
 
diff --git a/src/Filesystem/DirectoryCopier/StrategyFactory.php b/src/Filesystem/DirectoryCopier/StrategyFactory.php
@@ -49,7 +49,7 @@ public function create(string $strategy): StrategyInterface
                 break;
             default:
                 throw new \RuntimeException(
-                    sprintf('Strategy "%s" doesn\'t exist', $strategy)
+                    sprintf('Strategy "%s" does not exist', $strategy)
                 );
         }
 
diff --git a/src/Step/Deploy/InstallUpdate/ConfigUpdate/DbConnection.php b/src/Step/Deploy/InstallUpdate/ConfigUpdate/DbConnection.php
@@ -142,7 +142,7 @@ public function execute()
              * not by deploy scripts.
              */
             $this->logger->notice(
-                'Database relationship configuration doesn\'t exist'
+                'Database relationship configuration does not exist'
                 . ' and database is not configured through .magento.env.yaml or env variable.'
                 . ' Will be applied the previous database configuration.'
             );
diff --git a/src/Step/Deploy/SplitDbConnection.php b/src/Step/Deploy/SplitDbConnection.php
@@ -150,7 +150,7 @@ public function execute()
 
         if (!empty($missedSplitTypes)) {
             $this->logger->warning(
-                'Variable SPLIT_DB does not have data which were already split types: '
+                'The SPLIT_DB variable is missing the configuration for split connection types: '
                 . implode(', ', $missedSplitTypes),
                 ['errorCode' => Error::WARN_NOT_ENOUGH_DATA_SPLIT_DB_VAR]
             );
diff --git a/src/Step/Deploy/SplitDbConnection/SlaveConnection.php b/src/Step/Deploy/SplitDbConnection/SlaveConnection.php
@@ -96,7 +96,8 @@ public function update()
                 $this->logger->warning(
                     sprintf(
                         'Slave connection for \'%s\' connection not set. '
-                        . 'Relationships do not have the configuration for this slave connection',
+                        . 'The `relationships` configuration in the .magento.app.yaml file '
+                        . 'is missing the configuration for this slave connection',
                         $mageConnectionName
                     ),
                     ['errorCode' => Error::WARN_SLAVE_CONNECTION_NOT_SET]
diff --git a/src/Test/Functional/Acceptance/ReportDirNestingLevelCest.php b/src/Test/Functional/Acceptance/ReportDirNestingLevelCest.php
@@ -167,7 +167,7 @@ public function testErrorReportDirNestingLevelNotSet(\CliTester $I): void
             $log
         );
         $I->assertContains(
-            'You can configure the setting using the `config.report.dir_nesting_level`'
+            'You can configure the setting using the `config.report.dir_nesting_level` variable'
             . ' in the file ' . $this->expectedPathLocalXml,
             $log
         );
@@ -200,7 +200,7 @@ public function testWithInvalidLocalXmlFile(\CliTester $I): void
             $log
         );
         $I->assertContains(
-            "Config of the file {$this->expectedPathLocalXml} is invalid.",
+            "Invalid configuration in the {$this->expectedPathLocalXml} file.",
             $log
         );
         $I->assertContains(
diff --git a/src/Test/Functional/Acceptance/SplitDbCest.php b/src/Test/Functional/Acceptance/SplitDbCest.php
@@ -128,7 +128,7 @@ private function variationsDataPartWithoutSplitDbArch(): array
                     'Environment configuration is not valid.',
                     'Correct the following items in your .magento.env.yaml file:',
                     'The SPLIT_DB variable contains the invalid value.',
-                    'It should be array with next available values: [quote, sales].'
+                    'It should be an array with following values: [quote, sales].'
                 ],
                 'splitDbTypes' => ['checkout'],
             ],
@@ -138,7 +138,7 @@ private function variationsDataPartWithoutSplitDbArch(): array
                     'Environment configuration is not valid.',
                     'Correct the following items in your .magento.env.yaml file:',
                     'The SPLIT_DB variable contains the invalid value.',
-                    'It should be array with next available values: [quote, sales].',
+                    'It should be an array with following values: [quote, sales].',
                 ],
                 'splitDbTypes' => ['quote', 'checkout'],
             ]
@@ -162,13 +162,13 @@ private function variationsDataPartWithSplitDbArch(): array
             ],
             'Split Db type was deleted' => [
                 'splitDbTypes' => null,
-                'messages' => 'Variable SPLIT_DB does not have data which were already split types: quote',
+                'messages' => 'The SPLIT_DB variable is missing the configuration for split connection types: quote',
                 'expectedExists' => ['checkout'],
                 'expectedNotExist' => ['sales'],
             ],
             'Split Db  current type was deleted and new type added' => [
                 'splitDbTypes' => ['sales'],
-                'messages' => 'Variable SPLIT_DB does not have data which were already split types: quote',
+                'messages' => 'The SPLIT_DB variable is missing the configuration for split connection types: quote',
                 'expectedExists' => ['checkout'],
                 'expectedNotExist' => ['sales'],
             ],
diff --git a/src/Test/Integration/Schema/ValidatorTest.php b/src/Test/Integration/Schema/ValidatorTest.php
@@ -320,15 +320,15 @@ public function validateDataProvider(): array
                 DeployInterface::VAR_SPLIT_DB,
                 ['wrong'],
                 new Error(
-                    'The SPLIT_DB variable contains the invalid value. It should be array with next available values: '
+                    'The SPLIT_DB variable contains the invalid value. It should be an array with following values: '
                     . '[quote, sales].'
                 )
             ],
             [
                 DeployInterface::VAR_SPLIT_DB,
                 ['wrong', DeployInterface::SPLIT_DB_VALUE_QUOTE],
                 new Error(
-                    'The SPLIT_DB variable contains the invalid value. It should be array with next available values: '
+                    'The SPLIT_DB variable contains the invalid value. It should be an array with following values: '
                     . '[quote, sales].'
                 )
             ],
diff --git a/src/Test/Unit/Command/ErrorShowTest.php b/src/Test/Unit/Command/ErrorShowTest.php
@@ -153,6 +153,6 @@ public function testExecuteWithoutCodeEmptyLog()
         $tester = new CommandTester($this->command);
         $tester->execute([]);
 
-        $this->assertContains('The error log is empty or doesn\'t exist', $tester->getDisplay());
+        $this->assertContains('The error log is empty or does not exist', $tester->getDisplay());
     }
 }
diff --git a/src/Test/Unit/Config/Schema/Validator/SplitDbTest.php b/src/Test/Unit/Config/Schema/Validator/SplitDbTest.php
@@ -51,7 +51,7 @@ public function testValidateWithError(): void
         $this->assertEquals(
             new Error(
                 'The SOME_VARIABLE variable contains the invalid value.'
-                .' It should be array with next available values: [quote, sales].'
+                .' It should be an array with following values: [quote, sales].'
             ),
             $this->validator->validate('SOME_VARIABLE', ['invalid_value'])
         );
diff --git a/src/Test/Unit/Config/Validator/Deploy/ElasticSearchVersionTest.php b/src/Test/Unit/Config/Validator/Deploy/ElasticSearchVersionTest.php
@@ -135,10 +135,10 @@ public function testValidatePackageNotExists(): void
         $this->managerMock->expects($this->once())
             ->method('get')
             ->with('elasticsearch/elasticsearch')
-            ->willThrowException(new UndefinedPackageException('package doesn\'t exist'));
+            ->willThrowException(new UndefinedPackageException('package does not exist'));
         $this->loggerMock->expects($this->once())
             ->method('warning')
-            ->with('Can\'t validate version of elasticsearch: package doesn\'t exist');
+            ->with('Can\'t validate version of elasticsearch: package does not exist');
         $this->resultFactoryMock->expects($this->once())
             ->method('success');
 
diff --git a/src/Test/Unit/Config/Validator/Deploy/ReportDirNestingLevelTest.php b/src/Test/Unit/Config/Validator/Deploy/ReportDirNestingLevelTest.php
@@ -140,7 +140,7 @@ public function testValidateWithoutEnvVarWithoutConfigValue()
             ->method('error')
             ->with(
                 'The directory nesting level value for error reporting has not been configured.',
-                'You can configure the setting using the `config.report.dir_nesting_level`'
+                'You can configure the setting using the `config.report.dir_nesting_level` variable'
                 . ' in the file ' . $this->reportConfigFile
             );
 
@@ -166,7 +166,7 @@ public function testValidateWithoutEnvVarWithInvalidConfigFile()
         $this->resultFactoryMock->expects($this->once())
             ->method('error')
             ->with(
-                sprintf('Config of the file %s is invalid. Invalid xml', $this->reportConfigFile),
+                sprintf('Invalid configuration in the %s file. Invalid xml', $this->reportConfigFile),
                 'Fix the directory nesting level configuration for error reporting in the file '
                 . $this->reportConfigFile
             );
diff --git a/src/Test/Unit/Config/Validator/Deploy/SolrIntegrityTest.php b/src/Test/Unit/Config/Validator/Deploy/SolrIntegrityTest.php
@@ -111,7 +111,7 @@ public function testConfigSolr21()
                 [
                     'error' => 'Configuration for Solr was found in .magento.app.yaml.',
                     'suggestion' => 'Solr support has been deprecated in Magento 2.1. ' .
-                        'Update your search engine to Elasticsearch and remove this relationship.',
+                        'Remove this relationship and use Elasticsearch.',
                     'errorCode' => \Magento\MagentoCloud\App\Error::WARN_SOLR_DEPRECATED
                 ]
             )->willReturn($this->createMock(Error::class));
diff --git a/src/Test/Unit/DB/Data/ConnectionFactoryTest.php b/src/Test/Unit/DB/Data/ConnectionFactoryTest.php
@@ -67,7 +67,7 @@ public function testCreateSlaveAsMain()
     public function testCreateWithException()
     {
         $this->expectException(\RuntimeException::class);
-        $this->expectExceptionMessage('Connection with type dummy doesn\'t exist');
+        $this->expectExceptionMessage('Connection with type dummy does not exist');
         $this->factory->create('dummy');
     }
 }
diff --git a/src/Test/Unit/DB/Data/RelationshipConnectionFactoryTest.php b/src/Test/Unit/DB/Data/RelationshipConnectionFactoryTest.php
@@ -93,7 +93,7 @@ public function dataProvider(): array
     public function testCreateWithException()
     {
         $this->expectException(\RuntimeException::class);
-        $this->expectExceptionMessage('Connection with type dummy doesn\'t exist');
+        $this->expectExceptionMessage('Connection with type dummy does not exist');
 
         $this->factory->create('dummy');
     }
diff --git a/src/Test/Unit/Filesystem/DirectoryCopier/StrategyFactoryTest.php b/src/Test/Unit/Filesystem/DirectoryCopier/StrategyFactoryTest.php
@@ -88,7 +88,7 @@ public function createDataProvider(): array
     public function testCopyFromDirNotExists(): void
     {
         $this->expectException(\RuntimeException::class);
-        $this->expectExceptionMessage('Strategy "not_exists_strategy" doesn\'t exist');
+        $this->expectExceptionMessage('Strategy "not_exists_strategy" does not exist');
 
         $this->strategyFactory->create('not_exists_strategy');
     }
diff --git a/src/Test/Unit/Scenario/Collector/ScenarioTest.php b/src/Test/Unit/Scenario/Collector/ScenarioTest.php
@@ -85,7 +85,7 @@ public function testCollect()
     public function testCollectWithException()
     {
         $this->expectException(ValidationException::class);
-        $this->expectExceptionMessage('File doesn\'t exist');
+        $this->expectExceptionMessage('File does not exist');
 
         $scenario = '/path/to/scenario';
 
@@ -95,7 +95,7 @@ public function testCollectWithException()
             ->willReturn($scenario);
         $this->fileMock->expects($this->once())
             ->method('fileGetContents')
-            ->willThrowException(new FileSystemException('File doesn\'t exist'));
+            ->willThrowException(new FileSystemException('File does not exist'));
 
         $this->scenarioCollector->collect($scenario);
     }
diff --git a/src/Test/Unit/Step/Deploy/InstallUpdate/ConfigUpdate/DbConnectionTest.php b/src/Test/Unit/Step/Deploy/InstallUpdate/ConfigUpdate/DbConnectionTest.php
@@ -166,7 +166,7 @@ public function testExecuteWithoutDbConfigInEnvironment()
             ->willReturn([]);
         $this->loggerMock->expects($this->once())
             ->method('notice')
-            ->with('Database relationship configuration doesn\'t exist'
+            ->with('Database relationship configuration does not exist'
                 . ' and database is not configured through .magento.env.yaml or env variable.'
                 . ' Will be applied the previous database configuration.');
         $this->flagManagerMock->expects($this->never())
diff --git a/src/Test/Unit/Step/Deploy/SplitDbConnection/SlaveConnectionTest.php b/src/Test/Unit/Step/Deploy/SplitDbConnection/SlaveConnectionTest.php
@@ -194,11 +194,13 @@ public function testUpdateSlaveConnectionsNotAvailable()
             ->withConsecutive(
                 [
                     'Slave connection for \'checkout\' connection not set.'
-                    . ' Relationships do not have the configuration for this slave connection'
+                    . ' The `relationships` configuration in the .magento.app.yaml file'
+                    . ' is missing the configuration for this slave connection'
                 ],
                 [
                     'Slave connection for \'sales\' connection not set.'
-                    . ' Relationships do not have the configuration for this slave connection'
+                    . ' The `relationships` configuration in the .magento.app.yaml file'
+                    . ' is missing the configuration for this slave connection'
                 ]
             );
         $this->configWriterMock->create($mageConfig);
diff --git a/src/Test/Unit/Step/Deploy/SplitDbConnectionTest.php b/src/Test/Unit/Step/Deploy/SplitDbConnectionTest.php

Original file line number	Diff line number	Diff line change
`@@ -87,7 +87,7 @@ protected function execute(InputInterface $input, OutputInterface $output)`
`87`	`87`	`} else {`
`88`	`88`	`$errors = $this->reader->read();`
`89`	`89`	`if (empty($errors)) {`
`90`		`- $output->writeln('The error log is empty or doesn\'t exist');`
	`90`	`+ $output->writeln('The error log is empty or does not exist');`
`91`	`91`
`92`	`92`	`return 1;`
`93`	`93`	`}`
Original file line number	Diff line number	Diff line change
`@@ -66,8 +66,7 @@ public function validate(): ResultInterface`
`66`	`66`
`67`	`67`	`return $this->resultFactory->error(`
`68`	`68`	`'Debug logging is enabled in Magento',`
`69`		`- 'To save space in Magento Cloud, '`
`70`		`- . 'debug logging should not be enabled for your production environments.',`
	`69`	`+ 'To save space in Magento Cloud, disable debug logging for your production environments.',`
`71`	`70`	`Error::WARN_DEBUG_LOG_ENABLED`
`72`	`71`	`);`
`73`	`72`	`}`
Original file line number	Diff line number	Diff line change
`@@ -74,7 +74,7 @@ public function validate(): Validator\ResultInterface`
`74`	`74`
`75`	`75`	`return $this->resultFactory->error(`
`76`	`76`	`'Elasticsearch service is installed at infrastructure layer but is not used as a search engine.',`
`77`		`- 'Consider removing elasticsearch service from infrastructure layer for optimized resource usage.',`
	`77`	`+ 'Consider removing the Elasticsearch service from the infrastructure layer for optimized resource usage.',`
`78`	`78`	`Error::WARN_ES_INSTALLED_BUT_NOT_USED`
`79`	`79`	`);`
`80`	`80`	`}`
Original file line number	Diff line number	Diff line change
`@@ -102,7 +102,7 @@ public function validate(): Validator\ResultInterface`
`102`	`102`	`$currentPhpConstraint->getPrettyString()`
`103`	`103`	`),`
`104`	`104`	`"Change the version of PHP to the version that satisfies the restriction conditions.\n"`
`105`		`- . 'Change the PHP version in the .magento.app.yaml file or create a support ticket',`
	`105`	`+ . 'Change the PHP version in the .magento.app.yaml file, or create a support ticket',`
`106`	`106`	`Error::WARN_UPDATE_PHP_VERSION`
`107`	`107`	`);`
`108`	`108`	`}`
Original file line number	Diff line number	Diff line change
`@@ -63,7 +63,7 @@ public function validate(): ResultInterface`
`63`	`63`
`64`	`64`	`if ($this->magentoVersion->satisfies('2.1.*')) {`
`65`	`65`	`$args['suggestion'] = 'Solr support has been deprecated in Magento 2.1. ' .`
`66`		`- 'Update your search engine to Elasticsearch and remove this relationship.';`
	`66`	`+ 'Remove this relationship and use Elasticsearch.';`
`67`	`67`	`$args['errorCode'] = Error::WARN_SOLR_DEPRECATED;`
`68`	`68`	`}`
`69`	`69`
Original file line number	Diff line number	Diff line change
`@@ -68,7 +68,7 @@ public function create(string $connectionType): ConnectionInterface`
`68`	`68`	`return $this->getConnectionData(DbConfig::CONNECTION_SALES, false);`
`69`	`69`	`default:`
`70`	`70`	`throw new RuntimeException(`
`71`		`- sprintf('Connection with type %s doesn\'t exist', $connectionType)`
	`71`	`+ sprintf('Connection with type %s does not exist', $connectionType)`
`72`	`72`	`);`
`73`	`73`	`}`
`74`	`74`	`}`