Skip to:
Content

Opened 6 years ago

Last modified 6 weeks ago

#3889 new enhancement

When searching on the activity page, you cannot search using an authors name.

Reported by: fanquake Owned by:
Milestone: Future Release Priority: normal
Severity: minor Version:
Component: Activity Keywords: has-patch 2nd-opinion needs-testing needs-unit-tests
Cc:

Description

When using the search form on the Activity page, if you search for the name of the author of an activity item, nothing is returned. This functionality should included, then I think that it should be, as then you would be able to search for repeat offenders by name etc.

Attachments (1)

3889-user-searching.diff (1.1 KB) - added by tw2113 7 weeks ago.

Download all attachments as: .zip

Change History (11)

#1 in reply to: ↑ description @fanquake
6 years ago

Replying to fanquake:

When using the search form on the Activity page, if you search for the name of the author of an activity item, nothing is returned. This functionality should be included, as then you would be able to search for repeat offenders by name etc.

#2 @DJPaul
6 years ago

Admin or front end?

#3 @fanquake
6 years ago

Admin

#4 @DJPaul
6 years ago

  • Milestone changed from Awaiting Review to Future Release
  • Type changed from defect (bug) to enhancement

Right now we use the activity loop search parameter, which doesn't search on username. Will have to take a look at how best to do this in a future release.

#5 @boonebgorges
4 years ago

  • Keywords needs-patch added
  • Severity changed from normal to minor

#6 @tw2113
7 weeks ago

Big conflict with adding username searching is that BuddyPress doesn't store user information as username. It's by user ID.

Perhaps, we could do a boolean based filter to conditionally search users. Basically, if the filter returns true, we do a user lookup by username. If we find a valid user, then we add the sql clause for also searching for the ID column based on the found user's ID.

See diff attached.

#7 @tw2113
7 weeks ago

  • Keywords has-patch 2nd-opinion needs-testing added; needs-patch removed

#8 @tw2113
7 weeks ago

  • Milestone changed from Future Release to 2.9

This ticket was mentioned in Slack in #buddypress by hnla. View the logs.


6 weeks ago

#10 @tw2113
6 weeks ago

  • Keywords needs-unit-tests added
  • Milestone changed from 2.9 to Future Release
Note: See TracTickets for help on using tickets.