← Back to branch summary

~ubuntu-branches/ubuntu/vivid/php-codesniffer/vivid

« back to all changes in this revision

Viewing changes to PHP_CodeSniffer-1.5.5/CodeSniffer/Standards/Squiz/Sniffs/Classes/DuplicatePropertySniff.php

  • Committer: Package Import Robot
  • Author(s): David Prévot, Greg Sherwood, Alexey, Emily, David Prévot
  • Date: 2014-09-26 13:44:35 UTC
  • mfrom: (1.1.6)
  • Revision ID: package-import@ubuntu.com-20140926134435-wvjq16miqq4d60y0
Tags: 1.5.5-1
[ Greg Sherwood ]
* Improved closure support in Generic ScopeIndentSniff
* Improved indented PHP tag support in Generic ScopeIndentSniff
* Standards can now be located within hidden directories
 (further fix for bug #20323)
* Fixed bug #20373 : Inline comment sniff tab handling way
* Fixed bug #20378 : Report appended to existing file if no errors
  found in run
* Fixed bug #20381 : Invalid "Comment closer must be on a new line"
* PHP tokenizer no longer converts class/function names to special
  tokens types
* Fixed bug #20386 : Squiz.Commenting.ClassComment.SpacingBefore
  thrown if first block comment
* Squiz and PEAR FunctionCommentSnif now support _()
* PEAR ValidFunctionNameSniff no longer throws an error for _()
* Fixed bug #248 : FunctionCommentSniff expects ampersand on param name
* Fixed bug #248 in Squiz sniff as well
* Fixed bug #265 : False positives with type hints in ForbiddenFunctionsSniff
* Prepare for 1.5.5 release

[ Alexey ]
* Allowed single undersored methods and functions

[ Emily ]
* Added var_dump to discouraged functions sniff

[ David Prévot ]
* Revert "Add XS-Testsuite still needed for ci.d.n"
* Add self to uploaders
* Bump standards version to 3.9.6

Show diffs side-by-side

added added

removed removed

Lines of Context:
PHP_CodeSniffer-1.5.5/CodeSniffer/Standards/Squiz/Sniffs/Classes/DuplicatePropertySniff.php
 
1
<?php
 
2
/**
 
3
 * Squiz_Sniffs_Classes_DuplicatePropertySniff.
 
4
 *
 
5
 * PHP version 5
 
6
 *
 
7
 * @category  PHP
 
8
 * @package   PHP_CodeSniffer
 
9
 * @author    Greg Sherwood <gsherwood@squiz.net>
 
10
 * @copyright 2006-2014 Squiz Pty Ltd (ABN 77 084 670 600)
 
11
 * @license   https://github.com/squizlabs/PHP_CodeSniffer/blob/master/licence.txt BSD Licence
 
12
 * @link      http://pear.php.net/package/PHP_CodeSniffer
 
13
 */
 
14
 
 
15
/**
 
16
 * Squiz_Sniffs_Classes_DuplicatePropertySniff.
 
17
 *
 
18
 * Ensures JS classes don't contain duplicate property names.
 
19
 *
 
20
 * @category  PHP
 
21
 * @package   PHP_CodeSniffer
 
22
 * @author    Greg Sherwood <gsherwood@squiz.net>
 
23
 * @copyright 2006-2014 Squiz Pty Ltd (ABN 77 084 670 600)
 
24
 * @license   https://github.com/squizlabs/PHP_CodeSniffer/blob/master/licence.txt BSD Licence
 
25
 * @version   Release: 1.5.5
 
26
 * @link      http://pear.php.net/package/PHP_CodeSniffer
 
27
 */
 
28
class Squiz_Sniffs_Classes_DuplicatePropertySniff implements PHP_CodeSniffer_Sniff
 
29
{
 
30
 
 
31
    /**
 
32
     * A list of tokenizers this sniff supports.
 
33
     *
 
34
     * @var array
 
35
     */
 
36
    public $supportedTokenizers = array('JS');
 
37
 
 
38
 
 
39
    /**
 
40
     * Returns an array of tokens this test wants to listen for.
 
41
     *
 
42
     * @return array
 
43
     */
 
44
    public function register()
 
45
    {
 
46
        return array(T_OBJECT);
 
47
 
 
48
    }//end register()
 
49
 
 
50
 
 
51
    /**
 
52
     * Processes this test, when one of its tokens is encountered.
 
53
     *
 
54
     * @param PHP_CodeSniffer_File $phpcsFile The current file being processed.
 
55
     * @param int                  $stackPtr  The position of the current token in the
 
56
     *                                        stack passed in $tokens.
 
57
     *
 
58
     * @return void
 
59
     */
 
60
    public function process(PHP_CodeSniffer_File $phpcsFile, $stackPtr)
 
61
    {
 
62
        $tokens = $phpcsFile->getTokens();
 
63
        $start  = $tokens[$stackPtr]['scope_opener'];
 
64
        $end    = $tokens[$stackPtr]['scope_closer'];
 
65
 
 
66
        $properties   = array();
 
67
        $wantedTokens = array(
 
68
                         T_PROPERTY,
 
69
                         T_OPEN_CURLY_BRACKET,
 
70
                        );
 
71
 
 
72
        $next = $phpcsFile->findNext($wantedTokens, ($start + 1), $end);
 
73
        while ($next !== false && $next < $end) {
 
74
            // Skip nested objects.
 
75
            if ($tokens[$next]['code'] === T_OPEN_CURLY_BRACKET) {
 
76
                $next = $tokens[$next]['bracket_closer'];
 
77
            } else {
 
78
                $propName = $tokens[$next]['content'];
 
79
                if (isset($properties[$propName]) === true) {
 
80
                    $error = 'Duplicate property definition found for "%s"; previously defined on line %s';
 
81
                    $data  = array(
 
82
                              $propName,
 
83
                              $tokens[$properties[$propName]]['line'],
 
84
                             );
 
85
                    $phpcsFile->addError($error, $next, 'Found', $data);
 
86
                }
 
87
 
 
88
                $properties[$propName] = $next;
 
89
            }//end if
 
90
 
 
91
            $next = $phpcsFile->findNext($wantedTokens, ($next + 1), $end);
 
92
        }//end while
 
93
 
 
94
    }//end process()
 
95
 
 
96
 
 
97
}//end class
 
98
 
 
99
 
 
100
?>