Issue #5802

src/shared/components/Settings/Tools/Devices/index.jsx

@@ -84,46 +84,44 @@ export default class Devices extends ConsentComponent {
    */
   onHandleAddDevice(e) {
     e.preventDefault();
-    const { newDevice, deviceTrait, isEdit } = this.state;
+    const { newDevice, deviceTrait } = this.state;
     const { clearDeviceState } = this.props;
     this.setState({ isSubmit: true });
     if (this.onCheckFormValue(newDevice)) {
       return;
     }
-    if (!isEdit) {
-      const deviceItems = deviceTrait.traits
-        ? deviceTrait.traits.data.slice() : [];
-      let exist = false;
-      // eslint-disable-next-line no-restricted-syntax
-      for (const item of deviceItems) {
-        if (item.deviceType === newDevice.deviceType
-          && item.manufacturer === newDevice.manufacturer
-          && item.model === newDevice.model
-          && item.operatingSystem === newDevice.operatingSystem) {
-          exist = true;
-          break;
-        }
-      }
-      if (exist === true) {
-        const empty = {
-          deviceType: '',
-          manufacturer: '',
-          model: '',
-          operatingSystem: '',
-        };
-        this.setState({
-          newDevice: empty,
-          isEdit: false,
-          indexNo: null,
-          isSubmit: false,
-        });
-        clearDeviceState();
-        setImmediate(() => {
-          toastr.error('Looks like you\'ve already entered this device.');
-        });
-        return;
+    const deviceItems = deviceTrait.traits
+      ? deviceTrait.traits.data.slice() : [];
+    let exist = false;
+    // eslint-disable-next-line no-restricted-syntax
+    for (const item of deviceItems) {
+      if (item.deviceType === newDevice.deviceType
+        && item.manufacturer === newDevice.manufacturer
+        && item.model === newDevice.model
+        && item.operatingSystem === newDevice.operatingSystem) {
+        exist = true;
+        break;
       }
     }
+    if (exist === true) {
+      const empty = {
+        deviceType: '',
+        manufacturer: '',
+        model: '',
+        operatingSystem: '',
+      };
+      this.setState({
+        newDevice: empty,
+        isEdit: false,
+        indexNo: null,
+        isSubmit: false,
+      });
+      clearDeviceState();
+      setImmediate(() => {
+        toastr.error('Looks like you\'ve already entered this device.');
+      });
+      return;
+    }
     this.showConsent(this.onAddDevice.bind(this));
   }