Plugin Directory Admin: Rejecting a plugin with an existing name causes weird errors

[Ipstenu]

When a plugin is rejected its slug is changed from plugin-name to rejected-plugin-name-rejected

The problem is that people could submit the SAME plugin name again and get the slug plugin-name, which when rejected would be rejected-plugin-name-rejected ... again

This throws an error telling you that the name is already in use. If you go back and refresh the post, there is no more slug to edit and the name is kind of messed up.

In the case of REJECTED plugins, we shouldn't use the same if-name-exists check, and instead allow appending of a -2 as needed.

[Otto42]

This used to work and would add the -2 correctly, I thought. Might be worth looking at the changes to see when this broke.

[SergeyBiryukov]

In 5996:

Plugin Directory: Make sure that Status_Transitions::approved() and ::rejected() only run on status change, not on subsequent edits.

See #3180.

[SergeyBiryukov]

In 5997:

Plugin Directory: When rejecting a plugin, disable the check for existing slugs added in [5727], and let wp_unique_post_slug() do its job.

Fixes #3180.