Skip to content

Commit

Permalink
fix(shared): include integer keys in js::Object::GetKeys (#245)
Browse files Browse the repository at this point in the history
  • Loading branch information
xxshady authored Aug 24, 2024
1 parent 6b0ea96 commit 2ca2cb9
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion shared/src/helpers/JS.h
Original file line number Diff line number Diff line change
Expand Up @@ -298,7 +298,7 @@ namespace js
v8::MaybeLocal<v8::Array> maybePropNames = object->GetPropertyNames(GetContext(),
v8::KeyCollectionMode::kOwnOnly,
(v8::PropertyFilter)(v8::PropertyFilter::ONLY_ENUMERABLE | v8::PropertyFilter::SKIP_SYMBOLS),
v8::IndexFilter::kSkipIndices,
v8::IndexFilter::kIncludeIndices,
v8::KeyConversionMode::kConvertToString);
v8::Local<v8::Array> propNames;
if(!maybePropNames.ToLocal(&propNames)) return keys;
Expand Down

0 comments on commit 2ca2cb9

Please sign in to comment.