diff --git a/mod/lesson/action/insertpage.php b/mod/lesson/action/insertpage.php index d6533a63828..9643b80b696 100644 --- a/mod/lesson/action/insertpage.php +++ b/mod/lesson/action/insertpage.php @@ -148,7 +148,7 @@ $lesson->maxanswers = $lesson->maxanswers + 2; } for ($i = 0; $i < $lesson->maxanswers; $i++) { - if (trim(strip_tags($form->answer[$i]))) { // strip_tags because the HTML editor adds
...
+ if (!empty($form->answer[$i]) and trim(strip_tags($form->answer[$i]))) { // strip_tags because the HTML editor adds
...
$newanswer->lessonid = $lesson->id;
$newanswer->pageid = $newpageid;
$newanswer->timecreated = $timenow;
diff --git a/mod/lesson/action/updatepage.php b/mod/lesson/action/updatepage.php
index c4f053ad241..e0376bb9e82 100644
--- a/mod/lesson/action/updatepage.php
+++ b/mod/lesson/action/updatepage.php
@@ -83,7 +83,7 @@
for ($i = 0; $i < $lesson->maxanswers; $i++) {
// strip tags because the editor gives
...
// also save any answers where the editor is (going to be) used
- if (trim(strip_tags($form->answer[$i])) or isset($form->answereditor[$i]) or isset($form->responseeditor[$i])) {
+ if ((!empty($form->answer[$i]) and trim(strip_tags($form->answer[$i]))) or isset($form->answereditor[$i]) or isset($form->responseeditor[$i])) {
if ($form->answerid[$i]) {
$oldanswer = new stdClass;
$oldanswer->id = clean_param($form->answerid[$i], PARAM_INT);
@@ -162,7 +162,7 @@
error("Update page: answer $i not updated");
}
}
- } elseif ($form->answerid[$i]) {
+ } elseif (!empty($form->answerid[$i])) {
// need to delete blanked out answer
if (!delete_records("lesson_answers", "id", clean_param($form->answerid[$i], PARAM_INT))) {
error("Update page: unable to delete answer record");