[CSHARP-774] C# Driver still uses deprecated $within instead of $geoWithin Created: 14/Jul/13 Updated: 11/Mar/19 Resolved: 06/Apr/15 |
|
| Status: | Closed |
| Project: | C# Driver |
| Component/s: | None |
| Affects Version/s: | 1.8.1 |
| Fix Version/s: | None |
| Type: | Task | Priority: | Trivial - P5 |
| Reporter: | Jörg B. | Assignee: | Unassigned |
| Resolution: | Done | Votes: | 2 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||
| Backwards Compatibility: | Minor Change | ||||
| Description |
|
Good afternoon, I was reading over the geospatial query capabilities of mongodb in the documentation and in parallel was playing with the c# driver wroting some sample queries, but when I came to the $geoWithin operator, which apparently replace the deprecated $within one, I saw that the QueryBuilder.cs file still contains the aforementioned $within operator.. instead of $geoWithin (see https://github.com/mongodb/mongo-csharp-driver/blob/master/MongoDB.Driver/Builders/QueryBuilder.cs#L712). Not sure how this affects actual usage (atm it seems to work just fine), but I thought I'd report it nevertheless so the driver gets updated to $geoWithin as well. Cheers, |
| Comments |
| Comment by Craig Wilson [ 15/Jul/13 ] |
|
$within has been deprecated, but it's function still works. $geoWithin is just a better name. Think of these as aliases. The driver has to support multiple versions of the server. We typically support the current version and 2 prior versions. As such, we need to continue to use $within until server version 2.8. I'll put this in the TBD version so that we do remember to do it. Thanks, |