| user | course | role |
| teacher1 | C1 | editingteacher |
| manager1 | C1 | manager |
- And payment plugin "paypal" is enabled
And the following "core_payment > payment accounts" exist:
| name | gateways |
| Account1 | paypal |
And I select "Fee" from the "Add method" singleselect
And I set the following fields to these values:
| Payment account | Account1 |
- | Enrol cost | 10 |
+ | Enrolment fee | 123.45 |
| Currency | Euro |
And I press "Add method"
And I log out
And I log in as "student1"
And I am on course index
And I follow "Course 1"
- And I should see "This course requires a payment for entry."
- #And I should see "Cost: EUR 10.00" # TODO for some reason behat does not "see" this text.
- And I press "Pay enrolment fee"
- And I should see "PayPal" in the "Select Payment Type" "dialogue"
- And I click on "Cancel" "button" in the "Select Payment Type" "dialogue"
- And I log out
+ Then I should see "This course requires a payment for entry."
+ And I should see "123.45"
+ And I press "Select payment type"
+ And I should see "PayPal" in the "Select payment type" "dialogue"
+ And I click on "Cancel" "button" in the "Select payment type" "dialogue"
*/
$string['accountarchived'] = 'Archived';
-$string['accountdeleteconfirm'] = 'If this account has previous payments, they will be archived, otherwise all other data will be permanently deleted. Are you sure you want to continue?';
+$string['accountdeleteconfirm'] = 'If this account has previous payments, it will be archived, otherwise its configuration data will be permanently deleted. Are you sure you want to continue?';
+$string['accountconfignote'] = 'Payment gateways for this account will be configured separately';
+$string['accountidnumber'] = 'ID number';
+$string['accountidnumber_help'] = 'The ID number is only used when matching the account against external systems and is not displayed anywhere on the site. If the account has an official code name it may be entered, otherwise the field can be left blank.';
$string['accountname'] = 'Account name';
+$string['accountname_help'] = 'How this account will be identified for teachers or managers who set up payments (for example in the course enrolment plugin)';
$string['accountnotavailable'] = 'Not available';
+$string['paymentaccountsexplained'] = 'Create one or multiple payment accounts for this site. Each account includes configuration for available payment gateways. The person who configures payments on the site (for example, payment for the course enrolment) will be able to chose from the available accounts.';
$string['callbacknotimplemented'] = 'The callback is not implemented for component {$a}.';
$string['createaccount'] = 'Create payment account';
$string['deleteorarchive'] = 'Delete or archive';
$accounts = \core_payment\helper::get_payment_accounts_to_manage(context_system::instance(), $showarchived);
$table = new html_table();
-$table->head = [get_string('accountname', 'payment'), get_string('type_pg', 'plugin'), ''];
+$table->head = [get_string('accountname', 'payment'), get_string('type_pg_plural', 'plugin'), ''];
$table->colclasses = ['', '', 'mdl-right'];
$table->data = [];
foreach ($accounts as $account) {
$table->data[] = [$name, join(', ', $gateways), $OUTPUT->render($menu)];
}
+echo html_writer::div(get_string('paymentaccountsexplained', 'payment'), 'pb-2');
+
+if (has_capability('moodle/site:config', context_system::instance())) {
+ // For administrators add a link to "Manage payment gateways" page.
+ $link = html_writer::link(new moodle_url('/admin/settings.php', ['section' => 'managepaymentgateways']),
+ get_string('type_pgmanage', 'plugin'));
+ $text = get_string('gotomanageplugins', 'payment', $link);
+ echo html_writer::div($text, 'pb-2');
+}
+
echo html_writer::table($table);
$PAGE->requires->event_handler('[data-action=delete]', 'click', 'M.util.show_confirm_dialog',
echo $OUTPUT->single_button(new moodle_url('/payment/manage_account.php'), get_string('createaccount', 'payment'), 'get');
-if (has_capability('moodle/site:config', context_system::instance())) {
- // For administrators add a link to "Manage payment gateways" page.
- $link = html_writer::link(new moodle_url('/admin/settings.php', ['section' => 'managepaymentgateways']),
- get_string('type_pgmanage', 'plugin'));
- $text = get_string('gotomanageplugins', 'payment', $link);
- echo html_writer::div($text, 'pt-3');
-}
-
echo $OUTPUT->footer();
use core\persistent;
-defined('MOODLE_INTERNAL') || die();
-
/**
* Class account_gateway
*
*
* @return array
*/
- protected static function define_properties() : array {
+ protected static function define_properties(): array {
return array(
'accountid' => [
'type' => PARAM_INT,
],
'gateway' => [
'type' => PARAM_COMPONENT,
- // TODO select with options?
],
'enabled' => [
'type' => PARAM_BOOL,
* Return the gateway name ready for display
*
* @return string
- * @throws \coding_exception
*/
public function get_display_name(): string {
return get_string('pluginname', 'pg_' . $this->get('gateway'));
* Gateway management url
*
* @return \moodle_url
- * @throws \coding_exception
- * @throws \moodle_exception
*/
public function get_edit_url(): \moodle_url {
$params = $this->get('id') ? ['id' => $this->get('id')] :
* Get corresponding account
*
* @return account
- * @throws \coding_exception
*/
public function get_account(): account {
return new account($this->get('accountid'));
* Parse configuration from the json-encoded stored value
*
* @return array
- * @throws \coding_exception
*/
public function get_configuration(): array {
$config = @json_decode($this->get('config'), true);
use core\form\persistent;
-defined('MOODLE_INTERNAL') || die();
-
/**
* Class account
*
$mform->addElement('hidden', 'contextid');
$mform->addElement('text', 'name', get_string('accountname', 'payment'), 'maxlength="255"');
+ $mform->addHelpButton('name', 'accountname', 'payment');
$mform->setType('name', PARAM_TEXT);
$mform->addRule('name', get_string('required'), 'required', null, 'client');
$mform->addRule('name', get_string('maximumchars', '', 255), 'maxlength', 255, 'server');
- $mform->addElement('text', 'idnumber', get_string('idnumber'), 'maxlength="100"');
+ $mform->addElement('text', 'idnumber', get_string('accountidnumber', 'payment'), 'maxlength="100"');
+ $mform->addHelpButton('idnumber', 'accountidnumber', 'payment');
$mform->setType('idnumber', PARAM_RAW_TRIMMED);
$mform->addRule('idnumber', get_string('maximumchars', '', 100), 'maxlength', 100, 'server');
+ $mform->addElement('static', 'staticinfo', '', get_string('accountconfignote', 'payment'));
+
$mform->addElement('advcheckbox', 'enabled', get_string('enable'));
$this->add_action_buttons();
}
namespace core_payment\form;
use core\form\persistent;
-use core_payment\gateway;
-
-defined('MOODLE_INTERNAL') || die();
/**
* Class account_gateway
namespace core_payment;
-defined('MOODLE_INTERNAL') || die();
-
/**
* Base class for payment gateways.
*
use core_payment\event\account_deleted;
use core_payment\event\account_updated;
-defined('MOODLE_INTERNAL') || die();
-
/**
* Helper class for the payment subsystem.
*
--- /dev/null
+<?php
+// This file is part of Moodle - http://moodle.org/
+//
+// Moodle is free software: you can redistribute it and/or modify
+// it under the terms of the GNU General Public License as published by
+// the Free Software Foundation, either version 3 of the License, or
+// (at your option) any later version.
+//
+// Moodle is distributed in the hope that it will be useful,
+// but WITHOUT ANY WARRANTY; without even the implied warranty of
+// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+// GNU General Public License for more details.
+//
+// You should have received a copy of the GNU General Public License
+// along with Moodle. If not, see <http://www.gnu.org/licenses/>.
+
+/**
+ * pg_paypal installer script.
+ *
+ * @package pg_paypal
+ * @copyright 2020 Marina Glancy
+ * @license http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
+ */
+
+function xmldb_pg_paypal_install() {
+ // Enable the Paypal payment gateway on installation. It still needs to be configured and enabled for accounts.
+ $order = (!empty($CFG->pg_plugins_sortorder)) ? explode(',', $CFG->pg_plugins_sortorder) : [];
+ set_config('pg_plugins_sortorder', join(',', array_merge($order, ['paypal'])));
+}
When I log in as "admin"
And I navigate to "Payments > Payment accounts" in site administration
And I follow "Manage payment gateways"
- And I click on "Enable" "link" in the "PayPal" "table_row"
+ Then "Australian Dollar" "text" should exist in the "PayPal" "table_row"
And I follow "Payment accounts"
And I press "Create payment account"
And I set the field "Account name" to "TestAccount"
@javascript
Scenario: Configuring gateways on payment accounts
- Given payment plugin "paypal" is enabled
- And the following "core_payment > payment accounts" exist:
+ Given the following "core_payment > payment accounts" exist:
| name |
| Account1 |
| Account2 |
@javascript
Scenario: Deleting payment accounts
- Given payment plugin "paypal" is enabled
- And the following "core_payment > payment accounts" exist:
+ Given the following "core_payment > payment accounts" exist:
| name |
| Account1 |
| Account2 |
@javascript
Scenario: Archiving and restoring accounts
- Given payment plugin "paypal" is enabled
- And the following "users" exist:
+ Given the following "users" exist:
| username |
| user1 |
And the following "core_payment > payment accounts" exist:
+++ /dev/null
-<?php
-// This file is part of Moodle - http://moodle.org/
-//
-// Moodle is free software: you can redistribute it and/or modify
-// it under the terms of the GNU General Public License as published by
-// the Free Software Foundation, either version 3 of the License, or
-// (at your option) any later version.
-//
-// Moodle is distributed in the hope that it will be useful,
-// but WITHOUT ANY WARRANTY; without even the implied warranty of
-// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
-// GNU General Public License for more details.
-//
-// You should have received a copy of the GNU General Public License
-// along with Moodle. If not, see <http://www.gnu.org/licenses/>.
-
-/**
- * Steps definitions related with the payment API
- *
- * @package core_payment
- * @category test
- * @copyright 2020 Marina Glancy
- * @license http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
- */
-
-// NOTE: no MOODLE_INTERNAL test here, this file may be required by behat before including /config.php.
-
-use Behat\Gherkin\Node\TableNode as TableNode,
- Behat\Mink\Exception\ExpectationException as ExpectationException,
- Behat\Mink\Exception\ElementNotFoundException as ElementNotFoundException;
-
-/**
- * Steps definitions related with the payment API
- *
- * @package core_payment
- * @category test
- * @copyright 2020 Marina Glancy
- * @license http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
- */
-class behat_payment extends behat_question_base {
-
- /**
- * Enable payment plugin
- *
- * @Given /^payment plugin "(?P<plugin_name_string>(?:[^"]|\\")*)" is enabled$/
- * @param string $pluginname
- */
- public function payment_plugin_is_enabled($pluginname) {
- $plugins = \core\plugininfo\pg::get_enabled_plugins();
- \core\plugininfo\pg::set_enabled_plugins(array_merge($plugins, [$pluginname]));
- }
-}
* @license http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
*/
-defined('MOODLE_INTERNAL') || die();
-
/**
* Behat data generator for core_payment.
*/
class behat_core_payment_generator extends behat_generator_base {
+ /**
+ * Get a list of the entities that can be created.
+ *
+ * @return array entity name => information about how to generate.
+ */
protected function get_creatable_entities(): array {
return [
'payment accounts' => [
// You should have received a copy of the GNU General Public License
// along with Moodle. If not, see <http://www.gnu.org/licenses/>.
-defined('MOODLE_INTERNAL') || die();
-
/**
* Quiz module test data generator class
*
*/
class core_payment_generator extends component_generator_base {
+ /** @var int */
protected $accountcounter = 0;
/**
* @param array $data account data (name, idnumber, enabled) and additionally field 'gateways' that can include
* a list of gateways that should be mock-enabled for this account.
*/
- public function create_payment_account(array $data): \core_payment\account {
+ public function create_payment_account(array $data = []): \core_payment\account {
$this->accountcounter++;
$gateways = [];
if (!empty($data['gateways'])) {
$data['gateway'] = reset($gateways);
}
- $id = $DB->insert_record('payments', $data + [
- 'component' => 'testcomponent',
- 'paymentarea' => 'teatarea',
- 'componentid' => 0,
- 'currency' => 'AUD'
- ]);
+ $id = $DB->insert_record('payments', $data +
+ [
+ 'component' => 'testcomponent',
+ 'paymentarea' => 'teatarea',
+ 'componentid' => 0,
+ 'currency' => 'AUD',
+ ]);
return $id;
}
--- /dev/null
+<?php
+// This file is part of Moodle - http://moodle.org/
+//
+// Moodle is free software: you can redistribute it and/or modify
+// it under the terms of the GNU General Public License as published by
+// the Free Software Foundation, either version 3 of the License, or
+// (at your option) any later version.
+//
+// Moodle is distributed in the hope that it will be useful,
+// but WITHOUT ANY WARRANTY; without even the implied warranty of
+// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+// GNU General Public License for more details.
+//
+// You should have received a copy of the GNU General Public License
+// along with Moodle. If not, see <http://www.gnu.org/licenses/>.
+
+/**
+ * Testing generator in payments API
+ *
+ * @package core_payment
+ * @category test
+ * @copyright 2020 Marina Glancy
+ * @license http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
+ */
+
+namespace core_payment;
+
+/**
+ * Testing generator in payments API
+ *
+ * @package core_payment
+ * @category test
+ * @copyright 2020 Marina Glancy
+ * @license http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
+ */
+class helper_testcase extends \advanced_testcase {
+
+ public function test_create_account() {
+ global $DB;
+ $this->resetAfterTest();
+ /** @var \core_payment_generator $generator */
+ $generator = $this->getDataGenerator()->get_plugin_generator('core_payment');
+ $this->assertTrue($generator instanceof \core_payment_generator);
+
+ $account1 = $generator->create_payment_account();
+ $account2 = $generator->create_payment_account(['name' => 'My name', 'gateways' => 'paypal']);
+
+ $record1 = $DB->get_record('payment_accounts', ['id' => $account1->get('id')]);
+ $record2 = $DB->get_record('payment_accounts', ['id' => $account2->get('id')]);
+
+ $this->assertEquals(1, $record1->enabled);
+ $this->assertEquals('My name', $record2->name);
+
+ // First account does not have gateways configurations.
+ $this->assertEmpty($DB->get_records('payment_gateways', ['accountid' => $account1->get('id')]));
+ // Second account has.
+ $this->assertCount(1, $DB->get_records('payment_gateways', ['accountid' => $account2->get('id')]));
+ }
+
+ public function test_create_payment() {
+ global $DB;
+ $this->resetAfterTest();
+ /** @var \core_payment_generator $generator */
+ $generator = $this->getDataGenerator()->get_plugin_generator('core_payment');
+ $account = $generator->create_payment_account(['gateways' => 'paypal']);
+ $user = $this->getDataGenerator()->create_user();
+
+ $paymentid = $generator->create_payment(['accountid' => $account->get('id'), 'amount' => 10, 'userid' => $user->id]);
+
+ $this->assertEquals('testcomponent', $DB->get_field('payments', 'component', ['id' => $paymentid]));
+ }
+}
// along with Moodle. If not, see <http://www.gnu.org/licenses/>.
/**
- * Testing accounts management in payments API
+ * Testing helper class methods in payments API
*
* @package core_payment
* @category test
use core\plugininfo\pg;
/**
- * Testing accounts management in payments API
+ * Testing helper class methods in payments API
*
* @package core_payment
* @category test
if (!array_key_exists('paypal', \core_component::get_plugin_list('pg'))) {
return false;
}
- pg::set_enabled_plugins('paypal');
return true;
}
// Delete account with payments - it will be archived.
$this->setAdminUser();
$account = helper::save_payment_account((object)['name' => 'Test 1', 'idnumber' => '']);
- $DB->insert_record('payments', ['accountid' => $account->get('id'), 'component' => 'test', 'paymentarea' => 'test',
- 'componentid' => 1, 'userid' => $USER->id]);
+ $DB->insert_record('payments', [
+ 'accountid' => $account->get('id'),
+ 'component' => 'test',
+ 'paymentarea' => 'test',
+ 'componentid' => 1,
+ 'userid' => $USER->id,
+ ]);
helper::delete_payment_account(account::get_record(['id' => $account->get('id')]));
$this->assertEquals(1, $DB->get_field('payment_accounts', 'archived', ['id' => $account->get('id')]));
$this->assertEquals(0, $DB->get_field('payment_accounts', 'archived', ['id' => $account->get('id')]));
}
- public function test_format_cost() {
- $amount = 5.345;
+ /**
+ * Provider for format_cost test
+ *
+ * @return array
+ */
+ public function get_rounded_cost_provider(): array {
+ return [
+ 'IRR 0 surcharge' => [5.345, 'IRR', 0, 5],
+ 'IRR 12% surcharge' => [5.345, 'IRR', 12, 6],
+ 'USD 0 surcharge' => [5.345, 'USD', 0, 5.34],
+ 'USD 1% surcharge' => [5.345, 'USD', 1, 5.4],
+ ];
+ }
+
+ /**
+ * Provier for test_get_cost_as_string
+ *
+ * @return array[]
+ */
+ public function get_cost_as_string_provider(): array {
+ return [
+ 'IRR 0 surcharge' => [5.345, 'IRR', 0, 'IRR'."\xc2\xa0".'5'],
+ 'IRR 12% surcharge' => [5.345, 'IRR', 12, 'IRR'."\xc2\xa0".'6'],
+ 'USD 0 surcharge' => [5.345, 'USD', 0, 'USD'."\xc2\xa0".'5.34'],
+ 'USD 1% surcharge' => [5.345, 'USD', 1, 'USD'."\xc2\xa0".'5.40'],
+ ];
+ }
- $currency = 'IRR';
- $this->assertEquals(5, helper::get_rounded_cost($amount, $currency));
- $this->assertEquals('IRR'."\xc2\xa0".'5', helper::get_cost_as_string($amount, $currency));
- $this->assertEquals(6, helper::get_rounded_cost($amount, $currency, 12));
- $this->assertEquals('IRR'."\xc2\xa0".'6', helper::get_cost_as_string($amount, $currency, 12));
+ /**
+ * Test for test_format_cost function
+ *
+ * @dataProvider get_rounded_cost_provider
+ * @param float $amount
+ * @param string $currency
+ * @param float $surcharge
+ * @param string $expected
+ */
+ public function test_get_rounded_cost(float $amount, string $currency, float $surcharge, string $expected) {
+ $this->assertEquals($expected, helper::get_rounded_cost($amount, $currency, $surcharge));
+ }
- $currency = 'USD';
- $this->assertEquals(5.34, helper::get_rounded_cost($amount, $currency));
- $this->assertEquals('USD'."\xc2\xa0".'5.34', helper::get_cost_as_string($amount, $currency));
- $this->assertEquals(5.40, helper::get_rounded_cost($amount, $currency, 1));
- $this->assertEquals('USD'."\xc2\xa0".'5.40', helper::get_cost_as_string($amount, $currency, 1));
+ /**
+ * Test for get_cost_as_string function
+ *
+ * @dataProvider get_cost_as_string_provider
+ * @param float $amount
+ * @param string $currency
+ * @param float $surcharge
+ * @param string $expected
+ */
+ public function test_get_cost_as_string(float $amount, string $currency, float $surcharge, string $expected) {
+ $this->assertEquals($expected, helper::get_cost_as_string($amount, $currency, $surcharge));
}
}
