Discource-C
/
discourse
mirror of https://github.com/discourse/discourse.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

FIX: Make category-drop search subcategories (#25817) 

Subcategories were not returned when lazy loaded categories are enabled.
This commit implements the old behavior which displayed only top level
categories when there was no search term, but when there was one, it
searched subcategories too.

This commit also removes the client-side ordering of categories, because
the results are already ordered on the server-side.
This commit is contained in:
Bianca Nenciu 2024-02-22 21:35:35 +02:00 committed by GitHub
parent 57ab42d4ca
commit 35adf6046e
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
1 changed files with 19 additions and 15 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

34
app/assets/javascripts/select-kit/addon/components/category-drop.js
View File

@ -139,22 +139,26 @@ export default ComboBoxComponent.extend({
async search(filter) {
if (this.site.lazy_load_categories) {
const results = await Category.asyncSearch(filter, {
parentCategoryId: this.options.parentCategory?.id || -1,
includeUncategorized: this.siteSettings.allow_uncategorized_topics,
limit: 15,
});
return this.shortcuts.concat(
results.sort((a, b) => {
if (a.parent_category_id && !b.parent_category_id) {
return 1;
} else if (!a.parent_category_id && b.parent_category_id) {
return -1;
} else {
return 0;
}
let parentCategoryId;
if (this.options.parentCategory?.id) {
parentCategoryId = this.options.parentCategory.id;
} else if (!filter) {
// Only top-level categories should be displayed by default.
// If there is a search term, the term can match any category,
// including subcategories.
parentCategoryId = -1;
}
const results = (
await Category.asyncSearch(filter, {
parentCategoryId,
includeUncategorized: this.siteSettings.allow_uncategorized_topics,
includeAncestors: true,
limit: 15,
})
);
).categories;
return this.shortcuts.concat(results);
}
const opts = {