diff --git a/ui/apps/platform/src/Components/CheckboxSelect.tsx b/ui/apps/platform/src/Components/CheckboxSelect.tsx index ff9e8b20ac3cd..175119bfc8eb1 100644 --- a/ui/apps/platform/src/Components/CheckboxSelect.tsx +++ b/ui/apps/platform/src/Components/CheckboxSelect.tsx @@ -27,7 +27,7 @@ function CheckboxSelect({ const [isOpen, setIsOpen] = useState(false); const onToggleClick = () => { - setIsOpen(!isOpen); + setIsOpen((prev) => !prev); }; const onSelect = ( diff --git a/ui/apps/platform/src/Components/CollapsibleRow.jsx b/ui/apps/platform/src/Components/CollapsibleRow.jsx index 5696def6d46b4..c26330a01fb07 100644 --- a/ui/apps/platform/src/Components/CollapsibleRow.jsx +++ b/ui/apps/platform/src/Components/CollapsibleRow.jsx @@ -11,7 +11,7 @@ const CollapsibleRow = ({ header, isCollapsible, children, isCollapsibleOpen, ha if (!isCollapsible) { return; } - setOpen(!open); + setOpen((prev) => !prev); } const icons = { diff --git a/ui/apps/platform/src/Components/CollapsibleSection/CollapsibleSection.jsx b/ui/apps/platform/src/Components/CollapsibleSection/CollapsibleSection.jsx index 2a1af802dc999..8dbd7ebe650b9 100644 --- a/ui/apps/platform/src/Components/CollapsibleSection/CollapsibleSection.jsx +++ b/ui/apps/platform/src/Components/CollapsibleSection/CollapsibleSection.jsx @@ -20,7 +20,7 @@ const CollapsibleSection = ({ const [isOpen, setIsOpen] = useState(defaultOpen); function toggleOpen() { - setIsOpen(!isOpen); + setIsOpen((prev) => !prev); } const Icon = isOpen ? ( diff --git a/ui/apps/platform/src/Components/CompoundSearchFilter/components/AutocompleteSelect.tsx b/ui/apps/platform/src/Components/CompoundSearchFilter/components/AutocompleteSelect.tsx index 92e254fbc9479..94dd1c51b9a94 100644 --- a/ui/apps/platform/src/Components/CompoundSearchFilter/components/AutocompleteSelect.tsx +++ b/ui/apps/platform/src/Components/CompoundSearchFilter/components/AutocompleteSelect.tsx @@ -157,7 +157,7 @@ function AutocompleteSelect({ ); const onToggleClick = () => { - setIsOpen(!isOpen); + setIsOpen((prev) => !prev); }; const onSelect = ( diff --git a/ui/apps/platform/src/Components/CompoundSearchFilter/components/SearchFilterConditionText.tsx b/ui/apps/platform/src/Components/CompoundSearchFilter/components/SearchFilterConditionText.tsx index 84e94112a44ac..e49549f7b6445 100644 --- a/ui/apps/platform/src/Components/CompoundSearchFilter/components/SearchFilterConditionText.tsx +++ b/ui/apps/platform/src/Components/CompoundSearchFilter/components/SearchFilterConditionText.tsx @@ -95,7 +95,7 @@ function SearchFilterConditionText({ attribute, onSearch }: SearchFilterConditio const [isOpen, setIsOpen] = useState(false); const onToggleClick = () => { - setIsOpen(!isOpen); + setIsOpen((prev) => !prev); }; const onSelect = ( diff --git a/ui/apps/platform/src/Components/CompoundSearchFilter/components/SimpleSelect.tsx b/ui/apps/platform/src/Components/CompoundSearchFilter/components/SimpleSelect.tsx index 7be80cdb93b3e..4bb48ec76bbdb 100644 --- a/ui/apps/platform/src/Components/CompoundSearchFilter/components/SimpleSelect.tsx +++ b/ui/apps/platform/src/Components/CompoundSearchFilter/components/SimpleSelect.tsx @@ -25,7 +25,7 @@ function SimpleSelect({ const [isOpen, setIsOpen] = useState(false); const onToggleClick = () => { - setIsOpen(!isOpen); + setIsOpen((prev) => !prev); }; const onSelect = ( diff --git a/ui/apps/platform/src/Components/PatternFly/CheckboxSelect.tsx b/ui/apps/platform/src/Components/PatternFly/CheckboxSelect.tsx index e96caccb3457a..41b804545588a 100644 --- a/ui/apps/platform/src/Components/PatternFly/CheckboxSelect.tsx +++ b/ui/apps/platform/src/Components/PatternFly/CheckboxSelect.tsx @@ -100,7 +100,7 @@ function CheckboxSelect({ const selectRef = useRef(null); function onToggle() { - setIsOpen(!isOpen); + setIsOpen((prev) => !prev); } function handleBlur(event: FocusEvent) { diff --git a/ui/apps/platform/src/Components/PatternFly/MenuDropdown.tsx b/ui/apps/platform/src/Components/PatternFly/MenuDropdown.tsx index 484c416411d62..5bdf34f526d72 100644 --- a/ui/apps/platform/src/Components/PatternFly/MenuDropdown.tsx +++ b/ui/apps/platform/src/Components/PatternFly/MenuDropdown.tsx @@ -40,7 +40,7 @@ function MenuDropdown({ const [isOpen, setIsOpen] = useState(false); function onToggleClick() { - setIsOpen(!isOpen); + setIsOpen((prev) => !prev); } function onSelectHandler( diff --git a/ui/apps/platform/src/Components/ReportJob/PartialReportModal.tsx b/ui/apps/platform/src/Components/ReportJob/PartialReportModal.tsx index 4d862c9c64edb..7513b574231dd 100644 --- a/ui/apps/platform/src/Components/ReportJob/PartialReportModal.tsx +++ b/ui/apps/platform/src/Components/ReportJob/PartialReportModal.tsx @@ -25,7 +25,7 @@ function PartialReportModal({ failedClusters = [], onDownload }: PartialReportMo const [perPage, setPerPage] = useState(20); const handleModalToggle = () => { - setIsModalOpen(!isModalOpen); + setIsModalOpen((prev) => !prev); }; const startIndex = (page - 1) * perPage; diff --git a/ui/apps/platform/src/Components/SelectSingle/useSelectToggleState.ts b/ui/apps/platform/src/Components/SelectSingle/useSelectToggleState.ts index 9a29cbfbcc09f..c091d5a783998 100644 --- a/ui/apps/platform/src/Components/SelectSingle/useSelectToggleState.ts +++ b/ui/apps/platform/src/Components/SelectSingle/useSelectToggleState.ts @@ -15,7 +15,7 @@ function useSelectToggleState(onSelectionChange: (value: string) => void) { }; const onToggle = () => { - setIsOpen(!isOpen); + setIsOpen((prev) => !prev); }; return { diff --git a/ui/apps/platform/src/Components/TypeaheadSelect/TypeaheadSelect.tsx b/ui/apps/platform/src/Components/TypeaheadSelect/TypeaheadSelect.tsx index ef1fefb4028d7..315167dc88cf8 100644 --- a/ui/apps/platform/src/Components/TypeaheadSelect/TypeaheadSelect.tsx +++ b/ui/apps/platform/src/Components/TypeaheadSelect/TypeaheadSelect.tsx @@ -76,7 +76,7 @@ function TypeaheadSelect({ } function onToggle() { - setIsOpen(!isOpen); + setIsOpen((prev) => !prev); } function onInputChange(_event: FormEvent, text: string) { diff --git a/ui/apps/platform/src/Containers/Audit/ListeningEndpoints/ListeningEndpointsPage.tsx b/ui/apps/platform/src/Containers/Audit/ListeningEndpoints/ListeningEndpointsPage.tsx index b6dae4c88347f..e8fdccf64b915 100644 --- a/ui/apps/platform/src/Containers/Audit/ListeningEndpoints/ListeningEndpointsPage.tsx +++ b/ui/apps/platform/src/Containers/Audit/ListeningEndpoints/ListeningEndpointsPage.tsx @@ -172,14 +172,14 @@ function ListeningEndpointsPage() { setAutocompleteOpen(!autocompleteOpen)} + onClick={() => setAutocompleteOpen((prev) => !prev)} isExpanded={autocompleteOpen} isFullWidth > setAutocompleteOpen(!autocompleteOpen)} + onClick={() => setAutocompleteOpen((prev) => !prev)} onChange={(_event, value) => { setAutocompleteInputValue(value); updateSearchValue(value); diff --git a/ui/apps/platform/src/Containers/Audit/ListeningEndpoints/ListeningEndpointsTable.tsx b/ui/apps/platform/src/Containers/Audit/ListeningEndpoints/ListeningEndpointsTable.tsx index 93f1276f54047..91d0e9fb69465 100644 --- a/ui/apps/platform/src/Containers/Audit/ListeningEndpoints/ListeningEndpointsTable.tsx +++ b/ui/apps/platform/src/Containers/Audit/ListeningEndpoints/ListeningEndpointsTable.tsx @@ -73,7 +73,7 @@ function ListeningEndpointsTable({ // TODO Awkward type assertion here is fixed in PF 5 https://github.com/patternfly/patternfly-react/issues/8330 collapseAllAriaLabel: 'Expand or collapse all rows' as '', onToggle: () => { - setAllRowsExpanded(!areAllRowsExpanded); + setAllRowsExpanded((prev) => !prev); invertedExpansionRowSet.clear(); }, }} diff --git a/ui/apps/platform/src/Containers/Clusters/ClusterPage.tsx b/ui/apps/platform/src/Containers/Clusters/ClusterPage.tsx index 99e1eabb619e3..c2eb58048bba0 100644 --- a/ui/apps/platform/src/Containers/Clusters/ClusterPage.tsx +++ b/ui/apps/platform/src/Containers/Clusters/ClusterPage.tsx @@ -265,7 +265,7 @@ function ClusterPage({ clusterId }: ClusterPageProps): ReactElement { } function toggleSA() { - setCreateUpgraderSA(!createUpgraderSA); + setCreateUpgraderSA((prev) => !prev); } function onDownload() { diff --git a/ui/apps/platform/src/Containers/Clusters/DelegateScanning/Components/DelegatedScanningSettings.tsx b/ui/apps/platform/src/Containers/Clusters/DelegateScanning/Components/DelegatedScanningSettings.tsx index d71e3958f1853..993e3ff6249b7 100644 --- a/ui/apps/platform/src/Containers/Clusters/DelegateScanning/Components/DelegatedScanningSettings.tsx +++ b/ui/apps/platform/src/Containers/Clusters/DelegateScanning/Components/DelegatedScanningSettings.tsx @@ -63,7 +63,7 @@ function DelegatedScanningSettings({ setIsOpen(!isOpen)} + onClick={() => setIsOpen((prev) => !prev)} isDisabled={!isEditing} isExpanded={isOpen} > diff --git a/ui/apps/platform/src/Containers/ComplianceEnhanced/Coverage/components/ScanConfigurationSelect.tsx b/ui/apps/platform/src/Containers/ComplianceEnhanced/Coverage/components/ScanConfigurationSelect.tsx index dba9a8658f3ed..b03b4668352e5 100644 --- a/ui/apps/platform/src/Containers/ComplianceEnhanced/Coverage/components/ScanConfigurationSelect.tsx +++ b/ui/apps/platform/src/Containers/ComplianceEnhanced/Coverage/components/ScanConfigurationSelect.tsx @@ -36,7 +36,7 @@ function ScanConfigurationSelect({ const [isOpen, setIsOpen] = useState(false); const onToggleClick = () => { - setIsOpen(!isOpen); + setIsOpen((prev) => !prev); }; const onSelect = ( diff --git a/ui/apps/platform/src/Containers/NetworkGraph/NetworkGraphPage.tsx b/ui/apps/platform/src/Containers/NetworkGraph/NetworkGraphPage.tsx index 58df4f7f28306..584fb3eebefa8 100644 --- a/ui/apps/platform/src/Containers/NetworkGraph/NetworkGraphPage.tsx +++ b/ui/apps/platform/src/Containers/NetworkGraph/NetworkGraphPage.tsx @@ -281,7 +281,7 @@ function NetworkGraphPageContent() { }); } - setIsCIDRBlockFormOpen(!isCIDRBlockFormOpen); + setIsCIDRBlockFormOpen((prev) => !prev); } return ( diff --git a/ui/apps/platform/src/Containers/NetworkGraph/components/DefaultCIDRToggle.tsx b/ui/apps/platform/src/Containers/NetworkGraph/components/DefaultCIDRToggle.tsx index bd828b5ed5792..b597659322878 100644 --- a/ui/apps/platform/src/Containers/NetworkGraph/components/DefaultCIDRToggle.tsx +++ b/ui/apps/platform/src/Containers/NetworkGraph/components/DefaultCIDRToggle.tsx @@ -26,7 +26,7 @@ function DefaultCIDRToggle({ updateNetworkNodes = () => {} }): ReactElement { function toggleHandler(): void { setHideDefaultExternalSrcs(showDefaultExternalSrcs) .then(() => { - setShowDefaultExternalSrcs(!showDefaultExternalSrcs); + setShowDefaultExternalSrcs((prev) => !prev); setErrorMessage(''); updateNetworkNodes(); }) diff --git a/ui/apps/platform/src/Containers/NetworkGraph/simulation/NotifyYAMLModal.tsx b/ui/apps/platform/src/Containers/NetworkGraph/simulation/NotifyYAMLModal.tsx index b9ddc7adee93c..99ad5e8fc0291 100644 --- a/ui/apps/platform/src/Containers/NetworkGraph/simulation/NotifyYAMLModal.tsx +++ b/ui/apps/platform/src/Containers/NetworkGraph/simulation/NotifyYAMLModal.tsx @@ -46,7 +46,7 @@ function NotifyYAMLModal({ function onClose() { onClearAll(); setErrorMessage(null); - setIsModalOpen(!isModalOpen); + setIsModalOpen((prev) => !prev); } let content: ReactElement =
; diff --git a/ui/apps/platform/src/Containers/Policies/Wizard/Step1/PolicyCategoriesSelectField.tsx b/ui/apps/platform/src/Containers/Policies/Wizard/Step1/PolicyCategoriesSelectField.tsx index cedfbaa9920e8..deb1b9c6ae736 100644 --- a/ui/apps/platform/src/Containers/Policies/Wizard/Step1/PolicyCategoriesSelectField.tsx +++ b/ui/apps/platform/src/Containers/Policies/Wizard/Step1/PolicyCategoriesSelectField.tsx @@ -34,7 +34,7 @@ function PolicyCategoriesSelectField(): ReactElement { const selectedCategories: string[] = (field.value as string[]) ?? []; const onToggle = () => { - setIsOpen(!isOpen); + setIsOpen((prev) => !prev); }; const onSelect = ( diff --git a/ui/apps/platform/src/Containers/Policies/Wizard/Step3/PolicyCriteriaModal.tsx b/ui/apps/platform/src/Containers/Policies/Wizard/Step3/PolicyCriteriaModal.tsx index 8414c57edc332..83c0bb92a3dd4 100644 --- a/ui/apps/platform/src/Containers/Policies/Wizard/Step3/PolicyCriteriaModal.tsx +++ b/ui/apps/platform/src/Containers/Policies/Wizard/Step3/PolicyCriteriaModal.tsx @@ -186,7 +186,7 @@ function PolicyCriteriaModal({ - diff --git a/ui/apps/platform/src/Containers/Policies/Wizard/Step3/PolicySection.tsx b/ui/apps/platform/src/Containers/Policies/Wizard/Step3/PolicySection.tsx index 555851acbed4d..b52360ff02136 100644 --- a/ui/apps/platform/src/Containers/Policies/Wizard/Step3/PolicySection.tsx +++ b/ui/apps/platform/src/Containers/Policies/Wizard/Step3/PolicySection.tsx @@ -68,7 +68,7 @@ function PolicySection({ sectionIndex, descriptors, readOnly = false }: PolicySe diff --git a/ui/apps/platform/src/Containers/SystemHealth/DiagnosticBundle/DiagnosticBundleForm.tsx b/ui/apps/platform/src/Containers/SystemHealth/DiagnosticBundle/DiagnosticBundleForm.tsx index a48a35065fd8a..81ec2ec4c3ee3 100644 --- a/ui/apps/platform/src/Containers/SystemHealth/DiagnosticBundle/DiagnosticBundleForm.tsx +++ b/ui/apps/platform/src/Containers/SystemHealth/DiagnosticBundle/DiagnosticBundleForm.tsx @@ -69,7 +69,7 @@ function DiagnosticBundleForm({ }, [hasReadAccessForCluster]); function toggleClusterSelect() { - setClusterSelectOpen(!clusterSelectOpen); + setClusterSelectOpen((prev) => !prev); } function onSelect(_event: ReactMouseEvent | undefined, selection: string | number | undefined) { diff --git a/ui/apps/platform/src/Containers/Violations/Details/K8sCard.tsx b/ui/apps/platform/src/Containers/Violations/Details/K8sCard.tsx index 8b827ed9ef632..ea26e51605b42 100644 --- a/ui/apps/platform/src/Containers/Violations/Details/K8sCard.tsx +++ b/ui/apps/platform/src/Containers/Violations/Details/K8sCard.tsx @@ -28,7 +28,7 @@ function K8sCard({ message, keyValueAttrs = { attrs: [] }, time }: K8sCardProps) const [isExpanded, setIsExpanded] = useState(true); function onExpand() { - setIsExpanded(!isExpanded); + setIsExpanded((prev) => !prev); } return ( diff --git a/ui/apps/platform/src/Containers/Violations/Details/NetworkFlowCard.tsx b/ui/apps/platform/src/Containers/Violations/Details/NetworkFlowCard.tsx index a1fff5cd0afcc..208504d97e355 100644 --- a/ui/apps/platform/src/Containers/Violations/Details/NetworkFlowCard.tsx +++ b/ui/apps/platform/src/Containers/Violations/Details/NetworkFlowCard.tsx @@ -23,7 +23,7 @@ function NetworkFlowCard({ networkFlowInfo, message, time }: NetworkFlowCardProp const [isExpanded, setIsExpanded] = useState(true); function onExpand() { - setIsExpanded(!isExpanded); + setIsExpanded((prev) => !prev); } return ( diff --git a/ui/apps/platform/src/Containers/Violations/Details/TimestampedEventCard.tsx b/ui/apps/platform/src/Containers/Violations/Details/TimestampedEventCard.tsx index 4aabf110f1dc0..f13e20dc09917 100644 --- a/ui/apps/platform/src/Containers/Violations/Details/TimestampedEventCard.tsx +++ b/ui/apps/platform/src/Containers/Violations/Details/TimestampedEventCard.tsx @@ -31,7 +31,7 @@ function TimestampedEventCard({ const [isExpanded, setIsExpanded] = useState(true); function onExpand() { - setIsExpanded(!isExpanded); + setIsExpanded((prev) => !prev); } const timestamps = events diff --git a/ui/apps/platform/src/Containers/Violations/ViolationsTablePanel.tsx b/ui/apps/platform/src/Containers/Violations/ViolationsTablePanel.tsx index d3764b07eca7c..b09c137bead55 100644 --- a/ui/apps/platform/src/Containers/Violations/ViolationsTablePanel.tsx +++ b/ui/apps/platform/src/Containers/Violations/ViolationsTablePanel.tsx @@ -148,7 +148,7 @@ function ViolationsTablePanel({ ); function onToggleSelect() { - setIsSelectOpen(!isSelectOpen); + setIsSelectOpen((prev) => !prev); } // Handle setting confirmation modals for bulk actions diff --git a/ui/apps/platform/src/Containers/Vulnerabilities/VulnerablityReporting/forms/CollectionSelection.tsx b/ui/apps/platform/src/Containers/Vulnerabilities/VulnerablityReporting/forms/CollectionSelection.tsx index 2ed5463a604b2..1549025982aaf 100644 --- a/ui/apps/platform/src/Containers/Vulnerabilities/VulnerablityReporting/forms/CollectionSelection.tsx +++ b/ui/apps/platform/src/Containers/Vulnerabilities/VulnerablityReporting/forms/CollectionSelection.tsx @@ -154,7 +154,7 @@ function CollectionSelection({ } function onToggleClick() { - setIsOpen(!isOpen); + setIsOpen((prev) => !prev); if (isOpen) { setSearch(''); } diff --git a/ui/apps/platform/src/Containers/Vulnerabilities/WorkloadCves/components/CreateReportDropdown.tsx b/ui/apps/platform/src/Containers/Vulnerabilities/WorkloadCves/components/CreateReportDropdown.tsx index 0b81b85e7c4f4..dffeb62ed1c0f 100644 --- a/ui/apps/platform/src/Containers/Vulnerabilities/WorkloadCves/components/CreateReportDropdown.tsx +++ b/ui/apps/platform/src/Containers/Vulnerabilities/WorkloadCves/components/CreateReportDropdown.tsx @@ -11,7 +11,7 @@ function CreateReportDropdown({ onSelect }: CreateReportDropdownProps) { const [isOpen, setIsOpen] = useState(false); const onToggleClick = () => { - setIsOpen(!isOpen); + setIsOpen((prev) => !prev); }; const onSelectHandler = ( diff --git a/ui/apps/platform/src/Containers/Vulnerabilities/WorkloadCves/components/DefaultFilterModal.tsx b/ui/apps/platform/src/Containers/Vulnerabilities/WorkloadCves/components/DefaultFilterModal.tsx index 987f9214ba370..9b77cfc95ba8e 100644 --- a/ui/apps/platform/src/Containers/Vulnerabilities/WorkloadCves/components/DefaultFilterModal.tsx +++ b/ui/apps/platform/src/Containers/Vulnerabilities/WorkloadCves/components/DefaultFilterModal.tsx @@ -52,7 +52,7 @@ function DefaultFilterModal({ defaultFilters, setLocalStorage }: DefaultFilterMo if (isOpen) { setValues(defaultFilters).catch(() => {}); } - setIsOpen(!isOpen); + setIsOpen((prev) => !prev); } function handleSeverityChange(severity: VulnerabilitySeverityLabel, isChecked: boolean) { diff --git a/ui/apps/platform/src/hooks/patternfly/useSelectToggle.ts b/ui/apps/platform/src/hooks/patternfly/useSelectToggle.ts index 36dba3f78b32a..1d32881285fea 100644 --- a/ui/apps/platform/src/hooks/patternfly/useSelectToggle.ts +++ b/ui/apps/platform/src/hooks/patternfly/useSelectToggle.ts @@ -35,7 +35,7 @@ interface SelectToggleReturn { */ function useSelectToggle(defaultExpanded = false): SelectToggleReturn { const [isOpen, setIsOpen] = useState(defaultExpanded); - const onToggle = useCallback(() => setIsOpen(!isOpen), [isOpen, setIsOpen]); + const onToggle = useCallback(() => setIsOpen((prev) => !prev), [setIsOpen]); const toggleSelect = useCallback(setIsOpen, [setIsOpen]); const openSelect = useCallback(() => toggleSelect(true), [toggleSelect]); const closeSelect = useCallback(() => toggleSelect(false), [toggleSelect]); diff --git a/ui/apps/platform/src/hooks/useMultiSelect.tsx b/ui/apps/platform/src/hooks/useMultiSelect.tsx index ad36cf376bf2b..cdb0e530e168a 100644 --- a/ui/apps/platform/src/hooks/useMultiSelect.tsx +++ b/ui/apps/platform/src/hooks/useMultiSelect.tsx @@ -16,7 +16,7 @@ function useMultiSelect( const [isOpen, setIsOpen] = useState(false); function onToggle() { - setIsOpen(!isOpen); + setIsOpen((prev) => !prev); } function onSelect(_event, selection) {