Handle data from $.post request

Asked Apr 22 '18 at 22:04

Active Apr 22 '18 at 22:20

Viewed 19 times

I wanna get and handle the data from a POST request.

I got the code:

function GetSqlResult (sql_query) {
  var result = [];
  $.post("getsqlresult.php", {
    sql: sql_query
  },
  function(data){
    result.push(data);
  });

  return result;
};

I get the error "result.append(data) is not a function".

I don't know if that what I'm doing is possible, but I wanna collect the data in an array.

This "could be" a duplicate from "Data from post request" , but it didn't work with the responses from that question, that's why I'm asking with my own code.

edited Apr 22 '18 at 22:20

asked Apr 22 '18 at 22:04

David Leites

Javascript arrays do not have a function named `append`. See https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Array You want `result.push(data)` – Wesley Smith Apr 22 '18 at 22:06
oh shit, that's right, so sorry – David Leites Apr 22 '18 at 22:08
That said, we should mention that `return result` will not do what you expect. – Wesley Smith Apr 22 '18 at 22:09
@DelightedD0D, thank you very much, this was the dumbest thing ever, I really thought the problem was handleling the post response – David Leites Apr 22 '18 at 22:10
really?, How so? – David Leites Apr 22 '18 at 22:10
No worries, Ajax is an asynchronous operation. ie, the call happens in the background and further code execution inline continues. That is to say, `GetSqlResult` will call `return result` and return your empty array *before* the ajax finishes – Wesley Smith Apr 22 '18 at 22:12
oh, so what should I do? – David Leites Apr 22 '18 at 22:13
Instead of calling `result.append(data);`, make a new function called `handleResponse` and call `handleResponse(data);` then, do your stuff in that function :) – Wesley Smith Apr 22 '18 at 22:13
Beyond that little hiccup the whole approach is wrong since `$.post()` (which is a shortcut for `$.ajax()`) is **asynchronous** – charlietfl Apr 22 '18 at 22:15
You can even pass it in like `$.post("getsqlresult.php", {sql: sql_query}, handleResponse);` – Wesley Smith Apr 22 '18 at 22:15
1

Also a horrible security flaw if you are actually putting sql query strings in client side code where anyone can see them – charlietfl Apr 22 '18 at 22:19
@charlietfl , thank you, I didn't know that – David Leites Apr 22 '18 at 22:23
@DelightedD0D, I'm gonna try, thanks – David Leites Apr 22 '18 at 22:24
@MarianaLeites to charlietfl's comment, see https://en.wikipedia.org/wiki/SQL_injection – Wesley Smith Apr 22 '18 at 22:25

Handle data from $.post request

0 Answers0