From 0c94e43d015ee1fe64b362006ca4f8558e9c9d3b Mon Sep 17 00:00:00 2001
From: skodak <skodak>
Date: Sat, 8 Mar 2008 15:08:44 +0000
Subject: [PATCH] MDL-13848 backup form - do not include checkboxes for mods
 that are not used in course; merged from MOODLE_19_STABLE

---
 backup/backup_form.html | 45 +++++++++++++++++++----------------------
 1 file changed, 21 insertions(+), 24 deletions(-)

diff --git a/backup/backup_form.html b/backup/backup_form.html
index 2437564384..2aecf60d2e 100644
--- a/backup/backup_form.html
+++ b/backup/backup_form.html
@@ -15,7 +15,7 @@
             }
         }
     } else {
-        if (!has_capability('moodle/site:backup', get_context_instance(CONTEXT_SYSTEM, SITEID))) {
+        if (!has_capability('moodle/site:backup', get_context_instance(CONTEXT_SYSTEM))) {
             error("You need to be an admin user to use this page.", "$CFG->wwwroot/login/index.php");
         }
     }
@@ -142,8 +142,9 @@ function selectItemInCheckboxByName(formId, checkName, checked ) {
             $modname = $mod->name;
             $modbackup = $modname."_backup_mods";
             //If exists the lib & function
-            $var = "exists_".$modname;
-            if (isset($$var) && $$var) {
+            $var   = 'exists_'.$modname;
+            $exvar = 'exists_one_'.$modname;
+            if (!empty($$var) and !empty($$exvar)) {
                 //Print the full tr
                 echo "<tr class=\"r".$currentrow."\">";
                 echo "  <td align=\"right\">&nbsp;";
@@ -166,29 +167,25 @@ function selectItemInCheckboxByName(formId, checkName, checked ) {
                     echo '<input type="hidden" name="'.$var.'" value="0" />';
                 }
                 echo "</td></tr>\n";
-                // if we have the backup_one_mod function, loop here.
-                $var = 'exists_one_'.$modname;
-                if (!empty($$var)) {
-                    echo '<tr><td></td><td colspan="3"><table class="backup-form-instances">';
-                    $instances = get_all_instances_in_course($modname, $course, NULL, true);
-                    foreach ($instances as $instance) {
-                        echo '<tr><td>';
-                        $var = 'backup_'.$modname.'_instance_'.$instance->id;
-                        $$var = optional_param($var,1);
-                        print_checkbox($var,$$var,$$var,$instance->name,$instance->name,'this.form.elements[\'backup_'.$modname.'\'].checked=1;');
-                        echo '</td><td align="right">&nbsp;';
-                        $var = 'backup_user_info_'.$modname.'_instance_'.$instance->id;
-                        $$var = optional_param($var,1);
-                        if (empty($to)) {
-                            print_checkbox($var,$$var,$$var,get_string('userdata'),'','this.form.elements[\'backup_user_info_'.$modname.'\'].checked=1;');
-                        } else {
-                            echo '<input type="hidden" name="'.$var.'" value="0" />';
-                        }
-                        echo '</td></tr>';
+                echo '<tr><td></td><td colspan="3"><table class="backup-form-instances">';
+                $instances = get_all_instances_in_course($modname, $course, NULL, true);
+                $var = $exvar;
+                foreach ($instances as $instance) {
+                    echo '<tr><td>';
+                    $var = 'backup_'.$modname.'_instance_'.$instance->id;
+                    $$var = optional_param($var,1);
+                    print_checkbox($var,$$var,$$var,$instance->name,$instance->name,'this.form.elements[\'backup_'.$modname.'\'].checked=1;');
+                    echo '</td><td align="right">&nbsp;';
+                    $var = 'backup_user_info_'.$modname.'_instance_'.$instance->id;
+                    $$var = optional_param($var,1);
+                    if (empty($to)) {
+                        print_checkbox($var,$$var,$$var,get_string('userdata'),'','this.form.elements[\'backup_user_info_'.$modname.'\'].checked=1;');
+                    } else {
+                        echo '<input type="hidden" name="'.$var.'" value="0" />';
                     }
-                    echo '</table></td></tr>';
+                    echo '</td></tr>';
                 }
-
+                echo '</table></td></tr>';
             }
         }
         //Line
-- 
2.39.5