Homec4science

Fix Herald to accept new JSON encoding of sparse arrays

Authored by epriestley <git@epriestley.com> on Jul 7 2011, 21:35.

Description

Fix Herald to accept new JSON encoding of sparse arrays

Summary: JX.JSON was recently changed to use JSON.stringify (the native
implementation) if it is available. The native implementation has a behavioral
difference from the Javelin implementation, in that it does not compact sparse
arrays. Ignore nulls resulting from removals when processing the encoded action
and condition lists.
Test Plan: Removed conditions from Herald rules.
Reviewed By: aran
Reviewers: aran, cpojer, jungejason
CC: aran, epriestley
Differential Revision: 606

Details

Committed
epriestley <git@epriestley.com>Jul 8 2011, 00:27
Pushed
aubortJan 31 2017, 17:16
Parents
rPH49310391e0a3: Added subscriber view to Maniphest.
Branches
Unknown
Tags
Unknown

Event Timeline

epriestley <git@epriestley.com> committed rPH62532ef26d6e: Fix Herald to accept new JSON encoding of sparse arrays (authored by epriestley <git@epriestley.com>).Jul 8 2011, 00:27