Maniphest T195915

[Tracking] Trying to saving a page when Mediawiki is on readonly mode shows no message or error
Closed, ResolvedPublic
Actions

Assigned To

Authored By

	Framawiki
	May 29 2018, 8:13 PM

Description

During a read-only passage of Wikipedia today (T194939, https://gerrit.wikimedia.org/r/#/c/435755/), I wanted to try to save a page via the mobile site.
After opening the editor, makes a small change in the code, clicking the Save button displays no messages, no errors. Interface just tries to save the page, but it can't. After a few seconds, the button becomes clickable again. The user can click several times on the button infinitely, the site tries again.
It would simply appear that the software does not take this case into account.
This is quite problematic since the mediawiki desktop website displays an error message chosen by the administrator. Mobile users do not see this information. Nothing indicates that it is temporarily normal that the site is not editable.

Analysis (@Jdrewniak)

From my analysis, I think there are 4 distinct bugs here:

Prior to editing, there is no indication the site is in read-only mode.
In mobile VE, saving fails with a "<apierror-readonly>" message (correctly), but the actual message text does not show up. (note: this could be a local error, should verify on another machine).
In mobile source editor after the save fails, a toast is presented with a generic error message, but the toast is not visible until you scroll down, and the toast appears "underneath" the article content. The error message should probably display the "apierror-readonly" message EditorOverlay.js#L285
In the EditorGateway the failed save falls through as a generic "http" error. The actual nature of the error is not considered when presenting the error message. EditorGateway.js#L233

Acceptance criteria

All subtasks resolved

Related Objects
Search...

Status	Assigned	Task
Resolved	Jdrewniak	T195915 [Tracking] Trying to saving a page when Mediawiki is on readonly mode shows no message or error
Resolved	Ryasmeen	T197497 In readonly mode edit icon should show as locked
Resolved	Esanders	T197498 VisualEditor error message for read only mode not loading on mobile
Resolved	Jdlrobson	T197499 Wikitext editor error handling broken in read only mode

Event Timeline

Framawiki created this task.May 29 2018, 8:13 PM

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptMay 29 2018, 8:13 PM

pmiazga added a project: Web-Team-Backlog.May 30 2018, 11:22 AM

Jdlrobson moved this task from Incoming to Needs Prioritization on the Web-Team-Backlog board.May 30 2018, 12:02 PM

ovasileva triaged this task as Medium priority.May 30 2018, 3:45 PM

Jdrewniak claimed this task.Jun 4 2018, 8:58 AM

Jdrewniak updated the task description. (Show Details)

Jdrewniak subscribed.

Jdrewniak updated the task description. (Show Details)Jun 4 2018, 9:06 AM

@Jdrewniak LGTM.

In mobile VE, saving fails with a "<apierror-readonly>" message (correctly), but the actual message text does not show up. (note: this could be a local error, should verify on another machine).

I think this is how VE works, this task should be tagged VE as they'll be better equipped to address it. The other 3 feel like they could be easily resolved in one task. Any reason to split this into 4 tasks or could we just do it (seems straightforward enough to me for one task rather than complicating QA)?

Prior to editing, there is no indication the site is in read-only mode.

Note, I'd assume this would mean showing the edit locked icon - if so, this prevents all of the other 3 errors from occurring (with the slight edge case of an edit that's in progress), so it seems addressing that is the highest priority.

Subtasks created!

• Deskana closed subtask T197498: VisualEditor error message for read only mode not loading on mobile as Resolved.Jun 18 2018, 1:07 PM

• Vvjjkkii reopened subtask T197498: VisualEditor error message for read only mode not loading on mobile as Open.Jul 1 2018, 1:03 AM

• Vvjjkkii renamed this task from [Tracking] Trying to saving a page when Mediawiki is on readonly mode shows no message or error to 80baaaaaaa.Jul 1 2018, 1:07 AM

• Vvjjkkii removed Jdrewniak as the assignee of this task.

• Vvjjkkii raised the priority of this task from Medium to High.

• Vvjjkkii added projects: CheckUser, Connected-Open-Heritage-Batch-uploads (RAÄ-KMB_1_2017-02), Tamil-Sites, Gamepress, Hashtags, Jade, KartoEditor, Language-2018-Apr-June, New-Editor-Experiences, Mail, TCB-Team (now WMDE-TechWish).

• Vvjjkkii updated the task description. (Show Details)

• Vvjjkkii removed a subscriber: Aklapper.

CommunityTechBot renamed this task from 80baaaaaaa to [Tracking] Trying to saving a page when Mediawiki is on readonly mode shows no message or error.Jul 2 2018, 2:42 AM

CommunityTechBot assigned this task to Jdrewniak.

CommunityTechBot lowered the priority of this task from High to Medium.

CommunityTechBot updated the task description. (Show Details)