Modify

Opened 5 years ago

Closed 5 years ago

Last modified 5 years ago

#19011 closed defect (fixed)

JOSM imagery, geojson output is not correct geojson

Reported by: SimonPoole Owned by: Don-vip
Priority: normal Milestone: 20.03
Component: Core imagery Version:
Keywords: geojson Cc:

Description

See https://josm.openstreetmap.de/maps?format=geojson

The top level object should be a FeatureCollection, but is missing "type": "FeatureCollection" either before or after the "features" array.

Attachments (0)

Change History (8)

comment:1 by Stereo, 5 years ago

Thanks Simon! The fix is relatively easy, but has to happen internally in Trac.

On line 366 in Trac's JOSM.py, replace:

j = {"features": []}

with

j = {"type": "FeatureCollection", "features": []}

comment:2 by Don-vip, 5 years ago

Keywords: geojson added
Milestone: 20.03
Owner: changed from team to Don-vip
Status: newassigned

comment:3 by Don-vip, 5 years ago

Resolution: fixed
Status: assignedclosed

Thank you both!

comment:4 by simon04, 5 years ago

Just out of curiosity, where can the JOSM.py be found (to get an idea of the magic hidden behind JOSM Trac)?

comment:5 by Don-vip, 5 years ago

It's not public yet, I shared it with Stereo as he was interested to help us in imagery list. Check your e-mails :)

comment:6 by skyper, 5 years ago

Thanks, now JOSM is back in taginfo`s project listings.

comment:7 by Stereo, 5 years ago

taginfo has a project listing that depends on valid geojson?

comment:8 by Don-vip, 5 years ago

This is completely unrelated please don't mix different subjects.

Modify Ticket

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