Move dependency map resolution to after all promises are settled.

master
Joshua Sigona 4 years ago
parent f6568e54a7
commit 65c51606bd
  1. 4
      src/App.js

@ -474,6 +474,7 @@ function TableEditor(p) {
useEffect(()=>{ useEffect(()=>{
if (update) { if (update) {
setLoading(true) setLoading(true)
var dependency_map = {}
axios.get(BACKEND_URL+p.path) axios.get(BACKEND_URL+p.path)
.then((data)=>{ .then((data)=>{
var cols = data.data.fields var cols = data.data.fields
@ -481,7 +482,6 @@ function TableEditor(p) {
setFields(cols.filter((col,i)=>col.name!=="id"&&!(i===0&&col.name==="name"))) setFields(cols.filter((col,i)=>col.name!=="id"&&!(i===0&&col.name==="name")))
var dependency_map = {}
var promise_list = [] var promise_list = []
cols.filter((col)=>col.name!=="id"&&col.name.includes("_id")).forEach((col)=>{ cols.filter((col)=>col.name!=="id"&&col.name.includes("_id")).forEach((col)=>{
@ -490,11 +490,11 @@ function TableEditor(p) {
dependency_map[col.name]=data.data.rows.reverse() dependency_map[col.name]=data.data.rows.reverse()
})) }))
}) })
setDependencies(dependency_map)
setData(rows) setData(rows)
return Promise.allSettled(promise_list) return Promise.allSettled(promise_list)
}) })
.then(()=>{ .then(()=>{
setDependencies(dependency_map)
setLoading(false) setLoading(false)
}) })
setUpdate(false) setUpdate(false)

Loading…
Cancel
Save