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

Status: RESOLVED FIXED

(Webtools Graveyard :: Bonsai, defect)

Description

[Christopher Aillon (sabbatical, not receiving bugmail)]

Comment 1

[Christopher Aillon (sabbatical, not receiving bugmail)]

Attachment: Proposed fix

Untested since landfill seems to be down...

Comment 2

[timeless]

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

Comment 3

[Dave Miller [:justdave]]

reassign to patch author.

caillon: this got needs-worked.  were you going to update the patch?

Comment 4

[Hixie (not reading bugmail)]

justdave: seems unlikely

Comment 5

[cls]

Attachment: fix my peeps

Comment 6

[cls]

Patch has been checked in.