Merged from MOODLE_16_STABLE.

diff --git a/enrol/authorize/enrol.html b/enrol/authorize/enrol.html
index 5dabac7544356d56c09c29a18b18e5895fd42730..5dcc2267e6d854556493d486610d260226576814 100755 (executable)
--- a/enrol/authorize/enrol.html
+++ b/enrol/authorize/enrol.html
@@ -1,5 +1,6 @@
 <?php // $Id$
 
+$this->prevent_double_paid($course);
 $formvars = array('password', 'ccaddress', 'cccity', 'ccstate', 'cccountry', 'cczip',
                   'ccfirstname', 'cclastname', 'cc', 'ccexpiremm', 'ccexpireyyyy', 'cctype', 'cvv');
 

diff --git a/enrol/authorize/enrol.php b/enrol/authorize/enrol.php
index 0eb9a38ac678924decb6e209f77e665758de3af8..545b66d6e317239dbb8b028e471068d16b192a54 100755 (executable)
--- a/enrol/authorize/enrol.php
+++ b/enrol/authorize/enrol.php
@@ -100,6 +100,16 @@ class enrolment_plugin_authorize
     function print_entry($course) {
         global $CFG, $USER, $form;
 
+        $zerocost = $this->zero_cost($course);
+        if ($zerocost) {
+            $manual = enrolment_factory::factory('manual');
+            if (!empty($this->errormsg)) {
+                $manual->errormsg = $this->errormsg;
+            }
+            $manual->print_entry($course);
+            return;
+        }
+
         httpsrequired();
 
         if (empty($_SERVER['HTTPS']) || $_SERVER['HTTPS'] == 'off') {
@@ -114,31 +124,26 @@ class enrolment_plugin_authorize
 
         $strcourses = get_string('courses');
         $strloginto = get_string('loginto', '', $course->shortname);
-        $zerocost = $this->zero_cost($course);
 
         print_header($strloginto, $course->fullname, "<a href=\"$CFG->wwwroot/course/\">$strcourses</a> -> $strloginto");
         print_course($course, '80%');
 
-        if ($course->password && !$zerocost) {
+        if ($course->password) {
             print_heading(get_string('choosemethod', 'enrol_authorize'), 'center');
         }
 
-        print_simple_box_start('center');
-        if ($zerocost){
-            echo '<div align="center"><p>'.get_string('nocostyet', 'enrol_authorize').'</p></div>';
-        } else if (isguest()) {
+        if (isguest()) {
             $curcost = $this->get_course_cost($course);
             echo '<div align="center"><p>'.get_string('paymentrequired').'</p>';
             echo '<p><b>'.get_string('cost').": $curcost[currency] $curcost[cost]".'</b></p>';
             echo '<p><a href="'.$CFG->httpswwwroot.'/login/">'.get_string('loginsite').'</a></p>';
             echo '</div>';
         } else {
-            $this->prevent_double_paid($course);
+            print_simple_box_start('center');
             include($CFG->dirroot.'/enrol/authorize/enrol.html');
+            print_simple_box_end();
         }
 
-        print_simple_box_end();
-
         if ($course->password) {
             $password = '';
             $teacher = get_teacher($course->id);
@@ -157,10 +162,10 @@ class enrolment_plugin_authorize
      * @access public
      */
     function check_entry($form, $course) {
-        if (!empty($course->password) and !empty($form->password)) {
+        if ($this->zero_cost($course) or (!empty($course->password) and !empty($form->password))) {
             $manual = enrolment_factory::factory('manual');
             $manual->check_entry($form, $course);
-            if (isset($manual->errormsg)) {
+            if (!empty($manual->errormsg)) {
                 $this->errormsg = $manual->errormsg;
             }
         } elseif ((!empty($form->ccsubmit)) and $this->validate_enrol_form($form)) {