User "Wikinaut" posted a comment on MediaWiki.r89900.

Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/89900#c17943
Commit summary:

expressly set opensearch limit. Found a reproducible problem that on a large 
wiki a few case-insensitive matches were not shown when limit was not set to a 
meaningful value. But api.php has been found working by testing that via manual 
call from url. Perhaps a jQuery problem. Set limit to an value solved that 
problem immediately.

Comment:

I could (now) not immediately reproduce the problem, but can try again later 
(tomorrow+), as I did only change and reverted this line. (Yes, of course, I 
always clear the browsers cache after changing the javascsript. Setting the 
limit=1 is a good test while debugging this, to see, whether the correct script 
is actually used as wanted.)

The api call which I tested before committing this code change was
http://server/phase3/api.php?action=opensearch&search=ev&namespace=0&format=jsonfm
api output was like expected
["ev",["EVL","EVL-Redesign"]]

I could not detect any problem in api.php.

Just to remind you: I observed that:

when I entered in the search box (main page) the correct case page names
"EV" -> 2 suggestions
"ev" -> 0 Suggestions (wrong) (*)

After adding limit=10 I got
"EV" -> 2 suggestions
"ev" -> 2 Suggestions (correct)

Changing back gave the same as (*)


* Do you want me to open a bugzilla with links to this revision in order to 
further track this problem? 

_______________________________________________
MediaWiki-CodeReview mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-codereview

Reply via email to