jenkins-bot [Wed, 22 Nov 2017 20:50:19 +0000 (20:50 +0000)]
Merge "Preferences: Don't show the watchlist token; just link to ResetTokens"
jenkins-bot [Wed, 22 Nov 2017 20:45:47 +0000 (20:45 +0000)]
Merge "Deprecate OutputPage::parserOptions() as a setter"
Brad Jorsch [Wed, 22 Nov 2017 20:12:43 +0000 (15:12 -0500)]
Deprecate OutputPage::parserOptions() as a setter
The only known callers were using it to call ->setEditSections( false ),
which has been the default since MediaWiki 1.19.
Change-Id: Ib021d7e80e44782ba6ebe66cf8366bae9147ce00
Depends-On: I7de0102e6cac6b5aa0147b825fa2cb6ebd37b434
Depends-On: I7b0604773a8692c7cb37fffa3fcb8aaac83d81ba
jenkins-bot [Wed, 22 Nov 2017 19:33:08 +0000 (19:33 +0000)]
Merge "InstallDocFormatter: Add missing @covers scope"
Timo Tijhof [Wed, 22 Nov 2017 19:01:54 +0000 (11:01 -0800)]
InstallDocFormatter: Add missing @covers scope
The format() method is a one-line wrapper around execute(), which
is the real method being tested here. Given the class doesn't
contain any other methods, increase the scope to the whole class.
Change-Id: I8b2c7736c6708c0915a9ce15fef0dc85c63621ea
Timo Tijhof [Wed, 22 Nov 2017 18:48:27 +0000 (10:48 -0800)]
build: Add 'npm run qunit' command
The running of 'grunt qunit' is unconvenient due to it only working
if the user has grunt-cli installed globally, which should not be
needed because it is already installed in the local directory.
It could be worked around by instructing users to use
`./node_modules/.bin/grunt qunit`, but it would be much simpler
to instruct them to use `npm run qunit` instead.
Unlike 'composer', 'npm' does not come by default with a command
like 'composer exec' that one could pass a command directly
without needing to register it. This is fixed in more recent
versions through 'npx -c', but that's a bit too new to require
in the manual, so adding it as a run-script instead.
Change-Id: I2812b13dbed50612b1626a617ba65f92e212f01a
jenkins-bot [Wed, 22 Nov 2017 18:12:12 +0000 (18:12 +0000)]
Merge "RCFilters: Respect subpage in RCLinked"
jenkins-bot [Wed, 22 Nov 2017 18:05:44 +0000 (18:05 +0000)]
Merge "objectcache: add WANObjectCache::STALE_TTL_NONE constant"
Moriel Schottlender [Tue, 21 Nov 2017 23:59:44 +0000 (15:59 -0800)]
RCFilters: Respect subpage in RCLinked
Make sure that when we redirect a URL when there's a saved query,
we retain the information about a subpage.
Then, normalize the URL to always use &target=xxx so that the
system knows to correct the value if the user uses the form that
is, for the moment, outside the regular RCFilters interface.
Bug: T181100
Change-Id: I75cfb2b56a4da6357e6117b3f34f3178bfb2c90c
jenkins-bot [Wed, 22 Nov 2017 17:22:36 +0000 (17:22 +0000)]
Merge "RCFilters: Update URL even when we skip fetching"
jenkins-bot [Wed, 22 Nov 2017 16:35:56 +0000 (16:35 +0000)]
Merge "Add missing message dependency to oojs-ui-widgets"
Thiemo Mättig [Wed, 22 Nov 2017 14:59:59 +0000 (15:59 +0100)]
Add missing message dependency to oojs-ui-widgets
The depdency is just missing. The message is used in the code, but
not loaded, resulting in tooltips that show "<ooui-item-remove>".
This is currently an actual issue when playing around with the
AdvancedSearch prototype announced at
https://www.mediawiki.org/wiki/Help:Extension:AdvancedSearch
Change-Id: Ia89f2dd6c89b03fe63d289732ac9221eddf5db99
WMDE-Fisch [Wed, 22 Nov 2017 13:28:52 +0000 (14:28 +0100)]
Add tests for DiffEngine tooltips
Change-Id: I8e6560efc35a6bd875f96edb9b59c9961b81a956
WMDE-Fisch [Wed, 22 Nov 2017 10:51:43 +0000 (11:51 +0100)]
Fix regex on diff tooltips
Follow-up to I71600acc19319f7b5781b42dc88cd5e8c65c29a0
Change-Id: If307f6a02291005c12f28129a67aa00e423e5e9c
jenkins-bot [Wed, 22 Nov 2017 08:12:59 +0000 (08:12 +0000)]
Merge "Maintenance: add fatalError() method"
Fomafix [Wed, 22 Nov 2017 05:08:33 +0000 (06:08 +0100)]
Merge mediawiki.special.userrights.styles into mediawiki.special
Follows-up
5f18aae76eecf5.
This reduces the number of modules.
Bug: T180914
Change-Id: I4143e876495bad6530afe290ba686d7f26a43c58
Bryan Davis [Mon, 20 Nov 2017 00:36:54 +0000 (17:36 -0700)]
Maintenance: add fatalError() method
Deprecate the second argument to Maintenance::error() in favor of a new
Maintenance::fatalError() method. This is intended to make it easier to
review flow control in maintenance scripts.
Change-Id: I75699008638f7e99b11210c7bb9e2e131fca7c9e
Moriel Schottlender [Wed, 22 Nov 2017 00:37:32 +0000 (16:37 -0800)]
RCFilters: Update URL even when we skip fetching
In some cases, when the selected value of the filters have not
changed, we don't reload the results. However, we should still
update the URL values.
Change-Id: Iff81b4ca1b78848813b2eb8d55f0f5f5e614b424
Subramanya Sastry [Tue, 21 Nov 2017 22:07:03 +0000 (17:07 -0500)]
Sync up with Parsoid parserTests.txt
This now aligns with Parsoid commit
0723e5c47845ff4361b9635b591e7d386c975fdf
Change-Id: Ic78ee28a5cdeb9d32147332bf6c06bbe6ab19acd
Aaron Schulz [Tue, 21 Nov 2017 22:09:08 +0000 (14:09 -0800)]
objectcache: add WANObjectCache::STALE_TTL_NONE constant
Also improved the documentation around "staleTTL".
Change-Id: Iae3377cbf58d8aa2c70f75586f183089a8bec92f
Max Semenik [Mon, 13 Nov 2017 23:53:24 +0000 (15:53 -0800)]
{{anchorencode:}} output should be always usable in links
Bug: T179544
Change-Id: If8098e3e839e8ba1cf27b6053017e818c556a733
jenkins-bot [Tue, 21 Nov 2017 21:18:46 +0000 (21:18 +0000)]
Merge "Sanitizer::safeEncodeAttribute(): also encode ]"
jenkins-bot [Tue, 21 Nov 2017 21:18:40 +0000 (21:18 +0000)]
Merge "Add unit tests for Html helper methods and change messageBox visibility"
jdlrobson [Wed, 15 Nov 2017 19:11:02 +0000 (11:11 -0800)]
Add unit tests for Html helper methods and change messageBox visibility
* Messagebox is now private to Html class to discourage unconventional
usages
* Tests are added for all three helper methods added in
4e7021a2318
Bug: T166915
Change-Id: I1c3e4131b2439c0f4fb94ad4e616a909b52d6b78
Translation updater bot [Tue, 21 Nov 2017 20:57:45 +0000 (21:57 +0100)]
Localisation updates from https://translatewiki.net.
Change-Id: I290aac624dced5271a12cb1e582dc5649aea3eda
jenkins-bot [Tue, 21 Nov 2017 19:13:34 +0000 (19:13 +0000)]
Merge "Reduce lag waiting time in CategoryMembershipUpdateJob critical section"
jenkins-bot [Tue, 21 Nov 2017 19:13:30 +0000 (19:13 +0000)]
Merge "Make CategoryMembershipChangeJob query more readable"
jenkins-bot [Tue, 21 Nov 2017 15:22:23 +0000 (15:22 +0000)]
Merge "Display MAX_CHARS / MAX_BYTES in ApiSandbox"
jenkins-bot [Tue, 21 Nov 2017 15:21:26 +0000 (15:21 +0000)]
Merge "Add string length limits"
WMDE-Fisch [Tue, 7 Nov 2017 11:44:53 +0000 (12:44 +0100)]
Add localised tooltips for moved paragraph indicators
Bug: T166882
Change-Id: I71600acc19319f7b5781b42dc88cd5e8c65c29a0
Kunal Mehta [Sat, 4 Nov 2017 20:15:26 +0000 (13:15 -0700)]
DifferenceEngine: Improve cache invalidation
Invalidate the diff cache if the engine producing the diff changes, or
if a configuration setting that controls the diff output changes. This
is probably what most users expect, that changing the configuration will
result in a change for diffs that may have already been viewed.
For wikidiff2 specifically, a change in version or
$wgWikiDiff2MovedParagraphDetectionCutoff will invalidate the cache.
Refactor engine detection and sanity-checking into a private getEngine()
function.
As part of this getDiffBodyCacheKey() was deprecated, and subclasses
should implement getDiffBodyCacheKeyParams() instead. Drop the
deprecated and unused MW_DIFF_VERSION constant while we're at it, and
bump DIFF_VERSION since we're already changing the cache key format.
Bug: T180043
Change-Id: I4e386ca05bd2a2fb54208d760c131eb42e3a72ab
Gergő Tisza [Mon, 13 Nov 2017 00:37:21 +0000 (00:37 +0000)]
Display MAX_CHARS / MAX_BYTES in ApiSandbox
Change-Id: I0f15afe6656765e92582e4e4cead714b31987b8a
Gergő Tisza [Sun, 12 Nov 2017 09:51:34 +0000 (09:51 +0000)]
Add string length limits
Adds two new ApiBase::getAllowedParams() keys:
PARAM_MAX_BYTES and PARAM_MAX_CHARS, to set a length
limit for a (string-like) parameter.
This makes it easy to document and enforce database
field length limits (where relying on the database
would either result in unfriendly error messages or
silent truncation, depending on DB settings) and
also exposes them in structured form so API clients
can verify the length without doing roundtrips.
Change-Id: I2e784972d7e11cad79fdef887bbcde297dbd9ce0
Aaron Schulz [Tue, 21 Nov 2017 02:09:52 +0000 (18:09 -0800)]
Reduce lag waiting time in CategoryMembershipUpdateJob critical section
Bug: T180793
Change-Id: Icfe8dd16f4194c5d4f88d7547f732acae8b1cfe2
Aaron Schulz [Tue, 21 Nov 2017 02:43:40 +0000 (18:43 -0800)]
Make CategoryMembershipChangeJob query more readable
Previously, the INNER JOIN had "rc_timestamp >= rev_timestamp" which
complicates query planning. Even with "equals" it still was techinally
ambiguous. Instead, just use EXISTS and an exact equality operator.
Bug: T180793
Change-Id: I1e9ae7c2ce0f95484e09e867550283d816d151f0
Aaron Schulz [Tue, 21 Nov 2017 00:04:38 +0000 (16:04 -0800)]
objectcache: Make MemcachedBagOStuff::makeKeyInternal always have a key class
Even if a key is too long and shortened, it should still have some key class.
Change-Id: I006b6b03ad1302e9e49362bbd051332bc6105837
James D. Forrester [Mon, 20 Nov 2017 23:34:06 +0000 (15:34 -0800)]
Preferences: Don't show the watchlist token; just link to ResetTokens
Also make both the PasswordReset and ResetTokens forms appropriately
flag their action buttons as destructive.
Bug: T180710
Change-Id: I26649900f9360e5175fa93b87dc7840a7c1d4f93
jenkins-bot [Mon, 20 Nov 2017 22:47:54 +0000 (22:47 +0000)]
Merge "Special:UserRights: Prevent FOUC on loading"
jenkins-bot [Mon, 20 Nov 2017 22:27:57 +0000 (22:27 +0000)]
Merge "Crimean Tatar Transliteration"
tjones [Mon, 31 Jul 2017 22:35:39 +0000 (18:35 -0400)]
Crimean Tatar Transliteration
This is a first pass at Latin/Cyrillic translitertion for Crimean
Tatar (crh).
Includes transliteration tables, prefix/suffix mappings, regex
mappings, and exceptions lists for words and abbreviations.
Regularize CRH language name in messages/* files.
Fix "varient" typos in qqq.json.
Add unit tests for CRH transliteration.
Bug: T23582
Change-Id: I424703f99adf837f6217872b882d1ea26bfdd068
Translation updater bot [Mon, 20 Nov 2017 20:56:22 +0000 (21:56 +0100)]
Localisation updates from https://translatewiki.net.
Change-Id: I2541976a8c5c079f0e11d84f843aff0ffd540dab
jenkins-bot [Mon, 20 Nov 2017 20:51:26 +0000 (20:51 +0000)]
Merge "objectcache: fix some makeKey/makeGlobalKey IDEA warnings"
jenkins-bot [Mon, 20 Nov 2017 20:48:47 +0000 (20:48 +0000)]
Merge "Fix some broke cache key generations"
jenkins-bot [Mon, 20 Nov 2017 20:41:16 +0000 (20:41 +0000)]
Merge "objectcache: make sure variant keys are namespaced as keys should be"
Aaron Schulz [Mon, 20 Nov 2017 07:27:34 +0000 (23:27 -0800)]
objectcache: make sure variant keys are namespaced as keys should be
Change-Id: I02597d6dbc2febace25ef6d1981f69dc704efe8d
Moriel Schottlender [Mon, 20 Nov 2017 20:07:43 +0000 (12:07 -0800)]
RCFilters: Followup I08db859c571: Fix typo in "namespaces"
Bug: T180863
Change-Id: I1b65ed5dfceb99bea98cbcadea11ddab367c6ca7
Fomafix [Mon, 20 Nov 2017 07:32:41 +0000 (08:32 +0100)]
Special:UserRights: Prevent FOUC on loading
Implement the show/hide based on the group checkbox with CSS instead of
JavaScript.
Hide the expiry input field initially with CSS instead of JavaScript.
Change-Id: I33a4b0a13dab92156aeea96529573ad49fa2d616
addshore [Mon, 20 Nov 2017 18:52:46 +0000 (18:52 +0000)]
Deprecate access to TitleValue properties
This switches from private to protected access of TitleValue
properties, and marks them as deprecated instead as per the
deprecation policy.
Partial revert of I3d8315ade6aa70bda43d90b0b32b730d8c9cbd2e
Change-Id: Ie290b6a84635ebb5865e82ae24fe4b27224e601a
jenkins-bot [Mon, 20 Nov 2017 18:20:45 +0000 (18:20 +0000)]
Merge "RCFilters: Only apply excluded label to namespace items"
jenkins-bot [Mon, 20 Nov 2017 16:26:48 +0000 (16:26 +0000)]
Merge "Tests for TitleValue::__toString"
jenkins-bot [Mon, 20 Nov 2017 16:15:38 +0000 (16:15 +0000)]
Merge "Set wgCommentTableSchemaMigrationStage for some RevisionTests"
jenkins-bot [Mon, 20 Nov 2017 11:50:13 +0000 (11:50 +0000)]
Merge "Move styles for Special:UserRights to separate style module"
addshore [Mon, 20 Nov 2017 09:39:55 +0000 (09:39 +0000)]
Tests for TitleValue::__toString
Change-Id: Ibb157b8e6e4fdc75bdb27ecaa4a552304465c4b2
addshore [Mon, 20 Nov 2017 09:33:28 +0000 (09:33 +0000)]
Set wgCommentTableSchemaMigrationStage for some RevisionTests
Bug: T180922
Change-Id: Ib88b53e36a66042b3c177cae0663040b4fb14a4c
Reedy [Sun, 19 Nov 2017 18:36:34 +0000 (18:36 +0000)]
Move styles for Special:UserRights to separate style module
Bug: T180914
Change-Id: I0e3cf5e3d69f32a30e5fe6dc0d2fd964c4f5720c
Aaron Schulz [Mon, 20 Nov 2017 05:30:45 +0000 (21:30 -0800)]
Fix some broke cache key generations
Change-Id: Ib724fe0cfa866351caee89d3451b6c4b7b6a4578
Translation updater bot [Sun, 19 Nov 2017 20:55:12 +0000 (21:55 +0100)]
Localisation updates from https://translatewiki.net.
Change-Id: I571e77344da60ffec62d52ea79231ff94562c424
Aaron Schulz [Sat, 18 Nov 2017 20:39:21 +0000 (12:39 -0800)]
objectcache: fix some makeKey/makeGlobalKey IDEA warnings
Change-Id: I5fefda9676a685167326c63c34b2b7df8be2e7cd
jenkins-bot [Sat, 18 Nov 2017 21:44:15 +0000 (21:44 +0000)]
Merge "@since tags & private class properties for TitleValue"
jenkins-bot [Sat, 18 Nov 2017 21:42:34 +0000 (21:42 +0000)]
Merge "Add __toString method to LinkTarget interface"
Translation updater bot [Sat, 18 Nov 2017 20:53:19 +0000 (21:53 +0100)]
Localisation updates from https://translatewiki.net.
Change-Id: If8cc0f3abd7cde509909de175a32c9492f9388a6
addshore [Sat, 18 Nov 2017 17:34:41 +0000 (17:34 +0000)]
@since tags & private class properties for TitleValue
Change-Id: I3d8315ade6aa70bda43d90b0b32b730d8c9cbd2e
addshore [Sat, 18 Nov 2017 17:34:10 +0000 (17:34 +0000)]
Add __toString method to LinkTarget interface
Change-Id: I4f12e3175a308f88e240db01cd4e91e78b283057
daniel [Thu, 16 Nov 2017 19:44:44 +0000 (20:44 +0100)]
Introduce the UserIdentity interface.
This provides a narrow view on a user identity, providing access to
ID and name.
This has been extracted from I140f43a6fb443b for re-use with Actors,
on Anomie's request.
Change-Id: Ief00db5ce382537c5bf992159eae6baf096ae4be
addshore [Sat, 18 Nov 2017 18:22:24 +0000 (18:22 +0000)]
[MCR] tests for Revision::getQueryInfo
Bug: T180210
Change-Id: I5bdb5eed853e22bacd6b4c2546343e9d0f2d8c89
addshore [Sat, 18 Nov 2017 18:08:45 +0000 (18:08 +0000)]
[MCR] tests for Revision::getArchiveQueryInfo
Bug: T180210
Change-Id: Icb016be8d69dfdfa83f44bbc4fb259b5beb30678
addshore [Sat, 18 Nov 2017 17:49:22 +0000 (17:49 +0000)]
[MCR] Readd various field & cond method tests for Revision
This is a partial revert of:
Idcfd15568489d9f03a7ba4460e96610d33bc4089
which removed these tests.
Bug: T180210
Change-Id: Ib0617ee0a7bd4391ed25415b44a8ed077a985eaa
jenkins-bot [Sat, 18 Nov 2017 13:00:39 +0000 (13:00 +0000)]
Merge "Remove box-shadow from preference panels for ooui-apex"
jenkins-bot [Sat, 18 Nov 2017 03:54:22 +0000 (03:54 +0000)]
Merge "Fix RemexCompatMunger infinite recursion"
jenkins-bot [Sat, 18 Nov 2017 02:08:27 +0000 (02:08 +0000)]
Merge "SwiftFileBackend::resolveContainerPath() check the proper length"
Moriel Schottlender [Sat, 18 Nov 2017 00:57:40 +0000 (16:57 -0800)]
RCFilters: Only apply excluded label to namespace items
Bug: T180863
Change-Id: I08db859c571b82ed4e4793d97b99100875ff1e23
Aaron Schulz [Fri, 17 Nov 2017 23:27:49 +0000 (15:27 -0800)]
SwiftFileBackend::resolveContainerPath() check the proper length
The length sanity check should use this instead of urlencode()
see it is rawurlencode() that is actually used.
Change-Id: I5632e30c14c8ab27c8324c3e31311ca8bff7c162
Ed Sanders [Fri, 17 Nov 2017 22:02:58 +0000 (22:02 +0000)]
Follow-Up Iae63b6994: Add missing editfont dependency
Change-Id: I606a81576baf312891f3bb12f575892b00bca823
Ed Sanders [Fri, 17 Nov 2017 21:23:16 +0000 (21:23 +0000)]
Remove box-shadow from preference panels for ooui-apex
Change-Id: I7d42c75053b29cb634b18bae9e06e6b28ae1e967
Translation updater bot [Fri, 17 Nov 2017 21:03:49 +0000 (22:03 +0100)]
Localisation updates from https://translatewiki.net.
Change-Id: Iabf53d3c29d23ea48d35c2a45d88b2b3d937d5c6
addshore [Wed, 15 Nov 2017 10:37:29 +0000 (10:37 +0000)]
Revision::userCanBitfield test fallback to $wgUser
Bug: T180210
Change-Id: Ibda256d6fdd8762e0e90748d71034979e2de106f
Ed Sanders [Fri, 17 Nov 2017 17:15:44 +0000 (17:15 +0000)]
Work around Firefox 57 attribute parsing regression
Bug: T180138
Change-Id: I554d5adf88c448db64e6f135e19ee76a4ec28493
jenkins-bot [Fri, 17 Nov 2017 15:53:52 +0000 (15:53 +0000)]
Merge "Expose string->bool conversion as function"
Tim Starling [Fri, 17 Nov 2017 11:15:59 +0000 (22:15 +1100)]
Fix RemexCompatMunger infinite recursion
When TreeBuilder requests reparenting of all child nodes of a given
element, we do this by removing the existing child nodes, and then
inserting the proposed new parent under the old parent. However, when a
p-wrap diversion is in place, the insertion of the new parent is
diverted into the p-wrap, and the p-wrap then becomes a child of the new
parent, causing a reference loop, and ultimately infinite recursion in
Serializer.
Instead, divert the entire reparent request to the p-wrap, so that the
new parent is a child of the p-wrap. This makes sense since the new
parent is always a formatting element. The only caller of
reparentChildren(), apart from proxies, is AAA step 17, which reparents
children under the formatting element cloned from the AFE list.
Left in some debug code for next time.
Bug: T178632
Change-Id: Id77d21d99748e94c064ef24c43ee0033de627b8e
jenkins-bot [Thu, 16 Nov 2017 23:05:43 +0000 (23:05 +0000)]
Merge "Preferences: Improve visual appearance by “unboxing” sections"
jenkins-bot [Thu, 16 Nov 2017 23:03:20 +0000 (23:03 +0000)]
Merge "Cleanup, removed space"
Volker E [Wed, 15 Nov 2017 00:47:52 +0000 (16:47 -0800)]
Preferences: Improve visual appearance by “unboxing” sections
Instead let's work with whitespace, which also saves virtual space and
makes the appearance not as jarring.
Bug: T180538
Depends-on: I39088107e6ab07399f9826dd925df9e1b8dda006
Change-Id: I24d21eb3c0d188004dacbce8a9bc1ac3ad7e2a8f
Translation updater bot [Thu, 16 Nov 2017 21:34:02 +0000 (22:34 +0100)]
Localisation updates from https://translatewiki.net.
Change-Id: I20f8422f709fe37f212b65f2e995db30776009eb
jenkins-bot [Thu, 16 Nov 2017 20:34:31 +0000 (20:34 +0000)]
Merge "Use Remex in Sanitizer::stripAllTags()"
Max Semenik [Mon, 13 Nov 2017 22:57:23 +0000 (14:57 -0800)]
Sanitizer::safeEncodeAttribute(): also encode ]
...so that it wouldn't break wikilinks, too.
Bug: T29694
Bug: T51672
Bug: T72875
Change-Id: I189bdefbc9034cf8d221a89d7158195de1c0fa6c
jenkins-bot [Thu, 16 Nov 2017 17:31:44 +0000 (17:31 +0000)]
Merge "OOUIHTMLForm: Prevent duplicate FieldsetLayout wrapping"
jenkins-bot [Thu, 16 Nov 2017 17:22:39 +0000 (17:22 +0000)]
Merge "HTMLMultiSelectField: Fix OOUI\CheckboxMultiselectInputWidget to be infusable again"
jenkins-bot [Thu, 16 Nov 2017 17:05:14 +0000 (17:05 +0000)]
Merge "HTMLRadioField: Do not automatically infuse our RadioSelectInputWidgets"
Bartosz Dziewoński [Thu, 16 Nov 2017 16:14:29 +0000 (17:14 +0100)]
HTMLRadioField: Do not automatically infuse our RadioSelectInputWidgets
This is really a workaround for an issue in a completely different place:
JS RadioSelectInputWidget internally uses `<input type="hidden">`
rather than real radio buttons, which does not work correctly with the
code in mediawiki.special.preferences.confirmClose.js. Ideally we would
change RadioSelectInputWidget to not do such weird things.
However, I think this is actually a good thing to do in general.
From the user's perspective, PHP RadioSelectInputWidget and JS
RadioSelectInputWidget look and behave the same, so there's no reason
to infuse and rebuild them.
This behavior was implemented in
f50cee1375201a5d3fd76c0c262cfc7e66bd5d42
in which unfortunately I did not document the reason for it. For other
fields it makes obvious sense (the JS widgets have improvements like
autocompletion, or at least look "pretty"), but I have no idea why
I did it for this one.
Bug: T180643
Change-Id: I53e50f8cda39466b2396b374e642c154487888bb
Bartosz Dziewoński [Thu, 16 Nov 2017 10:24:47 +0000 (11:24 +0100)]
OOUIHTMLForm: Prevent duplicate FieldsetLayout wrapping
The code in formatSection() assumed it was only called for the
toplevel section (the whole form), while it's actually called
for every subsection too. I think it was written before we added
support for subsections in OOUIHTMLForm.
Move code for toplevel section wrapping to wrapForm().
As a bonus, this also fixes display of custom headers and error
or warning messages for forms with subsections.
Bug: T180535
Change-Id: I6a88184d302a951be78387490404137acde3fa1a
WMDE-Fisch [Thu, 16 Nov 2017 12:27:11 +0000 (13:27 +0100)]
Fixed hover circle for timeless skin
Bug: T180663
Change-Id: I5112636bcfae6f41e86ccc29524ccf5c9e2a6004
Bartosz Dziewoński [Thu, 16 Nov 2017 10:51:19 +0000 (11:51 +0100)]
HTMLMultiSelectField: Fix OOUI\CheckboxMultiselectInputWidget to be infusable again
Regression from
5a113417e5af9d0d0dbed63429649a9780784d45.
Bug: T180677
Change-Id: Id1b0ebe9d9a56a76d73deb2b4d17213ae5e45a04
jenkins-bot [Thu, 16 Nov 2017 10:04:35 +0000 (10:04 +0000)]
Merge "Hide empty OOUI FieldsetLayout headers"
Huji Lee [Sun, 12 Nov 2017 01:44:00 +0000 (20:44 -0500)]
UserGroupsChanged hook should specify the performer of the change
Otherwise, there will be a unit-testing error when Echo is enabled
Bug: T180292
Change-Id: Ibc185c82ad2a03e06e5727a633e6ab6bccce3345
jenkins-bot [Thu, 16 Nov 2017 01:33:21 +0000 (01:33 +0000)]
Merge "Move Sanitizer.php to includes/parser/"
jenkins-bot [Thu, 16 Nov 2017 01:32:46 +0000 (01:32 +0000)]
Merge "SanitizerTest: Add tests for stripAllTags"
Roan Kattouw [Tue, 14 Nov 2017 22:22:31 +0000 (14:22 -0800)]
Use Remex in Sanitizer::stripAllTags()
Using a real HTML tokenizer fixes bugs when < or > appear in attribute
values. The old implementation used delimiterReplace(), which didn't
handle this case:
> print Sanitizer::stripAllTags( '<p data-foo="a<b>c">Hello</p>' );
c">Hello
We also can't use PHP's built-in strip_tags() because it doesn't handle
<?php and <? correctly:
> print strip_tags('1<span class="<?php">2</span>3');
1
> print strip_tags('1<span class="<?">2</span>3');
1
Bug: T179978
Change-Id: I53b98e6c877c00c03ff110914168b398559c9c3e
Roan Kattouw [Wed, 15 Nov 2017 20:44:48 +0000 (12:44 -0800)]
Move Sanitizer.php to includes/parser/
Change-Id: Id08d91c747ec77d715459b89b03eee247ccd4e1b
Roan Kattouw [Tue, 14 Nov 2017 22:16:14 +0000 (14:16 -0800)]
SanitizerTest: Add tests for stripAllTags
Bug: T179978
Change-Id: I9776cfd51b1b3ec772d4216168fbe466f48f5892
Translation updater bot [Wed, 15 Nov 2017 20:54:46 +0000 (21:54 +0100)]
Localisation updates from https://translatewiki.net.
Change-Id: I746f9a0b5a9ffcfbe198b4d222e476b169bad2dc