rectorphp/rector-src issues and pull requests

#6520 - [Php70] Do not replace if method call if method exists in current class on call same method with parent on Php4ConstructorRector

Pull Request - State: closed - Opened by samsonasik 2 months ago - 2 comments

#6519 - [Php70] Handle parent method call on Php4ConstructorRector

Pull Request - State: closed - Opened by samsonasik 2 months ago - 2 comments

#6518 - [Printer] Try extends PHPStan Printer

Pull Request - State: closed - Opened by samsonasik 2 months ago - 2 comments

#6517 - Revert "[Printer] Remove AlwaysRememberedExpr check on BetterStandardPrinter"

Pull Request - State: closed - Opened by samsonasik 2 months ago - 9 comments

#6516 - [Php80] Mirror comments on assign on ChangeSwitchToMatchRector

Pull Request - State: closed - Opened by samsonasik 2 months ago - 2 comments

#6515 - Add `withSetProviders` method

Pull Request - State: closed - Opened by ghostwriter 2 months ago - 1 comment

#6514 - [TypeDeclaration] Skip with default value and assigned mixed on TypedPropertyFromAssignsRector

Pull Request - State: closed - Opened by samsonasik 2 months ago - 2 comments

#6513 - [Php83] Remove get traits class as ancestors collection on AddOverrideAttributeToOverriddenMethodsRector

Pull Request - State: closed - Opened by samsonasik 2 months ago - 2 comments

#6512 - [Php83] Allow indirect override abstract method on Grand Child on AddOverrideAttributeToOverriddenMethodsRector

Pull Request - State: closed - Opened by samsonasik 2 months ago - 3 comments

#6511 - Use createFromNode() from PhpDocInfoFactory on RemoveUnusedPrivateMethodRector

Pull Request - State: closed - Opened by samsonasik 2 months ago

#6510 - feat: add Behat annotations to attributes set

Pull Request - State: closed - Opened by carlos-granados 2 months ago - 1 comment

#6509 - [DeadCode] Skip private dataProvider method on RemoveUnusedPrivateMethodRector

Pull Request - State: closed - Opened by samsonasik 2 months ago - 5 comments

#6508 - Add custom skipper implementation.

Pull Request - State: closed - Opened by uncaught 2 months ago - 5 comments

#6507 - [Naming] Clean up unnecessary use of NodeTypeResolver on PropertyNaming

Pull Request - State: closed - Opened by samsonasik 2 months ago - 1 comment

#6506 - [CodeQuality][Renaming] Handle DynamicDocBlockPropertyToNativePropertyRector+RenameClassRector with aliased name

Pull Request - State: closed - Opened by samsonasik 2 months ago - 1 comment

#6505 - [Renaming] Clean up namespaced check on ClassRenamePhpDocNodeVisitor

Pull Request - State: closed - Opened by samsonasik 2 months ago - 1 comment

#6504 - [Renaming] Handle rename conflict docblock name on RenameClassRector

Pull Request - State: closed - Opened by samsonasik 2 months ago - 2 comments

#6503 - [PHPStanStaticTypeMapper] Ensure namespaced_name attribute set on Name node when create from ShortenedObjectType or AliasedObjectType

Pull Request - State: closed - Opened by samsonasik 2 months ago

#6502 - [Renaming] Add test for rename + auto import docblock with target name exist

Pull Request - State: closed - Opened by samsonasik 2 months ago

#6501 - [Renaming] add test for inner namespace rename docblock

Pull Request - State: closed - Opened by samsonasik 2 months ago

#6500 - [NodeTypeResolver] Clean up UseImportsResolver check on ClassRenamePhpDocNodeVisitor

Pull Request - State: closed - Opened by samsonasik 2 months ago - 1 comment

#6499 - [Renaming] More tests and clean up name check for rename aliased sub namespace on ClassRenamePhpDocNodeVisitor

Pull Request - State: closed - Opened by samsonasik 2 months ago

#6498 - [Renaming] Handle docblock aliased rename on RenameClassRector

Pull Request - State: closed - Opened by samsonasik 2 months ago - 2 comments

#6497 - [CodeQuality] Fix aliased object type on FlipTypeControlToUseExclusiveTypeRector

Pull Request - State: closed - Opened by samsonasik 2 months ago - 1 comment

#6496 - [TypeDeclaration] More tests for native type with sub namespace and aliased on TypedPropertyFromAssignsRector

Pull Request - State: closed - Opened by samsonasik 2 months ago

#6495 - [NodeTypeResolver] Clean up unnecessary check AliasedObjectType on getObjectClassReflections() call later

Pull Request - State: closed - Opened by samsonasik 2 months ago

#6494 - [Naming] More fixture tests for skip alias sub name and shot name with sub namespace on RenamePropertyToMatchTypeRector

Pull Request - State: closed - Opened by samsonasik 2 months ago

#6493 - [CodeQuality] Handle aliased object type from docblock on DynamicDocBlockPropertyToNativePropertyRector

Pull Request - State: closed - Opened by samsonasik 2 months ago - 3 comments

#6492 - [NodeTypeResolver] Remove no longer used FUNC_ARGS_TRAILING_COMMA constant

Pull Request - State: closed - Opened by samsonasik 3 months ago - 2 comments

#6491 - [Printer] Remove no longer needed pCommaSeparated() override method on BettterStandardPrinter

Pull Request - State: closed - Opened by samsonasik 3 months ago - 2 comments

#6490 - Update developing with docker: Fix php version to 8.2.

Pull Request - State: closed - Opened by uncaught 3 months ago - 1 comment

#6489 - Clean up comment on FileProcessor since printer no longer use ltrim

Pull Request - State: closed - Opened by samsonasik 3 months ago

#6488 - [Printer] Synchronize pParam() method on BetterStandardPrinter with parent Standard

Pull Request - State: closed - Opened by samsonasik 3 months ago - 1 comment

#6487 - [Printer] Remove no longer needed pSingleQuotedString method on BetterStandardPrinter

Pull Request - State: closed - Opened by samsonasik 3 months ago - 2 comments

#6486 - [Printer] Reuse clean heredoc/nowdoc start indent functionality on BetterStandardPrinter

Pull Request - State: closed - Opened by samsonasik 3 months ago - 1 comment

#6485 - [Printer] Use NewLineSplitter on BetterStandardPrinter for heredoc/nowdoc removal space

Pull Request - State: closed - Opened by samsonasik 3 months ago

#6484 - [Printer] Remove AlwaysRememberedExpr check on BetterStandardPrinter

Pull Request - State: closed - Opened by samsonasik 3 months ago - 1 comment

#6483 - [Strict] Using in_array() on CallLike instead of multiple calls on Strict Rules (part 2)

Pull Request - State: closed - Opened by samsonasik 3 months ago - 1 comment

#6482 - [Differ] Remove DefaultDiffer, use ConsoleDiffer instead to avoid double diffing process

Pull Request - State: closed - Opened by samsonasik 3 months ago - 4 comments

#6481 - Trigger CI

Pull Request - State: closed - Opened by samsonasik 3 months ago

#6480 - Use Type->getIterableKeyType() over ArrayType->getKeyType()

Pull Request - State: closed - Opened by staabm 3 months ago - 2 comments

#6479 - Use Type->getIterableValueType() over ArrayType->getItemType()

Pull Request - State: closed - Opened by staabm 3 months ago - 1 comment

#6478 - Utilize PHPStan Type interface in TypeNormalizer

Pull Request - State: closed - Opened by staabm 3 months ago - 1 comment

#6477 - [Scoper] Remove implements DocumentedRuleInterface on RectorInterface on scoper process

Pull Request - State: closed - Opened by samsonasik 3 months ago - 6 comments

#6476 - [Scoper] Add CommunityNodeVisitorAbstract to be used for community that include getRuleDefinition() empty method

Pull Request - State: closed - Opened by samsonasik 3 months ago - 20 comments

#6475 - [DeadCode] Handle crash on valid conditional type on RemoveUselessReturnTagRector

Pull Request - State: closed - Opened by samsonasik 3 months ago - 3 comments

#6474 - [CodeQuality] Skip compare nullable object on UseIdenticalOverEqualWithSameTypeRector

Pull Request - State: closed - Opened by samsonasik 3 months ago - 2 comments

#6473 - [CodeQuality] Skip used by array callable on LocallyCalledStaticMethodToNonStaticRector

Pull Request - State: closed - Opened by samsonasik 3 months ago - 2 comments

#6472 - [DeadCode] Skip used by get_object_vars() when implements JsonSerializable on RemoveUnusedPromotedPropertyRector

Pull Request - State: closed - Opened by samsonasik 3 months ago - 4 comments

#6471 - [Php83] Skip add #[\Override] on non-empty method from trait on AddOverrideAttributeToOverriddenMethodsRector

Pull Request - State: closed - Opened by samsonasik 3 months ago - 6 comments

#6470 - [Arguments] Handle parent call with static call on ArgumentAdderRector

Pull Request - State: closed - Opened by samsonasik 3 months ago - 1 comment

#6469 - Add fixture test for Doctrine annotation to attribute auto import FQCN (with @\ prefix)

Pull Request - State: closed - Opened by samsonasik 3 months ago

#6468 - feat: add the option to use the annotation value as an argument to the attribute

Pull Request - State: closed - Opened by carlos-granados 3 months ago - 3 comments

#6467 - [NodeTypeResolver] Reuse resolve fully qualified object type on NameImportingPhpDocNodeVisitor

Pull Request - State: closed - Opened by samsonasik 3 months ago

#6466 - [AutoImport] Handle doctrine aliased auto import

Pull Request - State: closed - Opened by samsonasik 3 months ago - 2 comments

#6465 - Revert " Fix regex space before star comment "

Pull Request - State: closed - Opened by samsonasik 3 months ago

#6464 - Fix regex space before star comment

Pull Request - State: closed - Opened by samsonasik 3 months ago - 2 comments

#6463 - [Renaming] Fix space on union docblock during rename on RenameClassRector

Pull Request - State: closed - Opened by samsonasik 3 months ago - 4 comments

#6462 - Fix ArrayItem scope refresh

Pull Request - State: closed - Opened by samsonasik 3 months ago

#6461 - [DeadCode] Skip by docblock definition on RemoveAlwaysTrueIfConditionRector

Pull Request - State: closed - Opened by samsonasik 3 months ago - 2 comments

#6460 - [Php71] Skip already array destruct on ListToArrayDestructRector

Pull Request - State: closed - Opened by samsonasik 3 months ago - 2 comments

#6459 - [Php80] Handle crash single quoted doctrine on AnnotationToAttributeRector

Pull Request - State: closed - Opened by samsonasik 3 months ago - 1 comment

#6458 - [CodeQuality] Handle crash on return array constant on ExplicitReturnNullRector

Pull Request - State: closed - Opened by samsonasik 3 months ago - 1 comment

#6457 - [DeadCode] Handle InterpolatedStringPart crash on RemoveUnusedForeachKeyRector

Pull Request - State: closed - Opened by samsonasik 3 months ago - 2 comments

#6456 - [Printer] Remove unnecessary override pExpr_Closure and pStmt_ClassMethod on BetterStandardPrinter

Pull Request - State: closed - Opened by samsonasik 3 months ago - 2 comments

#6455 - Fix along with other packages e2e test on rector/rector

Pull Request - State: closed - Opened by samsonasik 3 months ago

#6454 - [attributes] Make sure the target attribute class exists

Pull Request - State: closed - Opened by TomasVotruba 3 months ago

#6453 - Use vendor-patches main branch

Pull Request - State: closed - Opened by samsonasik 3 months ago

#6452 - [dx] Add ComposerSemver, to detect if packages of specific version is present

Pull Request - State: closed - Opened by TomasVotruba 3 months ago - 1 comment

#6451 - [dx] enable all attributes, if none are selected to streamline config

Pull Request - State: closed - Opened by TomasVotruba 3 months ago

#6450 - [dx] add validation to symfony xml/php config files

Pull Request - State: closed - Opened by TomasVotruba 3 months ago

#6449 - [chore] Set disableExtensionsAnalysis() on composer dependency analyser due to composer-dependency-analyser:^1.8

Pull Request - State: closed - Opened by samsonasik 3 months ago - 2 comments

#6448 - [chore] Remove left over config fixture test without test

Pull Request - State: closed - Opened by samsonasik 3 months ago - 1 comment

#6447 - Fix interface implementer in AddOverrideAttributeToOverriddenMethodsRector, add override only in case of overiding parent method with contents

Pull Request - State: closed - Opened by TomasVotruba 3 months ago

#6446 - [dx] Drop start with short open tag handling on FilesFinder

Pull Request - State: closed - Opened by samsonasik 3 months ago - 2 comments

#6445 - [dx] Add HTMLAverseRectorInterface to teach rules to skip HTML + PHP mix that would go wrong

Pull Request - State: closed - Opened by TomasVotruba 3 months ago

#6444 - [DX] Add default empty definition with add getRuleDefinition() method on AbstractRector

Pull Request - State: closed - Opened by samsonasik 3 months ago - 3 comments

#6443 - [DeadCode] Allow remove assign variable with next method call on RemoveDoubleAssignRector

Pull Request - State: closed - Opened by samsonasik 3 months ago - 2 comments

#6442 - Fix RemoveDoubleAssignRector in case of method call

Pull Request - State: closed - Opened by TomasVotruba 3 months ago

#6441 - Add "--only" option to process only a single rule

Pull Request - State: closed - Opened by cweiske 3 months ago - 17 comments

#6440 - [docs] getRuleDefinition() no longer required for custom rules

Pull Request - State: closed - Opened by TomasVotruba 3 months ago

#6439 - [fix] class phpdoc generic method

Pull Request - State: closed - Opened by lammafish 3 months ago - 1 comment

#6438 - [dx] Allow external rules without getRuleDefinition() to make them easier to write

Pull Request - State: closed - Opened by TomasVotruba 3 months ago - 2 comments

#6437 - Create doctrine_annotation_with_extra_parenthesis.php.inc

Pull Request - State: closed - Opened by caugner 3 months ago - 4 comments

#6436 - tv cleanup next

Pull Request - State: closed - Opened by TomasVotruba 3 months ago

#6435 - [dx] Show loaded composer based sets in case of --debug

Pull Request - State: closed - Opened by TomasVotruba 3 months ago

#6434 - [deprecation] Deprecate MixedTypeRector, to avoid mixed type filled blindly, use type declaration set instead

Pull Request - State: closed - Opened by TomasVotruba 3 months ago

#6433 - [dx] enable polyfills by default when PHP is on, to make config simpler

Pull Request - State: closed - Opened by TomasVotruba 3 months ago

#6432 - Support php-64bit as required in composer.json

Pull Request - State: closed - Opened by hans-thomas 3 months ago - 1 comment

#6431 - Upgrade to PHPParser 5 and PHPStan 2

Pull Request - State: closed - Opened by samsonasik 3 months ago - 25 comments

#6430 - [TypeDeclaration] Skip test methods with exception in ReturnNeverTypeRector, [php74] Skip conditinal assign in RestoreDefaultNullToNullableTypePropertyRector as most likely desired to assign or fail

Pull Request - State: closed - Opened by TomasVotruba 3 months ago

#6429 - Fix array key

Pull Request - State: closed - Opened by TomasVotruba 3 months ago

#6428 - [DX] Add version-based set loading based on installed package version

Pull Request - State: closed - Opened by TomasVotruba 3 months ago - 1 comment

#6427 - [Printer] Remove ltrim() on BetterStandardPrinter

Pull Request - State: closed - Opened by samsonasik 3 months ago - 2 comments

#6426 - [deprecated] Remove deprecated AbstractTestCase

Pull Request - State: closed - Opened by TomasVotruba 3 months ago

#6425 - [deprecation] Deprecate AbstractScopeAwareRector in favor of single AbstractRector

Pull Request - State: closed - Opened by TomasVotruba 3 months ago

#6424 - [deprecation] Remove deprecated PublicConstantVisibilityRector, cleanup deprecated SetListInterface

Pull Request - State: closed - Opened by TomasVotruba 3 months ago

#6423 - [Naming] Reduce instanceof ObjectType on ExpectedNameResolver (take 5)

Pull Request - State: closed - Opened by samsonasik 3 months ago - 2 comments

#6422 - Reduce instanceof ObjectType, use ->isObject()->yes() (take 5)

Pull Request - State: closed - Opened by samsonasik 3 months ago - 1 comment

#6421 - [StaticTypeMapper] Reduce instanceof TypeWithClassName usage via ClassNameFromObjectTypeResolver (take 4)

Pull Request - State: closed - Opened by samsonasik 3 months ago - 1 comment

GitHub / rectorphp/rector-src issues and pull requests