Maniphest T193364

SpecialBlock::getSuggestedDurations should not return 'other' unless if appropriate
Closed, ResolvedPublic
Actions

Assigned To

Authored By

	Huji
	Apr 30 2018, 1:35 AM

Description

In 3481e3b2e0af @dbarratt modified SpecialBlock::getSuggestedDurations to return an option named 'other' at the end of all other block duration suggestions. While this works well for MW core, he accidentally broke another dependency in AbuseFilter in includes/views/AbuseFilterViewEdit.php

I will make a patch shortly to add an optional variable to getSuggestedDurations to suppress the 'other' option whenever it is not desired.

Details

	Subject	Repo	Branch	Lines +/-
	Make the 'other' option superessable in getSuggestedDurations	mediawiki/core	master	+4 -2

Customize query in gerrit

Related Objects

Mentioned Here: rMW3481e3b2e0af: Create Expiry Widget with Date Time Selector

Event Timeline

Huji created this task.Apr 30 2018, 1:35 AM

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptApr 30 2018, 1:35 AM

Change 429708 had a related patch set uploaded (by Huji; owner: Huji):
[mediawiki/core@master] Make the 'other' option superessable in getSuggestedDurations

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

gerritbot added a project: Patch-For-Review.Apr 30 2018, 1:39 AM

• TBolliger moved this task from Backlog to User blocking on the MediaWiki-User-management board.May 18 2018, 4:03 PM

• Vvjjkkii renamed this task from SpecialBlock::getSuggestedDurations should not return 'other' unless if appropriate to 3zdaaaaaaa.Jul 1 2018, 1:13 AM

• Vvjjkkii triaged this task as High priority.

• Vvjjkkii added projects: CheckUser, Connected-Open-Heritage-Batch-uploads (RAÄ-KMB_1_2017-02), Tamil-Sites, Gamepress, Hashtags, Jade, KartoEditor, Language-2018-Apr-June, New-Editor-Experiences, Mail, TCB-Team (now WMDE-TechWish).

• Vvjjkkii updated the task description. (Show Details)

• Vvjjkkii removed subscribers: gerritbot, Huji, Aklapper.

AntiCompositeNumber renamed this task from 3zdaaaaaaa to SpecialBlock::getSuggestedDurations should not return 'other' unless if appropriate.Jul 1 2018, 12:32 PM

AntiCompositeNumber raised the priority of this task from High to Needs Triage.