66058 - "Changed" date should be more informative

Status: VERIFIED FIXED

(Bugzilla :: Bugzilla-General, defect)

Description

[Henrik Gemal]

If you're doing a Bug List the "Changed" field should be more informative.
What about the following:
If changed today: write the time (fx 12:24:34)
If changed this week: write short date plus short time (01/19 12:24)
If changed later than this week: write full date: 2001-01-19

Comment 1

[Stephan Niemz]

Good idea! Taking bug.

Comment 2

[Stephan Niemz]

How about this minor change:

If changed within the last 18 hours: time only (12:24:34)
If changed within the last  6 days:  weekday and time (Fri 12:24)
If changed later: current behaviour (2001-01-19)

Comment 3

[Stephan Niemz]

Cc'ing a few people to get comments.
I wouldn't want to put any effort in this to see it not being accepted later.

Comment 4

[Dave Miller [:justdave]]

I like this idea.  That would be very helpful.

Comment 5

[Stephan Niemz]

Attachment: patch, without whitespace changes

Comment 6

[Stephan Niemz]

Attachment: patch, also removing trailing whitespace

Comment 7

[Stephan Niemz]

Okay, it wasn't as difficult as I thought, so here is the patch.
Additionally, the "date bug opened" is treated the same way, if that
column is active.

The second patch also removes trailing whitespace everywhere.  If you don't
like that, please check in the first patch.

Comment 8

[Dave Miller [:justdave]]

r= [e-mail]

Works great on my install.

Comment 9

[Stephan Niemz]

Attachment: patch #2 (no space change)

Comment 10

[Stephan Niemz]

Attachment: patch #2 (removing trailing whitespace, too)

Comment 11

[Stephan Niemz]

Attached is a second patch, inspired by a discussion with timeless on IRC.
Now it works this way:

If changed within the last 18 hours: time only (12:24:34)
If changed within the last   6 days: weekday and time (Fri 12:24)
If changed within the last 100 days: month and day (01-19)
If changed later: current behaviour (2001-01-19)

Comment 12

[Dave Miller [:justdave]]

patch v2 is checked in.

Comment 13

[Jesse Ruderman]

I hope this doesn't mess up sorting by date...

Comment 14

[Dave Miller [:justdave]]

not at all.  This is all in the display code, not the calculation or DB storage.

Comment 15

[Tara Hernandez]

Sorry for the spam, but I needed to be able to query for all of these correctly.

Comment 16

[Gervase Markham [:gerv]]

VERIFIED. Very cool :-)

Gerv

Comment 17

[Dave Miller [:justdave]]

Moving closed bugs to Bugzilla product