Page MenuHomePhabricator

Allow history merge without a redirect
Open, Stalled, LowPublic

Description

When after a history merge a page has revisions, a redirect is created.
Users with the suppressredirect user right should have a check box to choose if this redirect should be created or not.
For ContentModels without redirects this checkbox should be greyed out, as on [[Special:MovePage]]


Version: 1.24rc
Severity: enhancement

Details

Reference
bz68096

Event Timeline

bzimport raised the priority of this task from to Low.Nov 22 2014, 3:36 AM
bzimport set Reference to bz68096.
bzimport added a subscriber: Unknown Object (MLST).
DannyS712 added a subscriber: DannyS712.

Implementation proposed:
If the user has the suppressredirect right

  1. Add a checkbox on Special:MergeHistory for leaving a redirect
  2. Have the checkbox "checked" by default
  3. Mimic Special:MovePage regarding content models' support for redirects

Question:
Should not leaving a redirect

  1. ...result in a different log type (merge/noredirect or similar) à la T152346: Separate log action for moves suppressing redirects
  2. ...save a log entry parameter that it was without a redirect, à la current move logging

Tagging Platform Engineering as a feature request to review

eprodromou added a subscriber: eprodromou.

This makes sense as a feature; using a single log line is probably fine here. We'll track this as it proceeds.

This makes sense as a feature; using a single log line is probably fine here. We'll track this as it proceeds.

Would that single log line be a different type, or with a parameter?

I think this should wait until after T134675: Convert Special:MergeHistory to OOUI - lets not add more to the output form right before the form is redone

DannyS712 changed the task status from Open to Stalled.Feb 26 2020, 5:17 AM

I think this should wait until after T134675: Convert Special:MergeHistory to OOUI - lets not add more to the output form right before the form is redone

Stalling

This makes sense as a feature; using a single log line is probably fine here. We'll track this as it proceeds.

Would that single log line be a different type, or with a parameter?

{{ping}} @eprodromou in case you missed the question