Opened 2 months ago

Last modified 7 weeks ago

#15524 new defect

[PATCH] building=yes is described as residential building

Reported by: mkoniecz Owned by: team
Priority: normal Milestone:
Component: Internal preset Version:
Keywords: template_report Cc:

Description (last modified by Klumbumbus)

What steps will reproduce the problem?

  1. create way
  2. tag building=yes

What is the expected result?

Not described as residential building

What happens instead?

described as residential building

Please provide any additional information below. Attach a screenshot if possible.

Repository:UUID: 0c6e7542-c601-0410-84e7-c038aed88b3b
Last:Changed Date: 2017-10-30 23:58:23 +0100 (Mon, 30 Oct 2017)
Build-Date:2017-10-31 02:33:18
Relative:URL: ^/trunk

Identification: JOSM/1.5 (13064 en) Linux Ubuntu 16.04.3 LTS
Memory Usage: 934 MB / 934 MB (459 MB allocated, but free)
Java version: 1.8.0_151-b12, Oracle Corporation, Java HotSpot(TM) 64-Bit Server VM
Screen: :0.0 1920x1080
Maximum Screen Size: 1920x1080
Dataset consistency test: No problems found

+ OpeningHoursEditor (33185)
+ buildings_tools (33735)
+ lakewalker (33712)
+ reverter (33572)
+ undelete (33480)

Map paint styles:
- ${HOME}/Documents/install_moje/josm_style_sidewalks/Styles_Sidewalks-style.mapcss

Validator rules:
+ ${HOME}/Documents/install_moje/josm_validator/josm.validator.mapcss

Last errors/warnings:
- E: Socket closed
- W: Read timed out
- W: Read timed out
- W: Read timed out
- W: Read timed out
- W: Read timed out
- W: Read timed out
- W: Read timed out
- W: Read timed out
- W: Read timed out

Attachments (1)

Selection_016.png (4.5 KB) - added by mkoniecz 2 months ago.

Download all attachments as: .zip

Change History (5)

Changed 2 months ago by mkoniecz

Attachment: Selection_016.png added

comment:1 Changed 2 months ago by naoliv

Summary: building=yes is described as residential building[PATCH] building=yes is described as residential building

Have the generic building=yes only at the Building preset.
Also roof

  • data/defaultpresets.xml

    68116811                  ru.href=""
    68126812                  uk.href="" />
    68136813            <space />
    6814             <combo key="building" text="Building" values="yes,residential,apartments,house,hotel,hut,bungalow,dormitory,terrace,detached,farm,roof" default="residential" match="keyvalue!" values_context="building" values_searchable="true" />
     6814            <combo key="building" text="Building" values="residential,apartments,house,hotel,hut,bungalow,dormitory,terrace,detached,farm" default="residential" match="keyvalue!" values_context="building" values_searchable="true" />
    68156815            <optional>
    68166816                <text key="name" text="Name" />
    68176817                <combo key="building:levels" text="Levels" values="1,2,3,4,5,6,7,8,9,10,11" text_context="building" />
Last edited 2 months ago by naoliv (previous) (diff)

comment:2 Changed 2 months ago by Klumbumbus

The value "yes" was added to the residential building preset in r9392 to link it in the Tags/Membership dialog on the right, so you can switch from the default value "yes" to an more precise value by clicking the link there. I agree this is not an optimal solution but I think it is better than if only the building preset is linked there and not the residential building preset too. I think with the current match implementation (TaggingPresets) there is no better solution possible.

(However we should remove roof from the residential building preset as this is not the default value.)

comment:3 Changed 2 months ago by Klumbumbus

Description: modified (diff)

comment:4 Changed 7 weeks ago by Klumbumbus

In 13185/josm:

see #15524 - remove roof from residential preset

Modify Ticket

Change Properties
Set your email in Preferences
as new The owner will remain team.
as The resolution will be set.
to The owner will be changed from team to the specified user.
The owner will change to mkoniecz
as duplicate The resolution will be set to duplicate.The specified ticket will be cross-referenced with this ticket
The owner will be changed from team to anonymous.

Add Comment

E-mail address and name can be saved in the Preferences.

Note: See TracTickets for help on using tickets.