0
0
mirror of https://github.com/phpv8/v8js.git synced 2024-11-08 15:48:40 +00:00

allow enumeration of methods

-> actually our properties said this shouldn't be possible
-> however v8js_named_property_enumerator enumerated the methods
-> older V8 versions listed the methods anyways
-> V8 6.3 started to not list the method now

refs #326
This commit is contained in:
Stefan Siegl 2017-09-24 13:59:34 +02:00
parent fb0b0c1f07
commit 8e8306ae3e
No known key found for this signature in database
GPG Key ID: B224EC6857AD7F9B

View File

@ -664,8 +664,7 @@ v8::Local<v8::Value> v8js_named_property_callback(v8::Local<v8::String> property
} }
} }
} else if (callback_type == V8JS_PROP_QUERY) { } else if (callback_type == V8JS_PROP_QUERY) {
// methods are not enumerable ret_value = V8JS_UINT(v8::ReadOnly|v8::DontDelete);
ret_value = V8JS_UINT(v8::ReadOnly|v8::DontEnum|v8::DontDelete);
} else if (callback_type == V8JS_PROP_SETTER) { } else if (callback_type == V8JS_PROP_SETTER) {
ret_value = set_value; // lie. this field is read-only. ret_value = set_value; // lie. this field is read-only.
} else if (callback_type == V8JS_PROP_DELETER) { } else if (callback_type == V8JS_PROP_DELETER) {