Page MenuHomePhabricator

Initial render of mention output in an empty (placeholder'ed) VE Flow reply is blank until it is changed
Closed, ResolvedPublic0 Estimated Story Points

Description

en.wiki does not have the issue.

  1. On any Talk page in ca.wiki or test.wiki - e.g. http://test.wikimedia.beta.wmflabs.org/wiki/Talk:Mavetuna or http://ca.wikipedia.beta.wmflabs.org/wiki/Discussi%C3%B3:DT6 - click on the Mention icon, select a suggestion and click Insert.
  2. There is no visible insert of Mention in UI. Switch to wikitext - the inserted template is displayed.

Clicking 'Save changes' in VE mode, saves the added Mention.

Event Timeline

Etonkovidova raised the priority of this task from to Needs Triage.
Etonkovidova updated the task description. (Show Details)
Etonkovidova subscribed.
Catrope renamed this task from [betalabs] cawiki and testwiki - adding Mention in VE is invisible until it is saved to Parsoid broken in beta for VisualEditor but not for Flow (RESTbase breakage?).Jul 29 2015, 11:18 PM
Catrope set Security to None.
Jdforrester-WMF renamed this task from Parsoid broken in beta for VisualEditor but not for Flow (RESTbase breakage?) to Initial render of mention output in an empty (placeholder'ed) VE Flow reply is blank until it is changed.Jul 29 2015, 11:23 PM
Jdforrester-WMF subscribed.

Title was sufficiently concerning to make me try this out myself. :-) Not a Parsoid or RESTbase issue AFAICT.

The preview request to api.php?action=visualeditor&paction=parsefragment results in a parsoidserver-http: HTTP 404 response, whereas api.php?action=flow-parsoid-utils requests to convert the same content do work. Is beta's RESTbase broken, perhaps?

Title was sufficiently concerning to make me try this out myself. :-) Not a Parsoid or RESTbase issue AFAICT.

It looks like RB is 404ing on anything and everything to do with missing pages. The reason we saw it in Flow was because we were testing on a new page.

Sounds like this is a bug in the VE API module not expecting RB's (legitimate) 404s.

Change 227918 had a related patch set uploaded (by Catrope):
Add {ar,ca}wiki and remove ee-prototype in beta labs RB config

https://gerrit.wikimedia.org/r/227918

Catrope claimed this task.
Catrope reassigned this task from Catrope to GWicke.

Change 227918 merged by Ori.livneh:
Add {ar,ca}wiki and remove ee-prototype in beta labs RB config

https://gerrit.wikimedia.org/r/227918