Error
MediaWiki version: n/a
Uncaught TypeError: Cannot read property 'length' of null
Jdlrobson | |
Aug 28 2020, 7:00 PM |
F32414945: image.png | |
Oct 26 2020, 7:25 PM |
F32414936: image.png | |
Oct 26 2020, 7:25 PM |
F32414960: image.png | |
Oct 26 2020, 7:25 PM |
F32386391: image.png | |
Sep 22 2020, 8:27 PM |
F32386396: image.png | |
Sep 22 2020, 8:27 PM |
MediaWiki version: n/a
Uncaught TypeError: Cannot read property 'length' of null
at VeUiFindAndReplaceDialog.ve.ui.FindAndReplaceDialog.renderRangeOfFragments https://ca.wikipedia.org/w/load.php?lang=ca&modules=ext.visualEditor.articleTarget,base,core,desktopArticleTarget,desktopTarget,diffing,icons,language,mediawiki,mwalienextension,mwcore,mwextensions,mwformatting,mwgallery,mwimage,mwlanguage,mwlink,mwmeta,mwsave,mwsignature,mwtransclusion,sanitize,switching,welcome|ext.visualEditor.core.desktop,utils|ext.visualEditor.mwextensions.desktop|ext.visualEditor.mwimage.core&skin=vector&version=zd0la:822:690 at VeUiFindAndReplaceDialog.ve.ui.FindAndReplaceDialog.renderFragments https://ca.wikipedia.org/w/load.php?lang=ca&modules=ext.visualEditor.articleTarget,base,core,desktopArticleTarget,desktopTarget,diffing,icons,language,mediawiki,mwalienextension,mwcore,mwextensions,mwformatting,mwgallery,mwimage,mwlanguage,mwlink,mwmeta,mwsave,mwsignature,mwtransclusion,sanitize,switching,welcome|ext.visualEditor.core.desktop,utils|ext.visualEditor.mwextensions.desktop|ext.visualEditor.mwimage.core&skin=vector&version=zd0la:821:843 at run https://ca.wikipedia.org/w/load.php?lang=ca&modules=jquery,oojs-ui-core,oojs-ui-widgets,site&skin=vector&version=1pfn7:157:645
Change 623069 had a related patch set uploaded (by Esanders; owner: Esanders):
[VisualEditor/VisualEditor@master] FindAndReplace: Guard against getSelectionRects failing
I couldn't reproduce this, but as the offending path tried to build a DOM range from a selection on a debounced event, I can see how that could result in the DOM range not being able to be constructed. Absent of reports of the dialog not behaving properly, I think it's fine to just give up if the rects can't be generated
Change 623069 merged by jenkins-bot:
[VisualEditor/VisualEditor@master] FindAndReplace: Guard against getSelectionRects failing
As we couldn't reproduce this, there is nothing to test. Hopefully it will stop appearing in the error logs.
Change 623875 had a related patch set uploaded (by Bartosz Dziewoński; owner: Bartosz Dziewoński):
[mediawiki/extensions/VisualEditor@master] Update VE core submodule to master (b49db9a5b)
Change 623875 merged by jenkins-bot:
[mediawiki/extensions/VisualEditor@master] Update VE core submodule to master (9534694b4)
These are probably all the same error (the error messages are different because they're generated by different browsers). The fix is only getting deployed this week, because there was no deployment train last week.
Change 626476 had a related patch set uploaded (by Esanders; owner: Esanders):
[VisualEditor/VisualEditor@master] More direct fix for find-and-replace running after teardown issues
Change 626476 merged by jenkins-bot:
[VisualEditor/VisualEditor@master] More direct fix for find-and-replace running after teardown issues
Change 626737 had a related patch set uploaded (by Bartosz Dziewoński; owner: Bartosz Dziewoński):
[mediawiki/extensions/VisualEditor@master] Update VE core submodule to master (22a771930)
Change 626737 merged by jenkins-bot:
[mediawiki/extensions/VisualEditor@master] Update VE core submodule to master (22a771930)
It looks like the "More direct fix" (https://gerrit.wikimedia.org/r/626476) regressed the issue.
At first the errors disappeared between 2020-09-09 and 2020-09-16 (corresponding to the first patch https://gerrit.wikimedia.org/r/623069), then reappeared (corresponding to https://gerrit.wikimedia.org/r/626476).
Chart for "rects is null": https://logstash.wikimedia.org/goto/91e5fcdf5bd39363519d06086f659b0e
Change 629214 had a related patch set uploaded (by Bartosz Dziewoński; owner: Bartosz Dziewoński):
[VisualEditor/VisualEditor@master] Restore check for getSelectionRects returning null
Change 629214 merged by jenkins-bot:
[VisualEditor/VisualEditor@master] Restore check for getSelectionRects returning null
Change 629450 had a related patch set uploaded (by Bartosz Dziewoński; owner: Bartosz Dziewoński):
[mediawiki/extensions/VisualEditor@master] Update VE core submodule to master (0a5e284c7)
Change 629450 merged by jenkins-bot:
[mediawiki/extensions/VisualEditor@master] Update VE core submodule to master (0a5e284c7)
I think this also shows up as` TypeError: rect is null` https://logstash.wikimedia.org/app/kibana#/doc/logstash-*/logstash-2020.10.14/clienterror/?id=AXUoB4XL2lgCwKx7G3H8
Stopped occurring after 13 October.
"rects.length": https://logstash.wikimedia.org/goto/d3fc8c9caa6ebeace47854fc2b397a0b
"Cannot read property 'length' of null": https://logstash.wikimedia.org/goto/d7479784897998032da012bd44ae07f2
(the same error message can easily occur for unrelated problems, and it still appears, but at greatly reduced volume)
"rects is null": https://logstash.wikimedia.org/goto/a68b172beab3d9751f336990a6b08986