Opened 6 years ago
Closed 6 years ago
#3254 closed defect (bug) (fixed)
Add forum search (and improve search keys as array handling)
Reported by: | johnjamesjacoby | Owned by: | johnjamesjacoby |
---|---|---|---|
Milestone: | 2.6 | Priority: | high |
Severity: | normal | Version: | |
Component: | Component - Search | Keywords: | |
Cc: |
Description
Topic & Reply searching is currently supported, but sites with many forums do not have a specific way to search for existing forums.
Let's add support for this.
While we're at it, let's also better handle query arguments that are in the shape of an array rather than a string, as there have been a few notices hitting the Dotorg logs.
Attachments (2)
Change History (3)
#1
@
6 years ago
- Resolution set to fixed
- Status changed from assigned to closed
In r6903:
Search: add support for fs
query-arg to search forums list:
- Introduces
bbp_sanitize_search_request()
to encapsulate duplicate code across forums/topics/replies - Introduces
bbp_get_search_type_ids()
to stub out future enhancements (tags, users, etc...) - Use these new functions where intended
- Update
bbp_get_search_terms()
to loop through known search-type IDs
This commit also fixes debug notices that would happen when these query arguments were not explicitly strings.
Fixes #3254.
Note: See
TracTickets for help on using
tickets.
Searching a forum listing for forums with "gen" in them