[Regression pre-wmf.36] Discussion page crashes with error: "ve.init.target.getSaveButtonLabel is not a function" when adding a new section/switching to the whole page editing mode with a draft comment
Closed, ResolvedPublic
Actions

Assigned To

Authored By

	Ryasmeen
	Mar 24 2021, 11:43 PM

Description

Steps to reproduce:

First, make sure you do not have the Quick Topic Adding option enabled on your preferences.
Go to any discussion page, for example: https://en.wikipedia.beta.wmflabs.org/wiki/User_talk:RYasmeen_(WMF)/sandbox
Click on a Reply link and draft a comment.
Without posting the comment, click on the "New Section" link on the top menu

Observed Result:
At this point you will see the progress bar, but it gets stuck and there is this error in the console:

Uncaught TypeError: ve.init.target.getSaveButtonLabel is not a function

Screen Shot 2021-03-24 at 3.59.04 PM.png (1×2 px, 498 KB)

Screen Shot 2021-03-24 at 4.00.14 PM.png (1×2 px, 682 KB)

If you go back to the Read mode now, you will still see the reply tool being open with the draft comment intact, which is expected I guess.

There is another way to reproduce this crash, by clicking on the "Edit Source" link on the top menu in step 4.

In this case as well, the whole page editing mode does not end up loading and you will see the stuck progress bar with the same error in the console.

And if you now go back to the Read mode, you will see this interesting "hybrid" mode:

Screen Shot 2021-03-24 at 4.22.23 PM.png (1×1 px, 528 KB)

Environment: Beta cluster.
Browser: Chrome.

Details

Related Changes in Gerrit:

	Subject	Repo	Branch	Lines +/-
	Avoid using mw.ArticleTarget methods on global ve.init.target in tools	mediawiki/extensions/VisualEditor	master	+13 -8

Customize query in gerrit

Event Timeline

Ryasmeen created this task.Mar 24 2021, 11:43 PM

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptMar 24 2021, 11:43 PM

Also, I couldn't reproduce this issue in production, so I am marking it as regression.

Ryasmeen updated the task description. (Show Details)Mar 24 2021, 11:54 PM

ppelberg added a project: Editing-team (Kanban Board).Mar 25 2021, 3:08 PM

ppelberg moved this task from Inbox to Upcoming on the Editing-team (Kanban Board) board.

Ryasmeen updated the task description. (Show Details)Mar 25 2021, 5:13 PM

Ryasmeen updated the task description. (Show Details)Mar 25 2021, 5:16 PM

ppelberg moved this task from Upcoming to Ready to Be Worked On on the Editing-team (Kanban Board) board.Mar 25 2021, 7:23 PM

ppelberg added a subscriber: DLynch.

@DLynch a question for you about the error @Ryasmeen identified [i]: What – if any – other workflows, beyond the workflow this task is written about, do you think this error could affect [i]?

i. Uncaught TypeError: ve.init.target.getSaveButtonLabel is not a function

This happens because the reply widget opens at the same time as the new wikitext editor (but it's not visible because NWE takes over the whole page), and it overwrites ve.init.target to point to itself, which confuses the editor.

ppelberg triaged this task as High priority.Mar 26 2021, 9:25 PM

In T278399#6947080, @matmarex wrote:

This happens because the reply widget opens at the same time as the new wikitext editor (but it's not visible because NWE takes over the whole page), and it overwrites ve.init.target to point to itself, which confuses the editor.

This context is helpful, @matmarex.

After talking about this with @Ryasmeen some, I've marked this task as "High" priority considering it is preventing a subset of people [i] from being able to access editing.

i. Note: I am assuming the population of people affected by this issue – NWE enabled and New Discussion Tool disabled – to be relatively small.

In T278399#6950202, @ppelberg wrote:

In T278399#6947080, @matmarex wrote:

This happens because the reply widget opens at the same time as the new wikitext editor (but it's not visible because NWE takes over the whole page), and it overwrites ve.init.target to point to itself, which confuses the editor.

This context is helpful, @matmarex.

After talking about this with @Ryasmeen some, I've marked this task as "High" priority considering it is preventing a subset of people [i] from being able to access editing.

i. Note: I am assuming the population of people affected by this issue – NWE enabled and New Discussion Tool disabled – to be relatively small.

Thanks @ppelberg.

Change 675582 had a related patch set uploaded (by Bartosz Dziewoński; author: Bartosz Dziewoński):
[mediawiki/extensions/VisualEditor@master] Avoid using global ve.init.target in tools

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

gerritbot added a project: Patch-For-Review.Mar 29 2021, 8:17 PM

matmarex claimed this task.Mar 29 2021, 8:17 PM

matmarex moved this task from Ready to Be Worked On to Code Review on the Editing-team (Kanban Board) board.

Change 675582 merged by jenkins-bot:

[mediawiki/extensions/VisualEditor@master] Avoid using mw.ArticleTarget methods on global ve.init.target in tools

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

ReleaseTaggerBot added a project: MW-1.36-notes (1.36.0-wmf.38; 2021-04-06).Apr 3 2021, 1:00 PM

Maintenance_bot removed a project: Patch-For-Review.Apr 3 2021, 1:10 PM

matmarex moved this task from Code Review to QA on the Editing-team (Kanban Board) board.Apr 5 2021, 7:08 PM

matmarex added a project: Editing QA.

The crash is not happening anymore. Thanks @matmarex! :)

Ryasmeen edited projects, added Verified; removed Editing QA.Apr 6 2021, 10:33 PM

Ryasmeen moved this task from QA to Ready for Sign Off on the Editing-team (Kanban Board) board.

ppelberg closed this task as Resolved.Apr 9 2021, 1:27 AM

Restricted Application added a project: User-Ryasmeen. · View Herald TranscriptApr 9 2021, 1:27 AM

	F34187682: Screen Shot 2021-03-24 at 4.22.23 PM.png
	Mar 24 2021, 11:43 PM

	F34187621: Screen Shot 2021-03-24 at 3.59.04 PM.png
	Mar 24 2021, 11:43 PM

	F34187624: Screen Shot 2021-03-24 at 4.00.14 PM.png
	Mar 24 2021, 11:43 PM

[Regression pre-wmf.36] Discussion page crashes with error: "ve.init.target.getSaveButtonLabel is not a function" when adding a new section/switching to the whole page editing mode with a draft commentClosed, ResolvedPublicActions

Description

Details

Event Timeline

[Regression pre-wmf.36] Discussion page crashes with error: "ve.init.target.getSaveButtonLabel is not a function" when adding a new section/switching to the whole page editing mode with a draft comment
Closed, ResolvedPublic
Actions