Commit graph

34 commits

Author SHA1 Message Date
thiemowmde 3a20f81f31 Reduce very deep indention in some larger if/elseif blocks
Change-Id: I176f3cd5d28fde83bb2e243cc72d9e7c77ca6f51
2023-01-10 11:56:28 +01:00
jenkins-bot acb8030591 Merge "Replace Parser::getFunctionLang() with ::getTargetLanguage()" 2022-11-16 21:50:41 +00:00
jenkins-bot c72e162c96 Merge "Use HookContainer to register hooks on the fly." 2022-11-09 14:49:06 +00:00
jenkins-bot e02ad426b8 Merge "Remove dead code from #titleparts implementation" 2022-11-08 22:46:18 +00:00
daniel f97da7df26 Use HookContainer to register hooks on the fly.
Manipulating $wgHooks is deprecated and unreliable.
Use HookContainer instead.

Needed-By: Iddcb760cf8961316d6527e81b9aa968657d8354c
Change-Id: I413cd2ae43fe4b13917d61a50e190b536fce7279
2022-11-05 20:02:09 +00:00
thiemowmde 928a21605a Remove dead code from #titleparts implementation
It's impossible that explode() returns an empty array. This is dead
code.

Also add tests for the previously untested #titleparts function to
make sure this patch doesn't change any behavior. Including weird
usages with negative numbers.

Change-Id: Ib0d2503fdb0900bf6db3ebbc26d2ff648dbfc86d
2022-11-03 18:30:49 +01:00
thiemowmde ee66f0bfff Simplify weirdly complex ParserFunctions::ifexistInternal()
Personally I find the code much more readable when the private
helper method returns a bool.

Note this patch is intentionally tiny and focusses exclusively on
this one detail. There are many parser tests to prove that nothing
changed.

Change-Id: Ibe07c27aab9cbfc4705769820be679cd54acb79b
2022-11-03 17:50:41 +01:00
thiemowmde 129745609b Remove unused default values from method parameters
These methods are private and never called without these parameters.
These values are dead code.

Change-Id: I422f2d28f60cf3caf6234e5421b3613f80c1e6f1
2022-11-03 07:32:04 +00:00
libraryupgrader 57d87a2e33 build: Updating mediawiki/mediawiki-phan-config to 0.12.0
Change-Id: Ica980c2a25fb00feb6f9775ecb2d7830a3559a95
2022-10-10 21:05:20 +00:00
C. Scott Ananian f11b1099d0 Replace Parser::getFunctionLang() with ::getTargetLanguage()
Parser::getFunctionLang() is being deprecated.  These two functions
have been identical since 7df3473cfea59df53debb7a9eefffed8a7f20fb3
in MW 1.19 (2012) and this extension already requires MW >= 1.35.

Bug: T318860
Change-Id: I87a7ceedce173f6de4bb6722ffe594273c7b0359
2022-09-28 15:59:38 -04:00
Fomafix 6468bf9ebb Replace deprecated Language::isValidBuiltInCode and Language::factory
Change-Id: I4c5bf4d68bcb79da33864dbffe395e3818050eb4
2022-09-17 10:00:50 +00:00
Reedy f647423039 Correct namespacing to match best practices
Bug: T303105
Change-Id: Ic09ed23c4248e74f93e5807212fe73d241251281
2022-03-05 15:06:59 +00:00
Thiemo Kreuz ce22d48123 Fix and update PHPDoc type hints
The parameters passed to these parser hook handler functions have
guaranteed types. They are either strings or an array of PPNode
objects, depending on the SFH_OBJECT_ARGS flag. When a parameter is
missing the default value from the function header is used.

Change-Id: Ie0196f43f94cf002110c07ad2de30adfa6b77179
2021-11-20 08:52:38 +00:00
libraryupgrader 5b572a6f6a build: Updating mediawiki/mediawiki-codesniffer to 37.0.0
Change-Id: I1f93907a1f7b9f3dace4b2e65f7532cedc11800f
2021-07-23 06:43:57 +00:00
TChin 66f7d36e68 Record #ifexist media in imagelinks table
`#ifexist` used to only record media links if the file existed, now it does it either way.

Bug: T245965
Change-Id: Ida5dd30669fbc3471d792fd4c202bb0ff74737ec
2021-07-01 19:33:50 +00:00
libraryupgrader 7df8411050 build: Updating composer dependencies
* mediawiki/mediawiki-codesniffer: 35.0.0 → 36.0.0
  The following sniffs now pass and were enabled:
  * MediaWiki.Commenting.PropertyDocumentation.MissingDocumentationPublic

* php-parallel-lint/php-parallel-lint: 1.2.0 → 1.3.0

Change-Id: Ifaca144ccfd753ea00d1dbba0d12941b0050e2a4
2021-05-04 08:49:27 +00:00
C. Scott Ananian ff89dae8ff Replace use of Parser::$mOutput, deprecated in 1.35
The replacement, Parser::getOutput(), has been in MediaWiki for over a
decade (since 1.12.2 in 2008).

Bug: T275160
Change-Id: I4f4f7b0118470741a6cdaba562f858e425fcf350
2021-02-18 18:51:56 -05:00
Reedy 3767db25f7 Fix PSR12.Properties.ConstantVisibility.NotFound
Bug: T253169
Change-Id: If4de0fff2886961481faeefe3f8021ae6760e116
2020-05-30 01:27:32 +01:00
ArtBaltai a3706c0d42 Use LanguageConverter::findVariantLink() instead of Language::findVariantLink()
Bug: T226832
Change-Id: I6d7e9019def4bbccdbffcdcfd523e614902fe1e6
2020-05-06 04:11:23 +03:00
Umherirrender b00143f151 Use MediaWikiServices::getLinkCache()
Since 1.28

Change-Id: I502726983b7ebddcf887d377a1c87740441b0a12
2020-03-14 21:25:00 +01:00
Umherirrender 2c97d15088 Use MediaWikiServices::getRepoGroup
This required MediaWiki 1.34

Change-Id: I1578dffaa20d80696378827397a0593d1c634477
2020-03-14 14:45:29 +01:00
libraryupgrader b718f1bf1a build: Updating mediawiki/mediawiki-codesniffer to 29.0.0
Additional changes:
* Also sorted "composer fix" command to run phpcbf last.

Change-Id: Id96905c5755a2705e434b3d1755827d2788c0efb
2020-01-14 09:00:15 +00:00
Max Semenik 1e8444fad1 Improve ExprParser test coverage
Had to change a bit how ExprError is used, to prevent having to turn unit
tests into ingergation tests, adding 5 seconds to run time.

Change-Id: Id4e158f3a4418d320b1c0bdac834ebbba03762af
2019-09-05 15:37:21 -07:00
Max Semenik 8d6e567e5a ParserFunctions class: tweak function declarations
* Force type hints
* Documentation
* Give links to documentation on mw.org to make high-level understanding of
  what each function does easier.

Change-Id: I7708343540b7980e57b68ce9b2d9816d2aee710f
2019-07-20 21:27:52 +00:00
jenkins-bot c8a1f28418 Merge "ParserFunctions class: make everything that doesn't need to be public private" 2019-07-20 21:19:01 +00:00
jenkins-bot dad921ea53 Merge "Remove function existence check" 2019-07-18 02:50:44 +00:00
Max Semenik cca4ec9af0 Remove function existence check
PPFrame::setTTL() is present in every supported MediaWiki version.

Change-Id: I7065723bfbd2170cd7c951d1f9b3c4a0318f9938
2019-07-18 02:38:17 +00:00
Max Semenik 3fd4ac43ac ParserFunctions class: make everything that doesn't need to be public private
Change-Id: I818315e5f0cd80700731d212ba11fbf341582ba1
2019-07-18 02:36:30 +00:00
Max Semenik 8b50403fe8 Get rid of *Obj() functions
They're a rudiment from the time when the extension had to be compatible
with both SFH_OBJECT_ARGS supporting and not versions of MediaWiki.
This is in the past, so unify them now by merging and or deleting the
old versions. The resulting functions don't end with "Obj" anymore.
One "common" function was left alone to be refactored at a later time,
however I've renamed it and made private.

Change-Id: Ice97e587bf7cc0469476c2e53268d2774ddfdd04
2019-07-18 02:35:53 +00:00
Max Semenik 439544224f Replace a single use function with a closure
Change-Id: Id6408bb203fc984f7baaaf3358cea79c281005bf
2019-04-04 17:22:04 -07:00
Max Semenik e721c193d9 Avoid using $wgContLang
Change-Id: If43a6aa5580397264b2ca9dff5ef7d0da859e3d6
2019-04-03 23:25:11 -07:00
Max Semenik e684068ed9 Convert variable used as a constant into constant proper
Change-Id: I1c7914cc9015c18f4c3877b4acc5b3be921ec4d5
2019-04-03 19:20:15 -07:00
Max Semenik 8e7a3ce5df Remove b/c code
New code requires 1.32, this extension requires 1.32.

Change-Id: I14b9d0074d057d6fb323d54502d0a4c64fc74879
2019-04-03 19:20:15 -07:00
Max Semenik f53debae7d Namespace this extension
Change-Id: I0c7719f17f74180a4c85f914888262f7e3b13111
2019-04-03 19:20:14 -07:00
Renamed from includes/ExtParserFunctions.php (Browse further)