[CSHARP-2335] Support value return on UpdateBuilder::PopFirst like array operations Created: 19/Jul/18 Updated: 15/Jan/19 Resolved: 14/Jan/19 |
|
| Status: | Closed |
| Project: | C# Driver |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Improvement | Priority: | Minor - P4 |
| Reporter: | Vick Mena (Inactive) | Assignee: | Unassigned |
| Resolution: | Won't Fix | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Case: | (copied to CRM) |
| Description |
|
Would say "Pop" is not good name for this action. Expect a value return here. Thank you Yue |
| Comments |
| Comment by Jeffrey Yemin [ 14/Jan/19 ] |
|
Hi vick.mena, this is a builder class, so all the methods return "this". Even if we wanted to, we can't change this without breaking binary compatibility. While the name is unfortunate given the usual semantics of methods with "pop" in their name, this particular one is actually appending a $pop to the update, so not sure how we could avoid it. |