Another subroutine to be moved out of globals.pl.
Created attachment 171593 [details] [diff] [review] Moves and re-names modtime Here, I moved and renamed ModTime, and I added some pod docs.
Attachment #171593 - Flags: review?
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+
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+
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
Last Resolved: 14 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.