lastdiffed should use NULL instead of the "beginning of time"

RESOLVED FIXED in Bugzilla 2.20

Status

()

P1
normal
RESOLVED FIXED
14 years ago
14 years ago

People

(Reporter: mkanat, Assigned: mkanat)

Tracking

2.19.2
Bugzilla 2.20
Dependency tree / graph
Bug Flags:
approval +

Details

Attachments

(1 attachment)

(Assignee)

Description

14 years ago
New bugs should have lastdiffed set to NULL, instead of having lastdiffed set to
"0000-00-00 00:00:00". I'm fairly sure that that representation isn't
cross-database compatible anyhow, I'd imagine. Even if it is, I think it makes
more sense to use NULL. We only use lastdiffed in a few places; it should be
pretty easy to change.
(Assignee)

Updated

14 years ago
Assignee: email-notifications → mkanat
Priority: -- → P1
Target Milestone: --- → Bugzilla 2.20
(Assignee)

Comment 1

14 years ago
Yeah, it wasn't too difficult. I found all the places that we used lastdiffed,
and I fixed them up, in addition to testing BugMail and making sure that it
still worked.
Attachment #177100 - Flags: review?(bugzilla)
Comment on attachment 177100 [details] [diff] [review]
Change lastdiffed to NULL

r=glob
Attachment #177100 - Flags: review?(bugzilla) → review+
(Assignee)

Updated

14 years ago
Flags: approval?
Flags: approval? → approval+
(Assignee)

Comment 3

14 years ago
Fixed bitrot on checkin.

Checking in checksetup.pl;
/cvsroot/mozilla/webtools/bugzilla/checksetup.pl,v  <--  checksetup.pl
new revision: 1.371; previous revision: 1.370
done
Checking in editproducts.cgi;
/cvsroot/mozilla/webtools/bugzilla/editproducts.cgi,v  <--  editproducts.cgi
new revision: 1.73; previous revision: 1.72
done
Checking in globals.pl;
/cvsroot/mozilla/webtools/bugzilla/globals.pl,v  <--  globals.pl
new revision: 1.316; previous revision: 1.315
done
Checking in sanitycheck.cgi;
/cvsroot/mozilla/webtools/bugzilla/sanitycheck.cgi,v  <--  sanitycheck.cgi
new revision: 1.87; previous revision: 1.86
done
Checking in Bugzilla/BugMail.pm;
/cvsroot/mozilla/webtools/bugzilla/Bugzilla/BugMail.pm,v  <--  BugMail.pm
new revision: 1.37; previous revision: 1.36
done
Checking in Bugzilla/DB/Schema.pm;
/cvsroot/mozilla/webtools/bugzilla/Bugzilla/DB/Schema.pm,v  <--  Schema.pm
new revision: 1.7; previous revision: 1.6
done
Checking in contrib/sendunsentbugmail.pl;
/cvsroot/mozilla/webtools/bugzilla/contrib/sendunsentbugmail.pl,v  <-- 
sendunsentbugmail.pl
new revision: 1.4; previous revision: 1.3
done
Status: NEW → RESOLVED
Last Resolved: 14 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.