katello 4.16.1 → 4.17.0.rc1

data/webpack/scenes/ContentViews/Details/ComponentContentViews/ComponentContentViewAddModal.js

@@ -1,10 +1,21 @@
 import React, { useState, useMemo } from 'react';
 import PropTypes from 'prop-types';
 import {
-  Flex, Modal, ModalVariant, Select, SelectVariant,
-  SelectOption, Checkbox, Form, FormGroup,
-  ActionGroup, Button, Tooltip,
+  Flex,
+  Modal,
+  ModalVariant,
+  Checkbox,
+  Form,
+  FormGroup,
+  ActionGroup,
+  Button,
+  Tooltip,
 } from '@patternfly/react-core';
+import {
+  Select,
+  SelectVariant,
+  SelectOption,
+} from '@patternfly/react-core/deprecated';
 import { OutlinedQuestionCircleIcon } from '@patternfly/react-icons';
 import { useDispatch, useSelector, shallowEqual } from 'react-redux';
 import { translate as __ } from 'foremanReact/common/I18n';
@@ -105,7 +116,7 @@ const ComponentContentViewAddModal = ({
             id="horzontal-form-title"
             name="horizontal-form-title"
             isOpen={cvVersionSelectOpen}
-            onToggle={isExpanded => setCvVersionSelectOpen(isExpanded)}
+            onToggle={(_event, isExpanded) => setCvVersionSelectOpen(isExpanded)}
             aria-label="CvVersion"
             ouiaId="select-cv-version"
             menuAppendTo="parent"
@@ -131,7 +142,7 @@ const ComponentContentViewAddModal = ({
               name="latest"
               label={__('Always update to latest version')}
               isChecked={formLatest}
-              onChange={checked => updateLatest(checked)}
+              onChange={(_event, checked) => updateLatest(checked)}
             />
             <Tooltip
               position="top"

data/webpack/scenes/ContentViews/Details/ComponentContentViews/ComponentContentViewBulkAddModal.js

@@ -1,10 +1,24 @@
 import React, { useState } from 'react';
 import PropTypes from 'prop-types';
 import {
-  Flex, Modal, ModalVariant, Select, SelectVariant,
-  SelectOption, Checkbox, Form, FormGroup,
-  ActionGroup, Button, Card, CardTitle, CardBody, Tooltip,
+  Flex,
+  Modal,
+  ModalVariant,
+  Checkbox,
+  Form,
+  FormGroup,
+  ActionGroup,
+  Button,
+  Card,
+  CardTitle,
+  CardBody,
+  Tooltip,
 } from '@patternfly/react-core';
+import {
+  Select,
+  SelectVariant,
+  SelectOption,
+} from '@patternfly/react-core/deprecated';
 import { OutlinedQuestionCircleIcon } from '@patternfly/react-icons';
 import { useDispatch } from 'react-redux';
 import { translate as __ } from 'foremanReact/common/I18n';
@@ -78,7 +92,7 @@ const ComponentContentViewBulkAddModal = ({ cvId, rowsToAdd, onClose }) => {
                   }
                   }
                   isOpen={cvVersionSelectOpen === componentCvName}
-                  onToggle={isExpanded => setCvVersionSelectOpen(isExpanded ? componentCvName : '')}
+                  onToggle={(_event, isExpanded) => setCvVersionSelectOpen(isExpanded ? componentCvName : '')}
                   id={`horzontal-form-title-${componentCvName}-${cvVersionSelectOpen[componentCvName]}`}
                   name="horizontal-form-title"
                   aria-label={`version-select-${componentCvName}`}
@@ -106,7 +120,7 @@ const ComponentContentViewBulkAddModal = ({ cvId, rowsToAdd, onClose }) => {
                     name="latest"
                     label={__('Always update to latest version')}
                     isChecked={selectedComponentLatest[componentCvName]}
-                    onChange={(checked) => {
+                    onChange={(_event, checked) => {
                       setSelectedComponentLatest({
                         ...selectedComponentLatest,
                         ...{ [componentCvName]: checked },

data/webpack/scenes/ContentViews/Details/ComponentContentViews/ComponentEnvironments.js

@@ -8,7 +8,7 @@ const ComponentEnvironments = ({ environments }) => environments.map((env, index
     style={{ margin: `4px 0 4px ${index > 0 ? '4px' : '0'}` }}
     color="purple"
     href={`/lifecycle_environments/${env.id}`}
-    isTruncated
+
   >
     {env.name}
   </Label>

data/webpack/scenes/ContentViews/Details/ComponentContentViews/ContentViewComponents.js

@@ -2,9 +2,18 @@ import React, { useState, useCallback } from 'react';
 import useDeepCompareEffect from 'use-deep-compare-effect';
 import { useDispatch, useSelector } from 'react-redux';
 import {
-  Bullseye, Split, SplitItem, Button, ActionList,
-  ActionListItem, Dropdown, DropdownItem, KebabToggle,
+  Bullseye,
+  Split,
+  SplitItem,
+  Button,
+  ActionList,
+  ActionListItem,
 } from '@patternfly/react-core';
+import {
+  Dropdown,
+  DropdownItem,
+  KebabToggle,
+} from '@patternfly/react-core/deprecated';
 import { TableVariant, fitContent, TableText } from '@patternfly/react-table';
 import { PencilAltIcon } from '@patternfly/react-icons';
 import { STATUS } from 'foremanReact/constants';

data/webpack/scenes/ContentViews/Details/ComponentContentViews/__tests__/compositeCVDetails.fixtures.json

@@ -1,6 +1,7 @@
 {
   "content_host_count": 0,
   "composite": true,
+  "rolling": false,
   "component_ids": [
     5,
     7
@@ -390,4 +391,4 @@
     "promote_or_remove_content_views": true
   },
   "errors": null
-}
+}

data/webpack/scenes/ContentViews/Details/ComponentContentViews/__tests__/contentViewComponents.test.js

@@ -1,5 +1,5 @@
 import React from 'react';
-import { renderWithRedux, patientlyWaitFor, fireEvent } from 'react-testing-lib-wrapper';
+import { renderWithRedux, patientlyWaitFor, fireEvent, act } from 'react-testing-lib-wrapper';
 import { nockInstance, assertNockRequest, mockAutocomplete } from '../../../../../test-utils/nockWrapper';
 import api from '../../../../../services/api';
 import CONTENT_VIEWS_KEY from '../../../ContentViewsConstants';
@@ -47,7 +47,8 @@ test('Can call API and show components on page load', async (done) => {
   await patientlyWaitFor(() => expect(getByText(firstComponent.content_view.label))
     .toBeInTheDocument());
   assertNockRequest(autocompleteScope);
-  assertNockRequest(scope, done);
+  assertNockRequest(scope);
+  done();
 });
 
 test('Can call API and show unpublished components', async (done) => {
@@ -71,7 +72,8 @@ test('Can call API and show unpublished components', async (done) => {
     expect(getAllByText('Not yet published')).toHaveLength(4);
   });
   assertNockRequest(autocompleteScope);
-  assertNockRequest(scope, done);
+  assertNockRequest(scope);
+  done();
 });
 
 test('Can link to view environment', async () => {
@@ -150,7 +152,8 @@ test('Can search for component content views in composite view', async (done) =>
 
   assertNockRequest(autocompleteScope);
   assertNockRequest(cvComponentsScope);
-  assertNockRequest(withSearchScope, done);
+  assertNockRequest(withSearchScope);
+  done();
 });
 
 test('Can handle no components being present', async (done) => {
@@ -183,7 +186,8 @@ test('Can handle no components being present', async (done) => {
   expect(queryByText(firstComponent.content_view.label)).toBeNull();
   await patientlyWaitFor(() => expect(queryByText('No content views to add yet')).toBeInTheDocument());
   assertNockRequest(autocompleteScope);
-  assertNockRequest(scope, done);
+  assertNockRequest(scope);
+  done();
 });
 
 test('Can add published component views to content view with modal', async (done) => {
@@ -216,33 +220,46 @@ test('Can add published component views to content view with modal', async (done
     .reply(200, compositeCvDetails);
 
   const {
-    getByText, getByLabelText, queryByLabelText, getAllByLabelText,
+    getByText, getByLabelText, queryByLabelText, getAllByLabelText, findAllByLabelText,
   } = renderWithRedux(
     <ContentViewComponents cvId={4} details={cvDetails} />,
     renderOptions,
   );
   await patientlyWaitFor(() => {
-    expect(getAllByLabelText('Actions')[2]).toHaveAttribute('aria-expanded', 'false');
+    expect(getAllByLabelText('Kebab toggle')[2]).toHaveAttribute('aria-expanded', 'false');
+  });
+  await act(async () => {
+    await findAllByLabelText('Kebab toggle');
+    fireEvent.click(getAllByLabelText('Kebab toggle')[2]);
   });
-  fireEvent.click(getAllByLabelText('Actions')[2]);
-  expect(getAllByLabelText('Actions')[2]).toHaveAttribute('aria-expanded', 'true');
+
   await patientlyWaitFor(() => {
     expect(getByText('Add')).toBeInTheDocument();
   });
-  fireEvent.click(getByText('Add'));
+
+  await act(async () => {
+    fireEvent.click(getByText('Add'));
+  });
+
   await patientlyWaitFor(() => {
     expect(getByText('Add content view')).toBeInTheDocument();
   });
-  fireEvent.click(getByLabelText('add_component'));
+
+  await act(async () => {
+    fireEvent.click(getByLabelText('add_component'));
+  });
+
   await patientlyWaitFor(() => {
     expect(queryByLabelText('add_component')).not.toBeInTheDocument();
   });
+
   assertNockRequest(autocompleteScope);
   assertNockRequest(scope);
   assertNockRequest(publishedComponentVersionsScope);
   assertNockRequest(addComponentScope);
   assertNockRequest(returnScope);
-  assertNockRequest(cvDetailsScope, done);
+  assertNockRequest(cvDetailsScope);
+  done();
 });
 
 test('Can add unpublished component views to content view', async (done) => {
@@ -274,17 +291,20 @@ test('Can add unpublished component views to content view', async (done) => {
     renderOptions,
   );
   await patientlyWaitFor(() => {
-    expect(getAllByLabelText('Actions').slice(-1)[0]).toHaveAttribute('aria-expanded', 'false');
+    expect(getAllByLabelText('Kebab toggle').slice(-1)[0]).toHaveAttribute('aria-expanded', 'false');
+  });
+  await act(async () => {
+    fireEvent.click(getAllByLabelText('Kebab toggle').slice(-1)[0]);
   });
-  fireEvent.click(getAllByLabelText('Actions').slice(-1)[0]);
-  expect(getAllByLabelText('Actions').slice(-1)[0]).toHaveAttribute('aria-expanded', 'true');
+  expect(getAllByLabelText('Kebab toggle').slice(-1)[0]).toHaveAttribute('aria-expanded', 'true');
   await patientlyWaitFor(() => expect(getByText('Add')).toBeInTheDocument());
   fireEvent.click(getByText('Add'));
   assertNockRequest(autocompleteScope);
   assertNockRequest(scope);
   assertNockRequest(addComponentScope);
   assertNockRequest(returnScope);
-  assertNockRequest(cvDetailsScope, done);
+  assertNockRequest(cvDetailsScope);
+  done();
 });
 
 test('Can remove component views from content view', async (done) => {
@@ -316,17 +336,20 @@ test('Can remove component views from content view', async (done) => {
     renderOptions,
   );
   await patientlyWaitFor(() => {
-    expect(getAllByLabelText('Actions')[0]).toHaveAttribute('aria-expanded', 'false');
+    expect(getAllByLabelText('Kebab toggle')[0]).toHaveAttribute('aria-expanded', 'false');
   });
-  fireEvent.click(getAllByLabelText('Actions')[0]);
-  expect(getAllByLabelText('Actions')[0]).toHaveAttribute('aria-expanded', 'true');
+  await act(async () => {
+    fireEvent.click(getAllByLabelText('Kebab toggle')[0]);
+  });
+  expect(getAllByLabelText('Kebab toggle')[0]).toHaveAttribute('aria-expanded', 'true');
   await patientlyWaitFor(() => expect(getByText('Remove')).toBeInTheDocument());
   fireEvent.click(getByText('Remove'));
   assertNockRequest(autocompleteScope);
   assertNockRequest(scope);
   assertNockRequest(removeComponentScope);
   assertNockRequest(returnScope);
-  assertNockRequest(cvDetailsScope, done);
+  assertNockRequest(cvDetailsScope);
+  done();
 });
 
 test('Can bulk add component views to content view with modal', async (done) => {
@@ -363,20 +386,38 @@ test('Can bulk add component views to content view with modal', async (done) =>
     expect(getByLabelText('Select row 2')).toBeInTheDocument();
     expect(getByLabelText('bulk_add_components')).toHaveAttribute('aria-disabled', 'true');
   });
-  fireEvent.click(getByLabelText('Select row 2'));
-  fireEvent.click(getByLabelText('Select row 3'));
+  await act(async () => {
+    fireEvent.click(getByLabelText('Select row 2'));
+  });
+
+  await act(async () => {
+    fireEvent.click(getByLabelText('Select row 3'));
+  });
+
   await patientlyWaitFor(() => {
     expect(getByLabelText('bulk_add_components')).toHaveAttribute('aria-disabled', 'false');
   });
-  fireEvent.click(getByLabelText('bulk_add_components'));
+
+  await act(async () => {
+    fireEvent.click(getByLabelText('bulk_add_components'));
+  });
+
   await patientlyWaitFor(() => {
     expect(getAllByText('Add content views')[1]).toBeInTheDocument();
     expect(getAllByRole('textbox')[0]).toHaveValue('Version 4.0 (3 days ago)');
   });
-  fireEvent.click(getAllByRole('textbox')[0]);
-  fireEvent.click(queryByText('Version 3.0'));
 
-  fireEvent.click(getByLabelText('add_components'));
+  await act(async () => {
+    fireEvent.click(getAllByRole('textbox')[0]);
+  });
+
+  await act(async () => {
+    fireEvent.click(queryByText('Version 3.0'));
+  });
+
+  await act(async () => {
+    fireEvent.click(getByLabelText('add_components'));
+  });
   await patientlyWaitFor(() => {
     expect(queryByText('Select available version of content views to use')).not.toBeInTheDocument();
     expect(getByLabelText('bulk_add_components')).toHaveAttribute('aria-disabled', 'false');
@@ -386,5 +427,6 @@ test('Can bulk add component views to content view with modal', async (done) =>
   assertNockRequest(scope);
   assertNockRequest(addComponentScope);
   assertNockRequest(returnScope);
-  assertNockRequest(cvDetailsScope, done);
+  assertNockRequest(cvDetailsScope);
+  done();
 });

data/webpack/scenes/ContentViews/Details/ComponentContentViews/__tests__/publishedContentViewDetails.fixtures.json

@@ -1,6 +1,7 @@
 {
   "content_host_count": 0,
   "composite": false,
+  "rolling": false,
   "component_ids": [],
   "default": false,
   "version_count": 33,
@@ -210,4 +211,4 @@
   },
   "duplicate_repositories_to_publish": [],
   "errors": null
-}
+}

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

@@ -10,11 +10,13 @@ import {
   Button,
   Flex,
   FlexItem,
+} from '@patternfly/react-core';
+import {
   Dropdown,
   DropdownItem,
   KebabToggle,
   DropdownPosition,
-} from '@patternfly/react-core';
+} from '@patternfly/react-core/deprecated';
 import { STATUS } from 'foremanReact/constants';
 import { ExternalLinkAltIcon } from '@patternfly/react-icons';
 import { translate as __ } from 'foremanReact/common/I18n';
@@ -81,31 +83,17 @@ export default () => {
   if (status === STATUS.PENDING) return (<Loading />);
   if (status === STATUS.ERROR) return (<EmptyStateMessage error={error} />);
 
-  const dropDownItems = [
-    <DropdownItem
-      key="copy"
-      ouiaId="cv-copy"
-      onClick={() => {
-        setCopying(true);
-      }}
-    >
-      {__('Copy')}
-    </DropdownItem>,
-    <DropdownItem
-      key="delete"
-      ouiaId="cv-delete"
-      onClick={() => {
-        setDeleting(true);
-      }}
-    >
-      {__('Delete')}
-    </DropdownItem>,
-  ];
-
   const {
-    name, composite, permissions, environments, versions,
+    name, composite, rolling, permissions, environments, versions,
     generated_for: generatedFor, import_only: importOnly,
   } = details;
+
+  const dropDownItems = [];
+  if (!rolling) {
+    dropDownItems.push(<DropdownItem key="copy" ouiaId="cv-copy" onClick={() => { setCopying(true); }} > {__('Copy')}</DropdownItem>);
+  }
+  dropDownItems.push(<DropdownItem key="delete" ouiaId="cv-delete" onClick={() => { setDeleting(true); }} > {__('Delete')}</DropdownItem>);
+
   const generatedContentView = generatedFor !== 'none';
   const detailsTab = {
     key: 'details',
@@ -141,11 +129,11 @@ export default () => {
   /* eslint-disable no-nested-ternary */
   const tabs = [
     detailsTab,
-    versionsTab,
+    ...(rolling ? [] : [versionsTab]),
     ...(composite ? [contentViewsTab] :
-      ((importOnly || generatedContentView) ?
+      ((importOnly || generatedContentView || rolling) ?
         [repositoriesTab] : [repositoriesTab, filtersTab])),
-    historyTab,
+    ...(rolling ? [] : [historyTab]),
   ];
   /* eslint-enable no-nested-ternary */
 
@@ -162,7 +150,11 @@ export default () => {
               <FlexItem>
                 <TextContent>
                   <Text ouiaId="cv-details-header-name" component={TextVariants.h1}>
-                    <ContentViewIcon count={truncate(name)} composite={composite} />
+                    <ContentViewIcon
+                      count={truncate(name)}
+                      composite={composite}
+                      rolling={rolling}
+                    />
                   </Text>
                 </TextContent>
               </FlexItem>
@@ -170,7 +162,8 @@ export default () => {
           </GridItem>
           <GridItem md={4} sm={12} style={{ minWidth: '380px' }}>
             <Flex justifyContent={{ lg: 'justifyContentFlexEnd', sm: 'justifyContentFlexStart' }}>
-              {hasPermission(permissions, 'publish_content_views') &&
+              {!rolling &&
+                hasPermission(permissions, 'publish_content_views') &&
                 <FlexItem>
                   <Button
                     ouiaId="cv-details-publish-button"
@@ -211,7 +204,7 @@ export default () => {
                   position={DropdownPosition.right}
                   ouiaId="cv-details-actions"
                   style={{ marginLeft: 'auto' }}
-                  toggle={<KebabToggle onToggle={setDropdownOpen} id="toggle-dropdown" />}
+                  toggle={<KebabToggle onToggle={(_event, val) => setDropdownOpen(val)} id="toggle-dropdown" />}
                   isOpen={dropDownOpen}
                   isPlain
                   dropdownItems={dropDownItems}

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

@@ -30,6 +30,7 @@ const ContentViewInfo = ({ cvId, details }) => {
     label,
     description,
     composite,
+    rolling,
     solve_dependencies: solveDependencies,
     auto_publish: autoPublish,
     import_only: importOnly,
@@ -41,6 +42,8 @@ const ContentViewInfo = ({ cvId, details }) => {
     if (val === details[attribute]) return;
     dispatch(updateContentView(cvId, { [attribute]: val }));
   };
+  let iconText = __('Content view');
+  if (composite) { iconText = __('Composite content view'); } else if (rolling) { iconText = __('Rolling content view'); }
 
   return (
     <TextContent className="margin-0-24">
@@ -71,7 +74,7 @@ const ContentViewInfo = ({ cvId, details }) => {
         <TextListItem component={TextListItemVariants.dd} className="foreman-spaced-list">
           <Flex>
             <FlexItem spacer={{ default: 'spacerXs' }}>
-              <ContentViewIcon composite={composite} description={composite ? __('Composite') : __('Content view')} />
+              <ContentViewIcon composite={composite} rolling={rolling} description={iconText} />
             </FlexItem>
           </Flex>
         </TextListItem>
@@ -99,7 +102,7 @@ const ContentViewInfo = ({ cvId, details }) => {
             boolean
             {...{ currentAttribute, setCurrentAttribute }}
           />) :
-          (<ActionableDetail
+          (!rolling && <ActionableDetail
             label={__('Solve dependencies')}
             attribute="solve_dependencies"
             key={solveDependencies}
@@ -155,6 +158,7 @@ ContentViewInfo.propTypes = {
     label: PropTypes.string,
     description: PropTypes.string,
     composite: PropTypes.bool,
+    rolling: PropTypes.bool,
     solve_dependencies: PropTypes.bool,
     auto_publish: PropTypes.bool,
     import_only: PropTypes.bool,

data/webpack/scenes/ContentViews/Details/Filters/Add/CVFilterAddModal.js

@@ -6,9 +6,22 @@ import { STATUS } from 'foremanReact/constants';
 import PropTypes from 'prop-types';
 import { translate as __ } from 'foremanReact/common/I18n';
 import {
-  Modal, ModalVariant, Form, FormGroup, TextInput, ActionGroup, Button, Radio, TextArea,
-  Split, SplitItem, Select, SelectOption,
+  Modal,
+  ModalVariant,
+  Form,
+  FormGroup,
+  TextInput,
+  ActionGroup,
+  Button,
+  Radio,
+  TextArea,
+  Split,
+  SplitItem,
 } from '@patternfly/react-core';
+import {
+  Select,
+  SelectOption,
+} from '@patternfly/react-core/deprecated';
 import { addCVFilterRule, createContentViewFilter, getRepositoryTypes } from '../../ContentViewDetailActions';
 import { selectCreateContentViewFilter, selectCreateContentViewFilterStatus,
   selectCreateContentViewFilterError, selectCreateFilterRule,
@@ -129,7 +142,7 @@ const CVFilterAddModal = ({ cvId, onClose }) => {
             ouiaId="input_name"
             name="name"
             value={name}
-            onChange={value => setName(value)}
+            onChange={(_event, value) => setName(value)}
           />
         </FormGroup>
         <FormGroup label={__('Content type')} isRequired fieldId="content_type">
@@ -137,7 +150,7 @@ const CVFilterAddModal = ({ cvId, onClose }) => {
             selections={type}
             onSelect={onSelect}
             isOpen={typeSelectOpen}
-            onToggle={setTypeSelectOpen}
+            onToggle={(_event, val) => setTypeSelectOpen(val)}
             ouiaId="content_type"
             id="content_type"
             name="content_type"
@@ -152,7 +165,7 @@ const CVFilterAddModal = ({ cvId, onClose }) => {
               <Radio
                 isChecked={inclusion}
                 name="radio-1"
-                onChange={checked => setInclusion(checked)}
+                onChange={(_event, checked) => setInclusion(checked)}
                 label={__('Include filter')}
                 id="include_filter"
                 ouiaId="include_filter"
@@ -164,7 +177,7 @@ const CVFilterAddModal = ({ cvId, onClose }) => {
               <Radio
                 isChecked={!inclusion}
                 name="radio-1"
-                onChange={checked => setInclusion(!checked)}
+                onChange={(_event, checked) => setInclusion(!checked)}
                 label={__('Exclude filter')}
                 id="exclude_filter"
                 ouiaId="exclude_filter"
@@ -184,7 +197,7 @@ const CVFilterAddModal = ({ cvId, onClose }) => {
             resizeOrientation="vertical"
             autoResize
             style={{ maxHeight: '200px', minHeight: '36px' }}
-            onChange={value => setDescription(value)}
+            onChange={(_event, value) => setDescription(value)}
           />
         </FormGroup>
         <ActionGroup>

data/webpack/scenes/ContentViews/Details/Filters/Add/__tests__/cvFilterAdd.test.js

@@ -39,7 +39,8 @@ test('Can save content view filter from form', (done) => {
 
   fireEvent.submit(getByLabelText('create_filter'));
   assertNockRequest(repoTypeScope);
-  assertNockRequest(createFilterscope, done);
+  assertNockRequest(createFilterscope);
+  done();
 });
 
 test('Closes content view filter form upon save', async (done) => {
@@ -59,5 +60,6 @@ test('Closes content view filter form upon save', async (done) => {
     expect(queryByText('Description')).not.toBeInTheDocument();
   });
   assertNockRequest(repoTypeScope);
-  assertNockRequest(createFilterscope, done);
+  assertNockRequest(createFilterscope);
+  done();
 });

data/webpack/scenes/ContentViews/Details/Filters/AffectedRepositories/AffectedRepositorySelection.js

@@ -1,7 +1,10 @@
 import React, { useState } from 'react';
 import { shallowEqual, useSelector, useDispatch } from 'react-redux';
 import PropTypes from 'prop-types';
-import { Select, SelectOption } from '@patternfly/react-core';
+import {
+  Select,
+  SelectOption,
+} from '@patternfly/react-core/deprecated';
 import { translate as __ } from 'foremanReact/common/I18n';
 import { selectCVFilterDetails } from '../../ContentViewDetailSelectors';
 import { editCVFilter, getCVFilterDetails } from '../../ContentViewDetailActions';
@@ -41,7 +44,7 @@ const AffectedRepositorySelection = ({
       selections={type}
       onSelect={onSelect}
       isOpen={typeSelectOpen}
-      onToggle={isExpanded => setTypeSelectOpen(isExpanded)}
+      onToggle={(_event, isExpanded) => setTypeSelectOpen(isExpanded)}
       id="affected_repos"
       name="affected_repos"
       aria-label="affected_repos"

data/webpack/scenes/ContentViews/Details/Filters/AffectedRepositories/AffectedRepositoryTable.js

@@ -8,10 +8,12 @@ import {
   Button,
   ActionList,
   ActionListItem,
+} from '@patternfly/react-core';
+import {
   Dropdown,
   DropdownItem,
   KebabToggle,
-} from '@patternfly/react-core';
+} from '@patternfly/react-core/deprecated';
 import { TableVariant, fitContent } from '@patternfly/react-table';
 import { omit } from 'lodash';
 import { STATUS } from 'foremanReact/constants';

data/webpack/scenes/ContentViews/Details/Filters/ArtifactsWithNoErrata.js

@@ -44,7 +44,7 @@ export const ArtifactsWithNoErrataRenderer = ({ filterDetails }) => {
     label={getLabel()}
     isChecked={artifactsNoErrata}
     isDisabled={isLoading}
-    onChange={checked => setArtifactsNoErrata(checked)}
+    onChange={(_event, checked) => setArtifactsNoErrata(checked)}
   />);
 };
 

data/webpack/scenes/ContentViews/Details/Filters/CVContainerImageFilterContent.js

@@ -10,11 +10,13 @@ import {
   TabTitleText,
   Split,
   SplitItem,
+  Button,
+} from '@patternfly/react-core';
+import {
   Dropdown,
   DropdownItem,
   KebabToggle,
-  Button,
-} from '@patternfly/react-core';
+} from '@patternfly/react-core/deprecated';
 import { STATUS } from 'foremanReact/constants';
 import { translate as __ } from 'foremanReact/common/I18n';
 import onSelect from '../../../../components/Table/helpers';

data/webpack/scenes/ContentViews/Details/Filters/CVDebFilterContent.js

@@ -3,7 +3,19 @@ import useDeepCompareEffect from 'use-deep-compare-effect';
 import PropTypes from 'prop-types';
 import { shallowEqual, useSelector, useDispatch } from 'react-redux';
 import { TableVariant } from '@patternfly/react-table';
-import { Tabs, Tab, TabTitleText, Split, SplitItem, Button, Dropdown, DropdownItem, KebabToggle } from '@patternfly/react-core';
+import {
+  Tabs,
+  Tab,
+  TabTitleText,
+  Split,
+  SplitItem,
+  Button,
+} from '@patternfly/react-core';
+import {
+  Dropdown,
+  DropdownItem,
+  KebabToggle,
+} from '@patternfly/react-core/deprecated';
 import { STATUS } from 'foremanReact/constants';
 import { translate as __ } from 'foremanReact/common/I18n';
 import onSelect from '../../../../components/Table/helpers';