[TASK] Add rule to prevent the @access annotation entering the core 97/58497/4
authorWouter Wolters <typo3@wouterwolters.nl>
Sun, 30 Sep 2018 13:19:14 +0000 (15:19 +0200)
committerAnja Leichsenring <aleichsenring@ab-softlab.de>
Sun, 30 Sep 2018 13:59:47 +0000 (15:59 +0200)
Add the phpdoc_no_access rule to the php-cs-fixer configuration to
prevent the @access annotation from entering the core again.

Resolves: #86475
Related: #86446
Releases: master
Change-Id: I9db729ae1375434bea49f92468f7d490a554b4fb
Reviewed-on: https://review.typo3.org/58497
Reviewed-by: Christian Kuhn <lolli@schwarzbu.ch>
Tested-by: Christian Kuhn <lolli@schwarzbu.ch>
Reviewed-by: Stefan Neufeind <typo3.neufeind@speedpartner.de>
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
Tested-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
Build/.php_cs
Build/Scripts/annotationChecker.php

index 53acb2a..03c5f57 100644 (file)
@@ -87,5 +87,6 @@ return PhpCsFixer\Config::create()
         'cast_spaces' => ['space' => 'none'],
         'declare_equal_normalize' => ['space' => 'single'],
         'dir_constant' => true,
+        'phpdoc_no_access' => true
     ])
     ->setFinder($finder);
index a4144ba..a446134 100755 (executable)
@@ -53,7 +53,7 @@ class NodeVisitor extends NodeVisitorAbstract
                     // Widely used tags (but not existent in phpdoc)
                     'fix', 'fixme', 'override',
                     // PHPDocumentor 1 tags
-                    'abstract', 'access', 'code', 'deprec', 'endcode', 'exception', 'final', 'ingroup', 'inheritdoc', 'inheritDoc', 'magic', 'name', 'toc', 'tutorial', 'private', 'static', 'staticvar', 'staticVar', 'throw',
+                    'abstract', 'code', 'deprec', 'endcode', 'exception', 'final', 'ingroup', 'inheritdoc', 'inheritDoc', 'magic', 'name', 'toc', 'tutorial', 'private', 'static', 'staticvar', 'staticVar', 'throw',
                     // PHPDocumentor 2 tags
                     'api', 'author', 'category', 'copyright', 'deprecated', 'example', 'filesource', 'global', 'ignore', 'internal', 'license', 'link', 'method', 'package', 'param', 'property', 'property-read', 'property-write', 'return', 'see', 'since', 'source', 'subpackage', 'throws', 'todo', 'TODO', 'usedby', 'uses', 'var', 'version',
                     // PHPUnit tags