Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use LanguageFactory class to prevent deprecation warning in MediaWiki 1.40 #28

Conversation

rti
Copy link
Contributor

@rti rti commented Nov 10, 2023

Since MediaWiki 1.35 Language::factory() is deprecated [1]. Since MediaWiki 1.40 this function emits a deprecation warning [2]. This patch uses the new way to access the Language instance.

[1] https://www.mediawiki.org/wiki/Release_notes/1.35
[2] https://www.mediawiki.org/wiki/Release_notes/1.40

Since MediaWiki 1.35 Language::factory() is deprecated [1]. Since
MediaWiki 1.40 this function emits a deprecation warning [2]. This patch
uses the new way to access the Language instance.

[1] https://www.mediawiki.org/wiki/Release_notes/1.35
[2] https://www.mediawiki.org/wiki/Release_notes/1.40
@rti
Copy link
Contributor Author

rti commented Nov 10, 2023

Relevant bug on phabricator https://phabricator.wikimedia.org/T350934

@JeroenDeDauw
Copy link
Member

What is the compatibility impact? With this patch, is the extension still compatible with MW 34, or does it now need a more recent version of MW?

@rti
Copy link
Contributor Author

rti commented Nov 13, 2023

According to the docs, the function is available since 1.32 [1]. I did not test that though.

[1] https://doc.wikimedia.org/mediawiki-core/master/php/classMediaWiki_1_1MediaWikiServices.html#af9a83ed422f1324938ce6bce42c80cfa

@JeroenDeDauw JeroenDeDauw merged commit 74c3f70 into ProfessionalWiki:master Nov 13, 2023
6 checks passed
@rti
Copy link
Contributor Author

rti commented Nov 15, 2023

Thanks a lot for merging!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants