Ruby on Rails | Screencasts | Download | Documentation | Weblog | Community | Source

Ticket #1836 (closed defect: fixed)

Opened 3 years ago

Last modified 2 years ago

Bug in Time.midnight

Reported by: rcolli2@tampabay.rr.com Assigned to: David
Priority: normal Milestone: 1.0
Component: ActiveSupport Version: 0.13.1
Severity: major Keywords:
Cc:

Description

Following code fails (returns random results) on BSD 5.4 and Tiger 10.4.2, Works on Slackware 10.1.0:

require 'rubygems' require 'active_support' class Time

def same_day?(other)

mid_self = self.midnight mid_other = other.midnight puts(mid_self) puts(mid_other) retval = (mid_self == mid_other) puts retval puts retval

end

end 10.times do |time|

midnight = Time.now.midnight midnight.same_day?(midnight + 1.hour)

end

Change History

07/28/05 16:56:19 changed by nzkoz

  • status changed from new to closed.
  • resolution set to invalid.

Thanks for the bug report, but the time classes rails uses comes from ruby itself

http://www.ruby-doc.org/stdlib/libdoc/time/rdoc/index.html

07/28/05 17:09:59 changed by rcolli2@tampabay.rr.com

I'm pretty sure that Time#midnight is an extention that is part of Rails:

http://api.rubyonrails.com/classes/ActiveSupport/CoreExtensions/Time/Calculations.html#M000202

07/28/05 19:22:27 changed by nzkoz

  • status changed from closed to reopened.
  • resolution deleted.
  • milestone set to 1.0.

So it is, sorry. I've been a bit trigger happy here it seems.

Marking for 1.0

07/31/05 09:16:40 changed by minam

  • status changed from reopened to closed.
  • resolution set to fixed.

[1959]

06/12/06 02:04:15 changed by anonymous

tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet tatunet