Skip to:

Opened 12 years ago

Closed 12 years ago

#3993 closed enhancement (fixed)

Supporting Message search

Reported by: sbrajesh's profile sbrajesh Owned by:
Milestone: 1.6 Priority: normal
Severity: trivial Version: 1.5.3
Component: Messages Keywords: has-patch commit
Cc: sbrajesh


Can we support message search for user messages. It will be very beneficial for custom themes loading message via ajax and/or allowing searching(I will submit an enhancement for bp-default later).

Please have a look at the attached patch. The patch is against r5690

Attachments (3)

bp-message-search.diff (6.3 KB) - added by sbrajesh 12 years ago.
bp-message-search-1.diff (5.9 KB) - added by sbrajesh 12 years ago.
bp-messages-search-patch-3.patch (9.5 KB) - added by sbrajesh 12 years ago.

Download all attachments as: .zip

Change History (9)

#1 @sbrajesh
12 years ago

oops, please do not use the bp-message-search.diff, I am adding another patch.

#2 @johnjamesjacoby
12 years ago

  • Keywords commit added
  • Milestone changed from Awaiting Review to 1.6
  • Version set to 1.5.3

Love it. Can you integrate it into bp-default also? For things like this, it's difficult to make API changes without a UI to test them with.

#3 @sbrajesh
12 years ago

thank you jjj :)
going to put a patch for bp default today.

#4 @johnjamesjacoby
12 years ago

Looking forward to it, thanks! Would like to get it committed sooner than later so we can test it through the beta period.

#5 @sbrajesh
12 years ago

Hi jjj,
Thank you for looking into it :)
here is a patch which includes the earlier patch and also does it for bp-default. For now, I have put the search box after notices. we may change the location later. It allows ajax search of messages(we may want to clean the javascript code a little bit more as I have copied and pasted from directory search, we do not need scope/extra I guess).

#6 @johnjamesjacoby
12 years ago

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

(In [5822]) Add ajax search to Private Messages inbox and sentbox. Fixes #3993. Props sbrajesh.

Note: See TracTickets for help on using tickets.