@@ -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 | } |