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

:wikipedia bug #10140

Closed
UKChris-osm opened this issue Feb 28, 2024 · 5 comments
Closed

:wikipedia bug #10140

UKChris-osm opened this issue Feb 28, 2024 · 5 comments
Labels
bug A bug - let's fix this!

Comments

@UKChris-osm
Copy link

iD seems to now be suggesting to add empty ":wikipedia" tags for NSI items. These tags were removed a long time ago, so I'm not sure why iD is now suggesting to add empty ones?

Ref: osmlab/name-suggestion-index#9284

nsi-bug

@arch0345
Copy link
Contributor

This seems to be caused by 4829f8a

@UKChris-osm
Copy link
Author

UKChris-osm commented Feb 28, 2024

This seems to be caused by 4829f8a

Yes, that's what I'm thinking - so not something we can fix from the NSI end?

@UKChris-osm
Copy link
Author

if (preset.tags['brand:wikidata']) { preset.addTags['brand:wikipedia'] = ''; }

... the code seems to be checking for a :wikidata tag, and just adding an empty :wikipedia tag to the item - was this intentional?

@tyrasd tyrasd added the bug A bug - let's fix this! label Feb 29, 2024
@tyrasd
Copy link
Member

tyrasd commented Feb 29, 2024

adding an empty :wikipedia tag to the item - was this intentional?

yes, this was intentional, as you can read in the source code comment:

nsi does not specify *:wikipedia (anymore):
clean up previous values to prevent that the wikidata/wikipedia information
is going to be out of sync, see #9103

I went with the fix proposed in #9103 (comment), but unfortunately this introduced/triggered a bug in the validation module, which no one found before the release. It should not be too hard to fix, though.

@tyrasd
Copy link
Member

tyrasd commented Feb 29, 2024

FYI: I created a patch release (v2.28.1) for this: see openstreetmap/openstreetmap-website#4551

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug A bug - let's fix this!
Projects
None yet
Development

No branches or pull requests

3 participants