Page MenuHomePhabricator
Create Task
Maniphest T383702

Document onConditionalDefaultOptionsAddCondition under mw:Manual:Hooks
Closed, DeclinedPublic
Actions

Description

In rMW37985a198de7: ConditionalDefaultsLookup: allow to dynamically add conditions for evaluation (T376918), @Sgs added the onConditionalDefaultOptionsAddCondition hook to conditional defaults. While filling T383701, I noticed the hook was not added to mw:Manual:Hooks, making it harder to discover and understand.

We should expand https://www.mediawiki.org/wiki/Manual:Hooks/ConditionalDefaultOptionsAddCondition with the hook documentation, and then add it to the table.

Related Objects

Event Timeline

Urbanecm_WMF created this task.Jan 14 2025, 4:38 PM
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptJan 14 2025, 4:38 PM
Urbanecm_WMF added a project: MediaWiki-Core-Preferences.Jan 14 2025, 4:39 PM
Urbanecm_WMF moved this task from Inbox to Estimated tasks backlog on the Growth-Team board.Jan 17 2025, 1:10 PM
KStoller-WMF triaged this task as Low priority.Jan 21 2025, 4:53 PM
KStoller-WMF edited projects, added: Growth-Team (Maintenance); removed: Growth-Team.Feb 3 2025, 11:53 PM
DMburugu edited projects, added: Growth Team Maintenance; removed: Growth-Team (Maintenance).Oct 8 2025, 11:35 AM
Restricted Application added a project: Growth-Team. · View Herald TranscriptOct 8 2025, 11:35 AM
KStoller-WMF moved this task from Estimated tasks backlog to Needs Estimation on the Growth-Team board.Oct 17 2025, 10:56 PM
Michael moved this task from Backlog to Optimize Production Resource Usage on the Growth Team Maintenance board.Dec 22 2025, 7:53 PM
Sgs added a comment.Feb 11 2026, 2:42 PM

Is this hook still worth keeping?

Michael subscribed.Mar 4 2026, 5:47 PM
Urbanecm_WMF moved this task from Needs Estimation to Needs Refinement on the Growth-Team board.Mar 23 2026, 3:14 PM

During the 2026-03-23 estimation meeting, we were leaning towards removing the hook altogether, rather than documenting it. Let's discuss this question in Backlog Refinement and make the call there.

Michael closed this task as Declined.Apr 20 2026, 3:21 PM
Michael mentioned this in T423889: Remove onConditionalDefaultOptionsAddCondition hook.

We talked about it and decided to remove that hook instead: T423889: Remove onConditionalDefaultOptionsAddCondition hook

Content licensed under Creative Commons Attribution-ShareAlike (CC BY-SA) 4.0 unless otherwise noted; code licensed under GNU General Public License (GPL) 2.0 or later and other open source licenses. By using this site, you agree to the Terms of Use, Privacy Policy, and Code of Conduct. · Wikimedia Foundation · Privacy Policy · Code of Conduct · Terms of Use · Disclaimer · CC-BY-SA · GPL · Credits