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

Fixups to r81266

(bug 26857) Fatal error during installation enabling 'Vector' extension (or any extension, really). The reason we're including extensions during setup is so they have a chance to register with LoadExtensionSchemaUpdates. Before including extensions, we now include DefaultSettings.php, so any operations on those variables won't result in the fatals described in the bug.

More importantly, this adds support for installing extension tables *at* install time. Previously, the best we could do was to add the require()s to LocalSettings. However, without a subsequent update, wikis will probably start having errors immediately after install (this is really bad!). So I've added an interface to the DatabaseUpdater passed to LoadExtensionSchemaUpdates. An extension can now call addNewExtension() which will allow the extension to be enabled on install and not just update. Example committed in CodeReview

Wrapping core modules (FIXME from r79929)
(Touch: r74088, r73046)

Follow-up r81209: Use the same param detection method as WebStore, so that thumbnails with different mimetypes than the original file are thumbnailed as well.

Add a fairly pants TRUNCATE vs DELETE benchmark...

Swap "or" to ||

Stupid references, stupid PHP. I blame Nikerabbit (fixing r81254)

Remove a bunch of useless $wgOuts. Just pass them like we should. Also make them protected so people don't call them

(bug 27046) Do not strip newlines following C++-style // comments. Prior to this commit, such newlines were always stripped, causing invalid JS output in certain cases. They are now always preserved, even if they would normally qualifiy for vertical whitespace stripping/collapsing when wgResourceLoaderMinifyJSVerticalSpace is enabled.

Cleanup for r66268, r66267: merge WikiImporter back into Import.php, where it started. It doesn't really make sense to have a class called WikiImporter in a file called ImportXMLReader.php, and a few little helper classes for it in a file called Import.php.

Localisation updates for core and extension messages from translatewiki.net (2011-01-30 22:23:00 UTC)

Remove usage of $dbw->escapeLike in RenameDbPrefix (per bug 270042)

Tweak install steps to be on the POST rather than GET. It makes more sense architecturally. It also gives us one last chance for them to say no (in case they clicked "I'm bored already" but changed their minds). Check final step of installer for fatals, and if everything's good, suppress the back button. It's just going to confuse people. And the opposite, suppress continue on fatals...no need pretending things went dandy if they didn't. It's basically now impossible to re-run 'page=Install' after it completes. This is probably a good thing as it throws errors. Coincidentally this also fixes bug 26947.

(Bug 26961) hide anons from watchlist preference was broken due to typo in
r39314.

Fix by duplicatebug.

Given that this preference has been broken since it was introduced,
and the only person to notice in 3 years has been the testers, that may be
a good argument to remove the preference altogether.

As per my comments on bug 27038 and my comments on r75906, use a /thumb and /file prefix for files fetched via Special:UploadStash. Make filenames more consistent with regular thumbs by using the /thumb/$fileName/$thumbName scheme. Now uses MediaHandler::parseParamString so that params like page are properly handled.

Follow-up r80775: Check for meta.tiff as well.
Bump wgStyleVersion

ParseMaster is designed to use big regexes, with many starting characters.
It makes sense to pcre_study() them.

A few return type hints

Per fixme on r78786, obey sortDescending in Cached mode (ie if sortDescending, force order by qc_value DESC)

(bug 27021) Session key was changed somewhere from an int to a string, which broke stashed uploads totally. Changed $wgRequest->getInt to $wgRequest->getText. The API never explicitly defined it as int, so should not be affected.
Note that this also may break API clients in certain cases for strongly, statically typed languages (e.g. int sessionKey = request->sessionKey;). I see no way around this, so I will announce a breaking change on the API mailing list shortly.

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

* (bug 22606) Follow-up r63059: don't send the "someone registred an account" message when setting email address (i.e. old one empty) in user preferences

* (bug 27016) Fix for r77638: dumpTextPass.php now consider the "output" parameter

Also added "output" parameter to the doc since nothing was mentioning it

fixing php Parse error: syntax error from r81169

Removing more, no longer needed, overrides for the white table background  (which is no longer the case since bug 26449 was solved in r79087 and r79091)
* table#mw-search-top-table (r52669 / 55748)
* table.allpageslist, table.mw-allpages-table-form, table.mw-allpages-table-chunk (r49500)
* (r32269, r55393, r78865)
Thanks to Fomafix!
See also r80495 CR

Update comments to take into account r80443 and r80614 changes, per CR.

Follow up r81100. Don't hardcode hiddencat quotes, per r80936 CR.

(bug 26938) "wikipedia compatible" should not be part of the license name. We already explain this in the help box, there's no need to repeat it here

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

(bug 26939) $wgMetaNamespace is not being generated

* Adding cursor:pointer when hovering the headers (which hide/show stuff)
* Remove unneeded parens from r81151.
* Whitespaces

(Follow-up r81151, r81152, r81126)

Made mediaWiki JS test suite table no longer sortable. This functionality broke with the introduction of hide/show clickers in r81126. But we don't need this so we might as well remove it.

Some adjustments to new functionality from r81133
* Floating result to the right (looked a little messy)
* In Safari the 0.8em looked really pixelated and unreadably. Font-size back to normal size.
* Converted variable names to camelCase
* Fixed bug from r81133 (first array item is 0, first counter is 1). Added a +1 to fix it for now (since the headings were overwritten and the first one was removed). Perhaps should be done better at a later time.

open a series of files for xml input, if supplied (used for multiple xml prefetch files)

open a series of files for xml input, if supplied (used for multiple xml prefetch files)

Random fixes in mediawiki.js: add a var, fix indentation, and get our ABC right

Restore global wrongly removed in r80892

JSMin is gone, please talk with JavaScriptDistiller
(r80656 follow up)

Remove empty statement (r80993) and unused global (r80766).

Follow-up r81132 removal of pre-8.3 support.
Still needs removal of $wgDBts2schema inside initial_setup()

call parent constructor so that include path and other things work right

goodNames isn't used anymore

Followup r81132, missed some more ts2 removal

Minor cleanup, docs

Per comment on bug 26612, we should drop the pre-Postgres 8.3 support with the TSearch2 schema.
This removes a bunch of back-compat code in the installer and search code.
Minimum postgres version is now 8.3

JS tests: fold headers, adapt style

Update the style by removing padding inherited from .wikitable and make
the font smaller (0.8em). Make easier to see the tests on a small screen.

I have folded out the tests by section (using headers as reference), added
some classes and id around to help selection.

TODO: we probably only want to hide tests which are OK

Implement prop=wikitext in action=parse, (optionally) returning the original wikitext. Requested by some Google folks wanting to get data about a revision in one request.

Add the resolved URL of each langlink to the API's action=parse output

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

(bug 26285) Extensions will be automatically generated on upload if the user specified a filename without extension.
Note that this still will throw a warning.

invalid 'var' in JS email validation
follow up r81101

Fix c&p error in r81103

(bug 26929) Introduced the edittools-upload message, which will be inserted under the upload form instead of edittools if available

Bug 26948 - hyphens incorrectly interpreted as range

This is the kind of easy to fix, hard to review bug. Email validation
make uses of strings listing characters, those strings are then
enclosed between brackets "[]". Inside brackets, the hyphen is used
to describe a range of character [a-d] being a b c d.
The string containing an unescaped hyphen, made JS/PHP validation
to match the incorrect comma ",".

* Backslash hyphen
* JS,PHP: add tests for commas and hyphens in username or domain
* JS: add var to rfc_1034_ldh_str
* JS: minor code cleanup

TESTS:

php phpunit.php -c suite.xml --filter ValidEmail
OK (13 tests, 32 assertions)

Special:BlankPage?action=mwutiltest&debug=true
Ran 66 tests. 66 passed test(s). 0 error(s). 0 partially passed test(s).

Followup r81067, use the right syntax for doing a join

Added triggerClientTestResources, which makes the use of separate test resourecs optional. Needed for testing installer functions.

* Fixed Oracle new installer support, broken by r80957. This is a minimal patch and doesn't address the architectural issues.
** Moved the responsibility for calling setupSchemaVars() on install to the DatabaseInstaller subclass. This allows it to be called after setupDatabase() has completed, as required by Oracle and PostgreSQL.
** Fixed OracleInstaller::getConnection() so that it respects $this->useSysDBA correctly.
** In OracleInstaller, added some more variables to the list of schema vars, which are needed by user.sql and tables.sql
** In SearchOracle, specify the database name when calling ctx_ddl.sync_index(). This fixes a fatal error in the createMainpage step, caused by the schema name not being equal to the current user.

* In oracle/tables.sql, fixed a couple of indexes with missing table prefixes.
* Improved debugging output in DatabaseInstaller::getConnection() and Installer::createMainpage().
* In DatabaseBase::selectDB(), set $this->mDBname correctly, as in DatabaseMysql.

Removed getConnectionOrDie(), accidentally added in r80957

Fixup query fail from r80936

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

Followup fixme on r80861,

Do cl_from <> 0 check not in SQL, remove order by.

Fixup casing error noticed also

Escape the output of $wgLang->timeanddate() in SpecialContributions. Was escaped in one code path but not in the other

* Whitespace fixups
* followup r66675 - Robla caught a problem with the parameter munging

MessageCache.php: fixed a typo, tweaked spacing, added braces where needed, etc.

Last $wgArticle junk in Skin(Template). Just pass the article on hand to lastModified(). Nothing outside of here calls it anyway

syntax error, stupid me

Followup r81034, remove the global statements

First round of $wgArticle removals

Hooks.php: trim trailing spaces, tweak docs, etc.

* Replaced $wgMessageCache by MessageCache::singleton(); since we only use one instance of this class (as for ParserCache, LinkCache)
* MessageCache::singleton() calls wfGetMessageCacheStorage() directly instead of using $messageMemc, just in case this would be called before that variable is set
* Per TimStarling: also removed deprecated methods in MessageCache class: addMessages() and related, [get|set|enable|disable]Transform(), loadAllMessages(), loadMessageFile() and some others. Same for the legacyData stuff in LocalisationCache that was only used by MessageCache::addMessages() and related.
* Converted remaining extensions

Create index only if it doesn't already exist

Provisionally reverting r80430 (bug 529, 12974)

As noted in CR there are some issues where we're trading one set of weird behavior for a different, but previously unknown, set of weird behavior which breaks existing markup that works around the old behavior.
I'd recommend keeping this in store for after the 1.17 stuff calms down so unexpected parser changes aren't cropping up in the middle of things for people working with trunk.

If these are the right changes to make then great -- but they should be done after the consequences are better understood and folks can prepare for changes.

Transforming a message uses automatically $wgTitle, which may not be set if running from eg. a script, usecase broken by r80819.
Creating a new title in transformMsg() in such case. Reverting r80987 (its revert), and r80898 (workaround).

Confirmed that now both the installer and extensions/Translate/scripts/sync-group.php work.

generateUpgradeKey() could produce NULL return, which means a fatal error when
Call to a member function isOk() in includes/installer/WebInstallerPage.php on line 1039

The function wfShorthandToInteger() is precisely for that.
The previous way only worked for megabytes, which made it
'raise' the memory limit from several gigabytes to 50M

Make the UploadStash repo specific by creating FileRepo::getUploadStash(). In practice this will probably not be used and makes getting an UploadStash object slightly more type work, but I think it is cleaner to have an upload stash explicitly bound to a repo.

Test uploading a file of size $wgMaxUploadSize

TODO: method testMaxUploadSize() shoud be able to use differents
settings.  The assertion could then get moved elsewhere in a nice
helper.

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

Uploads that throw a warning (UploadFromStash) a thumbnail is now shown.

HTMLForm: All section headers and footers. Added an optional second $section parameter to addHeaderText and addFooterText.

revert r80819 for now so we can use phpunit

function foo (Title $title = null ) {} does not work

Fix r80982, re-add accidental removed AND during debugging

Followup r80856, don't unconditionally add the "AND" as if '' is returned by a function, we can get some silly errors (Yay for crappily built SQL strings)

Remove trailing whitespace

Small fixes
* Missing space in SpecialPage.php
* Raising z-index of .suggestions (when used in a jQuery UI modal box the suggestion list appeared behind the modal instead of on top (ui modal has z-index: 1000; )

Resolves bug 26931, where comments were not only not being properly stripped out, but were being transformed into syntax errors.

LanguageConverter::captionConvert(): remove HTML tags and escape HTML special chars to prevent disrupting the layout.

whitespace fix (tab too many)

Follow r80973, fix a tiny error.

Update Chinese conversion tables.

Reset the cache used in User::idFromName(), otherwise tests will try to add the user '127.0.0.1' multiple times, resulting in a database error (duplicate key for `user_name` field)

Allow the $result parameter of the isValidPassword hook to be an array if the message requires parameters

* Fixed a bug causing the installer to ignore the "engine" and "charset" settings when installing a MySQL database.
* Fixed a bug causing the engine and charset settings to not be properly preserved when adding new tables on upgrade.
* Fixed total breakage of SQLite upgrade, by reusing the administrative connection to the SQLite database instead of creating a new one when wfGetDB() is called. Added LBFactory_Single to support this.
* Introduced a "schema variable" concept to DatabaseBase to avoid the use of globals for communication between the installer and the Database. Removed a lot of old global variable names from Database::replaceVars(), most were only added on a whim and were never used.
* Introduced DatabaseInstaller::getSchemaVars(), to allow schema variables to be supplied by the DatabaseInstaller child classes.
* Removed messages config-mysql-egine-mismatch [sic] and config-mysql-charset-mismatch. In the old installer it was possible for users to request a certain character set for an upgrade, but in the new installer the question is never asked. So these warnings were shown whenever a non-default character set or engine was used in the old database.
* In MysqlInstaller::preUpgrade(), fixed the incorrect strings used to identify the MySQL character sets: mysql5 instead of utf8 and mysql5-binary instead of binary.
* On install, initialise the site_stats table, using code copied from the old installer. Unlike the old installer, use SiteStats to increment the user count when the initial user is added.
* Fixed several instances of inappropriate call-by-reference.
* Replaced call_user_func_array() with call_user_func() where possible, it is shorter and simpler.
* Moved the caching boilerplate for DatabaseInstaller::getConnection() to the base class, and have the derived classes override an uncached function openConnection() instead. Updates r80892.
* In MysqlInstaller::getLocalSettings(), escape PHP strings correctly with LocalSettingsGenerator::escapePhpString().
* Reduce timeout for checks in dirIsExecutable() to 3 seconds, so that it doesn't take 30s to run when apache is in single-threaded mode for debugging.
* MySQL and SQLite have been tested and they appear to work. PostgreSQL upgrade is totally broken, apparently it was like that before I started. The Oracle code is untested.

fixed typo from r80924

Couple more raw SQL to query arrays

Missed the case where there is an equals.
It's ugly having to add that literal node there.

port rtl specific parts of modern RTL css to main.css

In the new auto flipped CSS, the "html > body div#mw_contentholder ul li" specific hacks were breaking layout on other browsers, so removing those.

Follow up to r80776

Integrate what was functionally left of common_rtl.css into oldshared.css
Commit the Resourceloader conversion of oldshared.css that I had forgotten to commit

Follow up to r80785

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

update colors (I could not see them)