From 175cd46bb80def562e093c4bf5432335842e5e47 Mon Sep 17 00:00:00 2001
From: Artem Andreev <andreev.artem@gmail.com>
Date: Sat, 19 May 2012 12:41:46 +0400
Subject: [PATCH] Fixed #13: course reset for attendance activities now works.

---
 lang/en/attforblock.php |  2 ++
 lang/ru/attforblock.php |  2 ++
 lib.php                 | 78 +++++++++++++++++++++++++++++------------
 3 files changed, 59 insertions(+), 23 deletions(-)

diff --git a/lang/en/attforblock.php b/lang/en/attforblock.php
index 608e500..99ce54d 100644
--- a/lang/en/attforblock.php
+++ b/lang/en/attforblock.php
@@ -17,6 +17,7 @@ $string['addsession'] = 'Add session';
 $string['allcourses'] = 'All courses';
 $string['all'] = 'All';
 $string['allpast'] = 'All past';
+$string['attendancedata'] = 'Attendance data';
 $string['attendanceforthecourse'] = 'Attendance for the course';
 $string['attendancegrade'] = 'Attendance grade';
 $string['attendancenotstarted'] = 'Attendance has not started yet for this course';
@@ -158,6 +159,7 @@ $string['sortedgrid'] = 'Sorted grid';
 $string['sortedlist'] = 'Sorted list';
 $string['startofperiod'] = 'Start of period';
 $string['status'] = 'Status';
+$string['statuses'] = 'Statuses';
 $string['statusdeleted'] = 'Status deleted';
 $string['strftimedm'] = '%d.%m';
 $string['strftimedmy'] = '%d.%m.%Y';
diff --git a/lang/ru/attforblock.php b/lang/ru/attforblock.php
index ade25cb..b7d142a 100644
--- a/lang/ru/attforblock.php
+++ b/lang/ru/attforblock.php
@@ -17,6 +17,7 @@ $string['addsession'] = 'Добавить занятие';
 $string['all'] = 'Все';
 $string['allcourses'] = 'Все курсы';
 $string['allpast'] = 'Все прошедшие';
+$string['attendancedata'] = 'Информация о посещаемости';
 $string['attendanceforthecourse'] = 'Посещаемость для курса';
 $string['attendancegrade'] = 'Оценка за посещаемость';
 $string['attendancenotstarted'] = 'Пока нет отметок о посещаемости в данном курсе.';
@@ -145,6 +146,7 @@ $string['sortedgrid'] = 'Таблица';
 $string['sortedlist'] = 'Список';
 $string['startofperiod'] = 'Начало периода';
 $string['status'] = 'Статус';
+$string['statuses'] = 'Статусы';
 $string['statusdeleted'] = 'Статус удален';
 $string['strftimedm'] = '%d.%m';
 $string['strftimedmy'] = '%d.%m.%Y';
diff --git a/lib.php b/lib.php
index 38f81ee..e81fe29 100644
--- a/lib.php
+++ b/lib.php
@@ -28,6 +28,17 @@ function attforblock_supports($feature) {
     }
 }
 
+function att_add_default_statuses($attid) {
+    global $DB;
+
+    $statuses = $DB->get_records('attendance_statuses', array('attendanceid'=> 0), 'id');
+    foreach($statuses as $st) {
+        $rec = $st;
+        $rec->attendanceid = $attid;
+        $DB->insert_record('attendance_statuses', $rec);
+    }
+}
+
 function attforblock_add_instance($attforblock) {
 /// Given an object containing all the necessary data, 
 /// (defined by the form in mod.html) this function 
@@ -40,13 +51,8 @@ function attforblock_add_instance($attforblock) {
 
     $attforblock->id = $DB->insert_record('attforblock', $attforblock);
 
-    $statuses = $DB->get_records('attendance_statuses', array('attendanceid'=> 0), 'id');
-    foreach($statuses as $st) {
-        $rec = $st;
-        $rec->attendanceid = $attforblock->id;
-        $DB->insert_record('attendance_statuses', $rec);
-    }
-						
+    att_add_default_statuses($attforblock->id);
+
     attforblock_grade_item_update($attforblock);
 //	attforblock_update_grades($attforblock);
     return $attforblock->id;
@@ -54,12 +60,12 @@ function attforblock_add_instance($attforblock) {
 
 
 function attforblock_update_instance($attforblock) {
-/// Given an object containing all the necessary data, 
-/// (defined by the form in mod.html) this function 
+/// Given an object containing all the necessary data,
+/// (defined by the form in mod.html) this function
 /// will update an existing instance with new data.
 
     global $DB;
-    
+
     $attforblock->timemodified = time();
     $attforblock->id = $attforblock->instance;
 
@@ -75,7 +81,7 @@ function attforblock_update_instance($attforblock) {
 
 function attforblock_delete_instance($id) {
     global $DB;
-    
+
     if (! $attforblock = $DB->get_record('attforblock', array('id'=> $id))) {
         return false;
     }
@@ -138,24 +144,50 @@ function attforblock_reset_course_form_defaults($course) {
 function attforblock_reset_userdata($data) {
     global $DB;
 
+    $status = array();
+
+    $attids = array_keys($DB->get_records('attforblock', array('course'=> $data->courseid), '', 'id'));
+
     if (!empty($data->reset_attendance_log)) {
-		$sess = $DB->get_records('attendance_sessions', array('courseid'=> $data->courseid), '', 'id');
-		$slist = implode(',', array_keys($sess));
-    	$DB->delete_records_select('attendance_log', "sessionid IN ($slist)");
-        $DB->set_field('attendance_sessions', 'lasttaken', 0, array('courseid' => $data->courseid));
+		$sess = $DB->get_records_list('attendance_sessions', 'attendanceid', $attids, '', 'id');
+        if (!empty($sess)) {
+            list($sql, $params) = $DB->get_in_or_equal(array_keys($sess));
+            $DB->delete_records_select('attendance_log', "sessionid $sql", $params);
+            list($sql, $params) = $DB->get_in_or_equal($attids);
+            $DB->set_field_select('attendance_sessions', 'lasttaken', 0, "attendanceid $sql", $params);
+
+            $status[] = array(
+                'component' => get_string('modulenameplural', 'attforblock'),
+                'item' => get_string('attendancedata', 'attforblock'),
+                'error' => false
+            );
+        }
     }
+
     if (!empty($data->reset_attendance_statuses)) {
-    	$DB->delete_records('attendance_statuses', array('courseid'=> $data->courseid));
-	    $statuses = $DB->get_records('attendance_statuses', array('courseid'=> 0), 'id');
-		foreach($statuses as $stat) {
-			$rec = $stat;
-			$rec->courseid = $data->courseid;
-			$DB->insert_record('attendance_statuses', $rec);
-		}
+    	$DB->delete_records_list('attendance_statuses', 'attendanceid', $attids);
+        foreach($attids as $attid) {
+            att_add_default_statuses($attid);
+        }
+
+        $status[] = array(
+            'component' => get_string('modulenameplural', 'attforblock'),
+            'item' => get_string('sessions', 'attforblock'),
+            'error' => false
+        );
     }
+
     if (!empty($data->reset_attendance_sessions)) {
-    	$DB->delete_records('attendance_sessions', array('courseid'=> $data->courseid));
+    	$DB->delete_records_list('attendance_sessions', 'attendanceid', $attids);
+
+        $status[] = array(
+            'component' => get_string('modulenameplural', 'attforblock'),
+            'item' => get_string('statuses', 'attforblock'),
+            'error' => false
+        );
     }
+
+    return $status;
 }
 
 function attforblock_user_outline($course, $user, $mod, $attforblock) {