Modify ↓

Opened 11 years ago

Closed 11 years ago

Last modified 11 years ago

#10691 closed enhancement (fixed)

[patch] include boundary=protected_area

Reported by: skyper Owned by: team
Priority: normal Milestone: 14.11
Component: Internal preset Version:
Keywords: boundary protected_area Cc: Klumbumbus

Description

As mentioned in #10698boundary=protected_area (​wiki) is missing in defaultpresets.

Attachments (4)

josm_10691.diff​ (1.6 KB ) - added by skyper 11 years ago.
patch
josm_10691_screenshot.png​ (29.1 KB ) - added by skyper 11 years ago.
screenshot
josm_10691v2.diff​ (1.6 KB ) - added by skyper 11 years ago.
patch version 2
josm_10691_wording.diff​ (704 bytes ) - added by skyper 11 years ago.
wording patch

Download all attachments as: .zip

Change History (20)

comment:1 by skyper, 11 years ago

I am not sure about the place to add it within the defaultpreset as boundary is only available under relations and protected_area is valid for closed ways.

comment:2 by Klumbumbus, 11 years ago

Cc: Klumbumbus added

comment:3 by Klumbumbus, 11 years ago

boundaries are also under geography -> boundaries

by skyper, 11 years ago

Attachment: josm_10691.diff​ added

patch

by skyper, 11 years ago

Attachment: josm_10691_screenshot.png​ added

screenshot

in reply to:  3 comment:4 by skyper, 11 years ago

Summary: include boundary=protected_area → [patch] include boundary=protected_area

Replying to Klumbumbus:

boundaries are also under geography -> boundaries

Thanks

Please find attached patch​ which looks like below:

screenshot

comment:5 by Klumbumbus, 11 years ago

There is a typo, it is protection_title not protected_title.

Of cause it is ok, to add all language specific wiki links, but it is enough if you add only the english one. The other languages will added automatically before the next stable release (a changeset like [7640]).

by skyper, 11 years ago

Attachment: josm_10691v2.diff​ added

patch version 2

in reply to:  5 comment:6 by skyper, 11 years ago

Replying to Klumbumbus:

There is a typo, it is protection_title not protected_title.

Thanks, please find attached a fixed version​

Of cause it is ok, to add all language specific wiki links, but it is enough if you add only the english one. The other languages will added automatically before the next stable release (a changeset like [7640]).

No problem to add them once but the continually update is stupid work for computers.

comment:7 by Klumbumbus, 11 years ago

Resolution: → fixed
Status: new → closed

In 7718/josm:

fix #10691 - add boundary=protected_area to preset (patch by skyper)

comment:8 by Klumbumbus, 11 years ago

Milestone: → 14.11

comment:9 by aceman, 11 years ago

Did you really mean "destination" ? Or did you want to say "designation" ? Something like ​http://wiki.openstreetmap.org/wiki/Key:designation ?

comment:10 by Klumbumbus, 11 years ago

Resolution: fixed
Status: closed → reopened

Yes, I think he meant "designation". So is "designation" the correct word used in this context? (I'm not a english native speaker.)

in reply to:  9 ; comment:11 by skyper, 11 years ago

Replying to aceman:

Did you really mean "destination" ? Or did you want to say "designation" ? Something like ​osmwiki:Key:designation ?

I did copy it from the ​wiki page. Please, go ahead and adjust the page if needed.

by skyper, 11 years ago

Attachment: josm_10691_wording.diff​ added

wording patch

in reply to:  11 comment:12 by skyper, 11 years ago

Replying to skyper:

Replying to aceman:

Did you really mean "destination" ? Or did you want to say "designation" ? Something like ​osmwiki:Key:designation ?

I did copy it from the ​wiki page. Please, go ahead and adjust the page if needed.

I did fix the wiki page and completely remove the word in presets with attached patch​.

comment:13 by Klumbumbus, 11 years ago

Resolution: → fixed
Status: reopened → closed

In 7723/josm:

fix #10691 - fix typo (patch by skyper)

comment:14 by Klumbumbus, 11 years ago

In 7972/josm:

see #10691 - fix typo in key of protect_class preset

comment:15 by Klumbumbus, 11 years ago

In 7973/josm:

see #10691 - add validator warning for protected_class

comment:16 by Klumbumbus, 11 years ago

The number of protected_class raised since this bug from 31 to 81. I just fixed all 81.

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 .
Attachments ↑ Description ↑
 
Note: See TracTickets for help on using tickets.