• This site uses cookies. By continuing to use this site, you are agreeing to our use of cookies. Learn more.

Lack of interest Remove ability to edit posts when merging, fixing hard-deleting content issue

Brogan

XenForo moderator
Staff member
#2
That would require two steps when merging posts which do require editing.

Why not just automatically apply a post edit history to the destination post as part of the merge function?
 

Divvens

Well-known member
#3
If a moderator abuses their power in that manner - to do something they aren't allowed to, then they shouldn't be a moderator in the first place.

I think editing while merging is very essential and should stay the way it is.
 
#5
Context:
The current behaviour of post merging implies the deletion of one or more posts from the database; only one of the merged posts (the one selected in the "merge into post" drop down) will remain in the database after the merge is completed.
Merging posts requires only the "Manage (move, merge, etc.) thread" general moderator permission.​

This behaviour fails to meet two expectations of the current permission model when a user only has the "Manage (move, merge, etc.) thread" permission:
  1. The user, without the "Edit post" permission, can edit a post using the merge function.
  2. The user, without the "Hard-delete post" permission, can delete a post from the database using the merge function.
As there is no way of "unmerging" posts this behaviour also makes it very difficult to recover from a merge by mistake, requiring either manual changes in the database and/or a partial restore of a backup, if—hopefully—there is one.


My suggestion is to change the behaviour of the post merge action to soft-delete the original posts, copying the content into a new post with the merged text. An alternative suggestion is to keep the original post data in some form in the database (perhaps a new message state?) and provide an action to revert a merge automatically.
A second, related, suggestion is to disallow editing when merging posts if the user doesn't have the adequate permission ("Edit post").
 
#7
Well, now I feel silly. I did not notice that thread when I searched this forum earlier, and it is quite recent too.
In a way I think my suggestion complements that one, but indeed the main issue had been reported there already.

May I suggest a merge with the thread you pointed out, @Brogan? ;)