Update phpmd/phpmd from 2.7.0 to 2.14.1 #150
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
violinist-bot
force-pushed
the
phpmdphpmd2702141
branch
from
05d7b04 to
52af0d3
Compare
violinist-bot
force-pushed
the
phpmdphpmd2702141
branch
from
52af0d3 to
bdd3afb
Compare
violinist-bot
force-pushed
the
phpmdphpmd2702141
branch
from
bdd3afb to
31e54d0
Compare
violinist-bot
force-pushed
the
phpmdphpmd2702141
branch
from
31e54d0 to
c4bdf2b
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
If you have a high test coverage index, and your tests for this pull request are passing, it should be both safe and recommended to merge this update.
Updated packages
Some times an update also needs new or updated dependencies to be installed. Even if this branch is for updating one dependency, it might contain other installs or updates. All of the updates in this branch can be found here:
Release notes
Here are the release notes for all versions released between your current running version, and the version this PR updates the package to.
List of release notes
Changed files
Here is a list of changed files between the version you use, and the version this pull request updates to:
List of changed files
Changelog
Here is a list of changes between the version you use, and the version this pull request updates to:
Merge pull request #1040 from phpmd/release/v2.14.1Require PDepend 2.15.1Add 2.14.1 changelogv2.14.1Merge pull request #1037 from frankdekker/Result-cache-does-not-work-without-baseline-fileUse array_key_exists instead of isset to check of data contains all the necessary keys.Merge pull request #1033 from tvbeek/prepare_2-14-0Add #989 #990 to the changelog.Merge pull request #1032 from tvbeek/pdepend_2-15-0Prepare for version 2.14.0Update PDepend to version 2.15.0Merge pull request #1031 from JustMisha/fix_call_to_undefined_getparent_methodFix syntaxAvoid using else statementChanged the variable nameMerge pull request #1029 from JustMisha/extra-line-in-excerpt-can-be-set-via-cli-argFix PHP Fatal Error: Uncaught error: Call to undefined method PDepend\Source\AST\AST\ASTFunction::getParent() in phpmd\src\main\php\PHPMD\Rule\CleanCode\BooleanArgumentFlag.php:60 for files like:Added description of the --extra-line-in-excerpt option to README.rst;Added tests for the extra line in excerpt optionsMade the $extraLineInExcerpt argument optional in the __construct method of the HTMLRenderer class.Added the option to the command line to set the number of extra lines in code snippets.Merge pull request #1023 from kylekatarnls/feature/error-details-outputSimple multiline comment & fix typoMerge branch 'master' into feature/error-details-outputComment load-coverage-tokens.php purposeExplain why to fix tokens on test runFix typo in help outputAdd parentheses for ternaryMerge pull request #1026 from phpmd/fix/issue-467-duplicate-camel-case-violationMerge pull request #1025 from kylekatarnls/feature/stdinDedupe violation for CamelCaseVariableNameAdd documentationAdd tests for glob and stdin featuresFix typo in testSupport glob patternAdd shortcut for STDINIgnore line number in testSwitch top xdebug 3Fix stream handlerAdd tests for cache optionsRun coverage with PHP 8.2Run tests also with PHP 8.3Test all renderersFix PHPDoc, simplify getListOfAvailableRenderers() and handle nll value from thereAdd trace of the error when debug verbosityMerge pull request #1022 from phpmd/feature/verbose-outputMerge pull request #1024 from kylekatarnls/feature/upgrade-phpcsRun PHPCS 3 on GitHub ActionsMerge branch 'master' into feature/verbose-outputMerge pull request #1021 from szepeviktor/fix-typosMerge --verbose and -v, -vv, -vvvAdd tests for --verbose and --color optionsDocument --verbose and --color optionsAdd --verbose and --color options to text rendererFix typosAdd debug information to cache results (#1015)Add prefix removal for class names and variable names (#962)Merge pull request #1017 from AJenbo/phpstan-ciPrevent nested list in AUTHORSRun PHPStan in CIMerge pull request #1005 from phpmd/authorsAdd @AJenbo to authors listMerge pull request #1011 from frankdekker/Add-support-for-cache-resultsMerge branch 'master' into Add-support-for-cache-resultsMerge branch 'phpmd:master' into masterMerge branch 'master' into authorsMerge pull request #1007 from tvbeek/1006-fix-sarif-testKeep Linux compatibility for SARIFRendererTestFix SARIFRendererTest Windows compatibilityFix SARIFRendererTest Windows compatibilityMerge pull request #1014 from tvbeek/phar-signingAdd phar signing.Update src/test/php/PHPMD/Renderer/SARIFRendererTest.phpUpdate src/test/php/PHPMD/Renderer/SARIFRendererTest.phpMerge pull request #1012 from DannyvdSluijs/UseGitHubbActionsV3Merge pull request #1013 from DannyvdSluijs/IncludePHP82InActionMatrixci: Include PHP 8.2 in Actions matrixci: Update to use GitHub Actions V3Fix codestyle issueAdd coverage Paths::concatAdd coverage NodeInfoFactoryAdd coverage NodeInfoFix ResultCacheKeyFactoryTestFix phpcs issuesAvoid reading or writing the cache when generating/updating the baselineAvoid reading or writing the cache when generating/updating the baselineAdd baseline file hash to cache keyAdd baseline file hash to cache keyAdd commandline cli informationAdd coverage ResultCacheStateAdd coverage ResultCacheStateAdd coverage ResultCacheStateUpdate gitignoreFix PHPMDTestFix coverage RuleViolationTestAdd coverage ResultCacheStateAdd coverage ResultCacheEngineFactoryAdd coverage ResultCacheUpdaterAdd coverage ResultCacheWriterAdd coverage ResultCacheKeyFactoryAdd coverage ResultCacheKeyFactoryAdd coverage ResultCacheStateFactoryAdd coverage ResultCacheFileFilterAdd coverage ResultCacheFileFilterAdd coverage ResultCacheEngineAdd coverage ResultCacheKeyChange mb5_file to sha1_fileRun phpcbfAdd composer to cache keyImprove ResultCacheStateFactoryCleanup ResultCacheConfigAdd cache key factoryAdd cache key factoryAdd cache key factoryAdd basics for ResultCacheKeyReorganize namespaceAdd cache strategy commandline argumentImprove ResultCacheEngineFactoryAdd commandline options to factoryUpdate documentationAdd commandline options for --cache and --cache-fileFix codestyleReorganize classesExtract ResultCacheFileFilterCreate RuleViolations from result cacheRead state from result cacheRead state from result cacheAdd ResultCacheIO to save and read the stateAdded method to add RuleViolation to ResultCacheStateExtract AbstractNode from RuleViolationAdd result cache config, state and engineAdd result cache config, state and engineAdd result cache config, state and engineAdd result cache config, state and engineMerge pull request #6 from phpmd/masterRevert the .sarif files.Remove the @package_version@ part.Set version number in renderer processing file.Remove the closure that shouldn't change anything.Add extra assertions to see which json data is invallid and see the full data.Codestyle fix, split a long line in 2 lines.Change line endings in the json to see if that fixes the test on Windows.Update authorsMerge pull request #1004 from tvbeek/pdepend_2-14-0Update PDepend to version 2.14.0Merge pull request #999 from tvbeek/add_dev_to_composerMerge pull request #1000 from DannyvdSluijs/patch-1Correct url by removing leading slashAdd dev as keyword to the composer.json file to suggest to install it as a require-dev package. See: https://github.com/composer/composer/pull/10960Merge pull request #997 from tvbeek/add_mastodon_to_the_websiteCheck if we make Stickler CI happy with padding of 0 in combination with an height.Add a follow on mastodon link to the footer on the website.fix: global import from an array valueMerge pull request #990 from andrey-tech/issue-989Improve documentationMerge pull request #972 from phpmd/AJenbo-patch-1Decrease BooleanArgumentFlag complexityMerge pull request #986 from pieterocp/fix/php81-html-render-deprecationFix deprecation message for passing null to a non-nullable internal functionMerge pull request #983 from kylekatarnls/fix/issue-924-bool-arg-in-constructorAdd exceptions and ignorepattern properties to BooleanArgumentFlag ruleMerge pull request #981 from phpmd/release/2.13.0Add date for 2.13.0 (tentative)Bump version numberMerge pull request #980 from kylekatarnls/feature/php-8-2Cleanup importsFix PHPDoc typoAdd testsMake rules aware of traits and enumsAdd test case with enumStart PHPMD 2.13.0 CHANGELOGRequire pdepend/pdepend 2.12.0Try PDepend 2.12.0Merge pull request #978 from koenhendriks/masterAdd trailing commaExpand test with github and gitlab file reportsAdd missing github & gitlab reportfile optionsMerge pull request #976 from szepeviktor/patch-1Avoid deprecated exceptionsUpdate phpunit_coverage.ymlMerge pull request #970 from phpmd/travisMerge pull request #969 from phpmd/codecovRemove travis badgeUpdate README.rstUpdate codecov intergrationMerge pull request #968 from kylekatarnls/fix/deprecate-substr-nullMerge branch 'master' into fix/deprecate-substr-nullOptimize parseRuleNode steps orderFix PHP 8.1 deprecation for substr null valueMerge pull request #963 from frankdekker/Static-access-add-ignore-pattern-for-methodsSolve phpcs issueUpdate documentationAdd ignorepattern to StaticAccess ruleMerge pull request #959 from frankdekker/Add-missing-ruleset-error-message-on-baselinethrow Exception when ruleset is absent file location can't be determined.throw Exception when ruleset is absent file location can't be determined.throw Exception when ruleset is absent file location can't be determined.Merge branch 'phpmd:master' into masterMerge pull request #957 from metalmini/gitlab_rendererRemove unneeded counterAdd Gitlab renderer for Gitlab CIAdd Gitlab renderer for Gitlab CIAdd Gitlab renderer for Gitlab CIMerge pull request #954 from modess/masterbetter npath complexity documentation with exampleMerge pull request #946 from jvipez/highlight-code-examplesMerge pull request #947 from tvbeek/prepare_2.12.0Merge branch 'master' into prepare_2.12.0Merge pull request #948 from phpmd/kylekatarnls-patch-1Fix setup submodule URLIncrease minimum version of PDependMove the git submodule from git to https because we don't use any auth.Prepare for version 2.12.0Highlight readme code examplesMerge pull request #944 from emretepedev/patch-1Update exclude-pattern.xmlMerge pull request #942 from kylekatarnls/add_php8.1_tests_configFix dependencies files for PHP 8.1 compatibilityMerge pull request #918 from tvbeek/add_php8.1_tests_configMerge pull request #833 from martzd/feature/disallow-capital-in-camel-case-property-nameMerge pull request #930 from franmomu/patch-1Allow install composer/xdebug-handler 3Merge pull request #929 from phpmd/release/2.11.1Add 2.11.1 changelogBump build.properties version number to 2.11.1Merge pull request #928 from YAhiru/fix-deprecation-messagesFix assignment to a variable in a conditional expression.remove redundant function callsFix deprecation messagesMerge pull request #926 from villfa/fix/php81Fix remaining deprecation messagesMerge pull request #925 from villfa/patch-1Don't call preg_match_all on nullMerge pull request #922 from phpmd/release/2.11.0Add Rule name in text output in changelogMerge pull request #817 from phpmd/feature/issue-815-rule-name-in-text-outputAdd fix #781 in the changelogMerge pull request #783 from phpmd/feature/issue-601-unused-propertyUpdate Changelog for 2.11.0Require at least PDepend 2.10.2Prepare build for 2.11 releaseMerge pull request #920 from ricfio/update-changelog-2-10-2-versionfix typo on version in CHANGELOGUpdate the phpunit to start testing on PHP 8.1Merge pull request #917 from phpmd/contrib-github-folderMention the contribution guideMove contribution guide to .github folderMerge pull request #911 from phpmd/add_twitter_link_to_footerRemove the box-sizingCombine 2 background parts and try a recheckReorder last part (thanks Stickler)Hanlde the Stickler-ci commentsAdd a follow on twitter link to the footerMerge pull request #908 from kylekatarnls/feature/mute-php-8-deprecationAdd commentsFix code styleHandle Twice in invokes countSplit long lineFix mergeMerge branch 'master' of https://github.com/phpmd/phpmd into feature/issue-601-unused-propertyUpdate unit tests#815 Add unit test for text output alignment of last columns#815 Add unit test for text output alignmentFix #815 add rule name in text outputFix PHP 8 deprecation notices on the fly for phar buildMerge pull request #4 from phpmd/masterMerge pull request #906 from Sylry/fix_false_positive_static_variable_870Fix typoFix false positive with static variable in function. fixes #870Merge pull request #905 from YAhiru/feature/issue-884-unused-propertyUnusedFormalParameter does not apply to property promotion parametersMerge pull request #904 from TJVB/prepare_2_10_2Prepare the release for version 2.10.2Merge pull request #903 from frankdekker/Improve_baseline_basepath_calculationRemoved unused importFixed coverage notationUpdated documentationAdd coverage for changed filesRemove absolute path from BaselineSet and adjust baseline matchingMerge pull request #3 from phpmd/masterMerge pull request #902 from himanshu007-creator/patch-2Update tests according to new vertical spacingUse writer getterUse PHP_EOLUpdate src/main/php/PHPMD/Renderer/AnsiRenderer.phpUpdate AnsiRenderer.phpadded newlineMerge pull request #894 from dmetzler1988/bugfix/duplicated-array-key-typoFixed typo in DuplicatedArrayKey title incl. link.Add changelog for 2.10.1 and bump project.version (#890)Merge pull request #885 from pOmelchenko/masteradd 2.0 version of thecomposer/xdebug-handler``Merge pull request #880 from phpmd/release/2.10.0Suppress EmptyCatchBlock for intended TODOAdd 2.10.0 changelogBump project.versionMerge pull request #878 from kylekatarnls/feature/php-8Merge pull request #828 from kylekatarnls/feature/issue-672-fallback-function-without-namespace-checkMerge pull request #879 from phpmd/document-junitMerge pull request #836 from kdebisschop/reference-to-root-namespace-is-not-a-missing-importFix tests after rebase from masterFix @covers that was broken by change in function nameMethod addProprty() second parameter is string, cannot be booleanuse early-returnMerge pull request #839 from TJVB/cleancode_rulesMerge pull request #856 from ffflabs/feature/checkstyle_rendererAlign PHPDoc asterixStick cast to variable (PHPCS compliance)Merge pull request #786 from phpmd/findChildrenOfVariableMerge branch 'master' into findChildrenOfVariableAdd link to junit.xslt config fileAdd junit.xslt configDocument junit export using third-partyUpdate mikey179/vfsstreamMerge branch 'master' into feature/checkstyle_rendererMerge branch 'master' of https://github.com/phpmd/phpmd into cleancode_rulesFix file pathsMerge branch 'master' into reference-to-root-namespace-is-not-a-missing-importMerge branch 'master' of https://github.com/phpmd/phpmd into feature/disallow-capital-in-camel-case-property-nameFix incompatible files before running testsMerge pull request #865 from lukasbestle/feature/sarif-rendererIndent long lineAllow PHP 8Merge branch 'master' into feature/sarif-rendererMerge pull request #876 from frankdekker/update-violation-baselineAppease scrutinizer: suppress PHP7.4 getMock errorUpdate documentationUpdate documentationUpdate documentationUpdate documentationUpdate documentationAdd coverage CommandTest for update baselineAdd coverage CommandRemove unused importsAdd tests for BaselineValidatorMerge pull request #2 from phpmd/masterFixes: unlink($tempFile): Resource temporarily unavailableAdd tests for BaselineValidatorAdd tests for BaselineValidatorAdd tests for BaselineValidatorAdded BaselineValidatorReplace BaselineSet with Report in PHPMD::processFiles methodAdd BaselineMode + Update CommandLineOptions to support --update-baselineMerge pull request #875 from frankdekker/violation-baseline-appveyor-fixesMerge pull request #874 from TJVB/update_contributingFix slashes when running tests on windows environmentMerge pull request #1 from phpmd/masterMerge pull request #873 from frankdekker/violation-baselineUpdate CONTRIBUTING.md - Add a link to gitter - Remove the link to IRC - Add using an issue template to the issue partFix scrutinizer warningsFix scrutinizer warningsRevert scrutinizer violationsRevert scrutinizer violationsFix scrutinizer violationsReplace STDOUT with older php notationRemove expectsreplace createMock with getMockBuilderFinal adjustmentsAdd coverage PHPMD::processFilesAdd coverage Command::mainAdd coverage Command generateBaselineAdd coverage Command generateBaselineRemoved basedir from BaselineFactoryAdded realpath + coverageAdded realpath + coverageUpdate documentationAdd coverage for method typeAdd coverage for method typeAdjust baseline renderer to include method nameAdjust baseline renderer to include method nameAdd coverage for CommandLineOptionsAdd coverage for RendererFactoryAdd coverage for RendererFactoryAdd coverage for RendererFactoryAdd coverage for BaselineRendererAdd coverage for baseline reportAdd coverage for baseline reportAdd coverage for StreamWriter getStreamComplete coverage on Paths getAbsolutePathComplete coverage on Paths concat + getRelativePathComplete coverage on BaselineFileFinderComplete coverage BaselineSetFactoryTestRemove unnecessary argumentsUpdated documentationUpdated documentationReduce lines of code in CommandLeave basedir argument out-of-scopeAdded BaselineFileFinder to find the baseline file.Added BaselineFileFinder to find the baseline file.Added BaselineFileFinder to find the baseline file.Added Baseline to PHPMD.phpAdded BaselineSet to ReportAdded BaselineSet to ReportAdded BaselineSet to ReportAdded coverage BaselineSetFactory.phpAdded coverage BaselineSet + ViolationBaselineAdded BaselineSet Added ViolationBaseline Added BaselineSetFactoryAdded generate-baseline and baseline-basedir to the cli params Added BaselineRenderer to write the baselineMerge pull request #861 from lukasbestle/feature/docs-github-rendererMerge pull request #869 from danepowell/issue-868Merge pull request #782 from phpmd/feature/issue-673-ignore-dynamic-class-namesUpdate src/site/rst/rules/cleancode.rstMerge pull request #844 from babanesma/compact_double_quotesMerge pull request #872 from lukasbestle/feature/514-error-exit-codeChange exit code on processing errorsUpdate src/site/rst/rules/cleancode.rstadd test casecode style fixfix testsadded docsFix #868: Option to ignore globally-namespaced classes from MissingImportAdd blank line after if blockMerge branch 'master' of https://github.com/phpmd/phpmd into feature/issue-672-fallback-function-without-namespace-checkMerge pull request #838 from TJVB/naming_rulesMerge pull request #772 from TJVB/rename_confusion_functions_in_PHPMD_and_deprecate_old_functionsMerge pull request #822 from phpmd/feature/issue-818-unit-testsMerge pull request #827 from phpmd/feature/issue-826-foreach-catchMerge pull request #849 from TJVB/remove_robotsMerge pull request #862 from phpmd/kylekatarnls-cleanup-generate-phar-ymlMerge branch 'master' of https://github.com/phpmd/phpmd into kylekatarnls-cleanup-generate-phar-ymlMerge pull request #788 from phpmd/feature/issue-739-class-begin-lineAdd SARIF rendererAbstractStaticTest: Add placeholder for work dirAbstractStaticTest: Support custom JSON comparisonPass message arguments to RuleViolationMerge pull request #852 from SanderSander/fix-test-underscoreMerge pull request #864 from kylekatarnls/fix/github-pagesAdd missing environment variablesCleanup generate_phar.ymlGitHub actions (#848)Docs for the GitHub rendererUnify both allow-underscore RegExpsMerge branch 'master' into feature/checkstyle_rendererMerge pull request #857 from lukasbestle/feature/github-rendererUpdate src/main/php/PHPMD/Renderer/CheckStyleRenderer.phpRemove scalar type declarations and keep file pathUse method 'maybeAdd' from parent rendererInherit from XMLRenderer where it's dueUpdate src/main/php/PHPMD/TextUI/CommandLineOptions.phpCleanupAdd GitHub renderer for GitHub Actionsrepairs PSR2 code style faults, adds description to READMEadds checkstyle compatible renderer, suitable for cs2pr or reviewdogImprove test documentationSimplify RegExpFix multiple underscores in method name when allow-underscore-test is enabled in CamelCaseMethodNameNested is not a method, MyClass() is.Add documentation for property controlling application to imports of root namespace.Add option to allow referencing root namespace without importRemove the incorrect robots.txt and the reference in the site.iniMerge pull request #847 from samuelwang23/masterNew Development Code Fragment TestsFix typoRevert "Update Development Code Fragment Tests"Update Development Code FragmentRevert "Fix Issue #846"Update Development Code Fragment TestsFix Issue #846Merge branch 'master' of https://github.com/phpmd/phpmd into feature/issue-818-unit-testsrecognize compact variables with double quotesApply suggestions from code reviewChange the naming error where possiblefix: Avoid assigning values to variables in if clauses and the likeAvoid using undefined variables such as ... which wil lead to PHP noticesfix the Missing class import via use statement errorsMerge pull request #835 from phpmd/feature/issue-829-getFileNameLink todo to issueMerge pull request #832 from kamioftea/add-external-info-urlMerge pull request #834 from martzd/feature/add-tool-to-xml-reportMerge pull request #830 from ComiR/patch-1Mute filename errorAdd unit tests for #829Changes according to reviewFix testsAdd "tool" property to main "pmd" tag in XML reportDisallow capital letter at the beginning in CamelCasePropertyName ruleMinor commit to retry build that failed due to 'Peer fingerprint did not match'Add test for missing externalInfoUrlAdd externalInfoUrl for UndefinedVariableAdd missing return/throws annotationKeep $functionName before calling getReflectionFunctionFromFunctionName()Rename getReflectionFunction() into getReflectionFunctionByName()Revert Annotation.php mistaken import#672 Try global function if namespaced one is not foundFix #826 consider foreach exception only for direct childrenMerge pull request #825 from phpmd/changlog-2.9.1Merge pull request #823 from kylekatarnls/feature/issue-714-fix-member-detectionAdd Changelog for 2.9.1Merge pull request #821 from phpmd/kylekatarnls-build-number#714 Add dedicated unit test for call with static memberFix #714 Improve static member detection#818 Don't check function arguments if the callee is a methodMerge pull request #818 from a-menshchikov/patch-1Upgrade build version number for next patchFixed undefined index referringMerge pull request #795 from TJVB/changelog_2.9.0Lets change the release date for 2.9.0 to 2020/09/02Merge pull request #1 from kylekatarnls/patch-1Add new items to the changelogMerge pull request #809 from phpmd/feature/issue-808-supress-warningsMerge pull request #807 from artem-prozorov/fix_for_short_variables_ruleMerge pull request #806 from artem-prozorov/change_private_methods_to_protected_in_rulesMerge pull request #811 from frankdekker/short-long-class-name-documentationUpdated documentation for Short and Long Class name rulesMerge pull request #1 from phpmd/masterArray declaration is changed from [] to arrayShort variables in cycles are made configurableMerge pull request #805 from TJVB/handle_nested_arraysReduce the complexity by removing 2 not needed checks.Merge branch 'fix_for_short_variables_rule' of github.com:artem-prozorov/phpmd into fix_for_short_variables_ruleFix #808 Ignore rule path for supression annotationFix wrong return typeRevert unwanted code removalAray initialization changed to support old PHP verionsFix PHP 5.3 compatibility for $parentsFix PHP 5.3 compatibility for $exceptionVariableRemoved empty lines in ShortVariableTestAdded fix for ShortVariable Rule to process names inside freach loopAdded regression test for issue 790Restoreg original array in src/main/php/PHPMD/Rule/UnusedFormalParameter.phpPrivate methods are switched to protected in Naming rulesPrivate methods are switched to protected in Design rulesPrivate methods are switched to protected in Controversial rulesPrivate methods are switched to protected in CleanCode rulesPrivate methods are changed to protected in basic rulesMention #765 in the changelogMerge pull request #765 from frankdekker/class-name-length-ruleCheck if the base is an instance of ASTNode before calling the getNode functionAdd a test to show the error in https://github.com/phpmd/phpmd/issues/802Merge pull request #791 from phpmd/feature/fix-import-phpunit-exceptionMerge pull request #793 from phpmd/feature/issue-580-clsoure-formal-parameter#739 Shorten variable name numberOfPublicMethods > publicMethodsCount#739 Upgrade minimum version of PDependMerge pull request #792 from phpmd/feature/issue-673-missing-imports-testingMerge pull request #794 from phpmd/feature/issue-540-unused-variable-declared-twice#540 Add PHPDoc for containsUsages() methodRemove UnusedLocalVariable property customizationFix -> FixedReorder getViolationFailureMessage parametersChunk the composition of violation failure messagesPrepare a changelog for version 2.9.0#580 Use ->isInstanceOf instead of instanceofFix #540 Detect unused variable declared multiple times#540 Add a unit test to demonstrate the double-assignation issue#580 Loop over every ASTFormalParameter in UnusedFormalParameter#580 Ignore ASTFormalParameter in UnusedLocalVariable rule#580 Add unit tests to demonstrate the rule mismatchMerge pull request #789 from TJVB/update_coding_standard_description_in_contributingAdd comment about how options are setRevert early-return groupingFix mergeMerge branch 'master' of https://github.com/phpmd/phpmd into feature/issue-601-unused-propertyMigrate MissingImport tests on the apply/notApply standardCleanup merge changesMerge branch 'master' into feature/issue-673-ignore-dynamic-class-namesImprove error message for apply and notApply testsAdd file basename for both apply and notApplyUpdate CONTRIBUTING to point out to the correct coding standardMerge pull request #787 from phpmd/feature/migrate-missing-import-testingImprove wordingCSCSImprove doc bockCorrect statement in method doc block#739 Add unit test to demonstrate the class begin line issueFix typehintPrefix dynamic file rule violation test exception with the file base name.Provide getClassNodeForTestFile in AbstractTest and rename getMethodNodeForTestFile into getNodeForTestFileMake getMethodNodeForTestFile return the testable FunctionNode if MethodNode is not availableMerge branch 'master' into findChildrenOfVariableMerge pull request #784 from phpmd/feature/issue-672-passing-by-referenceUse new convenience method AbstractNode::findChildrenOfVariable()Add convenience method AbstractNode::findChildrenOfTypeVariable()Merge pull request #785 from phpmd/feature/optimize-unused-formal-parameterFlip if statement around to return earlyCall ->getNode only if it's not yet the PDepend ASTNodeSplit early-returnRestore getNode protected visibilityMerge branch 'master' into feature/issue-601-unused-propertyMerge pull request #781 from phpmd/feature/issue-714-fix-self-and-static-properties-accessOptimize UnusedFormalParameter#672 Group variable definition rules#672 Provide isPassedByReference method in AbstractLocalVariableFix #672 Handle passing-by-reference in native PHP functions#672 Add a unit test to demonstrate the passing-by-reference issueAdd a dedicated method prependMemberPrimaryPrefixFix #601 Stop considering properties in anonymous class as local variablesCopy tooling for AbstractLocalVariable from #781#601 Add unit test to demonstrate the issueMigrate UnusedLocalVariable to the new testing systemMigrate MissingImport to the new testing systemFix #673 Ignore dynamic class name from missing importsStandardize PHPDoc styleHandle static properties definitionMove $selfReferences in the common AbstractLocalVariableProvide getVariableImage in the common AbstractLocalVariableFix #714 static:: and self:: properties accessUpdate the docblock with @this and add an empty line. Feedback from @kylekatarnlsRemove unwanted empty linesFix the mocking in the ParserFactoryTest after using the new functionsUpdate docblocks with feedback from @ravage84Merge pull request #773 from phpmd/cs-fixesUse elseif instead of else + nested ifUse elseif instead of else + nested ifUse elseif instead of else + nested ifUse foreach instead of forMerge pull request #775 from phpmd/move-exceptionsUse original exceptions againAdd class aliases for moved exceptionsUse elseif instead of nested else & ifMerge branch 'master' into cs-fixesReplace outdated join function with implodeFix grammarReplace outdated join function with implodeRemove now superfluous var annotationsUpdate index.rstMerge pull request #770 from phpmd/feature/issue-769-handle-deconstruction-assignationsMove all exception classes to their own sub namespace/folderRevert one illogical reformattingAutomatic code reformatting by PhpStorm 2020.1The keyword elseif SHOULD be used instead of else if so that all control keywords look like single words.The keyword elseif SHOULD be used instead of else if so that all control keywords look like single words.Automatic code reformatting by PhpStorm 2020.1Do not call function in loopCSCollapse long lineRemove unnecessary white linesMerge pull request #756 from phpmd/feature/simple-rule-testingUpdate the use of the deprecated functions from PHPMD in the ParserFactory.Rename setIgnorePattern to addIgnorePatterns and getIgnorePattern to getIgnorePatterns so they are less confusion.Fix test nameMerge pull request #768 from phpmd/alias-removalsAdd important noteAlways use relative path for data dirRemove notes of now defunct PEAR channelRemove PEAR installation workaroundRename $deconstructedVariable to shorter one $unpackedVariableMerge branch 'master' of https://github.com/phpmd/phpmd into feature/issue-769-handle-deconstruction-assignationsFix #769 Handle deconstruction assignation for undefined variableRemove doubly "long" in example class nameChanged Short Class name example fromIdtoFo``Suppress violation for long variable nameBump maximum threshold of LongClassNameImprove test case doc blocksImprove wordingClarify code through intermediate variablesCorrect coverage annotation and FQCNGet class or interface name only onceImprove wordingImprove wordingCSCSGet class or interface name only onceImprove wordingImprove wordingRename Support namespace to UtilityImprove test doc blocksUpdate test case names to changed method namesCorrect doc block statementUse correct coverage annotationRename Strings::split() to splitToList()Use double quotesRename Strings::length() to lengthWithoutSuffixes()Correct test case name & test fileCorrect test case name & test fileImprove test case doc blocksCorrect coverage annotation and FQCNImprove doc block wordingRemove superfluous class aliasesRemove class aliasMove length calculation inside the if for consistency amongst rules Moved "threshold" to the right side so it's better understandable. Tightened the LongVariable length constraints in the test case.Fixed threshold calculation.PHPUnit 4.8.3 expectException fix.Updated docblock of StringsTestAdded example to exceptions listUpdated documentation for the short class name ruleShortClassName rule + testcasesUpdated documentationLongVariable Rule + TestsUpdate LongVariable rule with getStringProperty and string split/lengthStrings support class: - length: to calculate string length subtracting certain suffixes - split: to split a string trimming and removing empty valuesStrings support class: - length: to calculate string length subtracting certain suffixes - split: to split a string trimming and removing empty valuesMerge pull request #764 from phpmd/feature/issue-718-handle-anonymous-class-for-undefiend-variable-ruleAdd PHPDoc for collect()Fix #718 Handle anonymous class in "undefined variable" ruleMerge branch 'master' into feature/simple-rule-testingMerge pull request #529 from smuuf/masterMerge pull request #747 from frankdekker/long-variable-subtract-suffixMerge pull request #496 from eeree/issue-62Add forgotten file doc blockMake use of new test convenience methodsCS fixesReturn early & return pre-calculated variable name lengthDo not calculate variable name length in loopFiltered empty suffixes from configuration Removed $length > 0 checked Added test for empty subtract-suffixes configurationRenamed -testRuleAppliesToVariableNameSmallerThanThresholdWithSuffixSubtract-testRuleAppliesToVariableNameLongerThanThresholdWithSuffixSubtract``Add new property to rule documentation.Improve doc blockRename $length to $suffixLengthAdded unit tests for LongVariableAdded unit tests for LongVariableAdded unit tests for LongVariableAdded unit tests for LongVariableAdded unit tests for LongVariableAllow to subtract certain suffixes from the total length of the variable name.Improve doc blocksSetup coverage annotationMake use of new getReportWithOneViolation() methodAdjust speed impact claimCS fixesMerge branch 'issue-62' of https://github.com/eeree/phpmd into eeree-issue-62Merge masterCSFixed closing tagMerge pull request #759 from phpmd/phpcs-configRemove files to check with PHPCS as they are not configuredAdd files to check wih PHPCS to its configurationRe-add dev requirement on PHP extensionsLet Stickler CI use our new PHPCS CS definitionSimplify class name in throws annotation to fit usageUse code style definition with Composer scriptsDisable slevomat rules as we cannot use them with PHPCS 2.xRevert upgrade to PHPCS 3.5 & addition of slevomat/coding-standardAdd SlevomatCodingStandard.Classes.MethodSpacing sniff & fixesAdd Squiz.Commenting.FunctionCommentThrowTag sniff & fixesAdd Generic.Files.LineLength sniffAdd SlevomatCodingStandard.Whitespaces.DuplicateSpaces sniff & fixesReorder sniffsAdd SlevomatCodingStandard.Exceptions.DeadCatch sniffAdd SlevomatCodingStandard.Variables.UnusedVariable sniffAdd SlevomatCodingStandard.Variables.DuplicateAssignmentToVariable sniffAdd SlevomatCodingStandard.Variables.DisallowSuperGlobalVariable sniffAdd SlevomatCodingStandard.PHP.UselessSemicolon sniffRemove useless parenthesisAdd SlevomatCodingStandard.Namespaces.UselessAlias sniffAdd SlevomatCodingStandard.Namespaces.UseFromSameNamespace sniff & fixesAdd SlevomatCodingStandard.Namespaces.UnusedUses sniff & fixesAdd SlevomatCodingStandard.Classes.EmptyLinesAroundClassBrace sniffAdd SlevomatCodingStandard.ControlStructures.UselessTernaryOperator sniffAdd many general sniffsAdd Squiz.Commenting.DocCommentAlignment sniff & fixesAdd Generic.CodeAnalysis.ForLoopWithTestFunctionCall sniff & fixesAdd Squiz.WhiteSpace sniffsRename file name to fit class nameAdd Generic.Formatting.NoSpaceAfterCast sniff & fixesRename class to fit file nameAdd Squiz.PHP.Eval sniffAdd Generic.PHP.DeprecatedFunctions sniffSimplify returnAdd SlevomatCodingStandard.Commenting.UselessInheritDocComment sniffAdd SlevomatCodingStandard.Functions.UnusedParameter sniff & suppress violationsAdd SlevomatCodingStandard.Functions.UnusedInheritedVariablePassedToClosure sniffReorder sniffsAdd SlevomatCodingStandard.Operators.RequireCombinedAssignmentOperator sniffReplace SlevomatCodingStandard.Operators.DisallowIncrementAndDecrementOperators sniff & fixesAdd SlevomatCodingStandard.Operators.DisallowIncrementAndDecrementOperators sniffAdd SlevomatCodingStandard.Operators.DisallowEqualOperators sniff & fixesAdd SlevomatCodingStandard.PHP.DisallowDirectMagicInvokeCall sniffAdd SlevomatCodingStandard.ControlStructures.DisallowEmpty sniffAdd SlevomatCodingStandard.ControlStructures.DisallowContinueWithoutIntegerOperandInSwitch sniffAdd SlevomatCodingStandard.ControlStructures.AssignmentInCondition sniff & fixesAdd SlevomatCodingStandard.Classes.UselessLateStaticBinding sniffAdd SlevomatCodingStandard.Classes.DisallowLateStaticBindingForConstants sniffAdd SlevomatCodingStandard.Arrays.DisallowImplicitArrayCreation sniffAdd SlevomatCodingStandard.Arrays.TrailingArrayComma sniffAdd SlevomatCodingStandard.Commenting.EmptyComment sniffAdd SlevomatCodingStandard.Classes.UnusedPrivateElements sniffAdd Composer script for explaining the code style standardAdd slevomat CS as dev dependencyPSR-2 CS fixesAdd PHPCS configuration fileUse PHPCS 3.5+ instead of 2.8+Ignoring any XML files in the rootRename file according to class nameMerge masterMerge branch 'master' of https://github.com/phpmd/phpmd into feature/simple-rule-testingAbstractTest Cleanup (#758)Revert grammar fix until a formula closer to the intent can be foundImprove wording & divide short & long descriptionAdd PHPDoc for getApplyingFiles and getNotApplyingFilesAdd PHPDoc for UndefinedVariableTest Rename methodsMerge branch 'master' of https://github.com/phpmd/phpmd into feature/simple-rule-testingMerge pull request #750 from phpmd/getStringProperty-fallbackMerge pull request #757 from phpmd/report-mock-convenience-methodsMerge pull request #754 from phpmd/feature/720-foreach-pass-by-referenceRefactor array_map duplicated codeKeep main test methods inside the test case sub-classFix PHPDocFix PHPDocFix unit testsRemove duplicated codeImplement expectRuleInvokesForFile methodCleanup PHPDocReplace magic literalsMake use of AbstractTest::getReportWithNoViolation()Make use of AbstractTest::getReportWithOneViolation()Add three report mock methods to improve test case readabilityEase testing of simple rule standard casesAdd more tests for #720 undefined variable in foreach loopsRecord variable definitions from unary expressions in foreach statementsAdd a test for #720 undefined variable in foreach when passed by referenceMerge pull request #752 from phpmd/config-file-code-redundancyMerge pull request #749 from tuc0w/feature/add_allow-underscore_optionAdd tests for controversial rule CamelCaseParameterNameRemove code redundancy around config file pathAlso cover code in newly extracted methodFixed scrutinizerReduce code duplicationAdjust testImprove fallback bevavior of getIntProperty()Improve fallback behavior of getBooleanProperty()Merge pull request #748 from phpmd/rule-pr-checklistsFixed typosDefine strictness of in_array haystack searchAdd fallback value for AbstractRule::getBooleanProperty()Add fallback value for AbstractRule::getIntProperty()Move test case to its siblingAdd fallback value for AbstractRule::getStringProperty()Set coverage annotationAdd variable type hintAdd properties to site rstRemove blank lineAdd allow-underscore rulesAdd allow-underscore propertiesAdd allow-underscore testAdd guiding check lists...Merge pull request #743 from sveneld/bugfix-235Move additional explanation to long descriptionAdd comments to explain how it is testedexclude pattern documentation exclude pattern unittestOutput for version #235Add SECURITY.md file to complete the Tidelift taskMerge pull request #719 from phpmd/feature/issue-717-restore-rst-files-in-websiteMerge pull request #737 from ondrejfuhrer/ondrejfuhrer-static-access-allow-custom-exclusionAdd the release date for 2.8.2 (#736)Allowing custom exclusion for StaticAccess by extending the classMerge pull request #733 from kylekatarnls/feature/732-serializeFix #732: Add unit testsFix #732: Add __serialize and __unserialize to list of ignored methods for camelCase rule.Use GitHub link for raw filesFix #717 Restore RST files on phpmd.orgMerge pull request #716 from phpmd/release-2.8.1Add 2.8.1 changelogBump version to 2.8.1Merge pull request #713 from phpmd-team/feature/fix-travis-ci-failure-detectionMerge branch 'master' into feature/fix-travis-ci-failure-detectionMerge pull request #711 from phpmd-team/feature/website-builder-libraryMerge branch 'master' of https://github.com/phpmd/phpmd into feature/website-builder-libraryUpdate easy-doc/easy-docUse EasyDoc publishPhar optionUse scrutinizer default tests settingsUse stronger typing in PhpMdEnvironment classAdd publishPhar script fo @tvbeek in the composer build-website commandMerge pull request #709 from kylekatarnls/realease-2.8.0Merge pull request #707 from BackEndTea/patch-1Merge pull request #712 from phpmd/release-2.8.0Bump library version to 2.8.0Merge branch 'master' of https://github.com/phpmd/phpmd into feature/website-builder-libraryMute phpunit-mock-objects PHP 7.4 deprecations[Travis-CI] Fix unit tests failure detectionRequire pdepend 2.6Merge pull request #708 from phpmd-team/release-2.8.0Merge pull request #682 from TJVB/add_the_latest_phar_file_to_the_github_pagesTest against php 7.4Add missing #659 PRMerge pull request #696 from phated/masterMerge pull request #690 from TJVB/add_issue_template_configComplete 2.8.0 changelogMerge pull request #697 from phated/patch-1Add the documentation for the UndefinedVariable rule (Resolves #693 ) (#698)Added #657 mentionAdd enterprise language to READMEAdd "For Enterprise" page to PHPMD websiteAdd keyword to let merged PRs auto-close issues (#695)Start PHPMD 2.8.0 changelogChangeChat to ustoChat with us(#691)Update the about text with feedback from the team.Add the new issue template config for GitHub See: https://github.blog/changelog/2019-10-28-new-issue-template-configuration-options/Remove code that I forgot to remove during the refactoring.Rebuild the phar script to get correct latest versionsMerge pull request #685 from ravage84/short-variable-apply-improvementsMerge pull request #686 from phpmd/ignore-all-phar-filesIgnore all PHAR files, not just Compoer'sSplit long descriptions into separate short and long descriptionsRename method applyNoClass to applyNonClassMerge pull request #644 from TJVB/fixing_phpmd_errorsMerge pull request #684 from phpmd/kylekatarnls-patch-1Correct readme explaining ruleset and report. (#679)Fix arrival version of UndefinedVariableMerge branch 'master' into issue-62Merge branch 'master' into feature/website-builder-libraryUse the GitHub api for the latest versionRemove the spaces around dots for consistency.Add the latest phar file to the GitHub Pages to have a consistent url.Merge pull request #680 from TJVB/add_a_pull_request_templateAdd a Pull Request template to guide new Pull RequestsMerge pull request #677 from kylekatarnls/feature/issue-676-fix-count-in-loop-method-suppressAdd unit tests for #676Fix #676 Handle @SuppressWarnings on each method for CountInLoopMerge pull request #671 from kylekatarnls/feature/fix-661-assignations-in-closuresFix #661 Scan only the statement inner expression (first one)Merge pull request #668 from kylekatarnls/feature/fix-661-assignations-in-closuresIgnore the long name (renaming was not a good solution for this one)Revert "Fix a naming rule error"Merge pull request #669 from jakzal/phpdbgReplace xdebug with phpdbg for code coverageSave a trim call by using a variable. (Thanks to @kylekatarnls for the feedback)Add missing docblocks to the new private functions in ShortVariable (thanks to @ravage84 )Fix #661 Exclude FunctionPostfix from IfStatementAssignmentMerge pull request #497 from raa0121/add-undefined-variableFix the ShortVariable RuleFix a naming rule errorFix the cleancode rulesMerge pull request #667 from TJVB/remove_the_prefer-lowest_while_building_the_phar_fileBump to minimal version 5.3.9Remove the --prefer-lowest and add a platform config to the composer commands for building the phar fileMerge pull request #666 from phpmd/feature/fix-readme-linkFix readme link RST formatMerge pull request #665 from kylekatarnls/feature/fix-table-alignmentFix documentation table misalignmentMerge pull request #662 from phpmd/feature/document-ignore-namespacesAdd 'ignore-namespaces' option documentationMerge pull request #571 from tariq86/fixes/dev-frag-ns-ignoreMerge pull request #641 from szepeviktor/level2Merge pull request #660 from kylekatarnls/feature/tidelift-linkAdd link to TideliftMerge pull request #653 from phpmd/phar-composer-optimize-autoloaderMerge pull request #659 from SailorMax/patch-1Update RuleSetFactory.phpMerge pull request #657 from Jir4/fix-ifstatementassigment-messageOptimize Composer autoloader when building PharMerge pull request #656 from kylekatarnls/feature/phpunit-update-deprecated-usagesRemove PHPUnit debugging groupFix IfStatementAssignment violation messageMerge branch 'fix-php5.3-compat' of https://github.com/phpmd/phpmd into feature/phpunit-update-deprecated-usagesMute PHP 5.3 bug due to strlen()Mock explicitly argumentsCreate all mocks with getMockBuilder()Update deprecated callsUse traditional array syntax to keep PHP 5.3 compatibilityMerge pull request #652 from kylekatarnls/feature/650-fix-tablesMerge pull request #595 from PtrTn/ansi-report-rendererFix code styleFix PHP 5.3 compatibilityFix PHP compatibilityFix PHP < 5.5 compatibilityFix #650 Use pretty tables to handle multi-lineFix #650 Use pretty tables to handle multi-lineMerge branch 'master' of https://github.com/phpmd/phpmd into feature/website-builder-libraryMerge pull request #651 from TJVB/fix_issue_templatesMerge pull request #648 from phpmd/IfStatementAssignment-short-descriptionMove the issue templates to the ISSUE_TEMPLATE directory in the .github directory to make them available. (#649)Move IfStatementAssignment to CleanCodeMerge pull request #646 from kylekatarnls/feature/missing-imports-documentationAdd MissingImport link in the menuMerge pull request #645 from poplary/fix-docMerge branch 'master' of https://github.com/phpmd/phpmd into feature/website-builder-libraryFix underlining length for DuplicatedArrayKeyFix the doc of Clean Code Rules.Allow to add scripts like some Matomo analytics via ENV variableMerge pull request #555 from AJenbo/masterMerge pull request #635 from TJVB/issue_templatesRemove renderers countMerge branch 'master' of https://github.com/phpmd/phpmd into fixes/dev-frag-ns-ignoreAdd .github to the .gitattributes file to prevent it from being downloaded by composer.Add the Operating system and version to the Bug template.Be more explicit"our" -> "in the"Add "only"Improve wordingImprove wordingMerge branch 'master' of https://github.com/phpmd/phpmd into feature/x-debugUpdate the issue templates with the feedback from @ravage84Merge branch 'master' of https://github.com/phpmd/phpmd into add-undefined-variableMove UndefinedVariable into CleanCode namespaceMerge branch 'master' of https://github.com/phpmd/phpmd into ansi-report-rendererVerify undefined variables does not apply to superglobalsFix PHPDoc blocks phpstan Level 2URL in capital lettersAdjusted wording, added missing word2 typos with thanks to @kylekatarnls for the feedback.Add a first version of the documentation and website issue templateAdd a first version of the question issue templateAdd a first version of the feature proposal issue templateAdd a first version of the bug issue templateFix URLsFix CHANGELOG dumpFix absolute linksFix level 4 titlesMerge branch 'master' of https://github.com/kylekatarnls/phpmd into feature/website-builder-libraryUse easy-doc/easy-doc instead of custom scriptPrepare config for the website builder libraryCleaned up a bit.Cast a possible "false" value from SplFileObject::current() into string.Merge remote-tracking branch 'upstream/master'Update src/main/php/PHPMD/Renderer/HTMLRenderer.phpAdd a renderer which writes an ansi report stringRevert Appveyor changes in branchFollow existing codebase conventions (no _ for privates)Add 'ignore-namespaces' to DevelopmentCodeFragment ruleUpdate PHPUnit to fix failing testFix AppVeyor path exportFix Appveyor buildUse XdebugHandler to avoid performance issuesRemoved "string" typehint (PHP<7 compatibility).HTML renderer fixed BC with PHP 5.4 (no const arrays) + adapted tests.Old HTML renderer replaced with the new one.Reworked the new HTML renderer. - Report's sub title contains more information about report and machine. - New "summary" section with summaries calculated across: - Problem's priority - PHP namespace - Rule set - Rule name - Different design. - File paths act as links. - Deep linking possible, problem index acts as an anchor. - Details are hidden by default - can be toggled. - Attempt at printer friendly CSS media-queries.Fixed tests for now.Display violation's index, too + tweaked styling a bit. - Line number is now more highlighter.Added a new "pretty HTML" renderer.Support Undefined VariableCloses phpmd/phpmd#62Working with this branch
If you find you need to update the codebase to be able to merge this branch (for example update some tests or rebuild some assets), please note that violinist will force push to this branch to keep it up to date. This means you should not work on this branch directly, since you might lose your work. Read more about branches created by violinist.io here.
This is an automated pull request from Violinist: Continuously and automatically monitor and update your composer dependencies. Have ideas on how to improve this message? All violinist messages are open-source, and can be improved here.