From 90a92bd0cdbdca285b6df8fda05b7679ef834041 Mon Sep 17 00:00:00 2001
From: Pierre Pichet <pichet.pierre@uqam.ca>
Date: Sat, 24 Oct 2009 03:03:34 +0000
Subject: [PATCH] MDL-20585 correcting so that the confirm check button appears
 only on reload if errors are detected

---
 question/type/multianswer/edit_multianswer_form.php | 11 ++++++-----
 1 file changed, 6 insertions(+), 5 deletions(-)

diff --git a/question/type/multianswer/edit_multianswer_form.php b/question/type/multianswer/edit_multianswer_form.php
index ea93ecd91d..c49359b3ed 100644
--- a/question/type/multianswer/edit_multianswer_form.php
+++ b/question/type/multianswer/edit_multianswer_form.php
@@ -175,12 +175,13 @@ class question_edit_multianswer_form extends question_edit_form {
     
             }
             echo '</div>';
-            if ( ($countsavedsubquestions > $countsubquestions) ||$this->type_change || ($this->used_in_quiz && ($countsavedsubquestions < $countsubquestions))){
+            $this->negative_diff =$countsavedsubquestions - $countsubquestions ;
+            if ( ($this->negative_diff > 0 ) ||$this->type_change || ($this->used_in_quiz && $this->negative_diff != 0)){
                     $mform->addElement('header', 'additemhdr', "WARNING");
                 }
-            if($countsavedsubquestions > $countsubquestions) {
+            if($this->negative_diff > 0) {
                 //$this->used_in_quiz
-                $this->negative_diff =$countsavedsubquestions - $countsubquestions ;
+                
                             $mform->addElement('static', 'alert1', "<strong>"."Question deleted"."</strong>","<strong>".$this->negative_diff.get_string(' questions less than in the multtianswer question stored in the database','qtype_multianswer')."</strong>");//$countsubquestions."-".$countsavedsubquestions
             }
             if($this->type_change )
@@ -193,14 +194,14 @@ class question_edit_multianswer_form extends question_edit_form {
             echo '</div>';
         }
         if( $this->used_in_quiz){
-        if($countsavedsubquestions < $countsubquestions) {
+        if($this->negative_diff < 0) {
             $diff = $countsubquestions - $countsavedsubquestions;
                         $mform->addElement('static', 'alert1', "<strong>"."Question added "."</strong>","<strong>".$diff.get_string(' questions more than in the multtianswer question stored in the database','qtype_multianswer')."</strong>");//$countsubquestions."-".$countsavedsubquestions
         }
                 $mform->addElement('header', 'additemhdr2', "This question is used in $this->nb_of_quiz  quiz(s), total attempt(s) : $this->nb_of_attempts ");
                              $mform->addElement('static', 'alertas', "<strong>"."YOU SHOULD NOT "."</strong>");//$countsubquestions."-".$countsavedsubquestions
          }
-        if ( $this->negative_diff || $this->used_in_quiz ){
+        if ( ($this->negative_diff > 0 || $this->used_in_quiz && ($this->negative_diff > 0 ||$this->negative_diff < 0 || $this->type_change ) ) &&  $this->reload ){
             $mform->addElement('header', 'additemhdr', get_string('The question will be saved as edited', 'qtype_calculatedsimple'));            
             $mform->addElement('checkbox', 'confirm','' ,get_string('I confirm that I want the question be saved as edited', 'qtype_calculatedsimple'));
             $mform->setDefault('confirm', 0);
-- 
2.39.5