Dan-nl has uploaded a new change for review.
https://gerrit.wikimedia.org/r/98686
Change subject: js-error-output
......................................................................
js-error-output
chris steipp requested that:
* Line 311: Don't include error message in that string, since it's sent to
.html().
Change-Id: Ie5f8d7da8fa1bc9ef7406c95f531f59ad88bbac8
---
M resources/js/ext.gwtoolset.js
1 file changed, 3 insertions(+), 10 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/GWToolset
refs/changes/86/98686/1
diff --git a/resources/js/ext.gwtoolset.js b/resources/js/ext.gwtoolset.js
index 93e2494..d92378f 100644
--- a/resources/js/ext.gwtoolset.js
+++ b/resources/js/ext.gwtoolset.js
@@ -305,16 +305,9 @@
*/
handleAjaxSuccess: function ( data ) {
if ( !data.edit || !data.edit.result ||
data.edit.result !== 'Success' ) {
- if ( data.error && data.error.info ) {
- gwtoolset.openDialog( {
- msg: mw.message(
'gwtoolset-save-mapping-failed' ).text() +
- ' ( ' + data.error.info
+ ' )'
- } );
- } else {
- gwtoolset.openDialog( {
- msg: mw.message(
'gwtoolset-save-mapping-failed' ).text()
- } );
- }
+ gwtoolset.openDialog( {
+ msg: mw.message(
'gwtoolset-save-mapping-failed' ).text()
+ } );
} else {
gwtoolset.openDialog( {
msg: mw.message(
'gwtoolset-save-mapping-succeeded' ).text()
--
To view, visit https://gerrit.wikimedia.org/r/98686
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ie5f8d7da8fa1bc9ef7406c95f531f59ad88bbac8
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/GWToolset
Gerrit-Branch: master
Gerrit-Owner: Dan-nl <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
