From c2f549fd4f38db3a26834f06f456bad400a4dd66 Mon Sep 17 00:00:00 2001 From: Jarek Radosz Date: Thu, 31 Oct 2024 10:56:06 +0100 Subject: [PATCH] DEV: Use the new `owner` arg in `shouldRender` (#314) --- .../bread-crumbs-right/solved-status-filter.js | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/assets/javascripts/discourse/connectors/bread-crumbs-right/solved-status-filter.js b/assets/javascripts/discourse/connectors/bread-crumbs-right/solved-status-filter.js index 0423cf8..cb6e6e1 100644 --- a/assets/javascripts/discourse/connectors/bread-crumbs-right/solved-status-filter.js +++ b/assets/javascripts/discourse/connectors/bread-crumbs-right/solved-status-filter.js @@ -1,7 +1,6 @@ import Component from "@glimmer/component"; import { action } from "@ember/object"; import { inject as service } from "@ember/service"; -import { getOwnerWithFallback } from "discourse-common/lib/get-owner"; import I18n from "I18n"; const QUERY_PARAM_VALUES = { @@ -16,17 +15,17 @@ const UX_VALUES = { }; export default class SolvedStatusFilter extends Component { - static shouldRender(args, helper) { - const router = getOwnerWithFallback(this).lookup("service:router"); + static shouldRender(args, context, owner) { + const router = owner.lookup("service:router"); if ( - !helper.siteSettings.show_filter_by_solved_status || + !context.siteSettings.show_filter_by_solved_status || router.currentRouteName === "discovery.categories" || args.editingCategory ) { return false; } else if ( - helper.siteSettings.allow_solved_on_all_topics || + context.siteSettings.allow_solved_on_all_topics || router.currentRouteName === "tag.show" ) { return true;