Revert "MDL-34794 mod_assign: Fixed up style after integration"
authorEloy Lafuente (stronk7) <stronk7@moodle.org>
Fri, 14 Sep 2012 08:47:04 +0000 (10:47 +0200)
committerEloy Lafuente (stronk7) <stronk7@moodle.org>
Fri, 14 Sep 2012 08:47:04 +0000 (10:47 +0200)
mod/assign/lib.php
mod/assign/locallib.php

index d28db35..8aadd07 100644 (file)
@@ -56,11 +56,9 @@ function assign_delete_instance($id) {
 
 /**
  * This function is used by the reset_course_userdata function in moodlelib.
- *
  * This function will remove all assignment submissions and feedbacks in the database
  * and clean up any related data.
- *
- * @param stdClass $data the data submitted from the reset course.
+ * @param $data the data submitted from the reset course.
  * @return array status array
  */
 function assign_reset_userdata($data) {
@@ -68,10 +66,10 @@ function assign_reset_userdata($data) {
     require_once($CFG->dirroot . '/mod/assign/locallib.php');
 
     $status = array();
-    $params = array('courseid' => $data->courseid);
-    $sql = "SELECT a.id FROM {assign} a WHERE a.course = :courseid";
-    $course = $DB->get_record('course', array('id' => $data->courseid), '*', MUST_EXIST);
-    if ($assigns = $DB->get_records_sql($sql, $params)) {
+    $params = array('courseid'=>$data->courseid);
+    $sql = "SELECT a.id FROM {assign} a WHERE a.course=:courseid";
+    $course = $DB->get_record('course', array('id'=> $data->courseid), '*', MUST_EXIST);
+    if ($assigns = $DB->get_records_sql($sql,$params)) {
         foreach ($assigns as $assign) {
             $cm = get_coursemodule_from_instance('assign', $assign->id, $data->courseid, false, MUST_EXIST);
             $context = context_module::instance($cm->id);
@@ -89,15 +87,12 @@ function assign_reset_userdata($data) {
  * @param string $type Optional type of assignment to limit the reset to a particular assignment type
  */
 function assign_reset_gradebook($courseid, $type='') {
-    global $DB;
+    global $CFG, $DB;
 
     $params = array('moduletype'=>'assign','courseid'=>$courseid);
     $sql = 'SELECT a.*, cm.idnumber as cmidnumber, a.course as courseid
-              FROM {assign} a, {course_modules} cm, {modules} m
-             WHERE m.name = :moduletype
-               AND m.id = cm.module
-               AND cm.instance = a.id
-               AND a.course = :courseid';
+            FROM {assign} a, {course_modules} cm, {modules} m
+            WHERE m.name=:moduletype AND m.id=cm.module AND cm.instance=a.id AND a.course=:courseid';
 
     if ($assignments = $DB->get_records_sql($sql,$params)) {
         foreach ($assignments as $assignment) {
@@ -109,7 +104,7 @@ function assign_reset_gradebook($courseid, $type='') {
 /**
  * Implementation of the function for printing the form elements that control
  * whether the course reset functionality affects the assignment.
- * @param moodleform $mform form passed by reference
+ * @param $mform form passed by reference
  */
 function assign_reset_course_form_definition(&$mform) {
     $mform->addElement('header', 'assignheader', get_string('modulenameplural', 'assign'));
index fef9171..1aab6bc 100644 (file)
@@ -563,12 +563,12 @@ class assign {
     }
 
     /**
-     * Actual implementation of the reset course functionality, delete all the
-     * assignment submissions for course $data->courseid.
-     *
-     * @param stdClass $data the data submitted from the reset course.
-     * @return array status array
-     */
+    * Actual implementation of the reset course functionality, delete all the
+    * assignment submissions for course $data->courseid.
+    *
+    * @param $data the data submitted from the reset course.
+    * @return array status array
+    */
     public function reset_userdata($data) {
         global $CFG,$DB;
 
@@ -579,40 +579,42 @@ class assign {
         if (!empty($data->reset_assign_submissions)) {
             // Delete files associated with this assignment.
             foreach ($this->submissionplugins as $plugin) {
+                $fileareas = array();
                 $fileareas = $plugin->get_file_areas();
                 foreach ($fileareas as $filearea) {
-                    $fs->delete_area_files($this->context->id, 'mod_assign', $filearea);
+                    $fs->delete_area_files($this->context->id,'mod_assign', $filearea);
                 }
 
                 if (!$plugin->delete_instance()) {
-                    $status[] = array('component' => $componentstr,
-                                      'item' => get_string('deleteallsubmissions', 'assign'),
-                                      'error' => $plugin->get_error());
+                    $status[] = array('component'=>$componentstr,
+                                      'item'=>get_string('deleteallsubmissions','assign'),
+                                      'error'=>$plugin->get_error());
                 }
             }
 
             foreach ($this->feedbackplugins as $plugin) {
+                $fileareas = array();
                 $fileareas = $plugin->get_file_areas();
                 foreach ($fileareas as $filearea) {
-                    $fs->delete_area_files($this->context->id, 'mod_assign', $filearea);
+                    $fs->delete_area_files($this->context->id,'mod_assign', $filearea);
                 }
 
                 if (!$plugin->delete_instance()) {
-                    $status[] = array('component' => $componentstr,
-                                      'item' => get_string('deleteallsubmissions', 'assign'),
-                                      'error' => $plugin->get_error());
+                    $status[] = array('component'=>$componentstr,
+                                      'item'=>get_string('deleteallsubmissions','assign'),
+                                      'error'=>$plugin->get_error());
                 }
             }
 
             $assignssql = "SELECT a.id
                              FROM {assign} a
-                           WHERE a.course = :course";
+                           WHERE a.course=:course";
             $params = array ("course" => $data->courseid);
 
             $DB->delete_records_select('assign_submission', "assignment IN ($assignssql)", $params);
-            $status[] = array('component' => $componentstr,
-                              'item' => get_string('deleteallsubmissions', 'assign'),
-                              'error' => false);
+            $status[] = array('component'=>$componentstr,
+                              'item'=>get_string('deleteallsubmissions','assign'),
+                              'error'=>false);
 
             if (empty($data->reset_gradebook_grades)) {
                 // Remove all grades from gradebook.
@@ -622,10 +624,10 @@ class assign {
         }
         // Updating dates - shift may be negative too.
         if ($data->timeshift) {
-            shift_course_mod_dates('assign', array('duedate', 'allowsubmissionsfromdate', 'cutoffdate'), $data->timeshift, $data->courseid);
-            $status[] = array('component' => $componentstr,
-                              'item' => get_string('datechanged'),
-                              'error' => false);
+            shift_course_mod_dates('assign', array('duedate', 'allowsubmissionsfromdate','cutoffdate'), $data->timeshift, $data->courseid);
+            $status[] = array('component'=>$componentstr,
+                              'item'=>get_string('datechanged'),
+                              'error'=>false);
         }
 
         return $status;
@@ -4066,7 +4068,7 @@ class assign {
      * @return void
      */
     private function process_revert_to_draft($userid = 0) {
-        global $DB, $USER;
+        global $DB;
 
         // Need grade permission
         require_capability('mod/assign:grade', $this->context);