400 mauvaise requête sur admin-ajax.php en utilisant uniquement le hook d'action wp_enqueue_scripts

Question

400 mauvaise requête sur admin-ajax.php en utilisant uniquement le hook d'action wp_enqueue_scripts

- 23
- 73659 2018-01-17
Jetravaille sur ajax ces dernierstemps. Lestutoriels que voustrouvez sur lenet sonttoustrès similaireset assezfaciles àmettreen œuvre. Maisj'obtienstoujours une mauvaise requête 400 surmonfichier ajax-admin.php .

Après une recherche longueet intensive,j'aimaintenant découvert que c'était à cause dutemps d'intégration.

Sij'utilise le hook d'action init pourinitialiser le scriptet wp_localize_script ,toutfonctionnebien. Le code lui-même doit donc être correct.

my-page-test-functions.php

fonction ajax_login_init () { wp_register_script ('ajax-login-script',get_stylesheet_directory_uri (). '/js/ajax-login-script.js',array ('jquery')); wp_enqueue_script ('ajax-login-script'); wp_localize_script ('ajax-login-script','ajax_login_object',array ('ajaxurl'=> admin_url ('admin-ajax.php'),'redirecturl'=> 'REDIRECT_URL_HERE','loadingmessage'=> __ ('Envoi d'informations sur l'utilisateur,veuillezpatienter ...'))); add_action ('wp_ajax_nopriv_ajaxlogin','ajax_login'); } if (!is_user_logged_in ()) { add_action ('init','ajax_login_init'); } function ajax_login () { //le champnonceest créé sur lapage check_ajax_referer ('ajax-login-nonce','sécurité'); //CODE mourir(); }

Mais sij'utiliseparexemple wp_enqeue_scripts crochet d'action Je reçoistoujours lamauvaise requête.

if (!is_user_logged_in ()) { add_action ('wp_enqueue_scripts','ajax_login_init'); }

Leproblème avec ceciest:

Je voudrais avoir lesfonctions dans unfichierphp supplémentaireet les charger uniquement sielles sontnécessaires sur unepageparticulière. Pour cela,j'aibesoin,parexemple,de is_page () . Mais is_page () fonctionne auplustôt lorsquej'accroche lafonction avec l'inclusion dans le hook d'action parse_query :

functions.php

fonction sw18_page_specific_functions () { if (is_page ('page-test')) { include_oncenom_rép (__ FILE __). '/includes/my-page-test-functions.php'; } } add_action ('parse_query','sw18_page_specific_functions');

Ainsi,lesfonctions accrochées au hook init dans lefichier my-page-test-functions.php ne se déclenchentpas,je suppose,car init vient avant parse_query .

Existe-t-il desbonnespratiquespour organiser cela,pour que celafonctionne? Ou commentpuis-je corriger lamauvaise requête admin-ajax.php lors de l'utilisation du hook d'action wp_enqeue_scripts ?
I've been working on ajax lately. The tutorials you find on the net are all very similar and quite easy to implement. But I always get a bad request 400 on my ajax-admin.php file.

After a long and intensive search, I have now found out that's because of the time of integration.

If I use the init action hook to initialize script and wp_localize_script, everything works fine. So the code itself must be correct.

my-page-test-functions.php

function ajax_login_init(){ wp_register_script('ajax-login-script',get_stylesheet_directory_uri().'/js/ajax-login-script.js',array('jquery')); wp_enqueue_script('ajax-login-script'); wp_localize_script('ajax-login-script','ajax_login_object',array('ajaxurl' => admin_url('admin-ajax.php'),'redirecturl' => 'REDIRECT_URL_HERE','loadingmessage' => __('Sending user info, please wait...'))); add_action('wp_ajax_nopriv_ajaxlogin','ajax_login'); } if(!is_user_logged_in()){ add_action('init','ajax_login_init'); } function ajax_login(){ //nonce-field is created on page check_ajax_referer('ajax-login-nonce','security'); //CODE die(); }

But if I use e.g. wp_enqeue_scripts action hook I always get the bad request.

if(!is_user_logged_in()){ add_action('wp_enqueue_scripts','ajax_login_init'); }

The problem with this is:

I would like to have the functions in an extra php file and load them only if they are needed on a particular page. For this I need, for example is_page(). But is_page() works at the earliest when I hook the function with the include into the parse_query action hook:

functions.php

function sw18_page_specific_functions(){ if(is_page('page-test')){ include_once dirname(__FILE__).'/includes/my-page-test-functions.php'; } } add_action('parse_query','sw18_page_specific_functions');

So then functions hooked to init hook in my-page-test-functions.php file does not triggered, I suppose, because init comes before parse_query.

Is there a best practices to organize this, so it works? Or how can I fix the admin-ajax.php bad request when using the wp_enqeue_scripts action hook?
functions ajax

3 réponses

votes

0

2019-04-14

N'oubliezpas d'ajouter lenom de lafonction 'action' à labalise wp_ajax_.

 function fetchorderrows() { // Want to run this func on ajax submit
  // Do awesome things here all day long
}

add_action('wp_ajax_fetchorderrows', 'fetchorderrows', 0);

Remember to have the 'action' function name appended to the wp_ajax_ tag.

function fetchorderrows() { // Want to run this func on ajax submit
  // Do awesome things here all day long
}

add_action('wp_ajax_fetchorderrows', 'fetchorderrows', 0);

- -3
- 2018-11-12
écrivez simplement die;à lafin comme ci-dessous ...

just write die; at the end like below...
- Salut Zee Xhan.Bienvenue sur le site.Votre réponse abesoin de quelques révisions.Premièrement,si votre réponseest un code,ne publiezpas de capture d'écran.À laplace,publiez le code sousforme d'extrait de codeet formatez-le sousforme de code (utilisez lebouton {}).C'estprobablement la raisonpour laquelle votre réponse a été rejetéeet refusée.En outre,unpeuplus d'explications serait utile - comme "pourquoi" écrivez simplement die (),et où cela va-t-ilexactementpar rapport au code dans l'OP (message original)?
  
  Hi Zee Xhan. Welcome to the site. Your answer needs some revisions. First, if your answer is code, don't post a screenshot. Instead, post the code as a snippet and format it as code (use the {} button). That's likely the reason your answer was downvoted and not accepted. Also, a little more explanation would be helpful - like "why" just write die(), and where exactly does this go in relation to the code in the OP (original post)?
  - 9
  - 2018-11-12
  - butlerblog

swissspidy · Accepted Answer · 2018-01-17

17

2018-01-17

Jepense que la seule chose quimanqueiciest que vous devez déplacer add_action('wp_ajax_nopriv_ajaxlogin','ajax_login');en dehors de ajax_login_init.

Ce codeenregistre votregestionnaire Ajax,mais lorsque vousne l'exécutez que sur wp_enqueue_scripts,ilest déjàtroptardet les wp_ajax_nopriv_ hooks sont déjàexécutés.

Alors,avez-vousessayé quelque chose comme ceci:

 function ajax_login_init(){
  if ( ! is_user_logged_in() || ! is_page( 'page-test' ) ) {
    return;
  }

  wp_register_script('ajax-login-script',get_stylesheet_directory_uri().'/js/ajax-login-script.js',array('jquery'));
  wp_enqueue_script('ajax-login-script');
  wp_localize_script('ajax-login-script','ajax_login_object',array('ajaxurl' => admin_url('admin-ajax.php'),'redirecturl' => 'REDIRECT_URL_HERE','loadingmessage' => __('Sending user info, please wait...')));
}

add_action( 'wp_enqueue_scripts','ajax_login_init' );

add_action( 'wp_ajax_nopriv_ajaxlogin','ajax_login' );

function ajax_login(){
  //nonce-field is created on page
  check_ajax_referer('ajax-login-nonce','security');
  //CODE
  die();
}

Ilestmaintenantplus clair que vousne souhaitez charger le JavaScript que sur cettepageparticulière. Cela signifie que vous devezmettre votre is_page() dans ajax_login_init(). J'aimis àjour le codeen conséquence.

Maintenant,pourquoi votre solutionn'a-t-ellepasfonctionné?

La vérification is_page() signifiait que votrefichier defonctionsn'était chargé que sur cettepage spécifique. ajax_login_init()est appeléet vos scriptsmisen file d'attente. Jusqu'icitout vabien.

Maintenant,votre scripteffectue l'appel ajax. Commementionné dans les commentaires,les appels ajaxne sontpas conscients de lapage actuelle sur laquelle vous voustrouvez. Il y a une raisonpour laquelle lefichier setrouve dans wp-admin/admin-ajax.php. Iln'y apas de WP_Queryet donc is_page()ne fonctionnepaspendant une requête ajax.

Puisque celane fonctionnepas,sw18_page_specific_functions()ne fera rien dans un contexte ajax. Cela signifie que votrefichier defonctionsn'estpas chargéet que votregestionnaire ajaxn'existepas.

C'estpourquoi vous deveztoujoursinclure cefichier defonctionset déplacer cette vérification is_page() dans ajax_login_init().

Donc au lieu de sw18_page_specific_functions() { … }exécutez simplement include_once dirname(__FILE__).'/includes/my-page-test-functions.php'; directement . Sans aucun appel add_action( 'parse_query' ).

I think the only thing missing here is that you need to move add_action('wp_ajax_nopriv_ajaxlogin','ajax_login'); outside ajax_login_init.

That code registers your Ajax handler, but when you only run it on wp_enqueue_scripts, it's already too late and wp_ajax_nopriv_ hooks are already run.

So, have you tried something like this:

function ajax_login_init(){
  if ( ! is_user_logged_in() || ! is_page( 'page-test' ) ) {
    return;
  }

  wp_register_script('ajax-login-script',get_stylesheet_directory_uri().'/js/ajax-login-script.js',array('jquery'));
  wp_enqueue_script('ajax-login-script');
  wp_localize_script('ajax-login-script','ajax_login_object',array('ajaxurl' => admin_url('admin-ajax.php'),'redirecturl' => 'REDIRECT_URL_HERE','loadingmessage' => __('Sending user info, please wait...')));
}

add_action( 'wp_enqueue_scripts','ajax_login_init' );

add_action( 'wp_ajax_nopriv_ajaxlogin','ajax_login' );

function ajax_login(){
  //nonce-field is created on page
  check_ajax_referer('ajax-login-nonce','security');
  //CODE
  die();
}

Edit:

Now it's more clear that you only want to load the JavaScript on that particular page. This means you need to put your is_page() inside ajax_login_init(). I've updated the code accordingly.

Now, why didn't your solution work?

The is_page() check meant that your functions file was only loaded on that specific page. ajax_login_init() gets called and your scripts enqueued. So far so good.

Now your script makes the ajax call. As mentioned in the comments, ajax calls are not aware of the current page you're on. There's a reason the file sits at wp-admin/admin-ajax.php. There's no WP_Query and thus is_page() does not work during an ajax request.

Since that does not work, sw18_page_specific_functions() won't do anything in an ajax context. This means your functions file is not loaded and your ajax handler does not exist.

That's why you need to always include that functions file and move that is_page() check inside ajax_login_init().

So instead of sw18_page_specific_functions() { … } just run include_once dirname(__FILE__).'/includes/my-page-test-functions.php'; directly. Without any add_action( 'parse_query' ) call.

Bonne suggestion.J'ai changé cela (toujours lamêmeerreur),mais leproblèmeesttoujours que lefichier contenant lesfonctions se chargeratroptard.Maisj'aibesoin d'unmoyen de distinguer quellepageest utilisée.- actuellementj'essaye ceci avecis_page () comme décrit ci-dessus.

Good suggestion. I have changed that (still the same error), but the problem still is that the file containing the functions will load too late. But I need a way to distinguish which page is used. - currently I try this with is_page () as described above.
- 0
- 2018-01-17
- Sin
Essayez-vous d'exécuter `is_page ()` depuis `ajax_login ()` ou depuis `ajax_login_init ()`.Lepremierne peutpasfonctionner car c'est dans un contexte Ajax.

Are you trying to run `is_page()` from within `ajax_login()` or from within `ajax_login_init()`. The former can't work because it's in an Ajax context.
- 0
- 2018-01-17
- swissspidy
J'ai énuméré lesfichiers dans lesquels setrouvent lesfonctions,commetexte descriptif ci-dessus.Is_page ()est utilisé dans lefunctions.phpet sert àinclure lefichier avec lesfonctions ajax uniquement lorsque celaestnécessaire.

I have enumerated the files in which the functions are, as descriptive text above. The is_page() is used in the functions.php and serves to include the file with the ajax functions only when needed.
- 0
- 2018-01-17
- Sin
@Sin Encore unefois,`is_page ()`ne fonctionnepas dans un contexte Ajax.J'aimis àjourma réponseen conséquence.

@Sin Again, `is_page()` does not work in an Ajax context. I have updated my answer accordingly.
- 0
- 2018-01-17
- swissspidy