Skip to content

Commit 5179d83

Browse files
rodrigoprimojrfnl
authored andcommitted
Add links to all issues in the 2.6.2, 2.6.1 and 2.6.0 sections of the changelog
Also fixes a typo.
1 parent ba02444 commit 5179d83

File tree

1 file changed

+120
-59
lines changed

1 file changed

+120
-59
lines changed

CHANGELOG.md

Lines changed: 120 additions & 59 deletions
Original file line numberDiff line numberDiff line change
@@ -3255,36 +3255,55 @@ Additionally, thanks to [Alexander Turek][@derrabus] for consulting on the repo
32553255

32563256
## [2.6.2] - 2016-07-14
32573257
### Changed
3258-
- Added a new --exclude CLI argument to exclude a list of sniffs from checking and fixing (request #904)
3258+
- Added a new --exclude CLI argument to exclude a list of sniffs from checking and fixing (request [#904])
32593259
- Accepts the same sniff codes as the --sniffs command line argument, but provides the opposite functionality
3260-
- Added a new -q command line argument to disable progress and verbose information from being printed (request #969)
3260+
- Added a new -q command line argument to disable progress and verbose information from being printed (request [#969])
32613261
- Useful if a coding standard hard-codes progress or verbose output but you want PHPCS to be quiet
32623262
- Use the command "phpcs --config-set quiet true" to turn quiet mode on by default
3263-
- Generic LineLength sniff no longer errors for comments that cannot be broken out onto a new line (request #766)
3263+
- Generic LineLength sniff no longer errors for comments that cannot be broken out onto a new line (request [#766])
32643264
- A typical case is a comment that contains a very long URL
3265-
- The comment is ignored if putting the URL on a indented new comment line would be longer than the allowed length
3265+
- The comment is ignored if putting the URL on an indented new comment line would be longer than the allowed length
32663266
- Settings extensions in a ruleset no longer causes PHP notices during unit testing
32673267
- Thanks to [Klaus Purer][@klausi] for the patch
32683268
- Version control reports now show which errors are fixable if you are showing sources
3269-
- Added a new sniff to enforce a single space after a NOT operator (request #1051)
3269+
- Added a new sniff to enforce a single space after a NOT operator (request [#1051])
32703270
- Include in a ruleset using the code Generic.Formatting.SpaceAfterNot
3271-
- The Squiz.Commenting.BlockComment sniff now supports tabs for indenting comment lines (request #1056)
3271+
- The Squiz.Commenting.BlockComment sniff now supports tabs for indenting comment lines (request [#1056])
32723272

32733273
### Fixed
3274-
- Fixed bug #790 : Incorrect missing @throws error in methods that use closures
3275-
- Fixed bug #908 : PSR2 standard is not checking that closing brace is on line following the body
3276-
- Fixed bug #945 : Incorrect indent behavior using deep-nested function and arrays
3277-
- Fixed bug #961 : Two anonymous functions passed as function/method arguments cause indentation false positive
3278-
- Fixed bug #1005 : Using global composer vendor autoload breaks PHP lowercase built-in function sniff
3274+
- Fixed bug [#790] : Incorrect missing @throws error in methods that use closures
3275+
- Fixed bug [#908] : PSR2 standard is not checking that closing brace is on line following the body
3276+
- Fixed bug [#945] : Incorrect indent behavior using deep-nested function and arrays
3277+
- Fixed bug [#961] : Two anonymous functions passed as function/method arguments cause indentation false positive
3278+
- Fixed bug [#1005] : Using global composer vendor autoload breaks PHP lowercase built-in function sniff
32793279
- Thanks to [Michael Butler][@michaelbutler] for the patch
3280-
- Fixed bug #1007 : Squiz Unreachable code detection is not working properly with a closure inside a case
3281-
- Fixed bug #1023 : PSR2.Classes.ClassDeclaration fails if class extends base class and "implements" is on trailing line
3282-
- Fixed bug #1026 : Arrays in comma delimited class properties cause ScopeIndent to increase indent
3283-
- Fixed bug #1028 : Squiz ArrayDeclaration incorrectly fixes multi-line array where end bracket is not on a new line
3284-
- Fixed bug #1034 : Squiz FunctionDeclarationArgumentSpacing gives incorrect error when first arg is a variadic
3285-
- Fixed bug #1036 : Adjacent assignments aligned analysis statement wrong
3286-
- Fixed bug #1049 : Version control reports can show notices when the report width is very small
3287-
- Fixed bug #21050 : PEAR MultiLineCondition sniff suppresses errors on last condition line
3280+
- Fixed bug [#1007] : Squiz Unreachable code detection is not working properly with a closure inside a case
3281+
- Fixed bug [#1023] : PSR2.Classes.ClassDeclaration fails if class extends base class and "implements" is on trailing line
3282+
- Fixed bug [#1026] : Arrays in comma delimited class properties cause ScopeIndent to increase indent
3283+
- Fixed bug [#1028] : Squiz ArrayDeclaration incorrectly fixes multi-line array where end bracket is not on a new line
3284+
- Fixed bug [#1034] : Squiz FunctionDeclarationArgumentSpacing gives incorrect error when first arg is a variadic
3285+
- Fixed bug [#1036] : Adjacent assignments aligned analysis statement wrong
3286+
- Fixed bug [#1049] : Version control reports can show notices when the report width is very small
3287+
- Fixed bug [#21050] : PEAR MultiLineCondition sniff suppresses errors on last condition line
3288+
3289+
[#766]: https://github.com/squizlabs/PHP_CodeSniffer/issues/766
3290+
[#790]: https://github.com/squizlabs/PHP_CodeSniffer/issues/790
3291+
[#904]: https://github.com/squizlabs/PHP_CodeSniffer/issues/904
3292+
[#908]: https://github.com/squizlabs/PHP_CodeSniffer/issues/908
3293+
[#945]: https://github.com/squizlabs/PHP_CodeSniffer/issues/945
3294+
[#961]: https://github.com/squizlabs/PHP_CodeSniffer/issues/961
3295+
[#969]: https://github.com/squizlabs/PHP_CodeSniffer/issues/969
3296+
[#1005]: https://github.com/squizlabs/PHP_CodeSniffer/issues/1005
3297+
[#1007]: https://github.com/squizlabs/PHP_CodeSniffer/issues/1007
3298+
[#1023]: https://github.com/squizlabs/PHP_CodeSniffer/issues/1023
3299+
[#1026]: https://github.com/squizlabs/PHP_CodeSniffer/issues/1026
3300+
[#1028]: https://github.com/squizlabs/PHP_CodeSniffer/issues/1028
3301+
[#1034]: https://github.com/squizlabs/PHP_CodeSniffer/issues/1034
3302+
[#1036]: https://github.com/squizlabs/PHP_CodeSniffer/issues/1036
3303+
[#1049]: https://github.com/squizlabs/PHP_CodeSniffer/pull/1049
3304+
[#1051]: https://github.com/squizlabs/PHP_CodeSniffer/issues/1051
3305+
[#1056]: https://github.com/squizlabs/PHP_CodeSniffer/issues/1056
3306+
[#21050]: https://pear.php.net/bugs/bug.php?id=21050
32883307

32893308
## [2.6.1] - 2016-05-31
32903309
### Changed
@@ -3293,43 +3312,60 @@ Additionally, thanks to [Alexander Turek][@derrabus] for consulting on the repo
32933312
- Thanks to [Jaroslav Hanslík][@kukulich] for the patch
32943313
- The default_standard config setting now allows multiple standards to be listed, like on the command line
32953314
- Thanks to [Michael Mayer][@schnittstabil] for the patch
3296-
- Installations done via composer now only include the composer autoloader for PHP 5.3.2+ (request #942)
3315+
- Installations done via composer now only include the composer autoloader for PHP 5.3.2+ (request [#942])
32973316
- Added a rollbackChangeset() method to the Fixer class to purposely rollback the active changeset
32983317

32993318
### Fixed
3300-
- Fixed bug #940 : Auto-fixing issue encountered with inconsistent use of braces
3301-
- Fixed bug #943 : Squiz.PHP.InnerFunctions.NotAllowed reported in anonymous classes
3302-
- Fixed bug #944 : PHP warning when running the latest phar
3303-
- Fixed bug #951 : InlineIfDeclaration: invalid error produced with UTF-8 string
3304-
- Fixed bug #957 : Operator spacing sniff errors when plus is used as part of a number
3319+
- Fixed bug [#940] : Auto-fixing issue encountered with inconsistent use of braces
3320+
- Fixed bug [#943] : Squiz.PHP.InnerFunctions.NotAllowed reported in anonymous classes
3321+
- Fixed bug [#944] : PHP warning when running the latest phar
3322+
- Fixed bug [#951] : InlineIfDeclaration: invalid error produced with UTF-8 string
3323+
- Fixed bug [#957] : Operator spacing sniff errors when plus is used as part of a number
33053324
- Thanks to [Klaus Purer][@klausi] for the patch
3306-
- Fixed bug #959 : Call-time pass-by-reference false positive if there is a square bracket before the ampersand
3325+
- Fixed bug [#959] : Call-time pass-by-reference false positive if there is a square bracket before the ampersand
33073326
- Thanks to [Konstantin Leboev][@realmfoo] for the patch
3308-
- Fixed bug #962 : Null coalescing operator (??) not detected as a token
3327+
- Fixed bug [#962] : Null coalescing operator (??) not detected as a token
33093328
- Thanks to [Joel Posti][@joelposti] for the patch
3310-
- Fixed bug #973 : Anonymous class declaration and PSR1.Files.SideEffects.FoundWithSymbols
3311-
- Fixed bug #974 : Error when file ends with "function"
3312-
- Fixed bug #979 : Anonymous function with return type hint is not refactored as expected
3313-
- Fixed bug #983 : Squiz.WhiteSpace.MemberVarSpacing.AfterComment fails to fix error when comment is not a docblock
3314-
- Fixed bug #1010 : Squiz NonExecutableCode sniff does not detect boolean OR
3329+
- Fixed bug [#973] : Anonymous class declaration and PSR1.Files.SideEffects.FoundWithSymbols
3330+
- Fixed bug [#974] : Error when file ends with "function"
3331+
- Fixed bug [#979] : Anonymous function with return type hint is not refactored as expected
3332+
- Fixed bug [#983] : Squiz.WhiteSpace.MemberVarSpacing.AfterComment fails to fix error when comment is not a docblock
3333+
- Fixed bug [#1010] : Squiz NonExecutableCode sniff does not detect boolean OR
33153334
- Thanks to [Derek Henderson][@2shediac] for the patch
3316-
- Fixed bug #1015 : The Squiz.Commenting.FunctionComment sniff doesn't allow description in @return tag
3335+
- Fixed bug [#1015] : The Squiz.Commenting.FunctionComment sniff doesn't allow description in @return tag
33173336
- Thanks to [Alexander Obuhovich][@aik099] for the patch
3318-
- Fixed bug #1022 : Duplicate spaces after opening bracket error with PSR2 standard
3319-
- Fixed bug #1025 : Syntax error in JS file can cause undefined index for parenthesis_closer
3337+
- Fixed bug [#1022] : Duplicate spaces after opening bracket error with PSR2 standard
3338+
- Fixed bug [#1025] : Syntax error in JS file can cause undefined index for parenthesis_closer
3339+
3340+
[#940]: https://github.com/squizlabs/PHP_CodeSniffer/issues/940
3341+
[#942]: https://github.com/squizlabs/PHP_CodeSniffer/issues/942
3342+
[#943]: https://github.com/squizlabs/PHP_CodeSniffer/issues/943
3343+
[#944]: https://github.com/squizlabs/PHP_CodeSniffer/issues/944
3344+
[#951]: https://github.com/squizlabs/PHP_CodeSniffer/issues/951
3345+
[#957]: https://github.com/squizlabs/PHP_CodeSniffer/pull/957
3346+
[#959]: https://github.com/squizlabs/PHP_CodeSniffer/issues/959
3347+
[#962]: https://github.com/squizlabs/PHP_CodeSniffer/issues/962
3348+
[#973]: https://github.com/squizlabs/PHP_CodeSniffer/issues/973
3349+
[#974]: https://github.com/squizlabs/PHP_CodeSniffer/issues/974
3350+
[#979]: https://github.com/squizlabs/PHP_CodeSniffer/issues/979
3351+
[#983]: https://github.com/squizlabs/PHP_CodeSniffer/issues/983
3352+
[#1010]: https://github.com/squizlabs/PHP_CodeSniffer/issues/1010
3353+
[#1015]: https://github.com/squizlabs/PHP_CodeSniffer/issues/1015
3354+
[#1022]: https://github.com/squizlabs/PHP_CodeSniffer/issues/1022
3355+
[#1025]: https://github.com/squizlabs/PHP_CodeSniffer/issues/1025
33203356

33213357
## [2.6.0] - 2016-04-04
33223358
### Changed
3323-
- Paths used when setting CLI arguments inside ruleset.xml files are now relative to the ruleset location (request #847)
3359+
- Paths used when setting CLI arguments inside ruleset.xml files are now relative to the ruleset location (request [#847])
33243360
- This change only applies to paths within ARG tags, used to set CLI arguments
33253361
- Previously, the paths were relative to the directory PHPCS was being run from
33263362
- Absolute paths are still allowed and work the same way they always have
33273363
- This change allows ruleset.xml files to be more portable
33283364
- Content passed via STDIN will now be processed even if files are specified on the command line or in a ruleset
3329-
- When passing content via STDIN, you can now specify the file path to use on the command line (request #934)
3365+
- When passing content via STDIN, you can now specify the file path to use on the command line (request [#934])
33303366
- This allows sniffs that check file paths to work correctly
33313367
- This is the same functionality provided by the phpcs_input_file line, except it is available on the command line
3332-
- Files processed with custom tokenizers will no longer be skipped if they appear minified (request #877)
3368+
- Files processed with custom tokenizers will no longer be skipped if they appear minified (request [#877])
33333369
- If the custom tokenizer wants minified files skipped, it can set a $skipMinified member var to TRUE
33343370
- See the included JS and CSS tokenizers for an example
33353371
- Config vars set in ruleset.xml files are now processed earlier, allowing them to be used during sniff registration
@@ -3340,12 +3376,12 @@ Additionally, thanks to [Alexander Turek][@derrabus] for consulting on the repo
33403376
- You can still manually set php_path to use a specific binary for testing
33413377
- Thanks to [Andrew Berry][@deviantintegral] for the patch
33423378
- The PHP-supplied T_POW_EQUAL token has been replicated for PHP versions before 5.6
3343-
- Added support for PHP7 use group declarations (request #878)
3379+
- Added support for PHP7 use group declarations (request [#878])
33443380
- New tokens T_OPEN_USE_GROUP and T_CLOSE_USE_GROUP are assigned to the open and close curly braces
3345-
- Generic ScopeIndent sniff now reports errors for every line that needs the indent changed (request #903)
3381+
- Generic ScopeIndent sniff now reports errors for every line that needs the indent changed (request [#903])
33463382
- Previously, it ignored lines that were indented correctly in the context of their block
33473383
- This change produces more technically accurate error messages, but is much more verbose
3348-
- The PSR2 and Squiz standards now allow multi-line default values in function declarations (request #542)
3384+
- The PSR2 and Squiz standards now allow multi-line default values in function declarations (request [#542])
33493385
- Previously, these would automatically make the function a multi-line declaration
33503386
- Squiz InlineCommentSniff now allows docblocks on require(_once) and include(_once) statements
33513387
- Thanks to [Gary Jones][@GaryJones] for the patch
@@ -3357,31 +3393,56 @@ Additionally, thanks to [Alexander Turek][@derrabus] for consulting on the repo
33573393
- Renamed Squiz.CSS.Opacity.SpacingAfterPoint to Squiz.CSS.Opacity.DecimalPrecision
33583394
- Please update your ruleset if you are referencing this error code directly
33593395
- Fixed PHP tokenizer problem that caused an infinite loop when checking a comment with specific content
3360-
- Generic Disallow Space and Tab indent sniffs now detect and fix indents inside embedded HTML chunks (request #882)
3396+
- Generic Disallow Space and Tab indent sniffs now detect and fix indents inside embedded HTML chunks (request [#882])
33613397
- Squiz CSS IndentationSniff no longer assumes the class opening brace is at the end of a line
33623398
- Squiz FunctionCommentThrowTagSniff now ignores non-docblock comments
33633399
- Squiz ComparisonOperatorUsageSniff now allows conditions like while(true)
33643400
- PEAR FunctionCallSignatureSniff (and the Squiz and PSR2 sniffs that use it) now correctly check the first argument
3365-
- Further fix for bug #698
3401+
- Further fix for bug [#698]
33663402

33673403
### Fixed
3368-
- Fixed bug #791 : codingStandardsChangeSetting settings not working with namespaces
3369-
- Fixed bug #872 : Incorrect detection of blank lines between CSS class names
3370-
- Fixed bug #879 : Generic InlineControlStructureSniff can create parse error when case/if/elseif/else have mixed brace and braceless definitions
3371-
- Fixed bug #883 : PSR2 is not checking for blank lines at the start and end of control structures
3372-
- Fixed bug #884 : Incorrect indentation notice for anonymous classes
3373-
- Fixed bug #887 : Using curly braces for a shared CASE/DEFAULT statement can generate an error in PSR2 SwitchDeclaration
3374-
- Fixed bug #889 : Closure inside catch/else/elseif causes indentation error
3375-
- Fixed bug #890 : Function call inside returned short array value can cause indentation error inside CASE statements
3376-
- Fixed bug #897 : Generic.Functions.CallTimePassByReference.NotAllowed false positive when short array syntax
3377-
- Fixed bug #900 : Squiz.Functions.FunctionDeclarationArgumentSpacing bug when no space between type hint and argument
3378-
- Fixed bug #902 : T_OR_EQUAL and T_POW_EQUAL are not seen as assignment tokens
3379-
- Fixed bug #910 : Unrecognized "extends" and indentation on anonymous classes
3380-
- Fixed bug #915 : JS Tokenizer generates errors when processing some decimals
3381-
- Fixed bug #928 : Endless loop when sniffing a PHP file with a git merge conflict inside a function
3382-
- Fixed bug #937 : Shebang can cause PSR1 SideEffects warning
3404+
- Fixed bug [#791] : codingStandardsChangeSetting settings not working with namespaces
3405+
- Fixed bug [#872] : Incorrect detection of blank lines between CSS class names
3406+
- Fixed bug [#879] : Generic InlineControlStructureSniff can create parse error when case/if/elseif/else have mixed brace and braceless definitions
3407+
- Fixed bug [#883] : PSR2 is not checking for blank lines at the start and end of control structures
3408+
- Fixed bug [#884] : Incorrect indentation notice for anonymous classes
3409+
- Fixed bug [#887] : Using curly braces for a shared CASE/DEFAULT statement can generate an error in PSR2 SwitchDeclaration
3410+
- Fixed bug [#889] : Closure inside catch/else/elseif causes indentation error
3411+
- Fixed bug [#890] : Function call inside returned short array value can cause indentation error inside CASE statements
3412+
- Fixed bug [#897] : Generic.Functions.CallTimePassByReference.NotAllowed false positive when short array syntax
3413+
- Fixed bug [#900] : Squiz.Functions.FunctionDeclarationArgumentSpacing bug when no space between type hint and argument
3414+
- Fixed bug [#902] : T_OR_EQUAL and T_POW_EQUAL are not seen as assignment tokens
3415+
- Fixed bug [#910] : Unrecognized "extends" and indentation on anonymous classes
3416+
- Fixed bug [#915] : JS Tokenizer generates errors when processing some decimals
3417+
- Fixed bug [#928] : Endless loop when sniffing a PHP file with a git merge conflict inside a function
3418+
- Fixed bug [#937] : Shebang can cause PSR1 SideEffects warning
33833419
- Thanks to [Clay Loveless][@claylo] for the patch
3384-
- Fixed bug #938 : CallTimePassByReferenceSniff ignores functions with return value
3420+
- Fixed bug [#938] : CallTimePassByReferenceSniff ignores functions with return value
3421+
3422+
[#542]: https://github.com/squizlabs/PHP_CodeSniffer/issues/542
3423+
[#698]: https://github.com/squizlabs/PHP_CodeSniffer/issues/698
3424+
[#791]: https://github.com/squizlabs/PHP_CodeSniffer/issues/791
3425+
[#847]: https://github.com/squizlabs/PHP_CodeSniffer/issues/847
3426+
[#872]: https://github.com/squizlabs/PHP_CodeSniffer/issues/872
3427+
[#877]: https://github.com/squizlabs/PHP_CodeSniffer/issues/877
3428+
[#878]: https://github.com/squizlabs/PHP_CodeSniffer/issues/878
3429+
[#879]: https://github.com/squizlabs/PHP_CodeSniffer/issues/879
3430+
[#882]: https://github.com/squizlabs/PHP_CodeSniffer/issues/882
3431+
[#883]: https://github.com/squizlabs/PHP_CodeSniffer/issues/883
3432+
[#884]: https://github.com/squizlabs/PHP_CodeSniffer/issues/884
3433+
[#887]: https://github.com/squizlabs/PHP_CodeSniffer/issues/887
3434+
[#889]: https://github.com/squizlabs/PHP_CodeSniffer/issues/889
3435+
[#890]: https://github.com/squizlabs/PHP_CodeSniffer/issues/890
3436+
[#897]: https://github.com/squizlabs/PHP_CodeSniffer/issues/897
3437+
[#900]: https://github.com/squizlabs/PHP_CodeSniffer/issues/900
3438+
[#902]: https://github.com/squizlabs/PHP_CodeSniffer/issues/902
3439+
[#903]: https://github.com/squizlabs/PHP_CodeSniffer/issues/903
3440+
[#910]: https://github.com/squizlabs/PHP_CodeSniffer/issues/910
3441+
[#915]: https://github.com/squizlabs/PHP_CodeSniffer/issues/915
3442+
[#928]: https://github.com/squizlabs/PHP_CodeSniffer/issues/928
3443+
[#934]: https://github.com/squizlabs/PHP_CodeSniffer/issues/934
3444+
[#937]: https://github.com/squizlabs/PHP_CodeSniffer/pull/937
3445+
[#938]: https://github.com/squizlabs/PHP_CodeSniffer/issues/938
33853446

33863447
## [2.5.1] - 2016-01-20
33873448
### Changed

0 commit comments

Comments
 (0)