Move PerformSubsts() out of globals.pl

RESOLVED FIXED in Bugzilla 2.20

Status

()

--
enhancement
RESOLVED FIXED
14 years ago
6 years ago

People

(Reporter: mkanat, Assigned: mkanat)

Tracking

2.19.1
Bugzilla 2.20
Bug Flags:
approval +

Details

Attachments

(1 attachment)

(Assignee)

Description

14 years ago
The PerformSubsts() function probably belongs in BugMail.pm.
(Assignee)

Updated

14 years ago
Severity: normal → enhancement
(Assignee)

Comment 1

14 years ago
Created attachment 172226 [details] [diff] [review]
Move PerformSubsts without a rename

OK, here we go. I moved the function, added some documentation, and I also
added a variable-list to the subroutine definition. I've tested every call of
PerformSubsts on landfill with this patch and they all work.
Attachment #172226 - Flags: review?
(Assignee)

Updated

14 years ago
Status: NEW → ASSIGNED

Updated

14 years ago
Attachment #172226 - Flags: review? → review+

Updated

14 years ago
Flags: approval?
Target Milestone: --- → Bugzilla 2.20
This patch has a line in it that I think belongs to bug 277437.  One or the
other of these is going to conflict anyway.  Probably to keep it clean you
should remove the MessageToMTA line from this one when you commit.
Flags: approval? → approval+
(Assignee)

Comment 3

14 years ago
OK, whoever checks this in -- please remove the MessageToMTA line from the
EXPORT statement. I agree with justdave. :-)

Comment 4

14 years ago
Max, your patches are beginning to conflict with themselves in some cases; I 
had to manually apply the showdependencygraph.cgi change because there was 
another 'use' from a previous move messing up the context. If there's an order 
in which your patches are to be applied, please make a note of that.

I removed the MessageToMTA line, as per instructions. New patches are much 
preferred, because I almost missed that line.


Checking in globals.pl;
/cvsroot/mozilla/webtools/bugzilla/globals.pl,v  <--  globals.pl
new revision: 1.296; previous revision: 1.295
done
Checking in showdependencygraph.cgi;
/cvsroot/mozilla/webtools/bugzilla/showdependencygraph.cgi,v  <--  
showdependencygraph.cgi
new revision: 1.38; previous revision: 1.37
done
Checking in Bugzilla/BugMail.pm;
/cvsroot/mozilla/webtools/bugzilla/Bugzilla/BugMail.pm,v  <--  BugMail.pm
new revision: 1.27; previous revision: 1.26
done
Status: ASSIGNED → RESOLVED
Last Resolved: 14 years ago
Resolution: --- → FIXED
QA Contact: matty_is_a_geek → default-qa
You need to log in before you can comment on or make changes to this bug.