@@ -101,7 +101,7 @@ discard block |
||
| 101 | 101 | * @uses FEATURE_GRADE_OUTCOMES |
| 102 | 102 | * @uses FEATURE_SHOW_DESCRIPTION |
| 103 | 103 | * @param string $feature |
| 104 | - * @return mixed True if yes (some features may use other values) |
|
| 104 | + * @return null|boolean True if yes (some features may use other values) |
|
| 105 | 105 | */ |
| 106 | 106 | function bigbluebuttonbn_supports($feature) { |
| 107 | 107 | if (!$feature) { |
@@ -220,7 +220,7 @@ discard block |
||
| 220 | 220 | * |
| 221 | 221 | * @param object $bigbluebuttonbn Id of the module instance |
| 222 | 222 | * |
| 223 | - * @return bool Success/Failure |
|
| 223 | + * @return boolean|null Success/Failure |
|
| 224 | 224 | */ |
| 225 | 225 | function bigbluebuttonbn_delete_instance_log($bigbluebuttonbn) { |
| 226 | 226 | global $DB; |
@@ -241,7 +241,7 @@ discard block |
||
| 241 | 241 | * @param object $mod |
| 242 | 242 | * @param object $bigbluebuttonbn |
| 243 | 243 | * |
| 244 | - * @return bool |
|
| 244 | + * @return string |
|
| 245 | 245 | */ |
| 246 | 246 | function bigbluebuttonbn_user_outline($course, $user, $mod, $bigbluebuttonbn) { |
| 247 | 247 | if ($completed = bigbluebuttonbn_user_complete($course, $user, $mod, $bigbluebuttonbn)) { |
@@ -782,7 +782,7 @@ discard block |
||
| 782 | 782 | * @param stdClass $context context object |
| 783 | 783 | * @param string $filearea file area |
| 784 | 784 | * |
| 785 | - * @return false|null false if file not valid |
|
| 785 | + * @return boolean false if file not valid |
|
| 786 | 786 | */ |
| 787 | 787 | function bigbluebuttonbn_pluginfile_valid($context, $filearea) { |
| 788 | 788 | |
@@ -46,7 +46,7 @@ discard block |
||
| 46 | 46 | public static function notification_process($bigbluebuttonbn, $action) { |
| 47 | 47 | global $USER; |
| 48 | 48 | // Prepare message. |
| 49 | - $msg = (object) array(); |
|
| 49 | + $msg = (object)array(); |
|
| 50 | 50 | // Build the message_body. |
| 51 | 51 | $msg->action = $action; |
| 52 | 52 | $msg->activity_type = ''; |
@@ -71,26 +71,26 @@ discard block |
||
| 71 | 71 | * @return string |
| 72 | 72 | */ |
| 73 | 73 | public static function notification_msg_html($msg) { |
| 74 | - $messagetext = '<p>'.$msg->activity_type.' "'.$msg->activity_title.'" '. |
|
| 75 | - get_string('email_body_notification_meeting_has_been', 'bigbluebuttonbn').' '.$msg->action.'.</p>'."\n"; |
|
| 76 | - $messagetext .= '<p><b>'.$msg->activity_title.'</b> '. |
|
| 77 | - get_string('email_body_notification_meeting_details', 'bigbluebuttonbn').':'."\n"; |
|
| 78 | - $messagetext .= '<table border="0" style="margin: 5px 0 0 20px"><tbody>'."\n"; |
|
| 79 | - $messagetext .= '<tr><td style="font-weight:bold;color:#555;">'. |
|
| 80 | - get_string('email_body_notification_meeting_title', 'bigbluebuttonbn').': </td><td>'."\n"; |
|
| 81 | - $messagetext .= $msg->activity_title.'</td></tr>'."\n"; |
|
| 82 | - $messagetext .= '<tr><td style="font-weight:bold;color:#555;">'. |
|
| 83 | - get_string('email_body_notification_meeting_description', 'bigbluebuttonbn').': </td><td>'."\n"; |
|
| 84 | - $messagetext .= $msg->activity_description.'</td></tr>'."\n"; |
|
| 85 | - $messagetext .= '<tr><td style="font-weight:bold;color:#555;">'. |
|
| 86 | - get_string('email_body_notification_meeting_start_date', 'bigbluebuttonbn').': </td><td>'."\n"; |
|
| 87 | - $messagetext .= $msg->activity_openingtime.'</td></tr>'."\n"; |
|
| 88 | - $messagetext .= '<tr><td style="font-weight:bold;color:#555;">'. |
|
| 89 | - get_string('email_body_notification_meeting_end_date', 'bigbluebuttonbn').': </td><td>'."\n"; |
|
| 90 | - $messagetext .= $msg->activity_closingtime.'</td></tr>'."\n"; |
|
| 91 | - $messagetext .= '<tr><td style="font-weight:bold;color:#555;">'.$msg->action.' '. |
|
| 92 | - get_string('email_body_notification_meeting_by', 'bigbluebuttonbn').': </td><td>'."\n"; |
|
| 93 | - $messagetext .= $msg->activity_owner.'</td></tr></tbody></table></p>'."\n"; |
|
| 74 | + $messagetext = '<p>' . $msg->activity_type . ' "' . $msg->activity_title . '" ' . |
|
| 75 | + get_string('email_body_notification_meeting_has_been', 'bigbluebuttonbn') . ' ' . $msg->action . '.</p>' . "\n"; |
|
| 76 | + $messagetext .= '<p><b>' . $msg->activity_title . '</b> ' . |
|
| 77 | + get_string('email_body_notification_meeting_details', 'bigbluebuttonbn') . ':' . "\n"; |
|
| 78 | + $messagetext .= '<table border="0" style="margin: 5px 0 0 20px"><tbody>' . "\n"; |
|
| 79 | + $messagetext .= '<tr><td style="font-weight:bold;color:#555;">' . |
|
| 80 | + get_string('email_body_notification_meeting_title', 'bigbluebuttonbn') . ': </td><td>' . "\n"; |
|
| 81 | + $messagetext .= $msg->activity_title . '</td></tr>' . "\n"; |
|
| 82 | + $messagetext .= '<tr><td style="font-weight:bold;color:#555;">' . |
|
| 83 | + get_string('email_body_notification_meeting_description', 'bigbluebuttonbn') . ': </td><td>' . "\n"; |
|
| 84 | + $messagetext .= $msg->activity_description . '</td></tr>' . "\n"; |
|
| 85 | + $messagetext .= '<tr><td style="font-weight:bold;color:#555;">' . |
|
| 86 | + get_string('email_body_notification_meeting_start_date', 'bigbluebuttonbn') . ': </td><td>' . "\n"; |
|
| 87 | + $messagetext .= $msg->activity_openingtime . '</td></tr>' . "\n"; |
|
| 88 | + $messagetext .= '<tr><td style="font-weight:bold;color:#555;">' . |
|
| 89 | + get_string('email_body_notification_meeting_end_date', 'bigbluebuttonbn') . ': </td><td>' . "\n"; |
|
| 90 | + $messagetext .= $msg->activity_closingtime . '</td></tr>' . "\n"; |
|
| 91 | + $messagetext .= '<tr><td style="font-weight:bold;color:#555;">' . $msg->action . ' ' . |
|
| 92 | + get_string('email_body_notification_meeting_by', 'bigbluebuttonbn') . ': </td><td>' . "\n"; |
|
| 93 | + $messagetext .= $msg->activity_owner . '</td></tr></tbody></table></p>' . "\n"; |
|
| 94 | 94 | return $messagetext; |
| 95 | 95 | } |
| 96 | 96 | |
@@ -106,13 +106,13 @@ discard block |
||
| 106 | 106 | $coursemodinfo = \course_modinfo::instance($bigbluebuttonbn->course); |
| 107 | 107 | $course = $coursemodinfo->get_course($bigbluebuttonbn->course); |
| 108 | 108 | // Complete message. |
| 109 | - $msg = (object) array(); |
|
| 109 | + $msg = (object)array(); |
|
| 110 | 110 | $msg->user_name = fullname($sender); |
| 111 | 111 | $msg->user_email = $sender->email; |
| 112 | 112 | $msg->course_name = $course->fullname; |
| 113 | - $message .= '<p><hr/><br/>'.get_string('email_footer_sent_by', 'bigbluebuttonbn').' '. |
|
| 114 | - $msg->user_name.'('.$msg->user_email.') '; |
|
| 115 | - $message .= get_string('email_footer_sent_from', 'bigbluebuttonbn').' '.$msg->course_name.'.</p>'; |
|
| 113 | + $message .= '<p><hr/><br/>' . get_string('email_footer_sent_by', 'bigbluebuttonbn') . ' ' . |
|
| 114 | + $msg->user_name . '(' . $msg->user_email . ') '; |
|
| 115 | + $message .= get_string('email_footer_sent_from', 'bigbluebuttonbn') . ' ' . $msg->course_name . '.</p>'; |
|
| 116 | 116 | // Process the message sending. |
| 117 | 117 | foreach (self::users_to_notify($bigbluebuttonbn->course) as $user) { |
| 118 | 118 | if ($user->id != $sender->id) { |