danielneis / moodle-enrol_pagseguro

lib.php

Doc Comments +2 added lines, -2 removed lines

@@ -44,7 +44,7 @@ 
      * of one type exist. One instance may also produce several icons.
      *
      * @param array $instances all enrol instances of this type in one course
-     * @return array of pix_icon
+     * @return pix_icon[] of pix_icon
      */
     public function get_info_icons(array $instances) {
         return array(new pix_icon('icon', get_string('pluginname', 'enrol_pagseguro'), 'enrol_pagseguro'));
@@ -72,7 +72,7 @@ 
     /**
      * Sets up navigation entries.
      *
-     * @param object $instance
+     * @param stdClass $instance
      * @return void
      */
     public function add_course_navigation($instancesnode, stdClass $instance) {

Indentation +2 added lines, -2 removed lines

@@ -77,7 +77,7 @@ 
      */
     public function add_course_navigation($instancesnode, stdClass $instance) {
         if ($instance->enrol !== 'pagseguro') {
-             throw new coding_exception('Invalid enrol instance type!');
+                throw new coding_exception('Invalid enrol instance type!');
         }
 
         $context = context_course::instance($instance->courseid);
@@ -159,7 +159,7 @@ 
 
         // Pass $view=true to filter hidden caps if the user cannot see them
         if ($users = get_users_by_capability($context, 'moodle/course:update', 'u.*', 'u.id ASC',
-                                             '', '', '', '', false, true)) {
+                                                '', '', '', '', false, true)) {
             $users = sort_by_roleassignment_authority($users, $context);
             $teacher = array_shift($users);
         } else {

Spacing +1 added lines, -1 removed lines

@@ -166,7 +166,7 @@
             $teacher = false;
         }
 
-        if ( (float) $instance->cost <= 0 ) {
+        if ((float) $instance->cost <= 0) {
             $cost = (float) $this->get_config('cost');
         } else {
             $cost = (float) $instance->cost;

process.php

Doc Comments +3 added lines

@@ -370,6 +370,9 @@
     header("Location: $success_returnurl");
 }
 
+/**
+ * @param stdClass $data
+ */
 function message_pagseguro_error_to_admin($subject, $data) {
     $admin = get_admin();
     $site = get_site();

Spacing +26 added lines, -26 removed lines

@@ -43,18 +43,18 @@ 
 define('COMMERCE_PAGSEGURO_STATUS_REFUNDED', 6);
 define('COMMERCE_PAGSEGURO_STATUS_CANCELED', 7);
 define('COMMERCE_PAYMENT_STATUS_SUCCESS', 'success');
-define('COMMERCE_PAYMENT_STATUS_FAILURE', 'failure') ;
+define('COMMERCE_PAYMENT_STATUS_FAILURE', 'failure');
 define('COMMERCE_PAYMENT_STATUS_PENDING', 'pending');
 
-$userid       =  $USER->id;
-$plugin       =  enrol_get_plugin('pagseguro');
-$email        =  $plugin->get_config('pagsegurobusiness');
-$token        =  $plugin->get_config('pagsegurotoken');
+$userid       = $USER->id;
+$plugin       = enrol_get_plugin('pagseguro');
+$email        = $plugin->get_config('pagsegurobusiness');
+$token        = $plugin->get_config('pagsegurotoken');
 
 $error_returnurl   = $CFG->wwwroot.'/enrol/pagseguro/return.php';
 $success_returnurl = $CFG->wwwroot.'/enrol/pagseguro/return.php';
 
-$instanceid  = optional_param('instanceid', 0, PARAM_INT);
+$instanceid = optional_param('instanceid', 0, PARAM_INT);
 
 $plugin_instance = $DB->get_record("enrol", array("id" => $instanceid, "status" => 0));
 $courseid     = $plugin_instance->courseid;
@@ -62,14 +62,14 @@ 
 $currency     = $plugin->get_config('currency');
 $encoding     = 'UTF-8';
 $item_id      = $courseid;
-$item_desc    = empty($course->fullname) ? null: $course->fullname;
-$item_qty     = (int)1;
+$item_desc    = empty($course->fullname) ? null : $course->fullname;
+$item_qty     = (int) 1;
 $item_cost    = empty($plugin_instance->cost) ? 0.00 : number_format($plugin_instance->cost, 2);
 $item_cost    = str_replace(',', '', $item_cost);
 $item_amount  = $item_cost;
 
-$redirect_url =  $CFG->wwwroot.'/enrol/pagseguro/process.php?instanceid='.$instanceid;
-$submitValue  =  get_string("sendpaymentbutton", "enrol_pagseguro");
+$redirect_url = $CFG->wwwroot.'/enrol/pagseguro/process.php?instanceid='.$instanceid;
+$submitValue  = get_string("sendpaymentbutton", "enrol_pagseguro");
 
 $submited = optional_param('usersubmited', 0, PARAM_INT);
 
@@ -79,7 +79,7 @@ 
 $transactionid = optional_param('transaction_id', '', PARAM_RAW);
 
 if ($submited) {
-    $url = "https://ws.pagseguro.uol.com.br/v2/checkout/?email=" . urlencode($email) . "&token=" . $token;
+    $url = "https://ws.pagseguro.uol.com.br/v2/checkout/?email=".urlencode($email)."&token=".$token;
 
     $xml = "<?xml version=\"1.0\" encoding=\"$encoding\" standalone=\"yes\"?>
         <checkout>
@@ -132,13 +132,13 @@ 
     $transaction = curl_exec($curl);
     curl_close($curl);
 
-    if ($transaction == 'Unauthorized'){
+    if ($transaction == 'Unauthorized') {
         // Error=1 Não autorizado.
         $error_returnurl .= "?id={$courseid}&error=1";
         header("Location: $error_returnurl");
-        exit;//Mantenha essa linha
+        exit; //Mantenha essa linha
     } else {
-        $transaction_data  = serialize(trim($transaction));
+        $transaction_data = serialize(trim($transaction));
         process_moodle($transaction_data, $instanceid, $courseid);
     }
 
@@ -148,7 +148,7 @@ 
 if (!empty($notificationCode)) {
     $transaction = null;
     // Sets the web service URL.
-    $url = "https://ws.pagseguro.uol.com.br/v2/transactions/notifications/" . $notificationCode . "?email=".$email."&token=".$token;
+    $url = "https://ws.pagseguro.uol.com.br/v2/transactions/notifications/".$notificationCode."?email=".$email."&token=".$token;
 
     $curl = curl_init($url);
     curl_setopt($curl, CURLOPT_SSL_VERIFYPEER, false);
@@ -156,19 +156,19 @@ 
     $transaction = curl_exec($curl);
     curl_close($curl);
 
-    if ($transaction == 'Unauthorized'){
+    if ($transaction == 'Unauthorized') {
         // Error=1 Não autorizado.
         $error_returnurl .= "?id={$courseid}&error=1";
         header("Location: $error_returnurl");
-        exit;//Mantenha essa linha
+        exit; //Mantenha essa linha
     } else {
-        $transaction_data  = serialize(trim($transaction));
+        $transaction_data = serialize(trim($transaction));
         process_moodle($transaction_data, $instanceid, $courseid);
     }
 }
 
 function process_moodle($transaction_data, $instanceid, $cid) {
-    global $CFG,$USER,$DB,$course;
+    global $CFG, $USER, $DB, $course;
 
     /// Read all the data from pagseguro and get it ready for later;
     /// we expect only valid UTF-8 encoding, it is the responsibility
@@ -188,16 +188,16 @@ 
     $transaction_xml = unserialize($transaction_data);
     $transaction = json_decode(json_encode(simplexml_load_string($transaction_xml)));
 
-    if($transaction) {
+    if ($transaction) {
         foreach ($transaction as $trans_key => $trans_value) {
             $trans_key = strtolower($trans_key);
-            if(!is_object($trans_value)) {
+            if (!is_object($trans_value)) {
                 $data->$trans_key = $trans_value;
             } else {
-                foreach($trans_value as $key => $value) {
+                foreach ($trans_value as $key => $value) {
                     $key = strtolower($key);
-                    if(is_object($value)) {
-                        foreach($value as $k => $v) {
+                    if (is_object($value)) {
+                        foreach ($value as $k => $v) {
                             $k = strtolower($k);
                             $k = $trans_key.'_'.$key.'_'.$k;
                             $data->$k = $v;
@@ -221,7 +221,7 @@ 
     $data->instanceid       = $instanceid;
     $data->timeupdated      = time();
 
-    if(!isset($data->reference) && empty($data->reference)) {
+    if (!isset($data->reference) && empty($data->reference)) {
         $data->reference    = $plugin->get_config('pagsegurobusiness');
     }
 
@@ -297,7 +297,7 @@ 
     $coursecontext = get_context_instance(CONTEXT_COURSE, $course->id);
 
     // Check that amount paid is the correct amount
-    if ( (float) $plugin_instance->cost <= 0 ) {
+    if ((float) $plugin_instance->cost <= 0) {
         $cost = (float) $plugin->get_config('cost');
     } else {
         $cost = (float) $plugin_instance->cost;

edit_form.php

Indentation +2 added lines, -2 removed lines

@@ -42,7 +42,7 @@ 
         $mform->setType('name', PARAM_TEXT);
 
         $options = array(ENROL_INSTANCE_ENABLED  => get_string('yes'),
-                         ENROL_INSTANCE_DISABLED => get_string('no'));
+                            ENROL_INSTANCE_DISABLED => get_string('no'));
         $mform->addElement('select', 'status', get_string('status', 'enrol_pagseguro'), $options);
         $mform->setDefault('status', $plugin->get_config('status'));
 
@@ -51,7 +51,7 @@ 
         $mform->setDefault('cost', $plugin->get_config('cost'));
 
         $mform->addElement('select', 'currency', get_string('currency', 'enrol_pagseguro'),
-                           \get_string_manager()->get_list_of_currencies());
+                            \get_string_manager()->get_list_of_currencies());
         $mform->setDefault('currency', $plugin->get_config('currency'));
         
         if ($instance->id) {

locallib.php

Indentation +2 added lines, -2 removed lines

@@ -42,8 +42,8 @@
 
         $pagseguroimgurl = "https://p.simg.uol.com.br/out/pagseguro/i/botoes/pagamentos/99x61-pagar-assina.gif";
         $mform->addElement('static', 'paymentrequired', '', 
-                           html_writer::empty_tag('img', array('alt' => get_string('pagseguroaccepted', 'enrol_pagseguro'),
-                                                               'src' => $pagseguroimgurl)));
+                            html_writer::empty_tag('img', array('alt' => get_string('pagseguroaccepted', 'enrol_pagseguro'),
+                                                                'src' => $pagseguroimgurl)));
 
         $mform->addElement('hidden', 'courseid', $instance->courseid);
         $mform->setType('courseid', PARAM_INT);

settings.php

Indentation +1 added lines, -1 removed lines

@@ -47,7 +47,7 @@
         get_string('enrolinstancedefaults', 'admin'), get_string('enrolinstancedefaults_desc', 'admin')));
 
     $options = array(ENROL_INSTANCE_ENABLED  => get_string('yes'),
-                     ENROL_INSTANCE_DISABLED => get_string('no'));
+                        ENROL_INSTANCE_DISABLED => get_string('no'));
     $settings->add(new admin_setting_configselect('enrol_pagseguro/status',
         get_string('status', 'enrol_pagseguro'), get_string('status_desc', 'enrol_pagseguro'), ENROL_INSTANCE_DISABLED, $options));
 

version.php

Spacing +3 added lines, -3 removed lines

@@ -27,7 +27,7 @@
 
 defined('MOODLE_INTERNAL') || die();
 
-$plugin->version   = 2016030900;        // The current plugin version (Date: YYYYMMDDXX)
-$plugin->requires  = 2011112901;        // Requires this Moodle version
+$plugin->version   = 2016030900; // The current plugin version (Date: YYYYMMDDXX)
+$plugin->requires  = 2011112901; // Requires this Moodle version
 $plugin->release   = '3';
-$plugin->component = 'enrol_pagseguro';    // Full name of the plugin (used for diagnostics)
+$plugin->component = 'enrol_pagseguro'; // Full name of the plugin (used for diagnostics)