Skip to:
Content

BuddyPress.org

Opened 11 years ago

Closed 11 years ago

Last modified 11 years ago

#6305 closed defect (bug) (fixed)

bulk delete messages doesn't work anymore

Reported by: wdfee Owned by:
Priority: normal Milestone: 2.2.2
Component: Appearance - Template Pack Version:
Severity: normal 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)

#1 @rogercoathup
11 years ago

Yes, it's correct (as per your fix) in buddypress.js in bp-legacy.

#2 @DJPaul
11 years ago

  • Component Component - Messaging → Appearance - Template Pack
  • Milestone Awaiting Review → 2.2.2

Yay, BP-Default.

#3 @DJPaul
11 years ago

  • Keywords good-first-bug added

#4 @DJPaul
11 years ago

  • Keywords good-first-bug removed
  • Resolution → fixed
  • Status new → closed
  • Version 2.2.1

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!

Last edited 11 years ago by DJPaul (previous) (diff)
Note: See TracTickets for help on using tickets.