War on wfElement() and friends. Call the Xml members directly, rather than using old wrappers.

* removed useless check for $wgAllowUserSkin since we're already checked for it 18 lines above
* wfHidden() -> Xml::hidden()

Follow-up to r44584: finishing comment.

API: (bug 15579) clshow considers all categories !hidden. Modified patch by Brad Jorsch

API: (bug 16549) Kill a filesort in list=allpages&apfilterlanglinks=withlanglinks by replacing DISTINCT with GROUP BY. Of course GROUP BY page_namespace, page_title would suffice on MySQL, but Postgres will reportedly whine about that being against the SQL standard. Using GROUP BY on all selected fields instead (should please pgsql) and taking care that page_namespace, page_title is on front, which shouldn't cause a filesort is MySQL is smart enough (5.0 is, let's just hope 4.1 is too)

API: Fix broken block expiry calculation in list=recentchanges

Localization fix.

better parser test coverage of {{ns:}}

Localisation updates for core messages from Betawiki (2008-14-10 10:36 CET)

Fix regression in parserTests.inc caused by r44560.

Remove comment

(bug 12645) File histories should be paged

Add some sanity checking. Return nicely if Http::get fails (in general) on fetchImageQuery(), and also if our query returns poorly in findBySha1(), similar to queryImage().

Expand ASC condition as it should be

FindBySha1 now works in ForeignApiRepo :)

Add a new FileCache class to wrap RepoGroup::findFile() and findFiles(), and make wfFindFile() use it by default.  This should improve performance for pages that refer to the same image several times, but the real benefit is that it allows batch file existence checks, à la LinkBatch, by collecting a set of titles (or DB keys) and calling FileCache::findFiles() on them to prefill the cache.
XXX: The code seems to more or less work, but it obviously needs more testing, regarding both stability and memory usage.  In particular, I have not tested file uploads yet -- there may be consistency issues there.

Make boundary inclusion optional in getHistory()

Followup to r44525: remove the broken $flags parameter from RepoGroup::findFiles() too.

Tweak comment

(bug 16635) Add TOC to Special:Watchlist/edit

Add a bit in RELEASE-NOTES about what prompted me to play with the installer: bug 16293, change the PD URL to point at Creative Commons.

GFDL 1.3 is out, need to update the installer appropriately.
* GFDL 1.2 now links to 1.2, instead of 1.3
* 1.3 has been added as an option, 1.2 still Wikipedia-compatible
* The rights codes were tweaked a bit (see RELEASE-NOTES)

(Actually, we can simplify that a bit more.)

Fix interwiki link regression from r44524.

* (bug 16626) action=delete now correctly handles empty "reason" param

Make getHistory() a bit more paging friendly

*sigh* Forgot db->timestamp

Tweak r44534: simplify this

Tweak timestamp handling to allow no lower limit

Localization update for he, and whitespace fixes.

Follow up r44528: Bump $wgStyleVersion

Follow up r44514: Add new message to messages.inc

(bug 1941) Add textarea { font-family: monospace; } to shared.css for Safari

* (bug 16623) Add missing </p> tag in Special:LockDB -- patch by Nakon

Remove it from the comments too. (Who reads those, anyway?)

Remove  parameter from FileRepo::findFiles(), it was fubar and nobody was using it anyway.  (What we really need is some way to pass timestamps to that function, which probably requires further rethinking of the interface.)

followup to r44520: simplify various bits by removing checks now made redundant

(bug 5506) Make Title::isAlwaysKnown() return true for known special pages and files in foreign repos.

Besides fixing the display of the latter in logs, edit summaries and other places relying on Linker::link(),
the change should allow considerable simplification (not included in this commit) of that function, as
well as of several related functions e.g. in the Parser and Skin classes.  To that end, I've also introduced
a new function Title::isKnown() to wrap the common condition "$title->exists() || $title->isAlwaysKnown()",
and have clarified the roles of the three methods.

Need to define $text

Complete rewrite of Special:Statistics:
* Now subclasses SpecialPage like it should
* All of the individual sections were split off into their own methods for readability
* Fixed Brion's comments on code review (r44509): not using <small> in the message, breaking to a new line, and keeping the description message separate
* Since -tooltip is a misleading name for the message, I renamed them to -desc, all message files and messages.inc have been updated

* (bug 2391) A warning is now shown for invalid ISBN numbers on Special:Booksources.
This could be called from the parser as well, but I'm a little hesitant just yet.
Parser tests currently contain some bogus ISBN numbers which need to be replaced with
legit ones... there's also the possibility that some books really are published with
bad ISBNs... on the other hand, Amazon won't let you search for them. :)
Plus the feedback as you're hitting preview might be nice... but it's also not clear
what's wrong -- that is, that it's an invalid number rather than that you just didn't
do something right. Do we want better feedback?

remove no longer needed line for Parser_OldPP (now banished!)

Remove these tooltips in favor of actually putting the information inline.

Manual exporting namespaces aliases from Betawiki (bug 44: renaming of Image namespace to File)
Only added alias for "Discussion Image" (Image talk) since the name of the "Image" namespace is the same in English and French and the alias is already present

svn:eol-style native

Moved the IE content type checks to their own class. Disassembled IE 5 and 6 and added the results to the class. The entry points now return an array giving MIME types for all versions. The most important version difference is the introduction of an early check for PNG headers in IE 7. Added application/x-msdownload to disallowed types, haven't been able to reproduce any vulnerability, but it's better to be on the safe side.

Localization update.

Localization update.

Quick hack to disable import for non privileged users. Prolly a regression from r43963

Follow up for r44497: Rebuild all message files

* Reword some resetpass related messages: It is a password change, always. IMO "reset" is misleading now.
* Rearrange resetpass related messages in messages.inc
* Remove 1 unused message.

Some tweaks for Special:ResetPass
* Use a more appropriate message for 'Retype new password:'
* Add missing CSS IDs for table and rows

(bug 44) Fix omission: Image -> File in namespace names for yue, zh-hans, and zh-hant

(bug 12716) Unprotecting a non-protected page leaves a log entry

Tweak JS focusing

Clarify the $section parameter a bit on Article::replaceSection() to indicate acceptable input values.
Throw in a strval() around the param on compare so if somebody thinks "hey I'll make it a === because it's 5ns faster" again it won't break anything. :)

Fix regression from r44015 "Various minor cleanup/spacing"
Damaged some section editing, or at least display of undo, by causing null section parameters in section replacement to be treated as section 0.

more NS alias breakage

Fix broken NS aliases.

Self revert. Need to focus when there's no results, not only when there's no term.

Per code review. Only set focus if we have a search term.

Throw in $wgEnableSerializedMessages which I can disable on test wikipedia so it'll now show up to date messages. Yay!

Localization update for he.

API: (bug 16594) Use strict comparison in list=allusers. This'll hopefully make [[User:]] on enwiki show up again; he's currently being dropped from the user list because false == ''

Loosen requirements for cascading protection: instead of requiring that both edit=sysop and move=sysop (and any other custom restriction types be set to sysop as well), just require edit=sysop and leave the other restriction types alone. This makes it possible to protect a page with e.g. edit=sysop and move=autoconfirmed and still use cascading protection, but only through the API: the UI won't do this (yet) because the JS greys out the cascade checkbox (that'll need to be fixed).

Fix log list regression and fix funky $allLogs case while at it

Make 'ChangePassword' be the default name form for the password change form instead of ResetPass.
Since it's something we're sending people to with an actual UI, for general password changing, and no longer specific to by-email password *resets*, it's nicer to use the cleaner name.

Consistency tweak: Add horizental line same as exists in Special:RecentChanges

Set focus

Fix encoding

*Disabled hit counter
*Added 'overwrite' option

Reverting r44454 "(bug 14843) IRC should flag redirects with "R" like it does with bots, etc."
This doesn't work as intended; when editing existing articles, this'll pull the *previous* redirection state of the article, rather than the current state, so it's wrong whenever a page is changed into, or away from, being a redirect.

Some more verbose output and fix infinite loop if you make the block size small :)

(bug 14843) IRC should flag redirects with "R" like it does with bots, etc.

Add a script to filecache content pages on the wiki

Pull up and tweak filecache check to make it much faster and able to for request to avoid a db hit if counters are off and job rate = 0

document SetupAfterCache hook ;)

Reduce queries for pagesize thingy

May as well set article ID in title

Fix this broken crap some more

Revert r44381 "Remove noisy profiling"
We... like profiling? :D

Revert r44365 "Cleanup Title::getTouched()"
Rather than a cleanup, this rev seems to be adding a per-Title-object cache of page_touched. I'm a little leery of adding extra caches like this, particularly since it's not clear that it'll get consistently updated.

Revert r44335 "Tweak profile calls"
Let the doUpdates() func get counted even when it has nothing to do; otherwise stats will be skewed to only times when there are multiple updates. Or.... hmmm. bleah.

Revert part of r44296 "*Skip autoloading of skin, always does inefficient strtolower() stuff and fails (returns false) anyway" and r44302 "Tweak r44296: use autoloader if it is set there".
Unless there's some really huge overriding reason to do this, there's no point poking around in the autoloader's guts to try to outsmart it. Just let it take a few nanoseconds to check its array and don't add yourself a dependency on its implementation details.
If there *is* some really huge overriding reason to complicate the code, please provide some data supporting it.

Revert r44292 "Assign by ref to make sure cache fields carry over"
This isn't PHP 4; the object's fields are changed on the object itself just fine. Unless people are changing the global variable $wgTitle to something different this should be fine. If there is a problem that this fixes, that's probably a problem itself and should be fixed in a cleaner way :D

(bug 16589) CoreParserFunctions error causes 500

Revert r44271 "(bug 12998) Weaken DISPLAYTITLE restictions (patch by 'rememberthedot@gmail.com')"
Behavior seems a bit hard to predict, as far as what's going to go in the header and what in the browser window etc. Pulling it back for further testing and discussion.

Configure extension: With DB handler, defer wfConfigureSetup() to a new hook, SetupAfterCache, which, as the name implies, is executed in Setup.php, after caches are initialised.
This avoids some weird issues with Configure and maintenance scripts.

Revert r44257 "Use boolean searching with '?' char"
Let's avoid making up our own syntax that nobody will know or think to try...
Lucene, Google, Yahoo!, and Windows Live search all understand "red OR lion" but see nothing special in "?red ?lion". If we're going to add it, let's make the OR thing work. :)

Revert r44254 "Cleanup to wfGetNull(): * Add override in case /dev/null isn't accessible (bug 11009)"
Doesn't seem like a proper fix here... At best, this'll be dumping random crap to some random file unless the user has a local copy of the /dev/null device file, which seems.... wrong. :)
For anything generating command lines, it probably won't make any difference (assuming exec() is enabled at all!) since open_basedir won't be searching through the command line (I think).
Assuming the core use case actually happens (open_basedir is set, but proc_open() is available to run tidy), a more correct fix is probably to go ahead and read in stderr and toss the results, or maybe better pass it through to PHP's stderr FD instead of opening /dev/null ourselves. Tidy has a -q option which should suppress random "hi i'm tidy version XYZ" if it's currently present.

Attempting to import a title that came out as an interwiki would sort of half-work somewhere then explode when trying to save the null revision with the import info:
Fatal error:  Call to a member function insertOn() on a non-object in /Library/WebServer/Documents/trunk/includes/specials/SpecialImport.php on line 305

Now such titles are skipped. You can probably force import of the page by forcing the target namespace...

Better long-term solution might include title renormalization, adapation of remote namespace names (eg Wikipedia -> Project), etc.

* API: (bug 16613) action=protect doesn't say when &cascade is set but cascading protection wasn't allowed
* Changed Article::updateRestrictions()'s $cascade parameter to a reference which is set to false if cascading protection isn't allowed
* Used this in ApiProtect
* Expand help message for &cascade a bit

Follow-up to r44245, r44250 (comment field on Special:Import)
Label now clickable for comment field on the transwiki form as well as the upload form; gave them both distinctive IDs.

Follow-up to r44216/r44217 "Followup to r44204. Hardcoding image because it should (in theory) work always and forever. If you have a wiki acting as repo and you've overridden the canonical name Image: it's your own fault."
It was noted that the existing code could fail for unusual article path settings including a suffix if only the 'articleUrl' was available. Pretty corner case, and probably breaks a lot of other things, but who knows. :)
Went ahead and refactored this a bit to handle the case and feel a little cleaner to me...
Dropped the private getDescBaseUrl() method since this seems a little easier to handle directly in getDescriptionUrl() now.

Fix regression in Special:Ipblocklist caused by r43047 -- options arrays in show/hide links broken by being merged in the wrong order when moved from wfArrayMerge() to "+" operator.

Revert r44119 "Remove extract() comment"
extract() is pretty much the devil and shouldn't ever be used for safe, legible code because it makes it much more difficult to analyse the code -- you have to run around trying to track down the source of local variables and there's generally a lot of pain.
Removing the FIXME comment isn't a substitute for fixing it. ;)

add debug lines for UDP output so it can be more easily tested offline

revert r44412 and part of r44406: keep the improved new page URLs, but use the old getInternalURL()/preg_replace() hack to generate them

Seems r44406 would've broken the RC feed on secure.wikimedia org: fix by introducing a new config variable $wgRC2UDPScriptUrl (defaulting to $wgServer . $wgScript if not set).
NOTE: Wikimedia's secure.php needs to be updated to use the new variable.  It might be possible to retire the fixupSquidUrl() hack once this is done, though I'm not sure if anything else relies on it.

(bug 4253, bug 16586) Don't repeat titles of new pages in URLs in the RC->IRC feed, use the shorter and more useful revision ID instead.
Also simplify URL construction: old code used to call Title::getInternalURL() to build the URLs and then undo most of the work with preg_replace().
(TODO: Maybe consider introducing new global $wgIRCLineURLPrefix to optionally override the $wgInternalServer.$wgScript prefix?)