From fd8ad938688cee107027ab084c5093470c701e68 Mon Sep 17 00:00:00 2001 From: Petr Skoda Date: Tue, 10 Nov 2009 09:34:40 +0000 Subject: [PATCH] MDL-12886 fixed TODOs --- webservice/lib.php | 4 ++-- webservice/testclient.php | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/webservice/lib.php b/webservice/lib.php index a0d6e2323d..23fe69f62c 100644 --- a/webservice/lib.php +++ b/webservice/lib.php @@ -148,7 +148,7 @@ abstract class webservice_server implements webservice_server_interface { } if (!$auth->user_login_webservice($this->username, $this->password)) { - // TODO: log failed login attempts + // TODO: MDL-12886 log failed login attempts throw new webservice_access_exception('Wrong username or password'); } @@ -156,7 +156,7 @@ abstract class webservice_server implements webservice_server_interface { } else { if (!$token = $DB->get_record('external_tokens', array('token'=>$this->token, 'tokentype'=>EXTERNAL_TOKEN_PERMANENT))) { - // TODO: log failed login attempts + // TODO: MDL-12886 log failed login attempts throw new webservice_access_exception('Invalid token'); } diff --git a/webservice/testclient.php b/webservice/testclient.php index ee5269b651..e652e1e047 100644 --- a/webservice/testclient.php +++ b/webservice/testclient.php @@ -34,7 +34,7 @@ $protocol = optional_param('protocol', '', PARAM_SAFEDIR); $PAGE->set_url('webservice/testclient.php'); require_login(); -require_capability('moodle/site:config', get_context_instance(CONTEXT_SYSTEM)); // TODO: do we need some new capability? +require_capability('moodle/site:config', get_context_instance(CONTEXT_SYSTEM)); // list of all available functions for testing $allfunctions = $DB->get_records('external_functions', array(), 'name ASC'); -- 2.39.5