mirrors/phpv8
mirrors/phpv8
0
0
Fork 0
mirror of https://github.com/phpv8/v8js.git synced 2024-09-18 23:15:18 +00:00
Code Releases Activity

Don't export V8Js methods even if overwritten

Browse Source
This commit is contained in:
Stefan Siegl 2016-01-09 19:11:30 +01:00
parent 1e86e2c9f7
commit d438624a3d
2 changed files with 46 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

44
tests/issue_183_004.phpt Normal file
View File

@ -0,0 +1,44 @@
--TEST--
Test V8::executeString() : Method access on derived classes (overridden V8Js methods)
--SKIPIF--
<?php require_once(dirname(__FILE__) . '/skipif.inc'); ?>
--FILE--
<?php
class Foo extends \V8Js
{
public function hello()
{
print("Hello World\n");
}
public function executeString($script, $identifier = NULL, $flags = NULL, $time_limit = NULL, $memory_limit = NULL)
{
var_dump("executeString");
return parent::executeString($script);
}
}
$JS = <<< EOT
var_dump(typeof PHP.hello);
var_dump(typeof PHP.executeString);
try {
PHP.executeString('print("blar")');
}
catch(e) {
var_dump('caught');
}
EOT;
$v8 = new Foo();
$v8->executeString($JS);
?>
===EOF===
--EXPECTF--
string(13) "executeString"
string(8) "function"
string(9) "undefined"
string(6) "caught"
===EOF===

2
v8js_class.cc
View File

@ -531,7 +531,7 @@ static PHP_METHOD(V8Js, __construct)
const zend_function_entry *fe;
for (fe = v8js_methods; fe->fname; fe ++) {
if (fe->fname == method_ptr->common.function_name) {
if (strcmp(fe->fname, method_ptr->common.function_name) == 0) {
break;
}
}