mw.DesktopArticleTarget.prototype#loadFail sometimes given (unexpected?) jqXHR
Closed, ResolvedPublic1 Estimated Story Points
Actions

Assigned To

Authored By

	Krinkle
	Nov 22 2016, 2:36 AM

Description

Loading VisualEditor with jQuery 3+Migrate (T124742; https://gerrit.wikimedia.org/r/322812) results in the following warning:

JQMIGRATE: jQXHR.error is deprecated and removed

..

ve.init.mw.DesktopArticleTarget.loadFail

..

ve.init.mw.ArticleTarget.load

ve.init.mw.DesktopArticleTarget.activate

It seems loadFail() is given an jqXHR as first parameter. This matches the original documentation in parent class mw.ArticleTarget#loadFail, however I'm not sure if the code still expects this.

Perhaps it can be cleaned up as to not be passed anymore, or altered to not access its .error property, which is in fact, not an error representation, but a deprecated jqXHR method that is an alias of jQuery.Deferred#fail(), which adds a callback to the promise rejection handler.

Details

	Subject	Repo	Branch	Lines +/-
	Fix loadFail parameter handling	mediawiki/extensions/VisualEditor	master	+27 -25

Customize query in gerrit

Related Objects

Mentioned Here: T124742: Upgrade to jQuery 3

Event Timeline

Krinkle created this task.Nov 22 2016, 2:36 AM

Restricted Application added a project: VisualEditor. · View Herald TranscriptNov 22 2016, 2:36 AM

Restricted Application added a subscriber: Aklapper. · View Herald Transcript

Jdforrester-WMF triaged this task as High priority.Nov 22 2016, 8:06 PM

Jdforrester-WMF set the point value for this task to 1.

Jdforrester-WMF moved this task from To Triage to TR0: Interrupt on the VisualEditor board.

Reproducible when you have VE configured to connect to RB directly from the client and it's not running. https://gerrit.wikimedia.org/r/#/c/327721/ seems to fix

Change 327721 had a related patch set uploaded (by Bartosz Dziewoński):
Fix loadFail parameter handling

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

gerritbot added a project: Patch-For-Review.Jan 4 2017, 10:31 PM

matmarex assigned this task to • AlexMonk-WMF.Jan 4 2017, 10:34 PM

Change 327721 merged by jenkins-bot:
Fix loadFail parameter handling

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

Jdforrester-WMF closed this task as Resolved.Jan 4 2017, 10:54 PM

Jdforrester-WMF removed a project: Patch-For-Review.