Permalink
Browse files

MDL-39653 enrol_imsenterprise: only mail admins on change

The existing beavhiour was sending useless emails to admins and
generating warnings.

Thanks to Matthew Orlando for the suggested fix.
  • Loading branch information...
danpoltawski committed Sep 26, 2013
1 parent 7f3836d commit 75e19bf52b3a7f0b676c99c19d2fd47f6efe4ae3
Showing with 2 additions and 3 deletions.
  1. +2 −3 enrol/imsenterprise/lib.php
@@ -85,6 +85,7 @@ function cron() {
$this->logfp = fopen($logtolocation, 'a');
}
$fileisnew = false;
if ( file_exists($filename) ) {
@set_time_limit(0);
$starttime = time();
@@ -107,10 +108,8 @@ function cron() {
if(empty($prev_path) || ($filename != $prev_path)) {
$fileisnew = true;
} elseif(isset($prev_time) && ($filemtime <= $prev_time)) {
$fileisnew = false;
$this->log_line('File modification time is not more recent than last update - skipping processing.');
} elseif(isset($prev_md5) && ($md5 == $prev_md5)) {
$fileisnew = false;
$this->log_line('File MD5 hash is same as on last update - skipping processing.');
} else {
$fileisnew = true; // Let's process it!
@@ -217,7 +216,7 @@ function cron() {
$this->log_line('File not found: '.$filename);
}
if (!empty($mailadmins)) {
if (!empty($mailadmins) && $fileisnew) {
$msg = "An IMS enrolment has been carried out within Moodle.\nTime taken: $timeelapsed seconds.\n\n";
if(!empty($logtolocation)){
if($this->logfp){

0 comments on commit 75e19bf

Please sign in to comment.