Last modified: 2010-05-15 15:59:43 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T19455, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 17455 - Could not send confirmation mail. Check address for invalid characters. Mailer returned: 1
Could not send confirmation mail. Check address for invalid characters. Maile...
Status: RESOLVED INVALID
Product: MediaWiki
Classification: Unclassified
Email (Other open bugs)
1.11.x
PC Linux
: Normal normal (vote)
: ---
Assigned To: Nobody - You can work on this!
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2009-02-11 23:52 UTC by Isdal
Modified: 2010-05-15 15:59 UTC (History)
3 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description Isdal 2009-02-11 23:52:48 UTC
On ubuntu 8.04: 
Mediawiki version: 1:1.11.2-2ubuntu0.2

======Description ============
With email confirmations enabled it will send out an confirmation email on account creation. However, after account creation an error message displays: 'Could not send confirmation mail. Check address for invalid characters. Mailer returned: 1'
The mail is sent properly despite the error message.

======Analysis================
UserMailer::userMailer is expected to return '' on success and [error message as a string] on error.
But, according to the pear api: ->send returns true on success and e Pear_error oject on failure
see: http://pear.php.net/manual/en/package.mail.mail.send.php

======Workaround=============
Edit mediawiki/includes/UserMailer.php:154

== previous code ==
else
   return $mail_object->send($dest, $headers, $body);
===================
 
== workaround =====
else {
   $e = $mail_object->send($dest, $headers, $body);
   if ( $e == true ) {
      return '';
   } else {
      return $e;
   }
}
Comment 1 Siebrand Mazeland 2009-02-15 00:39:10 UTC
Please update to a more recent version of MediaWiki like 1.13.4. UserMailer.php has changed since 1.11.2.
Comment 2 Chad H. 2009-05-15 17:59:15 UTC
Marking as INVALID.

UserMailer::send() should return true on success or a WikiError on failure, not a String. See r30896 for more info.

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links