Modify

Opened 8 years ago

Closed 7 years ago

Last modified 7 years ago

#12731 closed enhancement (fixed)

Add an option to completely prevent upload of a layer : e.g. "never" to upload=true/false

Reported by: jaakkoh Owned by: team
Priority: normal Milestone: 17.03
Component: Core Version:
Keywords: upload policy Cc:

Description (last modified by jaakkoh)

The most typical need for this extension might be training totally new people to the world of OSM, specifically using JOSM. There is commonly e.g. an example.osm file used to illustrate elements in OSM and how they show up & are manipulated in JOSM. The issue has been discussed at https://github.com/hotosm/learnosm/issues/335 . And the (currently last) comment from June 1, 2015.

Currently the most that can be done is to

  • use the upload=false and
  • then tell those that are trained not to upload their edited data -- and definitely not to "bypass" the prompt that warns not to upload the data (if someone does try to upload. And finally to
  • check the area where the example data is placed to see if anyone has ended uploading regardless of the above.

Experienced users and especially those that are more computer savvy than some might think that having upload=false is sufficient. But it's not. And the answer is not to put more emphasis to stressing the need to not upload when the upload tells not to.

So, simply said, there really should be an upload=never option, or something alike.

Attachments (1)

12731-upload-block.patch (10.3 KB ) - added by bastiK 7 years ago.

Download all attachments as: .zip

Change History (11)

comment:1 by jaakkoh, 8 years ago

Description: modified (diff)

(comment moved to the ticket itself)

Last edited 8 years ago by jaakkoh (previous) (diff)

comment:2 by althio, 7 years ago

That would be an interesting feature for our use case at LearnOSM https://github.com/hotosm/learnosm/issues/335

comment:3 by stoecker, 7 years ago

Why not simply use a wrong username/password?

by bastiK, 7 years ago

Attachment: 12731-upload-block.patch added

comment:4 by bastiK, 7 years ago

Attached patch implements the upload='never' flag. There is deliberately no GUI to toggle the flag, you need to edit the file in an editor to set or unset.

in reply to:  4 comment:5 by Don-vip, 7 years ago

Keywords: upload policy added; uploading removed
Milestone: 17.03

Replying to bastiK:

Attached patch implements the upload='never' flag.

xml_flag should be final (immutable enum)

Otherwise it's great !

comment:6 by bastiK, 7 years ago

Resolution: fixed
Status: newclosed

In 11709/josm:

fixed #12731 - Add an option to completely prevent upload of a layer : e.g. "never" to upload=true/false

to set this option, add XML attribute upload='never' to .osm file

comment:7 by bastiK, 7 years ago

In 11710/josm:

see #12731 - make enum field final + fix compiler warning

comment:8 by Don-vip, 7 years ago

In 11712/josm:

see #12731 - checkstyle/sonar

comment:9 by althio, 7 years ago

Thank you very much bastiK and Don-vip.
This works as intended on our side, we are very grateful for the implementation.

comment:10 by openstreetmap.org-user-d1g, 7 years ago

Thanks for implementing that!

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain team.
as The resolution will be set.
The resolution will be deleted. Next status will be 'reopened'.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.