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

Added title type hint to invalidateImageRedirect here too

* Added File::normalizeTitle() function and used it to consolidate file title validity checks and sanitation. (bug 32195)
* Broke a few long lines.

* Added some Title type hinting
* Used 'self' in some class construction instances

Localisation updates for core and extension messages from translatewiki.net

Windows compat for a couple of tests :(

Apply for Bug #31694 -- "During installation, tabbing order (cursor focus) goes to logo instead of 'continue'"
Patch From: Grunny

Apply patch from Bug #12262 -- "Indents and lists don't align"

Some cleanup to EditPage:
* Added displayPermissionsError() to show the permissions error page with the content if there's content to show or it will throw an normal exception if there's nothing to display; this restores a functionnality lost at some point and added new message 'viewyourtext' that is displayed when the user made changes to the content (e.g. when the block happens while the user is editing the page)
* Marked EditPage::readOnlyPage() as deprecated since displayPermissionsError() is now used. This also means that OutputPage::readOnlyPage() is no longer used to display such error pages (but still have to keep that functionnality since it's used by extensions, same for EditPage::readOnlyPage(), *sigh*)
* Removed blockedPage()'s display of content since it's nearly never called (a blocked user would trigger the path mentionned above, not the check in internalAttemptSave()) and also removed 'blockedoriginalsource' and 'blockededitsource' messages that are no longer used
* Use OutputPage::prepareErrorPage() to prepare error pages instead of doing all that stuff directly
* Moved noCreatePermission() near other error-related functions
* Throw an exception directly in attemptSave() instead of calling deprecated methods and marked noCreatePermission() and blockedPage() as deprecated

And while I'm at it, move token check to onView() so that if action=markpatrolled is called without parameters, the user will see 'markedaspatrollederror' error and not 'sessionfailure'

Updated docs for r101896

Fixes Bug #31722 — “ugly old toolbar icons - height inconsistent” —
patch from diebuche.

eol w/s

Fix history versions

1.17 -> 1.18

* fixed ipblocks.ipb_by_text field, removed default blank not null (fixed install&update)
* fixed Block->insert; ipblocks.ipb_id is autoincrement field (should use sequences for compatibility)

Use WikiPage instead of Article to call the hasViewableContent() method

* fixed a typo in oracle/tables.sql
* readded the sha1 fields (installer & upgrader)

Per Platonides, fix for r101630: correct class name

Removed debug code from r101944

Partial revert of r101912 -- restores r99315 'includexmlnamespace' parameter for API, but keeps the logic to keep that from clobbering alternate XML output formats with their own namespaces.

alpha -> beta

Merge RELEASE-NOTES changes

Documentation

Fix mime-type on image

Resync 1.18 RELEASE-NOTES files

Add Central Dusun language code to RELEASE-NOTES-1.18

* (bug 31878, bug 31542) Fix XML namespace output in API; removed now-unneeded includexmlnamespace parameter.

Any existing uses of includexmlnamespace should be ignored and harmless.

If output data already includes a specific default namespace as an xmlns attribute, it is now retained -- so for example the RSD API discovery module outputs the appropriate namespace instead of the generic API one.

Partial revert of r99135 which added the includexmlnamespace parameter as a temporary requirement to get the xmlns attribute included.

Followup to r88007 which added the xmlns originally, but in the wrong order so it overrode existing output data.

Follow-up r101808. Use canonical class name.

FU r96015: removed & in MakeGlobalVariablesScript hook

eol w/s

Bug 28172 - wfGetDB called when it shouldn't be

Fix fixme on r101177

Whoops, need a !

Localisation updates for core and extension messages from translatewiki.net

*grumble grumble* know it alls think I'm abusing $wgMiserMode. So fine, have a SHINY NEW GLOBAL INSTEAD. bug 32072, r92109, yada-yada.

This is not a "useless" error message, regardless of what the bug claims.

Fix for r101858: forgot the override those methods

bug 32183: remove the client-* classes added from user-agent-sniffing onto the <html> element.

Browser sniffing is bad, and having these here encourages unpleasantness.
Keeping the js/nojs ones as those are useful and non-evil (they're an example of feature detection, exposed via CSS here rather than through JS.)

Follow-up r101858: forgot to commit AutoLoader.php

Per Aaron, fix for r89405: introduced DerivativeRequest to allow to override the request parameters, but not headers, cookies and session. Updated ApiEditPage (for the bug) and ApiLogin (for future use)

Add to jquery.client's qunit tests by actually testing the test() function against the sample user-agents.

Also changed from hardcoded constant of how many user-agents to test to counting them so it won't get out of sync.

Added Firefox 10 and IE 10 sample user-agent strings in to make sure they test correctly.
They *do* test correctly *if and only if* you use raw numbers for a major-version test (eg > 7). If you tried > "7.0" you would find it fails on the version 10 browsers.

Followup r101834 - this time tested

Quick ugly fix to avoid duplicate log entries in rc feed.
I didn't think that save() would send lines, since there was function to do that (added in r45725 - duplicating existing code).
This part of codebase could use much TLC

let us force thumb rendering even if previous file exist

The new FORCE_RENDER flag should always be used with FORCE_NOW. It let us
override the file_exist() check and thus trigger a thumb rendering even if
the file exist.

Will be needed by the vipscaler extension.

w/s fixup

add \n to a wfDebug() call

follow up r48417

typo

comment that isValidURI must reject file:// URI

use isValidURI for redirect check

man page help pages generation

Tweaked mwdocgen.php to let us generates man pages. They will be saved to
./docs/man so make sure to update your MANPATH if you use this feature.

Usage:

 php mwdocgen.php --all --generate-man
or
 make man

eol w/s

followup r101490, remove extra tabs after talking to Nikerabbit.

(bug 31944). As a work around for bug 31944, don't extract tiff:YCbCrSubSampling from XMP.

Several files have wrong type for that property, XMPReader currently marks entire file as invalid instead of just specific property, if there is a type mismatch in XMP data.

* Set constant default values in class definition and not constructor
* Added getTitle()
* Mark constructor and getArticle() as public
* Removed a PHP4-ism in constructor

Cleanup to deletion related code:
* Use the new 'cannotdelete-title' message for the title on error instead of "internal error"
* Don't use OutputPage::showFatalError() but add the content by "normal" methods (related to the item above)
* Don't use the $DeleteReason and $DeleteReasonList member variables in Article and FileDeleteForm but local variables since they are not used anywhere else
* Moved down some variable definitions near where they are used so that they are only set when they will actually be used
* Same for the setPageTitle() call with 'delete-confirm' message in Article::delete() that is now in Article::confirmDelete()
* Added missing "true" as fourth parameter of WikiPage::doDeleteArticle() call that was missing
* Factorised getTitle() and getUser() calls
* Salt token with array( 'delete', 'page title' )

* Use WikiPage instead of Article and call doDeleteArticle() with a custom error message instead of calling doDelete()
* Call Title::getUserPermissionsErrors() instead of User::isAllowed() to know whether the user can delete the page

Typofix in comment

Address fixme on r101606, should also call from $this

Make use of Title::isSpecialPage() in various places

Per comments on r99911 move $wgMaxImageArea check back to normaliseParams(). Added hook BitmapHandlerCheckImageArea to override the area check. I'm not very happy with this overly specific hook, but I don't see a clear way to obtain the functionallity required otherwise.

Remove the width and height params from BitmapHandler::getImageArea(). There is really no reason for them to be there.

Localisation updates for core and extension messages from translatewiki.net

Early return if the page exists and redlink parameter was provided instead of going through all the normal process and then redirect; also moved a part of outside the "else" condition for consistency with the code below

avoid E_STRICT warnings when no --installdbpass is given

* Let HTMLFileCache constructor take in a Title *or* a the prefixed dbkey itself.
* Tweaked filecache fallback in fileCachedPage() to try the raw title param. If the DB is down, we can get most views of titles with colons in them to work this way. Previously, it could fail on an interwiki lookup.

* (bug 32126) Fix 1.18 regression in watchlist editor when items already removed from watchlist

Overriding validation in HTMLForm bits so the extras can be safely ignored.

Alias NS_MEDIA page views to NS_FILE. Fixes bug 32032.

(bug 31822) Error during upgrade due to output buffer reset in stdout.

In various places in Maintenance, we're overwriting our output buffer rather than appending to it. This doesn't matter too much in the CLI, but it can corrupt headers when upgrading from the web. Thanks Woozle for pointing this out.

FU r101629, made Pager only call doBatchLookups() if there are rows

* Use WikiPage instance of Article
* Changed to call to directly create a Revision instead of going through WikiPage or Article

* Made PermissionsError exception accept an optional second parameter for the description of the errors (as returned by Title::getUserPermissionsErrors())
* PermissionsError now calls OutputPage::showPermissionsErrorPage() to display the error (this is needed to make the item above work correctly)
* Removed the override of the HTML title in OutputPage::showPermissionsErrorPage() so that it shows "Permission errors - Sitename" instead of simply "Permission errors" for consistency with the other things
* Pass the error array returned by Title::getUserPermissionsErrors() to PermissionsError where available
* Converted direct calls to OutputPage::showPermissionsErrorPage() to throw an PermissionsError error instead
* Added 'action-rollback' message that will be displayed when accessing action=rollback without sufficient rights
* Changed getRestriction() in subclasses of Action to return null when they previously returned 'read' so that user rights can be check with Title::getUserPermissionsErrors()
* Reordered checks to do first user rights, then block (if needed) and finally read only (also if needed) so that users don't think the error is temporary when they both don't have right and the database is locked