Merge pull request #137 from influxdata/fix/more-alert-issues

fix: more alerting issues with mutation

package.json

@@ -1,6 +1,6 @@
 {
   "name": "flightsql-datasource",
-  "version": "1.0.2",
+  "version": "1.0.3",
   "description": "",
   "scripts": {
     "build": "webpack -c ./.config/webpack/webpack.config.ts --env production",

src/components/BuilderView.tsx

@@ -105,7 +105,8 @@ export function BuilderView({query, datasource, onChange, fromRawSql}: any) {
 
   useEffect(() => {
     if (where) {
-      handleWhereChange(where, setWhereValues, whereValues)
+      const copyWheres = [...whereValues]
+      handleWhereChange(where, setWhereValues, copyWheres)
     }
     // eslint-disable-next-line react-hooks/exhaustive-deps
   }, [where])

src/components/utils.ts

@@ -51,8 +51,13 @@ export const handleColumnChange = (column: any, setColumnValues: any, columnValu
   if (column.index === undefined) {
     column.index = lastIndex
   }
-  columnValues[column.index]['value'] = column?.value
-  setColumnValues(columnValues)
+  const newColumnValues = [
+    ...columnValues.slice(0, column.index),
+    {
+      value: column?.value
+    },
+  ]
+  setColumnValues(newColumnValues)
 }
 
 export const addColumns = (setColumnValues: any, columnValues: any) => {
@@ -66,8 +71,12 @@ export const removeColumns = (i: any, setColumnValues: any, columnValues: any) =
 }
 
 export const handleWhereChange = (where: any, setWhereValues: any, whereValues: any) => {
-  let newWhereValues = [...whereValues]
-  newWhereValues[where.index]['value'] = where.value
+  const newWhereValues = [
+    ...whereValues.slice(0, where.index),
+    {
+      value: where?.value
+    },
+  ]
   setWhereValues(newWhereValues)
 }