Prepare the road to \JsonException usage

[deguif]

No description provided.

[ruflin]

It seems a bit overkill to have an additional dependency for a single exception change? What is the benefit of having JsonException?

[deguif]

For preparing smooth upgrade to native json_encode and json_decode functions with JSON_THROW_ON_ERROR option (introduced in PHP 7.3) and fully get rid of Elastica\JSON util class and JSONParseException exception class.

[deguif]

This extra polyfill package can be ignored on project side by using replace composer feature.

For example that's what I use in many projects when they are running on modern PHP versions, this way these packages are not installed as they are useless:

    "replace": {
        "paragonie/random_compat": "*",
        "ralouphie/getallheaders": "*",
        "symfony/polyfill-apcu": "*",
        "symfony/polyfill-ctype": "*",
        "symfony/polyfill-iconv": "*",
        "symfony/polyfill-intl-grapheme": "*",
        "symfony/polyfill-intl-icu": "*",
        "symfony/polyfill-intl-idn": "*",
        "symfony/polyfill-intl-normalizer": "*",
        "symfony/polyfill-mbstring": "*",
        "symfony/polyfill-php73": "*",
        "symfony/polyfill-uuid": "*"
    },

[ruflin]

SGTM. You have a conflict on the changelog.