'Delete' on edit pages bottom dropdown doesnt work

[DD32]

Attempting to delete a page by selecting 'Delete' on the bottom form fails (On the Edit page listing)

[ryan]

I think we need to check for doaction or doaction2 like edit.php does.

[ryan]

(In [9603]) Fix delete bulk op at end of pages list. Props DD32. fixes #8135

[DD32]

I think we need to check for doaction or doaction2 like edit.php does.

Then that check needs to go higher in the page; Preferably on this line:

if ( isset($_GET['action']) && ( -1 != $_GET['action'] || -1 != $_GET['action2'] ) ) { 

I missed the fact that that line hadnt been set to check for action2 not being set

[azaozz]

(In [9607]) Add check for bulk-edit when deleting posts/pages, see #8135

[azaozz]

Replying to DD32:

Then that check needs to go higher in the page; Preferably on this line:
...
I missed the fact that that line hadnt been set to check for action2 not being set

Both 'action' and 'action2' are selects in the same form, so when one is set, the other should be set too when the form is submitted.

[DD32]

Both 'action' and 'action2' are selects in the same form, so when one is set, the other should be set too when the form is submitted.

Thats true, In which case, My initial patch should've been OK since 'action' is checked for on a line above (line 13), Its pointless checking for the items being set at that point, as it should only enter that code block if either of them are set.

[azaozz]

Yes, but this form has several submit buttons, so selecting 'delete' and clicking the Filter button would delete any selected posts too... Not an expected behaviour.