Alex Monk has uploaded a new change for review.
https://gerrit.wikimedia.org/r/314216
Change subject: Use a proper error message when VE fails to load due to not
being able to connect to RB
......................................................................
Use a proper error message when VE fails to load due to not being able to
connect to RB
Not just 'HTTP 0'
Change-Id: Icd1b5b64ca8ea4af80e3cf32f8502d8fb0d11ae2
---
M extension.json
M modules/ve-mw/i18n/en.json
M modules/ve-mw/i18n/qqq.json
M modules/ve-mw/init/ve.init.mw.ArticleTargetLoader.js
4 files changed, 11 insertions(+), 1 deletion(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/VisualEditor
refs/changes/16/314216/1
diff --git a/extension.json b/extension.json
index 3930bf9..6b92b96 100644
--- a/extension.json
+++ b/extension.json
@@ -337,6 +337,7 @@
],
"messages": [
"visualeditor-loaderror-message",
+ "visualeditor-loaderror-message-noconnect",
"visualeditor-loaderror-title"
],
"targets": [
diff --git a/modules/ve-mw/i18n/en.json b/modules/ve-mw/i18n/en.json
index a31646e..1287b7d 100644
--- a/modules/ve-mw/i18n/en.json
+++ b/modules/ve-mw/i18n/en.json
@@ -244,6 +244,7 @@
"visualeditor-linknodeinspector-add-label": "Add label",
"visualeditor-linknodeinspector-title": "Simple link",
"visualeditor-loaderror-message": "Error loading data from server: $1.",
+ "visualeditor-loaderror-message-noconnect": "Could not connect",
"visualeditor-loaderror-title": "The editor could not load",
"visualeditor-loadwarning": "Error loading data from server: $1. Would
you like to retry?",
"visualeditor-loadwarning-token": "Error loading edit token from
server: $1. Would you like to retry?",
diff --git a/modules/ve-mw/i18n/qqq.json b/modules/ve-mw/i18n/qqq.json
index 7916675..3896c70 100644
--- a/modules/ve-mw/i18n/qqq.json
+++ b/modules/ve-mw/i18n/qqq.json
@@ -255,6 +255,7 @@
"visualeditor-linknodeinspector-add-label": "Label of button that
converts an auto-numbered, external, labelless link into a labeled external
link",
"visualeditor-linknodeinspector-title": "Title of inspector for editing
auto-numbered, external, labelless links.\n\nSee also:\n*
{{msg-mw|Visualeditor-annotationbutton-linknode-tooltip}}",
"visualeditor-loaderror-message": "Text shown in a message box when the
editor fails to load properly due to a RESTBase call failing.\n\nParameters:\n*
$1 - an error message",
+ "visualeditor-loaderror-message-noconnect": "Error message used in
{{mw-msg|visualeditor-loaderror-message}} when the editor fails to load
properly due to a RESTBase call failing due to not being able to connect.",
"visualeditor-loaderror-title": "Text shown as the title of a message
box when the editor fails to load properly for some reason.",
"visualeditor-loadwarning": "Text (JavaScript confirm()) shown when the
editor fails to load properly.\n\nParameters:\n* $1 - the error message from
the server, in English. e.g. \"docserver-http-error\"",
"visualeditor-loadwarning-token": "Text (JavaScript confirm()) shown
when the editor fails to load properly.\n\nParameters:\n* $1 - the error
message from the server.",
diff --git a/modules/ve-mw/init/ve.init.mw.ArticleTargetLoader.js
b/modules/ve-mw/init/ve.init.mw.ArticleTargetLoader.js
index a75603c..3c4c065 100644
--- a/modules/ve-mw/init/ve.init.mw.ArticleTargetLoader.js
+++ b/modules/ve-mw/init/ve.init.mw.ArticleTargetLoader.js
@@ -204,7 +204,14 @@
// Page does not exist,
so let the user start with a blank document.
return
$.Deferred().resolve( [ '', undefined ] ).promise();
} else {
- window.alert( mw.msg(
'visualeditor-loaderror-message', 'HTTP ' + response.status ) );
+ if ( response.status ) {
+ window.alert(
mw.msg( 'visualeditor-loaderror-message', 'HTTP ' + response.status ) );
+ } else {
+ window.alert(
mw.msg(
+
'visualeditor-loaderror-message',
+ mw.msg(
'visualeditor-loaderror-message-noconnect' )
+ ) );
+ }
mw.log.warn( 'RESTBase
load failed: ' + response.statusText );
}
--
To view, visit https://gerrit.wikimedia.org/r/314216
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: Icd1b5b64ca8ea4af80e3cf32f8502d8fb0d11ae2
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/VisualEditor
Gerrit-Branch: master
Gerrit-Owner: Alex Monk <a...@wikimedia.org>
_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
