Commit graph

716 commits

Author SHA1 Message Date
jenkins-bot bd273db64b Merge "Add phpunit tests" 2016-11-16 22:51:45 +00:00
Baha 76108cba37 Add phpunit tests
Bug: T131105
Change-Id: Ib774f18e62f050f48783e5ccccaedc23000533d2
2016-11-16 16:34:19 -05:00
Translation updater bot 009be59a30 Localisation updates from https://translatewiki.net.
Change-Id: Id5b4cfda9bd479e2ec51b36e4b6554e971bb5881
2016-11-15 22:58:14 +01:00
Mukunda Modell c399c4156e Don't attempt to call methods on $file if it's not an object
This fixes a fatal which is blocking deployment of 1.29.0-wmf.1
refs T149059 fixes T149849

Bug: 149849
Change-Id: Ieb61585af3aa60b7af58597091151d0b494b2fd6
2016-11-03 18:36:32 +00:00
Chad Horohoe 912335ce39 Whoops, track not trace
Change-Id: I2dcc6c7e9dce5d924d74d3653fdfc7a18a02d053
2016-10-24 17:03:10 -07:00
Chad Horohoe 6665071188 Swapping defaultbranch for trace
The former is a maintenance nightmare when branching.

Bug:T146293
Change-Id: Ic34d551709c6f436994b5a9957047ff5de65b771
2016-10-24 16:38:04 -07:00
jenkins-bot 88481cab07 Merge "Convert PageImages to extension registration" 2016-10-17 16:10:59 +00:00
Reedy d52f94205f Convert PageImages to extension registration
Bug: T87953
Change-Id: Icc9096060899b9e401d53590f38865b0937a73ff
2016-10-16 20:45:54 +01:00
Translation updater bot eef1d08d98 Localisation updates from https://translatewiki.net.
Change-Id: I7f26a75c6881f15cce5bb9b6eda9be6591fb6b75
2016-09-20 22:40:14 +02:00
Translation updater bot 48bbbd1497 Localisation updates from https://translatewiki.net.
Change-Id: Ic798f2003eda92206eed5fb85b9d85bd08c15391
2016-08-08 22:34:40 +02:00
Translation updater bot 59fcfc3bee Localisation updates from https://translatewiki.net.
Change-Id: Id7e66681bcd69f36e97999dae931d938c57e1122
2016-08-05 22:33:48 +02:00
Translation updater bot 8faba0d003 Localisation updates from https://translatewiki.net.
Change-Id: I49971695d25b0b95c5f3364f38588d520f78e8aa
2016-04-12 22:55:40 +02:00
Kunal Mehta 27a48ae1bf Use DatabaseBase::addQuotes() for quoting query conditions
Change-Id: Ia63d5af2c01b9441e601c8ba11e8e9f1b5da95a9
2016-04-04 16:34:23 -07:00
Thiemo Mättig 6c53cac48e Refactor duplicate code in LinksUpdateHookHandlerTest
Change-Id: I9acaadf18e859758cd7bdcb7d260007402695060
2016-03-10 16:27:40 +01:00
Gergő Tisza 151a5d7248 Add sanity check test for LinksUpdateHookHandler::getMetadata
Change-Id: I840902c8397c8442def8239504ce1cfa8eafbb8e
2016-03-07 22:53:00 +00:00
Thiemo Mättig 4fee70fc1f Remove not needed "return true" from hook handlers
* Do not "return true" in hook handlers. I was told it's good practice
  to either return false on failure, or nothing/null.
* Remove "static" from method that does not need to be static.
* Make some type hints more specific.
* Add missing imports. I wonder how this can work without the imports.
  My PHPStorm complains.

Change-Id: Ia0e980ff99f0e004d700b22dd07ff17f04bed4ec
2016-03-07 12:07:06 +01:00
Translation updater bot 068f58843d Localisation updates from https://translatewiki.net.
Change-Id: I00112b162b94c56278e96896d510d0df41215aa2
2016-03-06 22:15:19 +01:00
Petr Matas 5a60c10b1a Thumbnails generated from SVG may be larger than original
Bug: T128278
Change-Id: I2b10ff28389fd9c99f0703d7ae4ee2d63e0bd000
2016-03-02 12:21:00 +00:00
Gergő Tisza 7c78ba622c Weigh images by copyright status
Fetch the extended metadata for every image, and if the NonFree key
(provided by CommonsMetadata) is present, rank down the image.

The check is run on LinksUpdate so changes to image copyright
templates won't have effect until the page is reparsed.

Unlike other weights, the work of extracting the weight factor from
the file/page is wholly done on LinksUpdate, instead of doing most
of it on parse and storing as ParserOutput extension metadata.
This is to avoid getting the article page parsing and the file
description page parsing conflicting with each other.

Bug: T124225
Change-Id: I21111ecbc80ded864806a2a93002f3254c3c68a9
2016-03-02 00:59:02 +00:00
Translation updater bot 8e9c6fbade Localisation updates from https://translatewiki.net.
Change-Id: I596741985b213b92be14b423744cc55a283046d9
2016-02-19 22:10:02 +01:00
Thiemo Mättig 0e8d85a333 Rename class to plural ParserFileProcessingHookHandlers
As discussed in I7fdc582.

Change-Id: I76ffd85323f2b6b8ad4a22ee4671f0bb5a95456d
2016-02-14 12:21:00 +01:00
jenkins-bot 9f09934913 Merge "Extract ParserMakeImageParams/AfterParserFetchFileAndTitle hook handlers" 2016-02-12 17:29:37 +00:00
Translation updater bot 1bf982fde1 Localisation updates from https://translatewiki.net.
Change-Id: I67072f1e8a99c44320d7506a8bb7640c5c178f56
2016-02-03 23:08:00 +01:00
Kunal Mehta 59afdff5d5 Don't pass default pref as fallback to User::getOption()
User::getOption() already handles that, and this code bypassed the hook
in User::getDefaultOptions().

Change-Id: I41f9df177988dffd62de0060cb691a97161729e4
2016-01-26 10:07:39 -08:00
jenkins-bot 74fde6270d Merge "Extract LinksUpdateHookHandler to separate file" 2016-01-25 18:20:41 +00:00
Translation updater bot 091012621b Localisation updates from https://translatewiki.net.
Change-Id: I806bb6c0b77019a4c9934c1d15209bc4e80fe37c
2016-01-18 21:36:58 +01:00
Thiemo Mättig fe0da50344 Extract ParserMakeImageParams/AfterParserFetchFileAndTitle hook handlers
This patch only moves existing code around, but does not change any
implementation detail.

I found it very suprising that all code called by these two hook handlers
is 100% exclusive to these hook handlers. There is zero interaction
between these hook handlers code and all other code. Why is it in the
same file then? And why is it all static? It doesn't have to be. I
had to change literally nothing, except cutting and pasting, removing
all "static" and replacing all "self::..." with "$this->...". That's
all.

Change-Id: I7fdc582db425d3b95f7d02934b439eb9c102e712
2016-01-18 13:00:21 +01:00
Translation updater bot cb184e1349 Localisation updates from https://translatewiki.net.
Change-Id: I50619e480ee1849f46a79f359154233d320d906d
2016-01-17 22:33:41 +01:00
Max Semenik c0ecb9cd0c Add license name to Special:Version
Change-Id: I176dd2bfd34b6a8f1476f78d78a8cd8efa523519
2016-01-12 13:55:31 -08:00
Thiemo Mättig d9df7178a0 Extract LinksUpdateHookHandler to separate file
This patch only moves existing code around, but does not change any
implementation detail.

I found it very suprising that all code called by this hook handler
is 100% exclusive to this hook handler. There is zero interaction
between this hook handlers code and all other code. Why is it in the
same file then? And why is it all static? It doesn't have to be. I
had to change literally nothing, except cutting and pasting, removing
all "static" and replacing all "self::..." with "$this->...". That's
all.

Change-Id: I5ffe6fdf4e57135e6f3b32636c80f22be758607c
2015-12-30 13:40:48 +01:00
Kunal Mehta 7312d0846e build: Updating development dependencies
* grunt-jsonlint: 1.0.4 → 1.0.7

Change-Id: I7748cc73763bc2d1396ab29262169809c0efd4db
2015-12-29 11:46:04 -08:00
Translation updater bot de2e4e6101 Localisation updates from https://translatewiki.net.
Change-Id: Ie5f3d5034564aea6f609f4f56b70deceb5d314bb
2015-12-22 22:11:20 +01:00
Thiemo Mättig 9b9d7d93c2 Author prefers full name in @author tag
Requested by the author in
https://gerrit.wikimedia.org/r/#/c/253327/1/includes/PageImages.php

Change-Id: I0e48ea1f3e7bc7098901562e7f048849c27e153a
2015-11-17 09:54:24 +01:00
Thiemo Mättig 369eda24b8 Add missing and fix wrong @license tags
Change-Id: I07f77735bb0050e2e59491d3c045b11705f3a08f
2015-11-16 15:59:34 +01:00
Translation updater bot 77dd2bb0f7 Localisation updates from https://translatewiki.net.
Change-Id: I2f89c63dbb83d38fa2e79b164d9c0d31077e79a9
2015-11-13 21:34:24 +01:00
Thiemo Mättig 9b65ac4def Organize code in includes/maintenance directories
Change-Id: I3a243074049a5b8b212de5bcd1e341c36e4a13e0
2015-11-12 08:33:43 +01:00
Translation updater bot 3f68a93660 Localisation updates from https://translatewiki.net.
Change-Id: I8cd6f4c5f2a7df3d5e82923baf0791963b73137a
2015-11-08 22:12:09 +01:00
Marius Hoch 4d81dbac99 Slightly improve type hints
Change-Id: I620135289b4f0ff95ed3013420194570c26b7a4f
2015-10-31 02:31:13 +01:00
jenkins-bot d55ef9d7e2 Merge "Use mExtensionData instead of undeclared class property" 2015-10-31 01:21:48 +00:00
jenkins-bot 41e8483a0b Merge "Add inline documentation and newlines for readability" 2015-10-31 01:21:46 +00:00
Translation updater bot abb498deeb Localisation updates from https://translatewiki.net.
Change-Id: Idd7c0d27efa9c935956183c24b5a20b849332698
2015-10-27 21:33:11 +01:00
Thiemo Mättig 1f1c72bb2c Use mExtensionData instead of undeclared class property
Change-Id: I155e37fda412cd467da7b34c52b34661c7c20224
2015-10-26 12:09:52 +01:00
Thiemo Mättig 23cf541f1b Add inline documentation and newlines for readability
Change-Id: I3a5a4ef67836cbfe0277f511f8b23030d248141c
2015-10-26 11:41:13 +01:00
Translation updater bot 00a663a893 Localisation updates from https://translatewiki.net.
Change-Id: I7aef32dd951c52468f64d9be0aa9e55d3b0343c1
2015-10-22 22:02:39 +02:00
Translation updater bot 7851ed8159 Localisation updates from https://translatewiki.net.
Change-Id: I2d02cb4e837400472a1d18252ab6b12e47b08447
2015-10-15 09:48:48 +02:00
Translation updater bot 4ac82d2836 Localisation updates from https://translatewiki.net.
Change-Id: I152f5aa28e847cccabf0062acf1e9723b7ac89a8
2015-10-13 22:12:24 +02:00
Translation updater bot e3488dd60b Localisation updates from https://translatewiki.net.
Change-Id: Ic594e7579032de698ff21b75a9e12f7eb9b073af
2015-10-12 21:05:25 +02:00
Kunal Mehta ca63a0c207 build: Updating development dependencies
* grunt-banana-checker: 0.2.2 → 0.4.0

Change-Id: I373da28af617774905140f8af53426a2935ae0d2
2015-10-11 11:54:23 -04:00
Translation updater bot 6c79e9833d Localisation updates from https://translatewiki.net.
Change-Id: I5b6a78f3d8bd320305a3c16f9aedc562063aef0e
2015-10-10 22:43:14 +02:00
Translation updater bot 4bf28a73d0 Localisation updates from https://translatewiki.net.
Change-Id: Ia30dc6ca5329d641dbd18c7fb56d187a3e253e8f
2015-10-09 22:32:51 +02:00