L-electron-Rare/moodle
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

MDL-33680 JavaScript: Improve error handling on user selector

Browse Source 
Since the previous commit to this issue removes the built-in debugging
tests, I've added these back using the M.core libraries.
This commit is contained in:
Andrew Robert Nicols
2013-03-16 10:52:02 +00:00
parent 28037331fb
commit c2f47c19ad
2 changed files with 10 additions and 22 deletions
Download Patch File Download Diff File Expand all files Collapse all files
user/selector/lib.php
+1 -1
View File
@@ -70,7 +70,7 @@ abstract class user_selector_base {
protected static $jsmodule = array(
'name' => 'user_selector',
'fullpath' => '/user/selector/module.js',
'requires' => array('node', 'event-custom', 'datasource', 'json'),
'requires' => array('node', 'event-custom', 'datasource', 'json', 'moodle-core-notification'),
'strings' => array(
array('previouslyselectedusers', 'moodle', '%%SEARCHTERM%%'),
array('nomatchingusers', 'moodle', '%%SEARCHTERM%%'),
user/selector/module.js
+9 -21
View File
@@ -151,8 +151,7 @@ M.core_user.init_user_selector = function (Y, name, hash, extrafields, lastsearc
method: 'POST',
data: 'selectorid='+hash+'&sesskey='+M.cfg.sesskey+'&search='+value + '&userselector_searchanywhere=' + this.get_option('searchanywhere'),
on: {
success:this.handle_response,
failure:this.handle_failure
complete: this.handle_response
},
context:this
});
@@ -175,26 +174,15 @@ M.core_user.init_user_selector = function (Y, name, hash, extrafields, lastsearc
}
this.listbox.setStyle('background','');
var data = Y.JSON.parse(response.responseText);
if (data.error) {
this.searchfield.addClass('error');
return new M.core.ajaxException(data);
}
this.output_options(data);
} catch (e) {
this.handle_failure(requestid);
}
},
/**
* Handles what happens when the ajax request fails.
*/
handle_failure : function(requestid) {
delete this.iotransactions[requestid];
if (!Y.Object.isEmpty(this.iotransactions)) {
// More searches pending. Wait until they are all done.
return;
}
this.listbox.setStyle('background','');
this.searchfield.addClass('error');
// If we are in developer debug mode, output a link to help debug the failure.
if (M.cfg.developerdebug) {
this.searchfield.insert(Y.Node.create('<a href="'+M.cfg.wwwroot +'/user/selector/search.php?selectorid='+hash+'&sesskey='+M.cfg.sesskey+'&search='+this.get_search_text()+'&debug=1">Ajax call failed. Click here to try the search call directly.</a>'));
this.listbox.setStyle('background','');
this.searchfield.addClass('error');
return new M.core.exception(e);
}
},
/**
@@ -379,4 +367,4 @@ M.core_user.init_user_selector_options_tracker = function(Y) {
user_selector_options_tracker.init();
// Return it just incase it is ever wanted
return user_selector_options_tracker;
};
};