Page MenuHomePhabricator

Add timestamp, reason and user to Special:ProtectedTitles
Open, LowPublicFeature

Description

The protected_titles table has some columns which are not part of the Special page, would be nice to see there on the special page.


Version: 1.23.0
Severity: enhancement

Details

Reference
bz63318

Event Timeline

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

The abandon message is "No longer support for this change set." Any idea why?

The abandon message is "No longer support for this change set." Any idea why?

The patch set was open for 6 months to review, but was not succesfully reviewed. After the 6 months I have abandon the patch set, because I did not want give longer support for that (rebasing for example). But I also give the sentence "Feel free to resubmit and/or modify under your own name.", so this task is open for implemenation, do your own or use the already existing patch set and try to apply to master.

The protected_titles table holds essentially the same information as page_restrictions, but the structures of the tables are fundamentally different. page_restrictions holds some information specific to existing pages, which is understandable. protected_titles has some unnecessary columns:

  • pt_user (the user who protected the title, note that page_restrictions.pr_user is reserved for per-user restrictions and currently unused)
  • pt_reason
  • pt_timestamp

Similarly, the Special:ProtectedTitles and Special:ProtectedPages pages provide fundamentally different information: the former selects from the protected_titles page itself, while the latter selects from the logging table.

I'd suggest to migrate Special:ProtectedTitles to use logging as well (all the info is already available there) and drop the unnecessary columns.

Aklapper changed the subtype of this task from "Task" to "Feature Request".Feb 4 2022, 11:13 AM