Maniphest T312095

Remove PermissionManager from Logger class
Closed, ResolvedPublic
Actions

Assigned To

Diesel_kapasule

Authored By

	Tchanders
	Jul 5 2022, 12:49 PM

Tags

Referenced Files

None

Subscribers

Description

The Logger used to depend on the PermissionManager service; however, it is no longer used.

Acceptance criteria

Remove the PermissionManager form the Logger
Update related tests, etc

Details

	Subject	Repo	Branch	Lines +/-
	Remove PermissionManager from Logger class and related tests	mediawiki/extensions/IPInfo	master	+0 -32

Customize query in gerrit

Event Timeline

Tchanders created this task.Jul 5 2022, 12:49 PM

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptJul 5 2022, 12:49 PM

Diesel_kapasule changed the task status from Open to In Progress.Jul 12 2022, 11:21 AM

Diesel_kapasule claimed this task.

Change 813223 had a related patch set uploaded (by Diesel kapasule; author: Diesel kapasule):

[mediawiki/extensions/IPInfo@master] Removal of PermissionManager from Logger class and related tests

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

gerritbot added a project: Patch-For-Review.Jul 12 2022, 11:45 AM

Tchanders moved this task from Triage/To be Estimated to AHaT Sprint 11: The Fascinator on the Anti-Harassment board.Jul 18 2022, 5:35 PM

Tchanders edited projects, added Anti-Harassment (AHaT Sprint 11: The Fascinator); removed Anti-Harassment.

Tchanders moved this task from Ready 🎬 (ONLY IF YOU HAVE NO MORE CODE TO REVIEW) to Code Review 🔍 on the Anti-Harassment (AHaT Sprint 11: The Fascinator) board.

ARamirez_WMF edited projects, added Anti-Harassment (AHaT Sprint 12: The Dutch Cap); removed Anti-Harassment (AHaT Sprint 11: The Fascinator).Jul 18 2022, 11:23 PM

ARamirez_WMF moved this task from Ready 🎬 (ONLY IF YOU HAVE NO MORE CODE TO REVIEW) to Code Review 🔍 on the Anti-Harassment (AHaT Sprint 12: The Dutch Cap) board.Jul 18 2022, 11:23 PM

Tchanders moved this task from Code Review 🔍 to QA/Testing 🐞 on the Anti-Harassment (AHaT Sprint 12: The Dutch Cap) board.Jul 20 2022, 10:21 AM

Change 813223 merged by jenkins-bot:

[mediawiki/extensions/IPInfo@master] Remove PermissionManager from Logger class and related tests

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

Maintenance_bot removed a project: Patch-For-Review.Jul 20 2022, 10:30 AM

Diesel_kapasule closed this task as Resolved.Jul 20 2022, 10:37 AM

ReleaseTaggerBot added a project: MW-1.39-notes (1.39.0-wmf.22; 2022-07-25).Jul 20 2022, 11:00 AM

• Prtksxna moved this task from Backlog to Closed on the IP Info board.Aug 8 2022, 4:05 AM

• Prtksxna moved this task from QA/Testing 🐞 to Done Q4 2022 on the Anti-Harassment (AHaT Sprint 12: The Dutch Cap) board.Aug 8 2022, 4:21 AM