MDL-38821 enrol_paypal: fix cost param type

NoelDeMartin · Apr 2, 2013 · 881afa4 · 881afa4
1 parent 6842ab4
commit 881afa4
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/enrol/paypal/edit.php b/enrol/paypal/edit.php
@@ -69,7 +69,7 @@
 
         $instance->status         = $data->status;
         $instance->name           = $data->name;
-        $instance->cost           = $data->cost;
+        $instance->cost           = unformat_float($data->cost);
         $instance->currency       = $data->currency;
         $instance->roleid         = $data->roleid;
         $instance->enrolperiod    = $data->enrolperiod;
@@ -83,7 +83,7 @@
         }
 
     } else {
-        $fields = array('status'=>$data->status, 'name'=>$data->name, 'cost'=>$data->cost, 'currency'=>$data->currency, 'roleid'=>$data->roleid,
+        $fields = array('status'=>$data->status, 'name'=>$data->name, 'cost'=>unformat_float($data->cost), 'currency'=>$data->currency, 'roleid'=>$data->roleid,
                         'enrolperiod'=>$data->enrolperiod, 'enrolstartdate'=>$data->enrolstartdate, 'enrolenddate'=>$data->enrolenddate);
         $plugin->add_instance($course, $fields);
     }

diff --git a/enrol/paypal/edit_form.php b/enrol/paypal/edit_form.php
@@ -47,7 +47,7 @@ function definition() {
         $mform->setDefault('status', $plugin->get_config('status'));
 
         $mform->addElement('text', 'cost', get_string('cost', 'enrol_paypal'), array('size'=>4));
-        $mform->setType('cost', PARAM_TEXT);
+        $mform->setType('cost', PARAM_RAW); // Use unformat_float to get real value.
         $mform->setDefault('cost', $plugin->get_config('cost'));
 
         $paypalcurrencies = $plugin->get_currencies();