Adium

Ticket #7713 (closed defect: fixed)

Opened 3 years ago

Last modified 2 years ago

Online Time Displayed with Useless Information (1 Week 4 Days *0 Hours* 43 Minutes

Reported by: persept Owned by: boredzo
Milestone: Adium 1.2 Component: Adium UI
Version: Severity: normal
Keywords: Online Time Hours Time Cc:
Patch Status:

Description

When dispalying the online time it will state that the person has been online for 1 Week 4 Days *0 Hours* 43 Minutes, even though it's pointless to say 0 Hours.

Attachments

Useless_Info.png (49.5 KB) - added by persept 3 years ago.

Change History

Changed 3 years ago by persept

Changed 3 years ago by boredzo

  • owner changed from nobody to boredzo
  • status changed from new to assigned

Changed 3 years ago by boredzo

  • status changed from assigned to closed
  • resolution set to fixed

(In [20757]) * Changed stringFromTimeInterval: implementation to use a mutable array and create the string by joining its elements.

  • Changed logic of non-approximate string-building to use the array, only adding each element if its associated number is not 0.
  • Changed seconds truth-tests to handle the floating-point number correctly (by comparing it to an epsilon value, which is currently 0.01). This was necessitated by r20756.

All tests now pass. Fixes #7713.

Changed 3 years ago by Robby

Milestone?

Changed 2 years ago by edr1084

  • milestone set to Adium X 1.2

Marking as 1.2 because I don't see that it was merged, but if it was and I missed it, please change accordingly.

Note: See TracTickets for help on using tickets.