coding style tweaks

Found an error when $res = false while running tests

Don't use dirname when you already have a directory.

Improves on r77693 by placing ResourceLoader "only styles" CSS links together with the other CSS links in buildCssLink rather than getHeadItems which is used for meta information.

Filenames with quotes fail here.

Improves on r77693 by ensuring site styles are loaded just before user styles.

compress images with pngout for a couple more skins

a few more images compressed with pngout

more lossless PNG compression with pngout

Localisation updates for core and extension messages from translatewiki.net (2010-12-07 20:38:00 UTC)

Typofix r77999

losslessly compress images with pngout

Per CR, no need for an empty constructor

Remove the $wgCaches[CACHE_DB] = false; line added in r74552.
It produces an strange failure when going out of ParserTest::setUp() scope.
(eg. $parserMemc becoming back false instead of a FakeMemcached, related to
using a reference in $wgMemc = &wfGetMainCache() )

Can be checked when running phpunit.php includes/ParserOptionsTest.php
See https://bugzilla.wikimedia.org/show_bug.cgi?id=26244#c4

No errors found in sqlite without it.

Per CR, fix debugging code from r70835

Logic error from r77999

References causing Bug #26244?  This should find a way to just use
what Setup.inc already does.

Follow up r78003, forgot to make MonoBook use the footericons code.

Commit getFooterLinks and getFooterIcons helpers to abstract the common code used to trim footerlinks down to only valid items, flatten it if needed, trim non-image icons out of footericons, and remove the copyright icon if needed.

Also ditch PHP  thing. We would've bailed for a bad version already

Remove various "success" messages from Welcome screen on installer, no need to confuse the user with extra info :) Now we only show warnings and failures

Followup r77980, for simplesearch I guess we'll have to stick with type=text instead of type=search.

follow-up r77981: Fix class: mw-js -> mw-css for the CSS section
Add comments with the full message keys to make sure grep will find them

Follow up r77893, add BaseTemplate to autoloader.

Follow up r77884.

Per Nikerabbit, follow-up to r77972: use a string instead of boolean for readability

Reduce code duplication in Vector by combining portal markup generation into a single method.

Follow up r77981

Show the source on preview for CSS and Javascript pages.
Add pages in MediaWiki namespace into that group (bug 10871).
Only normal <pre> formatting is being used since ShowRawCssJs
hook directly sends things out to the output.

Add makeSearchInput and makeSearchButton to BaseTemplate and make use of it in Vector and MonoBook to abstract away the various inputs inside search forms.

Remove PHP comment for message, parameters are documented in qqq

Bump $wgVersion to 1.18alpha. RELEASE-NOTES and HISTORY should be poked at too, but I'm not burning my hands on those

* Don't unset $wgTitle if it's null, just let it as is so that the variable is always defined (it is already set to null in in Setup.php). It was added in r12612 when all the code that set $wgTitle was moved in MediaWiki::checkInitialQueries()
* Removed comment that documented a line removed in r45126

Merged wfDoUpdates() and MediaWiki::doUpdates() in wfDoUpdates(); avoids code duplication

$fname -> __METHOD__

Spelling error from r77955.

(bug 18372) $wgFileExtensions will now override $wgFileBlacklist

Modify config-install-done, hopefully this is what Chad was looking for in CR r77948. Added a large, obvious download link with icon from the Crystal Project.

Fix breakage from r75542: FakeMaintenance needs to set at least $mSelf so when runChild() calls loadParamsAndArgs() it doesn't pass nothing and thus try to fall back on $argv. This breaks the web updater, where that is undefined.

Followup r77949, qqq message no longer relevant

Address fixme on r67621: use $wgDefaultSkin instead of hardcoding vector
Would like a UI to choose skin with like previews or something, but its not urgent

Typofix r77949

Followup r77883, rm the _ExternalHTTP stuff as well; cleans up IC msgs as well

* Pop up a download box for LocalSettings.php when the user goes to page=Complete. Committing for review: I'm not wedded to the concept, but we do need some way of making sure users download that file.
* Fix MIME type on LocalSettings.php download to not be something that IE considers ambiguous. In Firefox, this makes it say "which is a: PHP file" instead of "plain text document" in the download dialog box.

Fix r77650, notice about undefined $css; was causing the serif font issue too

Fix incorrect use of wfMsgReal() from r77929, causes several warnings.

* Fix issues brought up in r75832
* Really and truly add in a couple files that were supposed to go in with r77898, and fully revert r77860 (which was a reversion of r75832). (Ran previous commit for r77898 from wrong directory. Oops.)
* Fixed issue with help box text not being parsed.
* Modified help system to remove tooltips and instead open/show hidden boxes.
  - I am not happy with this design and future revisions will iterate upon it.
  - The biggest problem is everything sitting "on top" of itself.  I dislike this a great deal and plan on changing it to be more aesthetic and natural; however, I wanted to get a version of this in that allowed for cut-and-pasting and solved the text parsing issue.
  - With javascript disabled, help text is visible by default.
* Remove redundancy in checkboxes
* Added i18n entry for "help", which wasn't there for some reason.
* Removed references to tipsy library in installer html
  - Kept tipsy library in core; this is being used elsewhere.

(bug 26265) Many semicolons are missing from JavaScript files

And some more unterminated statements..

(bug 26265) Many semicolons are missing from JavaScript files

And some more unterminated statements..

(bug 26265) Many semicolons are missing from JavaScript files

More unterminated statements..

(bug 26265) Many semicolons are missing from JavaScript files

Unterminated statements..

Rm two unused vars

Change monobook to vector because vector is the default skin now
Per suggestion of The Evil IP address: http://translatewiki.net/wiki/Thread:Support/userinvalidcssjstitle

Improves on r77710 - no purpose in using call() here.

Added center option - will pass this upstream.

Revert unintended changes in r77908

(bug 26130) Revert changes to WebStart.php in r72349, which turn out to have been misguided. This should fix double-gzip issues

Localisation updates for core and extension messages from translatewiki.net (2010-12-06 20:22:00 UTC)

Revert r77860, which was a reversion of r75832.

Didn't see comments about escaping html and use of tipsy; will be addressed in next commit.

Part of fix/workaround to  bug 26163. API calls to get language links and categories as html

Reverting r77516. It ain't gonna work

More skin system improvements; Create a new BaseTemplate class extended from QuickTemplate, this class will house helper methods that interact with QuickTemplate data set by SkinTemplate to help cut down on boilerplate for common uses of that data.
For the first helper in this class add a helper to use in generating a toolbox. Include a list item and link generating method that can be used to generate most of our sidebar links with minimal code.
Eventually this toolbox code will be used in code abstracting common sidebar generating code away into common helpers. For now make MonoBook and Vector and the skins based off of them use this method of generating toolboxes and sidebar links.

Check for the "search" along with other checks to create the Title object. Since this was done after the call to $wgOut->setTitle(), the Title object holded in that instance was incorrect when a such request happened

Per Hashar, r77819, @private comment. Explicit public method definition

* (bug 26253) Removed $wgPostCommitUpdateList

As I said on the bug, this variable is not used since ages

Added localize registration.

Added fixes for tipsy font size - because tips are rendering outside of the body their text size is not affected by rules for bodyContent where the text is usually adjusted up or down a final time.

Added a light shadow to tips.

Cleanup r77883, contained unrelated debugging change

Remove "checking for latest version" feature from the new installer. Originally a proof-of-concept, and I'm not satisfied with its state of completion enough to include it for 1.17. Reverts r71107, r71110, r71111, r71115, r71564, r65863, r57624, probably some others.

Fix fatal error when admin name is blank

Remove some of the rubbish that has been accumulating in the default LocalSettings.php for years, especially code as opposed to configuration data:
* Don't set $IP. The entry point already sets it, so there's no point. Users can still set it if they can figure out a way to do it without breaking things.
* Don't include DefaultSettings.php. Doing that is code, not configuration, and it can easily be done in WebStart/Maintenance. Some non-standard entry points in extensions may be broken by this. That's their fault for being non-standard. Backwards compatibility is preserved thanks to require_once().
* Introduce $wgInvalidateCacheOnLocalSettingsChange, which when set, causes $wgCacheEpoch to be updated in the way that it previously was in the default LocalSettings.php.
* Don't set $wgLocalInterwiki to some nonsense value (possibly including spaces and punctuation in the new installer). It should be only for actual interwiki prefixes. Since most wikis don't have them, I set this to false by default and adjusted the referring code to accept this.
* Removed the guard for web invocation with $wgCommandLineMode set. This ancient code is redundant with modern protections in doMaintenance.php.
* In DefaultSettings.php, fixed fold terminator placement near $wgLoadScript
* Add a web entry point guard. That's one piece of code which really is necessary.

* Reformatted several text documents to be a hybrid between wikitext and plain text, so that they look nicer in the new installer.
* Removed the specialised getFileContents() functions, they are unnecessary now that I have reformatted COPYING into wikitext.

In WebInstaller_Document::formatTextFile():
* Introduced new tab-indented syntax, which is equivalent to colon indents in ordinary wikitext. This was useful for reformatting the GPL.
* Removed the special case for spaces preceding URLs, used tabs instead, with the above feature.
* Removed the "numbers in square brackets" feature. The place where this was used, in README, was an unfortunate case where the square brackets originally indicated a footnote and its reference. Then another two "footnotes" were added with no corresponding reference, they just talked about random license-related stuff that happened to be similar to the footnote. It was never actually meant to be a numbered list. Fixed in README.
* Convert from DOS line endings to UNIX early on, to simplify the regexes.

Adding reference to bug 26267. Follow-up on r77123

* Made it possible to run multiple installer instances in the same cookie domain without them overwriting or misinterpreting each others' sessions. This is mostly paranoia: one could imagine a situation where the installer from one version of MW becomes insecure when run on the session from another. Did this by making installerData an array indexed by a hash of path and $wgVersion. I could have used $wgCookiePrefix, but that would have broken NoLocalSettings.php, which has no access to AutoLoader and so can't access Installer methods.

* Moved most of the functionality of config/index.php from the file level to a function, so that we can have local variables.

bug 26265, a few explicit ; at end of blocks

* (bug 23119) WikiError class and subclasses are now marked as deprecated

Last usage from those classes in core was removed in r77800.

Follow-up r77864: Add new measage key to maintenance script directly helps to not being pinged :-)

When opening the subprocess fetchText.php, pass the complete wiki ID through the --wiki parameter so that it works for all setups and not only for wikimedia

Add a Skins category to Special:Version.

* Update NoLocalSettings.php for the new installer. Stop advising users to move LocalSettings.php from config/, the new installer doesn't create it. I imagine I'm not the only one with copies of LocalSettings.php lying around everywhere.
* Fix the installer session name to be something that makes sense, instead of my_wiki_session.

Reverted r75832 per my comments on CR, unanswered for 19 days. Moving all help information into tooltips is a step backwards for usability. Reverted followups r75836, r75866, r76314 and r76315.

Fix bug causing rebuildFileCache.php to create inappropriate cache files for redirects. (Actually, it would be better to create _correct_ cache files for them instead, but this doesn't seem to be something the file cache was designed to do, so I'll leave it for later.)

followup to r77843: fix off by one error

Add new config variable $wgFileCacheDepth to set the depth of the subdirectory hierarchy used for the file cache. Default value is 2, which matches former behavior. Setting this to 0 makes the paths simpler, and can be used with clever Apache rewrite rules to serve cached files directly without calling MediaWiki or PHP at all.

Follow up to r77411. Patch by DieBuche

1. Adds a line css to actually fix Bug 3770 (Sorry, I had forgotten to paste it
there as well)
2. Fixes a problem, where the box size would be slighty higher if the image
thumb has a height of less than 8px
3. Normalize the calculation of the vertical padding to be exactly the same one
as the horizontal padding. (This is only relevant for unusually large
galleries, eg. previously it returned a not-exactly-square box if you specified
500px as both width and height)

Localisation updates for core and extension messages from translatewiki.net (2010-12-05 19:53:00 UTC)

This can be private too

Fix copy+paste error from r77778

-rakkaus/#mediawiki-i18n- [05-Dec-2010 18:22:39] PHP Fatal error:  Call to private method CategoryViewer::getHTML() from context 'CategoryPage' in /www/w/includes/CategoryPage.php on line 74

Partial revert of r77809, remove making of methods private, but don't reinstate the comments. They're seemingly wrong, and useless.

Swap @private for setting function definition to private

Change some member calls to static, from r75709

Follow up r75705 typo.

Converted ImportStreamSource functions to return a Status object rather than ImportStreamSource-or-WikiError

This is a breaking change since the ImportStreamSource object is in the value member of the Status object; I will fix the extensions in my next commit

Converted WikiError to MWException so at least users can see the problem (nothing was checking the return value of WikiExporter::dumpFrom() and realted methods)

(bug 26228) On certain PHP 5.2 installs, finfo functions can be available while FILEINFO_MIME_TYPE is not. Check for the latter explicitly

Fix typo in r77714

Followup r77793, documentation for icon as a string was missing.

Make footerLink a public method, this method is useful for hooks now that r77732 allows a hook to add new footer links in this style.

Folowup r77763, add documentation for $wgFooterIcons.