From a54bf344c87aa3c86f551c55a9f27d0aa31360d5 Mon Sep 17 00:00:00 2001 From: Moodle HQ git importer Date: Thu, 7 May 2009 00:35:40 +0000 Subject: [PATCH] Fixing drift between CVS and git --- mod/lesson/action/addcluster.php | 8 ++------ mod/lesson/action/addendofbranch.php | 8 ++------ mod/lesson/action/addendofcluster.php | 8 ++------ mod/lesson/action/insertpage.php | 7 +------ mod/lesson/action/updatepage.php | 4 +--- mod/lesson/restorelib.php | 8 ++------ 6 files changed, 10 insertions(+), 33 deletions(-) diff --git a/mod/lesson/action/addcluster.php b/mod/lesson/action/addcluster.php index 1735e11d42..224acd3649 100644 --- a/mod/lesson/action/addcluster.php +++ b/mod/lesson/action/addcluster.php @@ -38,9 +38,7 @@ $newpageid = $DB->insert_record("lesson_pages", $newpage); // update the linked list... if ($pageid != 0) { - if (!$DB->set_field("lesson_pages", "nextpageid", $newpageid, array("id" => $pageid))) { - print_error('cannotupdatelink', 'lesson'); - } + $DB->set_field("lesson_pages", "nextpageid", $newpageid, array("id" => $pageid)); } if ($pageid == 0) { @@ -48,9 +46,7 @@ } if ($page->nextpageid) { // the new page is not the last page - if (!$DB->set_field("lesson_pages", "prevpageid", $newpageid, array("id" => $page->nextpageid))) { - print_error('cannotupdatelink', 'lesson'); - } + $DB->set_field("lesson_pages", "prevpageid", $newpageid, array("id" => $page->nextpageid)); } // ..and the single "answer" $newanswer = new stdClass; diff --git a/mod/lesson/action/addendofbranch.php b/mod/lesson/action/addendofbranch.php index a052aa7fca..96799673c0 100644 --- a/mod/lesson/action/addendofbranch.php +++ b/mod/lesson/action/addendofbranch.php @@ -39,14 +39,10 @@ $newpage->contents = get_string("endofbranch", "lesson"); $newpageid = $DB->insert_record("lesson_pages", $newpage); // update the linked list... - if (!$DB->set_field("lesson_pages", "nextpageid", $newpageid, array("id" => $pageid))) { - print_error('cannotupdatelink', 'lesson'); - } + $DB->set_field("lesson_pages", "nextpageid", $newpageid, array("id" => $pageid)); if ($page->nextpageid) { // the new page is not the last page - if (!$DB->set_field("lesson_pages", "prevpageid", $newpageid, array("id" => $page->nextpageid))) { - print_error('cannotupdatelink', 'lesson'); - } + $DB->set_field("lesson_pages", "prevpageid", $newpageid, array("id" => $page->nextpageid)); } // ..and the single "answer" $newanswer = new stdClass; diff --git a/mod/lesson/action/addendofcluster.php b/mod/lesson/action/addendofcluster.php index 15fff40ad6..cbbc7a365e 100644 --- a/mod/lesson/action/addendofcluster.php +++ b/mod/lesson/action/addendofcluster.php @@ -30,14 +30,10 @@ $newpage->contents = get_string("endofclustertitle", "lesson"); $newpageid = $DB->insert_record("lesson_pages", $newpage); // update the linked list... - if (!$DB->set_field("lesson_pages", "nextpageid", $newpageid, array("id" => $pageid))) { - print_error('cannotupdatelink', 'lesson'); - } + $DB->set_field("lesson_pages", "nextpageid", $newpageid, array("id" => $pageid)); if ($page->nextpageid) { // the new page is not the last page - if (!$DB->set_field("lesson_pages", "prevpageid", $newpageid, array("id" => $page->nextpageid))) { - print_error('cannotupdatelink', 'lesson'); - } + $DB->set_field("lesson_pages", "prevpageid", $newpageid, array("id" => $page->nextpageid)); } // ..and the single "answer" $newanswer = new stdClass; diff --git a/mod/lesson/action/insertpage.php b/mod/lesson/action/insertpage.php index 4f9d37fbcb..231ee07b02 100644 --- a/mod/lesson/action/insertpage.php +++ b/mod/lesson/action/insertpage.php @@ -109,13 +109,8 @@ $newpage->contents = trim($form->contents); $newpage->title = $newpage->title; $newpageid = $DB->insert_record("lesson_pages", $newpage); - if (!$newpageid) { - print_error('cannotfindfirstpage', 'lesson'); - } // update the linked list - if (!$DB->set_field("lesson_pages", "prevpageid", $newpageid, array("id" => $newpage->nextpageid))) { - print_error('cannotupdatelink', 'lesson'); - } + $DB->set_field("lesson_pages", "prevpageid", $newpageid, array("id" => $newpage->nextpageid)); } } // now add the answers diff --git a/mod/lesson/action/updatepage.php b/mod/lesson/action/updatepage.php index 85eab4e7f6..2f268d3884 100644 --- a/mod/lesson/action/updatepage.php +++ b/mod/lesson/action/updatepage.php @@ -47,9 +47,7 @@ $page->contents = trim($form->contents); $page->title = $page->title; - if (!$DB->update_record("lesson_pages", $page)) { - print_error('cannotupdatepage', 'lesson'); - } + $DB->update_record("lesson_pages", $page); if ($page->qtype == LESSON_ENDOFBRANCH || $page->qtype == LESSON_ESSAY || $page->qtype == LESSON_CLUSTER || $page->qtype == LESSON_ENDOFCLUSTER) { // there's just a single answer with a jump $oldanswer = new stdClass; diff --git a/mod/lesson/restorelib.php b/mod/lesson/restorelib.php index c71e3902e3..5e6a904b43 100644 --- a/mod/lesson/restorelib.php +++ b/mod/lesson/restorelib.php @@ -181,9 +181,7 @@ //Fix the forwards link of the previous page if ($prevpageid) { - if (!$DB->set_field("lesson_pages", "nextpageid", $newid, array("id"=>$prevpageid))) { - print_error('cannotupdatepage', 'lesson'); - } + $DB->set_field("lesson_pages", "nextpageid", $newid, array("id"=>$prevpageid)); } $prevpageid = $newid; @@ -251,9 +249,7 @@ // change the absolute page id $page = backup_getid($restore->backup_unique_code,"lesson_pages",$answer->jumpto); if ($page) { - if (!$DB->set_field("lesson_answers", "jumpto", $page->new_id, array("id"=>$answer->id))) { - print_error('cannotresetjump', 'lesson'); - } + $DB->set_field("lesson_answers", "jumpto", $page->new_id, array("id"=>$answer->id)); } } } -- 2.39.5