katello 4.3.0 → 4.3.1

data/webpack/components/extensions/HostDetails/Tabs/RemoteExecutionConstants.js

@@ -2,5 +2,5 @@ export const REX_JOB_INVOCATIONS_KEY = 'REX_JOB_INVOCATIONS';
 export const REX_FEATURES = {
   KATELLO_PACKAGE_INSTALL: 'katello_package_install',
   KATELLO_HOST_TRACER_RESOLVE: 'katello_host_tracer_resolve',
-  KATELLO_HOST_ERRATA_INSTALL: 'katello_errata_install',
+  KATELLO_HOST_ERRATA_INSTALL_BY_SEARCH: 'katello_errata_install_by_search',
 };

data/webpack/components/extensions/HostDetails/Tabs/__tests__/errataTab.test.js

@@ -981,9 +981,9 @@ test('Can bulk apply via remote execution', async (done) => {
   // eslint-disable-next-line camelcase
   const jobInvocationBody = ({ job_invocation: { inputs, feature, search_query } }) =>
     inputs[ERRATA_SEARCH_QUERY] === `errata_id ^ (${results[0].errata_id},${results[1].errata_id})` &&
-     feature === REX_FEATURES.KATELLO_HOST_ERRATA_INSTALL &&
-     // eslint-disable-next-line camelcase
-     search_query === `name ^ (${hostName})`;
+    feature === REX_FEATURES.KATELLO_HOST_ERRATA_INSTALL_BY_SEARCH &&
+    // eslint-disable-next-line camelcase
+    search_query === `name ^ (${hostName})`;
 
   const resolveErrataScope = nockInstance
     .post(jobInvocations, jobInvocationBody)
@@ -1076,7 +1076,7 @@ test('Can apply errata in bulk via customized remote execution', async (done) =>
   getByLabelText('Select row 0').click();
   getByLabelText('Select row 1').click();
   const errata = `${results[0].errata_id},${results[1].errata_id}`;
-  const feature = REX_FEATURES.KATELLO_HOST_ERRATA_INSTALL;
+  const feature = REX_FEATURES.KATELLO_HOST_ERRATA_INSTALL_BY_SEARCH;
   const actionMenu = getByLabelText('bulk_actions');
   actionMenu.click();
   const viaRexAction = queryByText('Apply via customized remote execution');
@@ -1176,7 +1176,7 @@ test('Can apply a single erratum to the host via customized remote execution', a
   const mockErrata = makeMockErrata({});
   const { results } = mockErrata;
   const { errata_id: errataId } = results[0];
-  const feature = REX_FEATURES.KATELLO_HOST_ERRATA_INSTALL;
+  const feature = REX_FEATURES.KATELLO_HOST_ERRATA_INSTALL_BY_SEARCH;
   const scope = nockInstance
     .get(hostErrata)
     .query(defaultQuery)

data/webpack/components/extensions/HostDetails/Tabs/customizedRexUrlHelpers.js

@@ -29,6 +29,6 @@ export const resolveTraceUrl = ({ hostname, search }) => createJob({
 
 export const errataInstallUrl = ({ hostname, search }) => createJob({
   hostname,
-  feature: REX_FEATURES.KATELLO_HOST_ERRATA_INSTALL,
+  feature: REX_FEATURES.KATELLO_HOST_ERRATA_INSTALL_BY_SEARCH,
   inputs: { [ERRATA_SEARCH_QUERY]: search },
 });

data/webpack/scenes/Content/ContentConfig.js

@@ -1,5 +1,4 @@
 import React from 'react';
-import { Link } from 'react-router-dom';
 import { urlBuilder } from 'foremanReact/common/urlHelpers';
 import { translate as __ } from 'foremanReact/common/I18n';
 import ContentInfo from './Details/ContentInfo';
@@ -7,7 +6,8 @@ import LastSync from '../ContentViews/Details/Repositories/LastSync';
 import ContentRepositories from './Details/ContentRepositories';
 import ContentCounts from '../ContentViews/Details/Repositories/ContentCounts';
 
-export default () => [
+// Keep in mind when editing this file that the ContentViewVersionDetailConfig consumes this array.
+export default [
   {
     names: {
       pluralTitle: __('Python Packages'),
@@ -18,7 +18,7 @@ export default () => [
       singularLabel: 'python_package',
     },
     columnHeaders: [
-      { title: __('Name'), getProperty: unit => (<Link to={urlBuilder(`content/python_packages/${unit?.id}`, '')}>{unit?.name}</Link>) },
+      { title: __('Name'), getProperty: unit => (<a href={urlBuilder(`content/python_packages/${unit?.id}`, '')}>{unit?.name}</a>) },
       { title: __('Version'), getProperty: unit => unit?.version },
       { title: __('Filename'), getProperty: unit => unit?.filename },
     ],
@@ -82,6 +82,57 @@ export default () => [
       pluralLabel: 'ostree_refs',
       singularLabel: 'ostree_ref',
     },
+    columnHeaders: [
+      { title: __('Name'), getProperty: unit => (<a href={urlBuilder(`content/ostree_refs/${unit?.id}`, '')}>{unit?.name}</a>) },
+      { title: __('Version'), getProperty: unit => unit?.version },
+    ],
+    tabs: [
+      {
+        tabKey: 'details',
+        title: __('Details'),
+        getContent: (contentType, id, tabKey) => <ContentInfo {...{ contentType, id, tabKey }} />,
+        columnHeaders: [
+          { title: __('Name'), getProperty: unit => unit?.name },
+          { title: __('Version'), getProperty: unit => unit?.version },
+        ],
+      },
+      {
+        tabKey: 'repositories',
+        title: __('Repositories'),
+        getContent: (contentType, id, tabKey) =>
+          <ContentRepositories {...{ contentType, id, tabKey }} />,
+        columnHeaders: [
+          {
+            title: __('Name'),
+            getProperty: unit =>
+              <a href={urlBuilder(`products/${unit?.product.id}/repositories/${unit?.id}`, '')}>{unit?.name}</a>,
+          },
+          {
+            title: __('Product'),
+            getProperty: unit =>
+              <a href={urlBuilder(`products/${unit?.product.id}`, '')}>{unit?.product.name}</a>,
+          },
+          {
+            title: __('Sync Status'),
+            getProperty: unit =>
+              (<LastSync
+                startedAt={unit?.started_at}
+                lastSyncWords={unit?.last_sync_words}
+                lastSync={unit?.last_sync}
+              />),
+          },
+          {
+            title: __('Content Count'),
+            getProperty: unit =>
+              (<ContentCounts
+                productId={unit.product.id}
+                repoId={unit.id}
+                counts={unit.content_counts}
+              />),
+          },
+        ],
+      },
+    ],
   },
   {
     names: {
@@ -93,11 +144,10 @@ export default () => [
       singularLabel: 'ansible_collection',
     },
     columnHeaders: [
-      { title: __('Name'), getProperty: unit => (<Link to={urlBuilder(`content/ansible_collections/${unit?.id}`, '')}>{unit?.name}</Link>) },
+      { title: __('Name'), getProperty: unit => (<a href={urlBuilder(`content/ansible_collections/${unit?.id}`, '')}>{unit?.name}</a>) },
       { title: __('Author'), getProperty: unit => unit?.namespace },
       { title: __('Version'), getProperty: unit => unit?.version },
       { title: __('Checksum'), getProperty: unit => unit?.checksum },
-
     ],
     tabs: [
       {

data/webpack/scenes/Content/ContentPage.js

@@ -27,7 +27,7 @@ const ContentPage = () => {
       const types = {};
       contentTypesResponse.forEach((type) => {
         if (type.generic_browser) {
-          const typeConfig = ContentConfig().find(config =>
+          const typeConfig = ContentConfig.find(config =>
             config.names.singularLabel === type.label);
           if (typeConfig) {
             const { names } = typeConfig;

data/webpack/scenes/Content/Details/ContentDetails.js

@@ -16,7 +16,7 @@ const ContentDetails = () => {
 
   const { id, content_type: contentType } = useParams();
   const contentId = Number(id);
-  const config = ContentConfig().find(type =>
+  const config = ContentConfig.find(type =>
     type.names.pluralLabel === contentType);
   const { pluralTitle, pluralLabel } = config.names;
 

data/webpack/scenes/Content/Details/ContentInfo.js

@@ -19,7 +19,7 @@ const ContentInfo = ({ contentType, id, tabKey }) => {
   const detailsResponse = useSelector(selectContentDetails);
   const detailsStatus = useSelector(selectContentDetailsStatus);
 
-  const config = contentConfig().find(type => type.names.pluralLabel === contentType);
+  const config = contentConfig.find(type => type.names.pluralLabel === contentType);
   const { columnHeaders } = config.tabs.find(header => header.tabKey === tabKey);
 
   useEffect(() => {

data/webpack/scenes/Content/Details/ContentRepositories.js

@@ -20,7 +20,7 @@ const ContentRepositories = ({ contentType, id, tabKey }) => {
   const [searchQuery, updateSearchQuery] = useState('');
   const { results, ...metadata } = response;
 
-  const config = contentConfig().find(type => type.names.pluralLabel === contentType);
+  const config = contentConfig.find(type => type.names.pluralLabel === contentType);
   const typeSingularLabel = config.names.singularLabel;
   const { columnHeaders } = config.tabs.find(header => header.tabKey === tabKey);
 

data/webpack/scenes/Content/Table/ContentTable.js

@@ -18,7 +18,7 @@ const ContentTable = ({
   const [searchQuery, updateSearchQuery] = useState('');
   const { results, ...metadata } = response;
 
-  const { columnHeaders } = contentConfig().find(type =>
+  const { columnHeaders } = contentConfig.find(type =>
     type.names.singularLabel === contentTypes[selectedContentType][0]);
 
   return (

data/webpack/scenes/ContentViews/ContentViewsConstants.js

@@ -1,4 +1,5 @@
 import { translate as __ } from 'foremanReact/common/I18n';
+import { toUpper } from 'lodash';
 
 const CONTENT_VIEWS_KEY = 'CONTENT_VIEWS';
 export const CREATE_CONTENT_VIEW_KEY = 'CONTENT_VIEW_CREATE';
@@ -22,11 +23,11 @@ export const RPM_PACKAGE_GROUPS_CONTENT = 'RPM_PACKAGE_GROUPS_CONTENT';
 export const REPOSITORY_CONTENT = 'REPOSITORY_CONTENT';
 export const ERRATA_CONTENT = 'ERRATA_CONTENT';
 export const DOCKER_TAGS_CONTENT = 'DOCKER_TAGS_CONTENT';
-export const ANSIBLE_COLLECTIONS_CONTENT = 'ANSIBLE_COLLECTIONS_CONTENT';
 export const MODULE_STREAMS_CONTENT = 'MODULE_STREAMS_CONTENT';
 export const DEB_PACKAGES_CONTENT = 'DEB_PACKAGES_CONTENT';
 export const RPM_PACKAGES_CONTENT = 'RPM_PACKAGES_CONTENT';
 export const FILE_CONTENT = 'FILE_CONTENT';
+export const generatedContentKey = pluralLabel => `${toUpper(pluralLabel)}_CONTENT`;
 export const cvDetailsKey = cvId => `${CONTENT_VIEWS_KEY}_${cvId}`;
 export const cvDetailsRepoKey = cvId => `${CONTENT_VIEWS_KEY}_REPOSITORIES_${cvId}`;
 export const cvFilterRepoKey = filterId => `CV_FILTER_REPOSITORIES_${filterId}`;

data/webpack/scenes/ContentViews/Details/ContentViewDetailActions.js

@@ -1,6 +1,7 @@
 import { API_OPERATIONS, APIActions, get, put, post } from 'foremanReact/redux/API';
 import { addToast } from 'foremanReact/redux/actions/toasts';
 import { translate as __ } from 'foremanReact/common/I18n';
+import { lowerCase } from 'lodash';
 import {
   RPM_PACKAGES_CONTENT,
   RPM_PACKAGE_GROUPS_CONTENT,
@@ -45,8 +46,8 @@ import {
   ERRATA_CONTENT,
   MODULE_STREAMS_CONTENT,
   DEB_PACKAGES_CONTENT,
-  ANSIBLE_COLLECTIONS_CONTENT,
   DOCKER_TAGS_CONTENT,
+  generatedContentKey,
 } from '../ContentViewsConstants';
 import api, { foremanApi, orgId } from '../../../services/api';
 import { getResponseErrorMsgs, apiError } from '../../../utils/helpers';
@@ -70,6 +71,14 @@ const cvUpdateSuccess = (response, dispatch) => {
   }));
 };
 
+export const getContent = (pluralLabel, params) => get({
+  type: API_OPERATIONS.GET,
+  key: generatedContentKey(pluralLabel),
+  url: api.getApiUrl(`/${pluralLabel}`),
+  params,
+  errorToast: error => __(`Something went wrong while fetching ${lowerCase(pluralLabel)}! ${getResponseErrorMsgs(error.response)}`),
+});
+
 export const getRPMPackages = params => get({
   type: API_OPERATIONS.GET,
   key: RPM_PACKAGES_CONTENT,
@@ -135,14 +144,6 @@ export const getDockerTags = params => get({
   errorToast: error => __(`Something went wrong while getting docker tags! ${getResponseErrorMsgs(error.response)}`),
 });
 
-export const getAnsibleCollections = params => get({
-  type: API_OPERATIONS.GET,
-  key: ANSIBLE_COLLECTIONS_CONTENT,
-  url: api.getApiUrl('/ansible_collections'),
-  params,
-  errorToast: error => __(`Something went wrong while getting ansible collections! ${getResponseErrorMsgs(error.response)}`),
-});
-
 export const getErrata = params => get({
   type: API_OPERATIONS.GET,
   key: ERRATA_CONTENT,
@@ -176,7 +177,7 @@ export const getRepositories = params => get({
 });
 
 export const getContentViewRepositories = (cvId, params, status) => {
-  const apiParams = { ...params };
+  const apiParams = { organization_id: orgId(), ...params };
   let apiUrl = `/content_views/${cvId}/repositories`;
 
   if (status === ALL_STATUSES) {

data/webpack/scenes/ContentViews/Details/ContentViewDetailSelectors.js

@@ -33,8 +33,8 @@ import {
   ERRATA_CONTENT,
   MODULE_STREAMS_CONTENT,
   DEB_PACKAGES_CONTENT,
-  ANSIBLE_COLLECTIONS_CONTENT,
   DOCKER_TAGS_CONTENT,
+  generatedContentKey,
 } from '../ContentViewsConstants';
 import { pollTaskKey } from '../../Tasks/helpers';
 
@@ -131,11 +131,11 @@ export const selectCVFilterRules = (state, filterId) =>
 export const selectCVFilterRulesStatus = (state, filterId) =>
   selectAPIStatus(state, cvFilterRulesKey(filterId)) || STATUS.PENDING;
 
-export const selectAnsibleCollections = state =>
-  selectAPIResponse(state, ANSIBLE_COLLECTIONS_CONTENT);
+export const selectContent = (pluralLabel, state) =>
+  selectAPIResponse(state, generatedContentKey(pluralLabel));
 
-export const selectAnsibleCollectionsStatus = state =>
-  selectAPIStatus(state, ANSIBLE_COLLECTIONS_CONTENT) || STATUS.PENDING;
+export const selectContentStatus = (pluralLabel, state) =>
+  selectAPIStatus(state, generatedContentKey(pluralLabel)) || STATUS.PENDING;
 
 export const selectDockerTags = state =>
   selectAPIResponse(state, DOCKER_TAGS_CONTENT);

data/webpack/scenes/ContentViews/Details/Repositories/ContentCounts.js

@@ -41,7 +41,7 @@ const ContentCounts = ({ productId, repoId, counts }) => {
   Object.keys(counts).forEach((type) => {
     const count = counts[type];
     let info = repoLabels[type];
-    const config = ContentConfig().find(typeConfig =>
+    const config = ContentConfig.find(typeConfig =>
       typeConfig.names.singularLabel === type);
 
     if (config) {

data/webpack/scenes/ContentViews/Details/Versions/ContentViewVersionContent.js

@@ -1,5 +1,6 @@
 import React from 'react';
 import PropTypes from 'prop-types';
+import { camelCase } from 'lodash';
 import { Link } from 'react-router-dom';
 import { translate as __ } from 'foremanReact/common/I18n';
 import { urlBuilder } from 'foremanReact/common/urlHelpers';
@@ -17,15 +18,16 @@ const ContentViewVersionContent = ({ cvId, versionId, cvVersion }) => {
   } = cvVersion;
 
 
-  const genericContentTypes = ContentConfig().filter(({ names: { singularLabel } }) => {
-    // Ansible Collections has a tab in version details so it's handled separately below.
-    if (singularLabel === 'ansible_collection') return false;
-    const countLabel = `${singularLabel}_count`;
-    return !!cvVersion[countLabel];
-  }).map(({ names: { singularLabel, singularLowercase, pluralLowercase } }) => {
+  const contentConfigTypes = ContentConfig.filter(({ names: { singularLabel } }) =>
+    !!cvVersion[`${singularLabel}_count`]).map(({
+    names: {
+      singularLabel, singularLowercase, pluralLowercase, pluralLabel,
+    },
+  }) => {
     const countParam = `${singularLabel}_count`;
     const count = cvVersion[countParam];
     return {
+      pluralLabel,
       label: count > 1 ? pluralLowercase : singularLowercase,
       count,
     };
@@ -34,7 +36,7 @@ const ContentViewVersionContent = ({ cvId, versionId, cvVersion }) => {
   const noCounts =
     !Number(debCount) && !Number(dockerManifestCount) && !Number(dockerTagCount) &&
     !Number(fileCount) && !Number(moduleStreamCount) && !Number(ansibleCollectionCount) &&
-    !genericContentTypes?.length;
+    !contentConfigTypes?.length;
 
   if (noCounts) {
     return <InactiveText text={__('No content')} />;
@@ -69,16 +71,13 @@ const ContentViewVersionContent = ({ cvId, versionId, cvVersion }) => {
           <a href={urlBuilder(`content_views/${cvId}#/versions/${versionId}/files`, '')}>{`${fileCount} Files`}</a><br />
         </>
       }
-      {ansibleCollectionCount > 0 &&
-        <>
-          <a href={urlBuilder(`content_views/${cvId}#/versions/${versionId}/ansibleCollections`, '')}>{`${ansibleCollectionCount} Collections`}</a><br />
-        </>
-      }
-      {genericContentTypes?.length > 0 &&
-        genericContentTypes.map(({ label, count }) => (
-          <span key={label} style={{ whiteSpace: 'pre-line' }}>
-            {`${count} ${label}`}
-          </span>))
+      {contentConfigTypes?.length > 0 &&
+        contentConfigTypes.map(({ label, count, pluralLabel }) => (
+          <React.Fragment key={label}>
+            <a href={urlBuilder(`content_views/${cvId}#/versions/${versionId}/${camelCase(pluralLabel)}`, '')}>
+              {`${count} ${label}`}
+            </a><br />
+          </React.Fragment>))
       }
     </>
   );

data/webpack/scenes/ContentViews/Details/Versions/ContentViewVersions.js

@@ -92,7 +92,7 @@ const ContentViewVersions = ({ cvId, details }) => {
       { title: <ContentViewVersionEnvironments {...{ environments }} /> },
       {
         title: Number(packageCount) ?
-          <a href={urlBuilder(`content_views/${cvId}#/versions/${versionId}/packages`, '')}>{packageCount}</a> :
+          <a href={urlBuilder(`content_views/${cvId}#/versions/${versionId}/rpmPackages`, '')}>{packageCount}</a> :
           <InactiveText text={__('No packages')} />,
       },
       { title: <ContentViewVersionErrata {...{ cvId, versionId, errataCounts }} /> },

data/webpack/scenes/ContentViews/Details/Versions/VersionDetails/ContentViewVersionDetailConfig.js

@@ -3,14 +3,13 @@ import PropTypes from 'prop-types';
 import { translate as __ } from 'foremanReact/common/I18n';
 import { urlBuilder } from 'foremanReact/common/urlHelpers';
 import LongDateTime from 'foremanReact/components/common/dates/LongDateTime';
-import { startCase } from 'lodash';
+import { startCase, camelCase } from 'lodash';
 import {
   BugIcon,
   SecurityIcon,
   EnhancementIcon,
 } from '@patternfly/react-icons';
 import {
-  getAnsibleCollections,
   getContentViewVersions,
   getDebPackages,
   getDockerTags,
@@ -20,10 +19,9 @@ import {
   getPackageGroups,
   getRepositories,
   getRPMPackages,
+  getContent,
 } from '../../ContentViewDetailActions';
 import {
-  selectAnsibleCollections,
-  selectAnsibleCollectionsStatus,
   selectCVVersions,
   selectCVVersionsStatus,
   selectDebPackages,
@@ -42,11 +40,15 @@ import {
   selectRPMPackageGroupsStatus,
   selectRPMPackages,
   selectRPMPackagesStatus,
+  selectContent,
+  selectContentStatus,
 } from '../../ContentViewDetailSelectors';
 import ContentViewVersionRepositoryCell from './ContentViewVersionRepositoryCell';
+import ContentConfig from '../../../../Content/ContentConfig';
 
 export const TableType = PropTypes.shape({
   name: PropTypes.string,
+  route: PropTypes.string,
   getCountKey: PropTypes.func,
   repoType: PropTypes.string,
   responseSelector: PropTypes.func,
@@ -64,6 +66,7 @@ export const TableType = PropTypes.shape({
 export default ({ cvId, versionId }) => [
   {
     name: __('Components'),
+    route: 'components',
     getCountKey: item => item?.component_view_count,
     responseSelector: state => selectCVVersions(state, cvId),
     statusSelector: state => selectCVVersionsStatus(state, cvId),
@@ -91,6 +94,7 @@ export default ({ cvId, versionId }) => [
   },
   {
     name: __('Repositories'),
+    route: 'repositories',
     getCountKey: item => item?.repositories?.length,
     responseSelector: state => selectRepositories(state),
     statusSelector: state => selectRepositoriesStatus(state),
@@ -127,6 +131,7 @@ export default ({ cvId, versionId }) => [
   },
   {
     name: __('RPM Packages'),
+    route: 'rpmPackages',
     repoType: 'yum',
     getCountKey: item => item?.rpm_count,
     responseSelector: state => selectRPMPackages(state),
@@ -148,6 +153,7 @@ export default ({ cvId, versionId }) => [
   },
   {
     name: __('RPM Package Groups'),
+    route: 'rpmPackageGroups',
     repoType: 'yum',
     getCountKey: item => item?.package_group_count,
     responseSelector: state => selectRPMPackageGroups(state),
@@ -161,6 +167,7 @@ export default ({ cvId, versionId }) => [
   },
   {
     name: __('Files'),
+    route: 'files',
     repoType: 'file',
     getCountKey: item => item?.file_count,
     responseSelector: state => selectFiles(state),
@@ -180,6 +187,7 @@ export default ({ cvId, versionId }) => [
   },
   {
     name: __('Errata'),
+    route: 'errata',
     repoType: 'yum',
     getCountKey: item => item?.erratum_count,
     responseSelector: state => selectErrata(state),
@@ -233,6 +241,7 @@ export default ({ cvId, versionId }) => [
   },
   {
     name: __('Module Streams'),
+    route: 'moduleStreams',
     repoType: 'yum',
     getCountKey: item => item?.module_stream_count,
     responseSelector: state => selectModuleStreams(state),
@@ -255,6 +264,7 @@ export default ({ cvId, versionId }) => [
   },
   {
     name: __('Deb Packages'),
+    route: 'debPackages',
     repoType: 'deb',
     getCountKey: item => item?.deb_count,
     responseSelector: state => selectDebPackages(state),
@@ -273,38 +283,9 @@ export default ({ cvId, versionId }) => [
       { title: __('Architecture'), getProperty: item => item?.architecture },
     ],
   },
-  {
-    name: __('Ansible Collections'),
-    repoType: 'ansible_collection',
-    getCountKey: item => item?.ansible_collection_count,
-    responseSelector: state => selectAnsibleCollections(state),
-    statusSelector: state => selectAnsibleCollectionsStatus(state),
-    autocompleteEndpoint: `/ansible_collections/auto_complete_search?content_view_version_id=${versionId}`,
-    fetchItems: params => getAnsibleCollections({ content_view_version_id: versionId, ...params }),
-    columnHeaders: [
-      {
-        title: __('Name'),
-        getProperty: item => (
-          <a href={urlBuilder(`ansible_collections/${item?.id}`, '')}>
-            {item?.name}
-          </a>),
-      },
-      {
-        title: __('Author'),
-        getProperty: item => item?.namespace,
-      },
-      {
-        title: __('Version'),
-        getProperty: item => item?.version,
-      },
-      {
-        title: __('Checksum'),
-        getProperty: item => item?.checksum,
-      },
-    ],
-  },
   {
     name: __('Docker Tags'),
+    route: 'dockerTags',
     repoType: 'docker',
     getCountKey: item => item?.docker_tag_count,
     responseSelector: state => selectDockerTags(state),
@@ -329,4 +310,19 @@ export default ({ cvId, versionId }) => [
       { title: __('Product Name'), getProperty: item => item?.product?.name },
     ],
   },
+  ...ContentConfig.map(({
+    names: { pluralTitle, pluralLabel, singularLabel },
+    columnHeaders,
+  }) => ({
+    name: pluralTitle,
+    route: camelCase(pluralLabel),
+    repoType: singularLabel,
+    getCountKey: item => item[`${singularLabel}_count`],
+    responseSelector: state => selectContent(pluralLabel, state),
+    statusSelector: state => selectContentStatus(pluralLabel, state),
+    autocompleteEndpoint: `/${pluralLabel}/auto_complete_search?content_view_version_id=${versionId}`,
+    fetchItems: params =>
+      getContent(pluralLabel, { content_view_version_id: versionId, ...params }),
+    columnHeaders,
+  })),
 ];

data/webpack/scenes/ContentViews/Details/Versions/VersionDetails/ContentViewVersionDetails.js

@@ -3,7 +3,7 @@ import useDeepCompareEffect from 'use-deep-compare-effect';
 import { useParams, Route, useHistory, useLocation, Redirect, Switch } from 'react-router-dom';
 import { useDispatch, useSelector, shallowEqual } from 'react-redux';
 import { STATUS } from 'foremanReact/constants';
-import { isEmpty, camelCase, first } from 'lodash';
+import { isEmpty, first } from 'lodash';
 import { Grid, Tabs, Tab, TabTitleText, Label } from '@patternfly/react-core';
 import { number, shape } from 'prop-types';
 import './ContentViewVersionDetails.scss';
@@ -68,7 +68,7 @@ const ContentViewVersionDetails = ({ cvId, details }) => {
   const filteredTableConfigs = tableConfigs.filter(({ getCountKey }) => !!getCountKey(response));
   const { repositories } = versionDetails;
   const showTabs = filteredTableConfigs.length > 0 && repositories;
-  const getCurrentActiveKey = tab ?? camelCase(first(filteredTableConfigs)?.name);
+  const getCurrentActiveKey = tab ?? first(filteredTableConfigs)?.route;
 
   return (
     <Grid>
@@ -84,10 +84,10 @@ const ContentViewVersionDetails = ({ cvId, details }) => {
             onSelect={onSelect}
             isVertical
           >
-            {filteredTableConfigs.map(({ name, getCountKey }) => (
+            {filteredTableConfigs.map(({ route, name, getCountKey }) => (
               <Tab
-                key={name}
-                eventKey={camelCase(name)}
+                key={route}
+                eventKey={route}
                 title={
                   <>
                     <TabTitleText>{name}</TabTitleText>
@@ -100,9 +100,9 @@ const ContentViewVersionDetails = ({ cvId, details }) => {
           <Switch>
             {filteredTableConfigs.map(config => (
               <Route
-                key={camelCase(config.name)}
+                key={config.route}
                 exact
-                path={`/versions/:versionId([0-9]+)/${camelCase(config.name)}`}
+                path={`/versions/:versionId([0-9]+)/${config.route}`}
               >
                 <ContentViewVersionDetailsTable
                   tableConfig={config}
@@ -111,7 +111,7 @@ const ContentViewVersionDetails = ({ cvId, details }) => {
               </Route>))
             }
             <Redirect
-              to={`/versions/${versionId}/${camelCase(first(filteredTableConfigs).name)}`}
+              to={`/versions/${versionId}/${first(filteredTableConfigs).route}`}
             />
           </Switch>
         </div>

data/webpack/scenes/ContentViews/Details/Versions/VersionDetails/ContentViewVersionRepositoryCell.js

@@ -67,7 +67,7 @@ const ContentViewVersionRepositoryCell = ({
     },
   };
 
-  ContentConfig().forEach((type) => {
+  ContentConfig.forEach((type) => {
     CONTENT_COUNTS[type.names.singularLabel] = {
       name: type.names.pluralLowercase,
       url: `products/${id}/repositories/${libraryInstanceId}/content/${type.names.pluralLabel}`,

data/webpack/scenes/ContentViews/Details/Versions/__tests__/contentViewVersions.test.js

@@ -122,7 +122,7 @@ test('Can show package and erratas and link to list page', async () => {
 
   await patientlyWaitFor(() => {
     expect(getAllByText(8)[0].closest('a'))
-      .toHaveAttribute('href', '/content_views/5#/versions/11/packages/');
+      .toHaveAttribute('href', '/content_views/5#/versions/11/rpmPackages/');
     expect(getAllByText(15)[0].closest('a'))
       .toHaveAttribute('href', '/content_views/5#/versions/11/errata/');
     expect(getByText(5)).toBeInTheDocument();