@@ -104,6 +104,13 @@ class Package < ActiveRecord::Base self.name.downcase <=> other.name.downcase end + # If updated_at is nil (which is the case for some older records), return + # created_at as the value for updated_at. This is required because e.g. the + # sitemap relies on proper updated_at values. + def updated_at + super || self.created_at + end + def description self.latest_version.description # for convenience end app/models/package.rb @@ -88,4 +88,14 @@ describe Package do (pkg.updated_at > prev_time).should be_true end + it "should return the created_at timestamp if updated_at is nil" do + pkg = Package.make + pkg.updated_at.should == pkg.created_at + sql = "UPDATE package SET updated_at = NULL where id = #{pkg.id}" + ActiveRecord::Base.connection.execute(sql) + pkg.reload + pkg.attributes["updated_at"].should == nil + pkg.updated_at.should == pkg.created_at + end + end spec/models/package_spec.rb 33cfe54954996c5f85ebc2a8c5b25155e2334e5d Bjørn Arild Mæland bjorn.maeland@gmail.com http://github.com/hadley/crantastic/commit/3371b29165fb04ac86938f648adffc2bff875d48 3371b29165fb04ac86938f648adffc2bff875d48 2009-07-03T07:27:41-07:00 2009-07-03T07:27:41-07:00 If package#updated_at is nil (which is the case for some older records), return created_at instead of nil. 8fd1c638c928cea038e1e101e0fa220279b975b6 Bjørn Arild Mæland bjorn.maeland@gmail.com