Stylise Setup.php

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

Remove ancient comment, collapse if, move USE INDEX into if further down doing the exact same

Followup r80324, add type property description

Start of bug 24650 Fix API to work with categorylinks changes

Addition of a type field for filtering (default is "all" ie no filter)

Adding of type to props, to allow it to be outputted

Remove debugging issue in user preferences

'tototo' message key does not exist that made the message dropped.
Could have used null as a value but it looks nicer to just set
the value.

Per CR on r80281

* (bug 26733) Wrap initial table creation in transaction

bug 10158 : do not mention allowing others to contact you if $wgEnableUserEmail=false

This required to tweak HTMLForm to support an array of message keys
for help message. I could reuse help-message but since this helper
also supports array, I created an additional helper for array of
messages.

To test use variations of $wgEnableUserEmail and $wgEmailConfirmToEdit.

Path adapted from jopiswezggzmw at mailinator dot com

use require( RUN_MAINTENANCE_IF_MAIN ) as per r80205

fix review on r80269

lame benchmark of if/elseif/else versus switch/case/default

if is faster :
$ php bench_if_switch.php --count 20000
20000 times: function bench_if_switch->doElseIf() :
   204.64ms (  0.01ms each)
20000 times: function bench_if_switch->doSwitch() :
   272.98ms (  0.01ms each)

Fix for r80248: undefined variable.

Fix remainder of the accidental ->empty() calls I used when converting wfEmptyMsg into Message::exists() calls.

fix syntax terror.

Implement Message::isBlank and Message::isDisabled.
And while we're at it... update a random assortment of code using wfEmptyMsg to use the new wfMessage class and our exists/isBlank/isDisabled methods.

Fix r79383, I used Message->plain() under the impression it was the proper replacement for wfMsg when Message->text() was actually the correct method to use.

Fix bug 14267 by adding support for a MediaWiki:Mainpage-nstab.
Additionally, *cough* *cough*:
* Add a Title::isMainPage helper for the fairly common $title->equals( Title::newMainPage() ); test.
* Update wfMessageFallback to also accept an array of message keys instead of requiring them listed as arguments to the function.
* Move the bulk of wfMessageFallback code into Message.php instead of leaving it in GlobalFunctions.php
* Change the wfMessageFallback implementation so that the Message class handles the fallbacks themselves eliminating any side effects caused by the fact that wfEmptyMsg always used usedb=false, language=userlang when one might actually use a different language or usedb setting in the message object that actually returned the text (this may be considered a wfEmptyMsg regression in 1.18).
* Make blank "" message contents fallback like nonexistant messages do.
* Re use the new tabAction array handling used to support mainpage-nstab in the talk and view tabs instead of making wfEmptyMsg calls directly in SkinTemplate.

Follow-up r80192, tweak docs

* changed variable list as per comment on r79954 left only wgDBtype
* all other variables get set trough ->getGlobalNames
* could somone test this on mysql, postgres and sqlite ... it should work, but to be sure

Support talk page tab messages like MediaWiki:Nstab-user_talk. Like the MediaWiki:Nstab-main MediaWiki:Nstab-user etc... we already support.

Followup r80205: scripts using commandLine.inc were failing the shouldExecute() check due to extra require()s.

Scripts using this entry point cannot be included standalone for uses as classes, they will always execute.

But then again, if you're not subclassing Maintenance then why are you bothering to write a class in a command line script?

Fix incorrect translations.

Removed redundant padding rule added in r80106.

Cleaned up some comments that were in the wrong places, missing, or misleading. Added more information about the CSS hacks being used for IE6 support. Merged some rules that were needlessly broken up into multiple sections.

whitespace clean up in shared.css

Oops, FF2 doesn't support inline-block display. Restoring that one.
FF2 ignores the latter display:inline-block and sticks to the -moz-inline-box
(Follow-up r78232,r 80214)

Removing redundant line (Follow-up r78232)
<code>
ul.gallery, li.gallerybox {
display: inline-block;
</code>
the display of ul.gallery is overwritten later.
the display of li.gallery is overwriting an earlier one.

Shouldn't change anything other then having one line less

Narrow the <textarea> so it doesn't get pushed below the right navbar

Follow up r80164. Define explicit direction on content and noflip on image thumbs. (like vector/screen.css)

Fix concern raised by Brion in r74108 (but has really existed since the maintenance rewrite). Right now, including a maintenance script causes it to execute. This is bad when you want to reuse the particular class but not have it start executing all by itself.

Until now, we relied on setting MW_NO_SETUP which was a) hacky, b) irreversable, and c) likely to be forgotten if you didn't use one of the wrappers like runChild().

Instead, move the freaky magic to doMaintenance and have *it* check if it's in a specific call stack that indicates this is being run from the file scope and should be executed. Rename DO_MAINTENANCE to RUN_MAINTENANCE_IF_MAIN so it's nice and clear what magic happens behind the require_once().

More hyphen consistency, bug 26685

Fix bug 26034; Make the View / Read tab in content_navigation style tabs (skins like Vector) stay selected when on &action=purge.

Followup r80192 which had an unclosed comment causing some php notices resulting from hidden var statements.

Follow up r75662 and r67090 — use UTF-8 minus instead of HTML entity

Fixing up spaces to tabs in r80192

Removing trailing whitespace

Fix documentation for the SkinTemplateToolboxEnd hook which has a $sk (QuickTemplate) argument rather than the $tools array it says it has.

OutputPage properties documentation

This is following a pair session with Zak Greant on november 12th 2010.
Please review, ammend and commit your changes if you have anything useful
to add.
Please note you will find some humoristics quotes.

Localisation updates for core and extension messages from translatewiki.net (2011-01-13 20:41:00 UTC)

Followup r80122, protected, just incase it's needed..

Mark as needing a Database, as noted in r79532 CR.
It is needed for calling getServerVersion() to provide dbversion

Spacing

Allow to disable the preprocessor cache with $wgPreprocessorCacheThreshold = false

I had this parser comparer in my working copy since the summer.

Improve a preprocessor comment

Coerce mtimes to TS_UNIX before outputting them, and fix a wrong timestamp format in ResourceLoaderUserModule. This should fix bug 26704

Remove rtl modes from Print stylesheet, since ResourceLoader now does this itself.

Fixes bug 26694 and relates to r65923

Convert simple skin to Resource loader.

I removed all of the useroptions generators that were dupe with resourceloaderuseroptions.
The 'highlightbroken' option differs from what is in resourceloaderuseroptions and still remains to be ported.

Bugzilla #26649

Return something too

Fix broken fileExistsBatch() in ForeignAPIRepo since r51673

Follow up r47852. Unlike forkWorkers(), the start() method doesn't get the $procs parameter.

Spaces to tabs

Localisation updates for core and extension messages from translatewiki.net (2011-01-12 21:05:00 UTC)

Fix undefined variable from r79828

61 uses of $this->opts in SpecialContributions, so explicitally defining variable in class

Fix properties in r80112

Fix issue with missing keys in $wgFileExtensions caused by an array_diff() somewhere. Doesn't seem needed on any of the other vars at first glance

Fixed IE bug, you must use html:msg for the msg elements that jquery.localize uses to translate things otherwise IE chokes when you insert them cause it doesn't know about the msg element.

Fixed suggestions positioning in IE6

Added compressed version of csshover 3.11. Also fixed major issue where including it at the bottom with the scripts caused it to be included too late, and thus not even be used. Now we are adding it through OutputPage::addHeadItem.

Fixed positioning of search icon in IE6, and other browsers as well.

Fixed watch icon positioning in IE6. Converted PNGs to 8bit to improve appearance in older browsers.

Per CR on r75575, remove array_keys() as getModuleNames has array_key'd the returned stuffs

Fix key name

Set $mRevision to null by default so we don't need the isset() call

(bug 26688) Proper warning message is not displayed for the blank 'Database host'

(bug 26687) Error msg needs full stop

DB name can have hyphens (fixed in r79767), bug 26685 pointed out the message discrepancy

Fix for r79741, setTargetLanguage and setUserLang want an object and string, respectively. Fixes bug 26686

Fix another typo in r80075

Fix typo from r80075

Rename table variable

Follwup r75575, honour table prefixes. Bad Roan ;)

Followup r75575

reedy@ubuntu64-esxi:~/mediawiki/trunk/phase3/maintenance$ php cleanupRemovedModules.php
Fatal error: Call to undefined method ResourceLoader::getModules() in /home/reedy/mediawiki/trunk/phase3/maintenance/cleanupRemovedModules.php on line 40

Not sure why I originally used array_key_exists(). Switch for isset()

Follow up r79845. No need to keep $wgCustomConvertCommand there.

Document return type.

Explicit class member variable definition

initFromQueryResult doesn't use the $db parameter.

Private method, updated callers to not pass it.

Follow up r80025.
The space is actually used inside extensions, such as Maps, covertly called as 'display_map' into
Validator, then converted into 'display map' by getName().

Fixed logic error - all tests were passing... :(

Reverts part of r80036 - jquery.client is already depended on by mediaWiki.util - which is laoded initially as well.

A wfProfileOut was missing (which messed up profiling sometimes).  Thanks to Owen Davis for finding this.

Fixed logic error that caused the test to always pass.

Included client in the initial payload. Restored functionality that used to exist in this plug-in that adds classes to the document's HTML element for browser and platform information. This will help reduce CSS hacks to only be used for non-JS rendered items.

Repair quickbar of standard and cologneblue skins to work under Resourceloader.
Refs bug #26649

Factoring this out into seperate stylesheets seemed silly. Instead just moved
that last css line from skins/common/quickbar.css and skins/common/quickbar-right.css
inline as well.

Remove "Heldergeovane" from both PT and PT_BR message files, per request on TW

Renamed user already exists as "Helder.wiki"

Forbid '<', '>', ' ', '\n', '\r' in parser hook names.
Registering a tag hook with < or > on its name resulted in a UNIQ marker.
Spaces, tabs or LF do work (CR fails due to the CR->LF transformation),
but it's easier to also forbid them, just as they are not allowed in other
markup languages.

Add tests for parser tag hooks.

Fix a few doQuery() -> query()

Fix callbacks for the final time so it actually works per documentation.
Other minor postgres fixes

No space after unary minus

Spell false with small latter

Use same wording as in skintemplate

Fix copy+paste mistake from r79989

Fix callback issue noted in r79989 (broken in r78774).
$this makes a difference.
I'm an idiot.

Followup r80003: array_unique() preserves keys, so we got module arrays that looked like {"0": "ext.liquidThreads", "58": "mediawiki.legacy.wikibits", "59": "mediawiki.util",  ... } . To prevent this from happening, use array_values() to reindex the arrays

rm stray tab

Apply array_unique() to module lists in OutputPage, should prevent lots of duplicate 'ext.liquidThreads' elements in the module array reported by thedj

Seems there was a logic error in r80001 :)

Simplify a bit:
* replaced calls to Article::getLatest() by Title::getLatestRevID()
* introduced Skin::isRevisionCurrent()
* made checks on Skin::pageStats() more coherent

Follow-up r79964: Use the existing message 'parentheses' instead of introducing an identical one.

I have no clue how the callbacks work, this at least sets up a database, everything after that is fubar

Parent in DatabaseInstaller is a CoreInstaller, not just an Installer