Skip to:
Content

BuddyPress.org

Opened 8 years ago

Closed 8 years ago

#7809 closed defect (bug) (fixed)

BP Nouveau: Starred box needs more work to be reachable by the Messages UI

Reported by: imath Owned by: imath
Priority: normal Milestone: 3.0
Component: Messages Version:
Severity: normal Keywords: has-patch commit
Cc:

Description

Since [11925] the filter to set the star feature parameters for the Messages loop, is conditionaly loaded only if the bp_current_action() is the star feature slug.

As a result, BP Nouveau now needs to include the bp-messages/screens/starred.php file during the Ajax request.

Attachments (2)

7809.patch​ (709 bytes ) - added by imath 8 years ago.
7809.02.patch​ (1.5 KB ) - added by r-a-y 8 years ago.

Download all attachments as: .zip

Change History (6)

@imath
8 years ago

#1 @r-a-y
8 years ago

Hi @imath, thanks for finding this bug.

I suggest moving the bp_messages_filter_starred_message_threads() function back to bp-messages-star.php. I believe this will fix the problem, but let me know if it doesn't.

@r-a-y
8 years ago

#2 @r-a-y
8 years ago

  • Component Core → Messages

#3 @imath
8 years ago

  • Keywords commit added

Awesome! Thanks a lot for the .2.patch @r-a-y I confirm it's fixing the issue.

#4 @imath
8 years ago

  • Owner set to imath
  • Resolution → fixed
  • Status new → closed

In 12055:

Nouveau: make sure the starred box is reachable by the Messages UI

Props r-a-y

Fixes #7809

Note: See TracTickets for help on using tickets.