Commit graph

4407 commits

Author SHA1 Message Date
jenkins-bot d94cc34649 Merge "Add deprecated variables to PHPUnit tests" 2018-08-22 12:44:23 +00:00
jenkins-bot a762c82fe7 Merge "Add aliases for "_text" and "article_" variables" 2018-08-22 12:44:20 +00:00
jenkins-bot 5561abe296 Merge "Add a placeholder for the no-js changetags input field" 2018-08-22 12:44:19 +00:00
jenkins-bot da12b6e087 Merge "Focus the Ace editor after having added text from dropdown" 2018-08-22 12:22:47 +00:00
jenkins-bot 777a86314e Merge "Improve code coverage for AbuseFilterParser" 2018-08-22 11:15:00 +00:00
Daimona Eaytoy 83ccb376f0 Make Ace config pass eslint
Just a tiny change to be safer.

Change-Id: Id0e188ab6e5d0e14ef5f0fdcce74be5ec7f24470
2018-08-22 11:21:13 +02:00
Daimona Eaytoy e526295123 Add a placeholder for the no-js changetags input field
There is already one for the js field, but we can't reuse it since "one
by one" doesn't make any sense here.

Change-Id: Iaf01e19f4006b3d578bb2201cf9108fe46d56085
2018-08-22 11:02:51 +02:00
Daimona Eaytoy 2fc44e9d18 Focus the Ace editor after having added text from dropdown
The classic editor (textarea) already takes focus, while for Ace we need
to explicitly call its focus() method.

Bug: T202480
Change-Id: I8dcf54e46e730b2879b09b4e21198b2ef1131706
2018-08-22 10:57:15 +02:00
Daimona Eaytoy cd30d5146f Add deprecated variables to PHPUnit tests
Check a bunch of them, they should be computed and be identical to the
ones with new syntax.

Bug: T173889
Depends-On: I5c370b54e6516889624088e27928ad3a1f48a821
Change-Id: I276913a98e06b5f2ff1c5f5f3ba5bcc7b1e8c997
2018-08-22 08:38:31 +00:00
jenkins-bot 47acf13b4d Merge "Show an extract of suppression log for Special:AbuseLog" 2018-08-22 02:38:53 +00:00
Translation updater bot 35a621470b Localisation updates from https://translatewiki.net.
Change-Id: Ica826b21c2e558ed486c5ed20b0035d43b1676ff
2018-08-21 23:37:36 +02:00
Daimona Eaytoy 66774b8d7a Show an extract of suppression log for Special:AbuseLog
Like we do in core for similar special pages. This is really helpful
when (un)hiding an entry.

Bug: T200645
Change-Id: I16450a2573e8987e31a83ec34f3dbb16fac94f81
2018-08-21 19:19:10 +02:00
jenkins-bot ae45c61870 Merge "Raise tolerance for time-related unit tests to 10 seconds" 2018-08-21 17:16:44 +00:00
Matěj Suchánek 10ad58a6f3 Migrate AbuseFilter suppress log
Also make entries in Special:Log/suppress filterable.

Change-Id: Ic23e724997e4748c8d0da8138aa73d31b17b7064
2018-08-21 16:05:54 +00:00
Daimona Eaytoy c962203ad2 Raise tolerance for time-related unit tests to 10 seconds
This helps avoiding failures with tests depending on execution time.

Bug: T202073
Change-Id: I4da859cfb3e49314ca20329e2ad4a3a7c4fae897
2018-08-21 17:18:24 +02:00
Daimona Eaytoy 6bc630cfef Add aliases for "_text" and "article_" variables
Variables regarding title (full list in task description) are quite
deceiving, since they use "text" instead of "title". As proposed in the
task, this is the first patch to add aliases for those variables and
slightly deprecate the old ones. In the future we may be able to replace
every occurrence (either with a search function or directly on the
database), but even a coexistence would be enough to avoid
confusion. A wfDebug log is generated whenever a deprecated variable is
parsed. The "article_" prefix is also changed to "title_", in the same
way as above.
Also, added a hook which other extension may use to specify their
deprecated variables, which will be handled the same as core ones.

Bug: T173889
Change-Id: I5c370b54e6516889624088e27928ad3a1f48a821
2018-08-21 16:59:56 +02:00
jenkins-bot a6c5aa8927 Merge "Avoid variable reuse to pass taint-check" 2018-08-20 23:19:33 +00:00
Translation updater bot 25151a949c Localisation updates from https://translatewiki.net.
Change-Id: Iec3d91a788e0d2249dff968f7cc1f1dbc32a22a4
2018-08-20 22:20:35 +02:00
Umherirrender 2b615cfa29 Avoid variable reuse to pass taint-check
Also set param-taint for value of WebRequest::getText

Bug: T197002
Change-Id: I9e52d24f88789c99c726e32df20840707d1b47ae
2018-08-20 19:54:20 +02:00
Daimona Eaytoy 4f3b020f5d Improve code coverage for AbuseFilterParser
Add some tests and improve others to raise coverage percentage. This
should lead to almost 100% for the AbuseFilterParser class. Aside from
this, a couple of changes:
* Remove an unused function
* Let equals_to_any return a genuine result with empty strings
* Remove an if which will never be true in skipOverBraces, since the
function is called after checking the same conditions.

Bug: T201193
Change-Id: I7020b2ed996236c38c5784d161ad98ec44163406
2018-08-20 14:38:40 +02:00
jenkins-bot 50a295a6e7 Merge "Include CheckUser in phan config" 2018-08-20 01:52:54 +00:00
Translation updater bot 50ff368194 Localisation updates from https://translatewiki.net.
Change-Id: Id215438322326b4954ccd3413b583c3dd472b102
2018-08-19 22:38:41 +02:00
jenkins-bot 55f0cd2580 Merge "Change priority order for messages in hidden abuselog entries" 2018-08-19 19:21:41 +00:00
Daimona Eaytoy b8645753ca Remove deprecated method in AbuseFilterVariableHolder
It was soft-deprecated in 2013 and nothing is using it in MW code.

Change-Id: I1300bb18c518b61a2dbce9ad43beeb69c1b615e5
2018-08-19 19:02:45 +02:00
Daimona Eaytoy 9d21c7d03d Change priority order for messages in hidden abuselog entries
Check if the entry is deleted first, since it's the strongest deletion
here (oversight level). Bonus: don't use implicit conversion when
checking the return value of SpecialAbuseLog::isHidden.

Bug: T200644
Change-Id: Ie5c4575ad29fe3dcb85a26cc74f1c59207df2852
2018-08-19 18:22:07 +02:00
Translation updater bot 01df24418a Localisation updates from https://translatewiki.net.
Change-Id: Ia99e4123423cad2fbb0e8b15e8f20d6879cc89f5
2018-08-17 23:36:22 +02:00
Umherirrender c954b412c6 Include CheckUser in phan config
Depends-On: I51421184485c3117bbab9ce3dd42f2dbb6c6180c
Change-Id: Ida17580b301ff4a6b0d3d0020c48f65eb1e21026
2018-08-17 17:38:01 +02:00
Translation updater bot a79dbf5488 Localisation updates from https://translatewiki.net.
Change-Id: Ifab4d658d65b2abee003fc0e8d31fb61538e77be
2018-08-16 22:28:09 +02:00
Daimona Eaytoy bb476e2c45 Fix wrong error message for PHPUnit
We're currently emitting the same error twice, but in one of those cases
it's completely wrong. Damned copy&pasting!

Bug: T202073
Change-Id: I7687826a85f3ef0abaf15d7cd973afc4e55758b2
2018-08-16 17:11:41 +00:00
Translation updater bot 0c496d0118 Localisation updates from https://translatewiki.net.
Change-Id: Id6f137359ab5718087a95d01d65e22c484d5b511
2018-08-15 22:18:15 +02:00
Translation updater bot 2148d06632 Localisation updates from https://translatewiki.net.
Change-Id: Iafe001dca01005c6ab5337738a8e9463773bd2be
2018-08-14 22:34:47 +02:00
Brian Wolff 429b7c6705 Update phan-taint-check to 1.3.0. Hopefully should pass now
Change-Id: If8f78dd08152b68e534cdb72704ccbd81afbd822
Bug: T197002
2018-08-13 23:57:29 +00:00
Translation updater bot 825e5939b4 Localisation updates from https://translatewiki.net.
Change-Id: Ib987d0e1eb5616ce01e57ac8418fae91f373d6a2
2018-08-13 22:14:00 +02:00
Translation updater bot 20f4fb9a8c Localisation updates from https://translatewiki.net.
Change-Id: If8f78dd08152b68e534cdb72704ccbd81afbd898
2018-08-12 22:17:08 +02:00
Translation updater bot 2716696221 Localisation updates from https://translatewiki.net.
Change-Id: I455d6d615d20dc3428eaeb16e95cee280c1b9207
2018-08-11 22:17:14 +02:00
Translation updater bot a3c9c8f5f1 Localisation updates from https://translatewiki.net.
Change-Id: Icff2ffe9422a34ade18bb55b2be3b24cdcc7eb4d
2018-08-10 22:17:58 +02:00
jenkins-bot e129650125 Merge "Add PHPUnit tests for various generic functions" 2018-08-09 20:44:09 +00:00
Translation updater bot 063ea65758 Localisation updates from https://translatewiki.net.
Change-Id: I9810aaddda0ec958d98ab3d2e0dadf9d693d864b
2018-08-09 22:23:18 +02:00
Daimona Eaytoy 0026a68a8a Add PHPUnit tests for various generic functions
Adding tests for generic functions in AbuseFilter class, ranging from
simple utility function to variable computation.

Bug: T42478
Change-Id: I903fb7ffbc436b27462e3e4611ab65ecb8a543ba
2018-08-09 19:20:46 +00:00
Translation updater bot cd0e28be12 Localisation updates from https://translatewiki.net.
Change-Id: I4f641e18316762a57b36d3b61adc67723bfbff31
2018-08-08 22:13:03 +02:00
Translation updater bot b3555fab56 Localisation updates from https://translatewiki.net.
Change-Id: Iba209b3478c2da06ed9891ad74f416afc3a5d993
2018-08-07 22:31:06 +02:00
Translation updater bot 94c33ddbc3 Localisation updates from https://translatewiki.net.
Change-Id: Ideb82bed16e01eec2af4ee1fcb2e3890fec763e0
2018-08-06 22:20:49 +02:00
Translation updater bot 976a0a2fa5 Localisation updates from https://translatewiki.net.
Change-Id: Id9db4f217b5752e883775ba2a1391423220a13b4
2018-08-02 22:19:08 +02:00
jenkins-bot 4b185b3749 Merge "Add phpunit tests for noparams and notenoughargs exceptions" 2018-08-02 08:41:46 +00:00
jenkins-bot f587230fea Merge "Use noparams exception and correctly count function parameters" 2018-08-02 08:16:12 +00:00
jenkins-bot 75729b6195 Merge "Add other phpunit test for AFPUserVisibleException" 2018-08-02 08:07:56 +00:00
Translation updater bot 7d9e345953 Localisation updates from https://translatewiki.net.
Change-Id: Iaec408a00dd6afa0c54560ab994353f5e6303a25
2018-08-01 22:10:07 +02:00
jenkins-bot c876f98d67 Merge "Remove HitCounters from AbuseFilter and use hooks instead" 2018-07-31 20:57:42 +00:00
Translation updater bot 2c85c70e91 Localisation updates from https://translatewiki.net.
Change-Id: Ia4f4fb6b89244a09bc3037ff67f16a104383e1e1
2018-07-31 22:32:39 +02:00
Huji Lee df21fb2b20 Remove HitCounters from AbuseFilter and use hooks instead
Goes with Ief573fb412d332bd4ad6ad8de3052dd85d534b82

Bug: T159069
Change-Id: I38cd7cbf3e595890b53624a477010bd49c9b8552
2018-07-31 03:56:20 +00:00