Execute scripts AJAX returns

Question

TIP: "you could simple send an json object / array from php to js, and execute every entry like "update_match('1');" using the eval() function, please stop sending js code that way"- Lucian Depold.

In the index.php, I have this code, which executes when the document is ready:

$.post('php/main.php', {elements: 1}, function(return_msg) {
    alert(return_msg);
});

The respond I get is a bunch of scripts, as expected, with the expected values. However, they do not execute! How to make them execute?

Here is the response:

<script>jsfunction(37069);</script><script>updateTextbox('');</script><script>update_match('1', '19.30 Friday 15/5', '1');</script><script>update_player('1', '1', 'recoba', 'cmf', 'teo', '0', '0');</script><script>update_player('1', '2', 'nesta', 'cb', 'tsoulou', '0', '0');</script><script>update_player('1', '3', 'raul', 'cf', 'striker', '0', '0');</script><script>update_player('1', '4', 'samuel', 'cb', 'mafia', '', '1');</script><script>update_player('1', '5', 'deisler', 'cmf', 'free_kick', '1', '');</script><script>update_player('1', '6', 'ferdinard', 'cb', 'strong', '1', '');</script><script>update_match('2', 'Match 2', '0');</script>

When I had the PHP code that produced these scripts in the bottom of the index.php, all the js functions where called correctly. Because of this question though, I had to move the code to another .php file.

are you sending all those "<script>" tags over ajax and execute them ??? — Lucian Depold
– Lucian Depold, Commented Sep 13, 2015 at 21:38
you could simply send an json object / array from php to js, and execute every entry like "update_match('1');" using the eval() function, please stop sending js code that way — Lucian Depold
– Lucian Depold, Commented Sep 13, 2015 at 21:42

Lucian Depold · Accepted Answer · 2015-09-13 22:52:47Z

2

Do it this way:

In PHP...

$arrayOfCalls = array();
$arrayOfCalls[]="update_match('1')";
$arrayOfCalls[]="update_match('2')";

$dummy = array();
$dummy['calls'] =$arrayOfCalls;
echo json_encode($dummy);

And in Javascript...

$.post('php/main.php', {elements: 1}, function(return_json) {
    return_json = JSON.parse(return_json);
    return_json.calls.forEach(function(code){
    eval(code);
    })
});

edited Sep 13, 2015 at 22:52

answered Sep 13, 2015 at 21:47

Lucian Depold

2,0372 gold badges16 silver badges25 bronze badges

Sign up to request clarification or add additional context in comments.

9 Comments

gsamaras Over a year ago

Don't we need to specify in the post call that the data we sent are of JSON format?

Lucian Depold Over a year ago

the data you send to php, or from php to js ?

Lucian Depold Over a year ago

you could specify the fourth parameter as JSON, but jquery takes an "intelligent guess" (api.jquery.com/jquery.post). i think it should work without.

gsamaras Over a year ago

I had it working with:

$.post('php/main.php', {elements: 1}, function(return_json) { 			//console.log(return_json); 			return_json = JSON.parse(return_json); 			return_json.calls.forEach(function(code){ 				eval(code); 			}) 		});

Lucian Depold Over a year ago

maybe the fourth parameter would also have done the trick, setting it to "JSON"...

|

Lauromine · Accepted Answer · 2015-09-13 21:34:22Z

1

You can append the result to your page :

$.post('php/main.php', {elements: 1}, function(return_msg) {
    $('body').append(return_msg);
});

The code will be executed but I'm not sure if it's safe to do that.

answered Sep 13, 2015 at 21:34

Lauromine

1,48310 silver badges19 bronze badges

Comments

Axel Amthor · Accepted Answer · 2015-09-13 21:39:11Z

1

The date delivered by the AJAX call is just data - or text. In order to interpret that as JavaScript, you must append it to the dom.

Two ways: first, by appending the data directly:

$.post('php/main.php', {elements: 1}, function(return_msg) {
    alert(return_msg);
    $('body').append(return_msg);
});

but there's also a shorthand method:

$.getScript('php/main.php?elements=' + 1, function () {
   // script loaded

});

Please read the docs, there are some caveats!

answered Sep 13, 2015 at 21:39

Axel Amthor

11.1k1 gold badge28 silver badges45 bronze badges

1 Comment

gsamaras Over a year ago

Thanks! +1 for the shorthand method.

Collectives™ on Stack Overflow

Execute scripts AJAX returns

3 Answers 3

9 Comments

Comments

1 Comment

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

9 Comments

Comments

1 Comment

Your Answer

Sign up or log in

Post as a guest

Linked

Related