Page MenuHomePhabricator

Remove StopForumSpam/updateBlacklist.php in favour of maintenance/updateBlacklist.php
Closed, ResolvedPublic

Description

updateBlacklist.php was moved in 3e7633e0bc11

Given that such file was renamed a while ago, maybe we should think about removing the file in the future, with a deprecation period beforehand.

Event Timeline

Change 454086 had a related patch set uploaded (by MarcoAurelio; owner: MarcoAurelio):
[mediawiki/extensions/StopForumSpam@master] Deprecate updateBlacklist.php file

https://gerrit.wikimedia.org/r/454086

Change 454086 merged by jenkins-bot:
[mediawiki/extensions/StopForumSpam@master] Soft-deprecate updateBlacklist.php file

https://gerrit.wikimedia.org/r/454086

Krinkle renamed this task from Deprecate updateBlacklist.php in favor of maintenance/updateBlacklist.php to Remove StopForumSpam/updateBlacklist.php in favour of core's maintenance/updateBlacklist.php.Feb 8 2019, 9:36 PM
Krinkle moved this task from Unsorted to Migrate / Replace on the Technical-Debt board.

Change 500186 had a related patch set uploaded (by MarcoAurelio; owner: MarcoAurelio):
[mediawiki/extensions/StopForumSpam@master] Follow-up I5a9c23ce: Hard-deprecated updateBlacklist.php

https://gerrit.wikimedia.org/r/500186

@Krinkle Patch for hard deprecation submitted above. Shall we move back to the Technical-Debt (Deprecation process) board?

No, that board is for tracking the deprecation cycle of MediaWiki core and other WMF-maintained software. This task seems well-suited on StopForumSpam's own board, and in addition on the generic tech-debt workboard for any usage/referenced that may need to be updated.

Change 500186 merged by jenkins-bot:
[mediawiki/extensions/StopForumSpam@master] Follow-up I5a9c23ce: Hard-deprecate updateBlacklist.php

https://gerrit.wikimedia.org/r/500186

Change 500836 had a related patch set uploaded (by MarcoAurelio; owner: MarcoAurelio):
[mediawiki/extensions/StopForumSpam@REL1_32] Follow-up I5a9c23ce: Hard-deprecate updateBlacklist.php

https://gerrit.wikimedia.org/r/500836

@Legoktm @Daimona The REL1_32 cherry pick fails on Abusefilter and EquivSet in https://integration.wikimedia.org/ci/job/quibble-vendor-mysql-hhvm-docker/42856/console

Any ideas?

D'oh: all PHP tests for StopForumSpam looks broken on REL1_32 :-/

@MarcoAurelio Yes, it's T189560. You have to vote V+2.

I see it mentions 1.31. It's REL1.32 also affected?

@MarcoAurelio Yes, it's T189560. You have to vote V+2.

I see it mentions 1.31. It's REL1.32 also affected?

Yes, any REL* branch is.

Alright. Thank you. Given that it is my own change I'll let others to force-merge this if they like.

Change 500836 merged by Umherirrender:
[mediawiki/extensions/StopForumSpam@REL1_32] Follow-up I5a9c23ce: Hard-deprecate updateBlacklist.php

https://gerrit.wikimedia.org/r/500836

Change 521229 had a related patch set uploaded (by Reedy; owner: Reedy):
[mediawiki/extensions/StopForumSpam@master] Remove old entry points

https://gerrit.wikimedia.org/r/521229

Reedy renamed this task from Remove StopForumSpam/updateBlacklist.php in favour of core's maintenance/updateBlacklist.php to Remove StopForumSpam/updateBlacklist.php in favour of maintenance/updateBlacklist.php.Jul 8 2019, 9:26 AM

Change 521229 merged by jenkins-bot:
[mediawiki/extensions/StopForumSpam@master] Remove old entry points

https://gerrit.wikimedia.org/r/521229

MarcoAurelio assigned this task to Reedy.

Closing as they've been removed on master. If a port to any REL is needed, feel free to cherry-pick.