Page MenuHomePhabricator

HTML for newly-created pages with VE on testwiki not stored in RESTBase
Open, MediumPublic

Description

Steps to reproduce:

  • go to a non-existing page on testwiki
  • edit in VisualEditor
  • add some content and save
  • open for editing again

You'll be faced with an empty edit area, the previously saved will be gone. (It's properly saved but VE does not see it.)
Neither waiting between the save and the edit nor reloading the page inbetween avoids the bug.
Seems somewhat non-deterministic - on repeated attempts to edit the page, sometimes the content does show up.

Details

Related Gerrit Patches:
mediawiki/services/restbase/deploy : masterRevert "[TEMP] Switch test2.wp back to Parsoid/JS"
mediawiki/services/restbase/deploy : master[TEMP] Switch test2.wp back to Parsoid/JS

Event Timeline

Tgr created this task.Wed, Nov 20, 1:53 AM
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptWed, Nov 20, 1:53 AM
Tgr updated the task description. (Show Details)Wed, Nov 20, 1:58 AM
Tgr added a subscriber: ssastry.

Ping @ssastry - this is on testwiki probably fairly recent so Parsoid-PHP seems like the most plausible culprit.

ssastry added a subscriber: mobrovac.

Parsoid doesn't directly fetch any content. So, I suspect this is likely some RESTBase issue.

Change 552023 had a related patch set uploaded (by Mobrovac; owner: Mobrovac):
[mediawiki/services/restbase/deploy@master] [TEMP] Switch test2.wp back to Parsoid/JS

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

Change 552023 merged by Mobrovac:
[mediawiki/services/restbase/deploy@master] [TEMP] Switch test2.wp back to Parsoid/JS

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

Mentioned in SAL (#wikimedia-operations) [2019-11-20T09:52:21Z] <mobrovac@deploy1001> Started deploy [restbase/deploy@c677063]: Switch test2.wp back to Parsoid/JS temporarily - T238716

Mentioned in SAL (#wikimedia-operations) [2019-11-20T10:07:15Z] <mobrovac@deploy1001> Finished deploy [restbase/deploy@c677063]: Switch test2.wp back to Parsoid/JS temporarily - T238716 (duration: 14m 54s)

Change 552030 had a related patch set uploaded (by Mobrovac; owner: Mobrovac):
[mediawiki/services/restbase/deploy@master] Revert "[TEMP] Switch test2.wp back to Parsoid/JS"

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

Change 552030 merged by Mobrovac:
[mediawiki/services/restbase/deploy@master] Revert "[TEMP] Switch test2.wp back to Parsoid/JS"

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

Mentioned in SAL (#wikimedia-operations) [2019-11-20T10:22:54Z] <mobrovac@deploy1001> Started deploy [restbase/deploy@daa7808]: Revert switching test2.wp to Parsoid/JS - T238716

Mentioned in SAL (#wikimedia-operations) [2019-11-20T10:36:50Z] <mobrovac@deploy1001> Finished deploy [restbase/deploy@daa7808]: Revert switching test2.wp to Parsoid/JS - T238716 (duration: 13m 56s)

This issue is not dependent on the Parsoid backend used (hence, not a new issue, luckily). Instead, it seems to be a deeper issue, but only with test wikipedias and for newly-created pages only. We'll need to investigate some more.

mobrovac renamed this task from VisualEditor does not see previously saved content on testwiki to HTML for newly-created pages with VE on testwiki not stored in RESTBase.Wed, Nov 20, 10:44 AM
mobrovac triaged this task as Medium priority.
JTannerWMF added a subscriber: JTannerWMF.

There doesn't appear to be action for the Editing-team so we will move this to external.