Opened 4 years ago
Closed 4 years ago
#18731 closed enhancement (fixed)
[PATCH] There should be a method to search for all primitives inside a bbox
| Reported by: | taylor.smock | Owned by: | Don-vip |
|---|---|---|---|
| Priority: | normal | Milestone: | 20.02 |
| Component: | Core | Version: | |
| Keywords: | Cc: |
Description (last modified by )
Currently, if I want to check all primitives in a bbox, I have to create a list and then make a call for searchNodes, searchWays, and searchRelations.
I'm marking this as an RFC since I think it would be better to put this in the OsmData class with a default method, but I don't know if it is worth the effort (I don't know how to cleanly check if N/W/R extend O).
I also haven't implemented any tests for it.
Attachments (3)
Change History (8)
Changed 4 years ago by
| Attachment: | 18731.patch added |
|---|
comment:1 Changed 4 years ago by
It's ok in DataSet. Can be moved to OsmData later if there's a need for.
Changed 4 years ago by
| Attachment: | 18138.1.patch added |
|---|
Add tests and fix an issue with a possible NPE (if bbox is null, there is an error in searchNodes)
comment:2 Changed 4 years ago by
| Description: | modified (diff) |
|---|---|
| Summary: | [PATCH RFC] There should be a method to search for all primitives inside a bbox → [PATCH] There should be a method to search for all primitives inside a bbox |
comment:3 Changed 4 years ago by
| Milestone: | → 20.02 |
|---|---|
| Owner: | changed from team to Don-vip |
| Status: | new → assigned |
Changed 4 years ago by
| Attachment: | 18731.1.patch added |
|---|
Add tests and fix an issue with a possible NPE (if bbox is null, there is an error in searchNodes)
this time, with the correct file
Initial patch for
searchPrimitives