#6305 closed defect (bug) (fixed)
bulk delete messages doesn't work anymore
| Reported by: |
|
Owned by: | |
|---|---|---|---|
| Milestone: | 2.2.2 | Priority: | normal |
| Severity: | normal | Version: | |
| Component: | Appearance - Template Pack | Keywords: | |
| Cc: |
Description
I think it's a wrong placed ")" in global.js line 1219:
if ( -1 == jq.inArray( this.id ), Array( 'delete_sentbox_messages', 'delete_inbox_messages' ) ) {
should be
if ( -1 == jq.inArray( this.id, Array( 'delete_sentbox_messages', 'delete_inbox_messages' ) ) ) {
(works for me with this change)
Change History (4)
#2
@
10 years ago
- Component changed from Component - Messaging to Appearance - Template Pack
- Milestone changed from Awaiting Review to 2.2.2
Yay, BP-Default.
#4
@
10 years ago
- Keywords good-first-bug removed
- Resolution set to fixed
- Status changed from new to closed
- Version 2.2.1 deleted
Fixed in https://github.com/buddypress/BP-Default/commit/8089d184bb043f41c8d1c828c679a18b40260525.
The updated theme will be pulled into 2.2.2. Thanks for pointing to the fix, @wdfee!
Note: See
TracTickets for help on using
tickets.
Yes, it's correct (as per your fix) in buddypress.js in bp-legacy.