Modify

Opened 4 years ago

Closed 4 years ago

#1896 closed defect (fixed)

[PATCH] GPX timestamp reading is broken if there's a timezone but no fractional seconds

Reported by: Teemu Koskinen <teemu.koskinen@…> Owned by: framm
Priority: major Component: Core
Version: latest Keywords:
Cc:

Description

Revision 1167 broke timestamp reading of gpx-files if the timestamp doesn't contain only contains whole seconds and after that a timezone. This affects mostly the coloring of tracks by velocity.

All my gpx files (exported from Viking, originally from a Garmin eTrex Legend HCx) have timestamps in format: "yyyy-MM-dd'T'HH:mm:ss'Z'", which can't be parsed with the changed format: "yyyy-MM-dd'T'HH:mm:ss.SSS". The old format "yyyy-MM-dd'T'HH:mm:ss" worked just fine.

Attachments (1)

gpxtime.patch (991 bytes) - added by Teemu Koskinen <teemu.koskinen@…> 4 years ago.
This patch should fix the bug

Download all attachments as: .zip

Change History (4)

Changed 4 years ago by Teemu Koskinen <teemu.koskinen@…>

This patch should fix the bug

comment:1 Changed 4 years ago by Teemu Koskinen <teemu.koskinen@…>

  • Summary changed from GPX timestamp reading is broken if there's a timezone but no fractional seconds to [PATCH] GPX timestamp reading is broken if there's a timezone but no fractional seconds

comment:2 Changed 4 years ago by anonymous

I guess this might also affect the display of live audio trace, as on my tracks (they have timezone but no fractional seconds) the play head is sticked to the end of the track. I'd be grateful if someone having the skills to incorporate this patch could release a fixed compiled version.

comment:3 Changed 4 years ago by anonymous

  • Resolution set to fixed
  • Status changed from new to closed

Applied in r1185.

Add Comment

Modify Ticket

Change Properties
<Author field>
Action
as closed .
as The resolution will be set. Next status will be 'closed'.
The resolution will be deleted. Next status will be 'reopened'.
Author


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

 
Note: See TracTickets for help on using tickets.