aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Don't accept empty option group names for dropdown elementsThiemo Kreuz2021-02-192-5/+10
| | | | | | | | | | | | | | | | | | | | | | | | | When this code parses a string that looks like this: * ** A ** B It creates an option group with 2 options, but the name of the group is an empty string. This makes the OOUI DropdownInputWidget fail later. Since the empty group name is useless anyway, drop it. This makes the code behave as if the parsed string looked like this: * A * B Live example for the issue: https://el.wikipedia.org/wiki/MediaWiki:Protect-dropdown Bug: T275125 Change-Id: I780c1be27740b0ed3b35aa569b5a528112d7238f
* Localisation updates from https://translatewiki.net.Translation updater bot2021-02-1917-85/+126
| | | | Change-Id: I0ff53f00037f72dfaa3516c10e1e587f327d87c2
* Merge "resourceloader: Mark ResourceLoaderContext::encodeJson() as stable"jenkins-bot2021-02-191-1/+1
|\
| * resourceloader: Mark ResourceLoaderContext::encodeJson() as stableBartosz Dziewoński2021-02-191-1/+1
| | | | | | | | | | | | | | | | | | | | | | Un-mark as @internal. It's a useful function for extensions that have custom modules. Example uses: * Id8de16ab683948eae096b43462118ea837f53038 * Ia16dad8b6c450f4cb095a67af1c1c2b487e8536b Change-Id: I03f0e7190fab62cd7e5c6a504a564142d0a9800d
* | objectcache: make SqlBagOStuff use getSerialized()Aaron Schulz2021-02-191-3/+3
| | | | | | | | | | | | This makes use of any values prepared in setNewPreparedValues() Change-Id: I2c9ec7d1f9183e060cda81de009e4bdfc889faf6
* | Merge "field descriptors in HTMLForm must have keys"jenkins-bot2021-02-191-10/+10
|\ \
| * | field descriptors in HTMLForm must have keysjdlrobson2021-02-191-10/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Use the name of the input for the value where possible. Additional change: Use `default` rather than `invert` for checkbox default state. Bug: T275018 Bug: T274980 Change-Id: Ie08b192ad4967ac6c6d80b9c76224acd183fc1e8
* | | Merge "DefaultSettings.php: Update $wgLogos doc"jenkins-bot2021-02-191-10/+43
|\ \ \
| * | | DefaultSettings.php: Update $wgLogos docAmmarpad2021-02-171-10/+43
| | | | | | | | | | | | | | | | | | | | Bug: T274900 Change-Id: Ib8c21ba4c87aff0d9e4c9d7cdf783bfa9457912d
* | | | Merge "HTMLMultiSelectField: Reject nested arrays early"jenkins-bot2021-02-191-0/+6
|\ \ \ \
| * | | | HTMLMultiSelectField: Reject nested arrays earlyBartosz Dziewoński2021-02-191-0/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Users can pass multidimensional arrays in query parameters to PHP (e.g. ?foo[a][b]=bar). While filterDataForSubmit() ensured that anyone using HTMLMultiSelectField in their form did not see them, internal code here did not handle them correctly when validating the values and generating the inputs, resulting in warnings deep in other code. Use is_scalar instead of is_string in case default values somewhere are integers or other non-string types. Bug: T274955 Change-Id: I072a722ed025d687bfe755261a9896457f68f2ef
* | | | | Merge "SpecialContributions: Remove unneeded code"jenkins-bot2021-02-191-3/+0
|\| | | | | |_|/ / |/| | |
| * | | SpecialContributions: Remove unneeded codeBartosz Dziewoński2021-02-191-3/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is not necessary to pass the value of the query parameter to the form. HTMLForm receives the entire context, and pulls out the values from it by itself. None of the other fields in this form do this. Bug: T274955 Change-Id: Ibc1345f2f4355ed9889276027ea7bc9163f870fc
* | | | Merge "uuid: Improve architecture rationale docs for GlobalIdGenerator"jenkins-bot2021-02-181-20/+20
|\ \ \ \ | |/ / / |/| | |
| * | | uuid: Improve architecture rationale docs for GlobalIdGeneratorTimo Tijhof2021-02-181-20/+20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Fix typos and grammer. - Fix indentation. - Remove "The drift is immediate" sentence. It doesn't read well, and I don't remember what I meant with it. Change-Id: I9c3dc69db437644f677cbb3797f7db40c5ec5eeb
* | | | Merge "build: Upgrade eslint-config-wikimedia from 0.17.0 to 0.18.1"jenkins-bot2021-02-1822-13824/+14099
|\ \ \ \
| * | | | build: Upgrade eslint-config-wikimedia from 0.17.0 to 0.18.1James D. Forrester2021-02-1822-13824/+14099
| | | | | | | | | | | | | | | | | | | | Change-Id: I5e3687be2b197134578126e1b890ee37dbc1bc1b
* | | | | Merge "Align mw-ui-button with latest design recommendation"jenkins-bot2021-02-181-1/+2
|\ \ \ \ \ | |_|/ / / |/| | | |
| * | | | Align mw-ui-button with latest design recommendationjdlrobson2021-02-181-1/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Update padding and add min-height Bug: T268241 Change-Id: I1f24155c9bd96b37f7a3edc373dbf85f0c27de75
* | | | | docs: Fix 'dependant' typosTimo Tijhof2021-02-189-21/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The intended word in all these cases was the adjective "dependent". Whilst the "dependant" does exist, it is a noun and generally refers to a person. The word is rare used in general, but especially so in a technology context. Change-Id: Ic7e2d2ea6a566f4139ff1fdb77f38b0e962ccd9c
* | | | | Sync up with Parsoid parserTests.txtArlo Breault2021-02-181-17/+12
| |/ / / |/| | | | | | | | | | | | | | | | | | | This now aligns with Parsoid commit 241a08fb80cd5b4b16146eb99054b25c0261998c Change-Id: I8d176b76891e10de096247f6ad3ed52ec6f5735e
* | | | Merge "Don't apply French spacing in raw text elements"jenkins-bot2021-02-186-15/+54
|\ \ \ \
| * | | | Don't apply French spacing in raw text elementsArlo Breault2021-02-166-15/+54
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This also means we don't need to take special care for French spacing in attributes, since it's no longer applied there. Adds a test that captures this change. Note that the test "Nowiki and french spacing" wonders whether this escaping should be applied to nowiki content. Bug: T255007 Change-Id: Ic8965e81882d7cf024bdced437f684064a30ac86
* | | | | Merge "API: Mention PARAM_SENSITIVE in markParamsSensitive() docs"jenkins-bot2021-02-181-1/+5
|\ \ \ \ \
| * | | | | API: Mention PARAM_SENSITIVE in markParamsSensitive() docsTimo Tijhof2021-02-181-1/+5
| | | | | | | | | | | | | | | | | | | | | | | | Change-Id: Ib4f86a653645d00c9e6e574581567db68550689c
* | | | | | Merge "Mark mRemoveTempFile as a bool"jenkins-bot2021-02-181-1/+1
|\ \ \ \ \ \
| * | | | | | Mark mRemoveTempFile as a boollokal-profil2021-02-181-1/+1
| | |_|_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | This is only ever used as a bool, so do not tag it as a string. Change-Id: I582a8f57b11ebf165ec2831c25d65d04c8d7cb3a
* | | | | | Merge "DerivedPageDataUpdater: Skip all data updates when page is deleted"jenkins-bot2021-02-182-7/+34
|\ \ \ \ \ \
| * | | | | | DerivedPageDataUpdater: Skip all data updates when page is deletedUmherirrender2021-02-172-7/+34
| | |_|_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | There is nothing the updates can do, LinksUpdate also fails if there is no id for the given title on master Bug: T271356 Change-Id: I7fb690bec8dbfadf71eb9caacb4edbfe6b785e73
* | | | | | Merge "Remove unused patch-img_exif.sql"jenkins-bot2021-02-181-3/+0
|\ \ \ \ \ \
| * | | | | | Remove unused patch-img_exif.sqlAmir Sarabadani2021-02-181-3/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This file is not needed since ee117b601d89296485c76eebd4f9e9bfc4bb234f (2005) Change-Id: I687d2fa38854dfa2ad9575de40817efa3a9b2d50
* | | | | | | Merge "SpecialImport: Fix typo in name of 'mapping' input"jenkins-bot2021-02-182-2/+2
|\ \ \ \ \ \ \
| * | | | | | | SpecialImport: Fix typo in name of 'mapping' inputEd Sanders2021-02-182-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Bug: T275031 Change-Id: I733c538b849eff2de7dc0560970aa2f588c98704
* | | | | | | | Merge "Localisation updates from https://translatewiki.net."jenkins-bot2021-02-1817-24/+37
|\ \ \ \ \ \ \ \ | |/ / / / / / / |/| | | | | | |
| * | | | | | | Localisation updates from https://translatewiki.net.Translation updater bot2021-02-1817-24/+37
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: Ibc5db6b5d079bd2fff067937bfcc6da26b87d53a
* | | | | | | | Merge "objectcache: make MediumSpecificBagOStuff::updateOpStats more robust"jenkins-bot2021-02-184-48/+55
|\ \ \ \ \ \ \ \ | |_|/ / / / / / |/| | | | | | |
| * | | | | | | objectcache: make MediumSpecificBagOStuff::updateOpStats more robustAaron Schulz2021-02-184-48/+55
| | |/ / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Make the method accept either an integer-indexed list of keys or a map of keys to the send/receive payload size information tuple. Make sure that no BagOStuff subclasses are passing in $keys directly from the call to the public BagOStuff methods; these allow keys in the form of a list or a map of arbitrary strings to keys. Change-Id: I9687d25a4dd1c7b4b304f9fd543cc0a26a595962
* | | | | | | Merge "PostgresUpdater: Move `bot_passwords` table addition down"jenkins-bot2021-02-181-1/+1
|\ \ \ \ \ \ \ | |_|/ / / / / |/| | | | | |
| * | | | | | PostgresUpdater: Move `bot_passwords` table addition downAmmarpad2021-02-171-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This should have no effect, but it helps in making this file consistent with Mysql/Sqlite updaters. Table was introduced in 1.27 (Iaa4015e) and was added in 'new tables' section, because then Postgres separated them per patch type instead of per release. Separating them per release (as MySql/Sqlit do) is more logical and it's even more important now that we are not keeping them indefinitely. Bug: T272199 Change-Id: I1fa6d81c9903fb6188f00b791794b3199a0a5aa3
* | | | | | | Merge "Drop 1.25 updgrade patch files."jenkins-bot2021-02-188-141/+0
|\| | | | | |
| * | | | | | Drop 1.25 updgrade patch files.Ammarpad2021-02-178-141/+0
| | |/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 5 files for MySql/Sqlite/Postgres. MysqlUpdater.php seems to have incorrect comment for 1.26 release The hitcounter table was dropped in 1.25 (I0e5006a), so all the files dropped here are part of schema change for 1.25. Bug: T272199 Change-Id: I6a509a54d31f68e423b3d76d26259b1f88aa22bd
* | | | | | Merge "MysqlUpdater::doUnsignedSyncronisation(): Increase grepability of ↵jenkins-bot2021-02-181-8/+10
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | filenames"
| * | | | | | MysqlUpdater::doUnsignedSyncronisation(): Increase grepability of filenamesReedy2021-02-131-8/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: Ia960fb7b945d6126e143b0ae8d856d0dc335e0d7
* | | | | | | Merge "MysqlUpdater::doLanguageLinksLengthSync(): Increase grepability of ↵jenkins-bot2021-02-181-4/+4
|\| | | | | | | |_|_|/ / / |/| | | | | | | | | | | filenames"
| * | | | | MysqlUpdater::doLanguageLinksLengthSync(): Increase grepability of filenamesReedy2021-02-131-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | Change-Id: Iee5f26d33b28a7c4bb383f5610b58297dfd729dd
* | | | | | Merge "Fix a typo in ExpirationAwareness (Idom => Idiom)"jenkins-bot2021-02-181-1/+1
|\ \ \ \ \ \
| * | | | | | Fix a typo in ExpirationAwareness (Idom => Idiom)Martin Urbanec2021-02-181-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: Ie181ec5d6dcd917f509af74e3af4717142a66e8d
* | | | | | | API: Prominently link to [[mw:API:Etiquette]] on api.phpKunal Mehta2021-02-171-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | api.php is the main documentation entry point, so we should have a prominent link to the Etiquette and usage guidelines that provide further pointers to the User-Agent policy and more. Change-Id: I7134e8525658fcf0065de4aae608e72175f451aa
* | | | | | | Merge "objectcache: factor out WANObjectCache::fetchKeys() method"jenkins-bot2021-02-181-67/+107
|\ \ \ \ \ \ \
| * | | | | | | objectcache: factor out WANObjectCache::fetchKeys() methodAaron Schulz2021-02-141-67/+107
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Avoid internal calls to methods with messy legacy signatures like get() and getMulti(). This avoids phan warnings about suspicious array access. Make WANObjectCache::PASS_BY_REF an empty array and tweak the default value for $info reference parameters. Tweak the parameter and usage documentation for get() and getMulti() to focus on the non-legacy case. Change-Id: Ia5b3df6d3df0c1e87821e78384c6e02991792575