Doesn’t work with non standard admin urls

  • Resolved themightyjmac

    (@themightyjmac)


    6 years, 1 month ago

    Hi, I just posted this information as part of a review, but this is probably a better place for it!

    There’s a bug with the plugin which prevents it from working if you’re using a non-standard wordpress ajax url for any reason. Using a distribution like Bedrock is one way your ajax url could be different.

    There’s a simple fix for the problem, just determine the admin ajax url in a better way, using the following code changes:
    recaptcha.php line 64:
    'ajax_url' => get_admin_url(null, 'admin-ajax.php'),

    recaptcha.js line 92:
    GriwpcAjax.post ( griwpco.ajax_url, { 'action' : 'griwpc_verify_recaptcha', 'resp' : griwpcr }, griwpcProcessAjaxResponse, true );

    Thanks for the great plugin, hopefully you can get these fixes in!

Viewing 6 replies - 1 through 6 (of 6 total)
  • Thread Starter themightyjmac

    (@themightyjmac)

    Also for some reason I’m not always getting ajaxResponse.statusText to be equal to “OK” (even when the response is successful), so I’ve had to remove this part of the ajax reponse verification.

    Plugin Author jmviade

    (@jmviade)

    Hello, themightyjmac,

    Thanks for your comments. Interessing point of view, other WP distributions…

    So, I’ll review the “fixed admin url” which will be substituted for a dynamic var -most probably- and also the behaviour of the “statusText”, both in the next minor plugin version… In fact, these last days I’m working on it due to the new WP version.

    Best,

    Joan Miquel

    Hello @jmviade,

    Please can you indicate when this fix will be put in place?

    Thanks,

    Luc

    Plugin Author jmviade

    (@jmviade)

    Hi, NewSky,

    Ups!… I’m sorry, that’s true … You’re absolutely right.

    I think this feature will be introduced during the next week. Thanks for the reminder ??

    Joan Miquel

    Nice !

    Thanks,

    Plugin Author jmviade

    (@jmviade)

    Hi, NewSky.

    Plugin Update to version 9.1.2 that includes non standard admin URL.

    Best,

    Joan Miquel

Viewing 6 replies - 1 through 6 (of 6 total)
  • The topic ‘Doesn’t work with non standard admin urls’ is closed to new replies.