From 94b5adac35e922e85f35feb7f0d6ab0f93b06231 Mon Sep 17 00:00:00 2001
From: Jelle van der Waa <jvanderwaa@redhat.com>
Date: Mon, 26 Aug 2024 17:44:23 +0200
Subject: [PATCH] Remove all unused passed arguments to functions or components

---
 src/app.jsx                                         | 3 +--
 src/components/networks/networkOverviewTab.jsx      | 5 ++---
 src/components/storagePools/storagePool.jsx         | 2 +-
 src/components/storagePools/storagePoolList.jsx     | 5 ++---
 src/components/vm/disks/vmDisksCard.jsx             | 4 +---
 src/components/vm/filesystems/vmFilesystemsCard.jsx | 5 ++---
 src/components/vm/hostdevs/hostDevAdd.jsx           | 4 ++--
 src/components/vm/nics/nicAdd.jsx                   | 5 ++---
 src/components/vm/snapshots/vmSnapshotsCard.jsx     | 2 +-
 src/helpers.js                                      | 2 +-
 10 files changed, 15 insertions(+), 22 deletions(-)

diff --git a/src/app.jsx b/src/app.jsx
index 043e65017..9501073b6 100644
--- a/src/app.jsx
+++ b/src/app.jsx
@@ -347,8 +347,7 @@ class AppActive extends React.Component {
                 <StoragePoolList storagePools={storagePools}
                     vms={vms}
                     loggedUser={systemInfo.loggedUser}
-                    libvirtVersion={systemInfo.libvirtVersion}
-                    onAddErrorNotification={this.onAddErrorNotification} />
+                    libvirtVersion={systemInfo.libvirtVersion} />
                 }
                 {path.length > 0 && path[0] == 'networks' &&
                 <NetworkList networks={networks}
diff --git a/src/components/networks/networkOverviewTab.jsx b/src/components/networks/networkOverviewTab.jsx
index 8c2702704..962ff6cb7 100644
--- a/src/components/networks/networkOverviewTab.jsx
+++ b/src/components/networks/networkOverviewTab.jsx
@@ -190,14 +190,13 @@ const NetworkAddStaticHostEntriesAction = ({ idPrefix, network, parentIndex }) =
                     onClick={() => setIsOpen(true)}>
                 {_("add entry")}
             </Button>
-            {isOpen && <NetworkAddStaticHostEntries idPrefix={idPrefix}
-                                                    network={network} parentIndex={parentIndex}
+            {isOpen && <NetworkAddStaticHostEntries network={network} parentIndex={parentIndex}
                                                     setIsOpen={setIsOpen} />}
         </>
     );
 };
 
-const NetworkAddStaticHostEntries = ({ idPrefix, network, parentIndex, setIsOpen }) => {
+const NetworkAddStaticHostEntries = ({ network, parentIndex, setIsOpen }) => {
     const [isSubmitted, setIsSubmitted] = useState(false);
     const [ipAddress, setIpAddress] = useState('');
     const [macAddress, setMacAddress] = useState('');
diff --git a/src/components/storagePools/storagePool.jsx b/src/components/storagePools/storagePool.jsx
index 8fba12997..049445d5f 100644
--- a/src/components/storagePools/storagePool.jsx
+++ b/src/components/storagePools/storagePool.jsx
@@ -41,7 +41,7 @@ import cockpit from 'cockpit';
 
 const _ = cockpit.gettext;
 
-export const getStoragePoolRow = ({ storagePool, vms, onAddErrorNotification }) => {
+export const getStoragePoolRow = ({ storagePool, vms }) => {
     const idPrefix = `${storagePoolId(storagePool.name, storagePool.connectionName)}`;
     const name = (
         <span id={`${idPrefix}-name`}>
diff --git a/src/components/storagePools/storagePoolList.jsx b/src/components/storagePools/storagePoolList.jsx
index 06d625e55..353c993f9 100644
--- a/src/components/storagePools/storagePoolList.jsx
+++ b/src/components/storagePools/storagePoolList.jsx
@@ -39,7 +39,7 @@ export class StoragePoolList extends React.Component {
     }
 
     render() {
-        const { storagePools, loggedUser, vms, onAddErrorNotification, libvirtVersion } = this.props;
+        const { storagePools, loggedUser, vms, libvirtVersion } = this.props;
         const sortFunction = (storagePoolA, storagePoolB) => storagePoolA.name.localeCompare(storagePoolB.name);
         const actions = (<CreateStoragePoolAction loggedUser={loggedUser} libvirtVersion={libvirtVersion} />);
 
@@ -77,7 +77,7 @@ export class StoragePoolList extends React.Component {
                                                       .map(storagePool => {
                                                           const filterVmsByConnection = vms.filter(vm => vm.connectionName == storagePool.connectionName);
 
-                                                          return getStoragePoolRow({ storagePool, vms: filterVmsByConnection, onAddErrorNotification });
+                                                          return getStoragePoolRow({ storagePool, vms: filterVmsByConnection });
                                                       })
                                               }
                                 />
@@ -92,6 +92,5 @@ export class StoragePoolList extends React.Component {
 StoragePoolList.propTypes = {
     storagePools: PropTypes.array.isRequired,
     vms: PropTypes.array.isRequired,
-    onAddErrorNotification: PropTypes.func.isRequired,
     libvirtVersion: PropTypes.number,
 };
diff --git a/src/components/vm/disks/vmDisksCard.jsx b/src/components/vm/disks/vmDisksCard.jsx
index 53bde465e..77e07ca19 100644
--- a/src/components/vm/disks/vmDisksCard.jsx
+++ b/src/components/vm/disks/vmDisksCard.jsx
@@ -101,7 +101,7 @@ export class VmDisksCardLibvirt extends React.Component {
         return areDiskStatsSupported;
     }
 
-    prepareDiskData(disk, diskStats, idPrefix, storagePools) {
+    prepareDiskData(disk, diskStats, storagePools) {
         diskStats = diskStats || {};
 
         let used = diskStats.allocation;
@@ -139,14 +139,12 @@ export class VmDisksCardLibvirt extends React.Component {
     render() {
         const { vm, vms, storagePools, supportedDiskBusTypes, onAddErrorNotification } = this.props;
 
-        const idPrefix = `${vmId(vm.name)}-disks`;
         const areDiskStatsSupported = this.getDiskStatsSupport(vm);
 
         const disks = Object.getOwnPropertyNames(vm.disks)
                 .sort() // by 'target'
                 .map(target => this.prepareDiskData(vm.disks[target],
                                                     vm.disksStats && vm.disksStats[target],
-                                                    `${idPrefix}-${target}`,
                                                     storagePools));
         return (
             <VmDisksCard
diff --git a/src/components/vm/filesystems/vmFilesystemsCard.jsx b/src/components/vm/filesystems/vmFilesystemsCard.jsx
index 7f3e15c27..f189c4b39 100644
--- a/src/components/vm/filesystems/vmFilesystemsCard.jsx
+++ b/src/components/vm/filesystems/vmFilesystemsCard.jsx
@@ -100,8 +100,7 @@ export const VmFilesystemActions = ({ connectionName, objPath, vmName, vmState }
     function open() {
         Dialogs.show(<VmFilesystemAddModal connectionName={connectionName}
                                            vmName={vmName}
-                                           objPath={objPath}
-                                           vmState={vmState} />);
+                                           objPath={objPath} />);
     }
 
     const addButton = (
@@ -116,7 +115,7 @@ export const VmFilesystemActions = ({ connectionName, objPath, vmName, vmState }
     return vmState == 'shut off' ? addButton : <Tooltip content={_("Adding shared directories is possible only when the guest is shut off")}>{addButton}</Tooltip>;
 };
 
-const VmFilesystemAddModal = ({ connectionName, objPath, vmName, vmState }) => {
+const VmFilesystemAddModal = ({ connectionName, objPath, vmName }) => {
     const Dialogs = useDialogs();
     const [additionalOptionsExpanded, setAdditionalOptionsExpanded] = useState(false);
     const [dialogError, setDialogError] = useState();
diff --git a/src/components/vm/hostdevs/hostDevAdd.jsx b/src/components/vm/hostdevs/hostDevAdd.jsx
index 6cbd7f512..ca0e4a0f8 100644
--- a/src/components/vm/hostdevs/hostDevAdd.jsx
+++ b/src/components/vm/hostdevs/hostDevAdd.jsx
@@ -41,7 +41,7 @@ import "./hostDevAdd.scss";
 
 const _ = cockpit.gettext;
 
-const TypeRow = ({ idPrefix, type, setType }) => {
+const TypeRow = ({ type, setType }) => {
     return (
         <FormGroup fieldId="usb_device"
                    label={_("Type")}
@@ -217,7 +217,7 @@ const AddHostDev = ({ idPrefix, vm }) => {
 
     const body = (
         <Form isHorizontal>
-            <TypeRow idPrefix={idPrefix} type={type} setType={setTypeWrapper} />
+            <TypeRow type={type} setType={setTypeWrapper} />
             <DevRow idPrefix={idPrefix} type={type} selectableDevices={selectableDevices} setSelectableDevices={setSelectableDevices} />
         </Form>
     );
diff --git a/src/components/vm/nics/nicAdd.jsx b/src/components/vm/nics/nicAdd.jsx
index ab512c99e..5748dfad4 100644
--- a/src/components/vm/nics/nicAdd.jsx
+++ b/src/components/vm/nics/nicAdd.jsx
@@ -85,7 +85,7 @@ const NetworkMacRow = ({ idPrefix, dialogValues, onValueChanged }) => {
     );
 };
 
-const PermanentChange = ({ idPrefix, onValueChanged, dialogValues, vm }) => {
+const PermanentChange = ({ idPrefix, onValueChanged, dialogValues }) => {
     // By default for a running VM, the iface is attached until shut down only. Enable permanent change of the domain.xml
     return (
         <FormGroup label={_("Persistence")} fieldId={`${idPrefix}-permanent`} hasNoPaddingTop>
@@ -198,8 +198,7 @@ export class AddNIC extends React.Component {
                 {domainIsRunning(vm.state) && vm.persistent &&
                 <PermanentChange idPrefix={idPrefix}
                                  dialogValues={this.state}
-                                 onValueChanged={this.onValueChanged}
-                                 vm={vm} />}
+                                 onValueChanged={this.onValueChanged} />}
             </Form>
         );
 
diff --git a/src/components/vm/snapshots/vmSnapshotsCard.jsx b/src/components/vm/snapshots/vmSnapshotsCard.jsx
index 7295577e5..ddb15eac9 100644
--- a/src/components/vm/snapshots/vmSnapshotsCard.jsx
+++ b/src/components/vm/snapshots/vmSnapshotsCard.jsx
@@ -44,7 +44,7 @@ export const VmSnapshotsActions = ({ vm, config, storagePools }) => {
     const Dialogs = useDialogs();
     const id = vmId(vm.name);
 
-    const isExternal = vmSupportsExternalSnapshots(config, vm, storagePools);
+    const isExternal = vmSupportsExternalSnapshots(config, vm);
 
     function open() {
         Dialogs.show(<CreateSnapshotModal idPrefix={`${id}-create-snapshot`}
diff --git a/src/helpers.js b/src/helpers.js
index b84234d68..0718740ae 100644
--- a/src/helpers.js
+++ b/src/helpers.js
@@ -872,7 +872,7 @@ export function getStoragePoolPath(storagePools, poolName, connectionName) {
     return pool?.target?.path;
 }
 
-export function vmSupportsExternalSnapshots(config, vm, storagePools) {
+export function vmSupportsExternalSnapshots(config, vm) {
     // External snapshot should only be used if the VM's os types/architecture allow it
     // and if snapshot features are present among guest capabilities:
     // https://libvirt.org/formatcaps.html#guest-capabilities