Skip to content

Commit d0e5a9a

Browse files
committed
Remove the externallib dependency
1 parent 9c9266a commit d0e5a9a

File tree

4 files changed

+10
-8
lines changed

4 files changed

+10
-8
lines changed

auth.php

+1-1
Original file line numberDiff line numberDiff line change
@@ -26,8 +26,8 @@
2626

2727
use auth_userkey\core_userkey_manager;
2828
use auth_userkey\userkey_manager_interface;
29+
use core_external\external_value;
2930

30-
require_once($CFG->libdir . "/externallib.php");
3131
require_once($CFG->libdir.'/authlib.php');
3232
require_once($CFG->dirroot . '/user/lib.php');
3333

externallib.php

+5-1
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,11 @@
2424

2525
defined('MOODLE_INTERNAL') || die();
2626

27-
require_once($CFG->libdir . "/externallib.php");
27+
use core_external\external_value;
28+
use core_external\external_api;
29+
use core_external\external_function_parameters;
30+
use core_external\external_single_structure;
31+
2832
require_once($CFG->dirroot . "/webservice/lib.php");
2933
require_once($CFG->dirroot . "/auth/userkey/auth.php");
3034

tests/auth_plugin_test.php

+1-2
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@
2121
use stdClass;
2222
use invalid_parameter_exception;
2323
use moodle_exception;
24-
use external_value;
24+
use core_external\external_value;
2525

2626
/**
2727
* Tests for auth_plugin_userkey class.
@@ -57,7 +57,6 @@ class auth_plugin_test extends advanced_testcase {
5757
public function setUp(): void {
5858
global $CFG;
5959

60-
require_once($CFG->libdir . "/externallib.php");
6160
require_once($CFG->dirroot . '/auth/userkey/tests/fake_userkey_manager.php');
6261
require_once($CFG->dirroot . '/auth/userkey/auth.php');
6362
require_once($CFG->dirroot . '/user/lib.php');

tests/externallib_test.php

+3-4
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@
1919
use advanced_testcase;
2020
use webservice_access_exception;
2121
use auth_userkey_external;
22-
use external_api;
22+
use core_external\external_api;
2323
use invalid_parameter_exception;
2424
use required_capability_exception;
2525
use context_system;
@@ -37,17 +37,16 @@ class externallib_test extends advanced_testcase {
3737
/**
3838
* User object.
3939
*
40-
* @var
40+
* @var stdClass
4141
*/
42-
protected $user = array();
42+
protected $user;
4343

4444
/**
4545
* Initial set up.
4646
*/
4747
public function setUp(): void {
4848
global $CFG;
4949

50-
require_once($CFG->libdir . "/externallib.php");
5150
require_once($CFG->dirroot . '/auth/userkey/externallib.php');
5251

5352
$this->resetAfterTest();

0 commit comments

Comments
 (0)