rfc:json_throw_on_error
Differences
This shows you the differences between two versions of the page.
Both sides previous revisionPrevious revisionNext revision | Previous revisionLast revisionBoth sides next revision | ||
rfc:json_throw_on_error [2017/09/13 21:24] – ajf | rfc:json_throw_on_error [2017/10/05 21:57] – close ajf | ||
---|---|---|---|
Line 1: | Line 1: | ||
====== PHP RFC: JSON_THROW_ON_ERROR ====== | ====== PHP RFC: JSON_THROW_ON_ERROR ====== | ||
- | * Version: 1.0.1 | + | * Version: 1.0.2 |
* Date: 2017-09-10 | * Date: 2017-09-10 | ||
* Author: Andrea Faulds | * Author: Andrea Faulds | ||
- | * Status: | + | * Status: |
* First Published at: http:// | * First Published at: http:// | ||
Line 12: | Line 12: | ||
===== Proposal ===== | ===== Proposal ===== | ||
- | This RFC instead proposes adding a new option flag value for < | + | This RFC instead proposes adding a new option flag value for < |
< | < | ||
- | At the present time, there would be no change to the default error behaviour. It would be worthwhile considering whether to eventually slowly deprecate not using < | + | Note that when given an invalid depth parameter, < |
+ | |||
+ | At the present time, there would be no change to the default error behaviour. It would be worthwhile considering whether to eventually slowly deprecate not using < | ||
===== Backward Incompatible Changes ===== | ===== Backward Incompatible Changes ===== | ||
Line 40: | Line 42: | ||
As mentioned earlier, it may be desirable to deprecate the default behaviour eventually. | As mentioned earlier, it may be desirable to deprecate the default behaviour eventually. | ||
- | ===== Proposed Voting Choices | + | ===== Vote ===== |
- | This is not a language change, merely a small addition to the JSON extension, so it only technically requires a 50%+1 majority. | + | This is not a language change, merely a small addition to the JSON extension, so it only technically requires a 50%+1 majority. However, it's a 2/3 vote. It is a simple Yes/No vote on whether to accept this RFC and merge the patch. |
+ | |||
+ | Voting started 2017-09-25 and ended 2017-10-05. | ||
- | It would be a simple Yes/No vote on whether to accept this RFC and merge the patch. | + | <doodle title=" |
+ | * Yes | ||
+ | * No | ||
+ | </ | ||
===== Patches and Tests ===== | ===== Patches and Tests ===== | ||
Line 66: | Line 73: | ||
===== Changelog ===== | ===== Changelog ===== | ||
- | * v1.0.1 - cover details of the patch not previously mentioned | + | * v1.0.2 - change behaviour to leave global error flag untouched, rather than clearing it; note depth, param-parsing errors |
- | * v1.0 - initial version | + | |
+ | * v1.0 - initial version |
rfc/json_throw_on_error.txt · Last modified: 2017/10/22 18:44 by ajf