Create template for mail by Remind Course Expiration cron - refs BT#9…

…898 #TMI
chamilo · Aug 5, 2015 · 0934761 · 0934761
1 parent a6de770
commit 0934761
Show file tree

Hide file tree

Showing 3 changed files with 35 additions and 24 deletions.
diff --git a/main/cron/remind_course_expiration.php b/main/cron/remind_course_expiration.php
@@ -68,7 +68,7 @@
     $query = "
         SELECT sessionUser.session_id, sessionUser.user_id, session.name, session.access_end_date 
         FROM $sessionUserTable AS sessionUser
-        INNER JOIN $sessionUserTable AS session
+        INNER JOIN $sessionTable AS session
         ON sessionUser.session_id = session.id
         WHERE
             session_id = $sessionId$userIds";
@@ -83,15 +83,6 @@
 
 if ($usersToBeReminded) {
     $today = date_create($today);
-    $platformLanguage = api_get_setting("platformLanguage");
-    $subject = sprintf(
-        get_lang(
-            "MailCronCourseExpirationReminderSubject",
-            null,
-            $platformLanguage
-        ),
-        api_get_setting("Institution")
-    );
     $administrator = array(
         'completeName' => api_get_person_name(
             api_get_setting("administratorName"),
@@ -125,24 +116,42 @@
                     'limit' => '1'
                 )
             );
-            $body = sprintf(
-                get_lang(
-                    'MailCronCourseExpirationReminderBody',
-                    null,
-                    $platformLanguage
-                ),
-                $userCompleteName,
-                $session['name'],
-                $session['access_end_date'],
-                $daysRemaining->format("%d"),
-                $result[0]['url'],
-                api_get_setting("siteName")
+
+            $subjectTemplate = new Template(null, false, false, false, false, false);
+            $subjectTemplate->assign(
+                'session_access_end_date',
+                $session['access_end_date']
+            );
+            $subjectTemplate->assign(
+                'remaining_days',
+                $daysRemaining->format("%d")
+            );
+
+            $subjectLayout = $subjectTemplate->get_template(
+                'mail/cron_remind_course_expiration_subject.tpl'
+            );
+
+            $bodyTemplate = new Template(null, false, false, false, false, false);
+            $bodyTemplate->assign('complete_user_name', $userCompleteName);
+            $bodyTemplate->assign('session_name', $session['name']);
+            $bodyTemplate->assign(
+                'session_access_end_date',
+                $session['access_end_date']
+            );
+            $bodyTemplate->assign(
+                'remaining_days',
+                $daysRemaining->format("%d")
             );
+
+            $bodyLayout = $bodyTemplate->get_template(
+                'mail/cron_remind_course_expiration_body.tpl'
+            );
+
             api_mail_html(
                 $userCompleteName,
                 $user['email'],
-                $subject,
-                $body,
+                $subjectTemplate->fetch($subjectLayout),
+                $bodyTemplate->fetch($bodyLayout),
                 $administrator['completeName'],
                 $administrator['email']
             );

diff --git a/main/template/default/mail/cron_remind_course_expiration_body.tpl b/main/template/default/mail/cron_remind_course_expiration_body.tpl
@@ -0,0 +1 @@
+{{ 'MailCronCourseExpirationReminderBody'|get_lang|format(complete_user_name, session_name, session_access_end_date, remaining_days, _p.web, _s.institution) }}
diff --git a/main/template/default/mail/cron_remind_course_expiration_subject.tpl b/main/template/default/mail/cron_remind_course_expiration_subject.tpl
@@ -0,0 +1 @@
+{{ 'MailCronCourseExpirationReminderSubject'|get_lang|format(_s.institution) }}