Permalink
Browse files

MDL-13572 message: added validation of the email address in the messa…

…ging preferences
  • Loading branch information...
1 parent 01f7f6f commit 925d068de11ff9e480c63f60f6867f0e25662a03 @andyjdavis andyjdavis committed Dec 13, 2011
Showing with 9 additions and 2 deletions.
  1. +9 −2 message/output/email/message_output_email.php
@@ -68,12 +68,19 @@ function send_message($eventdata) {
* @param object $mform preferences form class
*/
function config_form($preferences){
- global $USER;
- $string = get_string('email','message_email').': <input size="30" name="email_email" value="'.$preferences->email_email.'" />';
+ global $USER, $OUTPUT;
+
+ $inputattributes = array('size'=>'30', 'name'=>'email_email', 'value'=>$preferences->email_email);
+ $string = get_string('email','message_email') . ': ' . html_writer::empty_tag('input', $inputattributes);
if (empty($preferences->email_email) && !empty($preferences->userdefaultemail)) {
$string .= ' ('.get_string('default').': '.$preferences->userdefaultemail.')';
}
+
+ if (!empty($preferences->email_email) && !validate_email($preferences->email_email)) {
+ $string .= $OUTPUT->container(get_string('invalidemail'), 'error');
+ }
+
return $string;
}

0 comments on commit 925d068

Please sign in to comment.