danielneis / moodle-availability_paypal

ipn.php

Indentation +1 added lines, -1 removed lines

@@ -219,7 +219,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 +10 added lines, -10 removed lines

@@ -31,7 +31,7 @@ 
 
 require("../../../config.php");
 require_once($CFG->libdir.'/eventslib.php');
-require_once($CFG->libdir . '/filelib.php');
+require_once($CFG->libdir.'/filelib.php');
 
 // PayPal does not like when we return error messages here,
 // the custom handler just logs exceptions and stops.
@@ -76,15 +76,15 @@ 
 $custom = optional_param('custom', '', PARAM_TEXT);
 $custom = explode('-', $custom);
 $data->userid           = (int)$custom[0];
-$data->contextid       = (int)$custom[1];
+$data->contextid = (int)$custom[1];
 $data->timeupdated      = time();
 
-if (! $user = $DB->get_record("user", array("id" => $data->userid))) {
+if (!$user = $DB->get_record("user", array("id" => $data->userid))) {
     paypal_message_error_to_admin("Not a valid user id", $data);
     die;
 }
 
-if (! $context = context::instance_by_id($data->contextid, IGNORE_MISSING)) {
+if (!$context = context::instance_by_id($data->contextid, IGNORE_MISSING)) {
     paypal_message_error_to_admin("Not a valid context id", $data);
     die;
 }
@@ -166,8 +166,8 @@ 
 
         // If our status is not completed or not pending on an echeck clearance then ignore and die.
         // This check is redundant at present but may be useful if paypal extend the return codes in the future.
-        if (! ( $data->payment_status == "Completed" or
-               ($data->payment_status == "Pending" and $data->pending_reason == "echeck") ) ) {
+        if (!($data->payment_status == "Completed" or
+               ($data->payment_status == "Pending" and $data->pending_reason == "echeck"))) {
             die;
         }
 
@@ -201,10 +201,10 @@ 
         $coursecontext = context_course::instance($course->id, IGNORE_MISSING);
 
         // Check that amount paid is the correct amount.
-        if ( (float) $paypal->cost < 0 ) {
-            $cost = (float) 0;
+        if ((float)$paypal->cost < 0) {
+            $cost = (float)0;
         } else {
-            $cost = (float) $paypal->cost;
+            $cost = (float)$paypal->cost;
         }
 
         // Use the same rounding of floats as on the plugin form.
@@ -287,7 +287,7 @@ 
         }
         */
 
-    } else if (strcmp ($result, "INVALID") == 0) { // ERROR
+    } else if (strcmp($result, "INVALID") == 0) { // ERROR
         $DB->insert_record("paypal_transactions", $data, false);
         paypal_message_error_to_admin("Received an invalid payment notification!! (Fake payment?)", $data);
     }

version.php

Spacing +1 added lines, -1 removed lines

@@ -26,5 +26,5 @@
 $plugin->version = 2015051100;
 $plugin->requires = 2015050500;
 $plugin->release = 1;
-$plugin->maturity   = MATURITY_STABLE;
+$plugin->maturity = MATURITY_STABLE;
 $plugin->component = 'availability_paypal';

classes/condition.php

Indentation +1 added lines, -1 removed lines

@@ -95,7 +95,7 @@
         // should double-check with paypal everytime ?
         $context = $info->get_context();
         return $DB->record_exists('paypal_transactions',
-                                  array('userid' => $userid,
+                                    array('userid' => $userid,
                                         'contextid' => $context->id,
                                         'payment_status' => 'Completed'));
     }

Spacing +2 added lines, -2 removed lines

@@ -51,7 +51,7 @@ 
             $this->cost = $structure->cost;
         }
         if (isset($structure->itemname)) {
-            $this->itemname= $structure->itemname;
+            $this->itemname = $structure->itemname;
         }
         if (isset($structure->itemnumber)) {
             $this->itemnumber = $structure->itemnumber;
@@ -70,7 +70,7 @@ 
             $result->cost = $this->cost;
         }
         if ($this->itemname) {
-            $result->itemname= $this->itemname;
+            $result->itemname = $this->itemname;
         }
         if ($this->itemnumber) {
             $result->itemnumber = $this->itemnumber;