Skip to content

Replace call to removed MWHttpRequest::factory#493

Merged
tarrow merged 1 commit into
mainfrom
T407480
Oct 21, 2025
Merged

Replace call to removed MWHttpRequest::factory#493
tarrow merged 1 commit into
mainfrom
T407480

Conversation

@tarrow
Copy link
Copy Markdown
Contributor

@tarrow tarrow commented Oct 21, 2025

Since 1.34 MWHttpRequest::factory has been deprecated and it was removed in 1.41. This patch replaces it with what I believe is the obvious successor: HttpRequestFactory

This follows the recommended pattern from the mediawiki.org docs[1] of creating the factory from MediaWikiServices and then calls it with the options as before.

[1] https://www.mediawiki.org/wiki/Manual:HttpRequestFactory.php

Bug: T407480

Since 1.34 MWHttpRequest::factory has been deprecated and
it was removed in 1.41. This patch replaces it with what I
believe is the obvious successor: HttpRequestFactory

This follows the recommended pattern from the mediawiki.org
docs[1] of creating the factory from MediaWikiServices and then
calls it with the options as before.

[1] https://www.mediawiki.org/wiki/Manual:HttpRequestFactory.php

Bug: T407480
@tarrow tarrow requested a review from AndrewKostka October 21, 2025 08:30
@tarrow tarrow merged commit 5d2b121 into main Oct 21, 2025
9 checks passed
@tarrow tarrow deleted the T407480 branch October 21, 2025 13:04
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