Last modified: 2014-06-13 18:45:06 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 T67934, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 65934 - VisualEditor: [Regression wmf7] Characters and Pawns are getting added in each line and throwing error: ve.dm.Document.getNodeFromOffset(): offset -1 is out of bounds while trying to add a reference in Firefox
VisualEditor: [Regression wmf7] Characters and Pawns are getting added in eac...
Status: VERIFIED FIXED
Product: VisualEditor
Classification: Unclassified
Data Model (Other open bugs)
unspecified
All All
: High major
: VE-deploy-2014-06-05
Assigned To: Ed Sanders
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2014-05-30 06:15 UTC by Rummana Yasmeen
Modified: 2014-06-13 18:45 UTC (History)
6 users (show)

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


Attachments
Screenshot (47.88 KB, image/png)
2014-05-30 06:15 UTC, Rummana Yasmeen
Details

Description Rummana Yasmeen 2014-05-30 06:15:21 UTC
Created attachment 15528 [details]
Screenshot

Steps to reproduce

1.Open a page
2.Go to Insert>Reference
3.Type in a text as reference

Observed Result:
Each letters are getting added in each line , ends with a pawn then the text appears.The following error appears in the console:

Error: ve.dm.Document.getNodeFromOffset(): offset -1 is out of bounds

After clicking on "Insert Reference":

TypeError: sel.nativeSelection is null

After saving the page:
TypeError: elem.ownerDocument.defaultView is null

This issue is happening on Firefox only.

Environmen:test2
Comment 1 Ed Sanders 2014-05-30 19:49:03 UTC
Caused by https://gerrit.wikimedia.org/r/#/c/135012/ being a breaking change but not marked as such. Will be fixed once https://gerrit.wikimedia.org/r/#/c/135014 is merged.
Comment 2 Gerrit Notification Bot 2014-05-30 20:48:52 UTC
Change 136451 had a related patch set uploaded by Jforrester:
Cleanup createSurface methods

https://gerrit.wikimedia.org/r/136451
Comment 3 Gerrit Notification Bot 2014-05-30 21:17:22 UTC
Change 136451 merged by jenkins-bot:
Cleanup createSurface methods

https://gerrit.wikimedia.org/r/136451
Comment 4 Rummana Yasmeen 2014-05-30 23:12:48 UTC
Verified the fix in Betalabs
Comment 5 Rummana Yasmeen 2014-06-09 20:11:39 UTC
Verified the fix in test2
Comment 6 Rummana Yasmeen 2014-06-13 18:45:06 UTC
Verified the fix in en.wiki running wmf8

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


Navigation
Links