Wikipedia talk:Page mover/delete-redirect
Preliminary discussion
[edit]- The following discussion is closed. Please do not modify it. Subsequent comments should be made in a new section. A summary of the conclusions reached follows.
- I've opened the RFC --DannyS712 (talk) 23:17, 18 February 2021 (UTC)
@Pppery, Primefac, Uanfala, Wbm1058, and Wugapodes: (pinging those who participated in Wikipedia talk:Page mover/Archive 4#New user right allowing editors to move pages over one-revision pages that redirect to anywhere) Before I officially make this proposal, giving a chance for interested users to help flesh it out. Thoughts? --DannyS712 (talk) 06:38, 30 December 2020 (UTC)
- Without commenting on the substance, it's a simple and straight-forward proposal (which is what an RfC should be). Primefac (talk) 12:12, 30 December 2020 (UTC)
- Do administrators have the
delete-redirect
right yet? I would like to see how it works in practice. What sort of record is left in the logs, whether a deleted revision is left behind that an administrator could restore, etc. – wbm1058 (talk) 14:05, 30 December 2020 (UTC)- I don't see
delete-redirect
listed as an admin user-right at Special:ListGroupRights. Granted, admins don't need this right to perform this action. But, will admins doing this without the right be functionally different than page movers doing it with the right? wbm1058 (talk) 14:50, 30 December 2020 (UTC)- The only difference I can see is that it will show the resulting deletion as a normal deletion, rather than a move over a redirect, with no functionality changes. * Pppery * it has begun... 15:39, 30 December 2020 (UTC)
- Indeed - admins will log their deletions as
delete/delete
but page movers will be logged asdelete/delete_redir2
DannyS712 (talk) 20:30, 30 December 2020 (UTC)
- Indeed - admins will log their deletions as
- @Wbm1058 admins don't need this because they have normal
delete
rights DannyS712 (talk) 20:29, 30 December 2020 (UTC)
- The only difference I can see is that it will show the resulting deletion as a normal deletion, rather than a move over a redirect, with no functionality changes. * Pppery * it has begun... 15:39, 30 December 2020 (UTC)
- Administrators have every user right that page movers have. Before an RfC is started to expand this right to other users, it should first be granted to administrators. – wbm1058 (talk) 15:00, 30 December 2020 (UTC)
- The just-closed phabricator item says "The deletion is logged separately as
delete/delete_redir2
. What log is this separately logged in? What is the deletion tag log? wbm1058 (talk) 15:17, 30 December 2020 (UTC)- These deletions will still be logged in the deletion log. From my reading of the code, they will be indistinguishable from deleting single-revision redirects to the same target by moving over them in Special:Log (although I could be wrong here). The deletion tag log is an entirely unrelated feature, and entries in it are generated when a new page reviewer (or administrator) nominates a page for deletion (either speedy deletion or a deletion discussion) using the PageTriage extension * Pppery * it has begun... 15:39, 30 December 2020 (UTC)
- This matches my recollection from the coding - the log entries should be at https://en.wikipedia.org/wiki/Special:Log?type=delete&subtype=delete_redir just like for existing redirect deletions DannyS712 (talk) 20:31, 30 December 2020 (UTC)
- These deletions will still be logged in the deletion log. From my reading of the code, they will be indistinguishable from deleting single-revision redirects to the same target by moving over them in Special:Log (although I could be wrong here). The deletion tag log is an entirely unrelated feature, and entries in it are generated when a new page reviewer (or administrator) nominates a page for deletion (either speedy deletion or a deletion discussion) using the PageTriage extension * Pppery * it has begun... 15:39, 30 December 2020 (UTC)
- I don't see
- I don't see the need for the new revocation clause, because this proposal doesn't grant page movers the right to do anything significant they couldn't already do, only do things they can do in a less convoluted way. A page mover could already delete single-revision redirects by moving their targets over them, and then moving their former target back to the original location without leaving a redirect. Except for the deliberate misuse of the feature that I described above as already possible, any redirect deleted by a page mover falls under WP:G6 (
deleting redirects ... blocking page moves
). Aside from that, the proposal looks good. * Pppery * it has begun... 15:24, 30 December 2020 (UTC)- @Pppery removed, since the deliberate misuse of the feature you just described would already be a misuse of redirect suppression DannyS712 (talk) 03:40, 4 January 2021 (UTC)
Log event actions
[edit]mw:API:Logevents (API help) lists all the possible values (codes) of log actions. Where is it documented what all these codes mean? Specifically what do delete/delete
, delete/delete_redir
, and delete/delete_redir2
mean? Can you list each of these specific log actions on the Special:Log page? I hadn't noticed the "Type of deletion" dropdown on that page before. How is the "Type of deletion" queried using the API? wbm1058 (talk) 03:23, 31 December 2020 (UTC)
delete/delete
is a regular deletion by an administrator.delete/delete_redir
is any user moving a page over a single-revision redirect to the same target (the existing feature, not what is being proposed here).delete/delete_redir2
isn't currently used for anything, but will be used to log uses of the feature being proposed here if it gets approved. The "type of deletion" is queried by passing the relevant value to theleaction
parameter in the API. For example, all single-revision redirects you implicitly deleted as part of a page move, all pages you explicitly deleted not during a page move all pages you undeleted. * Pppery * it has begun... 18:13, 31 December 2020 (UTC)- Exactly ^^^ - sorry for not explaining, I've spent too much time coding and forgot it wasn't exposed to end users DannyS712 (talk) 23:16, 31 December 2020 (UTC)
Making a table:
API "action" | Special:Log "Type of deletion" | Description | Who can do this |
---|---|---|---|
"delete" | "Page deletion" | Deletion of a content page or a redirect that has multiple-edit history or doesn't target the page moved from | Administrators only? |
"delete_redir" | "Redirect overwrite" | Deletion of a redirect without multiple-edit history that targets the page moved from | All autoconfirmed editors |
"delete_redir2" | ?? | Deletion of a redirect without multiple-edit history that targets a different page than the page moved from, by a non-administrator granted the delete-redirect right |
Page-movers only |
How will "delete_redir2" be located in the deletion log? I don't see a new drop-down menu option for this.
API lookup for "delete_redir2" finds nothing, as expected because nobody has the delete-redirect
right yet.
Why does Pppery have 25 entries (in 2016) in the "Page deletion" log? If only admins are supposed to be able to do this? Software glitch? My first entry in this log was on 1 September 2015 (I was granted adminship at the end of August 2015).
But my first "Redirect overwrite" wasn't until 7 December 2016. I'm assuming that this wasn't broken out of the "Page deletion" bucket until later. Indeed, the first of these only appeared on 1 December 2016. – wbm1058 (talk) 23:19, 31 December 2020 (UTC)
- I don't believe that there was a separate log for redirect overwrite until after it was first introduced, which may explain why it was being logged as normal deletions originally, but I'll look through the history. For the "Type of deletion" on Special:Log,
delete_redir2
will be the same asdelete_redir
("Redirect overwrite") and the two subtypes will be shown together DannyS712 (talk) 23:48, 31 December 2020 (UTC)- @Wbm1058 see phab:T145991 - the separate "Redirect overwrite" log wasn't introduced until November 2016, which was after the original introduction of such deletions in phab:T106119. If you want the old log entries to be switched over, I suggest supporting phab:T154373 DannyS712 (talk) 23:51, 31 December 2020 (UTC)
- Moving a page over the redirect leaves an abandoned row in the revision table "
I think the issue here is that if you move a page over a redirect, the old redirect is gone forever, rather than being moved to the archive table.
" I had assumed that this was by design rather than a bug. The reason being it wasn't necessary to save the redirect since you knew where it redirected to. Why I have been sooo concerned that there would be no record of where these new "delete_redir2" redirects redirected to. It is reassuring to know now that these will be recorded in the log, and the deleted redirects will be recoverable from the deleted revisions. - My understanding of how moves worked formed from my earliest experience, before I became an administrator. My move log from early 2012 shows several "over redirect" moves but there are no deletion log entries from this timeframe, i.e. these "over redirect" moves did not leave an entry in the deletion log for the redirect they deleted. For example, at 15:19, 16 July 2012 I moved page Rental shop to Video rental shop over redirect. I don't see any deleted edits sitting under Video rental shop – that redirect is gone. I hadn't really noticed that this behavior had changed until now, on reviewing this. – wbm1058 (talk) 01:50, 1 January 2021 (UTC)
- Moving a page over the redirect leaves an abandoned row in the revision table "
- @Wbm1058 see phab:T145991 - the separate "Redirect overwrite" log wasn't introduced until November 2016, which was after the original introduction of such deletions in phab:T106119. If you want the old log entries to be switched over, I suggest supporting phab:T154373 DannyS712 (talk) 23:51, 31 December 2020 (UTC)
I just saw mw:Manual:Log actions, which is outdated. Can someone update it? wbm1058 (talk) 04:25, 31 December 2020 (UTC)
- Wow, its really out of date - I'll try to work on it when I have time DannyS712 (talk) 23:52, 31 December 2020 (UTC)
Wug's edit
[edit]Just got this ping and made some revisions. It looks good without them though, so feel free to revert whatever is not an improvement. The changes were too much to explain in an edit summary so I'm elaborating here. This seems good to go honestly, so I added some top-matter to get it ready to post. It includes the RFC tag and cats as well as a short question for the feedback request service bot. The {{draft proposal}} tag can be changed to {{proposal}} when ready, and then just remove the no wiki tags. I also added an empty discussion section.
I tried to streamline the background section. The first paragraph is substantially the same, but I tried to phrase it so that the antecedents are clear. I removed the paragraph on deletion with a note, though looking at it now I might want to add it back in the logging section. Getting into the weeds of how the deletions happen (or that they're deletions at all) is probably not necessary for the background and better to bring up on request in the discussion, imo. I added a paragraph on the page mover group and how it relates to these move types and this proposal.
I simplified the table and moved it into background. I removed the "API action" column since most people won't understand it and it may confuse the proposal. For example, delete_redir and delete_redir2 weren't mentioned and are easily confused with the delete-redirect
right under discussion. Since most people won't care about the API specifics, better to avoid any potential misunderstandings. I tried to clarify the "type of deletion" column header but maybe made it worse? May edit that again later. I copyedited the "page deletion" description as well. For all of them, I standardized the forat of the "who can do this" column stating the user right that allows the action and in parenthesis gave the user group that contains it.
I modified the proposal section so that the proposals are imperatives and start with verbs. I made "example use cases" a subheading but might undo that. Still, I think it would be worth writing up how this will simplify/obviate most round-robin moves which imo is the biggest gain from this proposal.
So like I said, looks good and revert what's not. Probably worth a listing at WP:CENT when it's live. Thanks for all the work you've put into this Danny! — Wug·a·po·des 22:07, 4 January 2021 (UTC)
- The table is now a bit confusing because the two types of redirect deletion have the same description on Special:Log/delete. Other than that, mostly looks good to me DannyS712 (talk) 00:06, 5 January 2021 (UTC)
Last call
[edit]@Primefac, Wbm1058, and Pppery: does this look ready to start / any final thoughts? --DannyS712 (talk) 02:52, 16 January 2021 (UTC)
- Looks ready to go to me. * Pppery * it has begun... 02:54, 16 January 2021 (UTC)
- OK. wbm1058 (talk) 04:00, 16 January 2021 (UTC)
RFC on granting delete-redirect to page movers
[edit]- The following discussion is an archived record of a request for comment. Please do not modify it. No further edits should be made to this discussion. A summary of the conclusions reached follows.
delete-redirect
to page movers. Terasail[✉] 14:05, 22 March 2021 (UTC) WP:NAC
Should the proposal at Wikipedia:Page mover/delete-redirect be adopted? --DannyS712 (talk) 23:25, 18 February 2021 (UTC)
Survey
[edit]- Support as proposer. Page movers can be trusted with this new right, and it would be helpful to have. --DannyS712 (talk) 23:26, 18 February 2021 (UTC)
- Support assuming it really does just make things more convenient, without adding any new "powers." davidwr/(talk)/(contribs) 23:48, 18 February 2021 (UTC)
- Support The right would only slightly expand on the abilities already granted to autoconfirmed users, and is on par with the suppress-redirect right already given out with page mover. The right would substantially lower the work load for page movers by eliminating the need for round robin moves in many cases. That process is complicated and error prone; if a page mover forgets to suppress a redirect anywhere in that process then they may need to start the whole thing over again. Not only will this eliminate that strategy for a whole class of page moves, it also allows page movers to fix mistakes (like not suppressing a redirect) they make in the course of more complicated page moves. Low risk, but high reward. Unconditional support. — Wug·a·po·des 23:54, 18 February 2021 (UTC)
- Support per above. * Pppery * it has begun... 00:00, 19 February 2021 (UTC)
- Support would be helpful and would lead to slightly cleaner and easier to understand page histories imo. Also what Wug said. ProcrastinatingReader (talk) 00:54, 19 February 2021 (UTC)
- Support For something that could be done without it, just in a more cumbersome round robin approach, I see only upsides for adding this to page movers. PaleAqua (talk) 02:56, 19 February 2021 (UTC)
- Support. After having read the details I don't see that this is at all likely to lead to problems or abuse. Thryduulf (talk) 14:09, 19 February 2021 (UTC)
- Support. I think I'll need to give one of my socks the page-mover right to see how it works. I think this is a first; a right that some non-admins will have, that admins won't. – wbm1058 (talk) 16:15, 19 February 2021 (UTC)
- without getting in to very esoteric permissions, some common permissions that non-admins have that admins do not are:
abusefilter-modify, bot, torunblocked
— xaosflux Talk 16:22, 19 February 2021 (UTC)
- without getting in to very esoteric permissions, some common permissions that non-admins have that admins do not are:
- Support. Seems like a reasonable right for pagemovers to have. —pythoncoder (talk | contribs) 20:06, 19 February 2021 (UTC)
- Neutral (if only to avoid a whiteout). All my pagemoves have been WP:ROBIN swaps, and I've never seen the need. On the other hand - you have to make a solid case to be granted the privilege, and I can't see any responsible editor who's jumped through that hoop and not been kicked out abusing this extra function. Narky Blert (talk) 01:13, 21 February 2021 (UTC)
- Support: Not much more than the regular
move
right; page movers can be trusted with this additional ability without too much additional potential for abuse. Twassman | Talk | Contribs 01:49, 27 February 2021 (UTC) - Support per Wugapodes. – SD0001 (talk) 17:20, 1 March 2021 (UTC)
- Support - seems like a pretty easy call here. Slightly reduces strain on administrative backlog, and moves things along faster. Users with the "page mover" right are already trusted to know how this works. Eagles 24/7 (C) 19:22, 1 March 2021 (UTC)
- Support for page movers. In the phabricator discussion at phab:T239277, granting it to autoconfirmed users was mentioned as an idea; I'd like to note that I oppose this idea as it allows any autoconfirmed user to overwrite redirects with vandalism pages, leaving damage behind that is cumbersome and requires sysop tools to repair. ~ ToBeFree (talk) 19:24, 1 March 2021 (UTC)
- Support since a redirect with no history isn't a problem but I'd log where the target was such as if someone moved "Apple" to "Orange" and "Orange" redirected to "Banana" you would see "Deleted redirect to "Banana"". Crouch, Swale (talk) 19:26, 1 March 2021 (UTC)
- support per SD0001. But to be honest, I like, and enjoy performing round robin moves. Most of the moves I've performed are round robins. —usernamekiran (talk) 19:38, 1 March 2021 (UTC)
- Support essentially per Wugapodes: seems like a reasonable slight expansion of the existing ability to overwrite redirects that autoconfirmed users already possess. Mz7 (talk) 19:42, 1 March 2021 (UTC)
- Support for pagemovers per ToBeFree. This is anyway already nearly de-facto possible for page-movers via the process of WP:ROBIN (an admittedly interesting process), but yes there are some cases were this might further simplify things, and if we trust the users then simplifying things is not a bad idea. RandomCanadian (talk / contribs) 21:03, 1 March 2021 (UTC)
- Support. I was rather taken aback when I saw the proposal, but having read the details, it seems fine. The piece that convinced me was that this only applies to single-revision redirects, and so opportunity for abuse is minimal. Vanamonde (Talk) 22:00, 1 March 2021 (UTC)
- Support - If they can be trusted to move pages, they can be trusted to do this singular delete. Dennis Brown - 2¢ 22:11, 1 March 2021 (UTC)
- Support - I thought pagemovers could already do this. It's sensible to allow it; it's nonsense that they currently can't. Ivanvector's squirrel (trees/nuts) 22:47, 1 March 2021 (UTC)
- Support - Seems like a sensible change with very low risk of harm. --Jack Frost (talk) 01:21, 2 March 2021 (UTC)
- Support A no-brainer low risk permission change really. jni(talk)(delete) 06:48, 2 March 2021 (UTC)
- Support has a good risk-reward ratio.
SSSB (talk) 09:53, 2 March 2021 (UTC) - Support This could sometimes be a time saver at Project Merge. GenQuest "scribble" 15:16, 2 March 2021 (UTC)
- Support Don't see how this could harm anything. Aervanath (talk) 21:57, 2 March 2021 (UTC)
- Support, mostly harmless. Obviously useful. Will save quite a few requests for housekeeping speedy deletions. - Nabla (talk) 18:34, 3 March 2021 (UTC)
- Support. It seems like the useful thing to do.Sea Ane (talk) 21:16, 3 March 2021 (UTC)
- Support - This seems useful and low risk. It's only a small expansion of the permissions of this group of users that already has a level of community trust. - tucoxn\talk 22:20, 3 March 2021 (UTC)
- Support still only single rev redirects? Not seeing a huge potential for abuse here. Elliot321 (talk | contribs) 08:23, 4 March 2021 (UTC)
- Support several page movers have stated it would be useful Asartea Talk | Contribs 12:49, 4 March 2021 (UTC)
- Support it is sensible that page movers should have this ability. Dreamy Jazz talk to me | my contributions 13:41, 4 March 2021 (UTC)
- Support Seems good. Since page movers are highly trusted editors, they should get this new right. —Nnadigoodluck███ 19:05, 4 March 2021 (UTC)
- Support: an uncontroversial technical improvement with no real behavioural considerations to factor in given the level of trust page movers are already given. — Bilorv (talk) 01:18, 8 March 2021 (UTC)
- Support. This would save a lot of time and free up admins for more important/higher priority tasks. -- Ϫ 04:10, 12 March 2021 (UTC)
- Support, though it's admittedly obvious what way the wind is blowing. This is a pretty trivial addition overall, and I say this as someone who's been vocal about the dangers of unbundling deletion even in part. Vaticidalprophet (talk) 12:20, 14 March 2021 (UTC)
- Support as this will reduce my need to send pages to G6 for page moves of broadcast stations. Every month, I have at least one G6 need while implementing the latest FCC Media Bureau Call Sign Actions list. Sammi Brie (she/her • t • c) 07:20, 18 March 2021 (UTC)
Discussion
[edit]- @DannyS712: does this permission exist in mediawiki? I don't see it assigned to anyone on any WMF project, could you point to the documentation on this? — xaosflux Talk 23:51, 18 February 2021 (UTC)
- @Xaosflux yes, it exists, but it hasn't been assigned to any group yet. See phab:T239277 for the history. If you want to try it out on the beta cluster I can set that up DannyS712 (talk) 23:52, 18 February 2021 (UTC)
- @Xaosflux: see #Log event actions above. It's documented, albeit cryptically, in the system-generated documentation at mw:API:Logevents as leaction
delete/delete_redir2
. mw:Manual:Log actions is still outdated. mw:Manual:MovePage.php isn't much of a manual; the class reference looks complicated. – wbm1058 (talk) 16:15, 19 February 2021 (UTC)
- mw:Manual:User rights#List of permissions doesn't list
delete-redirect
. – wbm1058 (talk) 17:18, 19 February 2021 (UTC)- nor does it list
tb-override
, norabusefilter-modify
, nortorunblocked
... wbm1058 (talk) 17:46, 19 February 2021 (UTC)- I've added it. Those other ones are from extensions DannyS712 (talk) 17:54, 19 February 2021 (UTC)
- nor does it list
- Up until now, only trusted users with experience who have shown a need get this perm. Even if the requirements arent decreased on paper, I just hope only competent, long term, and trustworthy editors will get this flag in the future. I mean, the mentality of granting admins shouldnt change over time. —usernamekiran (talk) 19:44, 1 March 2021 (UTC)
- I would imagine scrutiny of editors that get the pm bit in the future will be somewhat higher given the extra abilities, but not extraordinarily so. Any time you give someone the ability to delete any page, you would expect care in the selection. Dennis Brown - 2¢ 11:48, 8 March 2021 (UTC)
- This is a good idea, but when it's adopted, it will need to be very carefully advertised to all existing page-movers. At the very least, it makes certain mistakes possible: currently, if you mistakenly try to move over a redirect to a different target, you'll get an error message, but with the new userright, an unaware page-mover may not notice the mistake. Any announcement will also need to emphasise the need to ensure after a move that the previous target is accessible via a clear navigational path, for example from a hatnote. – Uanfala (talk) 21:07, 11 March 2021 (UTC)
- If I recall, the new behavior is, if the target is a single revision redirect pointing somewhere else, to show an error message that the target exists, but offer the options to delete it, like admins have for any existing target page DannyS712 (talk) 01:31, 12 March 2021 (UTC)
- Point which just occurred to me and that I haven't seen discussed prior: This sounds like it would grant page movers the technical ability to close RfDs as delete. Would that be permitted, or would it be considered a misuse of the tools? Alternately, am I misunderstanding something? Vaticidalprophet (talk) 07:51, 15 March 2021 (UTC)
- The new ability only works as part of a move: Wikipedia:Page mover/delete-redirect. – Uanfala (talk) 10:33, 15 March 2021 (UTC)
- Also any redirect under discussion at RfD should have more than one revision - the redirect, and the addition of the RfD tag DannyS712 (talk) 16:57, 15 March 2021 (UTC)
- I have made a request at Wikipedia:Administrators' noticeboard/Requests for closure for the RFC to be closed, since 30 days have elapsed. If closed with consensus in favor of the change, this will require a configuration change to be deployed - I'm happy to write the code for that if the closer pings me. Thanks, --DannyS712 (talk) 01:01, 22 March 2021 (UTC)
- @DannyS712: I don't think a formal closure is necessary for a discussion with unanimous approval (except for one neutral !vote) since the outcome is obvious. * Pppery * it has begun... 01:04, 22 March 2021 (UTC)
- @Pppery: I wanted to err on the safe side so that I could point to an official close when asking that the system administrators deploy the change DannyS712 (talk) 01:06, 22 March 2021 (UTC)
- @DannyS712: I don't think a formal closure is necessary for a discussion with unanimous approval (except for one neutral !vote) since the outcome is obvious. * Pppery * it has begun... 01:04, 22 March 2021 (UTC)
Implementation
[edit]Filed phab:T278131 to request that the configuration be changed and the right actually granted, once that is deployed I'll update the documentation --DannyS712 (talk) 14:28, 22 March 2021 (UTC)
- Task was resolved, configuration updated, and I've updated the documentation at Wikipedia:Page mover as well DannyS712 (talk) 18:18, 22 March 2021 (UTC)
Congrats to the first four page movers to take advantage of this new right!
- at 18:14, 22 March 2021 DannyS712 deleted redirect User:DannyS712/T278131 by overwriting (G6: Deleted to make way for move)
- at 18:54, 22 March 2021 Ammarpad deleted redirect Melbury Sampford by overwriting (G6: Deleted to make way for move)
- at 20:06, 22 March 2021 Lugnuts deleted redirect Alexander Beckett by overwriting (G6: Deleted to make way for move)
- at 14:34, 24 March 2021 TheAafi deleted redirect Massinissa by overwriting (G6: Deleted to make way for move)
- hmm, an edit conflict with Anthony Appleyard? Anthony restored 397 revisions, apparently including the redirect that was deleted to make way for the move.
- I re-deleted what I believe was the redirect that was deleted to make way for TheAafi's move. Note the previous move:
- Barcelona12345 moved page Masinissa to Massinissa of Numidia: There are quite a few Massinissa's (including other monarchs). He was the founder of the Numidian Kingdom and is known as Massinissa of Numidia.
- ...not only moved to "of Numidia" but also changed the spelling (added an "s").
Lugnuts, please take care to add a hatnote at the top of an article when you establish it as the primary topic by moving over a redirect to the former primary topic, as I did with this edit. – wbm1058 (talk) 17:14, 24 March 2021 (UTC)
- Wbm1058, Interesting. I had no idea of this. This is helpful. Thank y'all. ─ The Aafī (talk) 17:54, 24 March 2021 (UTC)
- I recall seeing the extra button when I made that move and thought it was pretty neat! Thanks. Lugnuts Fire Walk with Me 18:12, 24 March 2021 (UTC)
- this is a comment. —usernamekiran (talk) 18:17, 24 March 2021 (UTC)