Minor - format code.

1.9.x
Julio Montoya 11 years ago
parent da239fdb30
commit 074c81b67b
  1. 1
      main/exercice/exercise_result.php
  2. 24
      main/exercice/exercise_submit.php
  3. 370
      main/exercice/matching.class.php
  4. 1
      main/exercice/overview.php

@ -49,7 +49,6 @@ if ($debug) {
error_log('Entering exercise_result.php: '.print_r($_POST, 1));
}
// general parameters passed via POST/GET
// general parameters passed via POST/GET
if (empty($origin)) {
$origin = Security::remove_XSS($_REQUEST['origin']);

@ -794,8 +794,8 @@ if (!empty($error)) {
echo '<script>
$(function() {
//$(".exercise_save_now_button").hide();
$(".main_question").mouseover(function() {
//$(".exercise_save_now_button").hide();
$(".main_question").mouseover(function() {
//$(this).find(".exercise_save_now_button").show();
//$(this).addClass("question_highlight");
});
@ -807,17 +807,17 @@ if (!empty($error)) {
$(".no_remind_highlight").hide();
// if the users validates the form using return key, prevent form action and simulates click on validation button
$("#exercise_form").submit(function(){
$(".question-validate-btn").first().trigger("click");
return false;
});
// if the users validates the form using return key, prevent form action and simulates click on validation button
$("#exercise_form").submit(function(){
$(".question-validate-btn").first().trigger("click");
return false;
});
});
function previous_question(question_num) {
url = "exercise_submit.php?'.$params.'&num="+question_num;
window.location = url;
}
function previous_question(question_num) {
url = "exercise_submit.php?'.$params.'&num="+question_num;
window.location = url;
}
function previous_question_and_save(previous_question_id, question_id_to_save) {
url = "exercise_submit.php?'.$params.'&num="+previous_question_id;
@ -1048,7 +1048,7 @@ if (!empty($error)) {
//BUtton save and continue
switch ($objExercise->type) {
case ONE_PER_PAGE:
case ONE_PER_PAGE:
$exercise_actions .= $objExercise->show_button($questionId, $current_question);
break;
case ALL_ON_ONE_PAGE :

@ -1,8 +1,6 @@
<?php
/* For licensing terms, see /license.txt */
/**
* Code
*/
/**
*
* Class Matching
@ -14,76 +12,79 @@
* @author Eric Marguin
* @package chamilo.exercise
*/
class Matching extends Question {
static $typePicture = 'matching.gif';
static $explanationLangVar = 'Matching';
/**
* Constructor
*/
function Matching(){
parent::question();
$this -> type = MATCHING;
$this -> isContent = $this-> getIsContent();
}
/**
* function which redifines Question::createAnswersForm
* @param the formvalidator instance
*/
function createAnswersForm ($form) {
$defaults = array();
$navigator_info = api_get_navigator();
$nb_matches = $nb_options = 2;
if($form -> isSubmitted()) {
$nb_matches = $form -> getSubmitValue('nb_matches');
$nb_options = $form -> getSubmitValue('nb_options');
if(isset($_POST['lessMatches']))
$nb_matches--;
if(isset($_POST['moreMatches']))
$nb_matches++;
if(isset($_POST['lessOptions']))
$nb_options--;
if(isset($_POST['moreOptions']))
$nb_options++;
} else if(!empty($this -> id)) {
$answer = new Answer($this -> id);
$answer -> read();
if(count($answer->nbrAnswers)>0) {
$a_matches = $a_options = array();
$nb_matches = $nb_options = 0;
for($i=1 ; $i<=$answer->nbrAnswers ; $i++){
if ($answer -> isCorrect($i)) {
$nb_matches++;
$defaults['answer['.$nb_matches.']'] = $answer -> selectAnswer($i);
$defaults['weighting['.$nb_matches.']'] = float_format($answer -> selectWeighting($i),1);
$defaults['matches['.$nb_matches.']'] = $answer -> correct[$i];
} else {
$nb_options++;
$defaults['option['.$nb_options.']'] = $answer -> selectAnswer($i);
}
}
}
} else {
$defaults['answer[1]'] = get_lang('DefaultMakeCorrespond1');
$defaults['answer[2]'] = get_lang('DefaultMakeCorrespond2');
$defaults['matches[2]'] = '2';
$defaults['option[1]'] = get_lang('DefaultMatchingOptA');
$defaults['option[2]'] = get_lang('DefaultMatchingOptB');
}
$a_matches = array();
for($i=1 ; $i<=$nb_options ; ++$i) {
$a_matches[$i] = chr(64+$i); // fill the array with A, B, C.....
}
$form -> addElement('hidden', 'nb_matches', $nb_matches);
$form -> addElement('hidden', 'nb_options', $nb_options);
// DISPLAY MATCHES
$html='<table class="data_table">
class Matching extends Question
{
static $typePicture = 'matching.gif';
static $explanationLangVar = 'Matching';
/**
* Constructor
*/
public function Matching()
{
parent::question();
$this->type = MATCHING;
$this->isContent = $this-> getIsContent();
}
/**
* function which redefines Question::createAnswersForm
* @param FormValidator $form
*/
public function createAnswersForm ($form)
{
$defaults = array();
$navigator_info = api_get_navigator();
$nb_matches = $nb_options = 2;
if ($form -> isSubmitted()) {
$nb_matches = $form -> getSubmitValue('nb_matches');
$nb_options = $form -> getSubmitValue('nb_options');
if(isset($_POST['lessMatches']))
$nb_matches--;
if(isset($_POST['moreMatches']))
$nb_matches++;
if(isset($_POST['lessOptions']))
$nb_options--;
if(isset($_POST['moreOptions']))
$nb_options++;
} else if(!empty($this -> id)) {
$answer = new Answer($this -> id);
$answer -> read();
if(count($answer->nbrAnswers)>0) {
$a_matches = $a_options = array();
$nb_matches = $nb_options = 0;
for($i=1 ; $i<=$answer->nbrAnswers ; $i++){
if ($answer -> isCorrect($i)) {
$nb_matches++;
$defaults['answer['.$nb_matches.']'] = $answer -> selectAnswer($i);
$defaults['weighting['.$nb_matches.']'] = float_format($answer -> selectWeighting($i),1);
$defaults['matches['.$nb_matches.']'] = $answer -> correct[$i];
} else {
$nb_options++;
$defaults['option['.$nb_options.']'] = $answer -> selectAnswer($i);
}
}
}
} else {
$defaults['answer[1]'] = get_lang('DefaultMakeCorrespond1');
$defaults['answer[2]'] = get_lang('DefaultMakeCorrespond2');
$defaults['matches[2]'] = '2';
$defaults['option[1]'] = get_lang('DefaultMatchingOptA');
$defaults['option[2]'] = get_lang('DefaultMatchingOptB');
}
$a_matches = array();
for($i=1 ; $i<=$nb_options ; ++$i) {
$a_matches[$i] = chr(64+$i); // fill the array with A, B, C.....
}
$form -> addElement('hidden', 'nb_matches', $nb_matches);
$form -> addElement('hidden', 'nb_options', $nb_options);
// DISPLAY MATCHES
$html='<table class="data_table">
<tr>
<th width="10px">
'.get_lang('Number').'
@ -99,42 +100,42 @@ class Matching extends Question {
</th>
</tr>';
$form -> addElement ('label', get_lang('MakeCorrespond').'<br /> <img src="../img/fill_field.png">', $html);
if ($nb_matches < 1) {
$nb_matches = 1;
Display::display_normal_message(get_lang('YouHaveToCreateAtLeastOneAnswer'));
}
for($i = 1 ; $i <= $nb_matches ; ++$i) {
$form -> addElement ('html', '<tr><td>');
$group = array();
$puce = $form->createElement('text', null,null,'value="'.$i.'"');
$puce->freeze();
$group[] = $puce;
$group[] = $form->createElement('text', 'answer['.$i.']',null, 'size="60" style="margin-left: 0em;"');
$group[] = $form->createElement('select', 'matches['.$i.']',null,$a_matches);
$group[] = $form->createElement('text', 'weighting['.$i.']',null, array('class' => 'span1', 'value' => 10));
$form -> addGroup($group, null, null, '</td><td>');
$form -> addElement ('html', '</td></tr>');
}
$form -> addElement ('html', '</table></div></div>');
$group = array();
if ($navigator_info['name']=='Internet Explorer' && $navigator_info['version']=='6') {
$group[] = $form->createElement('submit', 'lessMatches', get_lang('DelElem'),'class="btn minus"');
$group[] = $form->createElement('submit', 'moreMatches', get_lang('AddElem'),'class="btn plus"');
} else {
$form -> addElement ('label', get_lang('MakeCorrespond').'<br /> <img src="../img/fill_field.png">', $html);
if ($nb_matches < 1) {
$nb_matches = 1;
Display::display_normal_message(get_lang('YouHaveToCreateAtLeastOneAnswer'));
}
for($i = 1 ; $i <= $nb_matches ; ++$i) {
$form -> addElement ('html', '<tr><td>');
$group = array();
$puce = $form->createElement('text', null,null,'value="'.$i.'"');
$puce->freeze();
$group[] = $puce;
$group[] = $form->createElement('text', 'answer['.$i.']',null, 'size="60" style="margin-left: 0em;"');
$group[] = $form->createElement('select', 'matches['.$i.']',null,$a_matches);
$group[] = $form->createElement('text', 'weighting['.$i.']',null, array('class' => 'span1', 'value' => 10));
$form -> addGroup($group, null, null, '</td><td>');
$form -> addElement ('html', '</td></tr>');
}
$form -> addElement ('html', '</table></div></div>');
$group = array();
if ($navigator_info['name']=='Internet Explorer' && $navigator_info['version']=='6') {
$group[] = $form->createElement('submit', 'lessMatches', get_lang('DelElem'),'class="btn minus"');
$group[] = $form->createElement('submit', 'moreMatches', get_lang('AddElem'),'class="btn plus"');
} else {
$group[] = $form->createElement('style_submit_button', 'moreMatches', get_lang('AddElem'),'class="btn plus"');
$group[] = $form->createElement('style_submit_button', 'lessMatches', get_lang('DelElem'),'class="btn minus"');
}
$group[] = $form->createElement('style_submit_button', 'lessMatches', get_lang('DelElem'),'class="btn minus"');
}
$form -> addGroup($group);
$form -> addGroup($group);
// DISPLAY OPTIONS
$html='<table class="data_table">
// DISPLAY OPTIONS
$html='<table class="data_table">
<tr style="text-align: center;">
<th width="10px">
'.get_lang('Number').'
@ -143,95 +144,102 @@ class Matching extends Question {
'.get_lang('Answer').'
</th>
</tr>';
//$form -> addElement ('html', $html);
//$form -> addElement ('html', $html);
$form -> addElement ('label', null, $html);
if ($nb_options < 1) {
$nb_options = 1;
Display::display_normal_message(get_lang('YouHaveToCreateAtLeastOneAnswer'));
}
for($i = 1 ; $i <= $nb_options ; ++$i) {
$form -> addElement ('html', '<tr><td>');
$group = array();
$puce = $form->createElement('text', null,null,'value="'.chr(64+$i).'"');
$puce->freeze();
$group[] = $puce;
$group[] = $form->createElement('text', 'option['.$i.']',null, array('class' =>'span6'));
$form -> addGroup($group, null, null, '</td><td>');
$form -> addElement ('html', '</td></tr>');
}
$form -> addElement ('html', '</table></div></div>');
$group = array();
global $text, $class;
if ($navigator_info['name']=='Internet Explorer' && $navigator_info['version']=='6') {
if ($nb_options < 1) {
$nb_options = 1;
Display::display_normal_message(get_lang('YouHaveToCreateAtLeastOneAnswer'));
}
for($i = 1 ; $i <= $nb_options ; ++$i) {
$form -> addElement ('html', '<tr><td>');
$group = array();
$puce = $form->createElement('text', null,null,'value="'.chr(64+$i).'"');
$puce->freeze();
$group[] = $puce;
$group[] = $form->createElement('text', 'option['.$i.']',null, array('class' =>'span6'));
$form -> addGroup($group, null, null, '</td><td>');
$form -> addElement ('html', '</td></tr>');
}
$form -> addElement ('html', '</table></div></div>');
$group = array();
global $text, $class;
if ($navigator_info['name']=='Internet Explorer' && $navigator_info['version']=='6') {
// setting the save button here and not in the question class.php
$group[] = $form->createElement('submit','submitQuestion',$text, 'class="'.$class.'"');
$group[] = $form->createElement('submit', 'lessOptions', get_lang('DelElem'),'class="minus"');
$group[] = $form->createElement('submit', 'moreOptions',get_lang('AddElem'),'class="plus"');
} else {
} else {
// setting the save button here and not in the question class.php
$group[] = $form->createElement('style_submit_button', 'lessOptions', get_lang('DelElem'),'class="minus"');
$group[] = $form->createElement('style_submit_button', 'moreOptions',get_lang('AddElem'),' class="plus"');
$group[] = $form->createElement('style_submit_button','submitQuestion',$text, 'class="'.$class.'"');
}
$form -> addGroup($group);
if (!empty($this -> id)) {
$form -> setDefaults($defaults);
} else {
if ($this -> isContent == 1) {
$form -> setDefaults($defaults);
}
}
$form->setConstants(array('nb_matches' => $nb_matches,'nb_options' => $nb_options));
}
/**
* abstract function which creates the form to create / edit the answers of the question
* @param the formvalidator instance
*/
function processAnswersCreation($form) {
$nb_matches = $form -> getSubmitValue('nb_matches');
$nb_options = $form -> getSubmitValue('nb_options');
$this -> weighting = 0;
$objAnswer = new Answer($this->id);
$position = 0;
// insert the options
for($i=1 ; $i<=$nb_options; ++$i) {
$position++;
$option = $form -> getSubmitValue('option['.$i.']');
$objAnswer->createAnswer($option, 0, '', 0, $position);
}
// insert the answers
for($i=1 ; $i<=$nb_matches ; ++$i) {
$position++;
$answer = $form -> getSubmitValue('answer['.$i.']');
$matches = $form -> getSubmitValue('matches['.$i.']');
$weighting = $form -> getSubmitValue('weighting['.$i.']');
$this -> weighting += $weighting;
$objAnswer->createAnswer($answer,$matches,'',$weighting,$position);
}
$objAnswer->save();
$this->save();
}
function return_header($feedback_type = null, $counter = null, $score = null) {
$header = parent::return_header($feedback_type, $counter, $score);
}
$form -> addGroup($group);
if (!empty($this -> id)) {
$form -> setDefaults($defaults);
} else {
if ($this -> isContent == 1) {
$form -> setDefaults($defaults);
}
}
$form->setConstants(array('nb_matches' => $nb_matches,'nb_options' => $nb_options));
}
/**
* abstract function which creates the form to create / edit the answers of the question
* @param FormValidator $form
*/
public function processAnswersCreation($form)
{
$nb_matches = $form -> getSubmitValue('nb_matches');
$nb_options = $form -> getSubmitValue('nb_options');
$this -> weighting = 0;
$objAnswer = new Answer($this->id);
$position = 0;
// insert the options
for($i=1 ; $i<=$nb_options; ++$i) {
$position++;
$option = $form -> getSubmitValue('option['.$i.']');
$objAnswer->createAnswer($option, 0, '', 0, $position);
}
// insert the answers
for($i=1 ; $i<=$nb_matches ; ++$i) {
$position++;
$answer = $form -> getSubmitValue('answer['.$i.']');
$matches = $form -> getSubmitValue('matches['.$i.']');
$weighting = $form -> getSubmitValue('weighting['.$i.']');
$this -> weighting += $weighting;
$objAnswer->createAnswer($answer,$matches,'',$weighting,$position);
}
$objAnswer->save();
$this->save();
}
/**
* @param null $feedback_type
* @param null $counter
* @param null $score
* @return string
*/
public function return_header($feedback_type = null, $counter = null, $score = null)
{
$header = parent::return_header($feedback_type, $counter, $score);
$header .= '<table class="'.$this->question_table_class .'">';
$header .= '<tr>
<th>'.get_lang('ElementList').'</th>
<th>'.get_lang('CorrespondsTo').'</th>
</tr>';
return $header;
}
}
}

@ -98,7 +98,6 @@ if (isset($exercise_stat_info['exe_id'])) {
}
//1. Check if this is a new attempt or a previous
//$countNotFinished = isset($exercise_stat_info['num_exe']) ? $exercise_stat_info['num_exe'] : null;
$label = get_lang('StartTest');
if ($time_control && !empty($clock_expired_time) || !empty($attempt_list)) {
$label = get_lang('ContinueTest');

Loading…
Cancel
Save