Last modified: 2013-11-20 11:04:58 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T58649, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 56649 - VisualEditor: Exception when undoing foreign image insertion
VisualEditor: Exception when undoing foreign image insertion
Status: RESOLVED FIXED
Product: VisualEditor
Classification: Unclassified
ContentEditable (Other open bugs)
unspecified
All All
: Normal normal
: VE-deploy-2013-11-14
Assigned To: Roan Kattouw
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-11-06 01:02 UTC by Roan Kattouw
Modified: 2013-11-20 11:04 UTC (History)
5 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments
Screenshot of the error (117.52 KB, image/png)
2013-11-15 19:01 UTC, Rummana Yasmeen
Details
Screenshot of the error (177.26 KB, image/png)
2013-11-15 19:03 UTC, Rummana Yasmeen
Details
Screenshot of errors (88.00 KB, image/png)
2013-11-15 19:09 UTC, Rummana Yasmeen
Details

Description Roan Kattouw 2013-11-06 01:02:03 UTC
1. Insert an image from Commons
2. Press Ctrl+Z quickly

When you undo the image insertion, there's still an API request pending. When it comes back, ve.ce.GeneratedContentNode.prototype.doneGenerating crashes because the node is now detached.

GCNode should protect against being detached, and should abort any pending requests when it's detached (the latter alone is not enough: GCNode promises aren't guaranteed to be abortable).
Comment 1 Gerrit Notification Bot 2013-11-06 05:19:26 UTC
Change 93920 had a related patch set uploaded by Catrope:
Guard against detached nodes in ve.ce.GeneratedContentNode#doneGenerating

https://gerrit.wikimedia.org/r/93920
Comment 2 Gerrit Notification Bot 2013-11-12 12:15:17 UTC
Change 93920 merged by jenkins-bot:
Guard against detached nodes in ve.ce.GeneratedContentNode#doneGenerating

https://gerrit.wikimedia.org/r/93920
Comment 3 Rummana Yasmeen 2013-11-15 18:55:18 UTC
I am getting this following errors while undoing an image insertion quickly on both FireFox 25 and Chrome Version 26.0.1410.65 with MAC OS X 10.8.5.

Uncaught TypeError: Cannot read property 'imageinfo' of null
ve.ui.MWMediaInsertDialog.teardown
OO.ui.Window.close
(anonymous function)
proxy load.php:10


See the screenshots.

Not sure if it is related with this bug or not.For now, I am reopening it.If it is a different issue I will create a separate ticket for that if required.
Comment 4 Rummana Yasmeen 2013-11-15 19:01:16 UTC
Created attachment 13806 [details]
Screenshot of the error
Comment 5 Rummana Yasmeen 2013-11-15 19:03:18 UTC
Created attachment 13807 [details]
Screenshot of the error
Comment 6 Rummana Yasmeen 2013-11-15 19:06:54 UTC
After getting this error,cannot open the media dialog box again on that session.Attaching some more screenshots of the error.
Comment 7 Rummana Yasmeen 2013-11-15 19:09:00 UTC
Created attachment 13808 [details]
Screenshot of errors
Comment 8 James Forrester 2013-11-20 11:04:58 UTC
(In reply to comment #3)
> I am getting this following errors while undoing an image insertion quickly
> on
> both FireFox 25 and Chrome Version 26.0.1410.65 with MAC OS X 10.8.5.
> 
> Uncaught TypeError: Cannot read property 'imageinfo' of null
> ve.ui.MWMediaInsertDialog.teardown
> OO.ui.Window.close
> (anonymous function)
> proxy load.php:10
> 
> 
> See the screenshots.
> 
> Not sure if it is related with this bug or not.For now, I am reopening it.If
> it
> is a different issue I will create a separate ticket for that if required.

Unrelated issue caused by a buggy gadget, "popups".

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links