[DRIVERS-331] BSON Regex flags must be alphabetically ordered Created: 25/Oct/16 Updated: 15/Apr/19 Resolved: 11/Dec/17 |
|
| Status: | Closed |
| Project: | Drivers |
| Component/s: | None |
| Fix Version/s: | None |
| Type: | Improvement | Priority: | Minor - P4 |
| Reporter: | David Golden | Assignee: | Unassigned |
| Resolution: | Done | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| Driver Compliance: |
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| Description |
|
The BSON Corpus added test cases for regex flags. See this diff for the complete changes. |
| Comments |
| Comment by Jeremy Mikola [ 13/Jan/17 ] |
|
Validating PHPC due to derick: I skipped HHVM since you may need to implement an equivalent of |
| Comment by Bernie Hackett [ 11/Nov/16 ] |
|
See also SERVER-26992. |
| Comment by Bernie Hackett [ 11/Nov/16 ] |
|
Oops, ignore my last comment. The sort algorithm is correct. |
| Comment by Bernie Hackett [ 11/Nov/16 ] |
|
Sadly, the sort algorithm in the server also appears to be incorrect (so this ticket may not matter at all). bjori is going to open another server ticket about that. |
| Comment by David Golden [ 11/Nov/16 ] |
|
We know sort order matters (now and historically), so let's proceed by getting that right and dropping the invalid checks from the corpus. |
| Comment by Bernie Hackett [ 11/Nov/16 ] |
|
Let's put this on hold until some decisions are made related to |
| Comment by Jeremy Mikola [ 07/Nov/16 ] |
|
Since the Extended JSON spec is still in progress, would it be worthwhile to add this requirement to its output as well? Noting the discussion in |