github-mirror/discourse
2
0
Fork 0
mirror of https://github.com/discourse/discourse.git synced 2024-11-26 04:56:18 +08:00
Code Issues Actions 6 Packages Projects Releases Wiki Activity

DEV: Remove findBy usage in admin-sidebar (#27813)

Browse Source 
We prefer not to use these Ember array extensions anymore
This commit is contained in:
Martin Brennan 2024-07-10 11:58:52 +10:00 committed by GitHub
parent eca4ed8f80
commit 54fba2005f
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
1 changed files with 22 additions and 14 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
app/assets/javascripts/discourse/app/lib/sidebar/admin-sidebar.js
View File

@ -184,7 +184,9 @@ export function useAdminNavConfig(navMap) {
for (const [sectionName, additionalLinks] of Object.entries(
additionalAdminSidebarSectionLinks
)) {
const section = navMap.findBy("name", sectionName);
const section = navMap.find(
(navSection) => navSection.name === sectionName
);
if (section && additionalLinks.length) {
section.links.push(...additionalLinks);
}
@ -286,7 +288,9 @@ export default class AdminSidebarPanel extends BaseCustomSidebarPanel {
const navMap = savedConfig || ADMIN_NAV_MAP;
if (!session.get("safe_mode")) {
const pluginLinks = navMap.findBy("name", "plugins").links;
const pluginLinks = navMap.find(
(section) => section.name === "plugins"
).links;
pluginAdminRouteLinks().forEach((pluginLink) => {
if (!pluginLinks.mapBy("name").includes(pluginLink.name)) {
pluginLinks.push(pluginLink);
@ -311,7 +315,9 @@ export default class AdminSidebarPanel extends BaseCustomSidebarPanel {
});
if (siteSettings.experimental_form_templates) {
navMap.findBy("name", "appearance").links.push({
navMap
.find((section) => section.name === "appearance")
.links.push({
name: "admin_customize_form_templates",
route: "adminCustomizeFormTemplates",
label: "admin.form_templates.nav_title",
@ -320,7 +326,9 @@ export default class AdminSidebarPanel extends BaseCustomSidebarPanel {
}
if (currentUser.show_experimental_flags_admin_page) {
navMap.findBy("name", "community").links.push({
navMap
.find((section) => section.name === "community")
.links.push({
name: "admin_moderation_flags",
route: "adminConfig.flags",
label: "admin.community.sidebar_link.moderation_flags",