Skip to:
Content

BuddyPress.org

Opened 11 years ago

Closed 11 years ago

#5347 closed defect (bug) (fixed)

Bulk Delete Messages Not Working BP1.91

Reported by: glyndavidson's profile glyndavidson Owned by: djpaul's profile djpaul
Milestone: 2.0 Priority: normal
Severity: normal Version: 1.9.1
Component: Messages Keywords:
Cc:

Description

I've never submitted a trac entry before so not sure what the correct etiquette is. Anyway, here goes:

Using the legacy theme templates, WP3.8, BP1.91, Bulk delete inbox messages doesn't work.

In /bp-themes/bp-default/_inc/global.js, line 1213
and /bp-templates/bp-legacy/js/buddypress.js line 1242

Shouldn’t

if ( -1 == jq.inArray( this.id ), Array( 'delete_sentbox_messages', 'delete_inbox_messages' ) ) {

be

if ( -1 == jq.inArray( this.id, Array( 'delete_sentbox_messages', 'delete_inbox_messages' ) )) {

?

I changed it and now my bulk delete works.

Change History (3)

#1 @glyndavidson
11 years ago

  • Summary changed from Bulk Delete Messages Note Working BP1.91 to Bulk Delete Messages Not Working BP1.91

#2 @DJPaul
11 years ago

  • Milestone changed from Awaiting Review to 2.0

Yep, good spot.

#3 @djpaul
11 years ago

  • Owner set to djpaul
  • Resolution set to fixed
  • Status changed from new to closed

In 7740:

Messages: fix bulk delete javascript on the inbox/sentbox screens.

Fixes #5347, props glyndavidson

Note: See TracTickets for help on using tickets.