JSONP has no dynamic callback

Still an issue on the Drupal 7 version.
Fix still works - line 28.
Thanks!

Having a dynamic way to set the callback name in jsonp is a must have ! here is the patch of this feature

This is really a must have otherwise you cannot make a JSONP AJAX request!

Patch still works and looks fine, so RTBC!

Marked #2010558: Option to add JSONP Prefix according to the callback query string parameter as duplicate.

Sorry but there is a problem if no $_GET parameter was set (e.g. fetching the JSON via PHP) so I re-created the patch.

Can somebody explain how this is supposed to work with AngularJS? What do I need to send as the 'jsonp_prefix' parameter from $http? Thanks!

Hi yannickoo,

I'm not sure your patch is correct.

In my opinion:

1) You use $_GET[$jsonp_prefix] which I think should be $_GET['jsonp_prefix']
2) You should be using || instead of &&.

Hey Chris, that's correct because we have to check whether the $json_prefix is available in the $_GET array. The key is not 'json_prefix' it is $json_prefix. It is dynamic so that you can set it in the Views UI.

Ah got it! Thanks for pointing that out!

But I would switch them, first the isset and then checking for a value...

Fixed with 6e9b6b9.

Fixed with 5c0c7aa.