Changed some magic __get method calls to accessors, were throwing notices

Moved action=deletetrackback and action=markpatrolled to Action class.
Also fixed order in DefaultSettings.php

safer check if mhash is available (and I'm about to make it unavailable by disabling the fn in the php.ini for lucid builds)

Update Chinese conversion tables.

Remove resize/arrow cursors from jquery.makeCollapsible
* In Safari they look nice (a visual clue which direction we're heading on-click) but on other browsers there is only a general arrow-icon that points both up and down, and other browsers have an even weirder cursor for this. Summarized: It's inconsistent and redundant with the text-label we already have.

(Poke r79083 CR)

Remove $wgCheckCopyrightUpload from DefaultSettings.php since the feature was removed about 5 years ago

Looks like the feature was partially removed back in r10168. In any case, its long gone out of trunk now.

sqlite.php: don't fail after --check-syntax on a non-SQLite DB

Fix jquery.tabIndex even more.
* Discovered through TestSwarm. Aside from IE6/IE7, which already needed extra checking, when IE6/IE7 is on Windows NT 5.2, it can also return NaN.

mediawiki.util.test.js: IE7 doesn't expose the CSSStyleSheet publicly (like it doesn't for many). Checking object type instead.

And I forgot to commit this file in r89874

* Made Special:RevisionDelete use the local variables stored in the RequestContext object instead of global ones and removed the $skin member
* Also updated classes in includes/revisiondelete that were using the $skin member of Special:RevisionDelete
* Handle action=revisiondelete (or rather action=historysubmit&revisiondelete=1) with an Action object that sets the RequestContext object of the SpecialPage object, and in this case get the target Title object through the context instead of $wgTitle

Removed usage of error suppression operator, also removed function_exists() check for timezone_open() since we now require PHP 5.2.3

Moved part of action handling functions near other ones

Followup r88873, remove setting of $wgEnableCreativeCommonsRdf in LocalSettingsGenerator

Changed more magic __get method calls to accessors, were throwing notices

RELEASE-NOTES for r89866

Upgrade jQuery from 1.4.4 to 1.6.1

Poke r88725, r88607, bug 28904;

Follow-up r89853: Documentation for completenesstest + minor fixes/whitespace

Add missing language name

Follow-up to r75241 / 75241: rename LanguageBat_smg to LanguageSgs

Fix for r89821, r89839: we can skip certain tests if the web user is the same as the install user.

Added jquery.qunit.completenessTest.js (A jQuery/QUnit test coverage utility)
* Added to /resources
* Conditionally loaded (condition being that the url parameter "completenesstest" has a truthy value)
* Fixed a test that was using === and true
* Setting an added method somewhere back to undefined so it won't be listed as a potential "missing test".

Follow-up r89835: Accidently comitted from a deeper dir than the diff. Comitting the rest now

Follow-up: r89845: Forgot to commit changes in the other directory

Review of unit test suites
* There's more than ok(), equal() and deepEqual(). Using the others as appropriate.
** Considered: strictEqual(a,b) tests the same as ok(a===b). But the latter doesn't include the values in the report when it fails. So strictEqual saves a lot of time in debugging (especially on TestSwarm where the report is all you have, there "not okay" or "Expected { foo: 500 }, Given: ['bar', '250']" is a big saver.
* Adding an "expect" to every test.
* Applying whitespace conventions and bringing consistency in the use of mw, mediaWiki, $, $j, jQuery. (except in the initial test where the aliases are being checked)
* Added cleanup for added CSSStyleSheet
* Making IPtest more complete (Based on IPTest.php)

Make Pg installer work in the “common” case: follow up r89821 so that $exists can be set true if same user is the same for install and web.

[Installer] .mw-help-hint and #config-live-log
* Merged padding rules into one
* Fixed the width issues of the textarea (see also r87975)

Localisation updates for core and extension messages from translatewiki.net (2011-06-10 16:40:00 UTC)

Simplify link construction a bit.

PostgreSQL install fixes:
* Made PG throw a DBQueryError when it gets a query error, instead of DBUnexpectedError. Apparently this mistake goes back to r14625, when exceptions were first introduced. Did it by removing reportQueryError(), the DatabaseBase version works fine.
* Fixed several places where there was an attempt to check for a query error by checking if the result of query() was false. This never worked. Used try/catch instead.
* Made the DBConnectionError messages go on one line so that they don't mess up the formatting in the installer.
* In DatabasePostgres::selectDB(), only disconnect and reconnect if the DB name is actually changing.
* Made DatabasePostgres::schemaExists() less weird and scary.
* Added DatabasePostgres::roleExists() for use by the installer.
* Removed the PostgreSQL-specific hack to make _InstallUser have a default other than "root". Made _InstallUser into a proper DBMS-specific internal variable instead, since every DBMS we support so far needs a different default.
* Removed the $dbName parameters from openConnection/getConnection, and got rid of $this->useAdmin. Implemented a more sophisticated caching scheme instead. Partial revert of r89389 and r81440.
* When connecting as the install user before DB creation, and when testing the web user's credentials, try a few different database names and use whichever one works.
* Instead of connecting as the web user to create tables, I used SET ROLE. It seems cleaner and more like what the other DBMSes do during installation. "SET ROLE wikiuser" requires the same privileges as "CREATE SCHEMA ... AUTHORIZATION wikiuser", so it's unlikely to break anything.
* In the area of web account creation, fixed various minor logic errors and introduced more informative error messages at the submit stage, pre-install. Show a helpful error message if the web user exists already and the install user can't do the relevant SET ROLE.
* Split schema creation out to a separate install step.
* When creating an account as a non-superuser, add the administrative account to the new account's group. This is necessary to avoid a fatal error during installation (bug 28845).
* Removed code which alters an existing web user to have appropriate search paths and permissions. This may break other apps and is not necessary. As in other DBMSes, If the web user exists, it is the responsibility of the sysadmin to ensure that it has appropriate permissions.
* Rewrote setupPLpgSQL() to use the query builder functions.

Fixed typo in comment.

Fix for r86741 which broke PG update due to the math table disappearing

Fix for logic error in r89529 causing a notice.

Reverted r89393. A single Database object certainly should not be attempting to manage multiple database connections, that is the job of LBFactory/LoadBalancer. I would like to see $wgSharedDB managed by LBFactory instead, for all DBMSes. Then the cruft in Database::tableName() can be removed.

r89393 caused tableExists() etc. to be completely broken, so PostgreSQL upgrade was broken too, see CR.

(bug 29332) don't let install progress if user requests mediawiki-announce subscription but does not enter email. Thanks Sumana :)

Self revert r89797: At least the block log link is wrong. I should not commit when bed calls me :/

r89767 didn't please overlords

Simplify link construction a bit.

Marked Article::doRedirect() as deprecated, was used by Article::insertNewArticle() and Article::updateArticle() that were removed. No usage left in core or extensions.

Localisation updates for core and extension messages from translatewiki.net (2011-06-09 20:44:00 UTC)

Recommit SpecialPage.php and SpecialPageFactory.php parts of r89406 (removal of magic __get() accessor for RequestContext)

Change whitespace so it is easier to read.

If this was ever passed an empty string it would spit "PHP Notice:  Uninitialized string offset: 0..."

Removed some usage of error suppression operator

MFT r89770

Attempt at fixing the issue raised in r82948.

* merge r89759 from REL1_17 into trunk
* changed addQuotes expected result for addQuotes in DatabaseTest

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

followup r88946 to add $1 to message

W/S fixups

Revert r89374, r88936 backports.

* fixed regex to work
* added missing directories

Localization update for he.

Revert r87145, bug 28752: Xcache doesn't work in cli mode. As pointed out on CR, this didn't fix it, it just hid the issue.

Revert r87173: adds 'wrap: break-word' which seems opposite to what pre blocks are meant to do.

Follow up r89707. No need for the explicit concatenations.

quick fix for bug28983 . Do not use $path in the loop. Even the remaining $e is dangerous subject to change from the require-once-loaded extensions. This is NOT A FINAL fix, just a small improvement

Reinstate r79122 (fix for bug 14404), reverting r83868. The real bug seem to have been r86131, fixed in r88902 (1.17) and  r88902 (1.18).
This is not merged with the r86131 change to Article::getParserOptions() since I don't see the point for the new function yet.
Reenabled its test ArticleTablesTest which was disabled in r85618

Fix (bug 29307) Line-height in <pre> is too narrow, edges overlap in some environments

Kill the extra indenting, and fix trailing whitespace

Revert r87292

Follow up r85410 & r89688. Try to make clearer that this is a message key.

Doc fix for r85410

Localisation updates for core and extension messages from translatewiki.net (2011-06-07 20:35:00 UTC)

Followup to r86061: add parser test case to confirm that '&apos' in wikitext doesn't make it to output.

'&apos;' is not defined in HTML 4, and some browsers (for instance IE at least through 8) do not support it.
It is however part of XML 1 (and thus XHTML 1), and is also defined in HTML 5 provisional, so it's fine to accept it.

Normalization seems to already handle turning the &apos; named char reference into &#39; so we don't have any compatibility worries.

Also added a comment on the def in Sanitizer that it shouldn't be used in output.

remove useless type hint (fu r85004)

Forgot to press save on the comment

Followup r87347, bug 22227, use makeTitleSafe.

Fix (Api)BlockTest for Sqlite. getRangeStart()/getRangeEnd() were returning null for user blocks, which was then happily inserted into a NOT NULL column. The IGNORE was just masking the real error. Thanks Brion and MaxSem. Oh, and ping r84475

Added phpunit test cases for r87497, r88076: ResourceLoader module name prefix collapsing

Provisional revert of r89648: "Another try at fixing bug 93 "tilde signatures inside nowiki tags sometimes get expanded (<includeonly><nowiki>~~~~</nowiki></includeonly>)""

This seems to disable PST for <includeonly>...</includeonly> in addition to the <includeonly><nowiki>...</nowiki></includeonly> that the bug is about.
This was caught by a parser test case, but the test case was changed to match... seems like the wrong change?

Also failed to update Preprocessor_HipHop.hphp to match the other two preprocessor classes.

Get rid of addNewExtension()/getNewExtensions() method of adding new extensions (added in r81266). Since r85021, this isn't necessary because we run old updates like we should. Fix the 3 extensions using it. Reverts most of r81266, all of r84868, tiny part of r86741. Also ping r89653 which is what caused me to look at this again.

Fix r85021: extension tables weren't created in some cases

Another try at fixing bug 93 "tilde signatures inside nowiki tags sometimes get expanded (<includeonly><nowiki>~~~~</nowiki></includeonly>)"

* Change the preprocessor to insert strip items for <ignore> nodes during pre-save-transform, just like <comment> nodes are handled already. This effectively disables all pre-save-transform steps inside <includeonly> tags.
* Adapt parser tests to the new behavior.

Fix broken r89643.

Add GENDER support for 'lockedbyandtime'. Suggested by Purodha Blissenbach.

Fix error noticed in r88174, wasn't actually validating sha1base36 hash

* Fix for r89624: fixed logo HTML escaping

* Restored the page shown when LocalSettings.php is missing to how it was before r85918. That template is a welcome page, not a "catastrophic error".
* Fixed attempted remote loading of the logo image in index.php's wfDie(). Use the distributed mediawiki.png instead, that's what it's there for.

* fixed table duplication for unit tests
* fixed sequence handling when using DB prefix
* deferring constraints on transaction, switching back to immediate on commit/rollback
* hardcoded NLS_NUMERIC_CHARACTERS as it breaks using floats in sql if connection is in a non-US-like NLS_LANG (all vars are bound as chars)

When detecting $wgServer, do not fall back to $_SERVER['HTTP_HOST']. It's unlikely that this is used by anything, since SERVER_NAME takes precedence, and SERVER_NAME is required by CGI 1.1 and appears to always be set by the major web servers. If it were ever used, it would open up a cache-poisoning vulnerability. Partially reverts r8010.

Partial revert of r85022. Removing added auto-margin that changed behaviour.

(bug 25124) Resource loader should respect $wgStyleDirectory
* Yay, one down, one resourceloader 1.0 blocker to go

Alright, IE7 is fixed by r89613. IE6 stays problematic, for some reason it contains non-nodes as well (presumably whitespace/textnodes). This oughta fix it forever (poke r89605)

Redo r89605 in a better way.

Follow up r89585. Clean up after the test, expand inContentLanguage() comment.

Fix bug that brakes the 'jquery.tabIndex > firstTabIndex' test in IE8/IE9. Some values default to a "unknown" string.

Localisation updates for core and extension messages from translatewiki.net (2011-06-06 20:43:00 UTC)

Fix bug that brakes the 'jquery.tabIndex > firstTabIndex' test in IE6/IE7 (Thanks TestSwarm).

Release notes for bug 27655 (r89545 and related revs)

jquery.client.js JSPERF+Unit test suite
- Using one closure function instead of two
- Comparing to undefined instead it's type (faster, [[mw:JSPERF]])
- Adding basic test suite

test thead / tfooter contains th elements

Implemented by r85922. Cover r85943.

TEST PLAN:

$ php parserTests.php --regex 'Table have th inside t'
This is MediaWiki version 1.19alpha (r89594).

Reading tests from "tests/parser/parserTests.txt"...
Running test Table have th inside tfoot... PASSED
Running test Table have th inside thead... PASSED

Reading tests from "tests/parser/extraParserTests.txt"...

Reading tests from "../mwexts/LabeledSectionTransclusion/lstParserTests.txt"...

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

add credits to bluehairedlawyer for thead/tbody

(bug 4740, r85922)

;

Revert r89537 till we decide a sane way to have it on/off as API bools suck

Follow-up r89586: actually delete the file

Revert r89263, r89297, r89347: unclear whether this is needed; DatabaseUpdater actions list is already extensible and it might just need some rearrangement?

Fix for bug 29274 - Message class ignores $wgForceUIMsgAsContentMsg
Also added tests, which pass

* (bug 27716) Make a method to do checking of 0 or 1 of the parameters existence (like requireOnlyOneParameter), but without needing one of the parameters

Followup r89542, validate hashes