Made DifferenceEngine use a context instead of global variables and updaters calls to constructor in core

Made setupGlobals() return the created context and use it instead of either global variables or recreating more instance of those classes

Removed FeedItem::getLastMod() and FeedItem::getDBPrefixedTitle() since they are not used and would throw an error since FeedItem::$Title is a string and not an object

move store writing out of recache()

Writing to cache is now handled in a new method. That makes recache() a bit
shorter and easier to read. Since there is no point in writing data to
/dev/null , return early in such a case (ie if store is LCStore_Null).

load up includes/Setup.php before Maintenance::finalSetup()

This let us get access to global functions and the autoloader when trying
to do some final setup for our maintenance scripts.

Follow-up r102628, r102612: More protocol relative URLs.
Also contains localisation updates from //translatewiki.net :)

* reverted changes to Block and ApiBlockTest made in r102625 ... PEBKAC

Follow-up r102612: Use protocol relative URLs for translatewiki.net also

* Added getInfinity to DatabaseOracle
* Block - replaced 'infinity' strings with DB->getInfinity calls
* UploadStash - added sequence value generation for ID

Add a new User::getDisplayName() to return the name that should be displayed in the interface.
Add a UserDisplayName hook to allow extensions to give custom display names for users.
Add a $wgRealNameInInterface to use the real name of a user as the display name.
To start of the first use of the display name functionality tweak SkinTemplate to declare the userdisplayname and use it inside of personal_urls.

Refactor core from Title::getLinkUrl to Title::getLinkURL to match Title's getLocalURL and getFullURL.
This shouldn't cause any back or forward compat issues since php treats method names as case-insensitive. So this change shouldn't break any old extensions, and extensions are free to start using getLinkURL and won't have any issue breaking with old versions of MediaWiki.

[mediawiki.Uri] Add overrideKeys option
* The behavior of turning foo=bar&foo=quux&lorem=ipsum into "{ lorem: 'ipsum', foo: ['bar', 'quux'] }", has annoyed me several times. As it is in contrary with the way PHP works (PHP only makes numeral arrays if the key ends in '[]'). Adding in an option to make mw.Uri behave like that.
* We may wanna make that the default at some point, leaving default behavior unchanged for now.
* Converted 'strictMode' into 'options', kept backwards compatibility
* Fixed documentation comment for example (q1=0 -> "{ q1: '0'")
* Updated unit tests to account for this option.

This SVN commit fixes bug32220 and makes links to mediawiki.org in language message files protocol relative

Follow-up r102521
Remove $wgUseExternalEditor globals.

Move tests that have likely never been executed.  Now they will be executed.

It might be a good idea to have a commit hook or similar to catch these.

You can find candidates for renaming with this command:
$ find tests/phpunit/includes/ -name '*.php' | grep -Ev 'Test.php$' | xargs grep -l MediaWikiTestCase

Reverted r101244 changes - these are not equivalent and may have been causing the assertion failures people (timo, myself) were getting.

essage=fix trumped line of code from svn update

Add wfRemoveDotSegments and unit tests.  This is a sane step towards fixing
bug 32168.  This implements RFC3986 Section 5.2.4.
http://tools.ietf.org/html/rfc3986#section-5.2.4

This is important because you need to remove dot segments in order to safely
compare URLs when limiting URLs to a particular path.

Fixed misleading getItem() comments...things like arrays or FileDependency objects and what not can be returned

essage=This bug fixes bug30483

Commit to fix bug 30513.

Bug 31995 - Addition of PPFrame::getTitle

Applied patch by Van de Bugger

Added documentation to go with

Added code to Preprocess_HipHop too

Update RELEASE-NOTES-1.18 from branch

Localisation updates for core and extension messages from translatewiki.net

Move RELEASE-NOTES from r102416, after r102542

(bug 32256) API list=search stops at first invalid result. Thanks to Lupo for finding the cause of the bug: in the code path for invalid titles, the iterator wasn't moved to the next result, so the loop kept processing the same result until it hit the limit.

move bug 29912 RN form 1.19 to 1.18

This follow up a manual merge of r101002 in REL1_18 made by r102535

Bug fix is:
(bug 29912) Unit tests break if parsertest tables are still present

Drop "selenium" user right. As far as I can tell, this isn't actually used anywhere (core or exts) and is leftover from the special page for selenium that was removed.

split getHooksFromDoc() function

if() { LOT OF CODE } else { LOT OF CODE } structures are hard to read
move the LOT OF CODE piece in its own function makes code easier to
understand.

* phpunit fixes & optimizations for oracle backend

Some updates to EditPage and ExternalEdit:
* Made ExternalEdit use a context
* Updated DifferenceEngine to use ExternalEdit to send the diff stuff to reduce code duplication
* Introduced ExternalEdit::useExternalEngine() to check whether to use the external edit or diff (except for the action, section and oldid which are still checked in Wiki.php) to remove code duplication; external diff can now also be controlled with externaledit or internaledit URL parameters
* Use $wgContLang to get the name of the "Special" namespace instead of user's language
* Modified the line breaks in the comment on the top of the control file so that the URL is on its own line
* Updated extension to call EditPage::edit() instead of EditPage::submit(), the latter will just call the former
* Updated extension to let core handle itself the ExternalEdit mode instead of doing this themself

Use WikiPage instead of Article

Throw a FatalError exception instead of calling OutputPage::showFatalError() (on usage of $wgOut less)

Allow \x1b (ascii ESC) in wfDebug() output

It is a local hack I had to let me pass colored debug message using a local
wrapper:

function wfDebugGreen($msg) {
$color = "\x1b[1;32m"; # Green
$reset = "\x1b[1;0m";
wfDebug( $color . $msg . $reset );
}

Ideally \x1b should only be kept if it is followed by /(\d+(;\d)?)?+m/
which should match the 'graphic rendering' ANSI escape code.

Comments out console.log calls

Catched by Schnark on CR of r102303

fix bug #29102: Upgrade fails "Unknown character set: 'mysql4'"

by replacing "mysql4" with "binary"

Trivial enough to be backported, I think.

Delete LanguageSr_el.deps.php. LanguageSr_ec.php doesn't
contain any base class used in LanguageSr_el. Not even in r13157.

Add name for code tly: Talysh ( en.wikipedia.org/wiki/Talysh_language ).
There are translations in that language for the toolserver and
some extensions (AdminLinks, Babel, CategoryTree, Contest, GlobalUsage,
LanguageSelector, Todo, Translate and WikiData).

Mark as FIXME. This three calls leak a png filename in sys_get_temp_dir() each,
tempnam() like, not the two-words filenames of RandomImageGenerator::getRandomFilenames()
They aren't produced by ApiTestCaseUpload::fakeUploadFile() either.

Test for bug 32057.
Latent bug in quite unused preprocessing path (expansion of heading nodes),
which was exposed by r101291 and fixed in r102440.
This test is used to exercise that code path.

Revert r99307 and its friend r99309, improper use of !important per CR

Attempt to fix translatewiki bug 32057
The different Preprocessor::newChild expect false (with strict comparison) if there's no title passed to them.

Localisation updates for core and extension messages from translatewiki.net

Fix for r102417: commiting OutputPage.php is actually a good idea

* Changed OutputPage's handling of subtitles to use an array and implode it with "<br />" so that callers don't have to bother whether there's already a subtitle when adding new ones
* Make OutputPage::setSubtitle(), OutputPage::addSubtitle() and OutputPage::appendSubtitle() accept a Message object in addition to a string
* Added new method OutputPage::addSubtitle() to replace appendSubtitle() and marked it as deprecated
* Clear the subtitles when showing an error page
* Always show the subtitle from Article::viewRedirect() when showing a page; left the parameter for use in EditPage
* Make Article::setOldSubtitle() add two subtitles and has to set "display:inline;" for the three possible divs to avoid too many spacing
* Removed the six different backlink subtitles and added one new message 'backlinksubtitle' to replace them and added OutputPage::addBacklinkSubtitle() to factorise common code
* Changed EditPage's view source to show "View source for <Page>" with the same backlink in subtitle for consistency with other back links and page titles

Bug 20148 - Change title of Special:Disambiguations

Changing the english title to something more accurate.

Cast the return value of getExtraUserToggles() to an array in case it's not in the dataCache

Fix fatal introduced in r97296. Seems to happen only after restarting memcached.

( ! ) Fatal error: Unsupported operand types in /srv/org/wikimedia/prototype/wikis/rl2/languages/Language.php on line 479
Call Stack
# Time Memory Function Location
1 0.0002 121976 {main}( ) ../index.php:0
2 0.0082 2959136 MediaWiki->run( ) ../index.php:58
3 0.0082 2959136 MediaWiki->main( ) ../Wiki.php:531
4 0.0082 2959136 MediaWiki->performRequest( ) ../Wiki.php:612
5 0.0165 5012424 MediaWiki->performAction( ) ../Wiki.php:252
6 0.0585 6947008 EditPage->submit( ) ../Wiki.php:506
7 0.0585 6947008 EditPage->edit( ) ../EditPage.php:360
8 0.1441 19979552 EditPage->showEditForm( ) ../EditPage.php:512
9 0.1704 22610032 EditPage->showStandardInputs( ) ../EditPage.php:1574
10 0.1778 22627752 Skin::makeInternalOrExternalUrl( ) ../EditPage.php:2046
11 0.1778 22627752 Skin::makeUrl( ) ../Skin.php:1006
12 0.1779 22627752 Skin::checkTitle( ) ../Skin.php:991
13 0.1780 22627752 Title::newFromText( ) ../Skin.php:1050
14 0.1780 22627752 Title->secureAndSplit( ) ../Title.php:148
15 0.1780 22627752 Language->getNsIndex( ) ../Title.php:2718
16 0.1781 22627752 Language->getNamespaceIds( ) ../Language.php:530
17 0.1782 22627752 Language->getNamespaceAliases( ) ../Language.php:505

test SQL for our QueryPages objects

Part of bug 32118: test special pages SQL queries against all supported DB

Still need to add all the other non QueryPage special pages and then setup
jenkins to support other databases.

Use the local context instead of $wgOut, now that we have one

* Use local context to get messages
* Changed three isset() to [!=]== null

correct typo in r102393

add quick comment from r102334 cr

clean RN 1.19 of bugs merged in 1.18

bug 16428, bug 30684, bug 30907

Prefix debug message in wfClientAcceptsGzip()

update ParserCache debug message

"Found." was not very helpfull when indicating we found a cached
ParserOutput object.

Removed old, unused, MW_IMG_* constants

Follow-up r102334: odd that this minus sign (yes, copied from a diff) was actually harmless.  Also explicitly define a variable that would otherwise be lazily initialised to zero but about which my IDE complains.

(bug 31502) (follow-up r84718) Restore ToC to Special:EditWatchlist if there is more than one namespace and more than 30 titles total (also resolves the commented TODO in the pre-r84718 code).

FU r102073:
* Added assertRepoDefined()/assertRepoTitle() functions to throw exceptions if a valid FileRepo or Title member is not set
* Fixed comment about UnregisteredLocalFile (it always sets a Title object)
* Note that $title in File constructor can be a string and that getTitle() can return false in docs

Localisation updates for core and extension messages from translatewiki.net

* Use 'new self' instead of hard-coded class name
* Added constructor comment

* Added specific page header when showing "search deleted pages" form
* Throw an exception directly instead of calling OutputPage::permissionRequired()
* Removed unreachable code paths
* Show nicer error messages when undeletion could not be executed
* Pass the subpage parameter to loadRequest() and define $mTarget and $mTargetObj there

re Bug #29283, r101507: Apply Vitaliy Filippov's followup fix:

    .... I've found another bug with the implementation - it also
    relies on the presence of key '...:preload' and doesn't check it
    when checking for the expired cache, which can lead to "Invalid or
    missing localisation cache" exceptions.

tests for bug 32047 / r102301

In table with class="sortable", thead is before caption

Fix Bug #32047: in table with class="sortable", thead is before
caption

Apply fomafix's patch for jquery.tablesorter.js (also reported upstream:
https://forum.jquery.com/topic/in-table-with-class-sortable-thead-is-before-caption
(currently in moderation)

Allow raw conditions on insertSelect, given that array syntax isn't always expressive enough (r96930)

Fix Bug #31547 - Category links need less spacing and better wrapping

Updated patch from Erwin Dokter

bug 31990 justify paragraphs pref adds extra space to category listing

When text justification is enabled in user preference, the category list
could show a large space before the first category. Overriding the element
to use left justification is enough to fix the spacing issue albeit the
categories will no more justified.

Thanks to lupo with bug 31990 comments number 8 hinting at the catlinks
element not being properly justified.

This is a regression in REL1_18 and need a backport.

bug 29110 - $wgFeedDiffCutoff doesn't affect new pages

Patch by John Du Hart. Rewritten to avoid code duplication, duplicate
code is now in a helper function.

fix badly named variable

Resource takes two S in french :D
follow up r102286

early returns to avoid long code in if / else

Instead of enclosing a lot of code in a if() {} block. I reverted the logic
to exit early. That makes code a bit easier to read.

Logic was:
 if( $title->getNamespace() >= 0 && !$accErrors && $newid ) {
   // LOT OF CODE
 }
 return;

Now:
 if( $title->getNamespace() < 0 || $accErrors || !$newid ) {
   return;
 }
 // LOT OF CODE

dieout when file is a boolean

On file operation errors, the file variable can be assigned boolean false
which is not a valid handle. Those backtrace can help users debug an issue
when generating a filemap.

Bug 30232 - add current time to message wlnote on Special:Watchlist

Patch by Umherirrender. I did remove the rcnote message being used most
probably before we had support for {{PLURAL}}. So now we just rely
on the wlnote message.

As requested by Niklas, the timeanddate parameter was dropped from
original patch.

Umherirrender is already in the CREDITS file.

Removed useless second parameter to Title::quickUserCan()

* Removed existence of Title object check since not having it would make other things crash in the same function
* Moved isArticleRelated() check one level up
* And pass the User object to Title::quickUserCan()

* Fix checks to show whether "edit" and "addsection" tabs should be marked as active so there's one, and only one active tab
* Follow-up r102192: simply call isProtect() to see if the page is protected, it also handles creation protection

Update date format for dsb and hsb: month names need the genitive, make them consistent
Per discussion on https://translatewiki.net/wiki/Thread:User_talk:Raymond/Formatierung_einer_Datumsvariable

Allow for installation of php in ~/.mwphp and don't die if maintenance/dev/data already existsy

Followup r102201, quote all variables to guard against paths with spaces in them.

Commit maintenance/dev/ a set of bash scripts that can quickly download php 5.4, install it in an isolated area, install a quick development copy of MediaWiki, and then start up a local webserver.

re r102027: split up long lines

Localisation updates for core and extension messages from translatewiki.net

Use WikiPage instead of Article to call updateCategoryCounts()

* Only show "view" tabs when the user hasn't the permission to read the page and put the whole stuff in a $userCanRead check instead of doing that for each check
* Factorise duplicate code (for the "protect" tab generation)
* Use Title::quickUserCan() check for delete and protect actions instead of User::isAllowed()
* Pass the User object to Title::quickUserCan()

* Merged Title::userCanRead() check in Title::getUserPermissionsErrors()
* (bug 26020) Setting $wgEmailConfirmToEdit to true no longer removes diffs from recent changes feeds
* Added second parameter to Title::userCan() and Title::quickUserCan() to allow callers to pass the User object to use for checks; this changes Title::userCan()'s second parameter from "do expensive queries" flag to User, but all callers should have been updated in r102183
* Updated callers that might throw a PermissionsError to use getUserPermissionsErrors() instead and pass the error array to the exception
* Refactored duplicate code in missingPermissionError()
* Moved Title::isNamespaceProtected() a bit upper and Title::userCanRead() near Title::userCan() to have related functions in the same location
* Some minor refactoring in permission-related functions in Title

Bug 26375 - Italics should be passed to TOC
http://en.wikipedia.org/w/index.php?title=Wikipedia:Village_pump_%28technical%29&oldid=403630551#Italics_in_ToC

Didn't we agree to use @since?

* Moved the language variant check inside the "not a special page" check instead of doing it again after
* Moved the definition of the $session variable near where it's used

Fixed doc typo

Make the CliInstaller use --server to set wgServer instead of always setting it to "localhost"

* Use local context to get messages
* Escape messages that were passed raw to the output

Localisation updates for core and extension messages from translatewiki.net

* Only call Title::getPageLanguage() when necessary, i.e. after "!$wgDisableLangConversion && $wgCanonicalLanguageLinks" check passed
* Removed creation of Special:Recentchanges' Title object and use Title::isSpecial() instead

* Made OuputPage::showPermissionsErrorPage() show a different messages for 'read', 'edit', 'create' and 'upload' actions to saying "You need to log in to do this action" when 1) The user is not logged in 2) The only error is a permissions error (no block or something else) and 3) The error can simply be avoided by logging in
* This replaces OuputPage::loginToUse() functionnality, made it simply throw a PermissionsEror exception and updated all calls in core
* Same for the check in SpecialUpload::execute(), EditPage::userNotLoggedInPage() and EditPage::noCreatePermission()
* Throw the same exception in EditPage::attemptSave() whether the user is logged in or not and let OuputPage::showPermissionsErrorPage() decide which message to display
* Replaced call to deprecated OutputPage::blockedPage() in SpecialUpload
* Displayed messages are the same as now, except the title is always "loginreqtitle"
* 'nocreatetitle' and 'uploadnologin' messages are still used by extensions, so I kept them, but the message 'whitelistedittitle' is not used anymore and has been removed

* Use WikiPage instead of Article for doPurge(), getting raw wikitext and ParserCache access
* Use ParserOptions::newFromContext()

* fixing Special:ListBots which behaved like Special:ListAdmins

Followup r100126, per brion remove htmlspecialchars() call

Swap back a code line to use $text

Swap some space indents to tabs

Followup r101241, after removing @private set private modifier as they are indeed private...

Partial revert to r101244 due to next() weirdness