1 | <?php |
---|
2 | |
---|
3 | /** |
---|
4 | * Sniff for detecting PHP4 constructors. |
---|
5 | */ |
---|
6 | |
---|
7 | /** |
---|
8 | * Warns about classes with PHP4 constructors that don't have PHP5 constructors. |
---|
9 | */ |
---|
10 | class JDGrimes_Sniffs_PHP7_PHP4ConstructorSniff extends JDGrimes_Sniff { |
---|
11 | |
---|
12 | public function register() { |
---|
13 | return array( T_FUNCTION ); |
---|
14 | } |
---|
15 | |
---|
16 | public function process( PHP_CodeSniffer_File $phpcsFile, $stackPtr ) { |
---|
17 | |
---|
18 | $tokens = $phpcsFile->getTokens(); |
---|
19 | |
---|
20 | if ( ! isset( $tokens[ $stackPtr ]['scope_closer'] ) ) { |
---|
21 | return; |
---|
22 | } |
---|
23 | |
---|
24 | $parent = end( $tokens[ $stackPtr ]['conditions'] ); |
---|
25 | |
---|
26 | if ( ! $parent || T_CLASS !== $parent ) { |
---|
27 | return; |
---|
28 | } |
---|
29 | |
---|
30 | $class_ptr = key( $tokens[ $stackPtr ]['conditions'] ); |
---|
31 | |
---|
32 | $method_ptr = $phpcsFile->findNext( T_STRING, $stackPtr + 1 ); |
---|
33 | $method_name = $tokens[ $method_ptr ]['content']; |
---|
34 | |
---|
35 | if ( '__construct' === $method_name ) { |
---|
36 | return $tokens[ $class_ptr ]['scope_closer']; |
---|
37 | } |
---|
38 | |
---|
39 | $class_name = $phpcsFile->getDeclarationName( $class_ptr ); |
---|
40 | |
---|
41 | if ( strtolower( $method_name ) === strtolower( $class_name ) ) { |
---|
42 | |
---|
43 | if ( isset( $phpcsFile->fixer ) ) { |
---|
44 | |
---|
45 | $fix = $phpcsFile->addFixableError( |
---|
46 | 'Found PHP4 constructor not proceeded by a PHP5 constructor.' |
---|
47 | , $stackPtr |
---|
48 | , 'NoPHP5Constructor' |
---|
49 | ); |
---|
50 | |
---|
51 | if ( $fix ) { |
---|
52 | |
---|
53 | $method_end_ptr = $tokens[ $stackPtr ]['scope_closer']; |
---|
54 | |
---|
55 | // Get the list of arguments accepted by the function. |
---|
56 | $args_def = $phpcsFile->getTokensAsString( |
---|
57 | $tokens[ $stackPtr ]['parenthesis_opener'] + 1 |
---|
58 | , $tokens[ $stackPtr ]['parenthesis_closer'] - $tokens[ $stackPtr ]['parenthesis_opener'] - 1 |
---|
59 | ); |
---|
60 | |
---|
61 | if ( ! empty( $args_def ) ) { |
---|
62 | $args_def = ' ' . trim( $args_def ) . ' '; |
---|
63 | $args_only = preg_replace( '/\s?=[^,]*(,)?/', '\1', $args_def ); |
---|
64 | $args_only .= ' '; |
---|
65 | } else { |
---|
66 | $args_only = ''; |
---|
67 | } |
---|
68 | |
---|
69 | $phpcsFile->fixer->beginChangeset(); |
---|
70 | |
---|
71 | // Change the name of the method to __construct(). |
---|
72 | $phpcsFile->fixer->replaceToken( $method_ptr, '__construct' ); |
---|
73 | |
---|
74 | $new_method = "\n\n\t/**"; |
---|
75 | $new_method .= "\n\t * PHP4 constructor."; |
---|
76 | $new_method .= "\n\t */"; |
---|
77 | $new_method .= "\n\tpublic function {$method_name}({$args_def}) {"; |
---|
78 | $new_method .= "\n\t\tself::__construct({$args_only});"; |
---|
79 | $new_method .= "\n\t}"; |
---|
80 | |
---|
81 | $phpcsFile->fixer->addContent( |
---|
82 | $method_end_ptr |
---|
83 | , $new_method |
---|
84 | ); |
---|
85 | /* |
---|
86 | // Add a the PHP4 constructor back, right after the PHP5 one. |
---|
87 | $phpcsFile->fixer->addNewline( $method_end_ptr ); |
---|
88 | $phpcsFile->fixer->addNewline( $method_end_ptr ); |
---|
89 | $phpcsFile->fixer->addContent( |
---|
90 | $method_end_ptr |
---|
91 | , "\tpublic function {$method_name} ({$args}) {" |
---|
92 | ); |
---|
93 | $phpcsFile->fixer->addNewline( $method_end_ptr ); |
---|
94 | $phpcsFile->fixer->addContent( |
---|
95 | $method_end_ptr |
---|
96 | , "\t\tself::__construct({$args});" |
---|
97 | ); |
---|
98 | $phpcsFile->fixer->addNewline( $method_end_ptr ); |
---|
99 | $phpcsFile->fixer->addContent( |
---|
100 | $method_end_ptr |
---|
101 | , "\t}" |
---|
102 | ); |
---|
103 | */ |
---|
104 | $phpcsFile->fixer->endChangeset(); |
---|
105 | } |
---|
106 | |
---|
107 | } else { |
---|
108 | |
---|
109 | $phpcsFile->addError( |
---|
110 | 'Found PHP4 constructor not proceeded by a PHP5 constructor.' |
---|
111 | , $stackPtr |
---|
112 | , 'NoPHP5Constructor' |
---|
113 | ); |
---|
114 | } |
---|
115 | |
---|
116 | return $tokens[ $class_ptr ]['scope_closer']; |
---|
117 | } |
---|
118 | } |
---|
119 | } |
---|
120 | |
---|
121 | // EOF |
---|