Opened 16 years ago
Closed 16 years ago
#9838 closed defect (bug) (fixed)
Don't show "Selected plugins deactivated." notification if all of selected plugins are deactivated before action.
Reported by: |
|
Owned by: |
|
---|---|---|---|
Milestone: | 2.8 | Priority: | normal |
Severity: | minor | Version: | 2.8 |
Component: | UI | Keywords: | needs-patch |
Focuses: | Cc: |
Description (last modified by )
Status completion messages should not be shown if no actions are performed. (Eg, Don't show "Selected plugins deactivated." notification if all of selected plugins are deactivated before action.)
A similar situation happens with plugin activation, as well as Plugin deletion.
There exists an issue where Active plugins can be deleted via the Bulk actions as well.
(Edited by dd32 to merge #9837 and mention similar bugs)
Attachments (5)
Change History (21)
#2
@
16 years ago
- Component changed from Plugins to UI
- Type changed from feature request to defect (bug)
#5
@
16 years ago
- Description modified (diff)
Patch pending #9835 closure due to code conflicts. (I could post a patch, But it'll only make one or the other stale, I'm going to get some feedback on the other ticket first before submitting this patch)
Note, I've also closed #9837 as a duplicate of this and merging the problems, Due to the actual issue being very similar
#7
@
16 years ago
attachment 9838.diff added
- Doesnt attempt to activate/deactivate an already activated/deactivated plugin via bulk action
- Bit of extra white space
- correction of paragraph around the Version and Author name
- Adds Bulk actions dropdown at the bottom of the table, like the other tables.
(Patch is about to be refreshed due to a new-line issue.)
#9
@
16 years ago
attachment 9838.3.diff added
- Same as before, Moves the Delete link to the 3rd position, moving the Visit plugin homepage back to the last item
- Some more spacing issues here and there.
- Also changed the $sep in the actions link code to use end() instead of a counter.. Its a bit more expensive timewise, but looks cleaner, and is only *just* a little slower..
#10
@
16 years ago
- Priority changed from low to normal
attachment 9838.4.diff added
- Final unrequested patch refresh.
- Doesnt have the Actions changes from the previous patches since its now no longer needed.
- Else, Everything else from the original patch is there. So..
- Doesnt attempt to activate/deactivate an already activated/deactivated plugin via bulk action
- Doesnt show completed messages when nothing was completed
- Adds the Actions to the bottom of the table as well.
#11
@
16 years ago
- Keywords needs-patch added; has-patch removed
Just noticed something else i missed in this patch.. I'll make a new one up this evening.
Also closed #9941 as a duplicate of this, It'll be corrected thanks to the array filtering in this ticket.
#14
@
16 years ago
- Keywords needs-patch added; has-patch removed
- Resolution fixed deleted
- Status changed from closed to reopened
Maybe some fixes were made, but problem mentioned in ticket is still not fixed.
#15
follow-up:
↓ 16
@
16 years ago
Status completion messages should not be shown if no actions are performed. (Eg, Don't show "Selected plugins deactivated." notification if all of selected plugins are deactivated before action.)
Seems fixed. Select active plugins, Activate = No message. Select inactive plugins, deactivate - no message. Select Active+inactive plugin, deactivate = message that selected plugins are deactivated (ok, semi true, not all selected are..)
A similar situation happens with plugin activation, as well as Plugin deletion.
Done too.
There exists an issue where Active plugins can be deleted via the Bulk actions as well.
Active plugins upon being deleted, are not shown in the list of "plugins to delete".
What problems are not fixed, And if its the first one here, What'd you expect to happen?
I'll take a look at this tomorrow