adulau/cakephp2-php8
1
0
Fork 0
mirror of https://github.com/kamilwylegala/cakephp2-php8.git synced 2025-02-07 12:36:25 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

Adding Shell::hasMethod and tests for it.

Browse source
This commit is contained in:
mark_story 2010-10-09 21:52:12 -04:00
parent 18c5a62445
commit cd18c8214c
2 changed files with 39 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

16
cake/console/libs/shell.php
View file

@ -305,7 +305,21 @@ class Shell extends Object {
* @return boolean * @return boolean
*/ */
public function hasMethod($name) { public function hasMethod($name) {
if (empty($this->_reflection)) {
$this->_reflection = new ReflectionClass($this);
}
try {
$method = $this->_reflection->getMethod($name);
if (!$method->isPublic() || substr($name, 0, 1) === '_') {
return false;
}
if ($method->getDeclaringClass() != $this->_reflection) {
return false;
}
return true;
} catch (ReflectionException $e) {
return false;
}
} }
/** /**

24
cake/tests/cases/console/libs/shell.test.php
View file

@ -57,6 +57,18 @@ class TestShell extends Shell {
protected function _stop($status = 0) { protected function _stop($status = 0) {
$this->stopped = $status; $this->stopped = $status;
} }
public function do_something() {
}
public function _secret() {
}
protected function no_access() {
}
} }
/** /**
@ -578,4 +590,16 @@ class ShellTest extends CakeTestCase {
$this->assertTrue($this->Shell->hasTask('db_config')); $this->assertTrue($this->Shell->hasTask('db_config'));
$this->assertTrue($this->Shell->hasTask('DbConfig')); $this->assertTrue($this->Shell->hasTask('DbConfig'));
} }
/**
* test the hasMethod
*
* @return void
*/
function testHasMethod() {
$this->assertTrue($this->Shell->hasMethod('do_something'));
$this->assertFalse($this->Shell->hasMethod('hr'), 'hr is callable');
$this->assertFalse($this->Shell->hasMethod('_secret'), '_secret is callable');
$this->assertFalse($this->Shell->hasMethod('no_access'), 'no_access is callable');
}
} }