From: ethem <ethem>
Date: Wed, 26 Aug 2009 10:13:01 +0000 (+0000)
Subject: Fix for make_checkbox
X-Git-Url: http://git.mjollnir.org/gw?a=commitdiff_plain;h=81c4689685a08686023c945e49a0520c8ed5cb85;p=moodle.git

Fix for make_checkbox
---

diff --git a/enrol/authorize/authorize.js b/enrol/authorize/authorize.js
index 122b20c4a5..df0b03c15b 100644
--- a/enrol/authorize/authorize.js
+++ b/enrol/authorize/authorize.js
@@ -1,5 +1,5 @@
 function authorize_jump_to_mypayments(e, args) {
-    var locationtogo = moodle_cfg.wwwroot + '/enrol/authorize/index.php?status' + args.status;
+    var locationtogo = moodle_cfg.wwwroot + '/enrol/authorize/index.php?status=' + args.status;
     locationtogo += '&user=' + (this.checked ? args.userid : '0');
     top.location.href = locationtogo;
 }
diff --git a/enrol/authorize/config_form.php b/enrol/authorize/config_form.php
index a8df96869c..58beeebbd4 100755
--- a/enrol/authorize/config_form.php
+++ b/enrol/authorize/config_form.php
@@ -101,8 +101,8 @@ if (!isset($frm->acceptechecktypes)) {
 
 <tr valign="top">
     <td align="right">delete_current:</td>
-    <td><?php echo $OUTPUT->checkbox(html_select_option::make_checkbox('1', !empty($frm->delete_current)), 'delete_current');?> <br />
-        <?php print_string("deletecheck", "moodle", get_string('oldpassword')) ?><br /><hr /></td>
+    <td><?php echo $OUTPUT->checkbox(html_select_option::make_checkbox('1', !empty($frm->delete_current), get_string("deletecheck", "moodle", get_string('oldpassword'))), 'delete_current');?> <br />
+        <hr /></td>
 </tr>
 
 <tr valign="top">
@@ -124,21 +124,21 @@ if (!isset($frm->acceptechecktypes)) {
 
 <tr valign="top">
     <td align="right">an_avs:</td>
-    <td><?php echo $OUTPUT->checkbox(html_select_option::make_checkbox('1', !empty($frm->an_avs)), 'an_avs'); ?><br />
-        <?php print_string("adminavs", "enrol_authorize") ?></td>
+    <td><?php echo $OUTPUT->checkbox(html_select_option::make_checkbox('1', !empty($frm->an_avs), get_string("adminavs", "enrol_authorize")), 'an_avs'); ?><br />
+    </td>
 </tr>
 
 <tr valign="top">
     <td align="right">an_authcode:</td>
-    <td><?php echo $OUTPUT->checkbox(html_select_option::make_checkbox('1', !empty($frm->an_authcode)), 'an_authcode');  ?>
+    <td><?php echo $OUTPUT->checkbox(html_select_option::make_checkbox('1', !empty($frm->an_authcode), get_string("adminauthcode", "enrol_authorize")), 'an_authcode');  ?>
         <?php echo $OUTPUT->help_icon(moodle_help_icon::make('authcode', 'authcode', 'enrol/authorize')); ?><br />
-        <?php print_string("adminauthcode", "enrol_authorize") ?></td>
+    </td>
 </tr>
 
 <tr valign="top">
     <td align="right">an_test:</td>
-    <td><?php echo $OUTPUT->checkbox(html_select_option::make_checkbox('1', !empty($frm->an_test)), 'an_test');  ?><br />
-        <?php print_string("antestmode", "enrol_authorize") ?></td>
+    <td><?php echo $OUTPUT->checkbox(html_select_option::make_checkbox('1', !empty($frm->an_test), get_string("antestmode", "enrol_authorize")), 'an_test');  ?><br />
+    </td>
 </tr>
 
 <tr valign="top">
@@ -182,9 +182,9 @@ if (!isset($frm->acceptechecktypes)) {
 
 <tr valign="top">
     <td align="right">an_review:</td>
-    <td><?php echo $OUTPUT->checkbox(html_select_option::make_checkbox('1', !empty($frm->an_review)), 'an_review'); ?>
+    <td><?php echo $OUTPUT->checkbox(html_select_option::make_checkbox('1', !empty($frm->an_review), get_string("adminreview", "enrol_authorize")), 'an_review'); ?>
         <?php echo $OUTPUT->help_icon(moodle_help_icon::make('review', get_string('adminhelpreviewtitle', 'enrol_authorize'), 'enrol/authorize')); ?><br />
-        <?php print_string("adminreview", "enrol_authorize") ?></td>
+    </td>
 </tr>
 
 <tr valign="top">
@@ -205,8 +205,8 @@ if (!isset($frm->acceptechecktypes)) {
 
 <tr valign="top">
     <td align="right">an_emailexpiredteacher:</td>
-    <td><?php echo $OUTPUT->checkbox(html_select_option::make_checkbox('1', !empty($frm->an_emailexpiredteacher)), 'an_emailexpiredteacher');  ?><br />
-        <?php print_string("adminemailexpiredteacher", "enrol_authorize") ?></td>
+    <td><?php echo $OUTPUT->checkbox(html_select_option::make_checkbox('1', !empty($frm->an_emailexpiredteacher), get_string("adminemailexpiredteacher", "enrol_authorize")), 'an_emailexpiredteacher');  ?><br />
+    </td>
 </tr>
 
 <tr valign="top">
@@ -224,20 +224,20 @@ if (!isset($frm->acceptechecktypes)) {
 
 <tr valign="top">
     <td align="right">enrol_mailstudents:</td>
-    <td><?php echo $OUTPUT->checkbox(html_select_option::make_checkbox('1', !empty($frm->enrol_mailstudents)), 'enrol_mailstudents'); ?><br />
-        <?php print_string("mailstudents") ?></td>
+    <td><?php echo $OUTPUT->checkbox(html_select_option::make_checkbox('1', !empty($frm->enrol_mailstudents), get_string("mailstudents")), 'enrol_mailstudents'); ?><br />
+    </td>
 </tr>
 
 <tr valign="top">
     <td align="right">enrol_mailteachers:</td>
-    <td><?php echo $OUTPUT->checkbox(html_select_option::make_checkbox('1', !empty($frm->enrol_mailteachers)), 'enrol_mailteachers');  ?><br />
-        <?php print_string("mailteachers") ?></td>
+    <td><?php echo $OUTPUT->checkbox(html_select_option::make_checkbox('1', !empty($frm->enrol_mailteachers), get_string("mailteachers")), 'enrol_mailteachers');  ?><br />
+    </td>
 </tr>
 
 <tr valign="top">
     <td align="right">enrol_mailadmins:</td>
-    <td><?php echo $OUTPUT->checkbox(html_select_option::make_checkbox('1', !empty($frm->enrol_mailadmins)), 'enrol_mailadmins');  ?><br />
-        <?php print_string("mailadmins") ?></td>
+    <td><?php echo $OUTPUT->checkbox(html_select_option::make_checkbox('1', !empty($frm->enrol_mailadmins), get_string("mailadmins")), 'enrol_mailadmins');  ?><br />
+    </td>
 </tr>
 
 </table>
diff --git a/enrol/authorize/enrol.php b/enrol/authorize/enrol.php
index b6eff1f84c..afa1dfa534 100755
--- a/enrol/authorize/enrol.php
+++ b/enrol/authorize/enrol.php
@@ -97,9 +97,9 @@ class enrolment_plugin_authorize
                 }
             }
 
-            $OUTPUT->box_start();
+            echo $OUTPUT->box_start();
             $frmenrol->display();
-            $OUTPUT->box_end();
+            echo $OUTPUT->box_end();
         }
 
         if ($course->password) {