From eb64eefaab7a13a3d8a1898d80dae776b9150de0 Mon Sep 17 00:00:00 2001 From: pichetp Date: Mon, 26 Jan 2009 01:26:03 +0000 Subject: [PATCH] MDL-17995 replacing != '' by !empty(). Merged from HEAD --- question/type/multianswer/edit_multianswer_form.php | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/question/type/multianswer/edit_multianswer_form.php b/question/type/multianswer/edit_multianswer_form.php index 8d0fed9d2df..77262d444da 100644 --- a/question/type/multianswer/edit_multianswer_form.php +++ b/question/type/multianswer/edit_multianswer_form.php @@ -36,7 +36,7 @@ class question_edit_multianswer_form extends question_edit_form { if ( isset($this->questiondisplay->options->questions) && is_array($this->questiondisplay->options->questions) ) { $countsubquestions =0; foreach($this->questiondisplay->options->questions as $subquestion){ - if ($subquestion != ''){ + if (!empty($subquestion)){ $countsubquestions++; } } @@ -95,7 +95,7 @@ class question_edit_multianswer_form extends question_edit_form { if (isset($question->id) and $question->id and $question->qtype and $question->questiontext) { foreach ($question->options->questions as $key => $wrapped) { - if($wrapped != ''){ + if(!empty($wrapped)){ // The old way of restoring the definitions is kept to gradually // update all multianswer questions if (empty($wrapped->questiontext)) {