Load function is not sending updated object to the front end, a value fetched from database

[elhananjair]

Hello guys,
I have multiple Postgres queries that I execute when the user hits refresh for an update. I store fetched data from the database into an object I have created and then put that inside the return.
But what I am facing is, that unless I restart the application when I refresh the page it still shows the old values of the sent object. Is there any way to avoid that or any issue with my code, this is what my code looks like:

let queryResult = null;
let keys = ['k1', 'k2', 'k3']
let counter = 0
let data = {
   k1: 0,
   k2: 0,
   k3: 0,
}
async function queryDB(){
   return await sql.begin(sql => {
      //multiple sql queries and it returns an array of results 
   })
}

async function overAllResult() {
    queryResult = await queryDB();
    queryResult.map(result => {
       data[keys[counter]] = result[0].count //Count is the key from query result 
    })
}
export const load = async ({locals}) => {
    ...
    await overAllResult()
    return {
       data
    }
}

[gregg-cbs]

Shouldnt you code be like this?

// these variables are global and will not reset after every query, 
// they will persist for the whole time your server runs. 
// If you do not want that, you should put these in the overAllResults function and use them there.
let queryResult = null;
let keys = ['k1', 'k2', 'k3']
let counter = 0
let data = {
   k1: 0,
   k2: 0,
   k3: 0,
}

async function queryDB(){
   return await sql.begin(sql => {
      //multiple sql queries and it returns an array of results 
   })
}

// updated
async function overAllResult() { 
     let resData = {...data};

    queryResult = await queryDB();
    queryResult.forEach(result => {
      resData[keys[counter]] = result[0].count //Count is the key from query result 
    });

    return resData;
}

// updated
export const load = async ({locals}) => { 
   const results = await overAllResult()

    return {
      results
    }
}

[elhananjair]

Hello @gregg-cbs thank you for the suggestion. Copying the data object to a new variable was not the problem
let resData = {...data};
return resData;

The real problem with using the variable counter is, for some reason it is causing a problem, therefore I replaced it with index from the map function parameter, and now it is working.