Modify

Opened 7 years ago

Closed 7 years ago

Last modified 7 years ago

#15173 closed enhancement (fixed)

Add OpenTopoMap as Imagery source

Reported by: Atalanttore Owned by: Atalanttore
Priority: normal Milestone:
Component: External imagery source Version: latest
Keywords: OpenTopoMap Cc: stoecker

Description (last modified by Atalanttore)

Hi,

please add https://opentopomap.org/ to the list of Imagery sources.

OpenTopoMap is very useful for adding mountain peaks at their highest position.

Regards,
Ettore

Attachments (0)

Change History (10)

comment:1 by Atalanttore, 7 years ago

Description: modified (diff)

comment:2 by Don-vip, 7 years ago

Owner: changed from team to Atalanttore
Status: newneedinfo

It's a wiki, you're allowed to add the entry yourself (less work for us!)

comment:3 by Don-vip, 7 years ago

Resolution: worksforme
Status: needinfoclosed

Please add it yourself, the bugtracker is meant to track things only JOSM developers can do.

comment:4 by Atalanttore, 7 years ago

The wiki doesn't allow my entry (see below). It reports: You are not allowed to change the default entries!

    <entry>
        <name>OpenTopoMap</name>
        <id>OpenTopoMap</id>
        <type>tms</type>
        <default>true</default>
        <url>http://tile.opentopomap.org/{zoom}/{x}/{y}.png</url>
        <attribution-text mandatory="true">Kartendaten: © OpenStreetMap-Mitwirkende, SRTM | Kartendarstellung: © OpenTopoMap (CC-BY-SA)</attribution-text>
        <attribution-url>https://tile.opentopomap.org/about#verwendung</attribution-url>
        <min-zoom>3</min-zoom> 
        <max-zoom>17</max-zoom>
        <valid-georeference>true</valid-georeference>
    </entry>

Regards,
Ettore

comment:5 by Don-vip, 7 years ago

Don't make it a default entry with: <default>false</default>

comment:6 by Atalanttore, 7 years ago

<default>false</default> didn't work as well (same error like before). I had to remove the line completely.

Regards,
Ettore

in reply to:  6 ; comment:7 by Don-vip, 7 years ago

Cc: stoecker added

Replying to Atalanttore:

<default>false</default> didn't work as well (same error like before). I had to remove the line completely.

@Dirk: sounds like a bug?

comment:8 by Don-vip, 7 years ago

Resolution: worksformefixed

comment:9 by Klumbumbus, 7 years ago

I don't think we need <default>false</default>. Currently there are only 7 default entries. Just omit the line for the other 599 entries.

Last edited 7 years ago by Klumbumbus (previous) (diff)

in reply to:  7 comment:10 by stoecker, 7 years ago

Replying to Don-vip:

Replying to Atalanttore:

<default>false</default> didn't work as well (same error like before). I had to remove the line completely.

@Dirk: sounds like a bug?

A design issue :-)

The checks aren't intelligent enough to detect the contents of <default>. It only prevents <default> completely, as there is no need for a false value at all.

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Atalanttore.
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.