Closed
Bug 278791
Opened 20 years ago
Closed 20 years ago
Move ModTime() to Bugzilla::Util
Categories
(Bugzilla :: Bugzilla-General, enhancement)
Tracking
()
RESOLVED
FIXED
Bugzilla 2.20
People
(Reporter: mkanat, Assigned: mkanat)
References
Details
Attachments
(1 file)
3.17 KB,
patch
|
goobix
:
review+
|
Details | Diff | Splinter Review |
Another subroutine to be moved out of globals.pl.
Assignee | ||
Comment 1•20 years ago
|
||
Here, I moved and renamed ModTime, and I added some pod docs.
Attachment #171593 -
Flags: review?
Assignee | ||
Updated•20 years ago
|
Version: 2.19.1 → 2.19.2
Assignee | ||
Updated•20 years ago
|
Status: NEW → ASSIGNED
Assignee | ||
Updated•20 years ago
|
Blocks: bz-globals
Comment 2•20 years ago
|
||
Comment on attachment 171593 [details] [diff] [review] Moves and re-names modtime The last documentation line is a little too long and should be split in 2 lines at least. But that looks fixable upon checkin.
Attachment #171593 -
Flags: review? → review+
Updated•20 years ago
|
Flags: approval?
Target Milestone: --- → Bugzilla 2.20
Comment 3•20 years ago
|
||
I was going to ask about tests since we have unit tests for most stuff in Bugzilla::Util now, but I guess it would be hard to test this since it depends on external data on the filesystem :)
Flags: approval? → approval+
Comment 4•20 years ago
|
||
Yay Max... One more down, 52 to go! :) Checking in globals.pl; /cvsroot/mozilla/webtools/bugzilla/globals.pl,v <-- globals.pl new revision: 1.287; previous revision: 1.286 done Checking in showdependencygraph.cgi; /cvsroot/mozilla/webtools/bugzilla/showdependencygraph.cgi,v <-- showdependencygraph.cgi new revision: 1.37; previous revision: 1.36 done Checking in Bugzilla/Util.pm; /cvsroot/mozilla/webtools/bugzilla/Bugzilla/Util.pm,v <-- Util.pm new revision: 1.18; previous revision: 1.17 done
Status: ASSIGNED → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•