MDL-14679 fixed remaining old style set_field()s

diff --git a/admin/xmldb/actions/test/test.class.php b/admin/xmldb/actions/test/test.class.php
index 44317de..f3f0aff 100644 (file)
--- a/admin/xmldb/actions/test/test.class.php
+++ b/admin/xmldb/actions/test/test.class.php
@@ -1059,19 +1059,16 @@ class test extends XMLDBAction {
             $rec->name = 'updatelobs';
         /// Calculate its length
             $textlen = $textlib->strlen($fulltext);
-            if ($DB->set_field('newnameforthetable', 'intro', $rec->intro, array('name'=>$rec->name))) {
-                if ($new = $DB->get_record('newnameforthetable', array('id'=>$rec->id))) {
-                    $newtextlen = $textlib->strlen($new->intro);
-                    if ($fulltext === $new->intro) {
-                        $test->sql = array($newtextlen . ' cc. (text) sent and received ok');
-                        $test->status = true;
-                    } else {
-                        $test->error = $DB->get_last_error();
-                        $test->sql = array($newtextlen . ' cc. (text) transfer failed. Data changed!');
-                        $test->status = false;
-                    }
+            $DB->set_field('newnameforthetable', 'intro', $rec->intro, array('name'=>$rec->name));
+            if ($new = $DB->get_record('newnameforthetable', array('id'=>$rec->id))) {
+                $newtextlen = $textlib->strlen($new->intro);
+                if ($fulltext === $new->intro) {
+                    $test->sql = array($newtextlen . ' cc. (text) sent and received ok');
+                    $test->status = true;
                 } else {
                     $test->error = $DB->get_last_error();
+                    $test->sql = array($newtextlen . ' cc. (text) transfer failed. Data changed!');
+                    $test->status = false;
                 }
             } else {
                 $test->error = $DB->get_last_error();
@@ -1089,19 +1086,16 @@ class test extends XMLDBAction {
             $rec->name = 'updatelobs';
         /// Calculate its length
             $textlen = strlen($rec->avatar);
-            if ($DB->set_field('newnameforthetable', 'avatar', $rec->avatar, array('name'=>$rec->name))) {
-                if ($new = $DB->get_record('newnameforthetable', array('id'=>$rec->id))) {
-                    $newtextlen = strlen($new->avatar);
-                    if ($rec->avatar === $new->avatar) {
-                        $test->sql = array($newtextlen . ' bytes (binary) sent and received ok');
-                        $test->status = true;
-                    } else {
-                        $test->error = $DB->get_last_error();
-                        $test->sql = array($newtextlen . ' bytes (binary) transfer failed. Data changed!');
-                        $test->status = false;
-                    }
+            $DB->set_field('newnameforthetable', 'avatar', $rec->avatar, array('name'=>$rec->name));
+            if ($new = $DB->get_record('newnameforthetable', array('id'=>$rec->id))) {
+                $newtextlen = strlen($new->avatar);
+                if ($rec->avatar === $new->avatar) {
+                    $test->sql = array($newtextlen . ' bytes (binary) sent and received ok');
+                    $test->status = true;
                 } else {
                     $test->error = $DB->get_last_error();
+                    $test->sql = array($newtextlen . ' bytes (binary) transfer failed. Data changed!');
+                    $test->status = false;
                 }
             } else {
                 $test->error = $DB->get_last_error();

diff --git a/auth/email/auth.php b/auth/email/auth.php
index 01612b6..4436097 100644 (file)
--- a/auth/email/auth.php
+++ b/auth/email/auth.php
@@ -131,12 +131,8 @@ class auth_plugin_email extends auth_plugin_base {
                 return AUTH_CONFIRM_ERROR;
 
             } else if ($user->secret == $confirmsecret) {   // They have provided the secret key to get in
-                if (!$DB->set_field("user", "confirmed", 1, array("id"=>$user->id))) {
-                    return AUTH_CONFIRM_FAIL;
-                }
-                if (!$DB->set_field("user", "firstaccess", time(), array("id"=>$user->id))) {
-                    return AUTH_CONFIRM_FAIL;
-                }
+                $DB->set_field("user", "confirmed", 1, array("id"=>$user->id));
+                $DB->set_field("user", "firstaccess", time(), array("id"=>$user->id));
                 return AUTH_CONFIRM_OK;
             }
         } else {

diff --git a/auth/ldap/auth.php b/auth/ldap/auth.php
index 5533439..0ef5c40 100644 (file)
--- a/auth/ldap/auth.php
+++ b/auth/ldap/auth.php
@@ -549,12 +549,8 @@ class auth_plugin_ldap extends auth_plugin_base {
                 if (!$this->user_activate($username)) {
                     return AUTH_CONFIRM_FAIL;
                 }
-                if (!$DB->set_field('user', 'confirmed', 1, array('id'=>$user->id))) {
-                    return AUTH_CONFIRM_FAIL;
-                }
-                if (!$DB->set_field('user', 'firstaccess', time(), array('id'=>$user->id))) {
-                    return AUTH_CONFIRM_FAIL;
-                }
+                $DB->set_field('user', 'confirmed', 1, array('id'=>$user->id));
+                $DB->set_field('user', 'firstaccess', time(), array('id'=>$user->id));
                 return AUTH_CONFIRM_OK;
             }
         } else {

diff --git a/auth/manual/auth.php b/auth/manual/auth.php
index e2b4906..68cef58 100644 (file)
--- a/auth/manual/auth.php
+++ b/auth/manual/auth.php
@@ -137,12 +137,8 @@ class auth_plugin_manual extends auth_plugin_base {
             if ($user->confirmed) {
                 return AUTH_CONFIRM_ALREADY;
             } else {
-                if (!$DB->set_field("user", "confirmed", 1, array("id"=>$user->id))) {
-                    return AUTH_CONFIRM_FAIL;
-                }
-                if (!$DB->set_field("user", "firstaccess", time(), array("id"=>$user->id))) {
-                    return AUTH_CONFIRM_FAIL;
-                }
+                $DB->set_field("user", "confirmed", 1, array("id"=>$user->id));
+                $DB->set_field("user", "firstaccess", time(), array("id"=>$user->id));
                 return AUTH_CONFIRM_OK;
             }
         } else  {

diff --git a/course/format/topics/format.php b/course/format/topics/format.php
index c1cb1cc..b40ecaa 100644 (file)
--- a/course/format/topics/format.php
+++ b/course/format/topics/format.php
@@ -43,9 +43,7 @@ $context = get_context_instance(CONTEXT_COURSE, $course->id);
 
 if (($marker >=0) && has_capability('moodle/course:setcurrentsection', $context) && confirm_sesskey()) {
     $course->marker = $marker;
-    if (! $DB->set_field("course", "marker", $marker, array("id"=>$course->id))) {
-        print_error("cannotmarktopic");
-    }
+    $DB->set_field("course", "marker", $marker, array("id"=>$course->id));
 }
 
 $streditsummary  = get_string('editsummary');

diff --git a/course/lib.php b/course/lib.php
index 09678a1..d450547 100644 (file)
--- a/course/lib.php
+++ b/course/lib.php
@@ -1735,9 +1735,7 @@ function rebuild_course_cache($courseid=0, $clearonly=false) {
     if ($rs = $DB->get_recordset("course", $select,'','id,fullname')) {
         foreach ($rs as $course) {
             $modinfo = serialize(get_array_of_activities($course->id));
-            if (!$DB->set_field("course", "modinfo", $modinfo, array("id"=>$course->id))) {
-                echo $OUTPUT->notification("Could not cache module information for course '" . format_string($course->fullname) . "'!");
-            }
+            $DB->set_field("course", "modinfo", $modinfo, array("id"=>$course->id));
             // update cached global COURSE too ;-)
             if ($course->id == $COURSE->id) {
                 $COURSE->modinfo = $modinfo;
@@ -2547,11 +2545,8 @@ function add_mod_to_section($mod, $beforemod=NULL) {
             $newsequence = "$section->sequence,$mod->coursemodule";
         }
 
-        if ($DB->set_field("course_sections", "sequence", $newsequence, array("id"=>$section->id))) {
-            return $section->id;     // Return course_sections ID that was used.
-        } else {
-            return 0;
-        }
+        $DB->set_field("course_sections", "sequence", $newsequence, array("id"=>$section->id));
+        return $section->id;     // Return course_sections ID that was used.
 
     } else {  // Insert a new record
         $section->course   = $mod->course;
@@ -2604,7 +2599,7 @@ function set_coursemodule_visible($id, $visible, $prevstateoverrides=false) {
     if ($grade_item !== false) {
         $grade_item->set_hidden(!$visible);
     }
-    
+
     if ($prevstateoverrides) {
         if ($visible == '0') {
             // Remember the current visible state so we can toggle this back.
@@ -2703,12 +2698,9 @@ function move_section($course, $section, $move) {
         return false;
     }
 
-    if (!$DB->set_field("course_sections", "section", $sectiondest, array("id"=>$sectionrecord->id))) {
-        return false;
-    }
-    if (!$DB->set_field("course_sections", "section", $section, array("id"=>$sectiondestrecord->id))) {
-        return false;
-    }
+    $DB->set_field("course_sections", "section", $sectiondest, array("id"=>$sectionrecord->id));
+    $DB->set_field("course_sections", "section", $section, array("id"=>$sectiondestrecord->id));
+
     // if the focus is on the section that is being moved, then move the focus along
     if (isset($USER->display[$course->id]) and ($USER->display[$course->id] == $section)) {
         course_set_display($course->id, $sectiondest);
@@ -2720,9 +2712,7 @@ function move_section($course, $section, $move) {
     $n = 0;
     foreach ($sections as $section) {
         if ($section->section != $n) {
-            if (!$DB->set_field('course_sections', 'section', $n, array('id'=>$section->id))) {
-                return false;
-            }
+            $DB->set_field('course_sections', 'section', $n, array('id'=>$section->id));
         }
         $n++;
     }

diff --git a/course/search.php b/course/search.php
index b7e81ac..9a12d09 100644 (file)
--- a/course/search.php
+++ b/course/search.php
@@ -68,9 +68,7 @@
                 $visible = 1;
             }
             if ($course) {
-                if (! $DB->set_field("course", "visible", $visible, array("id"=>$course->id))) {
-                    echo $OUTPUT->notification("Could not update that course!");
-                }
+                $DB->set_field("course", "visible", $visible, array("id"=>$course->id));
             }
         }
     }

diff --git a/lib/cronlib.php b/lib/cronlib.php
index c71d742..799f290 100644 (file)
--- a/lib/cronlib.php
+++ b/lib/cronlib.php
@@ -75,9 +75,7 @@ function cron_run() {
                     $pre_dbqueries = $DB->perf_get_queries();
                     $pre_time      = microtime(1);
                     if ($cron_function()) {
-                        if (!$DB->set_field("modules", "lastcron", $timenow, array("id"=>$mod->id))) {
-                            mtrace("Error: could not update timestamp for $mod->fullname");
-                        }
+                        $DB->set_field("modules", "lastcron", $timenow, array("id"=>$mod->id));
                     }
                     if (isset($pre_dbqueries)) {
                         mtrace("... used " . ($DB->perf_get_queries() - $pre_dbqueries) . " dbqueries");
@@ -106,9 +104,7 @@ function cron_run() {
                 if (method_exists($blockobj,'cron')) {
                     mtrace("Processing cron function for ".$block->name.'....','');
                     if ($blockobj->cron()) {
-                        if (!$DB->set_field('block', 'lastcron', $timenow, array('id'=>$block->id))) {
-                            mtrace('Error: could not update timestamp for '.$block->name);
-                        }
+                        $DB->set_field('block', 'lastcron', $timenow, array('id'=>$block->id));
                     }
                 /// Reset possible changes by blocks to time_limit. MDL-11597
                     @set_time_limit(0);
@@ -133,9 +129,7 @@ function cron_run() {
                     $pre_dbqueries = $DB->perf_get_queries();
                     $pre_time      = microtime(1);
                     if ($cron_function()) {
-                        if (!$DB->set_field('quiz_report', "lastcron", $timenow, array("id"=>$report->id))) {
-                            mtrace("Error: could not update timestamp for $report->name");
-                        }
+                        $DB->set_field('quiz_report', "lastcron", $timenow, array("id"=>$report->id));
                     }
                     if (isset($pre_dbqueries)) {
                         mtrace("... used " . ($DB->perf_get_queries() - $pre_dbqueries) . " dbqueries");

diff --git a/lib/grade/grade_item.php b/lib/grade/grade_item.php
index 5a03329..48f9903 100644 (file)
--- a/lib/grade/grade_item.php
+++ b/lib/grade/grade_item.php
@@ -431,10 +431,9 @@ class grade_item extends grade_object {
                 if (!empty($cm->idnumber)) {
                     return false;
                 }
-                if ($DB->set_field('course_modules', 'idnumber', $idnumber, array('id' => $cm->id))) {
-                    $this->idnumber = $idnumber;
-                    return $this->update();
-                }
+                $DB->set_field('course_modules', 'idnumber', $idnumber, array('id' => $cm->id));
+                $this->idnumber = $idnumber;
+                return $this->update();
             } else {
                 $this->idnumber = $idnumber;
                 return $this->update();

diff --git a/lib/listlib.php b/lib/listlib.php
index 4f16582..ce70893 100644 (file)
--- a/lib/listlib.php
+++ b/lib/listlib.php
@@ -340,9 +340,7 @@ class moodle_list {
     function reorder_peers($peers) {
         global $DB;
         foreach ($peers as $key => $peer) {
-            if (!$DB->set_field($this->table, "sortorder", $key, array("id"=>$peer))) {
-                print_error('listupdatefail');
-            }
+            $DB->set_field($this->table, "sortorder", $key, array("id"=>$peer));
         }
     }
 
@@ -363,13 +361,10 @@ class moodle_list {
             } else {
                 $newparent = 0; // top level item
             }
-            if (!$DB->set_field($this->table, "parent", $newparent, array("id"=>$item->id))) {
-                print_error('listupdatefail');
-            } else {
-                $oldparentkey = array_search($item->parentlist->parentitem->id, $newpeers);
-                $neworder = array_merge(array_slice($newpeers, 0, $oldparentkey+1), array($item->id), array_slice($newpeers, $oldparentkey+1));
-                $this->reorder_peers($neworder);
-            }
+            $DB->set_field($this->table, "parent", $newparent, array("id"=>$item->id));
+            $oldparentkey = array_search($item->parentlist->parentitem->id, $newpeers);
+            $neworder = array_merge(array_slice($newpeers, 0, $oldparentkey+1), array($item->id), array_slice($newpeers, $oldparentkey+1));
+            $this->reorder_peers($neworder);
         }
         return $item->parentlist->parentitem;
     }
@@ -387,17 +382,14 @@ class moodle_list {
         if (!isset($peers[$itemkey-1])) {
             print_error('listcantmoveright');
         } else {
-            if (!$DB->set_field($this->table, "parent", $peers[$itemkey-1], array("id"=>$peers[$itemkey]))) {
-                print_error('listupdatefail');
-            } else {
-                $newparent = $this->find_item($peers[$itemkey-1]);
-                if (isset($newparent->children)) {
-                    $newpeers = $newparent->children->get_child_ids();
-                }
-                if ($newpeers) {
-                    $newpeers[] = $peers[$itemkey];
-                    $this->reorder_peers($newpeers);
-                }
+            $DB->set_field($this->table, "parent", $peers[$itemkey-1], array("id"=>$peers[$itemkey]));
+            $newparent = $this->find_item($peers[$itemkey-1]);
+            if (isset($newparent->children)) {
+                $newpeers = $newparent->children->get_child_ids();
+            }
+            if ($newpeers) {
+                $newpeers[] = $peers[$itemkey];
+                $this->reorder_peers($newpeers);
             }
         }
     }

diff --git a/lib/questionlib.php b/lib/questionlib.php
index c359637..c0c3abb 100644 (file)
--- a/lib/questionlib.php
+++ b/lib/questionlib.php
@@ -748,9 +748,7 @@ function question_delete_course_category($category, $newcategory, $feedback=true
         if (!$newcontext = get_context_instance(CONTEXT_COURSECAT, $newcategory->id)) {
             return false;
         }
-        if (!$DB->set_field('question_categories', 'contextid', $newcontext->id, array('contextid'=>$context->id))) {
-            return false;
-        }
+        $DB->set_field('question_categories', 'contextid', $newcontext->id, array('contextid'=>$context->id));
         if ($feedback) {
             $a = new stdClass;
             $a->oldplace = print_context_name($context);
@@ -855,7 +853,7 @@ function question_delete_activity($cm, $feedback=true) {
  */
 function question_move_questions_to_category($questionids, $newcategoryid) {
     global $DB, $QTYPES;
-    $result = true;
+
     $ids = explode(',', $questionids);
     foreach ($ids as $questionid) {
         $questionid = (int)$questionid;
@@ -872,14 +870,14 @@ function question_move_questions_to_category($questionids, $newcategoryid) {
 
 
     // Move the questions themselves.
-    $result = $result && $DB->set_field_select('question', 'category', $newcategoryid, "id IN ($questionids)");
+    $DB->set_field_select('question', 'category', $newcategoryid, "id IN ($questionids)");
 
     // Move any subquestions belonging to them.
-    $result = $result && $DB->set_field_select('question', 'category', $newcategoryid, "parent IN ($questionids)");
+    $DB->set_field_select('question', 'category', $newcategoryid, "parent IN ($questionids)");
 
     // TODO Deal with datasets.
 
-    return $result;
+    return true;
 }
 
 /**
@@ -1957,9 +1955,7 @@ function question_process_comment($question, &$state, &$attempt, $comment, $grad
     $comment = trim($comment);
     $state->manualcomment = $comment;
     $state->newflaggedstate = $state->flagged;
-    if (!$DB->set_field('question_sessions', 'manualcomment', $comment, array('attemptid'=>$attempt->uniqueid, 'questionid'=>$question->id))) {
-        return get_string('errorsavingcomment', 'question', $question);
-    }
+    $DB->set_field('question_sessions', 'manualcomment', $comment, array('attemptid'=>$attempt->uniqueid, 'questionid'=>$question->id));
 
     // Update the attempt if the score has changed.
     if ($grade !== '' && (abs($state->last_graded->grade - $grade) > 0.002 || $state->last_graded->event != QUESTION_EVENTMANUALGRADE)) {

diff --git a/mod/assignment/lib.php b/mod/assignment/lib.php
index 99edfd2..37d5145 100644 (file)
--- a/mod/assignment/lib.php
+++ b/mod/assignment/lib.php
@@ -2511,10 +2511,7 @@ function assignment_cron () {
         $realuser = clone($USER);
 
         foreach ($submissions as $key => $submission) {
-            if (! $DB->set_field("assignment_submissions", "mailed", "1", array("id"=>$submission->id))) {
-                echo "Could not update the mailed field for id $submission->id.  Not mailed.\n";
-                unset($submissions[$key]);
-            }
+            $DB->set_field("assignment_submissions", "mailed", "1", array("id"=>$submission->id));
         }
 
         $timenow = time();

diff --git a/mod/forum/lib.php b/mod/forum/lib.php
index ab88df0..32b0249 100644 (file)
--- a/mod/forum/lib.php
+++ b/mod/forum/lib.php
@@ -2944,10 +2944,8 @@ function forum_get_course_forum($courseid, $type) {
         echo $OUTPUT->notification("Could not add the new course module to that section");
         return false;
     }
-    if (! $DB->set_field("course_modules", "section", $sectionid, array("id" => $mod->coursemodule))) {
-        echo $OUTPUT->notification("Could not update the course module with the correct section");
-        return false;
-    }
+    $DB->set_field("course_modules", "section", $sectionid, array("id" => $mod->coursemodule));
+
     include_once("$CFG->dirroot/course/lib.php");
     rebuild_course_cache($courseid);
 

diff --git a/mod/quiz/editlib.php b/mod/quiz/editlib.php
index d408fd0..b3219f9 100644 (file)
--- a/mod/quiz/editlib.php
+++ b/mod/quiz/editlib.php
@@ -139,9 +139,7 @@ function quiz_add_quiz_question($id, &$quiz, $page = 0) {
 
     // Save new questionslist in database
     $quiz->questions = implode(',', $questions);
-    if (!$DB->set_field('quiz', 'questions', $quiz->questions, array('id' => $quiz->id))) {
-        print_error('cannotsavequestion', 'quiz');
-    }
+    $DB->set_field('quiz', 'questions', $quiz->questions, array('id' => $quiz->id));
 
     // update question grades
     $quiz->grades[$id] = $DB->get_field('question', 'defaultgrade', array('id' => $id));

diff --git a/question/contextmove.php b/question/contextmove.php
index 8dbce82..0911d75 100644 (file)
--- a/question/contextmove.php
+++ b/question/contextmove.php
@@ -170,9 +170,7 @@
         }
         $sortorder = 0;
         foreach ($peers as $peer) {
-            if (! $DB->set_field('question_categories', "sortorder", $sortorder, array("id" => $peer))) {
-                print_error('listupdatefail', '', $onerrorurl);
-            }
+            $DB->set_field('question_categories', "sortorder", $sortorder, array("id" => $peer));
             $sortorder++;
         }
         //now move category

diff --git a/question/editlib.php b/question/editlib.php
index 73c6034..0df76ac 100644 (file)
--- a/question/editlib.php
+++ b/question/editlib.php
@@ -1448,10 +1448,9 @@ class question_bank_view {
                             $questionid = (int)$questionid;
                             question_require_capability_on($questionid, 'edit');
                             if ($DB->record_exists('quiz_question_instances', array('question' => $questionid))) {
-                                if (!$DB->set_field('question', 'hidden', 1, array('id' => $questionid))) {
-                                    question_require_capability_on($questionid, 'edit');
-                                    print_error('cannothidequestion', 'question');
-                                }
+                                $DB->set_field('question', 'hidden', 1, array('id' => $questionid));
+                                question_require_capability_on($questionid, 'edit');
+                                print_error('cannothidequestion', 'question');
                             } else {
                                 delete_question($questionid);
                             }
@@ -1467,9 +1466,7 @@ class question_bank_view {
         // Unhide a question
         if(($unhide = optional_param('unhide', '', PARAM_INT)) and confirm_sesskey()) {
             question_require_capability_on($unhide, 'edit');
-            if(!$DB->set_field('question', 'hidden', 0, array('id' => $unhide))) {
-                print_error('cannotunhidequestion', 'question');
-            }
+            $DB->set_field('question', 'hidden', 0, array('id' => $unhide));
             redirect($this->baseurl);
         }
     }

diff --git a/question/restorelib.php b/question/restorelib.php
index 79fb182..4c7cb41 100644 (file)
--- a/question/restorelib.php
+++ b/question/restorelib.php
@@ -417,7 +417,7 @@
             //                   see: $QTYPES['random']->get_question_options()
             if ($question->qtype == 'random' && $creatingnewquestion) {
                 $question->parent = $question->id;
-                $status = $DB->set_field('question', 'parent', $question->parent, array('id'=>$question->id));
+                $DB->set_field('question', 'parent', $question->parent, array('id'=>$question->id));
             }
 
             //Save newid to backup tables
@@ -457,10 +457,7 @@
                     if ($parent = backup_getid($restore->backup_unique_code,"question",$question->parent)) {
                         $question->parent = $parent->new_id;
                         if ($question->parent != $restored_questions[$i]->parent) {
-                            if (!$DB->set_field('question', 'parent', $question->parent, array('id'=>$newid))) {
-                                echo 'Could not update parent '.$question->parent.' for question '.$oldid.'<br />';
-                                $status = false;
-                            }
+                            $DB->set_field('question', 'parent', $question->parent, array('id'=>$newid));
                         }
                     } else {
                         echo 'Could not recode parent '.$question->parent.' for question '.$oldid.'<br />';

diff --git a/question/type/calculated/questiontype.php b/question/type/calculated/questiontype.php
index eda2ce0..fc8db07 100644 (file)
--- a/question/type/calculated/questiontype.php
+++ b/question/type/calculated/questiontype.php
@@ -394,9 +394,8 @@ class question_calculated_qtype extends default_questiontype {
         }
 
         // Set the legacy answer field
-        if (!$DB->set_field('question_states', 'answer', $responses, array('id'=> $state->id))) {
-            return false;
-        }
+        $DB->set_field('question_states', 'answer', $responses, array('id'=> $state->id));
+
         return true;
     }
 
@@ -623,9 +622,7 @@ class question_calculated_qtype extends default_questiontype {
             }
             $questionname ="#".$questionname;
         }
-        if (!$DB->set_field('question', 'name', $questionname, array("id" => $question->id))) {
-            return false ;
-        }
+        $DB->set_field('question', 'name', $questionname, array("id" => $question->id));
     }
 
     /**
@@ -703,9 +700,7 @@ class question_calculated_qtype extends default_questiontype {
             }else {
                 $options_synchronize = 0 ;
             }
-            if (!$DB->set_field('question_calculated_options', 'synchronize', $options_synchronize, array("question" => $question->id))) {
-                return false;
-            }
+            $DB->set_field('question_calculated_options', 'synchronize', $options_synchronize, array("question" => $question->id));
             if(isset($form->synchronize) && $form->synchronize == 2 ){
                 $this->addnamecategory($question);
             }

diff --git a/question/type/calculatedmulti/questiontype.php b/question/type/calculatedmulti/questiontype.php
index c648899..ffa30ee 100644 (file)
--- a/question/type/calculatedmulti/questiontype.php
+++ b/question/type/calculatedmulti/questiontype.php
@@ -241,9 +241,7 @@ class question_calculatedmulti_qtype extends question_calculated_qtype {
         $responses .= implode(',', $state->responses);
 
         // Set the legacy answer field
-        if (!$DB->set_field('question_states', 'answer', $responses, array('id'=> $state->id))) {
-            return false;
-        }
+        $DB->set_field('question_states', 'answer', $responses, array('id'=> $state->id));
         return true;
     }
 

diff --git a/question/type/match/questiontype.php b/question/type/match/questiontype.php
index be01f1c..469e11a 100644 (file)
--- a/question/type/match/questiontype.php
+++ b/question/type/match/questiontype.php
@@ -230,9 +230,7 @@ class question_match_qtype extends default_questiontype {
         $responses = implode(',', $responses);
 
         // Set the legacy answer field
-        if (!$DB->set_field('question_states', 'answer', $responses, array('id' => $state->id))) {
-            return false;
-        }
+        $DB->set_field('question_states', 'answer', $responses, array('id' => $state->id));
         return true;
     }
 

diff --git a/question/type/multianswer/questiontype.php b/question/type/multianswer/questiontype.php
index 9f1b47e..2288fec 100644 (file)
--- a/question/type/multianswer/questiontype.php
+++ b/question/type/multianswer/questiontype.php
@@ -210,9 +210,7 @@ class embedded_cloze_qtype extends default_questiontype {
         $responses = implode(',', $responses);
 
         // Set the legacy answer field
-        if (!$DB->set_field('question_states', 'answer', $responses, array('id' => $state->id))) {
-            return false;
-        }
+        $DB->set_field('question_states', 'answer', $responses, array('id' => $state->id));
         return true;
     }
 

diff --git a/question/type/multichoice/questiontype.php b/question/type/multichoice/questiontype.php
index c384377..ec60557 100644 (file)
--- a/question/type/multichoice/questiontype.php
+++ b/question/type/multichoice/questiontype.php
@@ -236,9 +236,7 @@ class question_multichoice_qtype extends default_questiontype {
         $responses .= implode(',', $state->responses);
 
         // Set the legacy answer field
-        if (!$DB->set_field('question_states', 'answer', $responses, array('id' => $state->id))) {
-            return false;
-        }
+        $DB->set_field('question_states', 'answer', $responses, array('id' => $state->id));
         return true;
     }
 

diff --git a/question/type/numerical/questiontype.php b/question/type/numerical/questiontype.php
index d3cba20..fe9ef21 100644 (file)
--- a/question/type/numerical/questiontype.php
+++ b/question/type/numerical/questiontype.php
@@ -416,9 +416,7 @@ class question_numerical_qtype extends question_shortanswer_qtype {
             $responses = $state->responses['answer'].'|||||';
         }
         // Set the legacy answer field
-        if (!$DB->set_field('question_states', 'answer', $responses, array('id' => $state->id))) {
-            return false;
-        }
+        $DB->set_field('question_states', 'answer', $responses, array('id' => $state->id));
         return true;
     }
 

diff --git a/question/type/random/questiontype.php b/question/type/random/questiontype.php
index f1acc34..fd85751 100644 (file)
--- a/question/type/random/questiontype.php
+++ b/question/type/random/questiontype.php
@@ -297,8 +297,7 @@ class random_qtype extends default_questiontype {
         $response = "random$realqid-$response";
 
         // ... and save it again.
-        if (!$DB->set_field('question_states', 'answer', $response, array('id' => $state->id))) {
-        }
+        $DB->set_field('question_states', 'answer', $response, array('id' => $state->id));
 
         // Restore the real id
         $wrappedquestion->id = $realqid;

diff --git a/repository/lib.php b/repository/lib.php
index ed5e3dc..691bc3c 100644 (file)
--- a/repository/lib.php
+++ b/repository/lib.php
@@ -1317,19 +1317,18 @@ abstract class repository {
             $DB->update_record('repository_instances', $r);
             unset($options['name']);
         }
-        $result = true;
         foreach ($options as $name=>$value) {
             if ($id = $DB->get_field('repository_instance_config', 'id', array('name'=>$name, 'instanceid'=>$this->id))) {
-                $result = $result && $DB->set_field('repository_instance_config', 'value', $value, array('id'=>$id));
+                $DB->set_field('repository_instance_config', 'value', $value, array('id'=>$id));
             } else {
                 $config = new object();
                 $config->instanceid = $this->id;
                 $config->name   = $name;
                 $config->value  = $value;
-                $result = $result && $DB->insert_record('repository_instance_config', $config);
+                $DB->insert_record('repository_instance_config', $config);
             }
         }
-        return $result;
+        return true;
     }
 
     /**

diff --git a/user/policy.php b/user/policy.php
index 6d8eb78..17dedc7 100644 (file)
--- a/user/policy.php
+++ b/user/policy.php
@@ -42,9 +42,7 @@ if (!isloggedin()) {
 
 if ($agree and confirm_sesskey()) {    // User has agreed
     if (!isguestuser()) {              // Don't remember guests
-        if (!$DB->set_field('user', 'policyagreed', 1, array('id'=>$USER->id))) {
-            print_error('cannotsaveagreement');
-        }
+        $DB->set_field('user', 'policyagreed', 1, array('id'=>$USER->id));
     }
     $USER->policyagreed = 1;