diff --git a/backup/backup_check.html b/backup/backup_check.html index 7e3a421e828..dba827e5996 100644 --- a/backup/backup_check.html +++ b/backup/backup_check.html @@ -46,41 +46,20 @@ //Check data //Check module info $var = "backup_".$modname; - if (!isset($$var)) { - $$var = 1; - } + $$var = optional_param( $var,0); //Check include user info $var = "backup_user_info_".$modname; - if (!isset($$var)) { - $$var = 1; - } + $$var = optional_param( $var,0); } } //Check other parameters - if (!isset($backup_metacourse)) { - $backup_metacourse = 1; - } - - if (!isset($backup_users)) { - $backup_users = 1; - } - - if (!isset($backup_logs)) { - $backup_logs = 0; - } - - if (!isset($backup_user_files)) { - $backup_user_files = 1; - } - - if (!isset($backup_course_files)) { - $backup_course_files = 2; - } - - if (!isset($backup_messages)) { - $backup_messages = 1; - } + $backup_metacourse = optional_param('backup_metacourse',1); + $backup_users = optional_param('backup_users',1); + $backup_logs = optional_param('backup_logs',0); + $backup_user_files = optional_param('backup_user_files',1); + $backup_course_files = optional_param('backup_course_files',1); + $backup_messages = optional_param('backup_messages',1); if ($count == 0) { notice("No backupable modules are installed!"); diff --git a/backup/backup_form.html b/backup/backup_form.html index 61100705983..ed79fa34774 100644 --- a/backup/backup_form.html +++ b/backup/backup_form.html @@ -45,41 +45,20 @@ //Check data //Check module info $var = "backup_".$modname; - if (!isset($$var)) { - $$var = 1; - } + $$var = optional_param( $var,1); //Check include user info $var = "backup_user_info_".$modname; - if (!isset($$var)) { - $$var = 1; - } + $$var = optional_param( $var,1); } } //Check other parameters - if (!isset($backup_metacourse)) { - $backup_metacourse = 1; - } - - if (!isset($backup_users)) { - $backup_users = 1; - } - - if (!isset($backup_logs)) { - $backup_logs = 0; - } - - if (!isset($backup_user_files)) { - $backup_user_files = 1; - } - - if (!isset($backup_course_files)) { - $backup_course_files = 1; - } - - if (!isset($backup_messages)) { - $backup_messages = 1; - } + $backup_metacourse = optional_param('backup_metacourse',1); + $backup_users = optional_param('backup_users',1); + $backup_logs = optional_param('backup_logs',0); + $backup_user_files = optional_param('backup_user_files',1); + $backup_course_files = optional_param('backup_course_files',1); + $backup_messages = optional_param('backup_messages',1); if ($count == 0) { notice("No backupable modules are installed!"); @@ -87,10 +66,69 @@ ?> -
+ + + "; + echo ""; + echo ""; + echo ""; + echo ""; + $currentrow = 0; foreach ($allmods as $mod) { $modname = $mod->name; $modbackup = $modname."_backup_mods"; @@ -98,35 +136,38 @@ $var = "exists_".$modname; if (isset($$var) && $$var) { //Print the full tr - echo ""; - echo " "; + echo " \n"; } } //Line - echo "\n"; + echo "\n"; if (empty($to)) { //Now print the Metacourse tr echo ""; - echo ""; - echo ""; - echo ""; - echo ""; - echo ""; - echo "
"; + echo ''.get_string("include").":"; + echo ""; + echo "". get_string("all")."/"; + echo "". get_string("none").""; + echo ""; + echo ' '; + echo ""; + echo "". get_string("all")."/"; + echo "". get_string("none").""; + echo "

"; - echo get_string("include")." ". get_string("modulenameplural",$modname).":"; - echo "\n"; + echo "
 "; + echo "\n"; $backup_options[0] = get_string("no"); $backup_options[1] = get_string("yes"); $var = "backup_".$modname; - choose_from_menu($backup_options, $var, $$var, ""); + //choose_from_menu($backup_options, $var, $$var, ""); + //Print the checkbox + print_checkbox($var, $$var, $$var, get_string("modulenameplural",$modname)); $var = "backup_user_info_".$modname; + echo " "; + echo ""; if (empty($to)) { $backup_user_options[0] = get_string("withoutuserdata"); $backup_user_options[1] = get_string("withuserdata"); - choose_from_menu($backup_user_options, $var, $$var, ""); - } - else { + //choose_from_menu($backup_user_options, $var, $$var, ""); + print_checkbox($var, $$var, $$var, get_string("userdata")); + } else { echo ''; } echo "


"; + echo ""; echo get_string ("metacourse").":"; - echo ""; + echo ""; $meta_options[0] = get_string("no"); $meta_options[1] = get_string("yes"); choose_from_menu($meta_options, "backup_metacourse", $backup_metacourse, ""); @@ -139,9 +180,9 @@ if (empty($to)) { //Now print the Users tr echo "
"; + echo ""; echo get_string("users").":"; - echo ""; + echo ""; $user_options[0] = get_string("all"); $user_options[1] = get_string("course"); $user_options[2] = get_string("none"); @@ -155,9 +196,9 @@ if (empty($to)) { //Now print the Logs tr echo "
"; + echo ""; echo get_string("logs").":"; - echo ""; + echo ""; $log_options[0] = get_string("no"); $log_options[1] = get_string("yes"); choose_from_menu($log_options, "backup_logs", $backup_logs, ""); @@ -170,9 +211,9 @@ if (empty($to)) { //Now print the User Files tr echo "
"; + echo ""; echo get_string ("userfiles").":"; - echo ""; + echo ""; $user_file_options[0] = get_string("no"); $user_file_options[1] = get_string("yes"); choose_from_menu($user_file_options, "backup_user_files", $backup_user_files, ""); @@ -183,9 +224,9 @@ } //Now print the Course Files tr echo "
"; + echo ""; echo get_string ("coursefiles").":"; - echo ""; + echo ""; $course_file_options[0] = get_string("no"); $course_file_options[1] = get_string("yes"); choose_from_menu($course_file_options, "backup_course_files", $backup_course_files, ""); @@ -194,9 +235,9 @@ if (empty($to) && $course->id == SITEID) { //If we are in a SITEID backup print the Messages tr echo "
"; + echo ""; echo get_string ('messages','message').":"; - echo ""; + echo ""; $mess_options[0] = get_string("no"); $mess_options[1] = get_string("yes"); choose_from_menu($mess_options, "backup_messages", $backup_messages, "");