Fixing sender in work

main/work/work.lib.php

@@ -1735,16 +1735,17 @@ function display_list_users_without_publication($task_id) {
  */
 function send_reminder_users_without_publication($task_data) {
 	global $_course, $currentUserFirstName, $currentUserLastName, $currentUserEmail;
+    $sender_name = api_get_person_name(api_get_setting('administratorName'), api_get_setting('administratorSurname'), null, PERSON_NAME_EMAIL_ADDRESS);
 
 	$task_id = $task_data['id'];
 	$task_title = !empty($task_data['title']) ? $task_data['title'] : basename($task_data['url']);
 
 	$emailsubject = '[' . api_get_setting('siteName') . '] ';
-	$sender_name = api_get_person_name($currentUserFirstName, $currentUserLastName, null, PERSON_NAME_EMAIL_ADDRESS);
-	$email_admin = $currentUserEmail;
+    
 	// The body can be as long as you wish, and any combination of text and variables
+    $emailbody_user = get_lang('Dear')." ".$currentUserFirstName .' '.$currentUserLastName .", \n\n";            
 	$emailbody_user .= get_lang('ReminderToSubmitPendingTask')."\n".get_lang('CourseName').' : '.$_course['name']."\n";
-	$emailbody_user .= get_lang('WorkName').' : '.$task_title."\n\n".get_lang('Teacher').' : '.api_get_person_name($currentUserFirstName, $currentUserLastName)."\n".get_lang('Email').' : '.$currentUserEmail;
+	$emailbody_user .= get_lang('WorkName').' : '.$task_title."\n";
 
 	$list_users = get_list_users_without_publication($task_id);
     
@@ -1755,7 +1756,6 @@ function send_reminder_users_without_publication($task_data) {
         $mails_sent_to[] = $name_user;                
 	}    
     return $mails_sent_to;    
-    
 }
 
 /**

main/work/work.php

@@ -954,10 +954,7 @@ switch ($action) {
 				}
 			
 				if (count($emailto) > 0) {			
-			
 					$emailto = implode(',', $emailto);				
-					$emailfromaddr = api_get_setting('emailAdministrator');
-					$emailfromname = api_get_setting('siteName');
 					$emailsubject = "[" . api_get_setting('siteName') . "] ";
 					$sender_name = api_get_setting('administratorName').' '.api_get_setting('administratorSurname');
 					$email_admin = api_get_setting('emailAdministrator');
@@ -969,11 +966,12 @@ switch ($action) {
 					$emailbody .= get_lang('DateSent')." : ".api_format_date(api_get_local_time())."\n";
 					$emailbody .= get_lang('FileName')." : ".$title."\n\n".get_lang('DownloadLink')."\n";
 					$emailbody .= api_get_path(WEB_CODE_PATH)."work/work.php?".api_get_cidreq()."&curdirpath=".$my_cur_dir_path."\n\n" . api_get_setting('administratorName') . " " . api_get_setting('administratorSurname') . "\n" . get_lang('Manager') . " " . api_get_setting('siteName') . "\n" . get_lang('Email') . " : " . api_get_setting('emailAdministrator');
+                    
 				    // Here we are forming one large header line
 					// Every header must be followed by a \n except the last
 					@api_mail('', $emailto, $emailsubject, $emailbody, $sender_name,$email_admin);
 				
-					$emailbody_user = get_lang('Dear')." ".$currentUserFirstName .' '.$currentUserLastName ."\n";
+					$emailbody_user = get_lang('Dear')." ".$currentUserFirstName .' '.$currentUserLastName .", \n\n";
 					$emailbody_user .= get_lang('MessageConfirmSendingOfTask')."\n".get_lang('CourseName')." : ".$_course['name']."\n";
 					$emailbody_user .= get_lang('WorkName')." : ".substr($my_cur_dir_path, 0, -1)."\n";
 					$emailbody_user .= get_lang('DateSent')." : ".api_format_date(api_get_local_time())."\n";