| Commit message (Collapse) | Author | Age | Files | Lines |
|---|
| |
|
|
|
|
| |
This now aligns with Parsoid commit 4d44dbd77363ad4e7c428ecd30029e906018fa6c
Change-Id: If883c36d599464aa6ed49edf71fd44dc880b3efd
|
| |
|
|
|
|
| |
This now aligns with Parsoid commit 13ccf50c0b4dbc87d5e28e7d474c3bcd1fcb7a2a
Change-Id: Iea302187ef1465cbe66fa6cc72e3b7e8ae796ad8
|
| |
|
|
|
|
|
| |
Per https://meta.wikimedia.org/wiki/Talk:Interwiki_map/Archives/2024#RFC
Bug: T71997
Change-Id: I2b6b63f7809e5801c78de370c828bbca5fe95e64
|
| |
|
|
|
|
|
|
|
|
|
|
|
| |
New options added: `iwl`, `links`, `special`, `extlinks`, and `templates`,
and handling of existing `ill` option tweaked to be consistent.
Added some tests to exercise these options, focusing on the handling
of title fragments. Attempted to make the output formatting consistent
among options; a future unification (I32df68714ffdf2f0745b974f47bc3ccceef1f41c)
should help DRY these out further.
Bug: T310512
Change-Id: Ic9c766ae4362969de124ad9d66eb47cfa68395c6
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Move Linker::makeExternalLink to the LinkRenderer service, as has been
done with the other static methods of Linker.
In order to allow phan's SecurityCheckPlugin to perform a more accurate
analysis of taintedness, tweak the API of Linker::makeExternalLink to
clearly indicate via the type system whether the link text has already
been escaped or not: a `string` argument will always be escaped, and
if the argument is already escaped it should be passed as an HtmlArmor
object. In refactoring, `Message` arguments were also common, and accept
them as-is to avoid the caller having to think about whether to call
Message::text() or Message::escaped().
This allows us to provide a more precise taint type to the $text argument,
avoids an opaque boolean argument, and avoids spurious errors from
SecurityCheck.
We also require the caller to explicitly pass a Title context, instead
of implicitly relying on the global $wgTitle. This works cleanly
everywhere except for CommentParser, which has a $selfLinkTarget which
generally works as the title context for the external link, but which
is nullable. The original Linker::makeExternalLink() used $wgTitle as
a fallback, but $wgTitle can also be null in some circumstances. The
title context only determines how $wgNoFollowNsExceptions is handled,
so existing code basically just ignored $wgNoFollowNsExceptions when
$wgTitle was null, which isn't terrible. A future refactor could/should
clean up CommentParser to ensure that there is always a non-null title
context that can be used.
Change-Id: I9bcf4780f388ba639a9cc882dd9dd42eda5736ae
|
| |
|
|
|
|
| |
This now aligns with Parsoid commit 6bc2181083b69fa94758da6ed35ef6d8e672dfa0
Change-Id: Ic48bf017f7ed89554bb8c7591f8bbb4b02c40458
|
| |
|
|
|
|
| |
This now aligns with Parsoid commit f4a186b9b63f9216b2d7a363aa1ed45aef59b0c4
Change-Id: Ie48f0c03a1539a575389f766b994c54a75bfb3e1
|
| |
|
|
|
|
| |
This now aligns with Parsoid commit 00d6c00c5972705c0f88d398e4b78f757ef86f4b
Change-Id: I051d1b29e480292078f681fc722a63d0d1588d15
|
|
|
This now aligns with Parsoid commit a3d35adf78c3c7e6a8f04e9a55b76f3efefaedb8
The monolithic parserTests.txt have been split up into a number of smaller
test files (more splitting to come) and three existing parser test files have
been renamed:
mediaParserTests.txt => media.txt
legacyMediaParserTests.txt => legacyMedia.txt
extraParserTests.txt => badCharacters.txt
This reduces redundancy and filename length now that we've got a dozen or so
of these.
Change-Id: I2c1461341fcb3f3cef8e49b5575cd30f3a38e500
|
