Last modified: 2014-06-20 21:26:39 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 T68692, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 66692 - VisualEditor: [Regression pre-wmf10] In Firefox, Saving the edit on a page everytime throws error: this.debugBar is null and cannot re-open VE after that
VisualEditor: [Regression pre-wmf10] In Firefox, Saving the edit on a page ev...
Status: VERIFIED FIXED
Product: VisualEditor
Classification: Unclassified
Editing Tools (Other open bugs)
unspecified
All All
: High normal
: VE-deploy-2014-06-12
Assigned To: Roan Kattouw
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2014-06-16 22:20 UTC by Rummana Yasmeen
Modified: 2014-06-20 21:26 UTC (History)
4 users (show)

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


Attachments
Screenshot (190.73 KB, image/png)
2014-06-16 22:20 UTC, Rummana Yasmeen
Details

Description Rummana Yasmeen 2014-06-16 22:20:06 UTC
Created attachment 15668 [details]
Screenshot

Steps to reproduce:

1.Open a page with VE in Firefox
2.Go to Insert>Language
3.Close the language inspector without making any change
4.Click on "Save Page"

Observed Result:
After step 3 you get the error: TypeError: sel.nativeSelection is null
After saving there is another error in the console:
TypeError: this.debugBar is null

If you had categories add on this page, they move to the top of the page
The context menu for Language Inspector is still open in read mode
Try to reopen VE , clicking on Edit button, VE does not open.

Environment: test2 and Betalabs
Comment 1 Rummana Yasmeen 2014-06-16 22:38:50 UTC
The second error: TypeError: this.debugBar is null
appears after selecting any language from the language selector
Comment 2 Gerrit Notification Bot 2014-06-16 22:54:41 UTC
Change 140039 had a related patch set uploaded by Catrope:
Followup 00af434: only tear down debug bar if it exists

https://gerrit.wikimedia.org/r/140039
Comment 3 Gerrit Notification Bot 2014-06-17 17:19:40 UTC
Change 140039 merged by jenkins-bot:
Followup 00af434: only tear down debug bar if it exists

https://gerrit.wikimedia.org/r/140039
Comment 4 James Forrester 2014-06-17 18:12:03 UTC
Are you sure this is a fault for test2? The code this follows up wasn't present for wmf9 unless I'm missing something.
Comment 5 Rummana Yasmeen 2014-06-17 18:31:42 UTC
Sorry, the language inspector throwing "TypeError:sel.nativeSelection is null"-
this is in both test2 and betalabs,but not the issue with tearing down the debug bar.
Initially I thought both of them are related.I am opening a different bug for that one.
Comment 6 James Forrester 2014-06-17 18:32:58 UTC
(In reply to ryasmeen from comment #5)
> Sorry, the language inspector throwing "TypeError:sel.nativeSelection is
> null"-
> this is in both test2 and betalabs,but not the issue with tearing down the
> debug bar.
> Initially I thought both of them are related.I am opening a different bug
> for that one.

OK, thanks.
Comment 7 Rummana Yasmeen 2014-06-17 18:34:07 UTC
Verified the fix in Betalabs
Comment 8 Rummana Yasmeen 2014-06-19 17:39:04 UTC
The backport did not work in test2
Comment 9 Rummana Yasmeen 2014-06-19 17:41:21 UTC
Ignore my last comment, added the comment on wrong bug
Comment 10 Rummana Yasmeen 2014-06-20 21:26:39 UTC
Verified the fix in production(en.wiki)

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


Navigation
Links