naconk wrote:

Adds email functionality to CheckUser

This puts a checkbox on the CheckUser form to send the results to the editor's
email address.

attachment test.patch ignored as obsolete

Needs to be a unified diff file, not a copy of the file itself. Please resubmit
as a unified diff.

Hmm.. actually, it looks like a unified diff file, but there's something funky
about what you've done with the first file in it. The second file in the diff
looks OK.

FT2.wiki wrote:

Crossref to bug 15129 (checkuser output formats) as well.

mike.lifeguard+bugs wrote:

(In reply to comment #3)

but there's something funky
about what you've done with the first file in it. The second file in the diff
looks OK.

Does the first one need to be redone then? Did you ever get a chance to review the good file?

mike.lifeguard+bugs wrote:

Please don't adjust the importance or severity fields unless you're a developer.

*Bulk BZ Change: +Patch to open bugs with patches attached that are missing the keyword*

john wrote:

Comment on attachment 1530
Adds email functionality to CheckUser

Thanks for submitting a patch to MediaWiki. Unfortunately I'm going to have to
mark it as obsolete because it is not a valid diff (And probably wouldn't merge anymore anyway). If you could provide a recent diff of this patch working we would be glad to accept it.

I'm afraid I'm going to WONTFIX this bug; though I can appreciate that it sounds sensible, e-mail is an inherently insecure system and we cannot build in a tool that would post confidential data as a matter of course to insecure communications methods.

Sorry.