Permalink
Browse files

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

…ging preferences
  • Loading branch information...
1 parent 464ad0a commit 63601aa655dd85e45991ea37b2e241c861af823c @andyjdavis andyjdavis committed with Sam Hemelryk 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 63601aa

Please sign in to comment.