MOBILE-2951 web-services: make it easier to debug web service calls
We add a redundant &wsfunction=web_service_method_name to the URL for web service requests, so that it is easier to find the call you are interested in when debugging using brower tools. Similarly, we add info= to calls to lib/ajax/service.php, matching what is done in Moodle web application JavaScript.main
parent
839fe09fa7
commit
6243eda097
|
@ -234,7 +234,9 @@ export class CoreWSProvider {
|
|||
args: this.convertValuesToString(data)
|
||||
}];
|
||||
|
||||
siteUrl = preSets.siteUrl + '/lib/ajax/service.php';
|
||||
// The info= parameter has no function. It is just to help with debugging.
|
||||
// We call it info to match the parameter name use by Moodle's AMD ajax module.
|
||||
siteUrl = preSets.siteUrl + '/lib/ajax/service.php?info=' + method;
|
||||
|
||||
const promise = this.http.post(siteUrl, JSON.stringify(ajaxData)).timeout(CoreConstants.WS_TIMEOUT).toPromise();
|
||||
|
||||
|
@ -547,8 +549,13 @@ export class CoreWSProvider {
|
|||
options['responseType'] = 'text';
|
||||
}
|
||||
|
||||
// We add the method name to the URL purely to help with debugging.
|
||||
// This duplicates what is in the ajaxData, but that does no harm.
|
||||
// POST variables take precedence over GET.
|
||||
const requestUrl = siteUrl + '&wsfunction=' + method;
|
||||
|
||||
// Perform the post request.
|
||||
const promise = this.http.post(siteUrl, ajaxData, options).timeout(CoreConstants.WS_TIMEOUT).toPromise();
|
||||
const promise = this.http.post(requestUrl, ajaxData, options).timeout(CoreConstants.WS_TIMEOUT).toPromise();
|
||||
|
||||
return promise.then((data: any) => {
|
||||
// Some moodle web services return null.
|
||||
|
|
Loading…
Reference in New Issue