Guard condition in `delete_temp_backup()` shouldn't return early

[johnbillion]

In the delete_temp_backup() function there is a guard condition which triggers an early return if the arguments for any of the temp backups are empty (​ref).

This prevents further iteration of the array of temp backups. In theory this prevents the deletion of a valid backup that exists in in a subsequent element in the array.

I think the return false should be replaced with continue.

[amitraj2203]

Patch file

[amitraj2203]

@johnbillion I have submitted the patch for the same.

[jamieblomerus]

During today's bug scrub, it was highlighted that the patch needs testing, and guidance was sought for from people who know more about this. The discussion focused on whether true or false should be returned when deletion is skipped, considering the potential impact on other parts of the code. It was noted that low-level APIs typically return true for successful deletion or when the item to be deleted does not exist.

Suggestions for handling broken backups included using unset(), adding an error, or utilizing wp_trigger_error() behind a WP_DEBUG flag.

We felt like a more extensive discussion is needed.

[nhrrob]

We are very close to 6.6 RC1.
Punting to the next milestone.

[stoyangeorgiev]

Moving this one to a Future Release, since additional discussion is needed in terms of handling this one properly. Also testing would be needed for this one.