Opened 17 years ago
Closed 17 years ago
#8135 closed defect (bug) (fixed)
'Delete' on edit pages bottom dropdown doesnt work
Reported by: |
|
Owned by: | |
---|---|---|---|
Milestone: | 2.7 | Priority: | normal |
Severity: | normal | Version: | 2.7 |
Component: | Administration | Keywords: | has-patch |
Focuses: | Cc: |
Description
Attempting to delete a page by selecting 'Delete' on the bottom form fails (On the Edit page listing)
Attachments (1)
Change History (10)
#4
follow-up:
↓ 6
@
17 years ago
- Resolution fixed deleted
- Status changed from closed to reopened
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
#6
in reply to:
↑ 4
@
17 years ago
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.
#7
@
17 years ago
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.
I think we need to check for doaction or doaction2 like edit.php does.