Page MenuHomePhabricator
Create Task
Maniphest T49581

VisualEditor: API paction=serialize should say so if Parsoid is unavailable (instead of status "success" with missing content)
Closed, ResolvedPublic
Actions

Description

  1. Parsoid not running, VisualEditor enabled on to-be-created page X
  2. Enter a few words
  3. "Review and Save"
  4. Loader keeps spinning indefinitely

Request:

action=visualeditor&paction=serialize&html=%3Cp+data-ve-changed%3D%22%7B%26quot%3Bcontent%26quot%3B%3A7%7D%22%3Edfgdg%3C%2Fp%3E&page=Sandbox%2F1&oldid=0&token=%2B%5C&format=json

Response:

{"visualeditor":{"result":"success","content":false}}

No errors.

Note this bug could also be reported as "Loader keeps spinning forever when creating a new page and trying to review/save it"

Version: unspecified
Severity: normal

Details

Reference
bz47581

Related Objects
Search...

Event Timeline

• bzimport raised the priority of this task from to Medium.Nov 22 2014, 1:32 AM
• bzimport added projects: VE-deploy-2013-05-27, VisualEditor-MediaWiki.
• bzimport set Reference to bz47581.
Krinkle created this task.Apr 23 2013, 10:51 PM
gerritbot added a comment.Apr 24 2013, 12:02 AM

Related URL: https://gerrit.wikimedia.org/r/60613 (Gerrit Change Icdf64a792c13a326f494e051be47f2946928d142)

Krinkle added a comment.Apr 24 2013, 12:21 AM

(In reply to comment #1)

Related URL: https://gerrit.wikimedia.org/r/60613 (Gerrit Change
Icdf64a792c13a326f494e051be47f2946928d142)

This fixed the visible problem described below:

(In reply to comment #0)

as "Loader keeps spinning forever when creating a new page and
trying to review/save it"

The error is now handled by stopping the spinner (as we always do after an aborted load) and display the error.

The error message in my case was "No Wikitext content in response from server" triggered from ve.init.mw.Target.onSerialize's patch where there are no errors and eventually finds this by type-checking the content property.

Lowering priority. Todo tasks:

  • Don't return status "success" if content is missing, but use status "error".
  • Make it not just status "error" but actually trigger a proper API response.error property with a more direct error, namely that the Parsoid server is unavailable.
gerritbot added a comment.May 15 2013, 3:30 PM

Related URL: https://gerrit.wikimedia.org/r/63861 (Gerrit Change I6bdd177bc9b5a0af9107cf08d8e46470c6778e12)

Jdforrester-WMF added a comment.May 15 2013, 5:26 PM

Merged into master; will go out with wmf5.

Jdforrester-WMF added a project: VisualEditor.Dec 3 2014, 2:21 AM
Content licensed under Creative Commons Attribution-ShareAlike (CC BY-SA) 4.0 unless otherwise noted; code licensed under GNU General Public License (GPL) 2.0 or later and other open source licenses. By using this site, you agree to the Terms of Use, Privacy Policy, and Code of Conduct. · Wikimedia Foundation · Privacy Policy · Code of Conduct · Terms of Use · Disclaimer · CC-BY-SA · GPL