Last modified: 2013-06-13 13:06:29 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 T33002, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 31002 - jquery.hotkeys.js breaks search suggestions
jquery.hotkeys.js breaks search suggestions
Status: RESOLVED FIXED
Product: MediaWiki extensions
Classification: Unclassified
BookManager (Other open bugs)
unspecified
All All
: Normal major (vote)
: ---
Assigned To: Raylton P. Sousa
: javascript
Depends on: 47509
Blocks: 28011
  Show dependency treegraph
 
Reported: 2011-09-19 17:45 UTC by Helder
Modified: 2013-06-13 13:06 UTC (History)
1 user (show)

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


Attachments
JS error (140.84 KB, image/png)
2011-09-19 17:45 UTC, Helder
Details

Description Helder 2011-09-19 17:45:48 UTC
Created attachment 9074 [details]
JS error

I'm using latest version of BookManager and of MW (r97515) and when I access a chater of some book, there is a Javascript error caused by jquery.hotkeys.js.

On Google Chrome 13.0.782.220:
> http://localhost/mediawiki/extensions/BookManager/client/jquery.hotkeys.js:112
> Uncaught TypeError: Cannot read property 'combi' of null

On Firefox 6.02:
> data is null @ > http://localhost/mediawiki/extensions/BookManager/client/jquery.hotkeys.js:112

Looking at the call stack on Google Chrome, I see that the error occurs when 

> $( '#searchInput, #searchInput2, #powerSearchText, #searchText' )
>      .suggestions(...)

is executed on
http://localhost/mediawiki/extensions/Vector/modules/ext.vector.simpleSearch.js

This makes the search suggestions not to start when the user types some characters in the search field.
Comment 1 Derk-Jan Hartman 2013-04-23 13:36:05 UTC
The jquery.hotkeys.js class is a rather old one, and it's overloading jquery.fn.bind() (and breaking it apparently). With the newer jquery interfaces, this should no longer be necessary and this hotkeys implementation should be replaced with the newer implementation:

https://github.com/tzuryby/jquery.hotkeys
Comment 2 Helder 2013-06-13 13:06:06 UTC
Fixed by fixing bug 47509.
Comment 3 Helder 2013-06-13 13:06:29 UTC
https://gerrit.wikimedia.org/r/#/c/60367/

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


Navigation
Links