fix: issue #5802

Author: yoution

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

@@ -90,40 +90,38 @@ export default class Devices extends ConsentComponent {
     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));
   }