Properly check permissions.

Localisation updates for core messages from Betawiki (2008-08-28 23:03 CEST)

Bump $wgStyleVersion for r40184

Make UploadFromBase::getTitle more logical by returning either a title or null

Replace the <p><br /></p> inserted between the toolbar and the preview area with a ontop class. In shared.css we give this a bottom margin of 1em to add the space between the toolbar and preview.
As a bonus, because display: none; is now used when the preview area is not used, ajax/live previews actually get the spacing instead of it not showing up.
preview.js has also been tweaked to remove the display: none; when previewing.

wfReadOnly now checked in ApiMain::requestWriteMode

(bug 15358) Article::doEdit() ignores $user when inserting a new revision

Use static methods to check whether upload is enabled

Localization update.

Full update for file

fix typo, format section

Implements #11884 : Now support Flash EXIF attribute

Credits for previous commit; patch was by Jeremy Baron

(bug 15348) prop=categories returns timestamps that aren't in ISO 8601 format

wfDebugMem() to report memory usage

Improve FakeTitle implementation:
* Make FakeTitle a subclass of Title so functions with a type hint like Parser::parse() accept FakeTitles too
* Remove two static functions from the FakeTitle class

Once we reset, no need to keep looping.

Localization update.

Modernize the email user form to our actual standards:
* Add fieldset
* RTL compatability
* CSS classes/IDs

Update files

Localization update.

Fixing 'recentchanges-legend' message text, now compatible to 'watchlist-options'.

Add colons to the form labels for consistency with all other forms.

Localization update.

Replace arguments before replacing variables in {{int:...}} parser function, for compatibility with messages that require it

* (bug 15172) 'Go' button of Special:Recentchanges now on the same line as the last input element (like Special:Watchlist too)

* Fix bug 15344. Was caused by the overwrite of $entry, assigned by reference to the last link and then overwritten in later code.
* Split doVariants() out from replaceInternal(), big functions are prone to variable reuse bugs

Add some caching to the group counts.

(bug 13471) Added NUMBERINGROUP magic word. Patch by Soxred93

(bug 15342) "Invert" checkbox now works correctly when selecting main namespace in Special:Watchlist

Localisation updates for core messages from Betawiki (2008-08-27 23:24 CEST)

Revert part of r40107 (Revert r40042 because of regressions). The changes to UserMailer.php had nothing to do with that bug.

Revert r40042 "* In Article::replaceSection(), actually return null when $section is bogus. Used this in my half-complete and now kind of abandoned attempt at rewriting EditPage.php"

This causes regression bug 15340 -- null-edits to a section destroy the rest of the page.

Set replace flag for headers

Make Special:Allpages/User:Foo actually work. Patch by Mormegil

Add an own message for the fieldset legend instead of duplicating the special page name

Localisation updates: Using a better Persian translation for "hide"

Fixup the function declarations that we're overridden from UploadFromBase.

Lets render BMP pictures to PNG.

Move the timestamp into the message of the current revision link to make the link consistent with the link to the previous version on the left side

* Remove a debug statement
* Return a status on unsaveTempFile

Splitting backend upload code from SpecialUpload.
* All common upload code resides in UploadFromBase. Then depending on the upload source, one of its derived classes is initiated by Special:Upload.
* SpecialUpload::ajaxGetExistsWarning now only returns warnings that are related to existence.
* Allow LocalFile::upload to attribute the upload to another user than $wgUser

This introduces breaking changes for upload extensions.

API:
* Add titlePartToKey() and keyPartToTitle() which use the substr() hack to preserve trailing spaces
* Migrate function calls where needed. ??continue parameters still use titleToKey() because they're generated using keyToTitle() and therefore can't contain trailing spaces

Fix fatal error caused by missing comma

Reverting my reversions of the fieldset on Special:Watchlist. I liked it, and apparently some other people did too (although some don't). Plus, it's been pointed out that semantically, a fieldset belongs around fields. Who knew? Could use some CSS tweaks maybe.

Can't add to a variable if it doesn't exist...

Fix a minor regression pointed out by aaron.
The spacer for between the preview area and the edit toolbar was being outputted even when not previewing.

Remove messages and rebuild all messages files per r40076.

Remove fieldset from Watchlist that apparently only I liked :)

Check on $wgEnableEmail && $wgEnableUserEmail

* Fixup validation methods in UserEmailForm a bit so that they don't return arrays when it's not necessary.
* Add email errors to the API's message map

Remove unused messages, function deleted with r39717.

Patch by Shlomi Fish to fix this test

Add $wgPhpCli var so shell outs know what exec name to use

Minor error... sorry bout that, forgot to remove the > when I redid the fix locally.

Recommit the EditPage changes:
Thanks for the heads up brion. $attribs was being overridden in EditPage::showTextbox rather than having individual things set which caused the tabindexes inputted to be ignored. Your test case is fixed.
I did a page diff, there are no more ui regressions. The only difference is the preview-toolbar <p><br /></p> has been moved from the inside of the preview to outside of it. There's no real difference in that.

Revert r40027 "Recommit my changes to EditPage without the regressions:"
This caused at least a regression in tab order which causes all my edits to fail:
1) Click "edit"
2) Hit "tab" to tab into edit field
3) Type something
4) Hit "tab" to tab into the comment field
5) Hit "enter" to save
Hey wait a minute, where's my edit?
When I go back and do it over, I find that I get tabbed from the edit box to the *cancel link*, so hitting enter goes back to the view page, destroying my edit.
Not good. :(

Haven't checked for other regressions; please be *very* careful with this edit page work, as seemingly small breakages can really damage the user experience and make things really badly broken.
Maybe we need a fuller UI test suite...

add more info

log strange anomaly

Reset password throttles on login

API: Remove the throttle from ApiLogin because
* it's been obsoleted by a similar throttle in core
* it uses time(), which causes problems when you have multiple servers with a significant clock skew between them

API: Improvements to action=emailuser:
* Add two messages from ApiEmailUser to ApiBase::$messageMap
* Add email tokens to prop=info

API: Fixed bug that caused all action=edit requests to return with the nochange flag set on wikis where DB_MASTER != DB_SLAVE

API: Throwing a more specific error message when the client tries to create empty new sections, which isn't allowed. Unknown error: ``228'' isn't very user-friendly.

-T

Fix for r40041. PHP strict notice. Thanks to Ialex

Localization update.

* (bug 15264) Underscores in Special:Search/Foo_bar parameters were taken
  literally; now converting them to spaces per expectation.

* In Article::replaceSection(), actually return null when $section is bogus. Used this in my half-complete and now kind of abandoned attempt at rewriting EditPage.php
* In UserMailer::send(), allow the caller to override the Content-type header so people can send multipart messages if they really want to. Using this in an extension I'm developing

* Add tooltips to rollback and undo links
At least on de.wiki the users with the review right have the rollback right too. They are confused about the difference rollback/undo. Hope that helps a bit

Quick hack to fix the broken hack noted in bug 15301

Use of:
$action_desc[0] = strtolower($action_desc[0]);
is horrible wrong for a few reasons:

1) The first-byte match fails for all non-ASCII characters -- use lcfirst()!
2) System strtolower() or lcfirst() may not work for UTF-8 or lang-specific issues properly; use $wgLang->lcfirst()
3) You shouldn't be forcing things to lowercase here anyway, you can't know it's correct gramattically.

This whole message output should be redone cleanly; either all the rights info messages should be designed to work in an inline list sentence, or they should be formatted as stand-alone text strings using a <ul> or something.

Recommit my changes to EditPage without the regressions:
* Content was double encoded by the call to htmlspecialchars and the use of Xml::textarea
* showFormBeforeText was returning instead of outputting (This caused saves to fail since there was no edittime)
* As a little extra no-one noticed, accidentally $encodedtext was left in showTextbox2 rather than $this->textbox2.

Fix path. Patch by rindolf on IRC.

Move @ingroup around so each of the classes are picked up by Doxygen.

Revert 40016, 40021, 40022. Breaks ability to save and has html entities problems.

Redo r39954, without regression this time. Sorry :(

* Remove duplicated function in EditPage.

Whoops, this was probably supposed to be an elseif.

* Revert back to my parser pseudo-branch again. Note: if you feel like reverting this, reverting Parser.php alone to r40010 will work just fine.
* Merged replaceFreeExternalLinks() with doMagicLinks(). Makes a lot of sense, very similar operations, doesn't break any parser tests. Stops free links from interacting with other parser stages, the same way ISBN links don't.
* The pass order change fixes Brion's complaint in r39980. Early link expansion, triggered by having more than 1000 links in the page, was outputting URLs which were destroyed by RFEL. Added parser test.
* Fixed an unrelated bug in LinkHolderArray::replace(): if a link to a redirect appears in two separate RLH calls, the second and subsequent calls do not add the mw-redirect class. Caused by an unmigrated LinkCache fetch.
* Added a parser test for a pass interaction bug that the pass order change fixes.
* The fuzzer told me to tell you that free external links in non-caption image parameters, which are and have always been invisible, are now not registered either.
* Miscellaneous supporting updates to the test infrastructure.

Localisation updates Cantonese, Chinese and Old/Late Time Chinese

Make the elements of the watchlist fieldset identical to the elements of the recent changes fieldset.

Commit initial portion of EditPage refactoring.
This isn't complete, but nothing is broken.
The goal here is to make it possible for extensions to subclass the editpage and override certain areas of it while allowing the rest to keep it's presence without needing to copy code from core which may change.
* Use $this->action instead of hardcoded 'submit's so this may be overridden for subclasses that use a different action=...
* Move all the header setting into a new setHeaders() function. This way subclasses can tweak the title, this was previously impossible because of how scattered setPageTitle calls were.
* Move the previewarea stuff that was being duplicated in 2 places into it's own function.
* Move some inputs into a beforetext function
* Move wpTextbox1 into a special function. And create a textbox function to unify textarea creation. wpTextbox1 and wpTextbox2 now have a unified style due to this. (Though the metadata textarea could use this applied to it as well)
* Move edittools into a function.
* In getPreviewText;
** Don't use $wgOut
** Move template and other parseroutput related code outside of the function and simply set a $this->parserOutput variable while we're in here. (We should probably also move the $wgOut->addParserOutputNoText call somewhere else, and move most of the preview parsing into one of the initialization functions.

Localisation updates for core messages from Betawiki (2008-08-26 14:37 CEST)

Added __destruct(), for stable pseudo-branch

mark getOldIDFromRequest as public

Revert r39949 "* Revert revert r39662 of my parser changes."
Causes weird regressions on http://meta.wikimedia.org/wiki/Talk:Spam_blacklist
Couldn't isolate to a parser test in a few minutes; some kind of template interaction perhaps.

Sample bad HTML like:
The associated page is used by the Mediawiki <a href="&lt;a href=" class="external free" title="http://www.mediawiki.org/wiki/Extension:SpamBlacklist" rel="nofollow">http://www.mediawiki.org/wiki/Extension:SpamBlacklist</a>" class="extiw" title="mw:Extension:SpamBlacklist"&gt;Spam Blacklist extension, and lists strings of text that may not be used in URLs in any page in Wikimedia Foundation projects (as well as many external wikis). Any meta <a href="/wiki/Administrator" title="Administrator">administrator</a> can edit the spam blacklist. There is also a more aggressive way to block spamming through direct use of <a href="/wiki/Anti-spam_features#.24wgSpamRegex" title="Anti-spam features">$wgSpamRegex</a>. Only <a href="/wiki/Developers" title="Developers" class="mw-redirect">developers</a> can make changes to $wgSpamRegex, and its use is to be avoided whenever possible.

Revert r39954, 39958 "Tweaks for SiteConfiguration:"
This seems to have broken upload dirs on live site (eg http://upload.wikimedia.org/wiki/commons/6/66/Diemetrie.png in place of http://upload.wikimedia.org/wikipedia/commons/6/66/Diemetrie.png)
Please make no further changes without some regression testing...

this looks backwards

mark these 'public'

* Add all HTMLDiff-related classes to the autoloader.
* Move all stuff @ingroup DifferenceEngine to /diff
* Split off Node and friends to Nodes.php to make HTMLDiff.php shorter.

Add @ingroup definitions.

Add separate debug output from the normal wfDebug() output, similar to the parser debug report. Only displayed if A) $wgEnableHtmlDiff is on and B) $wgDebugComments is on.

Removed Exception used for debugging. Disabled edit buttons for HTML diff.

*stab*

(bug 15303) Title conversion for templates wasn't working in some cases

Set svn:eol-style native on includes/api/ApiWatch.php.

Reported on IRC by rindolf.

rm useless comment :)

Tweaks for SiteConfiguration:
* Introduced SiteConfiguration::$siteParamsCallback, a callback function that returns an array of params to define custom parameters to replace and wiki tags so that they can be used at run time (e.g. for CentralAuth or SiteMatrix), can also override SiteConfiguration::siteFromDB(). See the documentation of that variable for further information. Backward compatibility is maintened for paramters passed to SiteConfiguration::get() and similar functions but the vaules returned by the callback function will override them.
* Added SiteConfiguration::arrayMerge() for merging arrays, written by Werdna. Warning: as array_merge(), it may change numeric keys, so don't use it for settings having namespaces index in keys.
* Removed SiteFromDB hook, deprecated by the callback function
* document a bit

* Revert revert r39662 of my parser changes.
* Fixed image link whitespace handling (Brion's complaint, r39662)
* Added fuzz test capability to parserTests.php
* Added __destruct() functions to Parser and Language, and called them explicitly from parserTests.inc, to avoid unconstrained memory usage during fuzz testing.
* Added unified diff to output of Parser_DiffTest
* Fixed whitespace change in Parser::doTableStuff() (found by fuzzing)
* Added feature to RELEASE-NOTES which I'd committed last time but forgotten to note: <gallery> will accept image names with no "Image:" prefix (rediscovered by fuzzing)
* Limit memory usage in Title::getInterwikiLink()
* Fixed chronic fail of all interwiki link parser tests (hid Siebrand's complaint, r39464)
* Fixed chronic fail of one of the LanguageConverter parser tests. Was actually an ignored bug.

Tweak for r39944: correct fix :)