Modify

Opened 8 weeks ago

Closed 9 days ago

Last modified 7 days 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 Changed 8 weeks ago by Atalanttore

Description: modified (diff)

comment:2 Changed 3 weeks ago by Don-vip

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 Changed 9 days ago by Don-vip

Resolution: worksforme
Status: needinfoclosed

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

comment:4 Changed 7 days ago by Atalanttore

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 Changed 7 days ago by Don-vip

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

comment:6 Changed 7 days ago by Atalanttore

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

Regards,
Ettore

comment:7 in reply to:  6 ; Changed 7 days ago by Don-vip

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 Changed 7 days ago by Don-vip

Resolution: worksformefixed

comment:9 Changed 7 days ago by Klumbumbus

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 days ago by Klumbumbus (previous) (diff)

comment:10 in reply to:  7 Changed 7 days ago by stoecker

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.

Add Comment


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

 
Note: See TracTickets for help on using tickets.