Marking notification as read causes "TypeError: sourceModel is null"
Closed, ResolvedPublic
Actions

Assigned To

Authored By

	He7d3r
	Jun 30 2016, 9:00 PM

Description

This JavaScript error happened right after I clicked on mark as read in the popup:

{
	"errorMessage": "TypeError: sourceModel is null",
	"url": "https://pt.wikipedia.org/w/load.php?debug=false&lang=pt&modules=ext.echo.controller%2Cdm%2Clogger%2Cui%7Cext.echo.styles.badge%2Cnotifications%7Cext.echo.ui.desktop%7Cmoment%2Coojs-ui-core%2Coojs-ui-widgets%7Coojs-ui-core.styles%7Coojs-ui.styles.icons%2Cicons-alerts%2Cicons-content%2Cicons-interactions%2Cicons-user%2Cindicators%2Ctextures%7Cschema.EchoInteraction&skin=vector&version=4377d2b7cb99",
	"lineNumber": 8,
	"columnNumber": 808,
	"errorObject": {}
}

in debug mode I get this:

{
	"errorMessage": "TypeError: sourceModel is null",
	"url": "https://pt.wikipedia.org/w/extensions/Echo/modules/controller/mw.echo.Controller.js?d173b",
	"lineNumber": 487,
	"columnNumber": 3,
	"errorObject": {}
}

from this part of the code:

		sourceModel = xwikiModel.getList().getGroupBySource( modelName );
		notifs = sourceModel.findByIds( itemIds );
		sourceModel.discardItems( notifs );

Details

Related Changes in Gerrit:

	Subject	Repo	Branch	Lines +/-
	Fix JS error when marking foreign notifications as read	mediawiki/extensions/Echo	master	+1 -1
	Fix mark xwiki as read	mediawiki/extensions/Echo	master	+11 -33

Customize query in gerrit

Related Objects

Mentioned In: T139339: Cross-wiki mark as read redirects to the page instead of actually mark as read

Event Timeline

He7d3r created this task.Jun 30 2016, 9:00 PM

Restricted Application added subscribers: Zppix, Aklapper. · View Herald TranscriptJun 30 2016, 9:00 PM

He7d3r renamed this task from Marking notification as read causes to Marking notification as read causes "TypeError: sourceModel is null".Jun 30 2016, 9:00 PM

He7d3r added a project: JavaScript.Jul 1 2016, 12:22 PM

He7d3r updated the task description. (Show Details)

Catrope assigned this task to Mooeypoo.Jul 1 2016, 1:29 PM

Catrope triaged this task as Unbreak Now! priority.

Catrope added a project: Collab-Team-Q1-July-Sep-2016.

Restricted Application added subscribers: Luke081515, TerraCodes, Urbanecm. · View Herald TranscriptJul 1 2016, 1:29 PM

Catrope moved this task from Untriaged to Ready for Pickup on the Collab-Team-Q1-July-Sep-2016 board.Jul 1 2016, 1:29 PM

Stack trace in betalabs for clicking on <span class="oo-ui-labelElement-label mw-echo-ui-menuItemWidget-content-label">Marca com a llegit</span>

TypeError: sourceModel is null
mw.echo.Controller.prototype.markCrossWikiItemsRead()
mw.echo.Controller.prototype.markSingleItemRead()
mw.echo.ui.SingleNotificationItemWidget.prototype.markRead()
mw.echo.ui.NotificationItemWidget.prototype.onPopupButtonWidgetChoose()
oo.EventEmitter.prototype.emit()
OO.ui.SelectWidget.prototype.chooseItem()
OO.ui.MenuSelectWidget.prototype.chooseItem()
OO.ui.SelectWidget.prototype.onMouseUp()
bound ()

Change 296958 had a related patch set uploaded (by Catrope):
Fix JS error when marking foreign notifications as read

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

gerritbot added a project: Patch-For-Review.Jul 1 2016, 7:47 PM

Catrope claimed this task.Jul 1 2016, 7:47 PM

Catrope moved this task from Ready for Pickup to Needs Review on the Collab-Team-Q1-July-Sep-2016 board.

Catrope added a subscriber: Mooeypoo.

Change 297262 had a related patch set uploaded (by Sbisson):
Fix mark xwiki as read

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

SBisson moved this task from Needs Review to QA Review on the Collab-Team-Q1-July-Sep-2016 board.Jul 4 2016, 6:51 PM

Change 297262 merged by jenkins-bot:
Fix mark xwiki as read

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

ReleaseTaggerBot added a project: MW-1.28-release (WMF-deploy-2016-07-05_(1.28.0-wmf.9)).Jul 4 2016, 7:00 PM

Change 296958 abandoned by Sbisson:
Fix JS error when marking foreign notifications as read

Reason:
The other fix was merged.

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

SBisson mentioned this in T139339: Cross-wiki mark as read redirects to the page instead of actually mark as read.Jul 5 2016, 12:53 PM

Quiddity lowered the priority of this task from Unbreak Now! to High.Jul 13 2016, 8:49 PM

Checked the fix in betalabs.

Etonkovidova moved this task from QA Review to Product Review on the Collab-Team-Q1-July-Sep-2016 board.Jul 13 2016, 10:38 PM

• jmatazzoni closed this task as Resolved.Jul 20 2016, 9:02 PM

Marking notification as read causes "TypeError: sourceModel is null"Closed, ResolvedPublicActions

Description

Details

Related Objects

Event Timeline

Marking notification as read causes "TypeError: sourceModel is null"
Closed, ResolvedPublic
Actions