Modify

Opened 6 years ago

Closed 4 years ago

#9498 closed defect (fixed)

[Patch] Released versions are visible for the day of release

Reported by: ak@… Owned by: Ryan J Ollos
Priority: normal Component: ExtendedVersionPlugin
Severity: normal Keywords:
Cc: Trac Release: 0.12

Description

Hi,

the test whether or not a version has been released actually only checks the date, not the datetime. This way a released version stays visible for the whole day and the label says "due in 33 sec" although it has been released 33sec ago.

Alex

Attachments (1)

release_versions_fix.patch (1.9 KB) - added by ak@… 6 years ago.

Download all attachments as: .zip

Change History (6)

Changed 6 years ago by ak@…

Attachment: release_versions_fix.patch added

comment:1 Changed 6 years ago by ak@…

Summary: Released versions are visible for the day of release[Patch] Released versions are visible for the day of release

comment:2 Changed 5 years ago by Malcolm Studd

ACK. This patch looks OK.

comment:3 Changed 5 years ago by Ryan J Ollos

Owner: changed from Malcolm Studd to Ryan J Ollos
Status: newassigned

One other related change I'd recommend is to make more extensive use of the trac.util.datefmt functions, which do a clean job of handling the user's timezone setting.

comment:4 Changed 4 years ago by Ryan J Ollos

Status: assignednew

comment:5 Changed 4 years ago by Ryan J Ollos

Resolution: fixed
Status: newclosed

This issue was fixed in [13310]. The patch in release_versions_fix.patch was mostly correct, except that the timestamp is stored in utc time, so the comparisons need to be done relative to current utc time, not the current user time. Conversion to utc time before storing in database is done here; conversion from utc time to the user's time is done in the template here. format_datetime is a partial function that is passed to the template after populating the req.tz argument (the actual call is through the user_time helper function now, but that detail doesn't matter much).

Modify Ticket

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