YourWishes/jerryscript
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix JSON.stringify (#2759)

Browse Source 
There was a wrong check on the property types, it did not check that the
property was name accessor. Also fixed a test case.

JerryScript-DCO-1.0-Signed-off-by: Istvan Miklos imiklos2@inf.u-szeged.hu
This commit is contained in:
Istvan Miklos
2019-02-19 11:29:13 +01:00
committed by László Langó
parent 60bbd731e5
commit 9c10d82e4a
2 changed files with 2 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files
jerry-core/ecma/builtin-objects/ecma-builtin-json.c
+1 -2
View File
@@ -1590,8 +1590,7 @@ ecma_builtin_json_object (ecma_object_t *obj_p, /**< the object*/
JERRY_ASSERT (ecma_is_property_enumerable (property)); JERRY_ASSERT (ecma_is_property_enumerable (property));
if (ECMA_PROPERTY_GET_TYPE (property) == ECMA_PROPERTY_TYPE_NAMEDDATA if (ECMA_PROPERTY_GET_TYPE (property) != ECMA_PROPERTY_TYPE_SPECIAL)
|| ECMA_PROPERTY_GET_TYPE (property) == ECMA_PROPERTY_TYPE_VIRTUAL)
{ {
ecma_append_to_values_collection (property_keys_p, *ecma_value_p, 0); ecma_append_to_values_collection (property_keys_p, *ecma_value_p, 0);
} }
tests/jerry/es2015/object-assign.js
+1 -1
View File
@@ -146,7 +146,7 @@ function completeAssign (target, sources) {
} }
var copy = completeAssign ({}, [obj]); var copy = completeAssign ({}, [obj]);
assert (JSON.stringify (copy) === '{"foo":1}'); assert (JSON.stringify (copy) === '{"foo":1,"bar":2}');
assert (copy.bar === 2); assert (copy.bar === 2);
// Test when target is not coercible to object // Test when target is not coercible to object