All users were logged out of Bugzilla on October 13th, 2018

"Mail everyone on this page (1 people)" should be changed to something which is grammatically correct.

RESOLVED FIXED

Status

--
trivial
RESOLVED FIXED
16 years ago
2 years ago

People

(Reporter: caillon, Assigned: cls)

Tracking

Details

(URL)

Attachments

(1 attachment, 1 obsolete attachment)

Created attachment 97412 [details] [diff] [review]
Proposed fix

Untested since landfill seems to be down...

Comment 2

16 years ago
Comment on attachment 97412 [details] [diff] [review]
Proposed fix

bonsai didn't like this: 
>+my $s, $people;
my ($s, $people); #worked

here's the output for the testcase:
CVS Checkins

Checkins to directory plague/newdir/ and to file foo_install.txt since the
beginning of time:

Modify Query
Mail the one person on this page (1)
Show commands which could be used to back out these changes
This is Bonsai: a query interface to the CVS source repository
Attachment #97412 - Flags: needs-work+
reassign to patch author.

caillon: this got needs-worked.  were you going to update the patch?
Assignee: tara → caillon
justdave: seems unlikely
Assignee: caillon → tara
(Assignee)

Comment 5

14 years ago
Created attachment 159794 [details] [diff] [review]
fix my peeps
Assignee: tara → cls
Attachment #97412 - Attachment is obsolete: true
Status: NEW → ASSIGNED

Updated

14 years ago
Attachment #159794 - Flags: review+
(Assignee)

Comment 6

14 years ago
Patch has been checked in.
Status: ASSIGNED → RESOLVED
Last Resolved: 14 years ago
Resolution: --- → FIXED
Product: Webtools → Webtools Graveyard
You need to log in before you can comment on or make changes to this bug.