Bugfix/select all crashes (equinor#3428)

* Move toggleAllSelected so it is clear which variables it is using

* Remove deeply suspicious useEffect

This resolves a crash that occurs when clicking select all 'too' quickly
which otherwise can trigger an infinite loop inside this useEffect.

packages/eds-core-react/src/components/Autocomplete/Autocomplete.tsx

@@ -367,14 +367,6 @@ function AutocompleteInner<T>(
     return _availableItems
   }, [_availableItems, showSelectAll])
 
-  const toggleAllSelected = () => {
-    if (selectedItems.length === inputOptions.length) {
-      setSelectedItems([])
-    } else {
-      setSelectedItems(inputOptions)
-    }
-  }
-
   useEffect(() => {
     const availableHash = JSON.stringify(inputOptions)
     const optionsHash = JSON.stringify(options)
@@ -446,6 +438,14 @@ function AutocompleteInner<T>(
     return 'NONE'
   }, [inputOptions, selectedItems])
 
+  const toggleAllSelected = () => {
+    if (selectedItems.length === inputOptions.length) {
+      setSelectedItems([])
+    } else {
+      setSelectedItems(inputOptions)
+    }
+  }
+
   const getLabel = useCallback(
     (item: T) => {
       //note: non strict check for null or undefined to allow 0
@@ -726,12 +726,6 @@ function AutocompleteInner<T>(
     reset: resetCombobox,
   } = useCombobox(comboBoxProps)
 
-  useEffect(() => {
-    if (isControlled) {
-      setSelectedItems(selectedOptions)
-    }
-  }, [isControlled, selectedOptions, setSelectedItems])
-
   const { x, y, refs, update, strategy } = useFloating<HTMLInputElement>({
     placement: 'bottom-start',
     middleware: [