Commit 046d8165 authored by Tim Hunt's avatar Tim Hunt
Browse files

MDL-47494 gapselect: Add lots of missing public/protected/private.

parent aa6a0ff0
...@@ -124,7 +124,7 @@ class restore_qtype_gapselect_plugin extends restore_qtype_plugin { ...@@ -124,7 +124,7 @@ class restore_qtype_gapselect_plugin extends restore_qtype_plugin {
/** /**
* Return the contents of this qtype to be processed by the links decoder * Return the contents of this qtype to be processed by the links decoder
*/ */
static public function define_decode_contents() { public static function define_decode_contents() {
$contents = array(); $contents = array();
......
...@@ -92,7 +92,7 @@ class qtype_gapselect_edit_form_base extends question_edit_form { ...@@ -92,7 +92,7 @@ class qtype_gapselect_edit_form_base extends question_edit_form {
* definition_inner adds all specific fields to the form. * definition_inner adds all specific fields to the form.
* @param object $mform (the form being built). * @param object $mform (the form being built).
*/ */
function definition_inner($mform) { protected function definition_inner($mform) {
global $CFG; global $CFG;
//add the answer (choice) fields to the form //add the answer (choice) fields to the form
...@@ -242,7 +242,7 @@ class qtype_gapselect_edit_form_base extends question_edit_form { ...@@ -242,7 +242,7 @@ class qtype_gapselect_edit_form_base extends question_edit_form {
return false; return false;
} }
function qtype() { public function qtype() {
return ''; return '';
} }
} }
\ No newline at end of file
...@@ -40,7 +40,7 @@ class qtype_gapselect_edit_form extends qtype_gapselect_edit_form_base { ...@@ -40,7 +40,7 @@ class qtype_gapselect_edit_form extends qtype_gapselect_edit_form_base {
// HTML tags allowed in answers (choices). // HTML tags allowed in answers (choices).
protected $allowedhtmltags = array(); protected $allowedhtmltags = array();
function qtype() { public function qtype() {
return 'gapselect'; return 'gapselect';
} }
} }
...@@ -57,7 +57,7 @@ class qtype_gapselect extends qtype_gapselect_base { ...@@ -57,7 +57,7 @@ class qtype_gapselect extends qtype_gapselect_base {
return 'selectgroup'; return 'selectgroup';
} }
function import_from_xml($data, $question, $format, $extra=null) { public function import_from_xml($data, $question, $format, $extra=null) {
if (!isset($data['@']['type']) || $data['@']['type'] != 'gapselect') { if (!isset($data['@']['type']) || $data['@']['type'] != 'gapselect') {
return false; return false;
} }
...@@ -98,7 +98,7 @@ class qtype_gapselect extends qtype_gapselect_base { ...@@ -98,7 +98,7 @@ class qtype_gapselect extends qtype_gapselect_base {
return $question; return $question;
} }
function export_to_xml($question, $format, $extra = null) { public function export_to_xml($question, $format, $extra = null) {
$output = ''; $output = '';
$output .= ' <shuffleanswers>' . $question->options->shuffleanswers . "</shuffleanswers>\n"; $output .= ' <shuffleanswers>' . $question->options->shuffleanswers . "</shuffleanswers>\n";
......
...@@ -308,7 +308,7 @@ abstract class qtype_gapselect_base extends question_type { ...@@ -308,7 +308,7 @@ abstract class qtype_gapselect_base extends question_type {
return $parts; return $parts;
} }
function move_files($questionid, $oldcontextid, $newcontextid) { public function move_files($questionid, $oldcontextid, $newcontextid) {
parent::move_files($questionid, $oldcontextid, $newcontextid); parent::move_files($questionid, $oldcontextid, $newcontextid);
$fs = get_file_storage(); $fs = get_file_storage();
......
Supports Markdown
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment