* (bug 28897) rvparse doesn’t seem to work with rvsection

Add missing word to r89505

* (bug 29265) Mediawiki API documentation incomplete

Add text about erroneous requests when sent to the api

Remove extra //, and add some documentation

Adding method documentation

Add a couple of explicit variable definitions

Add some braces

Add one false; to return under case self::AS_NO_CREATE_PERMISSION:

code documentation for $wgPreprocessorCacheThreshold

follow up r80180

SQLite-specific dropTable(): works a bit faster because it doesn't need a separate table existence check and does not rely on subtly broken tableExists() which I will fix a bit later

Revert r89254:
* It breaks if any extension uses any global except $IP, $wgHooks or $wgAutoloadClasses. Some unit tests that try updating a virtual database are already affected.
* If we have LocalSettings.php, we already have it loaded - are there code paths that don't have it?
* Parsing PHP with regexes is fragile and scary

pear -> port

Also support macports

Follow up r60599. Make normalizeUnicode() work even if $wgContLang is still not set,
just as it did before (eg. getGPCVal() check). This allows for early calls before
MediaWiki is completely initialised.

Fix for r89474: added missing () to function call

Fix for r89475: let's make this correctly

Added missing $PAGEEDITTIME replacement for impersonal mails

Follow up r89408. Magic >lang to getLang().

Use isAnon() so that the mail doesn't contain "anonymous user w.x.y.z" when a logged in user has "w.x.y.z" as real name and $wgEnotifUseRealName is true

Add BlockTest to the tests autoloader. Needed by ApiBlockTest.php
Added tabs to other items (compare ignoring whitespace).

Follow up to r89419:
Per Brion's comment on Code Review, a generically named ltr class is created and used to reduce the number of rules

Fixing fixme in r87164 (wrong if check)

* (bug 29265) Mediawiki API documentation incomplete

Flesh out documentation for the maxlag parameter

Added a note when this script is used

follow up r89278 — remove FTP support.

Followup r88124: fix the most glaring bug of the week

* (bug 29267) always give the servername for meta=siteinfo&siprop=dbrepllag

Provisional revert of r89230: per CR, benchmarking currently shows that it makes things slightly slower, not faster

http://www.mediawiki.org/wiki/Special:Code/MediaWiki/89230#c17571

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

Use getSearchEngine() instead of constructing class name manually

r89329 followup: roa-rup -> rup in addwiki.php's initial main page text.

(Is this script still used?)

Provisional revert of r89406, r89414: reference-related warnings need cleanup before applying code like this

Per CR http://www.mediawiki.org/wiki/Special:Code/MediaWiki/89406#c17545 :
'Here is a third one: Strict Standards: Only variables should be passed by reference in /www/sandwiki/includes/Wiki.php on line 177 '

Offending bit is this:
- SpecialPageFactory::executePath( $this->context->title, $this->context );
+ SpecialPageFactory::executePath( $this->getTitle(), $this->getContext() );

That function demands reference paramters for $title and $context, which is being violated here where we now pass function return values:

public static function executePath( Title &$title, RequestContext &$context, $including = false ) {

The $title does sometimes get replaced within the function body, but $context does not appear to ever be replaced (its *contents* are modified, which does not require passing by reference)
If replacing it is something it should be doing, then we need to be able to replace it upstream presumably, so $this->getTitle() probably isn't appropriate.
The $context probably should have the reference simply removed.

Revert r89409: provided a default Database->listTables() implementation that didn't actually check the database, thus missing extension tables, any custom tables, any other tables being managed deliberately, and would be missing a matching table prefix if any.

Extension to r89419, I was too excited fixing the bug

Extension to r89419, removing a bugous CSS code

(bug 29263) Add LTR class to the shared CSS to be used for left-to-right text such as SQL queries shown in dberrortext and similiar messages in RTL environements

Follow-up r89406 CR: another use of RequestContext::__get()

* suggestion for using tables.sql parsing instead of database query for tables list needed for phpunit
* would be better option IMO since the database is not necesserally dedicated to MW install

More unpicking of r85288.  I think this is all of the magic method calls, but they're very hard to grep for (part of the problem with them!), so let's leave the calls in with a wfDeprecated() for a while...

Don't execute the loop if there's no X-Forwarded-For header, also don't use isset() to check only for null

Start unpicking r85288 (magic __get() accessor for RequestContext).  Instead, bring back some of r86872 (abstract base class for classes providing access to RequestContext methods), which is a more 'classical' solution.

Deprecated wfGetAgent(), use $wgRequest->getHeader( 'User-Agent' ) instead

Removed usage of error suppression operator

(bug 29105) Splitlines completely messes up selection of inserted text. This is not a problem in practice (WikiEditor doesn't have multiline insertions with splitlines enabled since r89402), but just in case it ever becomes a problem I'm disabling selection of inserted text altogether when splitlines is enabled; instead, the cursor will be moved to the end of the inserted text.

(bug 29104) Fix r86622: make splitlines default to false, makes much more sense

(bug 29121) jquery.suggestions uses jquery.autoEllipsis but didn't have it in its dependency list, causing JS errors in non-Vector skins

Fix some of the weird freaky interaction between ApiBlockTest and BlockTest.
* By using the same summary for both, we allow both sets of tests to pass with --filter Block
* Just testing includes/BlockTest.php still passes

rvv: r89398. Tim wants me to wait

Kill off action=raw from index.php
* One less entry point to worry about
* Completely obsolete by load.php and api.php
* Left $wgDebugRawPage in place, also affects load.php requests (docs already reflect)
* Affects dumpHTML (probably broken anyway, has anyone RL-ified this?), and two abandoned/obsolete extensions

(bug 28840) If the query string hits bug 28235, redirect to a safer URL instead of showing an unhelpful error message. IE 6 will only use the extension of the final destination for its cache filename.

Remove useless bootstrap inclusions

Fix for r85229: getOutput() instead of getOut(), leftover $wgOut usage.

Apply a patch adapted from the one on Bug #16794

    This patch should allow you to use the $wgSharedDB [with Postgres]
    normally, as you would with mysql. Basically this patch creates a
    second connection with the shared database and when a query is
    made, we check on which connection we should send it.

Patch from Luca Fulchir

Missing ,

Add Maintenance and FakeMaintenance to the autoloader for paranoia

forward port r88929

Kill $wgWikiFarm. Only use in a wmf code path, and I can't find any evidence of this being use in WMF anywhere.

Fix for r89166: need self

Move fixes for config-error-box to mw-infobox in order to apply for warnings as well. And removed redundant clear: left. Poke r88465, 86060, r88109

Fix for bug #28172 (“wfGetDB called when it shouldn't be”).

forward ported from r88936 and r89374.

typo from r78201

* fixed table duplication for phpunit testing

Revert r88648 per CR

w/s changes.

Followup r89348

Fix code style, and remove hint that variable may be multi

Don't duplicate tables starting with unittest_ which can be left if a previous run ended with a fatal on SQLite

follow up r88502 with a fix to resource loader to restore tochidden
removal addition.

Revert r88722 (allow multiple values for letype): this won't play nice with query indexing

Fix for r89263: this to $this

Revert r88502 since this can be done with a gadget

Fix regression in r84638, causing ArticleDeleteComplete to be called twice on file deletions. doDeleteArticle() already calls it, there's no need for FileDeleteForm::doDelete() to call it again

Close wfProfileIn
Exit point missed by r85204

added @ßince tag

Add Debug hook as a new feature of 1.19

Release notes were not updated by r88843

properly move the jpegmeta.js file from util to libs.

Follow up r89083 (code change)
Fix r89084 (file move without history, reverted by r89334)

revert r89084 (file moved without keeping history)

Fix encoding (followup r89330)

Follow-up to r89329: Add MessagesRoa_rup.php as fallback to 'rup'

Renaming 'roa-rup' to 'rup' (see also bug 15988), and adding 'roa-rup' to $wgDummyLanguageCodes. I'm going to add MessagesRoa_rup.php as fallback, and update extension translations as well.

Removed useless call to wfLogProfilingData() since profiling is disabled in command-line mode

Removed obsolete comment that was mentioning $wgInputEncoding and $wgOutputEncoding

Explored some ideas for HipHop optimisation. Made a preprocessor implementation, based on a copy of Preprocessor_Hash, with a preprocessToObj() which is optimised. It takes 33% less time than Preprocessor_Hash for a certain realistic test case (the Barack Obama article). Some notes about what I did:
* Set EnableHipHopSyntax=true to enable string and integer type hints. I gave the file a .hphp extension to avoid false alarms in syntax checking scripts.
* Made sure almost all the local variables in preprocessToObj() have a specific type, instead of being variants. This is useful for integers, but has the largest impact for objects, since dynamic method calls can be avoided.
* Stopped using extract() since it forces all local variables to be variants, and adds some hashtable initialisation overhead.
* Found a way to cast a variant to a specific object class, by abusing argument type hinting. The method does not require special syntax; it is harmless in Zend PHP.
* Wrapped various internal function calls with type casts. strspn() and substr() need to be wrapped with intval() and strval() respectively, since they return a variant to support special error return values. HipHop isn't smart enough to know whether the error case will be triggered.
* Replaced most instances of double-equals with triple-equals. Profiling indicates that this makes a very large difference when comparing strings, much more so than in Zend.

Followup r89311: rv unrelated changes

Make user right selection in installer actually work, thanks Krinkle for spotting this

Revert r88997: fix for bug 93 caused some additional problems.
Follow-up to r89191: disable test case for bug 93 as it currently fails (bug left open).

Self-revert r87545

Mostly revert r89302, was slower than the original

Exclude Block tests until somebody fixes them, also cache listTables() result so we can skip some queries

Test db cleanup
* Fix Oracle stuff per CR on r88755
* Don't expect tests to clean up after themselves, do it in MediaWikiTestCase::run() (skipping user and interwiki, also per CR)
** Also don't do it from the destructor, phpunit calls this more often than you'd think
* Need to find: way to detect "is this the last test?" so we can drop our tables on completion

Add some doc comments on Cookie::validateCookieDomain() and Http::isValidUri(), which both have some failing test cases and are incomplete

Add more super-cookie test cases to HttpTest; things like org & .org aren't being handled correctly either. Not super high prio but... should probably mark these tests broken if can't be fixed easily yet.

Refactor validateCookieDomain test cases to use a data provider and report which bit's being tested so we can identify failures (there is still one failure in this set)

Not silly, but parser error (r89263)... fixing
PHP Parse error:  syntax error, unexpected T_STRING, expecting T_VARIABLE in ExtensionUpdater.php on line 7

'unblockself' right was never added to User::$mCoreRights

Noticed while poking bug 27514

Tweak HttpTest following r89278:
* remove FTP test case URL (shouldn't actually work, and as a directory listing, not super stable)
* remove old test case URLs that are now 30x redirects (example.com and a toolserver link), which broke the comparison tests
* added in http://en.wikipedia.org/robots.txt so we've got a nice basic fetch on one of our servers too

Now this test class is down to 3 failures, in URL & domain validation stuff

Document r89248,r89249 isQueryStringBad()

HttpTest regression fix: test URL on example.com no longer returns a 404 (now a 302 redirect), so switching the 404 tests to use an URL on mediawiki.org in the /xml/ dir where we stash actual files.

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

Fix Bug #29231: PhpHttpRequest doesn't support HTTPS

Tested fix with PHPUnit Tests that aren't currently executed by
CruiseControl b/c they're marked “broken” till someone comes up with
some reasonable unit tests that will work for everyone.

Add some documentation

Swap if ( $foo ) { $this->addFields( 'foo' ); } for $this->addFieldsIf( 'foo', $foo ); etc

Also swap $this->addFieldsIf( 'foo', $foo ); $this->addFieldsIf( 'foo2', $foo ); for $this->addFieldsIf( array( 'foo', 'foo2' ), $foo );

Micro-optimisation and readability!

Remove some unused variables, add/improve some documentation

* (bug 29221) Expose oldrevid in watchlist output

* suggestion for ExtensionUpdater that would handle DB install/update of extension from DatabaseUpdater
* could also generate default code for LocalSettingsGenerator
* revert if silly :D

* used Html static functions instead og Xml. (r89260#c17418)