Details
-
Type:
Bug
-
Status:
Closed
-
Priority:
Critical
-
Resolution: Fixed
-
Affects Version/s: 2.0.3, 2.1
-
Fix Version/s: 2.0.4
-
Component/s: Web Services
-
Labels:
-
Database:Any
-
Testing Instructions:Hide
TESTCASE 1:
Run testmoodlelib.php simpletest.Instructions to run testmoodlelib.php simpletest
1. Navigate to unit test (Development -> unit test)
2. enter "lib/simpletest/testmoodlelib.php" in textbox.
3. click "Run test"TESTCASE 2:
1. Enable Webservices
2. Run unit test (Site adminstration -> Development -> unit tests) "webservice/simpletest/testwebservice.php".How to Enable webservices
1. Enable web services (Site adminstration -> Advanced features)
2. Enable protocols (Site adminstration -> Plugins -> Web Services -> Manage Protocols ). Enable Rest &/ XMLRPC
3. Add External services ( Site adminstration -> Plugins -> Web Services -> External Services). Add function "moodle_user_create_users" to this external service
4. Manage Tokens ( Site adminstration -> Plugins -> Web Services -> Manage Tokens). Add the "External Service" which we created in step 3 and copy the token value.
5. Open webservice/simpletest/testwebservice.php and replace $this->testtoken value with above token value.
6. set " $this->testxmlrpc" or "$this->testrest" to true
7. set moodle_user_create_users to true.
8. Now you are ready to run simple test
ShowTESTCASE 1: Run testmoodlelib.php simpletest. Instructions to run testmoodlelib.php simpletest 1. Navigate to unit test (Development -> unit test) 2. enter "lib/simpletest/testmoodlelib.php" in textbox. 3. click "Run test" TESTCASE 2: 1. Enable Webservices 2. Run unit test (Site adminstration -> Development -> unit tests) "webservice/simpletest/testwebservice.php". How to Enable webservices 1. Enable web services (Site adminstration -> Advanced features) 2. Enable protocols (Site adminstration -> Plugins -> Web Services -> Manage Protocols ). Enable Rest &/ XMLRPC 3. Add External services ( Site adminstration -> Plugins -> Web Services -> External Services). Add function "moodle_user_create_users" to this external service 4. Manage Tokens ( Site adminstration -> Plugins -> Web Services -> Manage Tokens). Add the "External Service" which we created in step 3 and copy the token value. 5. Open webservice/simpletest/testwebservice.php and replace $this->testtoken value with above token value. 6. set " $this->testxmlrpc" or "$this->testrest" to true 7. set moodle_user_create_users to true. 8. Now you are ready to run simple test -
Affected Branches:MOODLE_20_STABLE, MOODLE_21_STABLE
-
Fixed Branches:MOODLE_20_STABLE
-
Pull from Repository:
-
Pull Master Branch:wip-mdl-27759-integration-master
-
Pull Master Diff URL:
Description
In create_user webservice function (/user/externallib.php), throw error for string timezones.
This is happening because parameter check for timezone is PARAM_ALPHANUMEXT and it doesn't allow '/' (which string timezone format).
Currently we don't have any PARAM defined for timzones, so probably we can define a new PARAM for timezone which should check for string to be either float or string with max one '/'
Activity
Ho, I'm keeping this under review to get these fixed:
1) right now it (master) does not merge clean. I guess other recent changes @ user/externallib.php are causing conflict.
2) In unit tests it's better to avoid using the $message param in common asertXXX() methods. Only if some extra information needs to be revealed they should be used. Else it's better to keep standard output to happen.
3) I think valid timezones like 'PST8PDT' are already supported by the regexp. I just would add one extra test for it.
Ciao
Could this also come in as a raw param and be processed by the script that uses it? That would depend on how often we need to parse a timezone parameter I suppose.
Michael;