Skip to content

Commit

Permalink
Update BetterReflection to version that uses PHPStan namespace
Browse files Browse the repository at this point in the history
  • Loading branch information
ondrejmirtes committed Feb 24, 2021
1 parent 5aa8a55 commit 06ef13e
Show file tree
Hide file tree
Showing 46 changed files with 194 additions and 203 deletions.
2 changes: 1 addition & 1 deletion build/baseline-7.4.neon
Original file line number Diff line number Diff line change
Expand Up @@ -93,6 +93,6 @@ parameters:
count: 1
path: ../src/Reflection/ReflectionProvider/SetterReflectionProviderProvider.php
-
message: "#^Class class@anonymous/src/Testing/TestCase\\.php\\:268 has an uninitialized property \\$reflectionProvider\\. Give it default value or assign it in the constructor\\.$#"
message: "#^Class class@anonymous/src/Testing/TestCase\\.php\\:267 has an uninitialized property \\$reflectionProvider\\. Give it default value or assign it in the constructor\\.$#"
count: 1
path: ../src/Testing/TestCase.php
5 changes: 0 additions & 5 deletions compiler/build/scoper.inc.php
Original file line number Diff line number Diff line change
Expand Up @@ -121,10 +121,6 @@ function (string $filePath, string $prefix, string $content): string {
return $class;
}

if (strpos($class, 'Roave\\BetterReflection\\') === 0) {
return $class;
}

if (strpos($class, '@') === 0) {
return $class;
}
Expand Down Expand Up @@ -217,7 +213,6 @@ function (string $filePath, string $prefix, string $content): string {
'PHPStan\*',
'PhpParser\*',
'Hoa\*',
'Roave\BetterReflection\*',
],
'whitelist-global-functions' => false,
'whitelist-global-classes' => false,
Expand Down
2 changes: 0 additions & 2 deletions compiler/src/Console/CompileCommand.php
Original file line number Diff line number Diff line change
Expand Up @@ -201,10 +201,8 @@ private function buildPreloadScript(): void
$output = '';
foreach ($finder->files()->name('*.php')->in([
$this->buildDir . '/src',
$this->buildDir . '/stubs/runtime',
$vendorDir . '/nikic/php-parser/lib/PhpParser',
$vendorDir . '/phpstan/phpdoc-parser/src',
$vendorDir . '/ondrejmirtes/better-reflection/src',
])->exclude([
'Testing',
]) as $phpFile) {
Expand Down
2 changes: 1 addition & 1 deletion composer.json
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@
"nette/utils": "^3.1.3",
"nikic/php-parser": "4.10.4",
"ondram/ci-detector": "^3.4.0",
"ondrejmirtes/better-reflection": "4.3.50",
"ondrejmirtes/better-reflection": "4.3.51",
"phpdocumentor/reflection-docblock": "4.3.4",
"phpstan/php-8-stubs": "^0.1.12",
"phpstan/phpdoc-parser": "^0.4.9",
Expand Down
18 changes: 9 additions & 9 deletions composer.lock

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

11 changes: 5 additions & 6 deletions conf/config.neon
Original file line number Diff line number Diff line change
Expand Up @@ -98,7 +98,6 @@ parameters:
- '#^PhpParser\\#'
- '#^PHPStan\\#'
- '#^Hoa\\#'
- '#^Roave\\BetterReflection\\#'
dynamicConstantNames:
- ICONV_IMPL
- LIBXML_VERSION
Expand Down Expand Up @@ -1331,7 +1330,7 @@ services:
- PHPStan\Reflection\ReflectionProvider

betterReflectionSourceLocator:
class: Roave\BetterReflection\SourceLocator\Type\SourceLocator
class: PHPStan\BetterReflection\SourceLocator\Type\SourceLocator
factory: @PHPStan\Reflection\BetterReflection\BetterReflectionSourceLocatorFactory::create
autowired: false

Expand Down Expand Up @@ -1405,17 +1404,17 @@ services:
implement: PHPStan\Reflection\BetterReflection\BetterReflectionProviderFactory

-
class: Roave\BetterReflection\SourceLocator\SourceStubber\PhpStormStubsSourceStubber
class: PHPStan\BetterReflection\SourceLocator\SourceStubber\PhpStormStubsSourceStubber
arguments:
phpParser: @php8PhpParser
phpVersionId: %phpVersion%
autowired:
- Roave\BetterReflection\SourceLocator\SourceStubber\PhpStormStubsSourceStubber
- PHPStan\BetterReflection\SourceLocator\SourceStubber\PhpStormStubsSourceStubber

-
class: Roave\BetterReflection\SourceLocator\SourceStubber\ReflectionSourceStubber
class: PHPStan\BetterReflection\SourceLocator\SourceStubber\ReflectionSourceStubber
autowired:
- Roave\BetterReflection\SourceLocator\SourceStubber\ReflectionSourceStubber
- PHPStan\BetterReflection\SourceLocator\SourceStubber\ReflectionSourceStubber

php8Lexer:
class: PhpParser\Lexer\Emulative
Expand Down
8 changes: 4 additions & 4 deletions conf/config.stubValidator.neon
Original file line number Diff line number Diff line change
Expand Up @@ -25,27 +25,27 @@ services:
autowired: false

stubClassReflector:
class: Roave\BetterReflection\Reflector\ClassReflector
class: PHPStan\BetterReflection\Reflector\ClassReflector
arguments:
sourceLocator: @stubSourceLocator
autowired: false

stubFunctionReflector:
class: Roave\BetterReflection\Reflector\FunctionReflector
class: PHPStan\BetterReflection\Reflector\FunctionReflector
arguments:
classReflector: @stubClassReflector
sourceLocator: @stubSourceLocator
autowired: false

stubConstantReflector:
class: Roave\BetterReflection\Reflector\ConstantReflector
class: PHPStan\BetterReflection\Reflector\ConstantReflector
arguments:
classReflector: @stubClassReflector
sourceLocator: @stubSourceLocator
autowired: false

stubSourceLocator:
class: Roave\BetterReflection\SourceLocator\Type\SourceLocator
class: PHPStan\BetterReflection\SourceLocator\Type\SourceLocator
factory: @PHPStan\PhpDoc\StubSourceLocatorFactory::create()
autowired: false

Expand Down
8 changes: 4 additions & 4 deletions src/Analyser/FileAnalyser.php
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,10 @@

use PhpParser\Comment;
use PhpParser\Node;
use PHPStan\BetterReflection\NodeCompiler\Exception\UnableToCompileNode;
use PHPStan\BetterReflection\Reflection\Exception\NotAClassReflection;
use PHPStan\BetterReflection\Reflection\Exception\NotAnInterfaceReflection;
use PHPStan\BetterReflection\Reflector\Exception\IdentifierNotFound;
use PHPStan\Dependency\DependencyResolver;
use PHPStan\Node\FileNode;
use PHPStan\Parser\Parser;
Expand All @@ -14,10 +18,6 @@
use PHPStan\Rules\NonIgnorableRuleError;
use PHPStan\Rules\Registry;
use PHPStan\Rules\TipRuleError;
use Roave\BetterReflection\NodeCompiler\Exception\UnableToCompileNode;
use Roave\BetterReflection\Reflection\Exception\NotAClassReflection;
use Roave\BetterReflection\Reflection\Exception\NotAnInterfaceReflection;
use Roave\BetterReflection\Reflector\Exception\IdentifierNotFound;
use function array_key_exists;
use function array_unique;

Expand Down
10 changes: 5 additions & 5 deletions src/Analyser/NodeScopeResolver.php
Original file line number Diff line number Diff line change
Expand Up @@ -46,6 +46,10 @@
use PhpParser\Node\Stmt\TryCatch;
use PhpParser\Node\Stmt\Unset_;
use PhpParser\Node\Stmt\While_;
use PHPStan\BetterReflection\Reflection\Adapter\ReflectionClass;
use PHPStan\BetterReflection\Reflector\ClassReflector;
use PHPStan\BetterReflection\SourceLocator\Ast\Strategy\NodeToReflection;
use PHPStan\BetterReflection\SourceLocator\Located\LocatedSource;
use PHPStan\DependencyInjection\Reflection\ClassReflectionExtensionRegistryProvider;
use PHPStan\File\FileHelper;
use PHPStan\File\FileReader;
Expand Down Expand Up @@ -114,10 +118,6 @@
use PHPStan\Type\TypeTraverser;
use PHPStan\Type\TypeUtils;
use PHPStan\Type\UnionType;
use Roave\BetterReflection\Reflection\Adapter\ReflectionClass;
use Roave\BetterReflection\Reflector\ClassReflector;
use Roave\BetterReflection\SourceLocator\Ast\Strategy\NodeToReflection;
use Roave\BetterReflection\SourceLocator\Located\LocatedSource;

class NodeScopeResolver
{
Expand Down Expand Up @@ -1257,7 +1257,7 @@ private function createAstClassReflection(Node\Stmt\ClassLike $stmt, Scope $scop
new LocatedSource(FileReader::read($scope->getFile()), $scope->getFile()),
$scope->getNamespace() !== null ? new Node\Stmt\Namespace_(new Name($scope->getNamespace())) : null
);
if (!$betterReflectionClass instanceof \Roave\BetterReflection\Reflection\ReflectionClass) {
if (!$betterReflectionClass instanceof \PHPStan\BetterReflection\Reflection\ReflectionClass) {
throw new \PHPStan\ShouldNotHappenException();
}

Expand Down
4 changes: 2 additions & 2 deletions src/DependencyInjection/ContainerFactory.php
Original file line number Diff line number Diff line change
Expand Up @@ -4,12 +4,12 @@

use Nette\DI\Extensions\PhpExtension;
use Phar;
use PHPStan\BetterReflection\BetterReflection;
use PHPStan\BetterReflection\SourceLocator\SourceStubber\PhpStormStubsSourceStubber;
use PHPStan\Broker\Broker;
use PHPStan\Command\CommandHelper;
use PHPStan\File\FileHelper;
use PHPStan\Php\PhpVersion;
use Roave\BetterReflection\BetterReflection;
use Roave\BetterReflection\SourceLocator\SourceStubber\PhpStormStubsSourceStubber;
use Symfony\Component\Finder\Finder;
use function sys_get_temp_dir;

Expand Down
14 changes: 7 additions & 7 deletions src/PhpDoc/StubSourceLocatorFactory.php
Original file line number Diff line number Diff line change
Expand Up @@ -2,15 +2,15 @@

namespace PHPStan\PhpDoc;

use PHPStan\BetterReflection\Reflector\FunctionReflector;
use PHPStan\BetterReflection\SourceLocator\Ast\Locator;
use PHPStan\BetterReflection\SourceLocator\SourceStubber\PhpStormStubsSourceStubber;
use PHPStan\BetterReflection\SourceLocator\Type\AggregateSourceLocator;
use PHPStan\BetterReflection\SourceLocator\Type\MemoizingSourceLocator;
use PHPStan\BetterReflection\SourceLocator\Type\PhpInternalSourceLocator;
use PHPStan\BetterReflection\SourceLocator\Type\SourceLocator;
use PHPStan\DependencyInjection\Container;
use PHPStan\Reflection\BetterReflection\SourceLocator\OptimizedSingleFileSourceLocatorRepository;
use Roave\BetterReflection\Reflector\FunctionReflector;
use Roave\BetterReflection\SourceLocator\Ast\Locator;
use Roave\BetterReflection\SourceLocator\SourceStubber\PhpStormStubsSourceStubber;
use Roave\BetterReflection\SourceLocator\Type\AggregateSourceLocator;
use Roave\BetterReflection\SourceLocator\Type\MemoizingSourceLocator;
use Roave\BetterReflection\SourceLocator\Type\PhpInternalSourceLocator;
use Roave\BetterReflection\SourceLocator\Type\SourceLocator;

class StubSourceLocatorFactory
{
Expand Down
34 changes: 17 additions & 17 deletions src/Reflection/BetterReflection/BetterReflectionProvider.php
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,17 @@

use PhpParser\PrettyPrinter\Standard;
use PHPStan\Analyser\Scope;
use PHPStan\BetterReflection\Identifier\Exception\InvalidIdentifierName;
use PHPStan\BetterReflection\NodeCompiler\Exception\UnableToCompileNode;
use PHPStan\BetterReflection\Reflection\Adapter\ReflectionClass;
use PHPStan\BetterReflection\Reflection\Adapter\ReflectionFunction;
use PHPStan\BetterReflection\Reflection\Exception\NotAClassReflection;
use PHPStan\BetterReflection\Reflection\Exception\NotAnInterfaceReflection;
use PHPStan\BetterReflection\Reflector\ClassReflector;
use PHPStan\BetterReflection\Reflector\ConstantReflector;
use PHPStan\BetterReflection\Reflector\Exception\IdentifierNotFound;
use PHPStan\BetterReflection\Reflector\FunctionReflector;
use PHPStan\BetterReflection\SourceLocator\Located\LocatedSource;
use PHPStan\Broker\AnonymousClassNameHelper;
use PHPStan\DependencyInjection\Reflection\ClassReflectionExtensionRegistryProvider;
use PHPStan\File\FileHelper;
Expand All @@ -23,17 +34,6 @@
use PHPStan\Type\Generic\TemplateTypeMap;
use PHPStan\Type\MixedType;
use PHPStan\Type\Type;
use Roave\BetterReflection\Identifier\Exception\InvalidIdentifierName;
use Roave\BetterReflection\NodeCompiler\Exception\UnableToCompileNode;
use Roave\BetterReflection\Reflection\Adapter\ReflectionClass;
use Roave\BetterReflection\Reflection\Adapter\ReflectionFunction;
use Roave\BetterReflection\Reflection\Exception\NotAClassReflection;
use Roave\BetterReflection\Reflection\Exception\NotAnInterfaceReflection;
use Roave\BetterReflection\Reflector\ClassReflector;
use Roave\BetterReflection\Reflector\ConstantReflector;
use Roave\BetterReflection\Reflector\Exception\IdentifierNotFound;
use Roave\BetterReflection\Reflector\FunctionReflector;
use Roave\BetterReflection\SourceLocator\Located\LocatedSource;

class BetterReflectionProvider implements ReflectionProvider
{
Expand All @@ -42,11 +42,11 @@ class BetterReflectionProvider implements ReflectionProvider

private \PHPStan\DependencyInjection\Reflection\ClassReflectionExtensionRegistryProvider $classReflectionExtensionRegistryProvider;

private \Roave\BetterReflection\Reflector\ClassReflector $classReflector;
private \PHPStan\BetterReflection\Reflector\ClassReflector $classReflector;

private \Roave\BetterReflection\Reflector\FunctionReflector $functionReflector;
private \PHPStan\BetterReflection\Reflector\FunctionReflector $functionReflector;

private \Roave\BetterReflection\Reflector\ConstantReflector $constantReflector;
private \PHPStan\BetterReflection\Reflector\ConstantReflector $constantReflector;

private \PHPStan\Type\FileTypeMapper $fileTypeMapper;

Expand Down Expand Up @@ -207,7 +207,7 @@ public function getAnonymousClassReflection(\PhpParser\Node\Stmt\Class_ $classNo
return self::$anonymousClasses[$className];
}

$reflectionClass = \Roave\BetterReflection\Reflection\ReflectionClass::createFromNode(
$reflectionClass = \PHPStan\BetterReflection\Reflection\ReflectionClass::createFromNode(
$this->classReflector,
$classNode,
new LocatedSource($this->printer->prettyPrint([$classNode]), $scopeFile),
Expand Down Expand Up @@ -313,7 +313,7 @@ public function resolveFunctionName(\PhpParser\Node\Name $nameNode, ?Scope $scop
try {
$this->functionReflector->reflect($name);
return true;
} catch (\Roave\BetterReflection\Reflector\Exception\IdentifierNotFound $e) {
} catch (\PHPStan\BetterReflection\Reflector\Exception\IdentifierNotFound $e) {
// pass
} catch (InvalidIdentifierName $e) {
// pass
Expand Down Expand Up @@ -358,7 +358,7 @@ public function resolveConstantName(\PhpParser\Node\Name $nameNode, ?Scope $scop
try {
$this->constantReflector->reflect($name);
return true;
} catch (\Roave\BetterReflection\Reflector\Exception\IdentifierNotFound $e) {
} catch (\PHPStan\BetterReflection\Reflector\Exception\IdentifierNotFound $e) {
// pass
} catch (UnableToCompileNode | NotAClassReflection | NotAnInterfaceReflection $e) {
// pass
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -2,9 +2,9 @@

namespace PHPStan\Reflection\BetterReflection;

use Roave\BetterReflection\Reflector\ClassReflector;
use Roave\BetterReflection\Reflector\ConstantReflector;
use Roave\BetterReflection\Reflector\FunctionReflector;
use PHPStan\BetterReflection\Reflector\ClassReflector;
use PHPStan\BetterReflection\Reflector\ConstantReflector;
use PHPStan\BetterReflection\Reflector\FunctionReflector;

interface BetterReflectionProviderFactory
{
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -12,16 +12,16 @@
use PHPStan\Reflection\BetterReflection\SourceLocator\OptimizedSingleFileSourceLocatorRepository;
use PHPStan\Reflection\BetterReflection\SourceLocator\PhpVersionBlacklistSourceLocator;
use PHPStan\Reflection\BetterReflection\SourceLocator\SkipClassAliasSourceLocator;
use Roave\BetterReflection\Reflector\FunctionReflector;
use Roave\BetterReflection\SourceLocator\Ast\Locator;
use Roave\BetterReflection\SourceLocator\SourceStubber\PhpStormStubsSourceStubber;
use Roave\BetterReflection\SourceLocator\SourceStubber\ReflectionSourceStubber;
use Roave\BetterReflection\SourceLocator\SourceStubber\SourceStubber;
use Roave\BetterReflection\SourceLocator\Type\AggregateSourceLocator;
use Roave\BetterReflection\SourceLocator\Type\EvaledCodeSourceLocator;
use Roave\BetterReflection\SourceLocator\Type\MemoizingSourceLocator;
use Roave\BetterReflection\SourceLocator\Type\PhpInternalSourceLocator;
use Roave\BetterReflection\SourceLocator\Type\SourceLocator;
use PHPStan\BetterReflection\Reflector\FunctionReflector;
use PHPStan\BetterReflection\SourceLocator\Ast\Locator;
use PHPStan\BetterReflection\SourceLocator\SourceStubber\PhpStormStubsSourceStubber;
use PHPStan\BetterReflection\SourceLocator\SourceStubber\ReflectionSourceStubber;
use PHPStan\BetterReflection\SourceLocator\SourceStubber\SourceStubber;
use PHPStan\BetterReflection\SourceLocator\Type\AggregateSourceLocator;
use PHPStan\BetterReflection\SourceLocator\Type\EvaledCodeSourceLocator;
use PHPStan\BetterReflection\SourceLocator\Type\MemoizingSourceLocator;
use PHPStan\BetterReflection\SourceLocator\Type\PhpInternalSourceLocator;
use PHPStan\BetterReflection\SourceLocator\Type\SourceLocator;

class BetterReflectionSourceLocatorFactory
{
Expand Down
Loading

0 comments on commit 06ef13e

Please sign in to comment.