MDL-69166 core_payment: Rename componentid to itemid
authorShamim Rezaie <shamim@moodle.com>
Fri, 16 Oct 2020 18:05:54 +0000 (05:05 +1100)
committerShamim Rezaie <shamim@moodle.com>
Tue, 27 Oct 2020 04:45:25 +0000 (15:45 +1100)
22 files changed:
enrol/fee/templates/payment_region.mustache
lib/db/install.xml
lib/db/upgrade.php
payment/amd/build/gateways_modal.min.js
payment/amd/build/gateways_modal.min.js.map
payment/amd/build/repository.min.js
payment/amd/build/repository.min.js.map
payment/amd/src/gateways_modal.js
payment/amd/src/repository.js
payment/classes/external/get_available_gateways.php
payment/classes/helper.php
payment/classes/local/callback/provider.php
payment/gateway/paypal/amd/build/gateways_modal.min.js
payment/gateway/paypal/amd/build/gateways_modal.min.js.map
payment/gateway/paypal/amd/build/repository.min.js
payment/gateway/paypal/amd/build/repository.min.js.map
payment/gateway/paypal/amd/src/gateways_modal.js
payment/gateway/paypal/amd/src/repository.js
payment/gateway/paypal/classes/external/get_config_for_js.php
payment/gateway/paypal/classes/external/transaction_complete.php
payment/tests/generator/lib.php
payment/tests/helper_test.php

index f39babe..6fb27c3 100644 (file)
@@ -25,7 +25,7 @@
     Data attributes required for JS:
     * data-component
     * data-paymentarea
-    * data-componentid
+    * data-itemid
     * data-cost
     * data-description
 
@@ -60,7 +60,7 @@
             id="gateways-modal-trigger-{{ uniqid }}"
             data-component="enrol_fee"
             data-paymentarea="fee"
-            data-componentid="{{instanceid}}"
+            data-itemid="{{instanceid}}"
             data-cost="{{cost}}"
             data-description={{# quote }}{{description}}{{/ quote }}
         >
index 5fbfe08..5974516 100755 (executable)
         <FIELD NAME="id" TYPE="int" LENGTH="10" NOTNULL="true" SEQUENCE="true"/>
         <FIELD NAME="component" TYPE="char" LENGTH="100" NOTNULL="true" SEQUENCE="false" COMMENT="The plugin this payment belongs to."/>
         <FIELD NAME="paymentarea" TYPE="char" LENGTH="50" NOTNULL="true" SEQUENCE="false" COMMENT="The name of payable area"/>
-        <FIELD NAME="componentid" TYPE="int" LENGTH="10" NOTNULL="true" SEQUENCE="false"/>
+        <FIELD NAME="itemid" TYPE="int" LENGTH="10" NOTNULL="true" SEQUENCE="false"/>
         <FIELD NAME="userid" TYPE="int" LENGTH="10" NOTNULL="true" SEQUENCE="false"/>
         <FIELD NAME="amount" TYPE="char" LENGTH="20" NOTNULL="true" SEQUENCE="false"/>
         <FIELD NAME="currency" TYPE="char" LENGTH="3" NOTNULL="true" SEQUENCE="false"/>
       </KEYS>
       <INDEXES>
         <INDEX NAME="gateway" UNIQUE="false" FIELDS="gateway"/>
-        <INDEX NAME="component-paymentarea-componentid" UNIQUE="false" FIELDS="component, paymentarea, componentid"/>
+        <INDEX NAME="component-paymentarea-itemid" UNIQUE="false" FIELDS="component, paymentarea, itemid"/>
       </INDEXES>
     </TABLE>
     <TABLE NAME="infected_files" COMMENT="Table to store infected file details.">
index 3ea16b5..b2f9bb8 100644 (file)
@@ -2925,7 +2925,7 @@ function xmldb_main_upgrade($oldversion) {
         $table->add_field('id', XMLDB_TYPE_INTEGER, '10', null, XMLDB_NOTNULL, XMLDB_SEQUENCE, null);
         $table->add_field('component', XMLDB_TYPE_CHAR, '100', null, XMLDB_NOTNULL, null, null);
         $table->add_field('paymentarea', XMLDB_TYPE_CHAR, '50', null, XMLDB_NOTNULL, null, null);
-        $table->add_field('componentid', XMLDB_TYPE_INTEGER, '10', null, XMLDB_NOTNULL, null, null);
+        $table->add_field('itemid', XMLDB_TYPE_INTEGER, '10', null, XMLDB_NOTNULL, null, null);
         $table->add_field('userid', XMLDB_TYPE_INTEGER, '10', null, XMLDB_NOTNULL, null, null);
         $table->add_field('amount', XMLDB_TYPE_CHAR, '20', null, XMLDB_NOTNULL, null, null);
         $table->add_field('currency', XMLDB_TYPE_CHAR, '3', null, XMLDB_NOTNULL, null, null);
@@ -2941,7 +2941,7 @@ function xmldb_main_upgrade($oldversion) {
 
         // Adding indexes to table payments.
         $table->add_index('gateway', XMLDB_INDEX_NOTUNIQUE, ['gateway']);
-        $table->add_index('component-paymentarea-componentid', XMLDB_INDEX_NOTUNIQUE, ['component', 'paymentarea', 'componentid']);
+        $table->add_index('component-paymentarea-itemid', XMLDB_INDEX_NOTUNIQUE, ['component', 'paymentarea', 'itemid']);
 
         // Conditionally launch create table for payments.
         if (!$dbman->table_exists($table)) {
index 32d97f4..fabf9e5 100644 (file)
Binary files a/payment/amd/build/gateways_modal.min.js and b/payment/amd/build/gateways_modal.min.js differ
index 84a00d7..6e82432 100644 (file)
Binary files a/payment/amd/build/gateways_modal.min.js.map and b/payment/amd/build/gateways_modal.min.js.map differ
index 7506386..9a31003 100644 (file)
Binary files a/payment/amd/build/repository.min.js and b/payment/amd/build/repository.min.js differ
index e36de5b..47d3a2a 100644 (file)
Binary files a/payment/amd/build/repository.min.js.map and b/payment/amd/build/repository.min.js.map differ
index cf74851..afeeb1b 100644 (file)
@@ -95,7 +95,7 @@ const show = async(rootNode, {
                 gateway,
                 rootNode.dataset.component,
                 rootNode.dataset.paymentarea,
-                rootNode.dataset.componentid,
+                rootNode.dataset.itemid,
                 rootNode.dataset.description,
                 ({success, message = ''}) => {
                     modal.hide();
@@ -127,8 +127,7 @@ const show = async(rootNode, {
         }
     });
 
-    const gateways = await getAvailableGateways(rootNode.dataset.component, rootNode.dataset.paymentarea,
-        rootNode.dataset.componentid);
+    const gateways = await getAvailableGateways(rootNode.dataset.component, rootNode.dataset.paymentarea, rootNode.dataset.itemid);
     const context = {
         gateways
     };
@@ -172,16 +171,16 @@ const updateCostRegion = async(root, defaultCost = '') => {
  * Process payment using the selected gateway.
  *
  * @param {string} gateway The gateway to be used for payment
- * @param {string} component Name of the component that the componentId belongs to
- * @param {string} paymentArea Name of the area in the component that the componentId belongs to
- * @param {number} componentId An internal identifier that is used by the component
+ * @param {string} component Name of the component that the itemId belongs to
+ * @param {string} paymentArea Name of the area in the component that the itemId belongs to
+ * @param {number} itemId An internal identifier that is used by the component
  * @param {string} description Description of the payment
  * @param {processPaymentCallback} callback The callback function to call when processing is finished
  * @returns {Promise<void>}
  */
-const processPayment = async(gateway, component, paymentArea, componentId, description, callback) => {
+const processPayment = async(gateway, component, paymentArea, itemId, description, callback) => {
     const paymentMethod = await import(`paygw_${gateway}/gateways_modal`);
-    paymentMethod.process(component, paymentArea, componentId, description, callback);
+    paymentMethod.process(component, paymentArea, itemId, description, callback);
 };
 
 /**
index 7b4ccbb..c3e71b2 100644 (file)
@@ -29,16 +29,16 @@ import Ajax from 'core/ajax';
  *
  * @param {string} component
  * @param {string} paymentArea
- * @param {number} componentId
+ * @param {number} itemId
  * @returns {Promise<{shortname: string, name: string, description: String}[]>}
  */
-export const getAvailableGateways = (component, paymentArea, componentId) => {
+export const getAvailableGateways = (component, paymentArea, itemId) => {
     const request = {
         methodname: 'core_payment_get_available_gateways',
         args: {
             component,
             paymentarea: paymentArea,
-            componentid: componentId,
+            itemid: itemId,
         }
     };
     return Ajax.call([request])[0];
index 3eba964..09fa896 100644 (file)
@@ -46,7 +46,7 @@ class get_available_gateways extends external_api {
         return new external_function_parameters([
             'component' => new external_value(PARAM_COMPONENT, 'Component'),
             'paymentarea' => new external_value(PARAM_AREA, 'Payment area in the component'),
-            'componentid' => new external_value(PARAM_INT, 'An identifier for payment area in the component')
+            'itemid' => new external_value(PARAM_INT, 'An identifier for payment area in the component')
         ]);
     }
 
@@ -55,23 +55,23 @@ class get_available_gateways extends external_api {
      *
      * @param string $component
      * @param string $paymentarea
-     * @param int $componentid
+     * @param int $itemid
      * @return \stdClass[]
      */
-    public static function execute(string $component, string $paymentarea, int $componentid): array {
+    public static function execute(string $component, string $paymentarea, int $itemid): array {
 
         $params = external_api::validate_parameters(self::execute_parameters(), [
             'component' => $component,
             'paymentarea' => $paymentarea,
-            'componentid' => $componentid,
+            'itemid' => $itemid,
         ]);
 
         $list = [];
-        $gateways = helper::get_available_gateways($params['component'], $params['paymentarea'], $params['componentid']);
+        $gateways = helper::get_available_gateways($params['component'], $params['paymentarea'], $params['itemid']);
         [
             'amount' => $amount,
             'currency' => $currency
-        ] = helper::get_cost($params['component'], $params['paymentarea'], $params['componentid']);
+        ] = helper::get_cost($params['component'], $params['paymentarea'], $params['itemid']);
 
         foreach ($gateways as $gateway) {
             $surcharge = helper::get_gateway_surcharge($gateway);
index 4dd64e9..0148a57 100644 (file)
@@ -62,17 +62,17 @@ class helper {
      *
      * @param string $component
      * @param string $paymentarea
-     * @param int $componentid
+     * @param int $itemid
      * @return string[]
      */
-    public static function get_available_gateways(string $component, string $paymentarea, int $componentid): array {
+    public static function get_available_gateways(string $component, string $paymentarea, int $itemid): array {
         $gateways = [];
 
         [
             'amount' => $amount,
             'currency' => $currency,
             'accountid' => $accountid,
-        ] = self::get_cost($component, $paymentarea, $componentid);
+        ] = self::get_cost($component, $paymentarea, $itemid);
         $account = new account($accountid);
         if (!$account->get('id') || !$account->get('enabled')) {
             return $gateways;
@@ -143,24 +143,24 @@ class helper {
     /**
      * Returns the attributes to place on a pay button.
      *
-     * @param string $component Name of the component that the componentid belongs to
+     * @param string $component Name of the component that the itemid belongs to
      * @param string $paymentarea
-     * @param int $componentid An internal identifier that is used by the component
+     * @param int $itemid An internal identifier that is used by the component
      * @param string $description Description of the payment
      * @return array
      */
-    public static function gateways_modal_link_params(string $component, string $paymentarea, int $componentid, string $description): array {
+    public static function gateways_modal_link_params(string $component, string $paymentarea, int $itemid, string $description): array {
         [
             'amount' => $amount,
             'currency' => $currency
-        ] = self::get_cost($component, $paymentarea, $componentid);
+        ] = self::get_cost($component, $paymentarea, $itemid);
 
         return [
             'id' => 'gateways-modal-trigger',
             'role' => 'button',
             'data-component' => $component,
             'data-paymentarea' => $paymentarea,
-            'data-componentid' => $componentid,
+            'data-itemid' => $itemid,
             'data-cost' => self::get_cost_as_string($amount, $currency),
             'data-description' => $description,
         ];
@@ -169,14 +169,14 @@ class helper {
     /**
      * Asks the cost from the related component.
      *
-     * @param string $component Name of the component that the componentid belongs to
+     * @param string $component Name of the component that the itemid belongs to
      * @param string $paymentarea
-     * @param int $componentid An internal identifier that is used by the component
+     * @param int $itemid An internal identifier that is used by the component
      * @return array['amount' => float, 'currency' => string, 'accountid' => int]
      * @throws \moodle_exception
      */
-    public static function get_cost(string $component, string $paymentarea, int $componentid): array {
-        $cost = component_class_callback("$component\\payment\\provider", 'get_cost', [$paymentarea, $componentid]);
+    public static function get_cost(string $component, string $paymentarea, int $itemid): array {
+        $cost = component_class_callback("$component\\payment\\provider", 'get_cost', [$paymentarea, $itemid]);
 
         if ($cost === null || !is_array($cost) || !array_key_exists('amount', $cost)
                 || !array_key_exists('currency', $cost) || !array_key_exists('accountid', $cost) ) {
@@ -191,14 +191,14 @@ class helper {
      *
      * @param string $component
      * @param string $paymentarea
-     * @param int $componentid
+     * @param int $itemid
      * @param string $gatewayname
      * @return array
      * @throws \moodle_exception
      */
-    public static function get_gateway_configuration(string $component, string $paymentarea, int $componentid,
+    public static function get_gateway_configuration(string $component, string $paymentarea, int $itemid,
             string $gatewayname): array {
-        $x = self::get_cost($component, $paymentarea, $componentid);
+        $x = self::get_cost($component, $paymentarea, $itemid);
         $gateway = null;
         $account = new account($x['accountid']);
         if ($account && $account->get('enabled')) {
@@ -215,15 +215,15 @@ class helper {
      *
      * @uses \core_payment\local\callback\provider::deliver_order()
      *
-     * @param string $component Name of the component that the componentid belongs to
+     * @param string $component Name of the component that the itemid belongs to
      * @param string $paymentarea
-     * @param int $componentid An internal identifier that is used by the component
+     * @param int $itemid An internal identifier that is used by the component
      * @param int $paymentid payment id as inserted into the 'payments' table, if needed for reference
      * @return bool Whether successful or not
      */
-    public static function deliver_order(string $component, string $paymentarea, int $componentid, int $paymentid): bool {
+    public static function deliver_order(string $component, string $paymentarea, int $itemid, int $paymentid): bool {
         $result = component_class_callback("$component\\payment\\provider", 'deliver_order',
-                [$paymentarea, $componentid, $paymentid]);
+                [$paymentarea, $itemid, $paymentid]);
 
         if ($result === null) {
             throw new \moodle_exception('callbacknotimplemented', 'core_payment', '', $component);
@@ -237,23 +237,23 @@ class helper {
      * Each payment gateway may then store the additional information their way.
      *
      * @param int $accountid Account id
-     * @param string $component Name of the component that the componentid belongs to
+     * @param string $component Name of the component that the itemid belongs to
      * @param string $paymentarea
-     * @param int $componentid An internal identifier that is used by the component
+     * @param int $itemid An internal identifier that is used by the component
      * @param int $userid Id of the user who is paying
      * @param float $amount Amount of payment
      * @param string $currency Currency of payment
      * @param string $gateway The gateway that is used for the payment
      * @return int
      */
-    public static function save_payment(int $accountid, string $component, string $paymentarea, int $componentid, int $userid,
+    public static function save_payment(int $accountid, string $component, string $paymentarea, int $itemid, int $userid,
             float $amount, string $currency, string $gateway): int {
         global $DB;
 
         $record = new \stdClass();
         $record->component = $component;
         $record->paymentarea = $paymentarea;
-        $record->componentid = $componentid;
+        $record->itemid = $itemid;
         $record->userid = $userid;
         $record->amount = $amount;
         $record->currency = $currency;
index e4c3b5a..8b6fa60 100644 (file)
@@ -38,16 +38,16 @@ interface provider {
 
     /**
      * @param string $paymentarea
-     * @param int $componentid An identifier that is known to the plugin
+     * @param int $itemid An identifier that is known to the plugin
      * @return array['amount' => float, 'currency' => string, 'accountid' => int]
      */
-    public static function get_cost(string $paymentarea, int $componentid): array;
+    public static function get_cost(string $paymentarea, int $itemid): array;
 
     /**
      * @param string $paymentarea
-     * @param int $componentid An identifier that is known to the plugin
+     * @param int $itemid An identifier that is known to the plugin
      * @param int $paymentid payment id as inserted into the 'payments' table, if needed for reference
      * @return bool Whether successful or not
      */
-    public static function deliver_order(string $paymentarea, int $componentid, int $paymentid): bool;
+    public static function deliver_order(string $paymentarea, int $itemid, int $paymentid): bool;
 }
index 08f064c..8faa42b 100644 (file)
Binary files a/payment/gateway/paypal/amd/build/gateways_modal.min.js and b/payment/gateway/paypal/amd/build/gateways_modal.min.js differ
index 343c7df..e3a4b5e 100644 (file)
Binary files a/payment/gateway/paypal/amd/build/gateways_modal.min.js.map and b/payment/gateway/paypal/amd/build/gateways_modal.min.js.map differ
index af79a9e..83d0ead 100644 (file)
Binary files a/payment/gateway/paypal/amd/build/repository.min.js and b/payment/gateway/paypal/amd/build/repository.min.js differ
index af94441..9f0f383 100644 (file)
Binary files a/payment/gateway/paypal/amd/build/repository.min.js.map and b/payment/gateway/paypal/amd/build/repository.min.js.map differ
index 59f4bb1..4d68227 100644 (file)
@@ -45,21 +45,21 @@ const showModalWithPlaceholder = async() => {
 /**
  * Process the payment.
  *
- * @param {string} component Name of the component that the componentId belongs to
- * @param {string} paymentArea The area of the component that the componentId belongs to
- * @param {number} componentId An internal identifier that is used by the component
+ * @param {string} component Name of the component that the itemId belongs to
+ * @param {string} paymentArea The area of the component that the itemId belongs to
+ * @param {number} itemId An internal identifier that is used by the component
  * @param {string} description Description of the payment
  * @param {processCallback} callback The callback function to call when processing is finished
  * @returns {Promise<void>}
  */
-export const process = async(component, paymentArea, componentId, description, callback) => {
+export const process = async(component, paymentArea, itemId, description, callback) => {
 
     const [
         modal,
         paypalConfig,
     ] = await Promise.all([
         showModalWithPlaceholder(),
-        Repository.getConfigForJs(component, paymentArea, componentId),
+        Repository.getConfigForJs(component, paymentArea, itemId),
     ]);
     const currency = paypalConfig.currency;
     const amount = paypalConfig.cost; // Cost with surcharge.
@@ -106,7 +106,7 @@ export const process = async(component, paymentArea, componentId, description, c
                     args: {
                         component,
                         paymentarea: paymentArea,
-                        componentid: componentId,
+                        itemid: itemId,
                         orderid: data.orderID,
                     },
                 }])[0]
index 627c95d..627753e 100644 (file)
@@ -27,18 +27,18 @@ import Ajax from 'core/ajax';
 /**
  * Return the PayPal JavaScript SDK URL.
  *
- * @param {string} component Name of the component that the componentid belongs to
- * @param {string} paymentArea The area of the component that the componentid belongs to
- * @param {number} componentId An internal identifier that is used by the component
+ * @param {string} component Name of the component that the itemId belongs to
+ * @param {string} paymentArea The area of the component that the itemId belongs to
+ * @param {number} itemId An internal identifier that is used by the component
  * @returns {Promise<{clientid: string, brandname: string}>}
  */
-export const getConfigForJs = (component, paymentArea, componentId) => {
+export const getConfigForJs = (component, paymentArea, itemId) => {
     const request = {
         methodname: 'paygw_paypal_get_config_for_js',
         args: {
             component,
             paymentarea: paymentArea,
-            componentid: componentId,
+            itemid: itemId,
         },
     };
 
index 0cd110f..5850dcb 100644 (file)
@@ -47,7 +47,7 @@ class get_config_for_js extends external_api {
         return new external_function_parameters([
             'component' => new external_value(PARAM_COMPONENT, 'Component'),
             'paymentarea' => new external_value(PARAM_AREA, 'Payment area in the component'),
-            'componentid' => new external_value(PARAM_INT, 'An identifier for payment area in the component'),
+            'itemid' => new external_value(PARAM_INT, 'An identifier for payment area in the component'),
         ]);
     }
 
@@ -56,18 +56,18 @@ class get_config_for_js extends external_api {
      *
      * @param string $component
      * @param string $paymentarea
-     * @param int $componentid
+     * @param int $itemid
      * @return string[]
      */
-    public static function execute(string $component, string $paymentarea, int $componentid): array {
+    public static function execute(string $component, string $paymentarea, int $itemid): array {
         self::validate_parameters(self::execute_parameters(), [
             'component' => $component,
             'paymentarea' => $paymentarea,
-            'componentid' => $componentid,
+            'itemid' => $itemid,
         ]);
 
-        $config = helper::get_gateway_configuration($component, $paymentarea, $componentid, 'paypal');
-        $cost = helper::get_cost($component, $paymentarea, $componentid);
+        $config = helper::get_gateway_configuration($component, $paymentarea, $itemid, 'paypal');
+        $cost = helper::get_cost($component, $paymentarea, $itemid);
         $surcharge = helper::get_gateway_surcharge('paypal');
 
         return [
index 251272d..dba305e 100644 (file)
@@ -48,7 +48,7 @@ class transaction_complete extends external_api {
         return new external_function_parameters([
             'component' => new external_value(PARAM_COMPONENT, 'The component name'),
             'paymentarea' => new external_value(PARAM_AREA, 'Payment area in the component'),
-            'componentid' => new external_value(PARAM_INT, 'The item id in the context of the component area'),
+            'itemid' => new external_value(PARAM_INT, 'The item id in the context of the component area'),
             'orderid' => new external_value(PARAM_TEXT, 'The order id coming back from PayPal'),
         ]);
     }
@@ -57,30 +57,30 @@ class transaction_complete extends external_api {
      * Perform what needs to be done when a transaction is reported to be complete.
      * This function does not take cost as a parameter as we cannot rely on any provided value.
      *
-     * @param string $component Name of the component that the componentid belongs to
+     * @param string $component Name of the component that the itemid belongs to
      * @param string $paymentarea
-     * @param int $componentid An internal identifier that is used by the component
+     * @param int $itemid An internal identifier that is used by the component
      * @param string $orderid PayPal order ID
      * @return array
      */
-    public static function execute(string $component, string $paymentarea, int $componentid, string $orderid): array {
+    public static function execute(string $component, string $paymentarea, int $itemid, string $orderid): array {
         global $USER, $DB;
 
         self::validate_parameters(self::execute_parameters(), [
             'component' => $component,
             'paymentarea' => $paymentarea,
-            'componentid' => $componentid,
+            'itemid' => $itemid,
             'orderid' => $orderid,
         ]);
 
-        $config = (object)helper::get_gateway_configuration($component, $paymentarea, $componentid, 'paypal');
+        $config = (object)helper::get_gateway_configuration($component, $paymentarea, $itemid, 'paypal');
         $sandbox = $config->environment == 'sandbox';
 
         [
             'amount' => $amount,
             'currency' => $currency,
             'accountid' => $accountid,
-        ] = payment_helper::get_cost($component, $paymentarea, $componentid);
+        ] = payment_helper::get_cost($component, $paymentarea, $itemid);
 
         // Add surcharge if there is any.
         $surcharge = helper::get_gateway_surcharge('paypal');
@@ -102,7 +102,7 @@ class transaction_complete extends external_api {
                         $success = true;
                         // Everything is correct. Let's give them what they paid for.
                         try {
-                            $paymentid = payment_helper::save_payment((int) $accountid, $component, $paymentarea, $componentid,
+                            $paymentid = payment_helper::save_payment((int) $accountid, $component, $paymentarea, $itemid,
                                 (int) $USER->id, $amount, $currency, 'paypal');
 
                             // Store PayPal extra information.
@@ -112,7 +112,7 @@ class transaction_complete extends external_api {
 
                             $DB->insert_record('paygw_paypal', $record);
 
-                            payment_helper::deliver_order($component, $paymentarea, $componentid, $paymentid);
+                            payment_helper::deliver_order($component, $paymentarea, $itemid, $paymentid);
                         } catch (\Exception $e) {
                             debugging('Exception while trying to process payment: ' . $e->getMessage(), DEBUG_DEVELOPER);
                             $success = false;
index 48f4649..8dd67e4 100644 (file)
@@ -73,7 +73,7 @@ class core_payment_generator extends component_generator_base {
             [
                 'component' => 'testcomponent',
                 'paymentarea' => 'teatarea',
-                'componentid' => 0,
+                'itemid' => 0,
                 'currency' => 'AUD',
             ]);
         return $id;
index 4177650..e1c5e29 100644 (file)
@@ -121,7 +121,7 @@ class accounts_testcase extends advanced_testcase {
             'accountid' => $account->get('id'),
             'component' => 'test',
             'paymentarea' => 'test',
-            'componentid' => 1,
+            'itemid' => 1,
             'userid' => $USER->id,
         ]);
         helper::delete_payment_account(account::get_record(['id' => $account->get('id')]));