Bug#610773: [moodle-packaging] Bug#610773: fixed patch

2011-01-29 Thread Mònica Ramírez Arceda
Hi, You're right :-) Maybe we could add an extra if in each branch? I attached the patch again with this idea (and with the "fullname" error fixed). Thanks for your work, Mònica --- moodlelib.php.ori 2010-08-24 21:25:20.0 +0200 +++ moodlelib.php 2011-01-29 18:42:49.0 +0100 @@ -4

Bug#610773: [moodle-packaging] Bug#610773: fixed patch

2011-01-29 Thread Hubert Chathi
Hello Mònica, Thank you for the suggestion and the patch. I don't think that it is is quite correct, though. If $replyto is specified, then the result will be that the mail will end up having two addresses in the reply-to field: the from address, and the $replyto address. But the intended behav

Bug#610773: fixed patch

2011-01-28 Thread Mònica Ramírez Arceda
Hi, The patch attached in the previous message is not correct. I've improved it and attached it again. Thanks for your work! --- moodlelib.php.ori 2010-08-24 21:25:20.0 +0200 +++ moodlelib.php 2011-01-28 22:07:35.0 +0100 @@ -4275,9 +4275,15 @@ if (is_string($from)) { // So we