From 8307210a95430053fa243e66d2529a9dc5ce2bb3 Mon Sep 17 00:00:00 2001 From: Pavan Soma Shekar Date: Thu, 30 Jan 2025 19:14:42 +0000 Subject: [PATCH] Fixes #38160 - Update hosts link to new hosts URL --- .../__mocks__/foremanReact/Root/Context/ForemanContext.js | 1 + .../scenes/ContentViews/__tests__/contentViewPage.test.js | 2 +- webpack/scenes/ContentViews/expansions/DetailsExpansion.js | 6 +++++- 3 files changed, 7 insertions(+), 2 deletions(-) diff --git a/webpack/__mocks__/foremanReact/Root/Context/ForemanContext.js b/webpack/__mocks__/foremanReact/Root/Context/ForemanContext.js index 3ad644d7f80..bff3666e3dd 100644 --- a/webpack/__mocks__/foremanReact/Root/Context/ForemanContext.js +++ b/webpack/__mocks__/foremanReact/Root/Context/ForemanContext.js @@ -1,3 +1,4 @@ // eslint-disable-next-line import/prefer-default-export export const useForemanSettings = () => ({ perPage: 20 }); export const useForemanVersion = () => 'nightly'; +export const useForemanHostsPageUrl = () => '/new/hosts'; diff --git a/webpack/scenes/ContentViews/__tests__/contentViewPage.test.js b/webpack/scenes/ContentViews/__tests__/contentViewPage.test.js index cc6acf9df04..d7c83bd9bca 100644 --- a/webpack/scenes/ContentViews/__tests__/contentViewPage.test.js +++ b/webpack/scenes/ContentViews/__tests__/contentViewPage.test.js @@ -132,7 +132,7 @@ test('Can expand cv and show activation keys and hosts', async (done) => { expect(queryByLabelText('activation_keys_link_2').textContent).toEqual('1'); // Displays hosts link with count - expect(queryByLabelText('host_link_2')).toHaveAttribute('href', '/hosts?search=content_view_id+%3D+2'); + expect(queryByLabelText('host_link_2')).toHaveAttribute('href', '/new/hosts?search=content_view_id%3D2'); expect(queryByLabelText('host_link_2').textContent).toEqual('1'); }); diff --git a/webpack/scenes/ContentViews/expansions/DetailsExpansion.js b/webpack/scenes/ContentViews/expansions/DetailsExpansion.js index 060bdddb944..7a8595e8c9c 100644 --- a/webpack/scenes/ContentViews/expansions/DetailsExpansion.js +++ b/webpack/scenes/ContentViews/expansions/DetailsExpansion.js @@ -1,6 +1,7 @@ import React from 'react'; import PropTypes from 'prop-types'; import { translate as __ } from 'foremanReact/common/I18n'; +import { useForemanHostsPageUrl } from 'foremanReact/Root/Context/ForemanContext'; import RelatedCompositeContentViewsModal from './RelatedCompositeContentViewsModal'; import RelatedContentViewComponentsModal from './RelatedContentViewComponentsModal'; @@ -10,6 +11,9 @@ const DetailsExpansion = ({ const activationKeyCount = activationKeys.length; const hostCount = hosts.length; + const baseHostsPageUrl = useForemanHostsPageUrl(); + const hostsPageUrl = `${baseHostsPageUrl}?search=${encodeURIComponent(`content_view_id=${cvId}`)}`; + const relatedContentViewModal = () => { if (cvComposite) { return ( @@ -36,7 +40,7 @@ const DetailsExpansion = ({
{__('Activation keys: ')}{activationKeyCount}
- {__('Hosts: ')}{hostCount} + {__('Hosts: ')}{hostCount}
{relatedContentViewModal()}