Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Can't update POI data #20846

Closed
luigifab opened this issue Sep 15, 2024 · 5 comments · Fixed by #21198
Closed

Can't update POI data #20846

luigifab opened this issue Sep 15, 2024 · 5 comments · Fixed by #21198
Assignees
Labels
Milestone

Comments

@luigifab
Copy link

luigifab commented Sep 15, 2024

Description

I'm displaying swimming pools POI on the map. I would like to add opening hours but I can't edit it, I can only add a new POI.

Steps to reproduce

Go to 55.52213 25.08419 and display swimming pools POI. Then try to edit it.

Actual result

Can't edit POI details.

Expected result

Allow to edit the POI.

Your Environment (required)

OsmAnd Version: 4.8.6
Android/iOS version: 11
Device model: Wiko
@yuriiurshuliak
Copy link

I was unable to reproduce the issue. To try reproducing it, go to the coordinates 55.52213, 25.08419, click on the POI of the pool, select "Actions," then choose "Modify POI" and "Add Opening hours."

Please ensure that the OpenStreetMap editing plugin is enabled.

OsmAnd~ 4.9.0#3193m, released: 2024-09-16

video_2024-09-16_16-24-48.mp4

@luigifab
Copy link
Author

yes plugin is enabled, and I tried again and I still can't edit this POI, for example next to the swimming pool there are bus stops, and I can edit them

@vshcherb vshcherb added this to the 4.9-android milestone Oct 29, 2024
@vshcherb
Copy link
Member

Bug reproduced; https://www.openstreetmap.org/way/322243903.
Context menu displays correct url to way.
To fix: all nodes and ways with correct url should be editable in OSM.

@nazar-kutz
Copy link
Contributor

nazar-kutz commented Oct 29, 2024

@vshcherb The problem seems to be caused by the "no_edit=true" attribute on the poi type of this point. Maybe we should fix it in xml.

Line in XML:
<poi_type name="swimming_pool" tag="leisure" value="swimming_pool" no_edit="true" poi_additional_category="nudism,fee,access_general"/>
https://github.com/osmandapp/OsmAnd-resources/blob/master/poi/poi_types.xml#L4196

Line in code that filtering this point:
isEditable = !amenity.getType().isWiki() && poiType != null && !poiType.isNotEditableOsm();
https://github.com/osmandapp/Osmand/blob/380a20fc064c6c10f830ffa3fb34b2aeee6791d4/OsmAnd/src/net/osmand/plus/plugins/osmedit/OsmEditingPlugin.java#L347
We get "poiType.isNotEditableOsm() == true" here, and so don't show "Modify POI" action.

@vshcherb
Copy link
Member

If internet is available we can allow edit without "predefined type" but all the tags will be loaded

nazar-kutz added a commit that referenced this issue Oct 31, 2024
@nazar-kutz nazar-kutz linked a pull request Oct 31, 2024 that will close this issue
Chumva added a commit that referenced this issue Nov 1, 2024
@Chumva Chumva closed this as completed Nov 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants