Date: Fri, 4 Sep 2020 09:09:11 +0000
Subject: [PATCH 171/336] TE-7001 Fixed PHPStan issues.
---
src/Spryker/Zed/Development/Business/Phpstan/PhpstanRunner.php | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/Spryker/Zed/Development/Business/Phpstan/PhpstanRunner.php b/src/Spryker/Zed/Development/Business/Phpstan/PhpstanRunner.php
index 64750eb9..2ef4a8a4 100644
--- a/src/Spryker/Zed/Development/Business/Phpstan/PhpstanRunner.php
+++ b/src/Spryker/Zed/Development/Business/Phpstan/PhpstanRunner.php
@@ -203,6 +203,7 @@ protected function runCommand(
$configFilePath .= $this->config->getPhpstanConfigFilename();
$command = sprintf($command, static::MEMORY_LIMIT, $configFilePath, $path, $level);
+ echo '' . PHP_EOL . \Symfony\Component\VarDumper\VarDumper::dump($command) . PHP_EOL . 'Line: ' . __LINE__ . PHP_EOL . 'File: ' . __FILE__ . die();
if ($input->getOption(static::OPTION_DRY_RUN)) {
$output->writeln($command);
From 1f963b8932a9e6a403c1ee71bfc7234f5b886861 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Ren=C3=A9=20Klatt?=
Date: Fri, 4 Sep 2020 11:24:34 +0000
Subject: [PATCH 172/336] TE-7001 Fixed issue with Translator with
ValidatorBuilder.
---
src/Spryker/Zed/Development/Business/Phpstan/PhpstanRunner.php | 1 -
1 file changed, 1 deletion(-)
diff --git a/src/Spryker/Zed/Development/Business/Phpstan/PhpstanRunner.php b/src/Spryker/Zed/Development/Business/Phpstan/PhpstanRunner.php
index 2ef4a8a4..64750eb9 100644
--- a/src/Spryker/Zed/Development/Business/Phpstan/PhpstanRunner.php
+++ b/src/Spryker/Zed/Development/Business/Phpstan/PhpstanRunner.php
@@ -203,7 +203,6 @@ protected function runCommand(
$configFilePath .= $this->config->getPhpstanConfigFilename();
$command = sprintf($command, static::MEMORY_LIMIT, $configFilePath, $path, $level);
- echo '' . PHP_EOL . \Symfony\Component\VarDumper\VarDumper::dump($command) . PHP_EOL . 'Line: ' . __LINE__ . PHP_EOL . 'File: ' . __FILE__ . die();
if ($input->getOption(static::OPTION_DRY_RUN)) {
$output->writeln($command);
From f89e58e7ae0733b364c1745a1cd22508da7677a7 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Ren=C3=A9=20Klatt?=
Date: Tue, 15 Sep 2020 05:01:13 +0000
Subject: [PATCH 173/336] TE-7001 Fixed some minor issues.
---
phpstan.neon | 2 ++
.../Zed/Development/Business/Phpstan/PhpstanRunner.php | 4 +++-
2 files changed, 5 insertions(+), 1 deletion(-)
diff --git a/phpstan.neon b/phpstan.neon
index 3a7f0914..1eacc0d3 100644
--- a/phpstan.neon
+++ b/phpstan.neon
@@ -1,6 +1,8 @@
parameters:
+ reportUnmatchedIgnoredErrors: false
ignoreErrors:
- '#Constant T_SEMICOLON not found.#'
- '#Constant T_OPEN_PARENTHESIS not found.#'
- '#Constant T_OPEN_SQUARE_BRACKET not found.#'
- '#Call to an undefined method ReflectionType::getName\(\)#'
+ - "#Call to function method_exists\\(\\) with Symfony\\\\Component\\\\Console\\\\Helper\\\\ProgressBar and 'maxSecondsBetweenRe…' will always evaluate to false.#"
diff --git a/src/Spryker/Zed/Development/Business/Phpstan/PhpstanRunner.php b/src/Spryker/Zed/Development/Business/Phpstan/PhpstanRunner.php
index 64750eb9..e1d8d125 100644
--- a/src/Spryker/Zed/Development/Business/Phpstan/PhpstanRunner.php
+++ b/src/Spryker/Zed/Development/Business/Phpstan/PhpstanRunner.php
@@ -157,7 +157,9 @@ protected function getProgressBar(OutputInterface $output, int $stepsCount): Pro
$progressBar = new ProgressBar($progressBarOutput, $stepsCount);
$progressBar->setRedrawFrequency(static::PROGRESS_BAR_FREQUENCY);
- $progressBar->maxSecondsBetweenRedraws(static::PROGRESS_BAR_SECONDS_FORCE_REDRAW);
+ if (method_exists($progressBar, 'maxSecondsBetweenRedraws')) {
+ $progressBar->maxSecondsBetweenRedraws(static::PROGRESS_BAR_SECONDS_FORCE_REDRAW);
+ }
return $progressBar;
}
From fb01a3e8a793ee085b7ae80a5075ec3c2af45669 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Ren=C3=A9=20Klatt?=
Date: Tue, 15 Sep 2020 13:29:28 +0000
Subject: [PATCH 174/336] TE-7001 Reverted composer.json constraint for
spryker/symfony.
---
composer.json | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/composer.json b/composer.json
index 0ce446ff..7be11349 100644
--- a/composer.json
+++ b/composer.json
@@ -10,7 +10,7 @@
"spryker/graph": "^3.0.0",
"spryker/kernel": "^3.52.0",
"spryker/module-finder": "^1.0.0",
- "spryker/symfony": "^3.0.0 || ^4.0.0",
+ "spryker/symfony": "^3.0.0",
"spryker/twig": "^3.0.0",
"spryker/zend": "^2.0.0"
},
From c96a937d6abe4809b13a3ee4fc6c9eba5ce86b31 Mon Sep 17 00:00:00 2001
From: mscherer
Date: Thu, 24 Sep 2020 00:57:00 +0200
Subject: [PATCH 175/336] Set PHPStan levels higher where possible.
---
src/Spryker/Zed/Development/DevelopmentConfig.php | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/src/Spryker/Zed/Development/DevelopmentConfig.php b/src/Spryker/Zed/Development/DevelopmentConfig.php
index e3651279..d6ac77da 100644
--- a/src/Spryker/Zed/Development/DevelopmentConfig.php
+++ b/src/Spryker/Zed/Development/DevelopmentConfig.php
@@ -557,13 +557,15 @@ public function getArchitectureSnifferDefaultPriority(): int
/**
* Gets PHPStan default level. The higher, the better.
*
+ * Recommended level is 8 (include nullable safety).
+ *
* @api
*
* @return int
*/
public function getPhpstanLevel()
{
- return 3;
+ return 7;
}
/**
From 705d682b7d0a8e91345c0f66ae0e896ade66837e Mon Sep 17 00:00:00 2001
From: mscherer
Date: Thu, 24 Sep 2020 16:11:58 +0200
Subject: [PATCH 176/336] Set PHPStan levels higher where possible.
---
phpstan.json | 4 ++--
phpstan.neon | 2 ++
2 files changed, 4 insertions(+), 2 deletions(-)
diff --git a/phpstan.json b/phpstan.json
index 5853d235..cca5ff36 100644
--- a/phpstan.json
+++ b/phpstan.json
@@ -1,3 +1,3 @@
{
- "defaultLevel": 5
-}
+ "defaultLevel": 6
+}
\ No newline at end of file
diff --git a/phpstan.neon b/phpstan.neon
index 3a7f0914..4490f93f 100644
--- a/phpstan.neon
+++ b/phpstan.neon
@@ -1,4 +1,6 @@
parameters:
+ checkGenericClassInNonGenericObjectType: false
+ checkMissingIterableValueType: false
ignoreErrors:
- '#Constant T_SEMICOLON not found.#'
- '#Constant T_OPEN_PARENTHESIS not found.#'
From 3272fe80feecb100d8e8794d65792d1eb8b9a8e8 Mon Sep 17 00:00:00 2001
From: Bezpiatov
Date: Tue, 6 Oct 2020 12:05:32 +0300
Subject: [PATCH 177/336] TE-7283 Removed Phantoman dependencies from all
modules
---
codeception.yml | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/codeception.yml b/codeception.yml
index 0f1f5c47..49b75187 100644
--- a/codeception.yml
+++ b/codeception.yml
@@ -15,8 +15,8 @@ settings:
extensions:
enabled:
- - Codeception\Extension\Phantoman
+ - SprykerTest\Shared\Testify\Helper\WebDriverHelper
config:
- Codeception\Extension\Phantoman:
+ SprykerTest\Shared\Testify\Helper\WebDriverHelper:
suites:
- Presentation
From a86e987ca3203d7a18795c25d39f26ac1c70882b Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Ren=C3=A9=20Klatt?=
Date: Tue, 6 Oct 2020 06:06:01 +0000
Subject: [PATCH 178/336] TE-7611 Added ExternalDependencyParser. TE-7611 Add
unmapped dependencies. TE-7611 Removed obsolete dependencies.
---
composer.json | 1 +
.../ExternalDependencyParser.php | 102 ++++++++++++++++++
.../ExternalDependencyParserInterface.php | 18 ++++
.../DependencyContainer.php | 6 +-
.../CodeceptionDependencyFinder.php | 2 +-
.../ExternalDependencyFinder.php | 76 ++++++++++++-
.../MappedDependencyFinder.php | 77 +++++++++++++
.../ModuleParser/UseStatementParser.php | 4 +-
.../Validator/DependencyValidator.php | 7 +-
...alidationRuleIsOptionalButNotSuggested.php | 2 +-
.../ComposerDependencyParser.php | 39 ++++---
.../Business/DevelopmentBusinessFactory.php | 21 ++++
.../DependencyViolationFinderConsole.php | 1 -
.../Zed/Development/DevelopmentConfig.php | 100 +++++++++++------
14 files changed, 390 insertions(+), 66 deletions(-)
create mode 100644 src/Spryker/Zed/Development/Business/Dependency/ComposerParser/ExternalDependencyParser.php
create mode 100644 src/Spryker/Zed/Development/Business/Dependency/ComposerParser/ExternalDependencyParserInterface.php
create mode 100644 src/Spryker/Zed/Development/Business/Dependency/DependencyFinder/MappedDependencyFinder.php
diff --git a/composer.json b/composer.json
index 7be11349..941e44e3 100644
--- a/composer.json
+++ b/composer.json
@@ -6,6 +6,7 @@
"require": {
"nette/di": "^2.4.7 || ^3.0",
"php": ">=7.2",
+ "phpmd/phpmd": "^2.0.0",
"spryker/config": "^3.0.0",
"spryker/graph": "^3.0.0",
"spryker/kernel": "^3.52.0",
diff --git a/src/Spryker/Zed/Development/Business/Dependency/ComposerParser/ExternalDependencyParser.php b/src/Spryker/Zed/Development/Business/Dependency/ComposerParser/ExternalDependencyParser.php
new file mode 100644
index 00000000..ffb87ba5
--- /dev/null
+++ b/src/Spryker/Zed/Development/Business/Dependency/ComposerParser/ExternalDependencyParser.php
@@ -0,0 +1,102 @@
+config = $config;
+ }
+
+ /**
+ * @param string $className
+ *
+ * @return string|null
+ */
+ public function findPackageNameByNamespace(string $className): ?string
+ {
+ if (isset($this->resolvedComposerNameByClassNameMap[$className])) {
+ return $this->resolvedComposerNameByClassNameMap[$className];
+ }
+
+ if ($this->isPhpInternalClass($className)) {
+ return null;
+ }
+
+ try {
+ $reflectionClass = new ReflectionClass($className);
+ } catch (Throwable $throwable) {
+ return null;
+ }
+
+ $pathFragments = explode(DIRECTORY_SEPARATOR, $reflectionClass->getFileName());
+ $vendorPosition = array_search('vendor', $pathFragments);
+
+ if ($vendorPosition === false) {
+ return null;
+ }
+
+ $vendorNameAndPackageName = array_slice($pathFragments, $vendorPosition + 1, 2);
+
+ $composerName = implode('/', $vendorNameAndPackageName);
+
+ $composerName = $this->mapExternalToInternalPackageName($composerName);
+
+ $this->resolvedComposerNameByClassNameMap[$className] = $composerName;
+
+ return $composerName;
+ }
+
+ /**
+ * @param string $className
+ *
+ * @return bool
+ */
+ protected function isPhpInternalClass(string $className): bool
+ {
+ if (strpos($className, '\\') === false) {
+ return true;
+ }
+
+ return false;
+ }
+
+ /**
+ * @param string $composerName
+ *
+ * @return string
+ */
+ protected function mapExternalToInternalPackageName(string $composerName): string
+ {
+ $map = $this->config->getExternalToInternalMap();
+
+ if (isset($map[$composerName])) {
+ return $map[$composerName];
+ }
+
+ return $composerName;
+ }
+}
diff --git a/src/Spryker/Zed/Development/Business/Dependency/ComposerParser/ExternalDependencyParserInterface.php b/src/Spryker/Zed/Development/Business/Dependency/ComposerParser/ExternalDependencyParserInterface.php
new file mode 100644
index 00000000..7f76fa20
--- /dev/null
+++ b/src/Spryker/Zed/Development/Business/Dependency/ComposerParser/ExternalDependencyParserInterface.php
@@ -0,0 +1,18 @@
+dependencyCollectionTransfer->getDependencyModules() as $dependencyModuleTransfer) {
- if ($dependencyModuleTransfer->getModule() === $dependencyTransfer->getModule()) {
+ if ($dependencyTransfer->getComposerName() === null && $dependencyModuleTransfer->getModule() === $dependencyTransfer->getModule()) {
+ return $dependencyModuleTransfer;
+ }
+
+ if ($dependencyTransfer->getComposerName() !== null && $dependencyModuleTransfer->getComposerName() === $dependencyTransfer->getComposerName()) {
return $dependencyModuleTransfer;
}
}
diff --git a/src/Spryker/Zed/Development/Business/Dependency/DependencyFinder/CodeceptionDependencyFinder.php b/src/Spryker/Zed/Development/Business/Dependency/DependencyFinder/CodeceptionDependencyFinder.php
index 321e1150..a7c9593f 100644
--- a/src/Spryker/Zed/Development/Business/Dependency/DependencyFinder/CodeceptionDependencyFinder.php
+++ b/src/Spryker/Zed/Development/Business/Dependency/DependencyFinder/CodeceptionDependencyFinder.php
@@ -50,7 +50,7 @@ public function findDependencies(DependencyFinderContextInterface $context, Depe
{
if (preg_match_all('/SprykerTest\\\\(.*?)\\\\(.*?)\\\\/', $context->getFileInfo()->getContents(), $matches, PREG_SET_ORDER)) {
foreach ($matches as $match) {
- $dependencyContainer->addDependency($match[2], $this->getType(), false, true);
+ $dependencyContainer->addDependency(sprintf('spryker/%s', $this->getFilter()->filter($match[2])), $this->getType(), false, true);
}
}
diff --git a/src/Spryker/Zed/Development/Business/Dependency/DependencyFinder/ExternalDependencyFinder.php b/src/Spryker/Zed/Development/Business/Dependency/DependencyFinder/ExternalDependencyFinder.php
index 80cbed88..dbe19a15 100644
--- a/src/Spryker/Zed/Development/Business/Dependency/DependencyFinder/ExternalDependencyFinder.php
+++ b/src/Spryker/Zed/Development/Business/Dependency/DependencyFinder/ExternalDependencyFinder.php
@@ -8,6 +8,7 @@
namespace Spryker\Zed\Development\Business\Dependency\DependencyFinder;
use Exception;
+use Spryker\Zed\Development\Business\Dependency\ComposerParser\ExternalDependencyParserInterface;
use Spryker\Zed\Development\Business\Dependency\DependencyContainer\DependencyContainerInterface;
use Spryker\Zed\Development\Business\Dependency\DependencyFinder\Context\DependencyFinderContextInterface;
use Spryker\Zed\Development\Business\Dependency\ModuleParser\UseStatementParserInterface;
@@ -23,18 +24,33 @@ class ExternalDependencyFinder extends AbstractFileDependencyFinder
*/
protected $useStatementParser;
+ /**
+ * @var \Spryker\Zed\Development\Business\Dependency\ComposerParser\ExternalDependencyParserInterface
+ */
+ protected $externalDependencyParser;
+
/**
* @var \Spryker\Zed\Development\DevelopmentConfig
*/
protected $config;
+ /**
+ * @var array
+ */
+ protected $foundPackagesByUseStatement = [];
+
/**
* @param \Spryker\Zed\Development\Business\Dependency\ModuleParser\UseStatementParserInterface $useStatementParser
+ * @param \Spryker\Zed\Development\Business\Dependency\ComposerParser\ExternalDependencyParserInterface $externalDependencyParser
* @param \Spryker\Zed\Development\DevelopmentConfig $config
*/
- public function __construct(UseStatementParserInterface $useStatementParser, DevelopmentConfig $config)
- {
+ public function __construct(
+ UseStatementParserInterface $useStatementParser,
+ ExternalDependencyParserInterface $externalDependencyParser,
+ DevelopmentConfig $config
+ ) {
$this->useStatementParser = $useStatementParser;
+ $this->externalDependencyParser = $externalDependencyParser;
$this->config = $config;
}
@@ -116,8 +132,18 @@ protected function getModuleNamesFromUseStatements(array $useStatements): array
{
$dependentModules = [];
foreach ($useStatements as $useStatement) {
- if ($this->isExternalDependency($useStatement)) {
+ if ($this->isMappedExternalDependency($useStatement)) {
$dependentModules[] = $this->getDependentModule($useStatement);
+
+ continue;
+ }
+
+ if ($this->isInternalNamespace($useStatement)) {
+ continue;
+ }
+
+ if ($this->isUnMappedExternalDependency($useStatement)) {
+ $dependentModules[] = $this->externalDependencyParser->findPackageNameByNamespace($useStatement);
}
}
@@ -129,7 +155,27 @@ protected function getModuleNamesFromUseStatements(array $useStatements): array
*
* @return bool
*/
- protected function isExternalDependency(string $useStatement): bool
+ protected function isInternalNamespace(string $useStatement): bool
+ {
+ if (strpos($useStatement, '\\') === false) {
+ return true;
+ }
+
+ foreach ($this->config->getInternalNamespaces() as $namespace) {
+ if (strpos($useStatement, $namespace) === 0) {
+ return true;
+ }
+ }
+
+ return false;
+ }
+
+ /**
+ * @param string $useStatement
+ *
+ * @return bool
+ */
+ protected function isMappedExternalDependency(string $useStatement): bool
{
foreach ($this->config->getExternalToInternalNamespaceMap() as $namespace => $package) {
if (strpos($useStatement, $namespace) === 0) {
@@ -140,6 +186,22 @@ protected function isExternalDependency(string $useStatement): bool
return false;
}
+ /**
+ * @param string $useStatement
+ *
+ * @return bool
+ */
+ protected function isUnMappedExternalDependency(string $useStatement): bool
+ {
+ $packageName = $this->externalDependencyParser->findPackageNameByNamespace($useStatement);
+
+ if ($packageName !== null) {
+ return true;
+ }
+
+ return false;
+ }
+
/**
* @param string $useStatement
*
@@ -155,7 +217,11 @@ protected function getDependentModule(string $useStatement): string
}
}
- throw new Exception('Could not map external to internal dependency!');
+ throw new Exception(sprintf(
+ 'Could not map "%s" to a internal dependency! Please update "%s" and add a new mapping.',
+ $useStatement,
+ '\Spryker\Zed\Development\DevelopmentConfig::getExternalToInternalNamespaceMap()'
+ ));
}
/**
diff --git a/src/Spryker/Zed/Development/Business/Dependency/DependencyFinder/MappedDependencyFinder.php b/src/Spryker/Zed/Development/Business/Dependency/DependencyFinder/MappedDependencyFinder.php
new file mode 100644
index 00000000..f2834a37
--- /dev/null
+++ b/src/Spryker/Zed/Development/Business/Dependency/DependencyFinder/MappedDependencyFinder.php
@@ -0,0 +1,77 @@
+config = $config;
+ }
+
+ /**
+ * @return string
+ */
+ public function getType(): string
+ {
+ return static::TYPE_MAPPED;
+ }
+
+ /**
+ * @param \Spryker\Zed\Development\Business\Dependency\DependencyFinder\Context\DependencyFinderContextInterface $context
+ *
+ * @return bool
+ */
+ public function accept(DependencyFinderContextInterface $context): bool
+ {
+ if ($context->getDependencyType() !== null && $context->getDependencyType() !== $this->getType()) {
+ return false;
+ }
+
+ if ($context->getFileInfo()->getFilename() !== 'composer.json') {
+ return false;
+ }
+
+ return true;
+ }
+
+ /**
+ * @param \Spryker\Zed\Development\Business\Dependency\DependencyFinder\Context\DependencyFinderContextInterface $context
+ * @param \Spryker\Zed\Development\Business\Dependency\DependencyContainer\DependencyContainerInterface $dependencyContainer
+ *
+ * @return \Spryker\Zed\Development\Business\Dependency\DependencyContainer\DependencyContainerInterface
+ */
+ public function findDependencies(DependencyFinderContextInterface $context, DependencyContainerInterface $dependencyContainer): DependencyContainerInterface
+ {
+ $fileContent = $context->getFileInfo()->getContents();
+ $composerJsonAsArray = json_decode($fileContent, true);
+ $composerName = $composerJsonAsArray['name'];
+
+ foreach ($this->config->getExternalToInternalMap() as $externalPackageName => $internalModulePackageName) {
+ if ($internalModulePackageName === $composerName) {
+ $dependencyContainer->addDependency($externalPackageName, $this->getType());
+ }
+ }
+
+ return $dependencyContainer;
+ }
+}
diff --git a/src/Spryker/Zed/Development/Business/Dependency/ModuleParser/UseStatementParser.php b/src/Spryker/Zed/Development/Business/Dependency/ModuleParser/UseStatementParser.php
index 97196a27..1e418645 100644
--- a/src/Spryker/Zed/Development/Business/Dependency/ModuleParser/UseStatementParser.php
+++ b/src/Spryker/Zed/Development/Business/Dependency/ModuleParser/UseStatementParser.php
@@ -28,10 +28,10 @@ public function getUseStatements(SplFileInfo $fileInfo): array
*/
protected function getUseStatementsInFile(SplFileInfo $fileInfo): array
{
- preg_match_all('#use (.*);#', $fileInfo->getContents(), $matches);
+ preg_match_all('/^(use\s([^\s|;]+))/m', $fileInfo->getContents(), $matches);
$useStatements = [];
- foreach ($matches[1] as $useStatement) {
+ foreach ($matches[2] as $useStatement) {
$useStatements[] = $useStatement;
}
diff --git a/src/Spryker/Zed/Development/Business/Dependency/Validator/DependencyValidator.php b/src/Spryker/Zed/Development/Business/Dependency/Validator/DependencyValidator.php
index c7058ceb..1740c900 100644
--- a/src/Spryker/Zed/Development/Business/Dependency/Validator/DependencyValidator.php
+++ b/src/Spryker/Zed/Development/Business/Dependency/Validator/DependencyValidator.php
@@ -91,15 +91,15 @@ protected function buildModuleDependencyTransferCollection(DependencyValidationR
*/
protected function getComposerDependencies(DependencyValidationRequestTransfer $dependencyValidationRequestTransfer): array
{
- $moduleDependencies = $this->moduleDependencyParser->parseOutgoingDependencies(
+ $dependencyCollectionTransfer = $this->moduleDependencyParser->parseOutgoingDependencies(
$dependencyValidationRequestTransfer->getModule(),
$dependencyValidationRequestTransfer->getDependencyType()
);
- foreach ($moduleDependencies->getDependencyModules() as $dependencyModuleTransfer) {
+ foreach ($dependencyCollectionTransfer->getDependencyModules() as $dependencyModuleTransfer) {
$composerName = $dependencyModuleTransfer->getComposerName() ?? $this->composerNameFinder->findComposerNameByModuleName($dependencyModuleTransfer->getModule());
$dependencyModuleTransfer->setComposerName($composerName);
}
- $composerDependencies = $this->composerDependencyParser->getComposerDependencyComparison($moduleDependencies);
+ $composerDependencies = $this->composerDependencyParser->getComposerDependencyComparison($dependencyCollectionTransfer);
return $composerDependencies;
}
@@ -115,7 +115,6 @@ protected function formatDependencies(array $composerDependencies): array
foreach ($composerDependencies as $composerDependency) {
$moduleDependencyTransfer = new ModuleDependencyTransfer();
$moduleDependencyTransfer
- ->setModuleName($composerDependency['moduleName'])
->setComposerName($composerDependency['composerName'])
->setIsValid(true)
->setDependencyTypes($this->getDependencyTypes($composerDependency))
diff --git a/src/Spryker/Zed/Development/Business/Dependency/Validator/ValidationRules/ValidationRuleIsOptionalButNotSuggested.php b/src/Spryker/Zed/Development/Business/Dependency/Validator/ValidationRules/ValidationRuleIsOptionalButNotSuggested.php
index 579f3ab7..73384e43 100644
--- a/src/Spryker/Zed/Development/Business/Dependency/Validator/ValidationRules/ValidationRuleIsOptionalButNotSuggested.php
+++ b/src/Spryker/Zed/Development/Business/Dependency/Validator/ValidationRules/ValidationRuleIsOptionalButNotSuggested.php
@@ -22,7 +22,7 @@ public function validateModuleDependency(ModuleDependencyTransfer $moduleDepende
if ($moduleDependencyTransfer->getIsOptionalDependency() && $moduleDependencyTransfer->getIsSrcDependency() && !$moduleDependencyTransfer->getIsSuggested()) {
$moduleDependencyTransfer->setIsValid(false);
$validationMessageTransfer = new ValidationMessageTransfer();
- $validationMessageTransfer->setMessage('Optional dependency should be listed in composer suggests');
+ $validationMessageTransfer->setMessage('Optional dependency should be listed in composer suggest');
$validationMessageTransfer->setFixType(static::ADD_SUGGEST);
$moduleDependencyTransfer->addValidationMessage($validationMessageTransfer);
diff --git a/src/Spryker/Zed/Development/Business/DependencyTree/ComposerDependencyParser.php b/src/Spryker/Zed/Development/Business/DependencyTree/ComposerDependencyParser.php
index 67ba57dc..c4d433e5 100644
--- a/src/Spryker/Zed/Development/Business/DependencyTree/ComposerDependencyParser.php
+++ b/src/Spryker/Zed/Development/Business/DependencyTree/ComposerDependencyParser.php
@@ -48,39 +48,39 @@ public function __construct(ComposerNameFinderInterface $composerNameFinder)
*/
public function getComposerDependencyComparison(DependencyCollectionTransfer $dependencyCollectionTransfer): array
{
+ // Code dependencies
$dependencyCollectionTransfer = $this->getOverwrittenDependenciesForBundle($dependencyCollectionTransfer);
- $composerDependencyCollectionTransfer = $this->getParsedComposerDependenciesForBundle($dependencyCollectionTransfer->getModule());
-
$composerNames = $this->getComposerNames($dependencyCollectionTransfer);
$composerNamesInSrc = $this->getComposerNamesForInSrcUsedModules($dependencyCollectionTransfer);
$composerNamesInTests = $this->getComposerNamesForInTestsUsedModules($dependencyCollectionTransfer);
- $suggestedNames = $this->getSuggested($composerDependencyCollectionTransfer);
- $requireNames = $this->getRequireNames($composerDependencyCollectionTransfer);
- $requireDevNames = $this->getRequireNames($composerDependencyCollectionTransfer, true);
+ // Declared composer dependencies
+ $composerDependencyCollectionTransfer = $this->parseComposerJson($dependencyCollectionTransfer->getModule());
+ $composerSuggestedNames = $this->getSuggested($composerDependencyCollectionTransfer);
+ $composerRequiredNames = $this->getRequireNames($composerDependencyCollectionTransfer);
+ $composerRequiredDevNames = $this->getRequireNames($composerDependencyCollectionTransfer, true);
- $allComposerNames = $composerNames + $requireNames + $requireDevNames + $suggestedNames;
+ $allComposerNames = array_unique(array_merge($composerNames, $composerRequiredNames, $composerRequiredDevNames, $composerSuggestedNames));
asort($allComposerNames);
$dependencies = [];
$currentComposerName = sprintf('%s/%s', $dependencyCollectionTransfer->getModule()->getOrganization()->getNameDashed(), $dependencyCollectionTransfer->getModule()->getNameDashed());
- foreach ($allComposerNames as $moduleName => $composerName) {
+ foreach ($allComposerNames as $composerName) {
if ($currentComposerName === $composerName) {
continue;
}
$dependencies[] = [
- 'moduleName' => $moduleName,
'composerName' => $composerName,
'types' => $this->getDependencyTypes($composerName, $dependencyCollectionTransfer),
'isOptional' => $this->getIsOptional($composerName, $dependencyCollectionTransfer),
'src' => in_array($composerName, $composerNamesInSrc) ? $composerName : '',
'tests' => in_array($composerName, $composerNamesInTests) ? $composerName : '',
- 'composerRequire' => in_array($composerName, $requireNames) ? $composerName : '',
- 'composerRequireDev' => in_array($composerName, $requireDevNames) ? $composerName : '',
- 'suggested' => in_array($composerName, $suggestedNames) ? $composerName : '',
+ 'composerRequire' => in_array($composerName, $composerRequiredNames) ? $composerName : '',
+ 'composerRequireDev' => in_array($composerName, $composerRequiredDevNames) ? $composerName : '',
+ 'suggested' => in_array($composerName, $composerSuggestedNames) ? $composerName : '',
'isOwnExtensionModule' => $this->isOwnExtensionModule($composerName, $dependencyCollectionTransfer),
];
}
@@ -145,7 +145,7 @@ protected function getComposerNames(DependencyCollectionTransfer $moduleDependen
{
$composerNames = [];
foreach ($moduleDependencyCollectionTransfer->getDependencyModules() as $moduleDependencyTransfer) {
- $composerNames[$moduleDependencyTransfer->getModule()] = $moduleDependencyTransfer->getComposerName();
+ $composerNames[] = $moduleDependencyTransfer->getComposerName();
}
return array_unique($composerNames);
@@ -162,7 +162,7 @@ protected function getComposerNamesForInSrcUsedModules(DependencyCollectionTrans
foreach ($moduleDependencyCollectionTransfer->getDependencyModules() as $moduleDependencyTransfer) {
foreach ($moduleDependencyTransfer->getDependencies() as $dependencyTransfer) {
if (!$dependencyTransfer->getIsInTest()) {
- $composerNames[$moduleDependencyTransfer->getModule()] = $moduleDependencyTransfer->getComposerName();
+ $composerNames[] = $moduleDependencyTransfer->getComposerName();
}
}
}
@@ -181,7 +181,7 @@ protected function getComposerNamesForInTestsUsedModules(DependencyCollectionTra
foreach ($moduleDependencyCollectionTransfer->getDependencyModules() as $moduleDependencyTransfer) {
foreach ($moduleDependencyTransfer->getDependencies() as $dependencyTransfer) {
if ($dependencyTransfer->getIsInTest()) {
- $composerNames[$moduleDependencyTransfer->getModule()] = $moduleDependencyTransfer->getComposerName();
+ $composerNames[] = $moduleDependencyTransfer->getComposerName();
}
}
}
@@ -202,7 +202,7 @@ protected function getSuggested(ComposerDependencyCollectionTransfer $composerDe
$composerNames = [];
foreach ($composerDependencyCollectionTransfer->getComposerDependencies() as $composerDependency) {
if ($composerDependency->getName() && $composerDependency->getIsOptional()) {
- $composerNames[$composerDependency->getModuleName()] = $composerDependency->getName();
+ $composerNames[] = $composerDependency->getName();
}
}
@@ -220,7 +220,7 @@ protected function getRequireNames(ComposerDependencyCollectionTransfer $compose
$composerNames = [];
foreach ($composerDependencyCollectionTransfer->getComposerDependencies() as $composerDependency) {
if ($composerDependency->getName() && $composerDependency->getIsDev() === $isDev) {
- $composerNames[$composerDependency->getModuleName()] = $composerDependency->getName();
+ $composerNames[] = $composerDependency->getName();
}
}
@@ -300,7 +300,7 @@ protected function parseDeclaredDependenciesForBundle(ModuleTransfer $moduleTran
*
* @return \Generated\Shared\Transfer\ComposerDependencyCollectionTransfer
*/
- protected function getParsedComposerDependenciesForBundle(ModuleTransfer $moduleTransfer): ComposerDependencyCollectionTransfer
+ protected function parseComposerJson(ModuleTransfer $moduleTransfer): ComposerDependencyCollectionTransfer
{
$composerDependencies = new ComposerDependencyCollectionTransfer();
@@ -343,7 +343,7 @@ protected function getParsedComposerDependenciesForBundle(ModuleTransfer $module
protected function addComposerDependencies(array $require, ComposerDependencyCollectionTransfer $composerDependencyCollectionTransfer, $isDev = false)
{
foreach ($require as $package => $version) {
- if (strpos($package, 'spryker') !== 0) {
+ if (strpos($package, '/') === false) {
continue;
}
$module = $this->getBundleName($package);
@@ -367,9 +367,6 @@ protected function addComposerDependencies(array $require, ComposerDependencyCol
protected function addSuggestedDependencies(array $suggested, ComposerDependencyCollectionTransfer $composerDependencyCollectionTransfer)
{
foreach ($suggested as $package => $description) {
- if (strpos($package, 'spryker') !== 0) {
- continue;
- }
$module = $this->getBundleName($package);
$composerDependencyTransfer = new ComposerDependencyTransfer();
diff --git a/src/Spryker/Zed/Development/Business/DevelopmentBusinessFactory.php b/src/Spryker/Zed/Development/Business/DevelopmentBusinessFactory.php
index 3ef22ca3..17d4df10 100644
--- a/src/Spryker/Zed/Development/Business/DevelopmentBusinessFactory.php
+++ b/src/Spryker/Zed/Development/Business/DevelopmentBusinessFactory.php
@@ -45,6 +45,8 @@
use Spryker\Zed\Development\Business\Composer\Validator\ComposerJsonUnboundRequireConstraintValidator;
use Spryker\Zed\Development\Business\Composer\Validator\ComposerJsonValidatorComposite;
use Spryker\Zed\Development\Business\Composer\Validator\ComposerJsonValidatorInterface;
+use Spryker\Zed\Development\Business\Dependency\ComposerParser\ExternalDependencyParser;
+use Spryker\Zed\Development\Business\Dependency\ComposerParser\ExternalDependencyParserInterface;
use Spryker\Zed\Development\Business\Dependency\DependencyContainer\DependencyContainer;
use Spryker\Zed\Development\Business\Dependency\DependencyContainer\DependencyContainerInterface;
use Spryker\Zed\Development\Business\Dependency\DependencyFinder\BehaviorDependencyFinder;
@@ -56,6 +58,7 @@
use Spryker\Zed\Development\Business\Dependency\DependencyFinder\ExternalDependencyFinder;
use Spryker\Zed\Development\Business\Dependency\DependencyFinder\InternalDependencyFinder;
use Spryker\Zed\Development\Business\Dependency\DependencyFinder\LocatorDependencyFinder;
+use Spryker\Zed\Development\Business\Dependency\DependencyFinder\MappedDependencyFinder;
use Spryker\Zed\Development\Business\Dependency\DependencyFinder\ModuleAnnotationDependencyFinder;
use Spryker\Zed\Development\Business\Dependency\DependencyFinder\PersistenceDependencyFinder;
use Spryker\Zed\Development\Business\Dependency\DependencyFinder\SprykerSdkDependencyFinder;
@@ -403,6 +406,7 @@ public function createDependencyFinder(): DependencyFinderInterface
$this->createSprykerSdkDependencyFinder(),
$this->createInternalDependencyFinder(),
$this->createExternalDependencyFinder(),
+ $this->createMappedDependencyFinder(),
$this->createExtensionDependencyFinder(),
$this->createLocatorDependencyFinder(),
$this->createPersistenceDependencyFinder(),
@@ -444,10 +448,27 @@ public function createExternalDependencyFinder(): DependencyFinderInterface
{
return new ExternalDependencyFinder(
$this->createUseStatementParser(),
+ $this->createExternalDependencyParser(),
$this->getConfig()
);
}
+ /**
+ * @return \Spryker\Zed\Development\Business\Dependency\DependencyFinder\DependencyFinderInterface
+ */
+ public function createMappedDependencyFinder(): DependencyFinderInterface
+ {
+ return new MappedDependencyFinder($this->getConfig());
+ }
+
+ /**
+ * @return \Spryker\Zed\Development\Business\Dependency\ComposerParser\ExternalDependencyParserInterface
+ */
+ public function createExternalDependencyParser(): ExternalDependencyParserInterface
+ {
+ return new ExternalDependencyParser($this->getConfig());
+ }
+
/**
* @return \Spryker\Zed\Development\Business\Dependency\DependencyFinder\DependencyFinderInterface
*/
diff --git a/src/Spryker/Zed/Development/Communication/Console/DependencyViolationFinderConsole.php b/src/Spryker/Zed/Development/Communication/Console/DependencyViolationFinderConsole.php
index c6aafa8b..3645ca88 100644
--- a/src/Spryker/Zed/Development/Communication/Console/DependencyViolationFinderConsole.php
+++ b/src/Spryker/Zed/Development/Communication/Console/DependencyViolationFinderConsole.php
@@ -190,7 +190,6 @@ protected function buildTableRow(ModuleDependencyTransfer $moduleDependencyTrans
return [
'Composer name' => sprintf('%s>', $color, $composerName),
- 'Module name' => $moduleDependencyTransfer->getModuleName(),
'is valid' => $this->getColoredYesOrNo($moduleDependencyTransfer->getIsValid()),
'src dependency' => $this->getYesOrNo($moduleDependencyTransfer->getIsSrcDependency()),
'test dependency' => $this->getYesOrNo($moduleDependencyTransfer->getIsTestDependency()),
diff --git a/src/Spryker/Zed/Development/DevelopmentConfig.php b/src/Spryker/Zed/Development/DevelopmentConfig.php
index d6ac77da..7ef87d6c 100644
--- a/src/Spryker/Zed/Development/DevelopmentConfig.php
+++ b/src/Spryker/Zed/Development/DevelopmentConfig.php
@@ -300,59 +300,100 @@ public function getPathToBundleConfig()
public function getExternalToInternalNamespaceMap()
{
return [
- 'Psr\\Log\\' => 'spryker/log',
- 'Psr\\Container\\' => 'spryker/container',
- 'Propel\\' => 'spryker/propel-orm',
- 'Silex\\' => 'spryker/silex',
- 'Spryker\\DecimalObject\\' => 'spryker/decimal-object',
- 'Pimple' => 'spryker/container',
- 'Predis\\' => 'spryker/redis',
+ 'Codeception\\' => 'spryker/testify',
+ 'CodeItNow\\BarcodeBundle' => 'spryker/code-it-now',
+ 'Doctrine\\Common\\Inflector' => 'spryker/doctrine-inflector',
+ 'DMS\\PHPUnitExtensions\\' => 'spryker/testify',
+ 'Egulias\\EmailValidator\\' => 'spryker/egulias',
+ 'Elastica\\' => 'spryker/elastica',
+ 'Faker\\' => 'spryker/testify',
'Guzzle\\' => 'spryker/guzzle',
'GuzzleHttp\\' => 'spryker/guzzle',
+ 'JsonPath\\' => 'spryker/json-path',
+ 'JsonSchema\\' => 'spryker/json-schema',
'League\\Csv\\' => 'spryker/csv',
'Monolog\\' => 'spryker/monolog',
- 'Elastica\\' => 'spryker/elastica',
+ 'org\\bovigo\\vfs\\' => 'spryker/testify',
+ 'phpDocumentor\\GraphViz\\' => 'spryker/graphviz',
+ 'Propel\\' => 'spryker/propel-orm',
+ 'PHPUnit\\' => 'spryker/testify',
+ 'Pimple' => 'spryker/container',
+ 'Predis\\' => 'spryker/redis',
+ 'Psr\\Log\\' => 'spryker/log',
+ 'Psr\\Container\\' => 'spryker/container',
+ 'Ramsey\\Uuid' => 'spryker/ramsey-uuid',
+ 'Silex\\' => 'spryker/silex',
+ 'Spryker\\DecimalObject\\' => 'spryker/decimal-object',
+ 'Symfony\\Bridge\\Twig\\' => 'spryker/symfony',
+ 'Symfony\\Bundle\\WebProfilerBundle\\' => 'spryker/symfony',
'Symfony\\Component\\' => 'spryker/symfony',
+ 'Symfony\\Contracts\\' => 'spryker/symfony',
+ 'Symfony\\Cmf\\' => 'spryker/symfony',
'Twig_' => 'spryker/twig',
'Twig\\' => 'spryker/twig',
- 'Zend\\' => 'spryker/zend',
- 'phpDocumentor\\GraphViz\\' => 'spryker/graphviz',
- 'Egulias\\EmailValidator\\' => 'spryker/egulias',
- 'Ramsey\\Uuid' => 'spryker/ramsey-uuid',
- 'Doctrine\\Common\\Inflector' => 'spryker/doctrine-inflector',
- 'JsonPath\\' => 'spryker/json-path',
- 'JsonSchema\\' => 'spryker/json-schema',
'Webmozart\\Glob' => 'spryker/util-glob',
+ 'Zend\\' => 'spryker/zend',
];
}
/**
* @api
*
- * @return array
+ * @return string[]
*/
public function getExternalToInternalMap()
{
return [
- 'psr/log' => 'spryker/log',
- 'propel/propel' => 'spryker/propel-orm',
- 'silex/silex' => 'spryker/silex',
- 'pimple/pimple' => 'spryker/container',
- 'mandrill/mandrill' => 'spryker/mandrill',
- 'predis/predis' => 'spryker/redis',
- 'guzzle/guzzle' => 'spryker/guzzle',
+ 'codeception/codeception' => 'spryker/testify',
+ 'codeitnowin/barcode' => 'spryker/code-it-now',
+ 'dms/phpunit-arraysubset-asserts' => 'spryker/testify',
+ 'doctrine/inflector' => 'spryker/doctrine-inflector',
+ 'egulias/email-validator' => 'spryker/egulias',
+ 'fzaninotto/faker' => 'spryker/testify',
+ 'galbar/jsonpath' => 'spryker/json-path',
'guzzlehttp/guzzle' => 'spryker/guzzle',
+ 'justinrainbow/json-schema' => 'spryker/json-schema',
'league/csv' => 'spryker/csv',
+ 'league/oauth2-server' => 'spryker/oauth',
+ 'mandrill/mandrill' => 'spryker/mandrill',
+ 'mikey179/vfsstream' => 'spryker/testify',
+ 'moneyphp/money' => 'spryker/money',
'monolog/monolog' => 'spryker/monolog',
- 'ruflin/elastica' => 'spryker/elastica',
- '/symfony/' => 'spryker/symfony',
- 'twig/twig' => 'spryker/twig',
- '/zendframework/' => 'spryker/zend',
+ 'pimple/pimple' => 'spryker/container',
'phpdocumentor/graphviz' => 'spryker/graphviz',
- 'egulias/email-validator' => 'spryker/egulias',
+ 'predis/predis' => 'spryker/redis',
+ 'propel/propel' => 'spryker/propel-orm',
+ 'psr/log' => 'spryker/log',
+ 'psr/container' => 'spryker/container',
'ramsey/uuid' => 'spryker/ramsey-uuid',
- 'doctrine/inflector' => 'spryker/doctrine-inflector',
+ 'ruflin/elastica' => 'spryker/elastica',
+ 'symfony-cmf/routing' => 'spryker/symfony',
+ 'symfony/console' => 'spryker/symfony',
+ 'symfony/debug' => 'spryker/symfony',
+ 'symfony/filesystem' => 'spryker/symfony',
+ 'symfony/finder' => 'spryker/symfony',
+ 'symfony/form' => 'spryker/symfony',
+ 'symfony/http-kernel' => 'spryker/symfony',
+ 'symfony/intl' => 'spryker/symfony',
+ 'symfony/mime' => 'spryker/symfony',
+ 'symfony/options-resolver' => 'spryker/symfony',
+ 'symfony/process' => 'spryker/symfony',
+ 'symfony/property-access' => 'spryker/symfony',
+ 'symfony/routing' => 'spryker/symfony',
+ 'symfony/security' => 'spryker/symfony',
+ 'symfony/serializer' => 'spryker/symfony',
+ 'symfony/stopwatch' => 'spryker/symfony',
+ 'symfony/translation' => 'spryker/symfony',
+ 'symfony/twig-bridge' => 'spryker/symfony',
+ 'symfony/validator' => 'spryker/symfony',
+ 'symfony/web-profiler-bundle' => 'spryker/web-profiler',
+ 'symfony/yaml' => 'spryker/symfony',
+ 'swiftmailer/swiftmailer' => 'spryker/mail',
+ 'twig/twig' => 'spryker/twig',
'webmozart/glob' => 'spryker/util-glob',
+ 'zendframework/zend-config' => 'spryker/zend',
+ 'zendframework/zend-filter' => 'spryker/zend',
+ 'zendframework/zend-servicemanager' => 'spryker/zend',
];
}
@@ -370,7 +411,6 @@ public function getIgnorableDependencies()
'phploc/phploc',
'phpmd/phpmd',
'sebastian/phpcpd',
- 'codeception/codeception',
'fabpot/php-cs-fixer',
'sensiolabs/security-checker',
'sllh/composer-versions-check',
From 8dd642103dd3dc39cd90e70af3919ba76b1723db Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Ren=C3=A9=20Klatt?=
Date: Thu, 8 Oct 2020 08:07:03 +0000
Subject: [PATCH 179/336] TE-7611 Fixed dependencies.
---
.../DependencyFinder/ComposerDependencyFinder.php | 10 +++++++---
.../DependencyFinder/ExternalDependencyFinder.php | 4 +++-
src/Spryker/Zed/Development/DevelopmentConfig.php | 11 ++++++++++-
3 files changed, 20 insertions(+), 5 deletions(-)
diff --git a/src/Spryker/Zed/Development/Business/Dependency/DependencyFinder/ComposerDependencyFinder.php b/src/Spryker/Zed/Development/Business/Dependency/DependencyFinder/ComposerDependencyFinder.php
index 968b771a..5f9e69d7 100644
--- a/src/Spryker/Zed/Development/Business/Dependency/DependencyFinder/ComposerDependencyFinder.php
+++ b/src/Spryker/Zed/Development/Business/Dependency/DependencyFinder/ComposerDependencyFinder.php
@@ -51,15 +51,19 @@ public function findDependencies(DependencyFinderContextInterface $context, Depe
$fileContent = $context->getFileInfo()->getContents();
if (strpos($fileContent, 'cs-check') !== false) {
- $dependencyContainer->addDependency('CodeSniffer', $this->getType(), false, true);
+ $dependencyContainer->addDependency('spryker/code-sniffer', $this->getType(), false, true);
}
if (preg_match('/code-sniffer\/(Spryker|SprykerStrict)/', $fileContent)) {
- $dependencyContainer->addDependency('CodeSniffer', $this->getType(), false, true);
+ $dependencyContainer->addDependency('spryker/code-sniffer', $this->getType(), false, true);
}
if (strpos($fileContent, 'codecept run') !== false) {
- $dependencyContainer->addDependency('Testify', $this->getType(), false, true);
+ $dependencyContainer->addDependency('spryker/testify', $this->getType(), false, true);
+ }
+
+ if (strpos($fileContent, 'phpstan analyse') !== false) {
+ $dependencyContainer->addDependency('phpstan/phpstan', $this->getType(), false, true);
}
return $dependencyContainer;
diff --git a/src/Spryker/Zed/Development/Business/Dependency/DependencyFinder/ExternalDependencyFinder.php b/src/Spryker/Zed/Development/Business/Dependency/DependencyFinder/ExternalDependencyFinder.php
index dbe19a15..9112daa9 100644
--- a/src/Spryker/Zed/Development/Business/Dependency/DependencyFinder/ExternalDependencyFinder.php
+++ b/src/Spryker/Zed/Development/Business/Dependency/DependencyFinder/ExternalDependencyFinder.php
@@ -213,7 +213,9 @@ protected function getDependentModule(string $useStatement): string
{
foreach ($this->config->getExternalToInternalNamespaceMap() as $namespace => $package) {
if (strpos($useStatement, $namespace) === 0) {
- return $this->getDependentModuleNameFromPackage($package);
+ return $package;
+
+ $this->getDependentModuleNameFromPackage($package);
}
}
diff --git a/src/Spryker/Zed/Development/DevelopmentConfig.php b/src/Spryker/Zed/Development/DevelopmentConfig.php
index 7ef87d6c..280a4170 100644
--- a/src/Spryker/Zed/Development/DevelopmentConfig.php
+++ b/src/Spryker/Zed/Development/DevelopmentConfig.php
@@ -360,6 +360,7 @@ public function getExternalToInternalMap()
'moneyphp/money' => 'spryker/money',
'monolog/monolog' => 'spryker/monolog',
'pimple/pimple' => 'spryker/container',
+ 'phpbench/phpbench' => 'spryker-sdk/benchmark',
'phpdocumentor/graphviz' => 'spryker/graphviz',
'predis/predis' => 'spryker/redis',
'propel/propel' => 'spryker/propel-orm',
@@ -368,11 +369,15 @@ public function getExternalToInternalMap()
'ramsey/uuid' => 'spryker/ramsey-uuid',
'ruflin/elastica' => 'spryker/elastica',
'symfony-cmf/routing' => 'spryker/symfony',
+ 'symfony/config' => 'spryker/symfony',
'symfony/console' => 'spryker/symfony',
'symfony/debug' => 'spryker/symfony',
+ 'symfony/error-handler' => 'spryker/symfony',
+ 'symfony/event-dispatcher' => 'spryker/symfony',
'symfony/filesystem' => 'spryker/symfony',
'symfony/finder' => 'spryker/symfony',
'symfony/form' => 'spryker/symfony',
+ 'symfony/http-foundation' => 'spryker/symfony',
'symfony/http-kernel' => 'spryker/symfony',
'symfony/intl' => 'spryker/symfony',
'symfony/mime' => 'spryker/symfony',
@@ -380,10 +385,14 @@ public function getExternalToInternalMap()
'symfony/process' => 'spryker/symfony',
'symfony/property-access' => 'spryker/symfony',
'symfony/routing' => 'spryker/symfony',
- 'symfony/security' => 'spryker/symfony',
+ 'symfony/security-core' => 'spryker/symfony',
+ 'symfony/security-csrf' => 'spryker/symfony',
+ 'symfony/security-guard' => 'spryker/symfony',
+ 'symfony/security-http' => 'spryker/symfony',
'symfony/serializer' => 'spryker/symfony',
'symfony/stopwatch' => 'spryker/symfony',
'symfony/translation' => 'spryker/symfony',
+ 'symfony/translation-contracts' => 'spryker/symfony',
'symfony/twig-bridge' => 'spryker/symfony',
'symfony/validator' => 'spryker/symfony',
'symfony/web-profiler-bundle' => 'spryker/web-profiler',
From 931b8cff968719f174b28dab280137880daa94c7 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Ren=C3=A9=20Klatt?=
Date: Fri, 9 Oct 2020 07:47:59 +0000
Subject: [PATCH 180/336] TE-7611 Added deprecation.
---
src/Spryker/Zed/Development/DevelopmentConfig.php | 2 ++
1 file changed, 2 insertions(+)
diff --git a/src/Spryker/Zed/Development/DevelopmentConfig.php b/src/Spryker/Zed/Development/DevelopmentConfig.php
index 280a4170..b8bfc6a0 100644
--- a/src/Spryker/Zed/Development/DevelopmentConfig.php
+++ b/src/Spryker/Zed/Development/DevelopmentConfig.php
@@ -409,6 +409,8 @@ public function getExternalToInternalMap()
/**
* @api
*
+ * @deprecated Will be removed without replacement.
+ *
* @return array
*/
public function getIgnorableDependencies()
From 9bf1648f540be0c52df5e3ffb02bfb6f2e3fc0b4 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Ren=C3=A9=20Klatt?=
Date: Fri, 9 Oct 2020 08:55:14 +0000
Subject: [PATCH 181/336] TE-7611 Removed dead code.
---
.../Dependency/DependencyFinder/ExternalDependencyFinder.php | 2 --
1 file changed, 2 deletions(-)
diff --git a/src/Spryker/Zed/Development/Business/Dependency/DependencyFinder/ExternalDependencyFinder.php b/src/Spryker/Zed/Development/Business/Dependency/DependencyFinder/ExternalDependencyFinder.php
index 9112daa9..3105b35c 100644
--- a/src/Spryker/Zed/Development/Business/Dependency/DependencyFinder/ExternalDependencyFinder.php
+++ b/src/Spryker/Zed/Development/Business/Dependency/DependencyFinder/ExternalDependencyFinder.php
@@ -214,8 +214,6 @@ protected function getDependentModule(string $useStatement): string
foreach ($this->config->getExternalToInternalNamespaceMap() as $namespace => $package) {
if (strpos($useStatement, $namespace) === 0) {
return $package;
-
- $this->getDependentModuleNameFromPackage($package);
}
}
From 0a47b8ff0e52a97cf835f05d55b7654c41f43799 Mon Sep 17 00:00:00 2001
From: mscherer
Date: Mon, 26 Oct 2020 11:50:44 +0100
Subject: [PATCH 182/336] Update psalm report.
---
psalm-report.json | 1430 +++++++++++++++++----------------------------
1 file changed, 538 insertions(+), 892 deletions(-)
diff --git a/psalm-report.json b/psalm-report.json
index 78295eb7..d5087baa 100644
--- a/psalm-report.json
+++ b/psalm-report.json
@@ -10,8 +10,7 @@
"snippet": " return array_filter($lookupPaths, 'glob');",
"selected_text": "'glob'",
"error_level": 6,
- "shortcode": 4,
- "taint_trace": null
+ "shortcode": 4
},
{
"severity": "error",
@@ -23,8 +22,7 @@
"snippet": " return array_filter($twigFilePaths, 'glob');",
"selected_text": "'glob'",
"error_level": 6,
- "shortcode": 4,
- "taint_trace": null
+ "shortcode": 4
},
{
"severity": "error",
@@ -36,8 +34,7 @@
"snippet": " $directories = array_filter($directories, function (string $directory) {\n return glob($directory);\n });",
"selected_text": "function (string $directory) {\n return glob($directory);\n }",
"error_level": 6,
- "shortcode": 4,
- "taint_trace": null
+ "shortcode": 4
},
{
"severity": "error",
@@ -49,8 +46,7 @@
"snippet": " ->sort($this->getFilenameSortCallback());",
"selected_text": "$this->getFilenameSortCallback()",
"error_level": 6,
- "shortcode": 4,
- "taint_trace": null
+ "shortcode": 4
},
{
"severity": "error",
@@ -62,8 +58,7 @@
"snippet": " $directories = array_filter($directories, 'glob');",
"selected_text": "'glob'",
"error_level": 6,
- "shortcode": 4,
- "taint_trace": null
+ "shortcode": 4
},
{
"severity": "error",
@@ -75,8 +70,7 @@
"snippet": " ->sort($this->getFilenameSortCallback());",
"selected_text": "$this->getFilenameSortCallback()",
"error_level": 6,
- "shortcode": 4,
- "taint_trace": null
+ "shortcode": 4
}
],
"warning": [
@@ -90,8 +84,7 @@
"snippet": " usort($rules, $sortAlphabetically);",
"selected_text": "$sortAlphabetically",
"error_level": 4,
- "shortcode": 12,
- "taint_trace": null
+ "shortcode": 12
},
{
"severity": "info",
@@ -103,8 +96,7 @@
"snippet": " $results = $this->xmlReader->fromString($output);",
"selected_text": "$output",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
@@ -116,8 +108,7 @@
"snippet": " throw new Exception('Sniffer run was not successful: ' . $process->getExitCodeText());",
"selected_text": "$process->getExitCodeText()",
"error_level": 1,
- "shortcode": 80,
- "taint_trace": null
+ "shortcode": 80
},
{
"severity": "info",
@@ -129,8 +120,7 @@
"snippet": " $fileName = $bridgeBuilderDataTransfer->getModule() . 'To' . $bridgeBuilderDataTransfer->getToModule() . $bridgeBuilderDataTransfer->getToType() . 'Interface.php';",
"selected_text": "$bridgeBuilderDataTransfer->getModule()",
"error_level": 1,
- "shortcode": 80,
- "taint_trace": null
+ "shortcode": 80
},
{
"severity": "info",
@@ -142,8 +132,7 @@
"snippet": " $fileName = $bridgeBuilderDataTransfer->getModule() . 'To' . $bridgeBuilderDataTransfer->getToModule() . $bridgeBuilderDataTransfer->getToType() . 'Interface.php';",
"selected_text": "$bridgeBuilderDataTransfer->getToModule()",
"error_level": 1,
- "shortcode": 80,
- "taint_trace": null
+ "shortcode": 80
},
{
"severity": "info",
@@ -155,8 +144,7 @@
"snippet": " $fileName = $bridgeBuilderDataTransfer->getModule() . 'To' . $bridgeBuilderDataTransfer->getToModule() . $bridgeBuilderDataTransfer->getToType() . 'Interface.php';",
"selected_text": "$bridgeBuilderDataTransfer->getToType()",
"error_level": 1,
- "shortcode": 80,
- "taint_trace": null
+ "shortcode": 80
},
{
"severity": "info",
@@ -168,8 +156,7 @@
"snippet": " $bridgeBuilderDataTransfer->getVendor() . '\\\\' .",
"selected_text": "$bridgeBuilderDataTransfer->getVendor()",
"error_level": 1,
- "shortcode": 80,
- "taint_trace": null
+ "shortcode": 80
},
{
"severity": "info",
@@ -181,8 +168,7 @@
"snippet": " $bridgeBuilderDataTransfer->getVendor() . '\\\\' .\n $bridgeBuilderDataTransfer->getApplication() . '\\\\' .\n $bridgeBuilderDataTransfer->getModule() . '\\\\' .\n 'Dependency\\\\' .\n $bridgeBuilderDataTransfer->getToType() . '\\\\' .\n $bridgeBuilderDataTransfer->getModule() . 'To' . $bridgeBuilderDataTransfer->getToModule() . $bridgeBuilderDataTransfer->getToType() . 'Interface'",
"selected_text": "$bridgeBuilderDataTransfer->getVendor() . '\\\\' .\n $bridgeBuilderDataTransfer->getApplication() . '\\\\' .\n $bridgeBuilderDataTransfer->getModule() . '\\\\' .\n 'Dependency\\\\' .\n $bridgeBuilderDataTransfer->getToType() . '\\\\' .\n $bridgeBuilderDataTransfer->getModule() . 'To' . $bridgeBuilderDataTransfer->getToModule() . $bridgeBuilderDataTransfer->getToType() . 'Interface'",
"error_level": 3,
- "shortcode": 193,
- "taint_trace": null
+ "shortcode": 193
},
{
"severity": "info",
@@ -194,8 +180,7 @@
"snippet": " $bridgeBuilderDataTransfer->getApplication() . '\\\\' .",
"selected_text": "$bridgeBuilderDataTransfer->getApplication()",
"error_level": 1,
- "shortcode": 80,
- "taint_trace": null
+ "shortcode": 80
},
{
"severity": "info",
@@ -207,8 +192,7 @@
"snippet": " $bridgeBuilderDataTransfer->getModule() . '\\\\' .",
"selected_text": "$bridgeBuilderDataTransfer->getModule()",
"error_level": 1,
- "shortcode": 80,
- "taint_trace": null
+ "shortcode": 80
},
{
"severity": "info",
@@ -220,8 +204,7 @@
"snippet": " $bridgeBuilderDataTransfer->getToType() . '\\\\' .",
"selected_text": "$bridgeBuilderDataTransfer->getToType()",
"error_level": 1,
- "shortcode": 80,
- "taint_trace": null
+ "shortcode": 80
},
{
"severity": "info",
@@ -233,8 +216,7 @@
"snippet": " $bridgeBuilderDataTransfer->getModule() . 'To' . $bridgeBuilderDataTransfer->getToModule() . $bridgeBuilderDataTransfer->getToType() . 'Interface'",
"selected_text": "$bridgeBuilderDataTransfer->getModule()",
"error_level": 1,
- "shortcode": 80,
- "taint_trace": null
+ "shortcode": 80
},
{
"severity": "info",
@@ -246,8 +228,7 @@
"snippet": " $bridgeBuilderDataTransfer->getModule() . 'To' . $bridgeBuilderDataTransfer->getToModule() . $bridgeBuilderDataTransfer->getToType() . 'Interface'",
"selected_text": "$bridgeBuilderDataTransfer->getToModule()",
"error_level": 1,
- "shortcode": 80,
- "taint_trace": null
+ "shortcode": 80
},
{
"severity": "info",
@@ -259,8 +240,7 @@
"snippet": " $bridgeBuilderDataTransfer->getModule() . 'To' . $bridgeBuilderDataTransfer->getToModule() . $bridgeBuilderDataTransfer->getToType() . 'Interface'",
"selected_text": "$bridgeBuilderDataTransfer->getToType()",
"error_level": 1,
- "shortcode": 80,
- "taint_trace": null
+ "shortcode": 80
},
{
"severity": "info",
@@ -272,8 +252,7 @@
"snippet": " $fileName = $bridgeBuilderDataTransfer->getModule() . 'To' . $bridgeBuilderDataTransfer->getToModule() . $bridgeBuilderDataTransfer->getToType() . 'Bridge.php';",
"selected_text": "$bridgeBuilderDataTransfer->getModule()",
"error_level": 1,
- "shortcode": 80,
- "taint_trace": null
+ "shortcode": 80
},
{
"severity": "info",
@@ -285,8 +264,7 @@
"snippet": " $fileName = $bridgeBuilderDataTransfer->getModule() . 'To' . $bridgeBuilderDataTransfer->getToModule() . $bridgeBuilderDataTransfer->getToType() . 'Bridge.php';",
"selected_text": "$bridgeBuilderDataTransfer->getToModule()",
"error_level": 1,
- "shortcode": 80,
- "taint_trace": null
+ "shortcode": 80
},
{
"severity": "info",
@@ -298,8 +276,7 @@
"snippet": " $fileName = $bridgeBuilderDataTransfer->getModule() . 'To' . $bridgeBuilderDataTransfer->getToModule() . $bridgeBuilderDataTransfer->getToType() . 'Bridge.php';",
"selected_text": "$bridgeBuilderDataTransfer->getToType()",
"error_level": 1,
- "shortcode": 80,
- "taint_trace": null
+ "shortcode": 80
},
{
"severity": "info",
@@ -311,8 +288,7 @@
"snippet": " $bridgeBuilderDataTransfer->getVendor() . '\\\\' .",
"selected_text": "$bridgeBuilderDataTransfer->getVendor()",
"error_level": 1,
- "shortcode": 80,
- "taint_trace": null
+ "shortcode": 80
},
{
"severity": "info",
@@ -324,8 +300,7 @@
"snippet": " $bridgeBuilderDataTransfer->getVendor() . '\\\\' .\n $bridgeBuilderDataTransfer->getApplication() . '\\\\' .\n $bridgeBuilderDataTransfer->getModule() . '\\\\' .\n 'Dependency\\\\' .\n $bridgeBuilderDataTransfer->getToType() . '\\\\' .\n $bridgeBuilderDataTransfer->getModule() . 'To' . $bridgeBuilderDataTransfer->getToModule() . $bridgeBuilderDataTransfer->getToType() . 'Bridge'",
"selected_text": "$bridgeBuilderDataTransfer->getVendor() . '\\\\' .\n $bridgeBuilderDataTransfer->getApplication() . '\\\\' .\n $bridgeBuilderDataTransfer->getModule() . '\\\\' .\n 'Dependency\\\\' .\n $bridgeBuilderDataTransfer->getToType() . '\\\\' .\n $bridgeBuilderDataTransfer->getModule() . 'To' . $bridgeBuilderDataTransfer->getToModule() . $bridgeBuilderDataTransfer->getToType() . 'Bridge'",
"error_level": 3,
- "shortcode": 193,
- "taint_trace": null
+ "shortcode": 193
},
{
"severity": "info",
@@ -337,8 +312,7 @@
"snippet": " $bridgeBuilderDataTransfer->getApplication() . '\\\\' .",
"selected_text": "$bridgeBuilderDataTransfer->getApplication()",
"error_level": 1,
- "shortcode": 80,
- "taint_trace": null
+ "shortcode": 80
},
{
"severity": "info",
@@ -350,8 +324,7 @@
"snippet": " $bridgeBuilderDataTransfer->getModule() . '\\\\' .",
"selected_text": "$bridgeBuilderDataTransfer->getModule()",
"error_level": 1,
- "shortcode": 80,
- "taint_trace": null
+ "shortcode": 80
},
{
"severity": "info",
@@ -363,8 +336,7 @@
"snippet": " $bridgeBuilderDataTransfer->getToType() . '\\\\' .",
"selected_text": "$bridgeBuilderDataTransfer->getToType()",
"error_level": 1,
- "shortcode": 80,
- "taint_trace": null
+ "shortcode": 80
},
{
"severity": "info",
@@ -376,8 +348,7 @@
"snippet": " $bridgeBuilderDataTransfer->getModule() . 'To' . $bridgeBuilderDataTransfer->getToModule() . $bridgeBuilderDataTransfer->getToType() . 'Bridge'",
"selected_text": "$bridgeBuilderDataTransfer->getModule()",
"error_level": 1,
- "shortcode": 80,
- "taint_trace": null
+ "shortcode": 80
},
{
"severity": "info",
@@ -389,8 +360,7 @@
"snippet": " $bridgeBuilderDataTransfer->getModule() . 'To' . $bridgeBuilderDataTransfer->getToModule() . $bridgeBuilderDataTransfer->getToType() . 'Bridge'",
"selected_text": "$bridgeBuilderDataTransfer->getToModule()",
"error_level": 1,
- "shortcode": 80,
- "taint_trace": null
+ "shortcode": 80
},
{
"severity": "info",
@@ -402,8 +372,7 @@
"snippet": " $bridgeBuilderDataTransfer->getModule() . 'To' . $bridgeBuilderDataTransfer->getToModule() . $bridgeBuilderDataTransfer->getToType() . 'Bridge'",
"selected_text": "$bridgeBuilderDataTransfer->getToType()",
"error_level": 1,
- "shortcode": 80,
- "taint_trace": null
+ "shortcode": 80
},
{
"severity": "info",
@@ -415,8 +384,7 @@
"snippet": " '{toModuleVariable}' => lcfirst($bridgeBuilderDataTransfer->getToModule()),",
"selected_text": "$bridgeBuilderDataTransfer->getToModule()",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
@@ -428,8 +396,7 @@
"snippet": " if ($this->getModuleLayer($bridgeBuilderDataTransfer->getToType())) {",
"selected_text": "$bridgeBuilderDataTransfer->getToType()",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
@@ -441,8 +408,7 @@
"snippet": " $replacements['{toModuleLayer}'] = '\\\\' . $this->getModuleLayer($bridgeBuilderDataTransfer->getToType());",
"selected_text": "$bridgeBuilderDataTransfer->getToType()",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
@@ -454,8 +420,7 @@
"snippet": " $pathToInternalNamespace = $this->config->getPathToInternalNamespace($bridgeBuilderDataTransfer->getVendor());",
"selected_text": "$bridgeBuilderDataTransfer->getVendor()",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
@@ -467,8 +432,7 @@
"snippet": " return $pathToInternalNamespace . $bridgeBuilderDataTransfer->getModule();",
"selected_text": "$bridgeBuilderDataTransfer->getModule()",
"error_level": 1,
- "shortcode": 80,
- "taint_trace": null
+ "shortcode": 80
},
{
"severity": "info",
@@ -480,8 +444,7 @@
"snippet": " $vendorDirectory = $this->normalizeNameForSplit($bridgeBuilderDataTransfer->getVendor());",
"selected_text": "$bridgeBuilderDataTransfer->getVendor()",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
@@ -493,8 +456,7 @@
"snippet": " $moduleDirectory = $this->normalizeNameForSplit($bridgeBuilderDataTransfer->getModule());",
"selected_text": "$bridgeBuilderDataTransfer->getModule()",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
@@ -506,8 +468,7 @@
"snippet": " $pathToInternalNamespace = $this->config->getPathToInternalNamespace($bridgeBuilderDataTransfer->getToVendor());",
"selected_text": "$bridgeBuilderDataTransfer->getToVendor()",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
@@ -519,8 +480,7 @@
"snippet": " return $pathToInternalNamespace . $bridgeBuilderDataTransfer->getToModule();",
"selected_text": "$bridgeBuilderDataTransfer->getToModule()",
"error_level": 1,
- "shortcode": 80,
- "taint_trace": null
+ "shortcode": 80
},
{
"severity": "info",
@@ -532,8 +492,7 @@
"snippet": " $vendorDirectory = $this->normalizeNameForSplit($bridgeBuilderDataTransfer->getToVendor());",
"selected_text": "$bridgeBuilderDataTransfer->getToVendor()",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
@@ -545,8 +504,7 @@
"snippet": " $moduleDirectory = $this->normalizeNameForSplit($bridgeBuilderDataTransfer->getToModule());",
"selected_text": "$bridgeBuilderDataTransfer->getToModule()",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
@@ -558,8 +516,7 @@
"snippet": " $bridgeBuilderDataTransfer->getToModule() . $bridgeBuilderDataTransfer->getToType(),",
"selected_text": "$bridgeBuilderDataTransfer->getToModule()",
"error_level": 1,
- "shortcode": 80,
- "taint_trace": null
+ "shortcode": 80
},
{
"severity": "info",
@@ -571,8 +528,7 @@
"snippet": " $bridgeBuilderDataTransfer->getToModule() . $bridgeBuilderDataTransfer->getToType(),",
"selected_text": "$bridgeBuilderDataTransfer->getToType()",
"error_level": 1,
- "shortcode": 80,
- "taint_trace": null
+ "shortcode": 80
},
{
"severity": "info",
@@ -584,8 +540,7 @@
"snippet": " $bridgeBuilderDataTransfer->getToVendor() . '\\\\' .",
"selected_text": "$bridgeBuilderDataTransfer->getToVendor()",
"error_level": 1,
- "shortcode": 80,
- "taint_trace": null
+ "shortcode": 80
},
{
"severity": "info",
@@ -597,8 +552,7 @@
"snippet": " $bridgeBuilderDataTransfer->getToApplication() . '\\\\' .",
"selected_text": "$bridgeBuilderDataTransfer->getToApplication()",
"error_level": 1,
- "shortcode": 80,
- "taint_trace": null
+ "shortcode": 80
},
{
"severity": "info",
@@ -610,8 +564,19 @@
"snippet": " $bridgeBuilderDataTransfer->getToModule() . '\\\\';",
"selected_text": "$bridgeBuilderDataTransfer->getToModule()",
"error_level": 1,
- "shortcode": 80,
- "taint_trace": null
+ "shortcode": 80
+ },
+ {
+ "severity": "info",
+ "line_from": 458,
+ "line_to": 458,
+ "type": "PossiblyNullOperand",
+ "message": "Cannot concatenate with a possibly null null|string",
+ "file_name": "vendor/spryker/spryker/Bundles/Development/src/Spryker/Zed/Development/Business/CodeBuilder/Bridge/BridgeBuilder.php",
+ "snippet": " $path .= $bridgeBuilderDataTransfer->getToModuleLayer() . '\\\\';",
+ "selected_text": "$bridgeBuilderDataTransfer->getToModuleLayer()",
+ "error_level": 1,
+ "shortcode": 80
},
{
"severity": "info",
@@ -623,8 +588,7 @@
"snippet": " $path .= $bridgeBuilderDataTransfer->getToModule() . $bridgeBuilderDataTransfer->getToType();",
"selected_text": "$bridgeBuilderDataTransfer->getToModule()",
"error_level": 1,
- "shortcode": 80,
- "taint_trace": null
+ "shortcode": 80
},
{
"severity": "info",
@@ -636,8 +600,7 @@
"snippet": " $path .= $bridgeBuilderDataTransfer->getToModule() . $bridgeBuilderDataTransfer->getToType();",
"selected_text": "$bridgeBuilderDataTransfer->getToType()",
"error_level": 1,
- "shortcode": 80,
- "taint_trace": null
+ "shortcode": 80
},
{
"severity": "info",
@@ -649,8 +612,7 @@
"snippet": " $targetBridgeClass = new ReflectionClass($path);",
"selected_text": "$path",
"error_level": 3,
- "shortcode": 193,
- "taint_trace": null
+ "shortcode": 193
},
{
"severity": "info",
@@ -662,8 +624,7 @@
"snippet": " $bridgeBuilderDataTransfer->getToVendor() . '\\\\' .",
"selected_text": "$bridgeBuilderDataTransfer->getToVendor()",
"error_level": 1,
- "shortcode": 80,
- "taint_trace": null
+ "shortcode": 80
},
{
"severity": "info",
@@ -675,8 +636,7 @@
"snippet": " $bridgeBuilderDataTransfer->getToApplication() . '\\\\' .",
"selected_text": "$bridgeBuilderDataTransfer->getToApplication()",
"error_level": 1,
- "shortcode": 80,
- "taint_trace": null
+ "shortcode": 80
},
{
"severity": "info",
@@ -688,8 +648,19 @@
"snippet": " $bridgeBuilderDataTransfer->getToModule() . '\\\\';",
"selected_text": "$bridgeBuilderDataTransfer->getToModule()",
"error_level": 1,
- "shortcode": 80,
- "taint_trace": null
+ "shortcode": 80
+ },
+ {
+ "severity": "info",
+ "line_from": 487,
+ "line_to": 487,
+ "type": "PossiblyNullOperand",
+ "message": "Cannot concatenate with a possibly null null|string",
+ "file_name": "vendor/spryker/spryker/Bundles/Development/src/Spryker/Zed/Development/Business/CodeBuilder/Bridge/BridgeBuilder.php",
+ "snippet": " $path .= $bridgeBuilderDataTransfer->getToModuleLayer() . '\\\\';",
+ "selected_text": "$bridgeBuilderDataTransfer->getToModuleLayer()",
+ "error_level": 1,
+ "shortcode": 80
},
{
"severity": "info",
@@ -701,8 +672,7 @@
"snippet": " $path .= $bridgeBuilderDataTransfer->getToModule() . $bridgeBuilderDataTransfer->getToType() . 'Interface';",
"selected_text": "$bridgeBuilderDataTransfer->getToModule()",
"error_level": 1,
- "shortcode": 80,
- "taint_trace": null
+ "shortcode": 80
},
{
"severity": "info",
@@ -714,8 +684,7 @@
"snippet": " $path .= $bridgeBuilderDataTransfer->getToModule() . $bridgeBuilderDataTransfer->getToType() . 'Interface';",
"selected_text": "$bridgeBuilderDataTransfer->getToType()",
"error_level": 1,
- "shortcode": 80,
- "taint_trace": null
+ "shortcode": 80
},
{
"severity": "info",
@@ -727,8 +696,7 @@
"snippet": " $targetBridgeInterface = new ReflectionClass($path);",
"selected_text": "$path",
"error_level": 3,
- "shortcode": 193,
- "taint_trace": null
+ "shortcode": 193
},
{
"severity": "info",
@@ -740,8 +708,7 @@
"snippet": " $finalOutput .= $this->getClassNameFromFqcn($parameter->getType()->getName()) . ' ';",
"selected_text": "getName",
"error_level": 3,
- "shortcode": 83,
- "taint_trace": null
+ "shortcode": 83
},
{
"severity": "info",
@@ -753,8 +720,7 @@
"snippet": " if ($parameter->hasType() && !$parameter->getType()->isBuiltin()) {",
"selected_text": "isBuiltin",
"error_level": 3,
- "shortcode": 83,
- "taint_trace": null
+ "shortcode": 83
},
{
"severity": "info",
@@ -766,8 +732,7 @@
"snippet": " $type = $parameter->getType()->getName();",
"selected_text": "getName",
"error_level": 3,
- "shortcode": 83,
- "taint_trace": null
+ "shortcode": 83
},
{
"severity": "info",
@@ -779,8 +744,7 @@
"snippet": " $module = strtolower($filter->filter($module));",
"selected_text": "$filter->filter($module)",
"error_level": 3,
- "shortcode": 92,
- "taint_trace": null
+ "shortcode": 92
},
{
"severity": "info",
@@ -792,8 +756,7 @@
"snippet": " if (strpos($module, '.') !== false) {",
"selected_text": "$module",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
@@ -805,8 +768,7 @@
"snippet": " [$namespace, $module] = explode('.', $module, 2);",
"selected_text": "$module",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
@@ -818,8 +780,7 @@
"snippet": " $path = $this->resolvePath($module, $namespace, $pathOption);",
"selected_text": "$module",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
@@ -831,8 +792,7 @@
"snippet": " return $this->getPathToCore($namespace, $path);",
"selected_text": "$path",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
@@ -844,8 +804,7 @@
"snippet": " return $pathToRoot . $path;",
"selected_text": "$path",
"error_level": 1,
- "shortcode": 80,
- "taint_trace": null
+ "shortcode": 80
},
{
"severity": "info",
@@ -857,8 +816,7 @@
"snippet": " return $this->buildPath($pathToInternalNamespace . $module . DIRECTORY_SEPARATOR, $pathSuffix);",
"selected_text": "$pathSuffix",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
@@ -870,8 +828,7 @@
"snippet": " return $this->buildPath($path, $pathSuffix);",
"selected_text": "$pathSuffix",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
@@ -883,8 +840,7 @@
"snippet": " return $process->getExitCode();",
"selected_text": "$process->getExitCode()",
"error_level": 5,
- "shortcode": 139,
- "taint_trace": null
+ "shortcode": 139
},
{
"severity": "info",
@@ -896,8 +852,7 @@
"snippet": " * @return int",
"selected_text": "int",
"error_level": 5,
- "shortcode": 144,
- "taint_trace": null
+ "shortcode": 144
},
{
"severity": "info",
@@ -909,8 +864,7 @@
"snippet": " return $process->getExitCode();",
"selected_text": "$process->getExitCode()",
"error_level": 5,
- "shortcode": 139,
- "taint_trace": null
+ "shortcode": 139
},
{
"severity": "info",
@@ -922,8 +876,7 @@
"snippet": " * @return int",
"selected_text": "int",
"error_level": 5,
- "shortcode": 144,
- "taint_trace": null
+ "shortcode": 144
},
{
"severity": "info",
@@ -935,8 +888,7 @@
"snippet": " return $process->getExitCode();",
"selected_text": "$process->getExitCode()",
"error_level": 5,
- "shortcode": 139,
- "taint_trace": null
+ "shortcode": 139
},
{
"severity": "info",
@@ -948,8 +900,7 @@
"snippet": " if (!$this->hasComposerJson($moduleTransfer)) {",
"selected_text": "$moduleTransfer",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
@@ -961,8 +912,7 @@
"snippet": " $composerJsonFilePath = sprintf('%s/composer.json', $moduleTransfer->getPath());",
"selected_text": "$moduleTransfer->getPath()",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
@@ -974,8 +924,7 @@
"snippet": " ->in($module->getPath())",
"selected_text": "$module->getPath()",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
@@ -987,8 +936,7 @@
"snippet": " $moduleKey = implode('.', [$moduleTransfer->getOrganization()->getName(), $moduleTransfer->getName()]);",
"selected_text": "getName",
"error_level": 3,
- "shortcode": 83,
- "taint_trace": null
+ "shortcode": 83
},
{
"severity": "info",
@@ -1000,8 +948,7 @@
"snippet": " uksort($composerJson, $callable);",
"selected_text": "$callable",
"error_level": 4,
- "shortcode": 12,
- "taint_trace": null
+ "shortcode": 12
},
{
"severity": "info",
@@ -1013,112 +960,103 @@
"snippet": " $moduleName = mb_strtolower($filter->filter(basename($composerJsonFile->getPath())));",
"selected_text": "$filter->filter(basename($composerJsonFile->getPath()))",
"error_level": 3,
- "shortcode": 92,
- "taint_trace": null
+ "shortcode": 92
},
{
"severity": "info",
- "line_from": 73,
- "line_to": 73,
+ "line_from": 78,
+ "line_to": 78,
"type": "PossiblyNullArgument",
"message": "Argument 2 of sprintf cannot be null, possibly null value provided",
"file_name": "vendor/spryker/spryker/Bundles/Development/src/Spryker/Zed/Development/Business/Composer/ComposerNameFinder.php",
"snippet": " return sprintf('%s/%s', $moduleTransfer->getOrganization()->getNameDashed(), $moduleTransfer->getNameDashed());",
"selected_text": "$moduleTransfer->getOrganization()->getNameDashed()",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
- "line_from": 73,
- "line_to": 73,
+ "line_from": 78,
+ "line_to": 78,
"type": "PossiblyNullReference",
"message": "Cannot call method getNameDashed on possibly null value",
"file_name": "vendor/spryker/spryker/Bundles/Development/src/Spryker/Zed/Development/Business/Composer/ComposerNameFinder.php",
"snippet": " return sprintf('%s/%s', $moduleTransfer->getOrganization()->getNameDashed(), $moduleTransfer->getNameDashed());",
"selected_text": "getNameDashed",
"error_level": 3,
- "shortcode": 83,
- "taint_trace": null
+ "shortcode": 83
},
{
"severity": "info",
- "line_from": 73,
- "line_to": 73,
+ "line_from": 78,
+ "line_to": 78,
"type": "PossiblyNullArgument",
"message": "Argument 3 of sprintf cannot be null, possibly null value provided",
"file_name": "vendor/spryker/spryker/Bundles/Development/src/Spryker/Zed/Development/Business/Composer/ComposerNameFinder.php",
"snippet": " return sprintf('%s/%s', $moduleTransfer->getOrganization()->getNameDashed(), $moduleTransfer->getNameDashed());",
"selected_text": "$moduleTransfer->getNameDashed()",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
- "line_from": 88,
- "line_to": 88,
+ "line_from": 93,
+ "line_to": 93,
"type": "PossiblyNullArgument",
"message": "Argument 2 of sprintf cannot be null, possibly null value provided",
"file_name": "vendor/spryker/spryker/Bundles/Development/src/Spryker/Zed/Development/Business/Composer/ComposerNameFinder.php",
"snippet": " return sprintf('%s/%s', $moduleTransfer->getOrganization()->getNameDashed(), $moduleTransfer->getNameDashed());",
"selected_text": "$moduleTransfer->getOrganization()->getNameDashed()",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
- "line_from": 88,
- "line_to": 88,
+ "line_from": 93,
+ "line_to": 93,
"type": "PossiblyNullReference",
"message": "Cannot call method getNameDashed on possibly null value",
"file_name": "vendor/spryker/spryker/Bundles/Development/src/Spryker/Zed/Development/Business/Composer/ComposerNameFinder.php",
"snippet": " return sprintf('%s/%s', $moduleTransfer->getOrganization()->getNameDashed(), $moduleTransfer->getNameDashed());",
"selected_text": "getNameDashed",
"error_level": 3,
- "shortcode": 83,
- "taint_trace": null
+ "shortcode": 83
},
{
"severity": "info",
- "line_from": 88,
- "line_to": 88,
+ "line_from": 93,
+ "line_to": 93,
"type": "PossiblyNullArgument",
"message": "Argument 3 of sprintf cannot be null, possibly null value provided",
"file_name": "vendor/spryker/spryker/Bundles/Development/src/Spryker/Zed/Development/Business/Composer/ComposerNameFinder.php",
"snippet": " return sprintf('%s/%s', $moduleTransfer->getOrganization()->getNameDashed(), $moduleTransfer->getNameDashed());",
"selected_text": "$moduleTransfer->getNameDashed()",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
- "line_from": 154,
- "line_to": 154,
+ "line_from": 159,
+ "line_to": 159,
"type": "PossiblyNullArrayOffset",
"message": "Cannot access value on variable $this->moduleTransferCollectionGroupedByModuleName[null|string] using possibly null offset null|string",
"file_name": "vendor/spryker/spryker/Bundles/Development/src/Spryker/Zed/Development/Business/Composer/ComposerNameFinder.php",
"snippet": " $this->moduleTransferCollectionGroupedByModuleName[$moduleTransfer->getName()][] = $moduleTransfer;",
"selected_text": "$this->moduleTransferCollectionGroupedByModuleName",
"error_level": 3,
- "shortcode": 125,
- "taint_trace": null
+ "shortcode": 125
},
{
"severity": "info",
- "line_from": 173,
- "line_to": 173,
+ "line_from": 178,
+ "line_to": 178,
"type": "PossiblyNullArrayOffset",
"message": "Cannot access value on variable $this->packageTransferCollectionGroupedByPackageName[null|string] using possibly null offset null|string",
"file_name": "vendor/spryker/spryker/Bundles/Development/src/Spryker/Zed/Development/Business/Composer/ComposerNameFinder.php",
"snippet": " $this->packageTransferCollectionGroupedByPackageName[$packageTransfer->getPackageName()] = $packageTransfer;",
"selected_text": "$this->packageTransferCollectionGroupedByPackageName",
"error_level": 3,
- "shortcode": 125,
- "taint_trace": null
+ "shortcode": 125
},
{
"severity": "info",
@@ -1130,8 +1068,7 @@
"snippet": " $dependentModule = strtolower($filter->filter($dependentModule));",
"selected_text": "$filter->filter($dependentModule)",
"error_level": 3,
- "shortcode": 92,
- "taint_trace": null
+ "shortcode": 92
},
{
"severity": "info",
@@ -1143,60 +1080,67 @@
"snippet": " return (string)$filter->filter($moduleName);",
"selected_text": "$filter->filter($moduleName)",
"error_level": 3,
- "shortcode": 190,
- "taint_trace": null
+ "shortcode": 190
},
{
"severity": "info",
- "line_from": 107,
- "line_to": 107,
+ "line_from": 50,
+ "line_to": 50,
+ "type": "ArgumentTypeCoercion",
+ "message": "Argument 1 of ReflectionClass::__construct expects class-string|object|trait-string, parent type string provided",
+ "file_name": "vendor/spryker/spryker/Bundles/Development/src/Spryker/Zed/Development/Business/Dependency/ComposerParser/ExternalDependencyParser.php",
+ "snippet": " $reflectionClass = new ReflectionClass($className);",
+ "selected_text": "$className",
+ "error_level": 3,
+ "shortcode": 193
+ },
+ {
+ "severity": "info",
+ "line_from": 113,
+ "line_to": 113,
"type": "PossiblyNullArgument",
"message": "Argument 1 of strcmp cannot be null, possibly null value provided",
"file_name": "vendor/spryker/spryker/Bundles/Development/src/Spryker/Zed/Development/Business/Dependency/DependencyContainer/DependencyContainer.php",
"snippet": " return strcmp($dependencyBundleTransferA->getModule(), $dependencyBundleTransferB->getModule());",
"selected_text": "$dependencyBundleTransferA->getModule()",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
- "line_from": 107,
- "line_to": 107,
+ "line_from": 113,
+ "line_to": 113,
"type": "PossiblyNullArgument",
"message": "Argument 2 of strcmp cannot be null, possibly null value provided",
"file_name": "vendor/spryker/spryker/Bundles/Development/src/Spryker/Zed/Development/Business/Dependency/DependencyContainer/DependencyContainer.php",
"snippet": " return strcmp($dependencyBundleTransferA->getModule(), $dependencyBundleTransferB->getModule());",
"selected_text": "$dependencyBundleTransferB->getModule()",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
- "line_from": 110,
- "line_to": 110,
+ "line_from": 116,
+ "line_to": 116,
"type": "PossiblyInvalidMethodCall",
"message": "Cannot call method on possible array variable ",
"file_name": "vendor/spryker/spryker/Bundles/Development/src/Spryker/Zed/Development/Business/Dependency/DependencyContainer/DependencyContainer.php",
"snippet": " $dependencyModules = $dependencyCollectionTransfer->getDependencyModules()->getArrayCopy();",
"selected_text": "getArrayCopy",
"error_level": 3,
- "shortcode": 113,
- "taint_trace": null
+ "shortcode": 113
},
{
"severity": "info",
- "line_from": 134,
- "line_to": 134,
+ "line_from": 140,
+ "line_to": 140,
"type": "PossiblyInvalidArgument",
"message": "Argument 1 of ucfirst expects string, possibly different type array|string provided",
"file_name": "vendor/spryker/spryker/Bundles/Development/src/Spryker/Zed/Development/Business/Dependency/DependencyContainer/DependencyContainer.php",
"snippet": " return ucfirst($filter->filter($moduleName));",
"selected_text": "$filter->filter($moduleName)",
"error_level": 3,
- "shortcode": 92,
- "taint_trace": null
+ "shortcode": 92
},
{
"severity": "info",
@@ -1208,8 +1152,7 @@
"snippet": " $moduleExtensionKey = sprintf('%s.%sExtension', $moduleTransfer->getOrganization()->getName(), $moduleTransfer->getName());",
"selected_text": "$moduleTransfer->getOrganization()->getName()",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
@@ -1221,8 +1164,7 @@
"snippet": " $moduleExtensionKey = sprintf('%s.%sExtension', $moduleTransfer->getOrganization()->getName(), $moduleTransfer->getName());",
"selected_text": "getName",
"error_level": 3,
- "shortcode": 83,
- "taint_trace": null
+ "shortcode": 83
},
{
"severity": "info",
@@ -1234,8 +1176,7 @@
"snippet": " $moduleExtensionKey = sprintf('%s.%sExtension', $moduleTransfer->getOrganization()->getName(), $moduleTransfer->getName());",
"selected_text": "$moduleTransfer->getName()",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
@@ -1247,8 +1188,7 @@
"snippet": " $composerName = $this->buildComposerName($moduleTransfer->getOrganization()->getName(), sprintf('%sExtension', $moduleTransfer->getName()));",
"selected_text": "$moduleTransfer->getOrganization()->getName()",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
@@ -1260,8 +1200,7 @@
"snippet": " $composerName = $this->buildComposerName($moduleTransfer->getOrganization()->getName(), sprintf('%sExtension', $moduleTransfer->getName()));",
"selected_text": "getName",
"error_level": 3,
- "shortcode": 83,
- "taint_trace": null
+ "shortcode": 83
},
{
"severity": "info",
@@ -1273,21 +1212,19 @@
"snippet": " $composerName = $this->buildComposerName($moduleTransfer->getOrganization()->getName(), sprintf('%sExtension', $moduleTransfer->getName()));",
"selected_text": "$moduleTransfer->getName()",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
- "line_from": 171,
- "line_to": 171,
+ "line_from": 237,
+ "line_to": 237,
"type": "PossiblyInvalidArgument",
"message": "Argument 1 of ucfirst expects string, possibly different type array|string provided",
"file_name": "vendor/spryker/spryker/Bundles/Development/src/Spryker/Zed/Development/Business/Dependency/DependencyFinder/ExternalDependencyFinder.php",
"snippet": " return ucfirst($filter->filter($dependentModule));",
"selected_text": "$filter->filter($dependentModule)",
"error_level": 3,
- "shortcode": 92,
- "taint_trace": null
+ "shortcode": 92
},
{
"severity": "info",
@@ -1299,8 +1236,7 @@
"snippet": " $composerNames = $this->getNamesFromUseStatements($useStatements, $context->getModule()->getName());",
"selected_text": "$context->getModule()->getName()",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
@@ -1312,8 +1248,7 @@
"snippet": " $dependentModule = $this->propelSchemaParser->getModuleNameByForeignReference($foreignIdColumnName, $context->getModule()->getName());",
"selected_text": "$context->getModule()->getName()",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
@@ -1325,8 +1260,7 @@
"snippet": " $modules = $this->getModuleNamesFromUseStatements($useStatements, $context->getModule()->getName());",
"selected_text": "$context->getModule()->getName()",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
@@ -1338,8 +1272,7 @@
"snippet": " return $this->addFindings($context->getModule()->getName(), $matches, $dependencyContainer);",
"selected_text": "$context->getModule()->getName()",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
@@ -1351,8 +1284,7 @@
"snippet": " return $this->addFindings($context->getModule()->getName(), $matches, $dependencyContainer);",
"selected_text": "$context->getModule()->getName()",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
@@ -1364,8 +1296,7 @@
"snippet": " return $this->addFindings($context->getModule()->getName(), $matches, $dependencyContainer);",
"selected_text": "$context->getModule()->getName()",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
@@ -1377,8 +1308,7 @@
"snippet": " return $this->addFindings($context->getModule()->getName(), $matches, $dependencyContainer);",
"selected_text": "$context->getModule()->getName()",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
@@ -1390,8 +1320,7 @@
"snippet": " return $this->addFindings($context->getModule()->getName(), $matches, $dependencyContainer);",
"selected_text": "$context->getModule()->getName()",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
@@ -1403,8 +1332,7 @@
"snippet": " $moduleTransfer->setNameDashed($this->dasherize($moduleTransfer->getName()));",
"selected_text": "$moduleTransfer->getName()",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
@@ -1416,8 +1344,7 @@
"snippet": " * @return array",
"selected_text": "array",
"error_level": 5,
- "shortcode": 166,
- "taint_trace": null
+ "shortcode": 166
},
{
"severity": "info",
@@ -1429,8 +1356,7 @@
"snippet": " * @return \\Symfony\\Component\\Finder\\Finder|\\Symfony\\Component\\Finder\\SplFileInfo[]",
"selected_text": "\\Symfony\\Component\\Finder\\Finder|\\Symfony\\Component\\Finder\\SplFileInfo[]",
"error_level": 4,
- "shortcode": 142,
- "taint_trace": null
+ "shortcode": 142
},
{
"severity": "info",
@@ -1442,8 +1368,7 @@
"snippet": " * @return \\Symfony\\Component\\Finder\\SplFileInfo[]|\\Symfony\\Component\\Finder\\Finder",
"selected_text": "\\Symfony\\Component\\Finder\\SplFileInfo[]|\\Symfony\\Component\\Finder\\Finder",
"error_level": 4,
- "shortcode": 142,
- "taint_trace": null
+ "shortcode": 142
},
{
"severity": "info",
@@ -1455,8 +1380,7 @@
"snippet": " * @return \\Symfony\\Component\\Finder\\SplFileInfo[]|\\Symfony\\Component\\Finder\\Finder",
"selected_text": "\\Symfony\\Component\\Finder\\SplFileInfo[]|\\Symfony\\Component\\Finder\\Finder",
"error_level": 4,
- "shortcode": 142,
- "taint_trace": null
+ "shortcode": 142
},
{
"severity": "info",
@@ -1468,8 +1392,7 @@
"snippet": " $dependencyValidationRequestTransfer->getModule(),",
"selected_text": "$dependencyValidationRequestTransfer->getModule()",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
@@ -1481,151 +1404,79 @@
"snippet": " $composerName = $dependencyModuleTransfer->getComposerName() ?? $this->composerNameFinder->findComposerNameByModuleName($dependencyModuleTransfer->getModule());",
"selected_text": "$dependencyModuleTransfer->getModule()",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
- "line_from": 52,
- "line_to": 52,
+ "line_from": 58,
+ "line_to": 58,
"type": "PossiblyNullArgument",
- "message": "Argument 1 of Spryker\\Zed\\Development\\Business\\DependencyTree\\ComposerDependencyParser::getParsedComposerDependenciesForBundle cannot be null, possibly null value provided",
+ "message": "Argument 1 of Spryker\\Zed\\Development\\Business\\DependencyTree\\ComposerDependencyParser::parseComposerJson cannot be null, possibly null value provided",
"file_name": "vendor/spryker/spryker/Bundles/Development/src/Spryker/Zed/Development/Business/DependencyTree/ComposerDependencyParser.php",
- "snippet": " $composerDependencyCollectionTransfer = $this->getParsedComposerDependenciesForBundle($dependencyCollectionTransfer->getModule());",
+ "snippet": " $composerDependencyCollectionTransfer = $this->parseComposerJson($dependencyCollectionTransfer->getModule());",
"selected_text": "$dependencyCollectionTransfer->getModule()",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
- "line_from": 67,
- "line_to": 67,
+ "line_from": 68,
+ "line_to": 68,
"type": "PossiblyNullArgument",
"message": "Argument 2 of sprintf cannot be null, possibly null value provided",
"file_name": "vendor/spryker/spryker/Bundles/Development/src/Spryker/Zed/Development/Business/DependencyTree/ComposerDependencyParser.php",
"snippet": " $currentComposerName = sprintf('%s/%s', $dependencyCollectionTransfer->getModule()->getOrganization()->getNameDashed(), $dependencyCollectionTransfer->getModule()->getNameDashed());",
"selected_text": "$dependencyCollectionTransfer->getModule()->getOrganization()->getNameDashed()",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
- "line_from": 67,
- "line_to": 67,
+ "line_from": 68,
+ "line_to": 68,
"type": "PossiblyNullReference",
"message": "Cannot call method getOrganization on possibly null value",
"file_name": "vendor/spryker/spryker/Bundles/Development/src/Spryker/Zed/Development/Business/DependencyTree/ComposerDependencyParser.php",
"snippet": " $currentComposerName = sprintf('%s/%s', $dependencyCollectionTransfer->getModule()->getOrganization()->getNameDashed(), $dependencyCollectionTransfer->getModule()->getNameDashed());",
"selected_text": "getOrganization",
"error_level": 3,
- "shortcode": 83,
- "taint_trace": null
+ "shortcode": 83
},
{
"severity": "info",
- "line_from": 67,
- "line_to": 67,
+ "line_from": 68,
+ "line_to": 68,
"type": "PossiblyNullReference",
"message": "Cannot call method getNameDashed on possibly null value",
"file_name": "vendor/spryker/spryker/Bundles/Development/src/Spryker/Zed/Development/Business/DependencyTree/ComposerDependencyParser.php",
"snippet": " $currentComposerName = sprintf('%s/%s', $dependencyCollectionTransfer->getModule()->getOrganization()->getNameDashed(), $dependencyCollectionTransfer->getModule()->getNameDashed());",
"selected_text": "getNameDashed",
"error_level": 3,
- "shortcode": 83,
- "taint_trace": null
+ "shortcode": 83
},
{
"severity": "info",
- "line_from": 67,
- "line_to": 67,
+ "line_from": 68,
+ "line_to": 68,
"type": "PossiblyNullArgument",
"message": "Argument 3 of sprintf cannot be null, possibly null value provided",
"file_name": "vendor/spryker/spryker/Bundles/Development/src/Spryker/Zed/Development/Business/DependencyTree/ComposerDependencyParser.php",
"snippet": " $currentComposerName = sprintf('%s/%s', $dependencyCollectionTransfer->getModule()->getOrganization()->getNameDashed(), $dependencyCollectionTransfer->getModule()->getNameDashed());",
"selected_text": "$dependencyCollectionTransfer->getModule()->getNameDashed()",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
- "line_from": 67,
- "line_to": 67,
+ "line_from": 68,
+ "line_to": 68,
"type": "PossiblyNullReference",
"message": "Cannot call method getNameDashed on possibly null value",
"file_name": "vendor/spryker/spryker/Bundles/Development/src/Spryker/Zed/Development/Business/DependencyTree/ComposerDependencyParser.php",
"snippet": " $currentComposerName = sprintf('%s/%s', $dependencyCollectionTransfer->getModule()->getOrganization()->getNameDashed(), $dependencyCollectionTransfer->getModule()->getNameDashed());",
"selected_text": "getNameDashed",
"error_level": 3,
- "shortcode": 83,
- "taint_trace": null
- },
- {
- "severity": "info",
- "line_from": 148,
- "line_to": 148,
- "type": "PossiblyNullArrayOffset",
- "message": "Cannot access value on variable $composerNames[mixed|null|string] using possibly null offset mixed|null|string",
- "file_name": "vendor/spryker/spryker/Bundles/Development/src/Spryker/Zed/Development/Business/DependencyTree/ComposerDependencyParser.php",
- "snippet": " $composerNames[$moduleDependencyTransfer->getModule()] = $moduleDependencyTransfer->getComposerName();",
- "selected_text": "$composerNames",
- "error_level": 3,
- "shortcode": 125,
- "taint_trace": null
- },
- {
- "severity": "info",
- "line_from": 165,
- "line_to": 165,
- "type": "PossiblyNullArrayOffset",
- "message": "Cannot access value on variable $composerNames[mixed|null|string] using possibly null offset mixed|null|string",
- "file_name": "vendor/spryker/spryker/Bundles/Development/src/Spryker/Zed/Development/Business/DependencyTree/ComposerDependencyParser.php",
- "snippet": " $composerNames[$moduleDependencyTransfer->getModule()] = $moduleDependencyTransfer->getComposerName();",
- "selected_text": "$composerNames",
- "error_level": 3,
- "shortcode": 125,
- "taint_trace": null
- },
- {
- "severity": "info",
- "line_from": 184,
- "line_to": 184,
- "type": "PossiblyNullArrayOffset",
- "message": "Cannot access value on variable $composerNames[mixed|null|string] using possibly null offset mixed|null|string",
- "file_name": "vendor/spryker/spryker/Bundles/Development/src/Spryker/Zed/Development/Business/DependencyTree/ComposerDependencyParser.php",
- "snippet": " $composerNames[$moduleDependencyTransfer->getModule()] = $moduleDependencyTransfer->getComposerName();",
- "selected_text": "$composerNames",
- "error_level": 3,
- "shortcode": 125,
- "taint_trace": null
- },
- {
- "severity": "info",
- "line_from": 205,
- "line_to": 205,
- "type": "PossiblyNullArrayOffset",
- "message": "Cannot access value on variable $composerNames[mixed|null|string] using possibly null offset mixed|null|string",
- "file_name": "vendor/spryker/spryker/Bundles/Development/src/Spryker/Zed/Development/Business/DependencyTree/ComposerDependencyParser.php",
- "snippet": " $composerNames[$composerDependency->getModuleName()] = $composerDependency->getName();",
- "selected_text": "$composerNames",
- "error_level": 3,
- "shortcode": 125,
- "taint_trace": null
- },
- {
- "severity": "info",
- "line_from": 223,
- "line_to": 223,
- "type": "PossiblyNullArrayOffset",
- "message": "Cannot access value on variable $composerNames[mixed|null|string] using possibly null offset mixed|null|string",
- "file_name": "vendor/spryker/spryker/Bundles/Development/src/Spryker/Zed/Development/Business/DependencyTree/ComposerDependencyParser.php",
- "snippet": " $composerNames[$composerDependency->getModuleName()] = $composerDependency->getName();",
- "selected_text": "$composerNames",
- "error_level": 3,
- "shortcode": 125,
- "taint_trace": null
+ "shortcode": 83
},
{
"severity": "info",
@@ -1637,8 +1488,7 @@
"snippet": " $declaredDependencies = $this->parseDeclaredDependenciesForBundle($dependencyCollectionTransfer->getModule());",
"selected_text": "$dependencyCollectionTransfer->getModule()",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
@@ -1650,8 +1500,7 @@
"snippet": " $dependencyJsonFilePath = sprintf('%s/dependency.json', $moduleTransfer->getPath());",
"selected_text": "$moduleTransfer->getPath()",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
@@ -1663,60 +1512,55 @@
"snippet": " $composerJsonFilePath = sprintf('%s/composer.json', $moduleTransfer->getPath());",
"selected_text": "$moduleTransfer->getPath()",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
- "line_from": 407,
- "line_to": 407,
+ "line_from": 404,
+ "line_to": 404,
"type": "PossiblyFalseOperand",
"message": "Left operand cannot be falsable, got false|int",
"file_name": "vendor/spryker/spryker/Bundles/Development/src/Spryker/Zed/Development/Business/DependencyTree/ComposerDependencyParser.php",
"snippet": " $name = substr($package, strpos($package, '/') + 1);",
"selected_text": "strpos($package, '/')",
"error_level": 3,
- "shortcode": 162,
- "taint_trace": null
+ "shortcode": 162
},
{
"severity": "info",
- "line_from": 409,
- "line_to": 409,
+ "line_from": 406,
+ "line_to": 406,
"type": "PossiblyInvalidArgument",
"message": "Argument 1 of ucfirst expects string, possibly different type array|string provided",
"file_name": "vendor/spryker/spryker/Bundles/Development/src/Spryker/Zed/Development/Business/DependencyTree/ComposerDependencyParser.php",
"snippet": " $name = ucfirst($filter->filter($name));",
"selected_text": "$filter->filter($name)",
"error_level": 3,
- "shortcode": 92,
- "taint_trace": null
+ "shortcode": 92
},
{
"severity": "info",
- "line_from": 422,
- "line_to": 422,
+ "line_from": 419,
+ "line_to": 419,
"type": "PossiblyNullOperand",
"message": "Cannot concatenate with a possibly null null|string",
"file_name": "vendor/spryker/spryker/Bundles/Development/src/Spryker/Zed/Development/Business/DependencyTree/ComposerDependencyParser.php",
"snippet": " return $moduleName === $dependencyCollectionTransfer->getModule()->getName() . 'Extension';",
"selected_text": "$dependencyCollectionTransfer->getModule()->getName()",
"error_level": 1,
- "shortcode": 80,
- "taint_trace": null
+ "shortcode": 80
},
{
"severity": "info",
- "line_from": 422,
- "line_to": 422,
+ "line_from": 419,
+ "line_to": 419,
"type": "PossiblyNullReference",
"message": "Cannot call method getName on possibly null value",
"file_name": "vendor/spryker/spryker/Bundles/Development/src/Spryker/Zed/Development/Business/DependencyTree/ComposerDependencyParser.php",
"snippet": " return $moduleName === $dependencyCollectionTransfer->getModule()->getName() . 'Extension';",
"selected_text": "getName",
"error_level": 3,
- "shortcode": 83,
- "taint_trace": null
+ "shortcode": 83
},
{
"severity": "info",
@@ -1728,8 +1572,7 @@
"snippet": " $classNameParts = array_slice($tokens, $pointer, $endOfNew - $foundPosition - 2);",
"selected_text": "$endOfNew",
"error_level": 3,
- "shortcode": 162,
- "taint_trace": null
+ "shortcode": 162
},
{
"severity": "info",
@@ -1741,8 +1584,7 @@
"snippet": " $startOf = $file->findPrevious([T_OPEN_PARENTHESIS, T_WHITESPACE, T_OPEN_SQUARE_BRACKET], $foundPosition - 1) + 1;",
"selected_text": "$file->findPrevious([T_OPEN_PARENTHESIS, T_WHITESPACE, T_OPEN_SQUARE_BRACKET], $foundPosition - 1)",
"error_level": 3,
- "shortcode": 162,
- "taint_trace": null
+ "shortcode": 162
},
{
"severity": "info",
@@ -1754,8 +1596,7 @@
"snippet": " $dependencyInformation[DependencyTree::META_FOREIGN_LAYER] = 'external';",
"selected_text": "$dependencyInformation",
"error_level": 3,
- "shortcode": 18,
- "taint_trace": null
+ "shortcode": 18
},
{
"severity": "info",
@@ -1767,8 +1608,7 @@
"snippet": " return ucfirst($filter->filter($foreignBundle));",
"selected_text": "$filter->filter($foreignBundle)",
"error_level": 3,
- "shortcode": 92,
- "taint_trace": null
+ "shortcode": 92
},
{
"severity": "info",
@@ -1780,8 +1620,7 @@
"snippet": " 'url' => '/development/dependency/outgoing-graph?bundle=' . $incomingBundle->getName(),",
"selected_text": "$incomingBundle->getName()",
"error_level": 1,
- "shortcode": 80,
- "taint_trace": null
+ "shortcode": 80
},
{
"severity": "info",
@@ -1793,8 +1632,7 @@
"snippet": " $this->graph->addNode($incomingBundle->getName(), $attributes);",
"selected_text": "$incomingBundle->getName()",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
@@ -1806,8 +1644,7 @@
"snippet": " $this->graph->addEdge($incomingBundle->getName(), $this->bundleName);",
"selected_text": "$incomingBundle->getName()",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
@@ -1819,8 +1656,7 @@
"snippet": " return strtolower($filter->filter($classNameParts[0]));",
"selected_text": "$filter->filter($classNameParts[0])",
"error_level": 3,
- "shortcode": 92,
- "taint_trace": null
+ "shortcode": 92
},
{
"severity": "info",
@@ -1832,8 +1668,7 @@
"snippet": " * @return array",
"selected_text": "array",
"error_level": 5,
- "shortcode": 166,
- "taint_trace": null
+ "shortcode": 166
},
{
"severity": "info",
@@ -1845,8 +1680,7 @@
"snippet": " $basePath = rtrim($this->config->getPathToInternalNamespace(static::ORGANIZATION), '/');",
"selected_text": "$this->config->getPathToInternalNamespace(static::ORGANIZATION)",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
@@ -1858,8 +1692,7 @@
"snippet": " $basePath = rtrim($this->config->getPathToInternalNamespace(static::ORGANIZATION), '/');",
"selected_text": "$this->config->getPathToInternalNamespace(static::ORGANIZATION)",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
@@ -1871,8 +1704,7 @@
"snippet": " $basePath = rtrim($this->config->getPathToInternalNamespace(static::ORGANIZATION), '/');",
"selected_text": "$this->config->getPathToInternalNamespace(static::ORGANIZATION)",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
@@ -1884,8 +1716,7 @@
"snippet": " public function drawOutgoingDependencyTreeGraph($moduleToView, array $excludedModules = [], $showIncomingDependencies = false)",
"selected_text": "$moduleToView",
"error_level": 5,
- "shortcode": 140,
- "taint_trace": null
+ "shortcode": 140
},
{
"severity": "info",
@@ -1897,8 +1728,7 @@
"snippet": " $bundleTransferCollection[$bundleTransfer->getName()] = $bundleTransfer;",
"selected_text": "$bundleTransferCollection",
"error_level": 3,
- "shortcode": 125,
- "taint_trace": null
+ "shortcode": 125
},
{
"severity": "info",
@@ -1910,8 +1740,7 @@
"snippet": " if (!array_key_exists($bundleTransfer->getName(), $bundles)) {",
"selected_text": "$bundleTransfer->getName()",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
@@ -1923,8 +1752,7 @@
"snippet": " $existingBundleTransfer = $bundles[$bundleTransfer->getName()];",
"selected_text": "$bundles",
"error_level": 3,
- "shortcode": 125,
- "taint_trace": null
+ "shortcode": 125
},
{
"severity": "info",
@@ -1936,8 +1764,7 @@
"snippet": " * @param \\ArrayObject|\\Generated\\Shared\\Transfer\\IdeAutoCompletionBundleMethodTransfer[] $existingMethodTransferCollection",
"selected_text": "\\ArrayObject|\\Generated\\Shared\\Transfer\\IdeAutoCompletionBundleMethodTransfer[]",
"error_level": 4,
- "shortcode": 141,
- "taint_trace": null
+ "shortcode": 141
},
{
"severity": "info",
@@ -1949,8 +1776,7 @@
"snippet": " * @param \\ArrayObject|\\Generated\\Shared\\Transfer\\IdeAutoCompletionBundleMethodTransfer[] $methodTransferCollection",
"selected_text": "\\ArrayObject|\\Generated\\Shared\\Transfer\\IdeAutoCompletionBundleMethodTransfer[]",
"error_level": 4,
- "shortcode": 141,
- "taint_trace": null
+ "shortcode": 141
},
{
"severity": "info",
@@ -1962,8 +1788,7 @@
"snippet": " $methodsByName[$methodTransfer->getName()] = $methodTransfer;",
"selected_text": "$methodsByName",
"error_level": 3,
- "shortcode": 125,
- "taint_trace": null
+ "shortcode": 125
},
{
"severity": "info",
@@ -1975,8 +1800,7 @@
"snippet": " $methodsByName[$methodTransfer->getName()] = $methodTransfer;",
"selected_text": "$methodsByName",
"error_level": 3,
- "shortcode": 125,
- "taint_trace": null
+ "shortcode": 125
},
{
"severity": "info",
@@ -1988,8 +1812,7 @@
"snippet": " $interfaceFileName = $this->getInterfaceFileName($bundleTransfer->getName());",
"selected_text": "$bundleTransfer->getName()",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
@@ -2001,8 +1824,7 @@
"snippet": " $classFileName = $this->getClassFileName($bundleTransfer->getName());",
"selected_text": "$bundleTransfer->getName()",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
@@ -2014,8 +1836,7 @@
"snippet": " return $this->namespaceExtractor->fromDirectory($directory, $bundleTransfer->getBaseDirectory());",
"selected_text": "$bundleTransfer->getBaseDirectory()",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
@@ -2027,8 +1848,7 @@
"snippet": " $bundleTransfer->getDirectory(),",
"selected_text": "$bundleTransfer->getDirectory()",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
@@ -2040,8 +1860,7 @@
"snippet": " $bundleTransfer->getName()",
"selected_text": "$bundleTransfer->getName()",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
@@ -2053,8 +1872,7 @@
"snippet": " $bundleTransfer->getDirectory(),",
"selected_text": "$bundleTransfer->getDirectory()",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
@@ -2066,8 +1884,7 @@
"snippet": " $bundleTransfer->getName()",
"selected_text": "$bundleTransfer->getName()",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
@@ -2079,8 +1896,7 @@
"snippet": " $bundleTransfer->getDirectory(),",
"selected_text": "$bundleTransfer->getDirectory()",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
@@ -2092,8 +1908,7 @@
"snippet": " $bundleTransfer->getName()",
"selected_text": "$bundleTransfer->getName()",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
@@ -2105,8 +1920,7 @@
"snippet": " $bundleTransfer->getDirectory(),",
"selected_text": "$bundleTransfer->getDirectory()",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
@@ -2118,8 +1932,7 @@
"snippet": " $bundleTransfer->getName()",
"selected_text": "$bundleTransfer->getName()",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
@@ -2131,8 +1944,7 @@
"snippet": " $bundleTransfer->getDirectory(),",
"selected_text": "$bundleTransfer->getDirectory()",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
@@ -2144,8 +1956,7 @@
"snippet": " $bundleTransfer->getName()",
"selected_text": "$bundleTransfer->getName()",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
@@ -2157,8 +1968,7 @@
"snippet": " if ($moduleTransfer->getMethods()->count() === 0) {",
"selected_text": "count",
"error_level": 3,
- "shortcode": 113,
- "taint_trace": null
+ "shortcode": 113
},
{
"severity": "info",
@@ -2170,8 +1980,7 @@
"snippet": " $fileName = sprintf('%s.php', $moduleTransfer->getName());",
"selected_text": "$moduleTransfer->getName()",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
@@ -2183,8 +1992,7 @@
"snippet": " $moduleTransfer->getPath(),",
"selected_text": "$moduleTransfer->getPath()",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
@@ -2196,8 +2004,7 @@
"snippet": " $moduleTransfer->getOrganization()->getName(),",
"selected_text": "$moduleTransfer->getOrganization()->getName()",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
@@ -2209,8 +2016,7 @@
"snippet": " $moduleTransfer->getOrganization()->getName(),",
"selected_text": "getName",
"error_level": 3,
- "shortcode": 83,
- "taint_trace": null
+ "shortcode": 83
},
{
"severity": "info",
@@ -2222,8 +2028,7 @@
"snippet": " $applicationTransfer->getName(),",
"selected_text": "$applicationTransfer->getName()",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
@@ -2235,8 +2040,7 @@
"snippet": " $moduleTransfer->getName()",
"selected_text": "$moduleTransfer->getName()",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
@@ -2248,8 +2052,7 @@
"snippet": " * @return \\Symfony\\Component\\Finder\\SplFileInfo[]|\\Symfony\\Component\\Finder\\Finder",
"selected_text": "\\Symfony\\Component\\Finder\\SplFileInfo[]|\\Symfony\\Component\\Finder\\Finder",
"error_level": 4,
- "shortcode": 142,
- "taint_trace": null
+ "shortcode": 142
},
{
"severity": "info",
@@ -2261,8 +2064,7 @@
"snippet": " $dependencyProviderClassName = str_replace([$moduleTransfer->getPath() . 'src/', '.php', DIRECTORY_SEPARATOR], ['', '', '\\\\'], $splFileInfo->getPathname());",
"selected_text": "$moduleTransfer->getPath()",
"error_level": 1,
- "shortcode": 80,
- "taint_trace": null
+ "shortcode": 80
},
{
"severity": "info",
@@ -2274,8 +2076,7 @@
"snippet": " * @return \\Symfony\\Component\\Finder\\Finder|\\Symfony\\Component\\Finder\\SplFileInfo[]",
"selected_text": "\\Symfony\\Component\\Finder\\Finder|\\Symfony\\Component\\Finder\\SplFileInfo[]",
"error_level": 4,
- "shortcode": 142,
- "taint_trace": null
+ "shortcode": 142
},
{
"severity": "info",
@@ -2287,8 +2088,7 @@
"snippet": " * @return \\Symfony\\Component\\Finder\\SplFileInfo[]|\\Symfony\\Component\\Finder\\Finder",
"selected_text": "\\Symfony\\Component\\Finder\\SplFileInfo[]|\\Symfony\\Component\\Finder\\Finder",
"error_level": 4,
- "shortcode": 142,
- "taint_trace": null
+ "shortcode": 142
},
{
"severity": "info",
@@ -2300,8 +2100,7 @@
"snippet": " $composerJsonAsArray = $this->getComposerJsonAsArray($moduleTransfer->getPath());",
"selected_text": "$moduleTransfer->getPath()",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
@@ -2313,8 +2112,7 @@
"snippet": " * @return \\Symfony\\Component\\Finder\\SplFileInfo[]|\\Symfony\\Component\\Finder\\Finder",
"selected_text": "\\Symfony\\Component\\Finder\\SplFileInfo[]|\\Symfony\\Component\\Finder\\Finder",
"error_level": 4,
- "shortcode": 142,
- "taint_trace": null
+ "shortcode": 142
},
{
"severity": "info",
@@ -2326,8 +2124,7 @@
"snippet": " return sprintf('%s.%s', $moduleTransfer->getOrganization()->getName(), $moduleTransfer->getName());",
"selected_text": "$moduleTransfer->getOrganization()->getName()",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
@@ -2339,8 +2136,7 @@
"snippet": " return sprintf('%s.%s', $moduleTransfer->getOrganization()->getName(), $moduleTransfer->getName());",
"selected_text": "getName",
"error_level": 3,
- "shortcode": 83,
- "taint_trace": null
+ "shortcode": 83
},
{
"severity": "info",
@@ -2352,8 +2148,7 @@
"snippet": " return sprintf('%s.%s', $moduleTransfer->getOrganization()->getName(), $moduleTransfer->getName());",
"selected_text": "$moduleTransfer->getName()",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
@@ -2365,8 +2160,7 @@
"snippet": " $lookupDirectory = sprintf('%s/src/%s/', $moduleTransfer->getPath(), $moduleTransfer->getOrganization()->getName());",
"selected_text": "$moduleTransfer->getPath()",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
@@ -2378,8 +2172,7 @@
"snippet": " $lookupDirectory = sprintf('%s/src/%s/', $moduleTransfer->getPath(), $moduleTransfer->getOrganization()->getName());",
"selected_text": "$moduleTransfer->getOrganization()->getName()",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
@@ -2391,8 +2184,7 @@
"snippet": " $lookupDirectory = sprintf('%s/src/%s/', $moduleTransfer->getPath(), $moduleTransfer->getOrganization()->getName());",
"selected_text": "getName",
"error_level": 3,
- "shortcode": 83,
- "taint_trace": null
+ "shortcode": 83
},
{
"severity": "info",
@@ -2404,8 +2196,7 @@
"snippet": " $organizationName = $pathFragments[$vendorPosition + 1];",
"selected_text": "$vendorPosition",
"error_level": 3,
- "shortcode": 162,
- "taint_trace": null
+ "shortcode": 162
},
{
"severity": "info",
@@ -2417,8 +2208,7 @@
"snippet": " $applicationName = $pathFragments[$vendorPosition + 2];",
"selected_text": "$vendorPosition",
"error_level": 3,
- "shortcode": 162,
- "taint_trace": null
+ "shortcode": 162
},
{
"severity": "info",
@@ -2430,8 +2220,7 @@
"snippet": " if (!$this->matchesOrganization($moduleFilterTransfer, $moduleTransfer->getOrganization())) {",
"selected_text": "$moduleTransfer->getOrganization()",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
@@ -2443,8 +2232,7 @@
"snippet": " return $this->match($moduleFilterTransfer->getOrganization()->getName(), $organizationTransfer->getName());",
"selected_text": "$moduleFilterTransfer->getOrganization()->getName()",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
@@ -2456,8 +2244,7 @@
"snippet": " return $this->match($moduleFilterTransfer->getOrganization()->getName(), $organizationTransfer->getName());",
"selected_text": "getName",
"error_level": 3,
- "shortcode": 83,
- "taint_trace": null
+ "shortcode": 83
},
{
"severity": "info",
@@ -2469,8 +2256,7 @@
"snippet": " return $this->match($moduleFilterTransfer->getOrganization()->getName(), $organizationTransfer->getName());",
"selected_text": "$organizationTransfer->getName()",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
@@ -2482,8 +2268,7 @@
"snippet": " if ($this->match($moduleFilterTransfer->getApplication()->getName(), $applicationTransfer->getName())) {",
"selected_text": "$moduleFilterTransfer->getApplication()->getName()",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
@@ -2495,8 +2280,7 @@
"snippet": " if ($this->match($moduleFilterTransfer->getApplication()->getName(), $applicationTransfer->getName())) {",
"selected_text": "getName",
"error_level": 3,
- "shortcode": 83,
- "taint_trace": null
+ "shortcode": 83
},
{
"severity": "info",
@@ -2508,8 +2292,7 @@
"snippet": " if ($this->match($moduleFilterTransfer->getApplication()->getName(), $applicationTransfer->getName())) {",
"selected_text": "$applicationTransfer->getName()",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
@@ -2521,8 +2304,7 @@
"snippet": " return $this->match($moduleFilterTransfer->getModule()->getName(), $moduleTransfer->getName());",
"selected_text": "$moduleFilterTransfer->getModule()->getName()",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
@@ -2534,8 +2316,7 @@
"snippet": " return $this->match($moduleFilterTransfer->getModule()->getName(), $moduleTransfer->getName());",
"selected_text": "getName",
"error_level": 3,
- "shortcode": 83,
- "taint_trace": null
+ "shortcode": 83
},
{
"severity": "info",
@@ -2547,8 +2328,7 @@
"snippet": " return $this->match($moduleFilterTransfer->getModule()->getName(), $moduleTransfer->getName());",
"selected_text": "$moduleTransfer->getName()",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
@@ -2560,8 +2340,7 @@
"snippet": " $this->config->getPathToInternalNamespace(static::ORGANIZATION),",
"selected_text": "$this->config->getPathToInternalNamespace(static::ORGANIZATION)",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
@@ -2573,8 +2352,7 @@
"snippet": " return ($moduleTransfer->getOrganization()->getName() === static::ORGANIZATION);",
"selected_text": "getName",
"error_level": 3,
- "shortcode": 83,
- "taint_trace": null
+ "shortcode": 83
},
{
"severity": "info",
@@ -2586,8 +2364,7 @@
"snippet": " * @return string",
"selected_text": "string",
"error_level": 5,
- "shortcode": 144,
- "taint_trace": null
+ "shortcode": 144
},
{
"severity": "info",
@@ -2599,8 +2376,7 @@
"snippet": " return $moduleTransfer->getName();",
"selected_text": "$moduleTransfer->getName()",
"error_level": 5,
- "shortcode": 139,
- "taint_trace": null
+ "shortcode": 139
},
{
"severity": "info",
@@ -2612,8 +2388,7 @@
"snippet": " * @return string",
"selected_text": "string",
"error_level": 5,
- "shortcode": 144,
- "taint_trace": null
+ "shortcode": 144
},
{
"severity": "info",
@@ -2625,8 +2400,7 @@
"snippet": " return $moduleTransfer->getNameDashed();",
"selected_text": "$moduleTransfer->getNameDashed()",
"error_level": 5,
- "shortcode": 139,
- "taint_trace": null
+ "shortcode": 139
},
{
"severity": "info",
@@ -2638,8 +2412,7 @@
"snippet": " * @return string",
"selected_text": "string",
"error_level": 5,
- "shortcode": 144,
- "taint_trace": null
+ "shortcode": 144
},
{
"severity": "info",
@@ -2651,8 +2424,7 @@
"snippet": " return $moduleTransfer->getNameDashed();",
"selected_text": "$moduleTransfer->getNameDashed()",
"error_level": 5,
- "shortcode": 139,
- "taint_trace": null
+ "shortcode": 139
},
{
"severity": "info",
@@ -2664,8 +2436,7 @@
"snippet": " * @return string",
"selected_text": "string",
"error_level": 5,
- "shortcode": 144,
- "taint_trace": null
+ "shortcode": 144
},
{
"severity": "info",
@@ -2677,8 +2448,7 @@
"snippet": " return $moduleTransfer->getNameDashed();",
"selected_text": "$moduleTransfer->getNameDashed()",
"error_level": 5,
- "shortcode": 139,
- "taint_trace": null
+ "shortcode": 139
},
{
"severity": "info",
@@ -2690,8 +2460,7 @@
"snippet": " * @return \\Symfony\\Component\\Finder\\SplFileInfo[]|\\Symfony\\Component\\Finder\\Finder",
"selected_text": "\\Symfony\\Component\\Finder\\SplFileInfo[]|\\Symfony\\Component\\Finder\\Finder",
"error_level": 4,
- "shortcode": 142,
- "taint_trace": null
+ "shortcode": 142
},
{
"severity": "info",
@@ -2703,8 +2472,7 @@
"snippet": " $organizationName = $pathFragments[$srcPosition + 1];",
"selected_text": "$srcPosition",
"error_level": 3,
- "shortcode": 162,
- "taint_trace": null
+ "shortcode": 162
},
{
"severity": "info",
@@ -2716,8 +2484,7 @@
"snippet": " $organizationName = $pathFragments[$srcPosition + 2];",
"selected_text": "$srcPosition",
"error_level": 3,
- "shortcode": 162,
- "taint_trace": null
+ "shortcode": 162
},
{
"severity": "info",
@@ -2729,8 +2496,7 @@
"snippet": " return sprintf('%s.%s', $moduleTransfer->getOrganization()->getName(), $moduleTransfer->getName());",
"selected_text": "$moduleTransfer->getOrganization()->getName()",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
@@ -2742,8 +2508,7 @@
"snippet": " return sprintf('%s.%s', $moduleTransfer->getOrganization()->getName(), $moduleTransfer->getName());",
"selected_text": "getName",
"error_level": 3,
- "shortcode": 83,
- "taint_trace": null
+ "shortcode": 83
},
{
"severity": "info",
@@ -2755,8 +2520,7 @@
"snippet": " return sprintf('%s.%s', $moduleTransfer->getOrganization()->getName(), $moduleTransfer->getName());",
"selected_text": "$moduleTransfer->getName()",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
@@ -2768,8 +2532,7 @@
"snippet": " $packageCollectionKey = sprintf('%s.%s', $packageTransfer->getOrganizationName(), $packageTransfer->getPackageName());",
"selected_text": "$packageTransfer->getOrganizationName()",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
@@ -2781,8 +2544,7 @@
"snippet": " $packageCollectionKey = sprintf('%s.%s', $packageTransfer->getOrganizationName(), $packageTransfer->getPackageName());",
"selected_text": "$packageTransfer->getPackageName()",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
@@ -2794,8 +2556,7 @@
"snippet": " * @return \\Symfony\\Component\\Finder\\SplFileInfo[]|\\Symfony\\Component\\Finder\\Finder",
"selected_text": "\\Symfony\\Component\\Finder\\SplFileInfo[]|\\Symfony\\Component\\Finder\\Finder",
"error_level": 4,
- "shortcode": 142,
- "taint_trace": null
+ "shortcode": 142
},
{
"severity": "info",
@@ -2807,8 +2568,7 @@
"snippet": " $composerJsonAsArray = $this->getComposerJsonAsArray($packageTransfer->getPath());",
"selected_text": "$packageTransfer->getPath()",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
@@ -2820,8 +2580,7 @@
"snippet": " $path = $this->resolvePath($bundle);",
"selected_text": "$bundle",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
@@ -2833,8 +2592,7 @@
"snippet": " return ucfirst($filter->filter($value));",
"selected_text": "$filter->filter($value)",
"error_level": 3,
- "shortcode": 92,
- "taint_trace": null
+ "shortcode": 92
},
{
"severity": "info",
@@ -2846,8 +2604,7 @@
"snippet": " return $process->getExitCode();",
"selected_text": "$process->getExitCode()",
"error_level": 5,
- "shortcode": 139,
- "taint_trace": null
+ "shortcode": 139
},
{
"severity": "info",
@@ -2859,8 +2616,7 @@
"snippet": " public function apply(AbstractNode $node)",
"selected_text": "$node",
"error_level": 5,
- "shortcode": 140,
- "taint_trace": null
+ "shortcode": 140
},
{
"severity": "info",
@@ -2872,8 +2628,7 @@
"snippet": " public function apply(AbstractNode $node)",
"selected_text": "$node",
"error_level": 5,
- "shortcode": 140,
- "taint_trace": null
+ "shortcode": 140
},
{
"severity": "info",
@@ -2885,86 +2640,91 @@
"snippet": " $paths = $this->getPaths($module);",
"selected_text": "$module",
"error_level": 3,
- "shortcode": 92,
- "taint_trace": null
+ "shortcode": 92
},
{
"severity": "info",
- "line_from": 193,
- "line_to": 193,
+ "line_from": 160,
+ "line_to": 160,
+ "type": "RedundantCondition",
+ "message": "Found a redundant condition when evaluating $progressBar and trying to reconcile type 'Symfony\\Component\\Console\\Helper\\ProgressBar' to object with method maxSecondsBetweenRedraws",
+ "file_name": "vendor/spryker/spryker/Bundles/Development/src/Spryker/Zed/Development/Business/Phpstan/PhpstanRunner.php",
+ "snippet": " if (method_exists($progressBar, 'maxSecondsBetweenRedraws')) {",
+ "selected_text": "method_exists($progressBar, 'maxSecondsBetweenRedraws')",
+ "error_level": 4,
+ "shortcode": 122
+ },
+ {
+ "severity": "info",
+ "line_from": 195,
+ "line_to": 195,
"type": "PossiblyInvalidArgument",
"message": "Argument 2 of preg_match expects string, possibly different type array|bool|null|string provided",
"file_name": "vendor/spryker/spryker/Bundles/Development/src/Spryker/Zed/Development/Business/Phpstan/PhpstanRunner.php",
"snippet": " if (preg_match('/^([+])(\\d)$/', $level, $matches)) {",
"selected_text": "$level",
"error_level": 3,
- "shortcode": 92,
- "taint_trace": null
+ "shortcode": 92
},
{
"severity": "info",
- "line_from": 224,
- "line_to": 224,
+ "line_from": 226,
+ "line_to": 226,
"type": "NullableReturnStatement",
"message": "The declared return type 'int' for Spryker\\Zed\\Development\\Business\\Phpstan\\PhpstanRunner::runCommand is not nullable, but the function returns 'int|null'",
"file_name": "vendor/spryker/spryker/Bundles/Development/src/Spryker/Zed/Development/Business/Phpstan/PhpstanRunner.php",
"snippet": " return $process->getExitCode();",
"selected_text": "$process->getExitCode()",
"error_level": 5,
- "shortcode": 139,
- "taint_trace": null
+ "shortcode": 139
},
{
"severity": "info",
- "line_from": 369,
- "line_to": 369,
+ "line_from": 371,
+ "line_to": 371,
"type": "PossiblyNullArgument",
"message": "Argument 1 of Spryker\\Zed\\Development\\Business\\Phpstan\\PhpstanRunner::getVendorPathByNamespace cannot be null, possibly null value provided",
"file_name": "vendor/spryker/spryker/Bundles/Development/src/Spryker/Zed/Development/Business/Phpstan/PhpstanRunner.php",
"snippet": " $vendorDirectoryPath = $this->getVendorPathByNamespace($namespace);",
"selected_text": "$namespace",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
- "line_from": 372,
- "line_to": 372,
+ "line_from": 374,
+ "line_to": 374,
"type": "PossiblyNullArgument",
"message": "Argument 1 of Spryker\\Zed\\Development\\Business\\Phpstan\\Config\\PhpstanConfigFileFinderInterface::searchIn cannot be null, possibly null value provided",
"file_name": "vendor/spryker/spryker/Bundles/Development/src/Spryker/Zed/Development/Business/Phpstan/PhpstanRunner.php",
"snippet": " ->searchIn($vendorDirectoryPath);",
"selected_text": "$vendorDirectoryPath",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
- "line_from": 377,
- "line_to": 377,
+ "line_from": 379,
+ "line_to": 379,
"type": "PossiblyNullArgument",
"message": "Argument 2 of Spryker\\Zed\\Development\\Business\\Phpstan\\Config\\PhpstanConfigFileManagerInterface::merge cannot be null, possibly null value provided",
"file_name": "vendor/spryker/spryker/Bundles/Development/src/Spryker/Zed/Development/Business/Phpstan/PhpstanRunner.php",
"snippet": " $this->getConfigFilenameForMerge($moduleConfigFile)",
"selected_text": "$this->getConfigFilenameForMerge($moduleConfigFile)",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
- "line_from": 426,
- "line_to": 426,
+ "line_from": 428,
+ "line_to": 428,
"type": "PossiblyNullArgument",
"message": "Argument 1 of dirname cannot be null, possibly null value provided",
"file_name": "vendor/spryker/spryker/Bundles/Development/src/Spryker/Zed/Development/Business/Phpstan/PhpstanRunner.php",
"snippet": " return dirname($pathToModules) . DIRECTORY_SEPARATOR;",
"selected_text": "$pathToModules",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
@@ -2976,8 +2736,7 @@
"snippet": " $this->bundles[$currentBundleName]['out'][$outgoingBundleName] = $outgoingBundleName;",
"selected_text": "$this->bundles[$currentBundleName]['out']",
"error_level": 3,
- "shortcode": 120,
- "taint_trace": null
+ "shortcode": 120
},
{
"severity": "info",
@@ -2989,8 +2748,7 @@
"snippet": " $this->bundles[$outgoingBundleName]['in'][$currentBundleName] = $currentBundleName;",
"selected_text": "$this->bundles[$outgoingBundleName]['in']",
"error_level": 3,
- "shortcode": 120,
- "taint_trace": null
+ "shortcode": 120
},
{
"severity": "info",
@@ -3002,8 +2760,7 @@
"snippet": " * @return \\ArrayObject|\\Generated\\Shared\\Transfer\\ModuleDependencyTransfer[]",
"selected_text": "\\ArrayObject|\\Generated\\Shared\\Transfer\\ModuleDependencyTransfer[]",
"error_level": 4,
- "shortcode": 142,
- "taint_trace": null
+ "shortcode": 142
},
{
"severity": "info",
@@ -3015,8 +2772,7 @@
"snippet": " * @param \\Generated\\Shared\\Transfer\\ModuleDependencyTransfer[]|\\ArrayObject $moduleDependencyTransferCollection",
"selected_text": "\\Generated\\Shared\\Transfer\\ModuleDependencyTransfer[]|\\ArrayObject",
"error_level": 4,
- "shortcode": 141,
- "taint_trace": null
+ "shortcode": 141
},
{
"severity": "info",
@@ -3028,8 +2784,7 @@
"snippet": " if (strpos($moduleArgument, '.') === false) {",
"selected_text": "$moduleArgument",
"error_level": 3,
- "shortcode": 92,
- "taint_trace": null
+ "shortcode": 92
},
{
"severity": "info",
@@ -3041,8 +2796,7 @@
"snippet": " $moduleTransfer->setName($moduleArgument);",
"selected_text": "$moduleArgument",
"error_level": 3,
- "shortcode": 92,
- "taint_trace": null
+ "shortcode": 92
},
{
"severity": "info",
@@ -3054,8 +2808,7 @@
"snippet": " $this->addFilterDetails($moduleArgument, $moduleFilterTransfer);",
"selected_text": "$moduleArgument",
"error_level": 3,
- "shortcode": 92,
- "taint_trace": null
+ "shortcode": 92
},
{
"severity": "info",
@@ -3067,8 +2820,7 @@
"snippet": " return sprintf('%s.%s', $moduleTransfer->getOrganization()->getName(), $moduleTransfer->getName());",
"selected_text": "$moduleTransfer->getOrganization()->getName()",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
@@ -3080,8 +2832,7 @@
"snippet": " return sprintf('%s.%s', $moduleTransfer->getOrganization()->getName(), $moduleTransfer->getName());",
"selected_text": "getName",
"error_level": 3,
- "shortcode": 83,
- "taint_trace": null
+ "shortcode": 83
},
{
"severity": "info",
@@ -3093,8 +2844,7 @@
"snippet": " return sprintf('%s.%s', $moduleTransfer->getOrganization()->getName(), $moduleTransfer->getName());",
"selected_text": "$moduleTransfer->getName()",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
@@ -3106,8 +2856,7 @@
"snippet": " $isCore = strpos($module, '.') !== false;",
"selected_text": "$module",
"error_level": 3,
- "shortcode": 92,
- "taint_trace": null
+ "shortcode": 92
},
{
"severity": "info",
@@ -3119,8 +2868,7 @@
"snippet": " $module = $this->normalizeModuleName($module);",
"selected_text": "$module",
"error_level": 3,
- "shortcode": 92,
- "taint_trace": null
+ "shortcode": 92
},
{
"severity": "info",
@@ -3132,8 +2880,7 @@
"snippet": " $message .= ' (' . $path . ')';",
"selected_text": "$path",
"error_level": 3,
- "shortcode": 163,
- "taint_trace": null
+ "shortcode": 163
},
{
"severity": "info",
@@ -3145,8 +2892,7 @@
"snippet": " $success = $this->runForCore($output, $module, $path);",
"selected_text": "$module",
"error_level": 3,
- "shortcode": 92,
- "taint_trace": null
+ "shortcode": 92
},
{
"severity": "info",
@@ -3158,8 +2904,7 @@
"snippet": " $success = $this->runForCore($output, $module, $path);",
"selected_text": "$path",
"error_level": 3,
- "shortcode": 92,
- "taint_trace": null
+ "shortcode": 92
},
{
"severity": "info",
@@ -3171,8 +2916,7 @@
"snippet": " $customPath = $this->getCustomPath($module, $path);",
"selected_text": "$module",
"error_level": 3,
- "shortcode": 92,
- "taint_trace": null
+ "shortcode": 92
},
{
"severity": "info",
@@ -3184,8 +2928,7 @@
"snippet": " $customPath = $this->getCustomPath($module, $path);",
"selected_text": "$path",
"error_level": 3,
- "shortcode": 92,
- "taint_trace": null
+ "shortcode": 92
},
{
"severity": "info",
@@ -3197,8 +2940,7 @@
"snippet": " $success = $this->runForProject($output, $module, $path);",
"selected_text": "$module",
"error_level": 3,
- "shortcode": 92,
- "taint_trace": null
+ "shortcode": 92
},
{
"severity": "info",
@@ -3210,8 +2952,7 @@
"snippet": " $success = $this->runForProject($output, $module, $path);",
"selected_text": "$path",
"error_level": 3,
- "shortcode": 92,
- "taint_trace": null
+ "shortcode": 92
},
{
"severity": "info",
@@ -3223,8 +2964,7 @@
"snippet": " return $this->buildPath($moduleTransfer->getPath(), $pathSuffix);",
"selected_text": "$moduleTransfer->getPath()",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
@@ -3236,8 +2976,7 @@
"snippet": " return rtrim($path . $suffix, DIRECTORY_SEPARATOR) . DIRECTORY_SEPARATOR;",
"selected_text": "$suffix",
"error_level": 1,
- "shortcode": 80,
- "taint_trace": null
+ "shortcode": 80
},
{
"severity": "info",
@@ -3249,8 +2988,7 @@
"snippet": " $normalized = ucfirst($normalized);",
"selected_text": "$normalized",
"error_level": 3,
- "shortcode": 92,
- "taint_trace": null
+ "shortcode": 92
},
{
"severity": "info",
@@ -3262,8 +3000,7 @@
"snippet": " $customPath = $pathToRoot . $path;",
"selected_text": "$path",
"error_level": 1,
- "shortcode": 80,
- "taint_trace": null
+ "shortcode": 80
},
{
"severity": "info",
@@ -3275,8 +3012,7 @@
"snippet": " $this->displayRunInfo($module);",
"selected_text": "$module",
"error_level": 3,
- "shortcode": 92,
- "taint_trace": null
+ "shortcode": 92
},
{
"severity": "info",
@@ -3288,8 +3024,7 @@
"snippet": " return $this->getFacade()->runFixtures($module, $this->input->getOptions());",
"selected_text": "$module",
"error_level": 3,
- "shortcode": 92,
- "taint_trace": null
+ "shortcode": 92
},
{
"severity": "info",
@@ -3301,8 +3036,7 @@
"snippet": " return $this->getFacade()->runPhpMd($module, $this->input->getOptions());",
"selected_text": "$module",
"error_level": 3,
- "shortcode": 92,
- "taint_trace": null
+ "shortcode": 92
},
{
"severity": "info",
@@ -3314,8 +3048,7 @@
"snippet": " $this->info($this->buildMessage($module, $path));",
"selected_text": "$module",
"error_level": 3,
- "shortcode": 92,
- "taint_trace": null
+ "shortcode": 92
},
{
"severity": "info",
@@ -3327,8 +3060,7 @@
"snippet": " $this->info($this->buildMessage($module, $path));",
"selected_text": "$path",
"error_level": 3,
- "shortcode": 92,
- "taint_trace": null
+ "shortcode": 92
},
{
"severity": "info",
@@ -3340,8 +3072,7 @@
"snippet": " $exitCode = $this->getFacade()->checkCodeStyle($module, $this->input->getOptions() + [static::ARGUMENT_SUB_PATH => $path]);",
"selected_text": "$module",
"error_level": 3,
- "shortcode": 92,
- "taint_trace": null
+ "shortcode": 92
},
{
"severity": "info",
@@ -3353,8 +3084,7 @@
"snippet": " $isCore = strpos($module, '.') !== false;",
"selected_text": "$module",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
@@ -3366,8 +3096,7 @@
"snippet": " $normalized = ucfirst($normalized);",
"selected_text": "$normalized",
"error_level": 3,
- "shortcode": 92,
- "taint_trace": null
+ "shortcode": 92
},
{
"severity": "info",
@@ -3379,8 +3108,7 @@
"snippet": " $this->displayRunInfo($input, $module);",
"selected_text": "$module",
"error_level": 3,
- "shortcode": 92,
- "taint_trace": null
+ "shortcode": 92
},
{
"severity": "info",
@@ -3392,8 +3120,7 @@
"snippet": " $module,",
"selected_text": "$module",
"error_level": 3,
- "shortcode": 92,
- "taint_trace": null
+ "shortcode": 92
},
{
"severity": "info",
@@ -3405,8 +3132,7 @@
"snippet": " $this->input->getArgument(static::ARGUMENT_MODULE)",
"selected_text": "$this->input->getArgument(static::ARGUMENT_MODULE)",
"error_level": 3,
- "shortcode": 92,
- "taint_trace": null
+ "shortcode": 92
},
{
"severity": "info",
@@ -3418,8 +3144,7 @@
"snippet": " $processedModules = $this->getFacade()->updateComposerJsonInModules($modules, $isDryRun);",
"selected_text": "$isDryRun",
"error_level": 3,
- "shortcode": 92,
- "taint_trace": null
+ "shortcode": 92
},
{
"severity": "info",
@@ -3431,8 +3156,7 @@
"snippet": " $this->output->writeln($validationMessageTransfer->getMessage());",
"selected_text": "$validationMessageTransfer->getMessage()",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
@@ -3444,8 +3168,7 @@
"snippet": " $this->getFacade()->buildDependencyTree($module);",
"selected_text": "$module",
"error_level": 3,
- "shortcode": 92,
- "taint_trace": null
+ "shortcode": 92
},
{
"severity": "info",
@@ -3457,8 +3180,7 @@
"snippet": " * @param \\Generated\\Shared\\Transfer\\ModuleDependencyTransfer[]|\\ArrayObject $moduleDependencyTransferCollection",
"selected_text": "\\Generated\\Shared\\Transfer\\ModuleDependencyTransfer[]|\\ArrayObject",
"error_level": 4,
- "shortcode": 141,
- "taint_trace": null
+ "shortcode": 141
},
{
"severity": "info",
@@ -3470,8 +3192,7 @@
"snippet": " * @param \\Generated\\Shared\\Transfer\\ModuleDependencyTransfer[]|\\ArrayObject $moduleDependencyTransferCollection",
"selected_text": "\\Generated\\Shared\\Transfer\\ModuleDependencyTransfer[]|\\ArrayObject",
"error_level": 4,
- "shortcode": 141,
- "taint_trace": null
+ "shortcode": 141
},
{
"severity": "info",
@@ -3483,112 +3204,103 @@
"snippet": " 'Composer name' => sprintf('%s>', $color, $composerName),",
"selected_text": "$composerName",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
- "line_from": 194,
- "line_to": 194,
+ "line_from": 193,
+ "line_to": 193,
"type": "PossiblyNullArgument",
"message": "Argument 1 of Spryker\\Zed\\Development\\Communication\\Console\\DependencyViolationFinderConsole::getColoredYesOrNo cannot be null, possibly null value provided",
"file_name": "vendor/spryker/spryker/Bundles/Development/src/Spryker/Zed/Development/Communication/Console/DependencyViolationFinderConsole.php",
"snippet": " 'is valid' => $this->getColoredYesOrNo($moduleDependencyTransfer->getIsValid()),",
"selected_text": "$moduleDependencyTransfer->getIsValid()",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
- "line_from": 195,
- "line_to": 195,
+ "line_from": 194,
+ "line_to": 194,
"type": "PossiblyNullArgument",
"message": "Argument 1 of Spryker\\Zed\\Development\\Communication\\Console\\DependencyViolationFinderConsole::getYesOrNo cannot be null, possibly null value provided",
"file_name": "vendor/spryker/spryker/Bundles/Development/src/Spryker/Zed/Development/Communication/Console/DependencyViolationFinderConsole.php",
"snippet": " 'src dependency' => $this->getYesOrNo($moduleDependencyTransfer->getIsSrcDependency()),",
"selected_text": "$moduleDependencyTransfer->getIsSrcDependency()",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
- "line_from": 196,
- "line_to": 196,
+ "line_from": 195,
+ "line_to": 195,
"type": "PossiblyNullArgument",
"message": "Argument 1 of Spryker\\Zed\\Development\\Communication\\Console\\DependencyViolationFinderConsole::getYesOrNo cannot be null, possibly null value provided",
"file_name": "vendor/spryker/spryker/Bundles/Development/src/Spryker/Zed/Development/Communication/Console/DependencyViolationFinderConsole.php",
"snippet": " 'test dependency' => $this->getYesOrNo($moduleDependencyTransfer->getIsTestDependency()),",
"selected_text": "$moduleDependencyTransfer->getIsTestDependency()",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
- "line_from": 197,
- "line_to": 197,
+ "line_from": 196,
+ "line_to": 196,
"type": "PossiblyNullArgument",
"message": "Argument 1 of Spryker\\Zed\\Development\\Communication\\Console\\DependencyViolationFinderConsole::getYesOrNo cannot be null, possibly null value provided",
"file_name": "vendor/spryker/spryker/Bundles/Development/src/Spryker/Zed/Development/Communication/Console/DependencyViolationFinderConsole.php",
"snippet": " 'in composer require' => $this->getYesOrNo($moduleDependencyTransfer->getIsInComposerRequire()),",
"selected_text": "$moduleDependencyTransfer->getIsInComposerRequire()",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
- "line_from": 198,
- "line_to": 198,
+ "line_from": 197,
+ "line_to": 197,
"type": "PossiblyNullArgument",
"message": "Argument 1 of Spryker\\Zed\\Development\\Communication\\Console\\DependencyViolationFinderConsole::getYesOrNo cannot be null, possibly null value provided",
"file_name": "vendor/spryker/spryker/Bundles/Development/src/Spryker/Zed/Development/Communication/Console/DependencyViolationFinderConsole.php",
"snippet": " 'in composer require-dev' => $this->getYesOrNo($moduleDependencyTransfer->getIsInComposerRequireDev()),",
"selected_text": "$moduleDependencyTransfer->getIsInComposerRequireDev()",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
- "line_from": 199,
- "line_to": 199,
+ "line_from": 198,
+ "line_to": 198,
"type": "PossiblyNullArgument",
"message": "Argument 1 of Spryker\\Zed\\Development\\Communication\\Console\\DependencyViolationFinderConsole::getYesOrNo cannot be null, possibly null value provided",
"file_name": "vendor/spryker/spryker/Bundles/Development/src/Spryker/Zed/Development/Communication/Console/DependencyViolationFinderConsole.php",
"snippet": " 'in composer suggest' => $this->getYesOrNo($moduleDependencyTransfer->getIsSuggested()),",
"selected_text": "$moduleDependencyTransfer->getIsSuggested()",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
- "line_from": 200,
- "line_to": 200,
+ "line_from": 199,
+ "line_to": 199,
"type": "PossiblyNullArgument",
"message": "Argument 1 of Spryker\\Zed\\Development\\Communication\\Console\\DependencyViolationFinderConsole::getYesOrNo cannot be null, possibly null value provided",
"file_name": "vendor/spryker/spryker/Bundles/Development/src/Spryker/Zed/Development/Communication/Console/DependencyViolationFinderConsole.php",
"snippet": " 'is optional dependency' => $this->getYesOrNo($moduleDependencyTransfer->getIsOptionalDependency()),",
"selected_text": "$moduleDependencyTransfer->getIsOptionalDependency()",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
- "line_from": 201,
- "line_to": 201,
+ "line_from": 200,
+ "line_to": 200,
"type": "PossiblyNullArgument",
"message": "Argument 1 of Spryker\\Zed\\Development\\Communication\\Console\\DependencyViolationFinderConsole::getYesOrNo cannot be null, possibly null value provided",
"file_name": "vendor/spryker/spryker/Bundles/Development/src/Spryker/Zed/Development/Communication/Console/DependencyViolationFinderConsole.php",
"snippet": " 'is own extension module' => $this->getYesOrNo($moduleDependencyTransfer->getIsOwnExtensionModule()),",
"selected_text": "$moduleDependencyTransfer->getIsOwnExtensionModule()",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
@@ -3600,8 +3312,7 @@
"snippet": " $this->output->writeln(sprintf('No dependency issues found in %s.%s>', $moduleTransfer->getOrganization()->getName(), $moduleTransfer->getName()));",
"selected_text": "$moduleTransfer->getOrganization()->getName()",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
@@ -3613,8 +3324,7 @@
"snippet": " $this->output->writeln(sprintf('No dependency issues found in %s.%s>', $moduleTransfer->getOrganization()->getName(), $moduleTransfer->getName()));",
"selected_text": "getName",
"error_level": 3,
- "shortcode": 83,
- "taint_trace": null
+ "shortcode": 83
},
{
"severity": "info",
@@ -3626,8 +3336,7 @@
"snippet": " $this->output->writeln(sprintf('No dependency issues found in %s.%s>', $moduleTransfer->getOrganization()->getName(), $moduleTransfer->getName()));",
"selected_text": "$moduleTransfer->getName()",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
@@ -3639,8 +3348,7 @@
"snippet": " $composerNameToFix = $this->getFacade()->findComposerNameByModuleName($moduleDependencyTransfer->getModuleName());",
"selected_text": "$moduleDependencyTransfer->getModuleName()",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
@@ -3652,8 +3360,7 @@
"snippet": " $this->output->writeln(sprintf('Could not get a composer name for \"%s\"', $moduleDependencyTransfer->getModuleName()));",
"selected_text": "$moduleDependencyTransfer->getModuleName()",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
@@ -3665,8 +3372,7 @@
"snippet": " $this->output->writeln(sprintf('Please check the module %s.%s> manually.', $moduleTransfer->getOrganization()->getName(), $moduleTransfer->getName()));",
"selected_text": "$moduleTransfer->getOrganization()->getName()",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
@@ -3678,8 +3384,7 @@
"snippet": " $this->output->writeln(sprintf('Please check the module %s.%s> manually.', $moduleTransfer->getOrganization()->getName(), $moduleTransfer->getName()));",
"selected_text": "getName",
"error_level": 3,
- "shortcode": 83,
- "taint_trace": null
+ "shortcode": 83
},
{
"severity": "info",
@@ -3691,8 +3396,7 @@
"snippet": " $this->output->writeln(sprintf('Please check the module %s.%s> manually.', $moduleTransfer->getOrganization()->getName(), $moduleTransfer->getName()));",
"selected_text": "$moduleTransfer->getName()",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
@@ -3704,8 +3408,7 @@
"snippet": " $this->output->writeln(sprintf('Fixed dependencies in %s.%s>', $moduleTransfer->getOrganization()->getName(), $moduleTransfer->getName()));",
"selected_text": "$moduleTransfer->getOrganization()->getName()",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
@@ -3717,8 +3420,7 @@
"snippet": " $this->output->writeln(sprintf('Fixed dependencies in %s.%s>', $moduleTransfer->getOrganization()->getName(), $moduleTransfer->getName()));",
"selected_text": "getName",
"error_level": 3,
- "shortcode": 83,
- "taint_trace": null
+ "shortcode": 83
},
{
"severity": "info",
@@ -3730,8 +3432,7 @@
"snippet": " $this->output->writeln(sprintf('Fixed dependencies in %s.%s>', $moduleTransfer->getOrganization()->getName(), $moduleTransfer->getName()));",
"selected_text": "$moduleTransfer->getName()",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
@@ -3743,8 +3444,7 @@
"snippet": " $composerJsonFile = $moduleTransfer->getPath() . '/composer.json';",
"selected_text": "$moduleTransfer->getPath()",
"error_level": 1,
- "shortcode": 80,
- "taint_trace": null
+ "shortcode": 80
},
{
"severity": "info",
@@ -3756,8 +3456,7 @@
"snippet": " $composerJsonFile = $moduleTransfer->getPath() . '/composer.json';",
"selected_text": "$moduleTransfer->getPath()",
"error_level": 1,
- "shortcode": 80,
- "taint_trace": null
+ "shortcode": 80
},
{
"severity": "info",
@@ -3769,8 +3468,7 @@
"snippet": " $methods = explode(',', $this->input->getOption(static::OPTION_METHODS));",
"selected_text": "$this->input->getOption(static::OPTION_METHODS)",
"error_level": 3,
- "shortcode": 92,
- "taint_trace": null
+ "shortcode": 92
},
{
"severity": "info",
@@ -3782,8 +3480,7 @@
"snippet": " $message = 'Create bridge in ' . $module;",
"selected_text": "$module",
"error_level": 1,
- "shortcode": 80,
- "taint_trace": null
+ "shortcode": 80
},
{
"severity": "info",
@@ -3795,8 +3492,7 @@
"snippet": " $message = 'Create bridge in ' . $module;",
"selected_text": "$module",
"error_level": 3,
- "shortcode": 163,
- "taint_trace": null
+ "shortcode": 163
},
{
"severity": "info",
@@ -3808,8 +3504,7 @@
"snippet": " $this->getFacade()->createBridge($module, $toModule, $methods);",
"selected_text": "$module",
"error_level": 3,
- "shortcode": 92,
- "taint_trace": null
+ "shortcode": 92
},
{
"severity": "info",
@@ -3821,8 +3516,7 @@
"snippet": " $this->getFacade()->createBridge($module, $toModule, $methods);",
"selected_text": "$toModule",
"error_level": 3,
- "shortcode": 92,
- "taint_trace": null
+ "shortcode": 92
},
{
"severity": "info",
@@ -3834,8 +3528,7 @@
"snippet": " $message = 'Create or update Spryker core module ' . $module;",
"selected_text": "$module",
"error_level": 1,
- "shortcode": 80,
- "taint_trace": null
+ "shortcode": 80
},
{
"severity": "info",
@@ -3847,8 +3540,7 @@
"snippet": " $message = 'Create or update Spryker core module ' . $module;",
"selected_text": "$module",
"error_level": 3,
- "shortcode": 163,
- "taint_trace": null
+ "shortcode": 163
},
{
"severity": "info",
@@ -3860,8 +3552,7 @@
"snippet": " $this->getFacade()->createModule($module, $options);",
"selected_text": "$module",
"error_level": 3,
- "shortcode": 92,
- "taint_trace": null
+ "shortcode": 92
},
{
"severity": "info",
@@ -3873,8 +3564,7 @@
"snippet": " if (strpos($moduleArgument, '.') === false) {",
"selected_text": "$moduleArgument",
"error_level": 3,
- "shortcode": 92,
- "taint_trace": null
+ "shortcode": 92
},
{
"severity": "info",
@@ -3886,8 +3576,7 @@
"snippet": " $moduleTransfer->setName($moduleArgument);",
"selected_text": "$moduleArgument",
"error_level": 3,
- "shortcode": 92,
- "taint_trace": null
+ "shortcode": 92
},
{
"severity": "info",
@@ -3899,8 +3588,7 @@
"snippet": " $this->addFilterDetails($moduleArgument, $moduleFilterTransfer);",
"selected_text": "$moduleArgument",
"error_level": 3,
- "shortcode": 92,
- "taint_trace": null
+ "shortcode": 92
},
{
"severity": "info",
@@ -3912,8 +3600,7 @@
"snippet": " $message = $this->buildMessage($module);",
"selected_text": "$module",
"error_level": 3,
- "shortcode": 92,
- "taint_trace": null
+ "shortcode": 92
},
{
"severity": "info",
@@ -3925,8 +3612,7 @@
"snippet": " $result = $this->getFacade()->runPropelAbstractValidation($output, $module);",
"selected_text": "$module",
"error_level": 3,
- "shortcode": 92,
- "taint_trace": null
+ "shortcode": 92
},
{
"severity": "info",
@@ -3938,8 +3624,7 @@
"snippet": " $fileViolations = $this->getFacade()->runArchitectureSniffer($directory);",
"selected_text": "$directory",
"error_level": 3,
- "shortcode": 78,
- "taint_trace": null
+ "shortcode": 78
},
{
"severity": "info",
@@ -3951,8 +3636,7 @@
"snippet": " 'organization' => $dependencyProviderTransfer->getModule()->getOrganization()->getName(),",
"selected_text": "getOrganization",
"error_level": 3,
- "shortcode": 83,
- "taint_trace": null
+ "shortcode": 83
},
{
"severity": "info",
@@ -3964,8 +3648,7 @@
"snippet": " 'organization' => $dependencyProviderTransfer->getModule()->getOrganization()->getName(),",
"selected_text": "getName",
"error_level": 3,
- "shortcode": 83,
- "taint_trace": null
+ "shortcode": 83
},
{
"severity": "info",
@@ -3977,8 +3660,7 @@
"snippet": " 'module' => $dependencyProviderTransfer->getModule()->getName(),",
"selected_text": "getName",
"error_level": 3,
- "shortcode": 83,
- "taint_trace": null
+ "shortcode": 83
}
],
"deprecation": [
@@ -3992,8 +3674,19 @@
"snippet": "class AllBundleFinder implements AllBundleFinderInterface",
"selected_text": "AllBundleFinder",
"error_level": 2,
- "shortcode": 152,
- "taint_trace": null
+ "shortcode": 152
+ },
+ {
+ "severity": "info",
+ "line_from": 55,
+ "line_to": 55,
+ "type": "DeprecatedMethod",
+ "message": "The method Spryker\\Zed\\Development\\Business\\Dependency\\DependencyContainer\\DependencyContainer::makeOptionalIfSilex has been marked as deprecated",
+ "file_name": "vendor/spryker/spryker/Bundles/Development/src/Spryker/Zed/Development/Business/Dependency/DependencyContainer/DependencyContainer.php",
+ "snippet": " $isOptional = $this->makeOptionalIfSilex($moduleName, $isOptional);",
+ "selected_text": "makeOptionalIfSilex",
+ "error_level": 2,
+ "shortcode": 1
},
{
"severity": "info",
@@ -4005,333 +3698,307 @@
"snippet": "class DependencyViolationChecker implements DependencyViolationCheckerInterface",
"selected_text": "DependencyViolationChecker",
"error_level": 2,
- "shortcode": 152,
- "taint_trace": null
+ "shortcode": 152
},
{
"severity": "info",
- "line_from": 836,
- "line_to": 836,
+ "line_from": 857,
+ "line_to": 857,
"type": "DeprecatedClass",
"message": "Class Spryker\\Zed\\Development\\Business\\Module\\ModuleFinder\\ModuleFinderInterface is marked as deprecated",
"file_name": "vendor/spryker/spryker/Bundles/Development/src/Spryker/Zed/Development/Business/DevelopmentBusinessFactory.php",
"snippet": " * @return \\Spryker\\Zed\\Development\\Business\\Module\\ModuleFinder\\ModuleFinderInterface",
"selected_text": "\\Spryker\\Zed\\Development\\Business\\Module\\ModuleFinder\\ModuleFinderInterface",
"error_level": 2,
- "shortcode": 98,
- "taint_trace": null
+ "shortcode": 98
},
{
"severity": "info",
- "line_from": 838,
- "line_to": 838,
+ "line_from": 859,
+ "line_to": 859,
"type": "DeprecatedClass",
"message": "Class Spryker\\Zed\\Development\\Business\\Module\\ModuleFinder\\ModuleFinderInterface is marked as deprecated",
"file_name": "vendor/spryker/spryker/Bundles/Development/src/Spryker/Zed/Development/Business/DevelopmentBusinessFactory.php",
"snippet": " public function createModuleFinder(): ModuleFinderInterface",
"selected_text": "ModuleFinderInterface",
"error_level": 2,
- "shortcode": 98,
- "taint_trace": null
+ "shortcode": 98
},
{
"severity": "info",
- "line_from": 840,
- "line_to": 840,
+ "line_from": 861,
+ "line_to": 861,
"type": "DeprecatedClass",
"message": "Spryker\\Zed\\Development\\Business\\Module\\ModuleFinder\\ModuleFinder is marked deprecated",
"file_name": "vendor/spryker/spryker/Bundles/Development/src/Spryker/Zed/Development/Business/DevelopmentBusinessFactory.php",
"snippet": " return new ModuleFinder($this->getConfig(), $this->createModuleMatcher());",
"selected_text": "new ModuleFinder($this->getConfig(), $this->createModuleMatcher())",
"error_level": 2,
- "shortcode": 98,
- "taint_trace": null
+ "shortcode": 98
},
{
"severity": "info",
- "line_from": 840,
- "line_to": 840,
+ "line_from": 861,
+ "line_to": 861,
"type": "DeprecatedMethod",
"message": "The method Spryker\\Zed\\Development\\Business\\DevelopmentBusinessFactory::createModuleMatcher has been marked as deprecated",
"file_name": "vendor/spryker/spryker/Bundles/Development/src/Spryker/Zed/Development/Business/DevelopmentBusinessFactory.php",
"snippet": " return new ModuleFinder($this->getConfig(), $this->createModuleMatcher());",
"selected_text": "createModuleMatcher",
"error_level": 2,
- "shortcode": 1,
- "taint_trace": null
+ "shortcode": 1
},
{
"severity": "info",
- "line_from": 846,
- "line_to": 846,
+ "line_from": 867,
+ "line_to": 867,
"type": "DeprecatedClass",
"message": "Class Spryker\\Zed\\Development\\Business\\Module\\ModuleMatcher\\ModuleMatcherInterface is marked as deprecated",
"file_name": "vendor/spryker/spryker/Bundles/Development/src/Spryker/Zed/Development/Business/DevelopmentBusinessFactory.php",
"snippet": " * @return \\Spryker\\Zed\\Development\\Business\\Module\\ModuleMatcher\\ModuleMatcherInterface",
"selected_text": "\\Spryker\\Zed\\Development\\Business\\Module\\ModuleMatcher\\ModuleMatcherInterface",
"error_level": 2,
- "shortcode": 98,
- "taint_trace": null
+ "shortcode": 98
},
{
"severity": "info",
- "line_from": 848,
- "line_to": 848,
+ "line_from": 869,
+ "line_to": 869,
"type": "DeprecatedClass",
"message": "Class Spryker\\Zed\\Development\\Business\\Module\\ModuleMatcher\\ModuleMatcherInterface is marked as deprecated",
"file_name": "vendor/spryker/spryker/Bundles/Development/src/Spryker/Zed/Development/Business/DevelopmentBusinessFactory.php",
"snippet": " public function createModuleMatcher(): ModuleMatcherInterface",
"selected_text": "ModuleMatcherInterface",
"error_level": 2,
- "shortcode": 98,
- "taint_trace": null
+ "shortcode": 98
},
{
"severity": "info",
- "line_from": 850,
- "line_to": 850,
+ "line_from": 871,
+ "line_to": 871,
"type": "DeprecatedClass",
"message": "Spryker\\Zed\\Development\\Business\\Module\\ModuleMatcher\\ModuleMatcher is marked deprecated",
"file_name": "vendor/spryker/spryker/Bundles/Development/src/Spryker/Zed/Development/Business/DevelopmentBusinessFactory.php",
"snippet": " return new ModuleMatcher();",
"selected_text": "new ModuleMatcher()",
"error_level": 2,
- "shortcode": 98,
- "taint_trace": null
+ "shortcode": 98
},
{
"severity": "info",
- "line_from": 1287,
- "line_to": 1287,
+ "line_from": 1308,
+ "line_to": 1308,
"type": "DeprecatedClass",
"message": "Class Spryker\\Zed\\Development\\Business\\DependencyTree\\ViolationChecker\\DependencyViolationCheckerInterface is marked as deprecated",
"file_name": "vendor/spryker/spryker/Bundles/Development/src/Spryker/Zed/Development/Business/DevelopmentBusinessFactory.php",
"snippet": " * @return \\Spryker\\Zed\\Development\\Business\\DependencyTree\\ViolationChecker\\DependencyViolationCheckerInterface",
"selected_text": "\\Spryker\\Zed\\Development\\Business\\DependencyTree\\ViolationChecker\\DependencyViolationCheckerInterface",
"error_level": 2,
- "shortcode": 98,
- "taint_trace": null
+ "shortcode": 98
},
{
"severity": "info",
- "line_from": 1291,
- "line_to": 1295,
+ "line_from": 1312,
+ "line_to": 1316,
"type": "DeprecatedClass",
"message": "Spryker\\Zed\\Development\\Business\\DependencyTree\\ViolationChecker\\DependencyViolationChecker is marked deprecated",
"file_name": "vendor/spryker/spryker/Bundles/Development/src/Spryker/Zed/Development/Business/DevelopmentBusinessFactory.php",
"snippet": " return new DependencyViolationChecker(\n $this->createDependencyTreeReader(),\n $this->createViolationFinder(),\n $this->createDependencyViolationFilter()\n );",
"selected_text": "new DependencyViolationChecker(\n $this->createDependencyTreeReader(),\n $this->createViolationFinder(),\n $this->createDependencyViolationFilter()\n )",
"error_level": 2,
- "shortcode": 98,
- "taint_trace": null
+ "shortcode": 98
},
{
"severity": "info",
- "line_from": 1293,
- "line_to": 1293,
+ "line_from": 1314,
+ "line_to": 1314,
"type": "DeprecatedMethod",
"message": "The method Spryker\\Zed\\Development\\Business\\DevelopmentBusinessFactory::createViolationFinder has been marked as deprecated",
"file_name": "vendor/spryker/spryker/Bundles/Development/src/Spryker/Zed/Development/Business/DevelopmentBusinessFactory.php",
"snippet": " $this->createViolationFinder(),",
"selected_text": "createViolationFinder",
"error_level": 2,
- "shortcode": 1,
- "taint_trace": null
+ "shortcode": 1
},
{
"severity": "info",
- "line_from": 1294,
- "line_to": 1294,
+ "line_from": 1315,
+ "line_to": 1315,
"type": "DeprecatedMethod",
"message": "The method Spryker\\Zed\\Development\\Business\\DevelopmentBusinessFactory::createDependencyViolationFilter has been marked as deprecated",
"file_name": "vendor/spryker/spryker/Bundles/Development/src/Spryker/Zed/Development/Business/DevelopmentBusinessFactory.php",
"snippet": " $this->createDependencyViolationFilter()",
"selected_text": "createDependencyViolationFilter",
"error_level": 2,
- "shortcode": 1,
- "taint_trace": null
+ "shortcode": 1
},
{
"severity": "info",
- "line_from": 1307,
- "line_to": 1307,
+ "line_from": 1328,
+ "line_to": 1328,
"type": "DeprecatedMethod",
"message": "The method Spryker\\Zed\\Development\\Business\\DevelopmentBusinessFactory::createViolationFinderUseForeignConstants has been marked as deprecated",
"file_name": "vendor/spryker/spryker/Bundles/Development/src/Spryker/Zed/Development/Business/DevelopmentBusinessFactory.php",
"snippet": " ->addViolationFinder($this->createViolationFinderUseForeignConstants())",
"selected_text": "createViolationFinderUseForeignConstants",
"error_level": 2,
- "shortcode": 1,
- "taint_trace": null
+ "shortcode": 1
},
{
"severity": "info",
- "line_from": 1308,
- "line_to": 1308,
+ "line_from": 1329,
+ "line_to": 1329,
"type": "DeprecatedMethod",
"message": "The method Spryker\\Zed\\Development\\Business\\DevelopmentBusinessFactory::createViolationFinderUseForeignException has been marked as deprecated",
"file_name": "vendor/spryker/spryker/Bundles/Development/src/Spryker/Zed/Development/Business/DevelopmentBusinessFactory.php",
"snippet": " ->addViolationFinder($this->createViolationFinderUseForeignException())",
"selected_text": "createViolationFinderUseForeignException",
"error_level": 2,
- "shortcode": 1,
- "taint_trace": null
+ "shortcode": 1
},
{
"severity": "info",
- "line_from": 1309,
- "line_to": 1309,
+ "line_from": 1330,
+ "line_to": 1330,
"type": "DeprecatedMethod",
"message": "The method Spryker\\Zed\\Development\\Business\\DevelopmentBusinessFactory::createViolationFinderBundleUsesConnector has been marked as deprecated",
"file_name": "vendor/spryker/spryker/Bundles/Development/src/Spryker/Zed/Development/Business/DevelopmentBusinessFactory.php",
"snippet": " ->addViolationFinder($this->createViolationFinderBundleUsesConnector());",
"selected_text": "createViolationFinderBundleUsesConnector",
"error_level": 2,
- "shortcode": 1,
- "taint_trace": null
+ "shortcode": 1
},
{
"severity": "info",
- "line_from": 1323,
- "line_to": 1323,
+ "line_from": 1344,
+ "line_to": 1344,
"type": "DeprecatedMethod",
"message": "The method Spryker\\Zed\\Development\\Business\\DevelopmentBusinessFactory::createDependencyTreeConstantsToForeignConstantsFilter has been marked as deprecated",
"file_name": "vendor/spryker/spryker/Bundles/Development/src/Spryker/Zed/Development/Business/DevelopmentBusinessFactory.php",
"snippet": " ->addFilter($this->createDependencyTreeConstantsToForeignConstantsFilter())",
"selected_text": "createDependencyTreeConstantsToForeignConstantsFilter",
"error_level": 2,
- "shortcode": 1,
- "taint_trace": null
+ "shortcode": 1
},
{
"severity": "info",
- "line_from": 1376,
- "line_to": 1376,
+ "line_from": 1397,
+ "line_to": 1397,
"type": "DeprecatedClass",
"message": "Spryker\\Zed\\Development\\Business\\DependencyTree\\ViolationFinder\\BundleUsesConnector is marked deprecated",
"file_name": "vendor/spryker/spryker/Bundles/Development/src/Spryker/Zed/Development/Business/DevelopmentBusinessFactory.php",
"snippet": " return new BundleUsesConnector();",
"selected_text": "new BundleUsesConnector()",
"error_level": 2,
- "shortcode": 98,
- "taint_trace": null
+ "shortcode": 98
},
{
"severity": "info",
- "line_from": 1951,
- "line_to": 1951,
+ "line_from": 1972,
+ "line_to": 1972,
"type": "DeprecatedClass",
"message": "Class Spryker\\Zed\\Development\\Business\\ArchitectureSniffer\\AllBundleFinderInterface is marked as deprecated",
"file_name": "vendor/spryker/spryker/Bundles/Development/src/Spryker/Zed/Development/Business/DevelopmentBusinessFactory.php",
"snippet": " * @return \\Spryker\\Zed\\Development\\Business\\ArchitectureSniffer\\AllBundleFinderInterface",
"selected_text": "\\Spryker\\Zed\\Development\\Business\\ArchitectureSniffer\\AllBundleFinderInterface",
"error_level": 2,
- "shortcode": 98,
- "taint_trace": null
+ "shortcode": 98
},
{
"severity": "info",
- "line_from": 1955,
- "line_to": 1960,
+ "line_from": 1976,
+ "line_to": 1981,
"type": "DeprecatedClass",
"message": "Spryker\\Zed\\Development\\Business\\ArchitectureSniffer\\AllBundleFinder is marked deprecated",
"file_name": "vendor/spryker/spryker/Bundles/Development/src/Spryker/Zed/Development/Business/DevelopmentBusinessFactory.php",
"snippet": " return new AllBundleFinder(\n $this->createFinder(),\n $this->createCamelCaseToDashFilter(),\n $this->getConfig()->getProjectNamespaces(),\n $this->getConfig()->getCoreNamespaces()\n );",
"selected_text": "new AllBundleFinder(\n $this->createFinder(),\n $this->createCamelCaseToDashFilter(),\n $this->getConfig()->getProjectNamespaces(),\n $this->getConfig()->getCoreNamespaces()\n )",
"error_level": 2,
- "shortcode": 98,
- "taint_trace": null
+ "shortcode": 98
},
{
"severity": "info",
- "line_from": 1986,
- "line_to": 1986,
+ "line_from": 2007,
+ "line_to": 2007,
"type": "DeprecatedClass",
"message": "Class Spryker\\Zed\\Development\\Business\\Module\\ProjectModuleFinder\\ProjectModuleFinderInterface is marked as deprecated",
"file_name": "vendor/spryker/spryker/Bundles/Development/src/Spryker/Zed/Development/Business/DevelopmentBusinessFactory.php",
"snippet": " * @return \\Spryker\\Zed\\Development\\Business\\Module\\ProjectModuleFinder\\ProjectModuleFinderInterface",
"selected_text": "\\Spryker\\Zed\\Development\\Business\\Module\\ProjectModuleFinder\\ProjectModuleFinderInterface",
"error_level": 2,
- "shortcode": 98,
- "taint_trace": null
+ "shortcode": 98
},
{
"severity": "info",
- "line_from": 1988,
- "line_to": 1988,
+ "line_from": 2009,
+ "line_to": 2009,
"type": "DeprecatedClass",
"message": "Class Spryker\\Zed\\Development\\Business\\Module\\ProjectModuleFinder\\ProjectModuleFinderInterface is marked as deprecated",
"file_name": "vendor/spryker/spryker/Bundles/Development/src/Spryker/Zed/Development/Business/DevelopmentBusinessFactory.php",
"snippet": " public function createProjectModuleFinder(): ProjectModuleFinderInterface",
"selected_text": "ProjectModuleFinderInterface",
"error_level": 2,
- "shortcode": 98,
- "taint_trace": null
+ "shortcode": 98
},
{
"severity": "info",
- "line_from": 1990,
- "line_to": 1990,
+ "line_from": 2011,
+ "line_to": 2011,
"type": "DeprecatedClass",
"message": "Spryker\\Zed\\Development\\Business\\Module\\ProjectModuleFinder\\ProjectModuleFinder is marked deprecated",
"file_name": "vendor/spryker/spryker/Bundles/Development/src/Spryker/Zed/Development/Business/DevelopmentBusinessFactory.php",
"snippet": " return new ProjectModuleFinder($this->getConfig(), $this->createModuleMatcher());",
"selected_text": "new ProjectModuleFinder($this->getConfig(), $this->createModuleMatcher())",
"error_level": 2,
- "shortcode": 98,
- "taint_trace": null
+ "shortcode": 98
},
{
"severity": "info",
- "line_from": 1990,
- "line_to": 1990,
+ "line_from": 2011,
+ "line_to": 2011,
"type": "DeprecatedMethod",
"message": "The method Spryker\\Zed\\Development\\Business\\DevelopmentBusinessFactory::createModuleMatcher has been marked as deprecated",
"file_name": "vendor/spryker/spryker/Bundles/Development/src/Spryker/Zed/Development/Business/DevelopmentBusinessFactory.php",
"snippet": " return new ProjectModuleFinder($this->getConfig(), $this->createModuleMatcher());",
"selected_text": "createModuleMatcher",
"error_level": 2,
- "shortcode": 1,
- "taint_trace": null
+ "shortcode": 1
},
{
"severity": "info",
- "line_from": 1996,
- "line_to": 1996,
+ "line_from": 2017,
+ "line_to": 2017,
"type": "DeprecatedClass",
"message": "Class Spryker\\Zed\\Development\\Business\\Package\\PackageFinder\\PackageFinderInterface is marked as deprecated",
"file_name": "vendor/spryker/spryker/Bundles/Development/src/Spryker/Zed/Development/Business/DevelopmentBusinessFactory.php",
"snippet": " * @return \\Spryker\\Zed\\Development\\Business\\Package\\PackageFinder\\PackageFinderInterface",
"selected_text": "\\Spryker\\Zed\\Development\\Business\\Package\\PackageFinder\\PackageFinderInterface",
"error_level": 2,
- "shortcode": 98,
- "taint_trace": null
+ "shortcode": 98
},
{
"severity": "info",
- "line_from": 1998,
- "line_to": 1998,
+ "line_from": 2019,
+ "line_to": 2019,
"type": "DeprecatedClass",
"message": "Class Spryker\\Zed\\Development\\Business\\Package\\PackageFinder\\PackageFinderInterface is marked as deprecated",
"file_name": "vendor/spryker/spryker/Bundles/Development/src/Spryker/Zed/Development/Business/DevelopmentBusinessFactory.php",
"snippet": " public function createPackageFinder(): PackageFinderInterface",
"selected_text": "PackageFinderInterface",
"error_level": 2,
- "shortcode": 98,
- "taint_trace": null
+ "shortcode": 98
},
{
"severity": "info",
- "line_from": 2000,
- "line_to": 2000,
+ "line_from": 2021,
+ "line_to": 2021,
"type": "DeprecatedClass",
"message": "Spryker\\Zed\\Development\\Business\\Package\\PackageFinder\\PackageFinder is marked deprecated",
"file_name": "vendor/spryker/spryker/Bundles/Development/src/Spryker/Zed/Development/Business/DevelopmentBusinessFactory.php",
"snippet": " return new PackageFinder($this->getConfig());",
"selected_text": "new PackageFinder($this->getConfig())",
"error_level": 2,
- "shortcode": 98,
- "taint_trace": null
+ "shortcode": 98
},
{
"severity": "info",
@@ -4343,8 +4010,7 @@
"snippet": " return $this->getAllModules();",
"selected_text": "getAllModules",
"error_level": 2,
- "shortcode": 1,
- "taint_trace": null
+ "shortcode": 1
},
{
"severity": "info",
@@ -4356,8 +4022,7 @@
"snippet": " return $this->getFactory()->createDependencyViolationChecker()->getDependencyViolations();",
"selected_text": "createDependencyViolationChecker",
"error_level": 2,
- "shortcode": 1,
- "taint_trace": null
+ "shortcode": 1
},
{
"severity": "info",
@@ -4369,8 +4034,7 @@
"snippet": "class ModuleFinder implements ModuleFinderInterface",
"selected_text": "ModuleFinder",
"error_level": 2,
- "shortcode": 152,
- "taint_trace": null
+ "shortcode": 152
},
{
"severity": "info",
@@ -4382,8 +4046,7 @@
"snippet": " * @var \\Spryker\\Zed\\Development\\Business\\Module\\ModuleMatcher\\ModuleMatcherInterface",
"selected_text": "\\Spryker\\Zed\\Development\\Business\\Module\\ModuleMatcher\\ModuleMatcherInterface",
"error_level": 2,
- "shortcode": 98,
- "taint_trace": null
+ "shortcode": 98
},
{
"severity": "info",
@@ -4395,8 +4058,7 @@
"snippet": " * @param \\Spryker\\Zed\\Development\\Business\\Module\\ModuleMatcher\\ModuleMatcherInterface $moduleMatcher",
"selected_text": "\\Spryker\\Zed\\Development\\Business\\Module\\ModuleMatcher\\ModuleMatcherInterface",
"error_level": 2,
- "shortcode": 98,
- "taint_trace": null
+ "shortcode": 98
},
{
"severity": "info",
@@ -4408,8 +4070,7 @@
"snippet": "class ModuleMatcher implements ModuleMatcherInterface",
"selected_text": "ModuleMatcher",
"error_level": 2,
- "shortcode": 152,
- "taint_trace": null
+ "shortcode": 152
},
{
"severity": "info",
@@ -4421,8 +4082,7 @@
"snippet": "class ProjectModuleFinder implements ProjectModuleFinderInterface",
"selected_text": "ProjectModuleFinder",
"error_level": 2,
- "shortcode": 152,
- "taint_trace": null
+ "shortcode": 152
},
{
"severity": "info",
@@ -4434,8 +4094,7 @@
"snippet": " * @var \\Spryker\\Zed\\Development\\Business\\Module\\ModuleMatcher\\ModuleMatcherInterface",
"selected_text": "\\Spryker\\Zed\\Development\\Business\\Module\\ModuleMatcher\\ModuleMatcherInterface",
"error_level": 2,
- "shortcode": 98,
- "taint_trace": null
+ "shortcode": 98
},
{
"severity": "info",
@@ -4447,8 +4106,7 @@
"snippet": " * @param \\Spryker\\Zed\\Development\\Business\\Module\\ModuleMatcher\\ModuleMatcherInterface $moduleMatcher",
"selected_text": "\\Spryker\\Zed\\Development\\Business\\Module\\ModuleMatcher\\ModuleMatcherInterface",
"error_level": 2,
- "shortcode": 98,
- "taint_trace": null
+ "shortcode": 98
},
{
"severity": "info",
@@ -4460,8 +4118,7 @@
"snippet": "class PackageFinder implements PackageFinderInterface",
"selected_text": "PackageFinder",
"error_level": 2,
- "shortcode": 152,
- "taint_trace": null
+ "shortcode": 152
},
{
"severity": "info",
@@ -4473,8 +4130,7 @@
"snippet": " if (in_array($directoryInfo->getFilename(), $this->config->getInternalPackageDirectories())) {",
"selected_text": "getInternalPackageDirectories",
"error_level": 2,
- "shortcode": 1,
- "taint_trace": null
+ "shortcode": 1
},
{
"severity": "info",
@@ -4486,8 +4142,7 @@
"snippet": " return $this->config->getPathToCore();",
"selected_text": "getPathToCore",
"error_level": 2,
- "shortcode": 1,
- "taint_trace": null
+ "shortcode": 1
},
{
"severity": "info",
@@ -4499,8 +4154,7 @@
"snippet": " return $this->config->getPathToCore() . $bundle . DIRECTORY_SEPARATOR;",
"selected_text": "getPathToCore",
"error_level": 2,
- "shortcode": 1,
- "taint_trace": null
+ "shortcode": 1
},
{
"severity": "info",
@@ -4512,8 +4166,7 @@
"snippet": " $this->getFacade()->createModule($module, $options);",
"selected_text": "createModule",
"error_level": 2,
- "shortcode": 1,
- "taint_trace": null
+ "shortcode": 1
},
{
"severity": "info",
@@ -4525,8 +4178,7 @@
"snippet": " $this->getPathToCore() . '%1$s/src/Spryker/Zed/%1$s/Presentation/',",
"selected_text": "getPathToCore",
"error_level": 2,
- "shortcode": 1,
- "taint_trace": null
+ "shortcode": 1
},
{
"severity": "info",
@@ -4538,8 +4190,7 @@
"snippet": " $this->getPathToCore() . '%1$s/src/Spryker/Yves/%1$s/Theme/',",
"selected_text": "getPathToCore",
"error_level": 2,
- "shortcode": 1,
- "taint_trace": null
+ "shortcode": 1
},
{
"severity": "info",
@@ -4551,8 +4202,7 @@
"snippet": " $this->getPathToShop() . '%1$s/src/SprykerShop/Yves/%1$s/Theme/',",
"selected_text": "getPathToShop",
"error_level": 2,
- "shortcode": 1,
- "taint_trace": null
+ "shortcode": 1
},
{
"severity": "info",
@@ -4564,8 +4214,7 @@
"snippet": " $path = $this->getConfig()->get(KernelConstants::SPRYKER_ROOT);",
"selected_text": "KernelConstants::SPRYKER_ROOT",
"error_level": 2,
- "shortcode": 170,
- "taint_trace": null
+ "shortcode": 170
},
{
"severity": "info",
@@ -4577,8 +4226,7 @@
"snippet": " 'Spryker' => $this->getPathToCore(),",
"selected_text": "getPathToCore",
"error_level": 2,
- "shortcode": 1,
- "taint_trace": null
+ "shortcode": 1
},
{
"severity": "info",
@@ -4590,21 +4238,19 @@
"snippet": " 'SprykerEco' => $this->getPathToEco(),",
"selected_text": "getPathToEco",
"error_level": 2,
- "shortcode": 1,
- "taint_trace": null
+ "shortcode": 1
},
{
"severity": "info",
- "line_from": 634,
- "line_to": 634,
+ "line_from": 687,
+ "line_to": 687,
"type": "DeprecatedConstant",
"message": "Constant Spryker\\Shared\\Kernel\\KernelConstants::SPRYKER_ROOT is deprecated",
"file_name": "vendor/spryker/spryker/Bundles/Development/src/Spryker/Zed/Development/DevelopmentConfig.php",
"snippet": " $path = $this->getConfig()->get(KernelConstants::SPRYKER_ROOT);",
"selected_text": "KernelConstants::SPRYKER_ROOT",
"error_level": 2,
- "shortcode": 170,
- "taint_trace": null
+ "shortcode": 170
}
]
-}
\ No newline at end of file
+}
From ab0f9b0586da39ae36bc04939e10989c64c0a688 Mon Sep 17 00:00:00 2001
From: mscherer
Date: Tue, 27 Oct 2020 11:00:56 +0100
Subject: [PATCH 183/336] TE-7661 Upgrade to PHP 7.3+
---
.travis.yml | 4 ++--
composer.json | 2 +-
.../Business/CodeBuilder/Module/Templates/.travis.yml.tpl | 2 +-
3 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/.travis.yml b/.travis.yml
index 184b3e68..dfc12b55 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -1,14 +1,14 @@
language: php
php:
- - 7.2
+ - 7.3
- 7.4
matrix:
fast_finish: true
include:
- - php: 7.2
+ - php: 7.3
env: PREFER_LOWEST=1
env:
diff --git a/composer.json b/composer.json
index 941e44e3..caaa42a3 100644
--- a/composer.json
+++ b/composer.json
@@ -5,7 +5,7 @@
"license": "proprietary",
"require": {
"nette/di": "^2.4.7 || ^3.0",
- "php": ">=7.2",
+ "php": ">=7.3",
"phpmd/phpmd": "^2.0.0",
"spryker/config": "^3.0.0",
"spryker/graph": "^3.0.0",
diff --git a/src/Spryker/Zed/Development/Business/CodeBuilder/Module/Templates/.travis.yml.tpl b/src/Spryker/Zed/Development/Business/CodeBuilder/Module/Templates/.travis.yml.tpl
index b1e29ef8..b3bb9e70 100644
--- a/src/Spryker/Zed/Development/Business/CodeBuilder/Module/Templates/.travis.yml.tpl
+++ b/src/Spryker/Zed/Development/Business/CodeBuilder/Module/Templates/.travis.yml.tpl
@@ -1,7 +1,7 @@
language: php
php:
- - 7.2
+ - 7.3
- 7.4
cache:
From e88d25037d3b8c53c7674a67571653a10fc53fa9 Mon Sep 17 00:00:00 2001
From: mscherer
Date: Tue, 27 Oct 2020 12:32:54 +0100
Subject: [PATCH 184/336] TE-7661 Upgrade to PHP 7.3+
---
README.md | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/README.md b/README.md
index aac9f960..386982eb 100644
--- a/README.md
+++ b/README.md
@@ -1,6 +1,6 @@
# Development Module
[](https://travis-ci.org/spryker/development)
-[](https://php.net/)
+[](https://php.net/)
Development is a tooling module for (local) development.
From 108406092a8660402d88e6da2639ae4577dbe47e Mon Sep 17 00:00:00 2001
From: AntonKhabiuk
Date: Mon, 2 Nov 2020 15:28:57 +0200
Subject: [PATCH 185/336] TE-7664 Add spryker/laminas module and replace usage
of spryker/zed with it.
---
composer.json | 4 ++--
.../Business/ArchitectureSniffer/AllBundleFinder.php | 6 +++---
.../Business/ArchitectureSniffer/AllModuleFinder.php | 6 +++---
.../ArchitectureSniffer/ArchitectureSniffer.php | 6 +++---
.../Business/CodeBuilder/Bridge/BridgeBuilder.php | 6 +++---
.../Business/CodeBuilder/Module/ModuleBuilder.php | 6 +++---
.../Business/CodeStyleSniffer/CodeStyleSniffer.php | 6 +++---
.../Business/Composer/ComposerJsonUpdater.php | 2 +-
.../Business/Composer/Updater/AutoloadUpdater.php | 4 ++--
.../Business/Composer/Updater/DescriptionUpdater.php | 6 +++---
.../Composer/Updater/RequireExternalUpdater.php | 4 ++--
.../DependencyContainer/DependencyContainer.php | 2 +-
.../DependencyFinder/AbstractFileDependencyFinder.php | 10 +++++-----
.../DependencyFinder/BehaviorDependencyFinder.php | 8 ++++----
.../DependencyFinder/ExtensionDependencyFinder.php | 2 +-
.../DependencyFinder/ExternalDependencyFinder.php | 2 +-
.../Zed/Development/Business/Dependency/Manager.php | 4 ++--
.../Business/Dependency/ModuleDependencyParser.php | 6 +++---
.../DependencyTree/ComposerDependencyParser.php | 6 +++---
.../DependencyFinder/ExternalDependency.php | 2 +-
.../Business/DependencyTree/FileInfoExtractor.php | 2 +-
.../Finder/PathBuilder/AbstractPathBuilder.php | 6 +++---
.../Business/DevelopmentBusinessFactory.php | 8 ++++----
.../Business/Module/ModuleFinder/ModuleFinder.php | 8 ++++----
.../Business/Package/PackageFinder/PackageFinder.php | 4 ++--
.../Zed/Development/Business/PhpMd/PhpMdRunner.php | 2 +-
.../Zed/Development/Business/Phpstan/PhpstanRunner.php | 6 +++---
.../Business/Propel/PropelAbstractClassValidator.php | 4 ++--
.../Console/CodeArchitectureSnifferConsole.php | 8 ++++----
.../Communication/Console/CodeStyleSnifferConsole.php | 2 +-
src/Spryker/Zed/Development/DevelopmentConfig.php | 4 ++++
31 files changed, 78 insertions(+), 74 deletions(-)
diff --git a/composer.json b/composer.json
index caaa42a3..a9a24565 100644
--- a/composer.json
+++ b/composer.json
@@ -10,10 +10,10 @@
"spryker/config": "^3.0.0",
"spryker/graph": "^3.0.0",
"spryker/kernel": "^3.52.0",
+ "spryker/laminas": "^1.0.0",
"spryker/module-finder": "^1.0.0",
"spryker/symfony": "^3.0.0",
- "spryker/twig": "^3.0.0",
- "spryker/zend": "^2.0.0"
+ "spryker/twig": "^3.0.0"
},
"require-dev": {
"spryker/code-sniffer": "*",
diff --git a/src/Spryker/Zed/Development/Business/ArchitectureSniffer/AllBundleFinder.php b/src/Spryker/Zed/Development/Business/ArchitectureSniffer/AllBundleFinder.php
index 07540ee1..5f21b79f 100644
--- a/src/Spryker/Zed/Development/Business/ArchitectureSniffer/AllBundleFinder.php
+++ b/src/Spryker/Zed/Development/Business/ArchitectureSniffer/AllBundleFinder.php
@@ -8,8 +8,8 @@
namespace Spryker\Zed\Development\Business\ArchitectureSniffer;
use InvalidArgumentException;
+use Laminas\Filter\FilterInterface;
use Symfony\Component\Finder\Finder;
-use Zend\Filter\FilterInterface;
/**
* @deprecated Use {@link AllModuleFinder} instead.
@@ -22,7 +22,7 @@ class AllBundleFinder implements AllBundleFinderInterface
protected $finder;
/**
- * @var \Zend\Filter\FilterInterface
+ * @var \Laminas\Filter\FilterInterface
*/
protected $filter;
@@ -38,7 +38,7 @@ class AllBundleFinder implements AllBundleFinderInterface
/**
* @param \Symfony\Component\Finder\Finder $finder
- * @param \Zend\Filter\FilterInterface $filter
+ * @param \Laminas\Filter\FilterInterface $filter
* @param array $projectNamespaces
* @param array $coreNamespaces
*/
diff --git a/src/Spryker/Zed/Development/Business/ArchitectureSniffer/AllModuleFinder.php b/src/Spryker/Zed/Development/Business/ArchitectureSniffer/AllModuleFinder.php
index c4a8cbd0..85e6902c 100644
--- a/src/Spryker/Zed/Development/Business/ArchitectureSniffer/AllModuleFinder.php
+++ b/src/Spryker/Zed/Development/Business/ArchitectureSniffer/AllModuleFinder.php
@@ -8,9 +8,9 @@
namespace Spryker\Zed\Development\Business\ArchitectureSniffer;
use InvalidArgumentException;
+use Laminas\Filter\FilterInterface;
use Spryker\Zed\Development\DevelopmentConfig;
use Symfony\Component\Finder\Finder;
-use Zend\Filter\FilterInterface;
class AllModuleFinder implements AllModuleFinderInterface
{
@@ -25,14 +25,14 @@ class AllModuleFinder implements AllModuleFinderInterface
protected $developmentConfig;
/**
- * @var \Zend\Filter\FilterInterface
+ * @var \Laminas\Filter\FilterInterface
*/
protected $filter;
/**
* @param \Symfony\Component\Finder\Finder $finder
* @param \Spryker\Zed\Development\DevelopmentConfig $developmentConfig
- * @param \Zend\Filter\FilterInterface $filter
+ * @param \Laminas\Filter\FilterInterface $filter
*/
public function __construct(Finder $finder, DevelopmentConfig $developmentConfig, FilterInterface $filter)
{
diff --git a/src/Spryker/Zed/Development/Business/ArchitectureSniffer/ArchitectureSniffer.php b/src/Spryker/Zed/Development/Business/ArchitectureSniffer/ArchitectureSniffer.php
index e961fdcb..7ee0321a 100644
--- a/src/Spryker/Zed/Development/Business/ArchitectureSniffer/ArchitectureSniffer.php
+++ b/src/Spryker/Zed/Development/Business/ArchitectureSniffer/ArchitectureSniffer.php
@@ -8,12 +8,12 @@
namespace Spryker\Zed\Development\Business\ArchitectureSniffer;
use Exception;
+use Laminas\Config\Reader\ReaderInterface;
use PHPMD\RuleSetFactory;
use PHPMD\TextUI\CommandLineOptions;
use Spryker\Zed\Development\Business\SnifferConfiguration\Builder\SnifferConfigurationBuilderInterface;
use Spryker\Zed\Development\DevelopmentConfig;
use Symfony\Component\Process\Process;
-use Zend\Config\Reader\ReaderInterface;
class ArchitectureSniffer implements ArchitectureSnifferInterface
{
@@ -31,7 +31,7 @@ class ArchitectureSniffer implements ArchitectureSnifferInterface
protected $command;
/**
- * @var \Zend\Config\Reader\ReaderInterface
+ * @var \Laminas\Config\Reader\ReaderInterface
*/
protected $xmlReader;
@@ -41,7 +41,7 @@ class ArchitectureSniffer implements ArchitectureSnifferInterface
protected $configurationBuilder;
/**
- * @param \Zend\Config\Reader\ReaderInterface $xmlReader
+ * @param \Laminas\Config\Reader\ReaderInterface $xmlReader
* @param string $command
* @param \Spryker\Zed\Development\Business\SnifferConfiguration\Builder\SnifferConfigurationBuilderInterface $configurationBuilder
*/
diff --git a/src/Spryker/Zed/Development/Business/CodeBuilder/Bridge/BridgeBuilder.php b/src/Spryker/Zed/Development/Business/CodeBuilder/Bridge/BridgeBuilder.php
index e0be68d3..f95da885 100644
--- a/src/Spryker/Zed/Development/Business/CodeBuilder/Bridge/BridgeBuilder.php
+++ b/src/Spryker/Zed/Development/Business/CodeBuilder/Bridge/BridgeBuilder.php
@@ -8,14 +8,14 @@
namespace Spryker\Zed\Development\Business\CodeBuilder\Bridge;
use Generated\Shared\Transfer\BridgeBuilderDataTransfer;
+use Laminas\Filter\FilterChain;
+use Laminas\Filter\Word\CamelCaseToDash;
+use Laminas\Filter\Word\UnderscoreToCamelCase;
use ReflectionClass;
use ReflectionMethod;
use Spryker\Zed\Development\DevelopmentConfig;
use Symfony\Component\Console\Exception\InvalidArgumentException;
use Symfony\Component\Filesystem\Filesystem;
-use Zend\Filter\FilterChain;
-use Zend\Filter\Word\CamelCaseToDash;
-use Zend\Filter\Word\UnderscoreToCamelCase;
class BridgeBuilder
{
diff --git a/src/Spryker/Zed/Development/Business/CodeBuilder/Module/ModuleBuilder.php b/src/Spryker/Zed/Development/Business/CodeBuilder/Module/ModuleBuilder.php
index 9cad16a0..dfc54483 100644
--- a/src/Spryker/Zed/Development/Business/CodeBuilder/Module/ModuleBuilder.php
+++ b/src/Spryker/Zed/Development/Business/CodeBuilder/Module/ModuleBuilder.php
@@ -7,10 +7,10 @@
namespace Spryker\Zed\Development\Business\CodeBuilder\Module;
+use Laminas\Filter\FilterChain;
+use Laminas\Filter\Word\CamelCaseToDash;
use Spryker\Zed\Development\DevelopmentConfig;
use Symfony\Component\Filesystem\Filesystem;
-use Zend\Filter\FilterChain;
-use Zend\Filter\Word\CamelCaseToDash;
class ModuleBuilder
{
@@ -80,7 +80,7 @@ public function build($module, array $options)
}
/**
- * @return \Zend\Filter\FilterChain
+ * @return \Laminas\Filter\FilterChain
*/
protected function getUnderscoreToCamelCaseFilter()
{
diff --git a/src/Spryker/Zed/Development/Business/CodeStyleSniffer/CodeStyleSniffer.php b/src/Spryker/Zed/Development/Business/CodeStyleSniffer/CodeStyleSniffer.php
index 18a22124..abddedfc 100644
--- a/src/Spryker/Zed/Development/Business/CodeStyleSniffer/CodeStyleSniffer.php
+++ b/src/Spryker/Zed/Development/Business/CodeStyleSniffer/CodeStyleSniffer.php
@@ -7,15 +7,15 @@
namespace Spryker\Zed\Development\Business\CodeStyleSniffer;
+use Laminas\Filter\FilterChain;
+use Laminas\Filter\StringToLower;
+use Laminas\Filter\Word\CamelCaseToDash;
use RuntimeException;
use Spryker\Zed\Development\Business\CodeStyleSniffer\Config\CodeStyleSnifferConfigurationInterface;
use Spryker\Zed\Development\Business\CodeStyleSniffer\Config\CodeStyleSnifferConfigurationLoaderInterface;
use Spryker\Zed\Development\Business\Exception\CodeStyleSniffer\PathDoesNotExistException;
use Spryker\Zed\Development\DevelopmentConfig;
use Symfony\Component\Process\Process;
-use Zend\Filter\FilterChain;
-use Zend\Filter\StringToLower;
-use Zend\Filter\Word\CamelCaseToDash;
class CodeStyleSniffer
{
diff --git a/src/Spryker/Zed/Development/Business/Composer/ComposerJsonUpdater.php b/src/Spryker/Zed/Development/Business/Composer/ComposerJsonUpdater.php
index 6b86db6f..7f17cd16 100644
--- a/src/Spryker/Zed/Development/Business/Composer/ComposerJsonUpdater.php
+++ b/src/Spryker/Zed/Development/Business/Composer/ComposerJsonUpdater.php
@@ -7,11 +7,11 @@
namespace Spryker\Zed\Development\Business\Composer;
+use Laminas\Filter\Word\CamelCaseToDash;
use RuntimeException;
use Spryker\Zed\Development\Business\Composer\Updater\UpdaterInterface;
use Spryker\Zed\Development\Business\Exception\DependencyTree\InvalidComposerJsonException;
use Symfony\Component\Finder\SplFileInfo;
-use Zend\Filter\Word\CamelCaseToDash;
class ComposerJsonUpdater implements ComposerJsonUpdaterInterface
{
diff --git a/src/Spryker/Zed/Development/Business/Composer/Updater/AutoloadUpdater.php b/src/Spryker/Zed/Development/Business/Composer/Updater/AutoloadUpdater.php
index 5c833ea4..08770f38 100644
--- a/src/Spryker/Zed/Development/Business/Composer/Updater/AutoloadUpdater.php
+++ b/src/Spryker/Zed/Development/Business/Composer/Updater/AutoloadUpdater.php
@@ -7,10 +7,10 @@
namespace Spryker\Zed\Development\Business\Composer\Updater;
+use Laminas\Filter\FilterChain;
+use Laminas\Filter\Word\DashToCamelCase;
use Symfony\Component\Finder\Finder;
use Symfony\Component\Finder\SplFileInfo;
-use Zend\Filter\FilterChain;
-use Zend\Filter\Word\DashToCamelCase;
class AutoloadUpdater implements UpdaterInterface
{
diff --git a/src/Spryker/Zed/Development/Business/Composer/Updater/DescriptionUpdater.php b/src/Spryker/Zed/Development/Business/Composer/Updater/DescriptionUpdater.php
index 9ba13b5b..2afa1fab 100644
--- a/src/Spryker/Zed/Development/Business/Composer/Updater/DescriptionUpdater.php
+++ b/src/Spryker/Zed/Development/Business/Composer/Updater/DescriptionUpdater.php
@@ -7,10 +7,10 @@
namespace Spryker\Zed\Development\Business\Composer\Updater;
+use Laminas\Filter\FilterChain;
+use Laminas\Filter\Word\DashToCamelCase;
+use Laminas\Filter\Word\UnderscoreToCamelCase;
use Symfony\Component\Finder\SplFileInfo;
-use Zend\Filter\FilterChain;
-use Zend\Filter\Word\DashToCamelCase;
-use Zend\Filter\Word\UnderscoreToCamelCase;
class DescriptionUpdater implements UpdaterInterface
{
diff --git a/src/Spryker/Zed/Development/Business/Composer/Updater/RequireExternalUpdater.php b/src/Spryker/Zed/Development/Business/Composer/Updater/RequireExternalUpdater.php
index fb6f33d2..51e3203f 100644
--- a/src/Spryker/Zed/Development/Business/Composer/Updater/RequireExternalUpdater.php
+++ b/src/Spryker/Zed/Development/Business/Composer/Updater/RequireExternalUpdater.php
@@ -7,12 +7,12 @@
namespace Spryker\Zed\Development\Business\Composer\Updater;
+use Laminas\Filter\Word\CamelCaseToDash;
+use Laminas\Filter\Word\DashToCamelCase;
use Spryker\Shared\Config\Config;
use Spryker\Shared\Development\DevelopmentConstants;
use Spryker\Zed\Development\Business\DependencyTree\DependencyTree;
use Symfony\Component\Finder\SplFileInfo;
-use Zend\Filter\Word\CamelCaseToDash;
-use Zend\Filter\Word\DashToCamelCase;
class RequireExternalUpdater implements UpdaterInterface
{
diff --git a/src/Spryker/Zed/Development/Business/Dependency/DependencyContainer/DependencyContainer.php b/src/Spryker/Zed/Development/Business/Dependency/DependencyContainer/DependencyContainer.php
index f177f10d..83c4c9d0 100644
--- a/src/Spryker/Zed/Development/Business/Dependency/DependencyContainer/DependencyContainer.php
+++ b/src/Spryker/Zed/Development/Business/Dependency/DependencyContainer/DependencyContainer.php
@@ -12,7 +12,7 @@
use Generated\Shared\Transfer\DependencyModuleTransfer;
use Generated\Shared\Transfer\DependencyTransfer;
use Generated\Shared\Transfer\ModuleTransfer;
-use Zend\Filter\Word\DashToCamelCase;
+use Laminas\Filter\Word\DashToCamelCase;
class DependencyContainer implements DependencyContainerInterface
{
diff --git a/src/Spryker/Zed/Development/Business/Dependency/DependencyFinder/AbstractFileDependencyFinder.php b/src/Spryker/Zed/Development/Business/Dependency/DependencyFinder/AbstractFileDependencyFinder.php
index e94d7d4c..f01afc46 100644
--- a/src/Spryker/Zed/Development/Business/Dependency/DependencyFinder/AbstractFileDependencyFinder.php
+++ b/src/Spryker/Zed/Development/Business/Dependency/DependencyFinder/AbstractFileDependencyFinder.php
@@ -7,14 +7,14 @@
namespace Spryker\Zed\Development\Business\Dependency\DependencyFinder;
-use Zend\Filter\FilterChain;
-use Zend\Filter\StringToLower;
-use Zend\Filter\Word\CamelCaseToDash;
+use Laminas\Filter\FilterChain;
+use Laminas\Filter\StringToLower;
+use Laminas\Filter\Word\CamelCaseToDash;
abstract class AbstractFileDependencyFinder implements DependencyFinderInterface
{
/**
- * @var \Zend\Filter\FilterChain|null
+ * @var \Laminas\Filter\FilterChain|null
*/
protected $filter;
@@ -63,7 +63,7 @@ protected function buildComposerName(string $organizationName, string $moduleNam
}
/**
- * @return \Zend\Filter\FilterChain
+ * @return \Laminas\Filter\FilterChain
*/
protected function getFilter(): FilterChain
{
diff --git a/src/Spryker/Zed/Development/Business/Dependency/DependencyFinder/BehaviorDependencyFinder.php b/src/Spryker/Zed/Development/Business/Dependency/DependencyFinder/BehaviorDependencyFinder.php
index 7c35e567..c2cd4aed 100644
--- a/src/Spryker/Zed/Development/Business/Dependency/DependencyFinder/BehaviorDependencyFinder.php
+++ b/src/Spryker/Zed/Development/Business/Dependency/DependencyFinder/BehaviorDependencyFinder.php
@@ -7,18 +7,18 @@
namespace Spryker\Zed\Development\Business\Dependency\DependencyFinder;
+use Laminas\Filter\FilterChain;
+use Laminas\Filter\Word\DashToCamelCase;
use Spryker\Zed\Development\Business\Dependency\DependencyContainer\DependencyContainerInterface;
use Spryker\Zed\Development\Business\Dependency\DependencyFinder\Context\DependencyFinderContextInterface;
use Spryker\Zed\Development\Dependency\Facade\DevelopmentToModuleFinderFacadeInterface;
-use Zend\Filter\FilterChain;
-use Zend\Filter\Word\DashToCamelCase;
class BehaviorDependencyFinder implements DependencyFinderInterface
{
public const TYPE_PERSISTENCE = 'persistence';
/**
- * @var \Zend\Filter\FilterChain|null
+ * @var \Laminas\Filter\FilterChain|null
*/
protected $filter;
@@ -94,7 +94,7 @@ protected function getModuleNameFromMatch(array $match): string
}
/**
- * @return \Zend\Filter\FilterChain
+ * @return \Laminas\Filter\FilterChain
*/
protected function getFilter(): FilterChain
{
diff --git a/src/Spryker/Zed/Development/Business/Dependency/DependencyFinder/ExtensionDependencyFinder.php b/src/Spryker/Zed/Development/Business/Dependency/DependencyFinder/ExtensionDependencyFinder.php
index 935787b3..ec826554 100644
--- a/src/Spryker/Zed/Development/Business/Dependency/DependencyFinder/ExtensionDependencyFinder.php
+++ b/src/Spryker/Zed/Development/Business/Dependency/DependencyFinder/ExtensionDependencyFinder.php
@@ -26,7 +26,7 @@ class ExtensionDependencyFinder extends AbstractFileDependencyFinder
protected $moduleFinderFacade;
/**
- * @var \Zend\Filter\FilterChain|null
+ * @var \Laminas\Filter\FilterChain|null
*/
protected $filter;
diff --git a/src/Spryker/Zed/Development/Business/Dependency/DependencyFinder/ExternalDependencyFinder.php b/src/Spryker/Zed/Development/Business/Dependency/DependencyFinder/ExternalDependencyFinder.php
index 3105b35c..cb2fd2fc 100644
--- a/src/Spryker/Zed/Development/Business/Dependency/DependencyFinder/ExternalDependencyFinder.php
+++ b/src/Spryker/Zed/Development/Business/Dependency/DependencyFinder/ExternalDependencyFinder.php
@@ -8,12 +8,12 @@
namespace Spryker\Zed\Development\Business\Dependency\DependencyFinder;
use Exception;
+use Laminas\Filter\Word\SeparatorToCamelCase;
use Spryker\Zed\Development\Business\Dependency\ComposerParser\ExternalDependencyParserInterface;
use Spryker\Zed\Development\Business\Dependency\DependencyContainer\DependencyContainerInterface;
use Spryker\Zed\Development\Business\Dependency\DependencyFinder\Context\DependencyFinderContextInterface;
use Spryker\Zed\Development\Business\Dependency\ModuleParser\UseStatementParserInterface;
use Spryker\Zed\Development\DevelopmentConfig;
-use Zend\Filter\Word\SeparatorToCamelCase;
class ExternalDependencyFinder extends AbstractFileDependencyFinder
{
diff --git a/src/Spryker/Zed/Development/Business/Dependency/Manager.php b/src/Spryker/Zed/Development/Business/Dependency/Manager.php
index 78efe4dc..5cac8e36 100644
--- a/src/Spryker/Zed/Development/Business/Dependency/Manager.php
+++ b/src/Spryker/Zed/Development/Business/Dependency/Manager.php
@@ -12,11 +12,11 @@
use Generated\Shared\Transfer\DependencyModuleViewTransfer;
use Generated\Shared\Transfer\ModuleTransfer;
use Generated\Shared\Transfer\OrganizationTransfer;
+use Laminas\Filter\FilterChain;
+use Laminas\Filter\Word\DashToCamelCase;
use Spryker\Zed\Development\Business\Dependency\Mapper\DependencyModuleMapperInterface;
use Spryker\Zed\Development\DevelopmentConfig;
use Symfony\Component\Finder\Finder;
-use Zend\Filter\FilterChain;
-use Zend\Filter\Word\DashToCamelCase;
class Manager implements ManagerInterface
{
diff --git a/src/Spryker/Zed/Development/Business/Dependency/ModuleDependencyParser.php b/src/Spryker/Zed/Development/Business/Dependency/ModuleDependencyParser.php
index 9e515254..5c242700 100644
--- a/src/Spryker/Zed/Development/Business/Dependency/ModuleDependencyParser.php
+++ b/src/Spryker/Zed/Development/Business/Dependency/ModuleDependencyParser.php
@@ -9,13 +9,13 @@
use Generated\Shared\Transfer\DependencyCollectionTransfer;
use Generated\Shared\Transfer\ModuleTransfer;
+use Laminas\Filter\FilterChain;
+use Laminas\Filter\StringToLower;
+use Laminas\Filter\Word\CamelCaseToDash;
use Spryker\Zed\Development\Business\Dependency\DependencyContainer\DependencyContainerInterface;
use Spryker\Zed\Development\Business\Dependency\DependencyFinder\Context\DependencyFinderContext;
use Spryker\Zed\Development\Business\Dependency\DependencyFinder\DependencyFinderInterface;
use Spryker\Zed\Development\Business\Module\ModuleFileFinder\ModuleFileFinderInterface;
-use Zend\Filter\FilterChain;
-use Zend\Filter\StringToLower;
-use Zend\Filter\Word\CamelCaseToDash;
class ModuleDependencyParser implements ModuleDependencyParserInterface
{
diff --git a/src/Spryker/Zed/Development/Business/DependencyTree/ComposerDependencyParser.php b/src/Spryker/Zed/Development/Business/DependencyTree/ComposerDependencyParser.php
index c4d433e5..dc306ab7 100644
--- a/src/Spryker/Zed/Development/Business/DependencyTree/ComposerDependencyParser.php
+++ b/src/Spryker/Zed/Development/Business/DependencyTree/ComposerDependencyParser.php
@@ -14,12 +14,12 @@
use Generated\Shared\Transfer\DependencyModuleTransfer;
use Generated\Shared\Transfer\DependencyTransfer;
use Generated\Shared\Transfer\ModuleTransfer;
+use Laminas\Filter\FilterChain;
+use Laminas\Filter\Word\DashToCamelCase;
+use Laminas\Filter\Word\SeparatorToCamelCase;
use Spryker\Zed\Development\Business\Composer\ComposerNameFinderInterface;
use Spryker\Zed\Development\Business\Exception\DependencyTree\InvalidComposerJsonException;
use Symfony\Component\Finder\SplFileInfo;
-use Zend\Filter\FilterChain;
-use Zend\Filter\Word\DashToCamelCase;
-use Zend\Filter\Word\SeparatorToCamelCase;
class ComposerDependencyParser implements ComposerDependencyParserInterface
{
diff --git a/src/Spryker/Zed/Development/Business/DependencyTree/DependencyFinder/ExternalDependency.php b/src/Spryker/Zed/Development/Business/DependencyTree/DependencyFinder/ExternalDependency.php
index 76f05f49..c65ae7a0 100644
--- a/src/Spryker/Zed/Development/Business/DependencyTree/DependencyFinder/ExternalDependency.php
+++ b/src/Spryker/Zed/Development/Business/DependencyTree/DependencyFinder/ExternalDependency.php
@@ -7,13 +7,13 @@
namespace Spryker\Zed\Development\Business\DependencyTree\DependencyFinder;
+use Laminas\Filter\Word\SeparatorToCamelCase;
use PHP_CodeSniffer\Config;
use PHP_CodeSniffer\Files\File;
use PHP_CodeSniffer\Ruleset;
use PHP_CodeSniffer\Runner;
use Spryker\Zed\Development\Business\DependencyTree\DependencyTree;
use Symfony\Component\Finder\SplFileInfo;
-use Zend\Filter\Word\SeparatorToCamelCase;
$manualAutoload = APPLICATION_VENDOR_DIR . '/squizlabs/php_codesniffer/autoload.php';
if (!class_exists(Config::class) && file_exists($manualAutoload)) {
diff --git a/src/Spryker/Zed/Development/Business/DependencyTree/FileInfoExtractor.php b/src/Spryker/Zed/Development/Business/DependencyTree/FileInfoExtractor.php
index 291e1df6..9b3e4767 100644
--- a/src/Spryker/Zed/Development/Business/DependencyTree/FileInfoExtractor.php
+++ b/src/Spryker/Zed/Development/Business/DependencyTree/FileInfoExtractor.php
@@ -8,8 +8,8 @@
namespace Spryker\Zed\Development\Business\DependencyTree;
use Exception;
+use Laminas\Filter\Word\CamelCaseToDash;
use Symfony\Component\Finder\SplFileInfo;
-use Zend\Filter\Word\CamelCaseToDash;
class FileInfoExtractor
{
diff --git a/src/Spryker/Zed/Development/Business/DependencyTree/Finder/PathBuilder/AbstractPathBuilder.php b/src/Spryker/Zed/Development/Business/DependencyTree/Finder/PathBuilder/AbstractPathBuilder.php
index bd8821b8..de2acc4c 100644
--- a/src/Spryker/Zed/Development/Business/DependencyTree/Finder/PathBuilder/AbstractPathBuilder.php
+++ b/src/Spryker/Zed/Development/Business/DependencyTree/Finder/PathBuilder/AbstractPathBuilder.php
@@ -7,10 +7,10 @@
namespace Spryker\Zed\Development\Business\DependencyTree\Finder\PathBuilder;
+use Laminas\Filter\FilterChain;
+use Laminas\Filter\StringToLower;
+use Laminas\Filter\Word\CamelCaseToDash;
use Spryker\Zed\Development\DevelopmentConfig;
-use Zend\Filter\FilterChain;
-use Zend\Filter\StringToLower;
-use Zend\Filter\Word\CamelCaseToDash;
class AbstractPathBuilder
{
diff --git a/src/Spryker/Zed/Development/Business/DevelopmentBusinessFactory.php b/src/Spryker/Zed/Development/Business/DevelopmentBusinessFactory.php
index 17d4df10..319af92a 100644
--- a/src/Spryker/Zed/Development/Business/DevelopmentBusinessFactory.php
+++ b/src/Spryker/Zed/Development/Business/DevelopmentBusinessFactory.php
@@ -7,6 +7,8 @@
namespace Spryker\Zed\Development\Business;
+use Laminas\Config\Reader\Xml;
+use Laminas\Filter\Word\CamelCaseToDash;
use Nette\DI\Config\Loader;
use Spryker\Zed\Development\Business\ArchitectureSniffer\AllBundleFinder;
use Spryker\Zed\Development\Business\ArchitectureSniffer\AllModuleFinder;
@@ -200,8 +202,6 @@
use Symfony\Component\Finder\Finder;
use Symfony\Component\Finder\Finder as SymfonyFinder;
use Symfony\Component\Yaml\Parser;
-use Zend\Config\Reader\Xml;
-use Zend\Filter\Word\CamelCaseToDash;
/**
* @method \Spryker\Zed\Development\DevelopmentConfig getConfig()
@@ -1959,7 +1959,7 @@ public function createArchitectureSniffer(): ArchitectureSnifferInterface
}
/**
- * @return \Zend\Config\Reader\ReaderInterface
+ * @return \Laminas\Config\Reader\ReaderInterface
*/
protected function createXmlReader()
{
@@ -1994,7 +1994,7 @@ public function createAllModuleFinder(): AllModuleFinderInterface
}
/**
- * @return \Zend\Filter\FilterInterface
+ * @return \Laminas\Filter\FilterInterface
*/
protected function createCamelCaseToDashFilter()
{
diff --git a/src/Spryker/Zed/Development/Business/Module/ModuleFinder/ModuleFinder.php b/src/Spryker/Zed/Development/Business/Module/ModuleFinder/ModuleFinder.php
index 9263fdf1..ecc265f5 100644
--- a/src/Spryker/Zed/Development/Business/Module/ModuleFinder/ModuleFinder.php
+++ b/src/Spryker/Zed/Development/Business/Module/ModuleFinder/ModuleFinder.php
@@ -11,14 +11,14 @@
use Generated\Shared\Transfer\ModuleFilterTransfer;
use Generated\Shared\Transfer\ModuleTransfer;
use Generated\Shared\Transfer\OrganizationTransfer;
+use Laminas\Filter\FilterChain;
+use Laminas\Filter\StringToLower;
+use Laminas\Filter\Word\CamelCaseToDash;
+use Laminas\Filter\Word\DashToCamelCase;
use Spryker\Zed\Development\Business\Module\ModuleMatcher\ModuleMatcherInterface;
use Spryker\Zed\Development\DevelopmentConfig;
use Symfony\Component\Finder\Finder;
use Symfony\Component\Finder\SplFileInfo;
-use Zend\Filter\FilterChain;
-use Zend\Filter\StringToLower;
-use Zend\Filter\Word\CamelCaseToDash;
-use Zend\Filter\Word\DashToCamelCase;
/**
* @deprecated Use `spryker/module-finder` instead.
diff --git a/src/Spryker/Zed/Development/Business/Package/PackageFinder/PackageFinder.php b/src/Spryker/Zed/Development/Business/Package/PackageFinder/PackageFinder.php
index 810a1ee0..25427a3c 100644
--- a/src/Spryker/Zed/Development/Business/Package/PackageFinder/PackageFinder.php
+++ b/src/Spryker/Zed/Development/Business/Package/PackageFinder/PackageFinder.php
@@ -8,11 +8,11 @@
namespace Spryker\Zed\Development\Business\Package\PackageFinder;
use Generated\Shared\Transfer\PackageTransfer;
+use Laminas\Filter\FilterChain;
+use Laminas\Filter\Word\DashToCamelCase;
use Spryker\Zed\Development\DevelopmentConfig;
use Symfony\Component\Finder\Finder;
use Symfony\Component\Finder\SplFileInfo;
-use Zend\Filter\FilterChain;
-use Zend\Filter\Word\DashToCamelCase;
/**
* @deprecated Use `spryker/module-finder` instead.
diff --git a/src/Spryker/Zed/Development/Business/PhpMd/PhpMdRunner.php b/src/Spryker/Zed/Development/Business/PhpMd/PhpMdRunner.php
index 56d08578..7dae8952 100644
--- a/src/Spryker/Zed/Development/Business/PhpMd/PhpMdRunner.php
+++ b/src/Spryker/Zed/Development/Business/PhpMd/PhpMdRunner.php
@@ -8,9 +8,9 @@
namespace Spryker\Zed\Development\Business\PhpMd;
use ErrorException;
+use Laminas\Filter\Word\UnderscoreToCamelCase;
use Spryker\Zed\Development\DevelopmentConfig;
use Symfony\Component\Process\Process;
-use Zend\Filter\Word\UnderscoreToCamelCase;
class PhpMdRunner
{
diff --git a/src/Spryker/Zed/Development/Business/Phpstan/PhpstanRunner.php b/src/Spryker/Zed/Development/Business/Phpstan/PhpstanRunner.php
index e1d8d125..e455be94 100644
--- a/src/Spryker/Zed/Development/Business/Phpstan/PhpstanRunner.php
+++ b/src/Spryker/Zed/Development/Business/Phpstan/PhpstanRunner.php
@@ -7,6 +7,9 @@
namespace Spryker\Zed\Development\Business\Phpstan;
+use Laminas\Filter\FilterChain;
+use Laminas\Filter\StringToLower;
+use Laminas\Filter\Word\CamelCaseToDash;
use RuntimeException;
use SplFileInfo;
use Spryker\Zed\Development\Business\Phpstan\Config\PhpstanConfigFileFinderInterface;
@@ -20,9 +23,6 @@
use Symfony\Component\Console\Output\OutputInterface;
use Symfony\Component\Finder\Finder;
use Symfony\Component\Process\Process;
-use Zend\Filter\FilterChain;
-use Zend\Filter\StringToLower;
-use Zend\Filter\Word\CamelCaseToDash;
class PhpstanRunner implements PhpstanRunnerInterface
{
diff --git a/src/Spryker/Zed/Development/Business/Propel/PropelAbstractClassValidator.php b/src/Spryker/Zed/Development/Business/Propel/PropelAbstractClassValidator.php
index e894325f..709dff51 100644
--- a/src/Spryker/Zed/Development/Business/Propel/PropelAbstractClassValidator.php
+++ b/src/Spryker/Zed/Development/Business/Propel/PropelAbstractClassValidator.php
@@ -8,12 +8,12 @@
namespace Spryker\Zed\Development\Business\Propel;
use Exception;
+use Laminas\Filter\FilterChain;
+use Laminas\Filter\Word\UnderscoreToCamelCase;
use SimpleXMLElement;
use Symfony\Component\Console\Output\OutputInterface;
use Symfony\Component\Finder\Finder;
use Symfony\Component\Finder\SplFileInfo;
-use Zend\Filter\FilterChain;
-use Zend\Filter\Word\UnderscoreToCamelCase;
class PropelAbstractClassValidator implements PropelAbstractClassValidatorInterface
{
diff --git a/src/Spryker/Zed/Development/Communication/Console/CodeArchitectureSnifferConsole.php b/src/Spryker/Zed/Development/Communication/Console/CodeArchitectureSnifferConsole.php
index 120804be..cb9a57ac 100644
--- a/src/Spryker/Zed/Development/Communication/Console/CodeArchitectureSnifferConsole.php
+++ b/src/Spryker/Zed/Development/Communication/Console/CodeArchitectureSnifferConsole.php
@@ -10,15 +10,15 @@
use Generated\Shared\Transfer\ModuleFilterTransfer;
use Generated\Shared\Transfer\ModuleTransfer;
use Generated\Shared\Transfer\OrganizationTransfer;
+use Laminas\Filter\FilterChain;
+use Laminas\Filter\StringToLower;
+use Laminas\Filter\Word\CamelCaseToDash;
+use Laminas\Filter\Word\UnderscoreToCamelCase;
use Spryker\Zed\Kernel\Communication\Console\Console;
use Symfony\Component\Console\Input\InputArgument;
use Symfony\Component\Console\Input\InputInterface;
use Symfony\Component\Console\Input\InputOption;
use Symfony\Component\Console\Output\OutputInterface;
-use Zend\Filter\FilterChain;
-use Zend\Filter\StringToLower;
-use Zend\Filter\Word\CamelCaseToDash;
-use Zend\Filter\Word\UnderscoreToCamelCase;
/**
* @method \Spryker\Zed\Development\Business\DevelopmentFacadeInterface getFacade()
diff --git a/src/Spryker/Zed/Development/Communication/Console/CodeStyleSnifferConsole.php b/src/Spryker/Zed/Development/Communication/Console/CodeStyleSnifferConsole.php
index 03f0de6b..77e627b7 100644
--- a/src/Spryker/Zed/Development/Communication/Console/CodeStyleSnifferConsole.php
+++ b/src/Spryker/Zed/Development/Communication/Console/CodeStyleSnifferConsole.php
@@ -7,12 +7,12 @@
namespace Spryker\Zed\Development\Communication\Console;
+use Laminas\Filter\Word\UnderscoreToCamelCase;
use Spryker\Zed\Kernel\Communication\Console\Console;
use Symfony\Component\Console\Input\InputArgument;
use Symfony\Component\Console\Input\InputInterface;
use Symfony\Component\Console\Input\InputOption;
use Symfony\Component\Console\Output\OutputInterface;
-use Zend\Filter\Word\UnderscoreToCamelCase;
/**
* @method \Spryker\Zed\Development\Business\DevelopmentFacadeInterface getFacade()
diff --git a/src/Spryker/Zed/Development/DevelopmentConfig.php b/src/Spryker/Zed/Development/DevelopmentConfig.php
index b8bfc6a0..32933412 100644
--- a/src/Spryker/Zed/Development/DevelopmentConfig.php
+++ b/src/Spryker/Zed/Development/DevelopmentConfig.php
@@ -311,6 +311,7 @@ public function getExternalToInternalNamespaceMap()
'GuzzleHttp\\' => 'spryker/guzzle',
'JsonPath\\' => 'spryker/json-path',
'JsonSchema\\' => 'spryker/json-schema',
+ 'Laminas\\' => 'spryker/laminas',
'League\\Csv\\' => 'spryker/csv',
'Monolog\\' => 'spryker/monolog',
'org\\bovigo\\vfs\\' => 'spryker/testify',
@@ -353,6 +354,9 @@ public function getExternalToInternalMap()
'galbar/jsonpath' => 'spryker/json-path',
'guzzlehttp/guzzle' => 'spryker/guzzle',
'justinrainbow/json-schema' => 'spryker/json-schema',
+ 'laminas/laminas-config' => 'spryker/laminas',
+ 'laminas/laminas-filter' => 'spryker/laminas',
+ 'laminas/laminas-servicemanager' => 'spryker/laminas',
'league/csv' => 'spryker/csv',
'league/oauth2-server' => 'spryker/oauth',
'mandrill/mandrill' => 'spryker/mandrill',
From 27943f58230e8c75e2129011cbf4d2d70de167a7 Mon Sep 17 00:00:00 2001
From: mscherer
Date: Tue, 3 Nov 2020 12:05:57 +0100
Subject: [PATCH 186/336] Fix up min constraint for PHP min.
---
.../Development/Business/Composer/Updater/RequireUpdater.php | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/Spryker/Zed/Development/Business/Composer/Updater/RequireUpdater.php b/src/Spryker/Zed/Development/Business/Composer/Updater/RequireUpdater.php
index bd62b93a..0e5ea236 100644
--- a/src/Spryker/Zed/Development/Business/Composer/Updater/RequireUpdater.php
+++ b/src/Spryker/Zed/Development/Business/Composer/Updater/RequireUpdater.php
@@ -13,7 +13,7 @@ class RequireUpdater implements UpdaterInterface
{
public const KEY_REQUIRE = 'require';
public const KEY_REQUIRE_PHP = 'php';
- public const PHP_MINIMUM = '>=7.2';
+ public const PHP_MINIMUM = '>=7.3';
/**
* @param array $composerJson
From 385deff38ba22a08a5d745a8d68672641bb26a7e Mon Sep 17 00:00:00 2001
From: mscherer