Saving a post with Quick Edit active on a comment results in the post not being saved

[druesome]

While trying to save a post or page, and having Quick Edit open in the comments section, pressing the Update button results in the post not being saved and redirects to the Edit Posts Screen.

Steps to replicate:

1. Edit any post or page that has at least one comment in the post editor.
2. Make changes to the post content by typing in new text.
3. Scroll down and click Quick Edit on any comment and leave it open.
4. Scroll up and press the Update button.
5. Expected: Post is saved and loads the post editor.

Actual: Post isn’t saved and loads the Edit Posts Screen (wp-admin/edit.php).

[MikeHansenMe]

I was able to reproduce this using the steps described.

[MikeHansenMe]

So after some digging it seems strange but line 395 and 407 use c.after( editRow ) and without these the redirect does not happen. In fact if you pass string in place of editRow the string will show and the redirect will not happen. So that leads me to beleive it may be something with editRow. If anyone else has some ideas from there that would be great.

[polevaultweb]

The issue here is that when the quick edit or comment reply form is open, a comment <form> element is added within the post edit form, which also contains an input[name="action"] overriding the input[name="action"] value on the edit post form.

Patch added to close the comment reply/edit form, which removes the form from the post edit form on submission of the post.

[chriscct7]

With the patch what it should do first is ensure that the comment hasn't been edited, or better confirm that you're going to lose the comment changes if you continue. If I change a comment while quick editing and forget to save it and then try to publish the post I'd lose my changes on the comment.

Once that is corrected, it looks good

[polevaultweb]

@chriscct7 I agree, however, I think what you propose is outside the scope of the ticket.

If you quick edit a comment and then try to reload the page via the browser, any changes are lost and there is no prompt, unlike the post editor.

This patch fixes the update post bug, but adding change detection and prompt functionality for the post comments meta box feels like it should be in its own ticket. What do you think?

[chriscct7]

@polevaultweb Sure I'm good with that.

[chriscct7]

Ticket for comment:4 's proposal is #32818

[obenland]

In 33024:

Close comment quick edit before updating a post.

Fixes a bug where the user gets redirected to the All Posts screen, after
updating a post while quick editing a comment.

Props polevaultweb.
Fixes #29457.

[obenland]

In 33066:

Check if commentReply exists before trying to work with it.

Fixes a bug that caused the save/publish flow to be interrupted by an AYS.

Introduced in [33024].
See #29457.