cleanup to mwsuggest.js & fix some things suggested by JSLint

Pass parser by reference on line 3344 for bug 22154 extension broken

(line 3351 passes by reference

Don't show empty edit box for invalid sections requested by users who cannot edit

In LanguageConverter:
* Rewrote convertArray() as an RD parser (with inline tokenizer) as suggested on CR r60986. Fixes unclosed rule issue (with parser test). Fixes O(N^2) timing.
* Removed $this->mMarkup abstraction. Life is complicated enough as it is.
* Replaced a couple of instances of explode() with StringUtils::explode(), limited element count in a couple more.

In ConverterRule:
* Removed mConvTable initialisation from the constructor, unnecessary
* Optimised the "-{xxx}-" tight loop by replacing function calls such as count() and in_array() with language constructs such as isset(). Reduced execution time from 356us to 275us.
* Cached $varsep_pattern for further reduction to 243us.
* A couple more parseFlags() hacks brings it back to 230us.
* Split out $this->mVariantFlags from $this->mFlags. Rearranged flag detection into a foreach/switch to avoid unnecessary isset() calls. 189us.
* Added a special-case optimisation to generateConvTable() for the case where there are no tables defined inline in the article. 116us.
* Fixed bug from r37499: "!R || !N" is always true since they are mutually exclusive, "!R && !N" was intended (with parser test).
* Fixed E_NOTICE from "-{N|foo}-"

Update js2stopgap.min.js for at least r61137 and r61075

Localisation updates for core and extension messages from translatewiki.net (2010-01-18 22:00 UTC)

Factored MySQL-specific munging out of Language::stripForSearch() to DatabaseMysql. This will also allow other backends to provide seamlessly their own munging algorithms in the future.

* (bug 21936) When a revision has been patrolled, there's now a link back to the article
* Break lines at 80 chars in RELEASE-NOTES

rv bit accidentally committed in r61180

Tweak __call failure text again

removed obsolete search term validation used in validation

Fix stupid r61199, unrelated changes checkin

Add bug #

Strip some microdata attributes when invalid

Style fixes to Sanitizer and Parser

All parser tests pass, no functional change.

* EditPageNoSuchSection hook
* Added message to bad section error for reviewed pages
* A few tiny cleanups

Code clarity tweak

Add some microdata parser tests

Per ^demon, follow-up r61173: remove useless "@return nothing"

Ran stylize.php on DatabaseSqlite

Fixed error suppressing suggested in r58559 by Tim. Also fixed some warnings and bugs in Oracle abstraction.

Localisation updates for core and extension messages from translatewiki.net (2010-01-17 19:30 UTC)

Follow-up to r60744: one more const != $var <--> $var != const switch

Fix some doxygen warnings

Follow up r59331.
*Check for false on $rev->getText()
*Use prefixed title on the messages.
*Add --show-private option to bypass revision delete.

Avoid r60976 breakage of unexpected line breaks inside maybeHelp

Updated link to docs

Use getTitle() instead of mTitle

Might prevent fatal errors caused by r61071, as suggested by code review
there.

Localisation updates Cantonese, Chinese and Literary Chinese

(bug 22122) __call() bug check outputs incorrect version.

Readd line mistakenly removed on r61150

Bug 19996 (backend hooks) Finally adding the four hooks.

Minor tweaks: reduce use of Title Case, consistent use of "IP address", line breaks after sensence for translators.

* added inline section edit to wikiEditor ( using js2 / mwEmbed conventions )
** added "loaders" to "WikiEditor" and UsabilityInitiative core js
** added >= 1.17 script include calls
* some general mwEmbed loader fixes (around loadDone )
* fixed $j.cookie jquery wrapping
* more jsClassLoader clean up.
* unified wgJSAutoloadLocalClasses var

Localisation updates for core and extension messages from translatewiki.net (2010-01-16 16:30 UTC)

Localization update for he.

Followup r61083,r61084: add RELEASE-NOTES entry

Fix for r60809: wrong argument name

MediaWikiBagOStuff -> SqlBagOStuff in comment, both are the same since r55079

* (bug 22061) API: add prop=headitems to action=parse

Patch contributed by Paul Copperman

provided option for showLogExtract to wrap result. Added log for create protected page

Made noSuchSectionPage() use mTitle

Tweaked nosuchsection messages

Bump style version for r61112

(bug 5061) Thumbnail styles in main.css and commonprint.css are too imprecise.
Use the more precise thumbcaption thumbimage and thumbinner classes for image divs on skins chick, simple and commonPrint.css

This just replicates the changes of r18303 on monobook.
Patch by Derk-Jan Hartman

follow up r61101, fix cut-n-paste error

follow up r60832 and follow up r60763

* Don't set Parser::$mTitle to random garbage.
* Remove ParserOutput::$displayTitle, make setDisplayTitle() and
  getDisplayTitle() wrappers for their *TitleText() equivalents.
* Remove Parser::$mDo*Convert member variables, move test for
  $mDoubleUnderScores[] directives closer to the action.
* Remove bogus "global $wgContLang".
* Use accessor to get at $mConvRuleTitle
* Fix up showtitle option in parserTests.inc
* TODO: refactor FakeConverter class away

Localisation updates for core and extension messages from translatewiki.net (2010-01-15 19:00 UTC)

Revert r61078/r61079 for now. Throws a lot of PHP warnings at translatewiki. See CR for details

* add ForkController class to Maintenance group
* removed trailing spaces

Add <guid> to RSS to avoid duplicates

Bug 7346.  Patch by Jools Wills (buzz).

Tabs to spaces in the middle of the line

* use require_once for ProfilerStub.php, GlobalFunctions.php and Hooks.php, these files might already be loaded when reaching this point
* always use the complete path (with $IP) when loading files with require(_once)
* include AutoLoader before LocalSettings.php as we do in WebStart.php
* don't load DefaultSettings.php, it's already done by LocalSettings.php
* define MW_PHPUNIT_TEST constant
* added a description for the file

And IE 5.5 too, while we're at it.

Kill off IE50Fixes. Adds extra clutter to the head, and market share is down below 0.03% (is.gd/6jyWo).

Reference 0.4 XML schema. Lets <redirect /> validate.

Fixed fatal error by defineing a blank function in parent class

follow-up r60811 clean up code, write some tests for the existing uses of HttpFunctions.  Still need to test background download, proxy tests, etc.

* puts jquery computeStyle in try catch ( un-breaks usability / js2 integration )

Cleanup on ancient JS file that's awful anyway

Not only hardcoded browser version checks, but hardcoded browser version
checks *for NN4*.  Probably a waste of time, but oh well.

PHP 5.3.2 might not be out until February, by which time a number of distros will probably be using PHP 5.3.1. Replacing the version number check with a direct bug #50394 check, so that distros or other users can backport the one-line fix without having to upgrade to 5.3.2.

Merge all skins' output of opening <body> tag

This fixes a few minor discrepancies, like Vector outputting dir=""
(redundant to the one on <html>), and non-Monobook-based skins omitting
the capitalize-all-nouns class (!).  This adds Html::openElement() and
refactors Html::rawElement() accordingly, so I checked that all parser
tests still pass.

I wasn't able to figure out if I broke some feature of right-floating
quickbars in the Standard skin, because I wasn't able to figure out what
the feature was in the first place.  Hopefully either it works, or
nobody cares, or someone else will figure out what it was supposed to
do.  (This is the stuff in getBodyOptions() in Standard.php I deleted;
I'm not sure the addition to sticky.js does what I want.)

Remove OutputPage::{get,set}OnloadHandler()

Didn't work at all in Monobook, Modern, Vector, or any derived skin, for
who knows how long.  Only one caller, which was using it to autofocus,
which is a bad idea anyway because it will cause the focus to happen too
late, often after the user has already interacted with the page.  Any
legitimate callers may as well just add the onload in pure JS without
adding an attribute to the body.

Remove body_onload, was always set false anyway

Localisation updates for core and extension messages from translatewiki.net (2010-01-14 20:00 UTC)

* (bug 21303) Comments are no longer stripped from MediaWiki:Common.js and skin-specific JS pages (those message no longer have their wikitext variables replaced)
* Removed a trailing space in RELEASE-NOTES

(bug 19226) First line renders differently on many UI messages
Added newlines before $1 to wrapWikiMsgs of <div>s

Also fixes r61052 utf-8 screw-up on RELEASE-NOTES.

(bug 22034) use wfClientAcceptsGzip() in wfGzipHandler instead of reimplementing it.
$wgUseGzip check on wfClientAcceptsGzip() is removed since HTMLFileCache (the only wfClientAcceptsGzip() user) already checked that before calling it. This way, $wgUseGzip stays applying only for file cache, and $wgDisableOutputCompression for  output handler.

Fix UTF-8 broken on r61052

* (bug 18765) Increased consistency of bold-italic markup for unbalanced quotes.
  The representation of six quotes is now improved (changes the meaning of some dubious markup).

Fixup r61005, seems i failed at applying the patch

follow up r60832: Remove un-necessary test for stringiness since "" converts it.

Update global variable settings in preparation for background, async downloads

Update RELEASE-NOTES to match r61037

follow up r61026 - use Atom by default

follow up r60999 - remove reference to $wgUseNormalUser

Removing the gradient from #page-base

Only advertise RSS feeds by default, not Atom

Advertising both is pointless, since every feed reader in the universe
supports both.  Their functionality is essentially identical.  And users
might be given a choice of multiple feeds to read when there's really
only one distinct feed.  Plus, it cuts a line from the <head>.  The Atom
feed still exists, and you can get the <head> link back if you really
want it with $wgAdvertisedFeedTypes.

* Document a bit
* Fix some doxygen warnings

Don't load nonexistent fix files for non-Monobook

Bug 14717.  I just hardcoded "&& skin == 'monobook'" in wikibits.js.  As
far as I know, no other skin actually uses fix files -- Vector, for
instance, just uses selector hacks in main-{ltr,rtl}.css as far as I
know.  Using separate files is arguably a bad idea, because it
gratuitously adds extra loads, encourages style changes to fall out of
sync, etc.  Plus, happily, the era of CSS fixes seems to be drawing to a
close -- Firefox 3.0, IE8, Opera 10, and all Safari and Chrome get by
fine with no fix files for any skin.

* (bug 21738) Fix for r57119: OutputPage::setSyndicated() now works again (was breaking Special:RecentChanges' feeds, ...)

Refactoring of Article::incViewCount(): use Database::getType() instead of $wgDBtype, and lockTables() instead of constructing SQL manually

Reapply style fixes from r61006 to IEFixes.js

Also fixes indentation problems that seem to have cropped up somewhere.

Localisation updates for core and extension messages from translatewiki.net (2010-01-13 19:00 UTC)

Make 'start' not required.  As requested by ^demon on IRC...

Keep IEFixes separate, load conditionally in JS

Per Code Review suggestion by Bryan on r60994.  Saves about a kilobyte
for browsers other than IE<7 after gzipping.

Release notes for r61008.

Renamed $wgSearchMySQLTotalHits to $wgCountTotalSearchHits, since such functionality also exists in SearchSqlite. No migration needed, as it was added in 1.16.

Upload license preview now uses the API instead of action=ajax

JavaScript cleanup: add spacing, missing braces, use single quotes instead of double quotes where appropriate and fix some comparisons

* (bug 16281) Show copyright system message on special pages

Patch by Jools Wills

add in session download script

follow-up r60832: make sure $t always ends up a Title.

Merge IEFixes.js into wikibits.js

Not actually tested, but it should theoretically work fine.  This
previously wasn't getting loaded for skins other than Monobook, Modern,
and Vector, and each one had separate code to load it.

I also removed <meta http-equiv="imagetoolbar" content="no" />.  This
was added by Gabriel Wicke without explanation as part of the original
davinci.pt back in r2786, in March 2004.  As far as I can tell, it
disables some minor feature of IE6 for some reason.  I assume it can be
removed completely at this point.  If not, it should be readded without
the conditional comment, which is probably unnecessary.

follow-up r60986: Revert earlier test change - philip showed me the error of my ways.

Per CR comments on r59593: use binary transparency for texvc by default. My testing indicates that it looks OK as long as the background colour is around #ccc or brighter.

follow-up r60986. fix for parser test failed.

document a bit

1. Add a new feature to LanguageConverter, for supporting nested using of manual convert syntax like "-{-{}-}-".
2. Fixed a little bug find in getURLVariant().

Fix regression from r60593: The title attribute for links to NS_MEDIA titles no longer contains the namespace as before.

Passed 559 of 559 tests (100%)... ALL TESTS PASSED!