EditRolesForm.js in foreman_ansible-8.0.0

- old
+ new

@@ -1,9 +1,9 @@
 import React, { useState } from 'react';
 import { translate as __ } from 'foremanReact/common/I18n';
 import PropTypes from 'prop-types';
-import { isEqual } from 'lodash';
+
 import { useMutation } from '@apollo/client';
 
 import { Button, Modal, Spinner } from '@patternfly/react-core';
 import { encodeId } from '../../../../../globalIdHelper';
 import assignAnsibleRoles from '../../../../../graphql/mutations/assignAnsibleRoles.gql';
@@ -19,14 +19,16 @@
     hostId,
     baseModalProps,
     actions,
   } = props;
 
-  const initAvailableOpt = availableRoles.map(item => item.name);
-  const initChosenOpt = assignedRoles.map(item => item.name);
-  const [availableOptions, setAvailableOptions] = useState(initAvailableOpt);
-  const [chosenOptions, setChosenOptions] = useState(initChosenOpt);
+  const [availableOptions, setAvailableOptions] = useState(
+    availableRoles.map(item => item.name)
+  );
+  const [chosenOptions, setChosenOptions] = useState(
+    assignedRoles.map(item => item.name)
+  );
 
   const onListChange = (nextAvailable, nextChosen) => {
     setAvailableOptions(nextAvailable);
     setChosenOptions(nextChosen);
   };
@@ -41,19 +43,15 @@
   const variables = {
     id: encodeId('Host', hostId),
     ansibleRoleIds: roleNamesToIds(allRoles, chosenOptions),
   };
 
-  const didNotModifyOptions = () =>
-    isEqual(initAvailableOpt.sort(), availableOptions.sort()) &&
-    isEqual(initChosenOpt, chosenOptions); // The order of the chosen options is important.
-
   const formActions = [
     <Button
       key="confirm"
       variant="primary"
       onClick={() => callMutation({ variables })}
-      isDisabled={loading || didNotModifyOptions()}
+      isDisabled={loading}
       aria-label="submit ansible roles"
     >
       {__('Confirm')}
     </Button>,
     ...actions,