@@ -35,6 +35,9 @@ discard block |
||
35 | 35 | 'ConstantPatcher', |
36 | 36 | ]; |
37 | 37 | |
38 | + /** |
|
39 | + * @param string $message |
|
40 | + */ |
|
38 | 41 | public static function log($message) |
39 | 42 | { |
40 | 43 | if (! self::$debug) |
@@ -303,6 +306,9 @@ discard block |
||
303 | 306 | self::$load_patchers = true; |
304 | 307 | } |
305 | 308 | |
309 | + /** |
|
310 | + * @param string $source |
|
311 | + */ |
|
306 | 312 | protected static function execPatchers($source) |
307 | 313 | { |
308 | 314 | $patched = false; |
@@ -11,7 +11,6 @@ |
||
11 | 11 | namespace Kenjis\MonkeyPatch; |
12 | 12 | |
13 | 13 | use LogicException; |
14 | -use RuntimeException; |
|
15 | 14 | use PhpParser\ParserFactory; |
16 | 15 | use Kenjis\MonkeyPatch\Patcher\FunctionPatcher; |
17 | 16 |
@@ -41,7 +41,7 @@ discard block |
||
41 | 41 | |
42 | 42 | public static function log($message) |
43 | 43 | { |
44 | - if (! self::$debug) |
|
44 | + if ( ! self::$debug) |
|
45 | 45 | { |
46 | 46 | return; |
47 | 47 | } |
@@ -80,7 +80,7 @@ discard block |
||
80 | 80 | } |
81 | 81 | if (is_null(self::$log_file)) |
82 | 82 | { |
83 | - self::$log_file = __DIR__ . '/debug.log'; |
|
83 | + self::$log_file = __DIR__.'/debug.log'; |
|
84 | 84 | } |
85 | 85 | } |
86 | 86 | |
@@ -93,10 +93,10 @@ discard block |
||
93 | 93 | else |
94 | 94 | { |
95 | 95 | // APPPATH is constant in CodeIgniter |
96 | - Cache::setProjectRootDir(APPPATH . '../'); |
|
96 | + Cache::setProjectRootDir(APPPATH.'../'); |
|
97 | 97 | } |
98 | 98 | |
99 | - if (! isset($config['cache_dir'])) |
|
99 | + if ( ! isset($config['cache_dir'])) |
|
100 | 100 | { |
101 | 101 | throw new LogicException('You have to set "cache_dir"'); |
102 | 102 | } |
@@ -105,7 +105,7 @@ discard block |
||
105 | 105 | |
106 | 106 | protected static function setPaths(array $config) |
107 | 107 | { |
108 | - if (! isset($config['include_paths'])) |
|
108 | + if ( ! isset($config['include_paths'])) |
|
109 | 109 | { |
110 | 110 | throw new LogicException('You have to set "include_paths"'); |
111 | 111 | } |
@@ -166,7 +166,7 @@ discard block |
||
166 | 166 | // Updated? |
167 | 167 | if ($cached !== $current) |
168 | 168 | { |
169 | - MonkeyPatchManager::log('clear_src_cache: from ' . __METHOD__); |
|
169 | + MonkeyPatchManager::log('clear_src_cache: from '.__METHOD__); |
|
170 | 170 | Cache::clearSrcCache(); |
171 | 171 | Cache::writeTmpIncludePaths($current); |
172 | 172 | } |
@@ -177,7 +177,7 @@ discard block |
||
177 | 177 | // Updated? |
178 | 178 | if ($cached !== $current) |
179 | 179 | { |
180 | - MonkeyPatchManager::log('clear_src_cache: from ' . __METHOD__); |
|
180 | + MonkeyPatchManager::log('clear_src_cache: from '.__METHOD__); |
|
181 | 181 | Cache::clearSrcCache(); |
182 | 182 | Cache::writeTmpExcludePaths($current); |
183 | 183 | } |
@@ -190,7 +190,7 @@ discard block |
||
190 | 190 | // Updated? |
191 | 191 | if ($cached !== self::$patcher_list) |
192 | 192 | { |
193 | - MonkeyPatchManager::log('clear_src_cache: from ' . __METHOD__); |
|
193 | + MonkeyPatchManager::log('clear_src_cache: from '.__METHOD__); |
|
194 | 194 | Cache::clearSrcCache(); |
195 | 195 | Cache::writeTmpPatcherList(self::$patcher_list); |
196 | 196 | } |
@@ -204,7 +204,7 @@ discard block |
||
204 | 204 | // Updated? |
205 | 205 | if ($cached !== $current) |
206 | 206 | { |
207 | - MonkeyPatchManager::log('clear_src_cache: from ' . __METHOD__); |
|
207 | + MonkeyPatchManager::log('clear_src_cache: from '.__METHOD__); |
|
208 | 208 | Cache::clearSrcCache(); |
209 | 209 | Cache::writeTmpFunctionWhitelist($current); |
210 | 210 | } |
@@ -266,7 +266,7 @@ discard block |
||
266 | 266 | */ |
267 | 267 | public static function patch($path) |
268 | 268 | { |
269 | - if (! is_readable($path)) |
|
269 | + if ( ! is_readable($path)) |
|
270 | 270 | { |
271 | 271 | throw new LogicException("Can't read '$path'"); |
272 | 272 | } |
@@ -274,17 +274,17 @@ discard block |
||
274 | 274 | // Check cache file |
275 | 275 | if ($cache_file = Cache::getValidSrcCachePath($path)) |
276 | 276 | { |
277 | - self::log('cache_hit: ' . $path); |
|
277 | + self::log('cache_hit: '.$path); |
|
278 | 278 | return fopen($cache_file, 'r'); |
279 | 279 | } |
280 | 280 | |
281 | - self::log('cache_miss: ' . $path); |
|
281 | + self::log('cache_miss: '.$path); |
|
282 | 282 | $source = file_get_contents($path); |
283 | 283 | |
284 | 284 | list($new_source, $patched) = self::execPatchers($source); |
285 | 285 | |
286 | 286 | // Write to cache file |
287 | - self::log('write_cache: ' . $path); |
|
287 | + self::log('write_cache: '.$path); |
|
288 | 288 | Cache::writeSrcCacheFile($path, $new_source); |
289 | 289 | |
290 | 290 | $resource = fopen('php://memory', 'rb+'); |
@@ -300,12 +300,12 @@ discard block |
||
300 | 300 | return; |
301 | 301 | } |
302 | 302 | |
303 | - require __DIR__ . '/Patcher/AbstractPatcher.php'; |
|
304 | - require __DIR__ . '/Patcher/Backtrace.php'; |
|
303 | + require __DIR__.'/Patcher/AbstractPatcher.php'; |
|
304 | + require __DIR__.'/Patcher/Backtrace.php'; |
|
305 | 305 | |
306 | 306 | foreach (self::$patcher_list as $classname) |
307 | 307 | { |
308 | - require __DIR__ . '/Patcher/' . $classname . '.php'; |
|
308 | + require __DIR__.'/Patcher/'.$classname.'.php'; |
|
309 | 309 | } |
310 | 310 | |
311 | 311 | self::$load_patchers = true; |
@@ -316,7 +316,7 @@ discard block |
||
316 | 316 | $patched = false; |
317 | 317 | foreach (self::$patcher_list as $classname) |
318 | 318 | { |
319 | - $classname = 'Kenjis\MonkeyPatch\Patcher\\' . $classname; |
|
319 | + $classname = 'Kenjis\MonkeyPatch\Patcher\\'.$classname; |
|
320 | 320 | $patcher = new $classname; |
321 | 321 | list($source, $patched_this) = $patcher->patch($source); |
322 | 322 | $patched = $patched || $patched_this; |
@@ -70,8 +70,7 @@ |
||
70 | 70 | if (isset($config['root_dir'])) |
71 | 71 | { |
72 | 72 | Cache::setProjectRootDir($config['root_dir']); |
73 | - } |
|
74 | - else |
|
73 | + } else |
|
75 | 74 | { |
76 | 75 | // APPPATH is constant in CodeIgniter |
77 | 76 | Cache::setProjectRootDir(APPPATH . '../'); |
@@ -20,6 +20,9 @@ |
||
20 | 20 | 'ConstantPatcher' => 0, |
21 | 21 | ]; |
22 | 22 | |
23 | + /** |
|
24 | + * @param string $patcher |
|
25 | + */ |
|
23 | 26 | public static function getInfo($patcher, $backtrace) |
24 | 27 | { |
25 | 28 | if (! isset(self::$map[$patcher])) |
@@ -21,7 +21,7 @@ discard block |
||
21 | 21 | |
22 | 22 | public static function getInfo($patcher, $backtrace) |
23 | 23 | { |
24 | - if (! isset(self::$map[$patcher])) |
|
24 | + if ( ! isset(self::$map[$patcher])) |
|
25 | 25 | { |
26 | 26 | throw new LogicException("No such a patcher: $patcher"); |
27 | 27 | } |
@@ -41,12 +41,12 @@ discard block |
||
41 | 41 | $line = isset($backtrace[$offset]['line']) |
42 | 42 | ? $backtrace[$offset]['line'] : null; |
43 | 43 | |
44 | - if (isset($backtrace[$offset+2])) |
|
44 | + if (isset($backtrace[$offset + 2])) |
|
45 | 45 | { |
46 | - $class = isset($backtrace[$offset+2]['class']) |
|
47 | - ? $backtrace[$offset+2]['class'] |
|
46 | + $class = isset($backtrace[$offset + 2]['class']) |
|
47 | + ? $backtrace[$offset + 2]['class'] |
|
48 | 48 | : null; |
49 | - $function = $backtrace[$offset+2]['function']; |
|
49 | + $function = $backtrace[$offset + 2]['function']; |
|
50 | 50 | } |
51 | 51 | else |
52 | 52 | { |
@@ -57,7 +57,7 @@ discard block |
||
57 | 57 | if (isset($class)) |
58 | 58 | { |
59 | 59 | $method = $function; |
60 | - $class_method = $class . '::' . $function; |
|
60 | + $class_method = $class.'::'.$function; |
|
61 | 61 | $function = null; |
62 | 62 | } |
63 | 63 | else |
@@ -47,8 +47,7 @@ discard block |
||
47 | 47 | ? $backtrace[$offset+2]['class'] |
48 | 48 | : null; |
49 | 49 | $function = $backtrace[$offset+2]['function']; |
50 | - } |
|
51 | - else |
|
50 | + } else |
|
52 | 51 | { |
53 | 52 | $class = null; |
54 | 53 | $function = null; |
@@ -59,8 +58,7 @@ discard block |
||
59 | 58 | $method = $function; |
60 | 59 | $class_method = $class . '::' . $function; |
61 | 60 | $function = null; |
62 | - } |
|
63 | - else |
|
61 | + } else |
|
64 | 62 | { |
65 | 63 | $method = null; |
66 | 64 | $class_method = null; |
@@ -14,7 +14,6 @@ |
||
14 | 14 | require __DIR__ . '/ConstantPatcher/Proxy.php'; |
15 | 15 | |
16 | 16 | use LogicException; |
17 | - |
|
18 | 17 | use Kenjis\MonkeyPatch\Patcher\ConstantPatcher\NodeVisitor; |
19 | 18 | |
20 | 19 | class ConstantPatcher extends AbstractPatcher |
@@ -10,8 +10,8 @@ |
||
10 | 10 | |
11 | 11 | namespace Kenjis\MonkeyPatch\Patcher; |
12 | 12 | |
13 | -require __DIR__ . '/ConstantPatcher/NodeVisitor.php'; |
|
14 | -require __DIR__ . '/ConstantPatcher/Proxy.php'; |
|
13 | +require __DIR__.'/ConstantPatcher/NodeVisitor.php'; |
|
14 | +require __DIR__.'/ConstantPatcher/Proxy.php'; |
|
15 | 15 | |
16 | 16 | use LogicException; |
17 | 17 |
@@ -181,8 +181,7 @@ discard block |
||
181 | 181 | if (is_string($token)) |
182 | 182 | { |
183 | 183 | $new_source .= $token; |
184 | - } |
|
185 | - elseif ($i == $replacement['key']) |
|
184 | + } elseif ($i == $replacement['key']) |
|
186 | 185 | { |
187 | 186 | $new_source .= $replacement['value']; |
188 | 187 | $replacement['key'] = key(self::$replacement); |
@@ -192,8 +191,7 @@ discard block |
||
192 | 191 | { |
193 | 192 | $replacement = false; |
194 | 193 | } |
195 | - } |
|
196 | - else |
|
194 | + } else |
|
197 | 195 | { |
198 | 196 | $new_source .= $token[1]; |
199 | 197 | } |
@@ -15,7 +15,6 @@ |
||
15 | 15 | use PhpParser\Node\Name; |
16 | 16 | use PhpParser\Node\Name\FullyQualified; |
17 | 17 | use PhpParser\NodeVisitorAbstract; |
18 | - |
|
19 | 18 | use Kenjis\MonkeyPatch\Patcher\ConstantPatcher; |
20 | 19 | |
21 | 20 | class NodeVisitor extends NodeVisitorAbstract |
@@ -24,7 +24,7 @@ discard block |
||
24 | 24 | |
25 | 25 | public function enterNode(Node $node) |
26 | 26 | { |
27 | - $callback = [$this, 'before' . ucfirst($node->getType())]; |
|
27 | + $callback = [$this, 'before'.ucfirst($node->getType())]; |
|
28 | 28 | if (is_callable($callback)) { |
29 | 29 | call_user_func_array($callback, [$node]); |
30 | 30 | } |
@@ -32,9 +32,9 @@ discard block |
||
32 | 32 | |
33 | 33 | public function leaveNode(Node $node) |
34 | 34 | { |
35 | - if (! ($node instanceof ConstFetch)) |
|
35 | + if ( ! ($node instanceof ConstFetch)) |
|
36 | 36 | { |
37 | - $callback = [$this, 'rewrite' . ucfirst($node->getType())]; |
|
37 | + $callback = [$this, 'rewrite'.ucfirst($node->getType())]; |
|
38 | 38 | if (is_callable($callback)) { |
39 | 39 | call_user_func_array($callback, [$node]); |
40 | 40 | } |
@@ -47,23 +47,23 @@ discard block |
||
47 | 47 | return; |
48 | 48 | } |
49 | 49 | |
50 | - if (! ($node->name instanceof Name)) |
|
50 | + if ( ! ($node->name instanceof Name)) |
|
51 | 51 | { |
52 | 52 | return; |
53 | 53 | } |
54 | 54 | |
55 | - if (! $node->name->isUnqualified()) |
|
55 | + if ( ! $node->name->isUnqualified()) |
|
56 | 56 | { |
57 | 57 | return; |
58 | 58 | } |
59 | 59 | |
60 | - if (! ConstantPatcher::isBlacklisted((string) $node->name)) |
|
60 | + if ( ! ConstantPatcher::isBlacklisted((string) $node->name)) |
|
61 | 61 | { |
62 | - $replacement = new FullyQualified('\__ConstProxy__::get(\'' . (string) $node->name . '\')'); |
|
62 | + $replacement = new FullyQualified('\__ConstProxy__::get(\''.(string) $node->name.'\')'); |
|
63 | 63 | |
64 | 64 | $pos = $node->getAttribute('startTokenPos'); |
65 | 65 | ConstantPatcher::$replacement[$pos] = |
66 | - '\__ConstProxy__::get(\'' . (string) $node->name .'\')'; |
|
66 | + '\__ConstProxy__::get(\''.(string) $node->name.'\')'; |
|
67 | 67 | |
68 | 68 | $node->name = $replacement; |
69 | 69 | } |
@@ -51,6 +51,9 @@ discard block |
||
51 | 51 | self::$patches_to_apply = []; |
52 | 52 | } |
53 | 53 | |
54 | + /** |
|
55 | + * @param string $constant |
|
56 | + */ |
|
54 | 57 | protected static function logInvocation($constant) |
55 | 58 | { |
56 | 59 | if (MonkeyPatchManager::$debug) |
@@ -68,6 +71,9 @@ discard block |
||
68 | 71 | } |
69 | 72 | } |
70 | 73 | |
74 | + /** |
|
75 | + * @param string $constant |
|
76 | + */ |
|
71 | 77 | protected static function checkCalledMethod($constant) |
72 | 78 | { |
73 | 79 | $trace = debug_backtrace(); |
@@ -13,14 +13,9 @@ |
||
13 | 13 | class_alias('Kenjis\MonkeyPatch\Patcher\ConstantPatcher\Proxy', '__ConstProxy__'); |
14 | 14 | |
15 | 15 | use LogicException; |
16 | -use ReflectionConstant; |
|
17 | -use ReflectionException; |
|
18 | - |
|
19 | 16 | use Kenjis\MonkeyPatch\Patcher\ConstantPatcher; |
20 | 17 | use Kenjis\MonkeyPatch\Patcher\Backtrace; |
21 | 18 | use Kenjis\MonkeyPatch\MonkeyPatchManager; |
22 | -use Kenjis\MonkeyPatch\Cache; |
|
23 | -use Kenjis\MonkeyPatch\InvocationVerifier; |
|
24 | 19 | |
25 | 20 | class Proxy |
26 | 21 | { |
@@ -63,7 +63,7 @@ discard block |
||
63 | 63 | $method = isset($info['class_method']) ? $info['class_method'] : $info['function']; |
64 | 64 | |
65 | 65 | MonkeyPatchManager::log( |
66 | - 'invoke_const: ' . $constant . ') on line ' . $line . ' in ' . $file . ' by ' . $method . '()' |
|
66 | + 'invoke_const: '.$constant.') on line '.$line.' in '.$file.' by '.$method.'()' |
|
67 | 67 | ); |
68 | 68 | } |
69 | 69 | } |
@@ -106,12 +106,12 @@ discard block |
||
106 | 106 | { |
107 | 107 | self::logInvocation($constant); |
108 | 108 | |
109 | - if (! empty(self::$patches_to_apply[$constant])) |
|
109 | + if ( ! empty(self::$patches_to_apply[$constant])) |
|
110 | 110 | { |
111 | - if (! self::checkCalledMethod($constant)) |
|
111 | + if ( ! self::checkCalledMethod($constant)) |
|
112 | 112 | { |
113 | 113 | MonkeyPatchManager::log( |
114 | - 'invoke_const: ' . $constant . ' not patched (out of scope)' |
|
114 | + 'invoke_const: '.$constant.' not patched (out of scope)' |
|
115 | 115 | ); |
116 | 116 | return constant($constant); |
117 | 117 | } |
@@ -119,12 +119,12 @@ discard block |
||
119 | 119 | |
120 | 120 | if (array_key_exists($constant, self::$patches)) |
121 | 121 | { |
122 | - MonkeyPatchManager::log('invoke_const: ' . $constant . ' patched'); |
|
122 | + MonkeyPatchManager::log('invoke_const: '.$constant.' patched'); |
|
123 | 123 | return self::$patches[$constant]; |
124 | 124 | } |
125 | 125 | |
126 | 126 | MonkeyPatchManager::log( |
127 | - 'invoke_const: ' . $constant . ' not patched (no patch)' |
|
127 | + 'invoke_const: '.$constant.' not patched (no patch)' |
|
128 | 128 | ); |
129 | 129 | return constant($constant); |
130 | 130 | } |
@@ -84,6 +84,9 @@ discard block |
||
84 | 84 | $this->node_visitor = new NodeVisitor(); |
85 | 85 | } |
86 | 86 | |
87 | + /** |
|
88 | + * @param string $error_msg |
|
89 | + */ |
|
87 | 90 | protected static function checkLock($error_msg) |
88 | 91 | { |
89 | 92 | if (self::$lock_function_list) |
@@ -110,6 +113,9 @@ discard block |
||
110 | 113 | return self::$whitelist; |
111 | 114 | } |
112 | 115 | |
116 | + /** |
|
117 | + * @param string $function_name |
|
118 | + */ |
|
113 | 119 | public static function addBlacklist($function_name) |
114 | 120 | { |
115 | 121 | self::checkLock("You can't add to blacklist after initialization"); |
@@ -14,7 +14,6 @@ |
||
14 | 14 | require __DIR__ . '/FunctionPatcher/Proxy.php'; |
15 | 15 | |
16 | 16 | use LogicException; |
17 | - |
|
18 | 17 | use Kenjis\MonkeyPatch\Patcher\FunctionPatcher\NodeVisitor; |
19 | 18 | |
20 | 19 | class FunctionPatcher extends AbstractPatcher |
@@ -10,8 +10,8 @@ |
||
10 | 10 | |
11 | 11 | namespace Kenjis\MonkeyPatch\Patcher; |
12 | 12 | |
13 | -require __DIR__ . '/FunctionPatcher/NodeVisitor.php'; |
|
14 | -require __DIR__ . '/FunctionPatcher/Proxy.php'; |
|
13 | +require __DIR__.'/FunctionPatcher/NodeVisitor.php'; |
|
14 | +require __DIR__.'/FunctionPatcher/Proxy.php'; |
|
15 | 15 | |
16 | 16 | use LogicException; |
17 | 17 |
@@ -181,8 +181,7 @@ discard block |
||
181 | 181 | if (is_string($token)) |
182 | 182 | { |
183 | 183 | $new_source .= $token; |
184 | - } |
|
185 | - elseif ($i == $replacement['key']) |
|
184 | + } elseif ($i == $replacement['key']) |
|
186 | 185 | { |
187 | 186 | $new_source .= $replacement['value']; |
188 | 187 | $replacement['key'] = key(self::$replacement); |
@@ -192,8 +191,7 @@ discard block |
||
192 | 191 | { |
193 | 192 | $replacement = false; |
194 | 193 | } |
195 | - } |
|
196 | - else |
|
194 | + } else |
|
197 | 195 | { |
198 | 196 | $new_source .= $token[1]; |
199 | 197 | } |
@@ -21,15 +21,11 @@ |
||
21 | 21 | |
22 | 22 | namespace Kenjis\MonkeyPatch\Patcher\FunctionPatcher; |
23 | 23 | |
24 | -use ReflectionFunction; |
|
25 | -use ReflectionException; |
|
26 | - |
|
27 | 24 | use PhpParser\Node; |
28 | 25 | use PhpParser\Node\Expr\FuncCall; |
29 | 26 | use PhpParser\Node\Name; |
30 | 27 | use PhpParser\Node\Name\FullyQualified; |
31 | 28 | use PhpParser\NodeVisitorAbstract; |
32 | - |
|
33 | 29 | use Kenjis\MonkeyPatch\Patcher\FunctionPatcher; |
34 | 30 | |
35 | 31 | class NodeVisitor extends NodeVisitorAbstract |
@@ -36,17 +36,17 @@ discard block |
||
36 | 36 | { |
37 | 37 | public function leaveNode(Node $node) |
38 | 38 | { |
39 | - if (! ($node instanceof FuncCall)) |
|
39 | + if ( ! ($node instanceof FuncCall)) |
|
40 | 40 | { |
41 | 41 | return; |
42 | 42 | } |
43 | 43 | |
44 | - if (! ($node->name instanceof Name)) |
|
44 | + if ( ! ($node->name instanceof Name)) |
|
45 | 45 | { |
46 | 46 | return; |
47 | 47 | } |
48 | 48 | |
49 | - if (! $node->name->isUnqualified()) |
|
49 | + if ( ! $node->name->isUnqualified()) |
|
50 | 50 | { |
51 | 51 | return; |
52 | 52 | } |
@@ -55,11 +55,11 @@ discard block |
||
55 | 55 | FunctionPatcher::isWhitelisted((string) $node->name) |
56 | 56 | && ! FunctionPatcher::isBlacklisted((string) $node->name) |
57 | 57 | ) { |
58 | - $replacement = new FullyQualified('\__FuncProxy__::' . (string) $node->name); |
|
58 | + $replacement = new FullyQualified('\__FuncProxy__::'.(string) $node->name); |
|
59 | 59 | |
60 | 60 | $pos = $node->getAttribute('startTokenPos'); |
61 | 61 | FunctionPatcher::$replacement[$pos] = |
62 | - '\__FuncProxy__::' . (string) $node->name; |
|
62 | + '\__FuncProxy__::'.(string) $node->name; |
|
63 | 63 | |
64 | 64 | $node->name = $replacement; |
65 | 65 | } |
@@ -37,7 +37,7 @@ discard block |
||
37 | 37 | * |
38 | 38 | * @param string $function function name |
39 | 39 | * @param mixed $return_value return value or callable |
40 | - * @param string $class_name class::method to apply this patch |
|
40 | + * @param string $class_method class::method to apply this patch |
|
41 | 41 | * |
42 | 42 | * @throws LogicException |
43 | 43 | */ |
@@ -86,6 +86,9 @@ discard block |
||
86 | 86 | InvocationVerifier::verify(self::$expected_invocations, self::$invocations); |
87 | 87 | } |
88 | 88 | |
89 | + /** |
|
90 | + * @param string $function |
|
91 | + */ |
|
89 | 92 | protected static function logInvocation($function, $arguments) |
90 | 93 | { |
91 | 94 | if (MonkeyPatchManager::$debug) |
@@ -111,6 +114,9 @@ discard block |
||
111 | 114 | } |
112 | 115 | } |
113 | 116 | |
117 | + /** |
|
118 | + * @param string $function |
|
119 | + */ |
|
114 | 120 | protected static function checkCalledMethod($function) |
115 | 121 | { |
116 | 122 | $trace = debug_backtrace(); |
@@ -184,6 +190,9 @@ discard block |
||
184 | 190 | return call_user_func_array($function, $arguments); |
185 | 191 | } |
186 | 192 | |
193 | + /** |
|
194 | + * @param string $function |
|
195 | + */ |
|
187 | 196 | protected static function checkPassedByReference($function) |
188 | 197 | { |
189 | 198 | $ref_func = new ReflectionFunction($function); |
@@ -15,7 +15,6 @@ |
||
15 | 15 | use LogicException; |
16 | 16 | use ReflectionFunction; |
17 | 17 | use ReflectionException; |
18 | - |
|
19 | 18 | use Kenjis\MonkeyPatch\Patcher\FunctionPatcher; |
20 | 19 | use Kenjis\MonkeyPatch\Patcher\Backtrace; |
21 | 20 | use Kenjis\MonkeyPatch\MonkeyPatchManager; |
@@ -51,7 +51,7 @@ discard block |
||
51 | 51 | self::outputMessage($msg); |
52 | 52 | throw new LogicException($msg); |
53 | 53 | } |
54 | - if (! FunctionPatcher::isWhitelisted($function)) |
|
54 | + if ( ! FunctionPatcher::isWhitelisted($function)) |
|
55 | 55 | { |
56 | 56 | $msg = "<red>Can't patch on '$function'. It is not in whitelist. If you want to patch it, please add it to 'functions_to_patch' in 'tests/Bootstrap.php'. But note that there are some limitations. See <https://github.com/kenjis/ci-phpunit-test/blob/master/docs/HowToWriteTests.md#patching-functions> for details.</red>"; |
57 | 57 | self::outputMessage($msg); |
@@ -97,16 +97,16 @@ discard block |
||
97 | 97 | $line = $info['line']; |
98 | 98 | $method = isset($info['class_method']) ? $info['class_method'] : $info['function']; |
99 | 99 | |
100 | - $log_args = function () use ($arguments) { |
|
100 | + $log_args = function() use ($arguments) { |
|
101 | 101 | $output = ''; |
102 | 102 | foreach ($arguments as $arg) { |
103 | - $output .= var_export($arg, true) . ', '; |
|
103 | + $output .= var_export($arg, true).', '; |
|
104 | 104 | } |
105 | 105 | $output = rtrim($output, ', '); |
106 | 106 | return $output; |
107 | 107 | }; |
108 | 108 | MonkeyPatchManager::log( |
109 | - 'invoke_func: ' . $function . '(' . $log_args() . ') on line ' . $line . ' in ' . $file . ' by ' . $method . '()' |
|
109 | + 'invoke_func: '.$function.'('.$log_args().') on line '.$line.' in '.$file.' by '.$method.'()' |
|
110 | 110 | ); |
111 | 111 | } |
112 | 112 | } |
@@ -148,10 +148,10 @@ discard block |
||
148 | 148 | |
149 | 149 | if (isset(self::$patches_to_apply[$function])) |
150 | 150 | { |
151 | - if (! self::checkCalledMethod($function)) |
|
151 | + if ( ! self::checkCalledMethod($function)) |
|
152 | 152 | { |
153 | 153 | MonkeyPatchManager::log( |
154 | - 'invoke_func: ' . $function . '() not patched (out of scope)' |
|
154 | + 'invoke_func: '.$function.'() not patched (out of scope)' |
|
155 | 155 | ); |
156 | 156 | self::checkPassedByReference($function); |
157 | 157 | return call_user_func_array($function, $arguments); |
@@ -160,7 +160,7 @@ discard block |
||
160 | 160 | |
161 | 161 | if (array_key_exists($function, self::$patches)) |
162 | 162 | { |
163 | - MonkeyPatchManager::log('invoke_func: ' . $function . '() patched'); |
|
163 | + MonkeyPatchManager::log('invoke_func: '.$function.'() patched'); |
|
164 | 164 | |
165 | 165 | if (is_callable(self::$patches[$function])) |
166 | 166 | { |
@@ -178,7 +178,7 @@ discard block |
||
178 | 178 | } |
179 | 179 | |
180 | 180 | MonkeyPatchManager::log( |
181 | - 'invoke_func: ' . $function . '() not patched (no patch)' |
|
181 | + 'invoke_func: '.$function.'() not patched (no patch)' |
|
182 | 182 | ); |
183 | 183 | self::checkPassedByReference($function); |
184 | 184 | return call_user_func_array($function, $arguments); |
@@ -231,7 +231,7 @@ discard block |
||
231 | 231 | $msg = str_replace( |
232 | 232 | ['<red>', '</red>'], [$red_begin, $red_end], $msg |
233 | 233 | ); |
234 | - echo $msg . "\n"; |
|
234 | + echo $msg."\n"; |
|
235 | 235 | } |
236 | 236 | |
237 | 237 | /** |
@@ -13,8 +13,6 @@ |
||
13 | 13 | require __DIR__ . '/MethodPatcher/NodeVisitor.php'; |
14 | 14 | require __DIR__ . '/MethodPatcher/PatchManager.php'; |
15 | 15 | |
16 | -use LogicException; |
|
17 | - |
|
18 | 16 | use Kenjis\MonkeyPatch\Patcher\MethodPatcher\NodeVisitor; |
19 | 17 | |
20 | 18 | class MethodPatcher extends AbstractPatcher |
@@ -10,8 +10,8 @@ discard block |
||
10 | 10 | |
11 | 11 | namespace Kenjis\MonkeyPatch\Patcher; |
12 | 12 | |
13 | -require __DIR__ . '/MethodPatcher/NodeVisitor.php'; |
|
14 | -require __DIR__ . '/MethodPatcher/PatchManager.php'; |
|
13 | +require __DIR__.'/MethodPatcher/NodeVisitor.php'; |
|
14 | +require __DIR__.'/MethodPatcher/PatchManager.php'; |
|
15 | 15 | |
16 | 16 | use LogicException; |
17 | 17 | |
@@ -64,11 +64,11 @@ discard block |
||
64 | 64 | { |
65 | 65 | if ($start_method && $token === '{') |
66 | 66 | { |
67 | - if(self::isVoidFunction($tokens, $key)){ |
|
68 | - $new_source .= '{ ' . self::CODENORET; |
|
67 | + if (self::isVoidFunction($tokens, $key)) { |
|
68 | + $new_source .= '{ '.self::CODENORET; |
|
69 | 69 | } |
70 | - else{ |
|
71 | - $new_source .= '{ ' . self::CODE; |
|
70 | + else { |
|
71 | + $new_source .= '{ '.self::CODE; |
|
72 | 72 | } |
73 | 73 | $start_method = false; |
74 | 74 | $replacement['key'] = key(self::$replacement); |
@@ -100,22 +100,22 @@ discard block |
||
100 | 100 | * @param $key |
101 | 101 | * @return bool |
102 | 102 | */ |
103 | - protected static function isVoidFunction($tokens, $key){ |
|
104 | - if($key - 1 <= 0){ |
|
103 | + protected static function isVoidFunction($tokens, $key) { |
|
104 | + if ($key - 1 <= 0) { |
|
105 | 105 | return false; |
106 | 106 | } |
107 | 107 | $token = $tokens[$key - 1]; |
108 | - if(is_array($token)){ |
|
108 | + if (is_array($token)) { |
|
109 | 109 | $token = $token[1]; |
110 | 110 | } |
111 | 111 | //Loop backwards though the start of the function block till you either find "void" or the end of the |
112 | 112 | //parameters declaration. |
113 | - while($token !== ")"){ |
|
114 | - if(strpos($token, "void") !== false){ |
|
113 | + while ($token !== ")") { |
|
114 | + if (strpos($token, "void") !== false) { |
|
115 | 115 | return true; |
116 | 116 | } |
117 | 117 | $token = $tokens[$key - 1]; |
118 | - if(is_array($token)){ |
|
118 | + if (is_array($token)) { |
|
119 | 119 | $token = $token[1]; |
120 | 120 | } |
121 | 121 | $key--; |
@@ -66,8 +66,7 @@ discard block |
||
66 | 66 | { |
67 | 67 | if(self::isVoidFunction($tokens, $key)){ |
68 | 68 | $new_source .= '{ ' . self::CODENORET; |
69 | - } |
|
70 | - else{ |
|
69 | + } else{ |
|
71 | 70 | $new_source .= '{ ' . self::CODE; |
72 | 71 | } |
73 | 72 | $start_method = false; |
@@ -78,13 +77,11 @@ discard block |
||
78 | 77 | { |
79 | 78 | $replacement = false; |
80 | 79 | } |
81 | - } |
|
82 | - else |
|
80 | + } else |
|
83 | 81 | { |
84 | 82 | $new_source .= $token; |
85 | 83 | } |
86 | - } |
|
87 | - else |
|
84 | + } else |
|
88 | 85 | { |
89 | 86 | $new_source .= $token[1]; |
90 | 87 | } |