Set envelope sender to the same as the From: address when PEAR mail is used.  This is so that mail “bounces” to the expected place.

Note that this only works when $wgSMTP is set and (as a result) the PEAR mailer is used.

When $wgSMTP is not set, the envelope sender is set using $wgAdditionalMailParams set to something like “-f from@example.com”.  Someone should create some Envelope sender sanity in how the envelope sender is set.

Suggestion: if $wgAdditionalMailParams is not set, then use $wgEmergencyContact to create “-f $wgEmergencyContact” for php mailer and to set the Return-Path header for PEAR mailer.  I used “From:” for Return-Path here because it seemed most sensible.

See also: http://pear.php.net/bugs/bug.php?id=5017

Removing calls to deprecated functionality in favor of the new versions. Old version worked fine but shouldn't be used.

Follows-up:
* r75275: Introduced the updateTooltipAccessKeys function in the new library but didn't call it on document ready and left the deprecated one in the onloadhook-run in wikibits.js untouched
* r75287: Introduced jquery.checkboxShiftClick and called on-load but left the load call for the legacy version untouched. Depending on the load order at any given time it may not have been used.

Also reordering the if-else case in mw.util.updateTooltipAccessKeys to allow a call without arguments ("undefined instanceof Foo" throws exception)

w/s fixup before patch

* Don't use isset() to check only for null
* Also pass the Title object in RequestContext::msg()

Override IndexPager::getTitle() in ImageListPager, this is the only class subclassing TablePager that don't do that in core

Removed usage of error suppression operator

Partial (?) fix for bug 29753 -- wrong Firefox version comparison for non-Mac Firefox 2+ accesskey tooltip. However, this code IS NOT YET USED which is very suspicious. An earlier version of this code is wikibits.js is still being used for some reason, which indicates a maintenance problem.

(bug 27410) The tag filter on a history page should be within a <label> element. Patch by The Evil IP Address.

Also remove inclusion protection for the 800th time: if a file only has classes, you don't need to do this :)

Reduce some duplication

(Update for r91646) Use mw-content-ltr/rtl so bullet list shows properly where user direction != site direction. Also add lang & dir attribute, and fix typo in shared.css.

Rephrasing msg clearyourcache per Bug 27550 : Adding Chrome, moving up IE, making it a bulltetd list. Thanks Amir E. Aharoni

Bug #29755: Apply patch from Vitaliy Filippov so that MW's HTTP client
respects no_proxy env setting

Partial revert of r90833, capitalisation change was not intented

Replace bullet-icon.png with an 8-bit version (became 24-bit due to r78011 which removed the color palette)
* Fixes (bug 19514) Unordered list list-style-image should be IE6-compatible (8-bit)

Followup r91609, helps to actually remove the @

Bunch of error suppression operator fixes (bug

* (bug 28626) Validate JavaScript files and pages loaded via ResourceLoader before minification, protecting separate modules from interference

This is possibly not perfect but seems to serve for a start; follows up on r91591 that adds JSMin+ to use it in some unit tests. May want to adjust some related bits.

- $wgResourceLoaderValidateJs on by default (can be disabled)
- when loading a JS file through ResourceLoaderFileModule or ResourceLoaderWikiModule, parse it using JSMinPlus's JSParser class. If the parser throws an exception, the JS code of the offending file will be replaced by a JS exception throw listing the file or page name, line number (in original form), and description of the error from the parser.
- parsing results are cached based on md5 of content to avoid re-parsing identical text
- for JS pages loaded via direct load.php request, the parse error is thrown and visible in the JS console/error log

Issues:
- the primary use case for this is when a single load.php request implements multiple modules via mw.loader.implement() -- the loader catches the exception and skips on to the next module (good) but doesn't re-throw the exception for the JS console. It does log to console if present, but it'll only show up as a regular debug message, not an error. This can suppress visibility of errors in a module that's loaded together with other modules (such as a gadget).
- have not done performance testing on the JSParser
- have not done thorough unit testing with the JSParser

Followup r91602, forgot svn add

Clean up the mess that is wfDie (resolves r85918). wfDie() doesn't exist anymore, so don't use it.

The only two usages left since r91590 are php version checks for index/load/api.php, and $wgAPIEnabled check in api.php. Consolidate all of the "bailing for invalid version" into PHPVersionError.php. Leaving $wgAPIEnabled as the only user was silly, so just echo and die like wfDie() would've done

Correct the documentation of srprop.

Actually return the title of the section, instead of the pagetitle when requesting srprop=sectiontitle from API search.

Fixes bug 29746

* (bug 29745) Fatal error in API search

Properly name objects being used

Remove 'upload-wasdeleted' message. Its no longer used.

Original feature was (accidently?) removed/broken in r57868 and then re-created
using the message 'upload-recreate-warning' in r65339.

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

Followup r83885: add JSMin+ 1.3 to use its parser to verify output of JavaScriptMinifierTest unit test cases. Had to change some of the test cases because they were not valid JavaScript programs -- one of the quoting tests was incorrectly quoted in the PHP sources, and several tests around return, continue, and break keywords failed due to not using them in the contexts that they require.

http://crisp.tweakblogs.net/blog/1856/jsmin+-version-13.html
JSMin+ under MPL 1.1/ GPL 2.0 / LGPL 2.1 license.

Remove last wfDie()s from maintenance

Partial revert of r91106: followup to r91127.

Suppressing all warning output from unserialize() in ExifBitmap::formatMetadata will hide actual errors; we're now checking explicitly for the bad-metadata and no-metadata markers so the case this was added for should not happen.

Move wfShowingResultsNum() back into SpecialSearch where it belongs. No need for a global function for something thats only used once in core or extensions

refixing bug 27338 with all parser test fixes, reverts r91557

Call parent::setUp() and allow it to do lang setup stuff

Declared some member fields

Remove notes about being destructive. We work properly on cloned tables now

setFileVersion (r91435) was missing default $val

(follow-up r57868) Check if there exists a deleted page if we have a filename to check, not if we don't have a filename.

Also the showDeletionLog function was removed way back and folded into a different function in special:upload, so remove the call to that.

public static

Followup r91270: didn't need to be so paranoid about $_SESSION, and actually was causing more issues than it solved.

(Bug 19725) Do not include suppressed edits in the "View X deleted edits" message, and when doing prefix search of special:undelete.

I'm not 100% sure this is the right thing to do, see the bug for the details. But basically this doesn't include an edit in the count if its text is hidden and its hidden from admins. (Not sure if it should not be included only if everything is hidden). Its also weird to show people different things depending if they have suppress rights, without really indicating that.

Minor db note: This causes the query to no longer use a covering index. I don't think that matters but just thought i'd mention.

p.s. The upload page show deleted edits link is broken right now, (from before) I'll fix in a follow-up.

* Pass the WebRequest object to User::newFromSession() and use it instead of $wgRequest (will still fall back to $wgRequest if not passed or another method to get an User object)
* Always call User::newFromSession(), not only when not in CLI mode
* Removed User::$mSkin, unused and not set

Removed usage of error suppression operator

Revert r91426 and followups r91427, r91430: Breaks Gallery-related parser tests

Changed all dynamic calls to Linker methods into static ones in ImagePage.php.
ImageHistoryList is not used in extension, so I also removed ImageHistoryList::getSkin().

Override Pager::getTitle() in core subclasses of ReverseChronologicalPager that don't have it

Redoing r87173 with scrollbars instead of word-wrap per CR

Revert r86047 for now per CR.

(bug 6100; follow-up to r91315) Being bold and removing $wgBetterDirectionality (and dependent wfUILang) in core, as most or all work is finished.
Also:
* Introduce classes mw-float-end, mw-float-start so we don't have to use inline css depending on wfUILang()/$wgLang (see HistoryPage and SpecialFileDuplicateSearch)
* Add direction mark to protection log
* Remove specialpageattributes as it is obsoleted by this commit (also fixes bug 28572)
* Add two direction marks in wfSpecialList, which makes ltr links on rtl wiki (and vice versa) display nicely as well (only on those special pages however)
* Revert r91340 partially: use mw-content-ltr/rtl class anyway in shared.css. Both ways have their [dis]advantages...
* Set the direction of input fields by default to the content language direction (except buttons etc.) in shared.css

(bug 29680, bug 22911) Add GetDeaultSortkey hook in order to override default sortkey used on pages (default is {{PAGENAME}})

Some people want it to default to subpage or to fullpagename this hook would let
them do what they want :)

(Follow-up r91414) Also take the page name in account when considering how much of page to put in auto-summary.

Move limit from 200 bytes to 250 bytes - 5 left over just cause (+4 bytes for the $1 and $2). The other auto-summary still uses 200 as the limit (I suppose they don't want it too big or soemthing?).

code readability is everything

Division by zero

mw.util.wikiGetlink default to wgPageName
* (bug 29723) mw.util.wikiGetlink() should default to wgPageName
* Solution by mybugs.mail

* Adding unit tests

another fix for old jQuery UI on live

mw.util optimalizations:
* Remove redundant mw>mediaWiki passing through IIFE
* Using local alias to global mw.util

Reverted part of r91246, fixes: Fatal error: Call to undefined method RawPage::getSkin() in C:\wamp\www\MW_trunk\includes\RawPage.php on line 156

Localisation updates for core and extension messages from translatewiki.net (2011-07-05 21:00:00 UTC)

Per Brion, fix for r91460: make this a bit more robust

Override IndexPager::getTitle() in AlphabeticPager instance of core to not rely on $wgTitle

* Don't create a WebRequest obhject in CLI mode but a FauxRequest; avoids some useless notices about headers already sent (I know this is more a PHP silliness, but anyway)
* Added HTTP response code parsing (sending a "HTTP/1.x code" header was throwing a NOTICE about undefined index on the result of the explode() call) and storage; added FauxResponse::getStatusCode() to retrieve it

Hashar seems to have missed one occurence of OutputPage::getStatusMessage() in r91425

(bug 5800) Added $formCallback as a parameter to the hook EditPage::showEditForm:initial

Removed usage of error suppression operator

Removed CategoryViewer::getSkin() and replaced by static calls to Linker; no use in extensions.

Added percentages to ProfilerSimpleText

(follow-up r91436) add new hook to hooks.txt

I totally updated the doc file, and then committed my other changes without the doc file.

(Follow-up r90759 per CR) Use a hook to register new Collations instead of just taking the collation name as a class name

* Added getFileVersion()/setFileVersion() functions to OutputPage
* Removed getDisplayedFile() from FlaggedPage and simplified getFile()
* Cleaned up getIncludeParams() to just do formatting
* Made template/file IDs mandatory for RevisionReviewFormUI

Follow-up r91432: I always forget release notes..

(see bug 29712) Removing the defaultUserOptionOverrides in MessagesXx.php because it is broken (and has been for a long time) and secondly, it is only used for things that can be done otherwise:
* 'editfont' => 'sans-serif'; -> trivial (can be set in CSS)
* 'underline' => 0; by languages written in scripts that are hard to read with underlines -> now set in shared.css, which makes it work also when they are used as interface language (note that [lang="xx"] is not yet present on wikitext content by default but will be soon as part of my work on RTL improvements)
* 'quickbar' => 2; by RTL languages. For this, I introduced a new option (5) that sets left/right according to the directionality of your interface language (and automatic is better than setting it in MessagesXx files!). Note that the broken feature was in this case "corrected" by CSS flipping, causing preferences to say "left" while it was actually "right". This commit fixes that bug as well.
Third, it actually would have made more sense to have it them load for wgLang instead of wgContLang, but that's not possible because wgLang is itself dependent on preferences

Followup r89617, r91428 call $pager->doQuery explicitally so we get a set result object, from which, we can then get a row count

Expliticly marking doQuery() in pager as public

using proper margin heights: 14 + 68 * 2 = 150

Revert r90678 per CR

better fix for bug 27338 - doesnt rely on line-height

fixing rounding problem, per comment at r82309

HTTP status messages move: OutputPage -> HttpStatus

OutputPage::getStatusMessage() is a method to convert a numeric HTTP status
code to an english message.  It does not really belong to the OutputPage were
it was for historical reason.
This patch move the basic function to a non MediaWiki dependant class in our
directory includes/libs.  We could potentially enhances it, but I do not see
any use cases for us yet.
I have renamed the method to the shorter 'getMessage' since the word 'status'
is now in the class name.

Summary of changes:
* OutputPage::getStatusMessage becomes HttpStatus::getMessage
* Method moved to the new includes/libs/HttpStatus.php
* Autoloader updated
* History kept by using 'svn copy'
* No functional changes
* No input/output format changes
* Old occurences modified in phase3
* OutputPage::getStatusMessages() marked as deprecated

PHPUnit test suite is fine.
Tested manually using a redirection

mediawiki.util.test.js: TestSwarm reports breakage in Chrome4-Linux
- It only appears on Linux not in Mac OS X with the same Chrome version
- Adding extra checks and stops() in attempt to get a more useful report than then sudden stop at:
http://toolserver.org/~krinkle/testswarm/?state=runresults&run_id=1258&client_id=3092#

Localisation updates for core and extension messages from translatewiki.net (2011-07-04 19:53:00 UTC)

tests for Html::testExpandAttributes()

FIXME: seems a code duplication of Xml::expandAttributes()

follow up r81571 (skip attributes with null value)

Removed doPurge() from ImagePage.php, already moved to WikiFilePage

Revert old page navigation back to a subtitle

Please reapply to trunk for inclusion in 1.19.

reverts r85322 & r90883

jquery.client unit testing
- More elaborate caching in the plugin in order for the code to be testable
- Added support for passing custom user agents (although the default behavior remains unchanged)
- Added support for passing custom profile-objects to $.client.test
- Added test case for all supported browsers according to http://www.mediawiki.org/wiki/Compatibility#Browser
- Replaced userAgent.toLowerCase() with userAgent since this string was already converted to lower case a few lines up

This will likely prevent bugs like bug 27652 and bug 29446.

(bug 29109) Allow the automatic edit summary for redirect creation
show the first bit of the new redirect page.

Adds $2 to the autoredircomment (not used by default) that contains the first 150-ish characters of the newly created redirect page.

Meant so that the Wikipedian's can see their {{R from foo}} templates.

* Removed usage of error suppression operator in includes/db
* Changed an usage of $_REQUEST to $wgRequest

Use WebRequest::getValues() instead of raw $_REQUEST

Allow SqlBagOStuff data to be split over many tables, to avoid lock contention performance issues on servers with a high write load. See bugs.mysql.com/bug.php?id=61735 and http://bugs.mysql.com/bug.php?id=61736 .

Fix for r90337 and r90743: Use $this->title instead of $wgTitle, also groupped global declarations at the top

Changed calls to Linker::tooltipAndAccesskeyAttribs() to be static and fixed casing in method's name

Fix for the broken tests introduced in r91387.

Check input before assuming there are no arguments and the values object is expected.

Adding tests for expected reponse when passing invalid values to Map.get
* These are currently @broken
* Map.get should (as documented) handle an object for multiple or a string for a single selection and return the value(s). Right now, however, the function returns the values object if the first two if-cases are not matched. It needs an additional check to verify that there are indeed no arguments passed.
* Adding comments in the test suite

Minor optimalizations and jsperfs in mediawiki.js
* Use strict comparison to undefined where possible (arguments and object/array members)
* Make sure $ is not implied global (using jQuery.isFunction at the bottom instead of creating another 1-line closure)

Use Linker::linkKnown per Siebrand on r91377.

Localisation updates for core and extension messages from translatewiki.net (2011-07-03 19:50:00 UTC)

enhance tests name (for --testdox option)

Changing back to $wgUser in SpecialImport.php. Importing pages returned "call to undefined method ImportReporter::getSkin()" due to r91246.

split tests for ArticleTest

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

Moved doCascadeProtectionUpdates() to WikiPage.php

Fix for r91344: removed straggling instances of $this->special in item classes

Bug #29699: Minor adjustments to Vector search button

Patch from Erwin Dokter as an alternative(?) to r73427.

Applying here so that it can be tested and compared.

The User object parameter of WikiPage::doRollback() and WikiPage::commitRollback() was not passed by the Article implementation because it was not present.
While I'm a it, WikiPage::doRollback() and WikiPage::commitRollback() now require that parameter, moved the check to Article.

Removed one hidden usage of $wgUser

Moved action=rollback to an Action subclass