mirror_php/moodle
1
0
Fork 0
mirror of https://github.com/moodle/moodle.git synced 2025-01-18 05:58:34 +01:00
Code Issues Projects Releases Wiki Activity

MDL-63225 webservice: Return proper debuginfo on WS exceptions

Browse Source 
In order to make developers easy, we should give some clues about the type of the data generating exceptions.
This commit is contained in:
Juan Leyva 2018-08-24 12:58:17 +01:00
parent 81bbf426f6
commit 27190655b0
2 changed files with 23 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
lib/externallib.php
View File

@ -397,8 +397,9 @@ class external_api {
return (bool)$response;
}
}
$responsetype = gettype($response);
$debuginfo = 'Invalid external api response: the value is "' . $response .
'", the server was expecting "' . $description->type . '" type';
'" of PHP type "' . $responsetype . '", the server was expecting "' . $description->type . '" type';
try {
return validate_param($response, $description->type, $description->allownull, $debuginfo);
} catch (invalid_parameter_exception $e) {

21
lib/tests/externallib_test.php
View File

@ -256,6 +256,27 @@ class core_externallib_testcase extends advanced_testcase {
$settings->set_filter($currentfilter);
}
/**
* Test for clean_returnvalue() for testing that returns the PHP type.
*/
public function test_clean_returnvalue_return_php_type() {
$returndesc = new external_single_structure(
array(
'value' => new external_value(PARAM_RAW, 'Some text', VALUE_OPTIONAL, null, NULL_NOT_ALLOWED)
)
);
// Check return type on exception because the external values does not allow NULL values.
$testdata = array('value' => null);
try {
$cleanedvalue = external_api::clean_returnvalue($returndesc, $testdata);
} catch (moodle_exception $e) {
$this->assertInstanceOf('invalid_response_exception', $e);
$this->assertContains('of PHP type "NULL"', $e->debuginfo);
}
}
/**
* Test for clean_returnvalue().
*/