"TypeError: dataMatrixLine is undefined" when trying to add a column to a table with an empty <tr> at the end
Closed, ResolvedPublic1 Estimated Story Points
Actions

Assigned To

Authored By

	daniel
	Dec 5 2016, 12:03 PM

Description

When trying to add a column to the big table on https://www.wikidata.org/w/index.php?title=User:Glorian_Yapinus_(WMDE)/quantity_change_properties&oldid=414054606, I get:

TypeError: dataMatrixLine is undefined
ve.ui.TableAction.prototype.insertRowOrCol()        load.php:875
ve.ui.TableAction.prototype.insert()                load.php:867
ve.ui.Surface.prototype.execute()                   load.php:789
ve.ui.Command.prototype.execute()                   load.php:815
ve.ui.TableLineContextItem.prototype.onActionButtonClick()

I tried action=edit with debug=1, but then I don't even get any toolbar buttons.

This is with FF 49.0 on Ubuntu.

Details

	Project	Branch	Lines +/-	Subject
	mediawiki/extensions/VisualEditor	master	+1 -1	Update VE core submodule to master (33012af)
	VisualEditor/VisualEditor	master	+39 -1	ui.TableAction: don't try to add cells if none have been provided

Customize query in gerrit

Related Objects

Mentioned In: T152806: Wikitext representing an empty <tr> produces technically-invalid output
T152659: [Regression?] Cannot add columns to sparse tables in VisualEditor
Mentioned Here: rGVED33012affe066: Update UnicodeJS to v0.1.6

Event Timeline

daniel created this task.Dec 5 2016, 12:03 PM

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptDec 5 2016, 12:03 PM

daniel updated the task description. (Show Details)Dec 5 2016, 12:04 PM

I'm getting:

Uncaught TypeError: Cannot read property 'cells' of undefined
    at VeUiTableAction.ve.ui.TableAction.insertRowOrCol
    at VeUiTableAction.ve.ui.TableAction.insert
    at VeUiDesktopSurface.ve.ui.Surface.execute

… which is equally not good.

The table seems to be incomplete / parsed into VE as incomplete, or so I'm guessing from this rendering:

(Note the partial row at the bottom.)

The error is because the code assumes that if it reaches a not-present cell index, dataMatrixLine.cells must be present with a new cell to insert there.

Specifically, the Parsoid output for this page includes an empty table row at the end: https://www.wikidata.org/api/rest_v1/page/html/User%3AGlorian_Yapinus_(WMDE)%2Fquantity_change_properties/414054606?redirect=false

Which is fair enough, because the wikitext markup for it includes an empty row at the end:

{| class="wikitable"
|'''Properties'''
|'''Exact Value'''
|'''Measurement'''
|'''I Don’t Know/Am Not Sure'''
|----
|generation time (P3337)
|
|X
|
|----
|operating income (P3362)
|
|X
|
|----
|}

Jdforrester-WMF renamed this task from "TypeError: dataMatrixLine is undefined" when trying to add table coloumn using VE to "TypeError: dataMatrixLine is undefined" when trying to add a column to a table with an empty <tr> at the end.Dec 6 2016, 10:16 PM

DLynch mentioned this in T152659: [Regression?] Cannot add columns to sparse tables in VisualEditor.Dec 9 2016, 7:57 PM

Change 326162 had a related patch set uploaded (by DLynch):
ui.TableAction: don't try to add cells if none have been provided

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

gerritbot added a project: Patch-For-Review.Dec 9 2016, 8:05 PM

DLynch mentioned this in T152806: Wikitext representing an empty <tr> produces technically-invalid output.Dec 9 2016, 9:56 PM

Change 326162 merged by jenkins-bot:
ui.TableAction: don't try to add cells if none have been provided

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

Change 326141 had a related patch set uploaded (by Jforrester):
Update VE core submodule to master (33012af)

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

Jdforrester-WMF closed this task as Resolved.Dec 10 2016, 12:10 AM

Jdforrester-WMF claimed this task.

Jdforrester-WMF reassigned this task from Jdforrester-WMF to DLynch.

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

Change 326141 merged by jenkins-bot:
Update VE core submodule to master (33012af)