[SERVER-50276] Remove unused 'modifiedPaths' parameter in update code. Created: 12/Aug/20 Updated: 17/Sep/20 Resolved: 17/Sep/20 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Improvement | Priority: | Major - P3 |
| Reporter: | Arun Banala | Assignee: | Arun Banala |
| Resolution: | Won't Do | Votes: | 0 |
| Labels: | qexec-team | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Sprint: | Query 2020-09-07, Query 2020-09-21 |
| Participants: |
| Description |
|
The modifiedPaths variable in ApplyParams is no longer being used anywhere except in unit tests. We can safely get rid of all the code that was setting it. This seems to also being used by stitch_support.cpp |
| Comments |
| Comment by Arun Banala [ 17/Sep/20 ] |
|
I spoke to mpobrien from Stitch team. It looks like they are still using modifiedPaths to identify whether a user has permission to modify certain field while updating a document. So we cannot remove this code. I'm closing this ticket as 'Won't Do', |