Jump to content

Welcome, Guest!

By registering with us, you'll be able to discuss, share and private message with other members of our community.

oliiiix

reCaptcha not working at all, ei captcha used as replacement

Recommended Posts

Dear All

The TB reCaptcha module doesn't work for me at all. First of all: after installation, clearing the cache didn't get rid of the 500 error message. I had to delete the whole cache folder and re upload the one from the TB Zip file.

Then, configuration seems to work without problems. The captcha is displayed on the contact form as well, but it doesn't matter as I can do everything without clicking on the captcha at all. So it doesn't stop me from sending contact messages even though I haven't checked the captcha checkbox. It's there, but it's not working. I can send as many contact messages without ever touching the captcha to confirm me not being a robot.

Anyone else having the same issues? I have configured the module completely and set it to appear everywhere it can be configured to appear. I also have overrides enabled etc under performance. Luckily I can continue with ei captcha which I used for prestashop which is working as it should 😉

Here's the link for everyone else who's having trouble:

https://github.com/nenes25/eicaptcha

You'll have to use the latest version as previous versions were not working when JS was disabled!

All the best,

Oli

Share this post


Link to post
Share on other sites

Can you check if the module's overrides are installed correctly?

Share this post


Link to post
Share on other sites
1 hour ago, datakick said:

Can you check if the module's overrides are installed correctly?

How Can I do that?

Share this post


Link to post
Share on other sites

Ah, so Yes, I see that file with date modification from today so it has to be from the module installation.

Share this post


Link to post
Share on other sites

Wtf. I was sure it its working in my store. Just tested too and the captcha is not needed 😂

Share this post


Link to post
Share on other sites
5 minutes ago, wakabayashi said:

Wtf. I was sure it its working in my store. Just tested too and the captcha is not needed 😂

😂

Share this post


Link to post
Share on other sites

oi, there's a new bug in core 🙂

When NoCaptchaRecaptcha module is installed, the call to

Module::isEnabled('NoCaptchaRecaptcha')

returns false, while call

Module::isEnabled('nocaptcharecaptcha')

returns expected true.

I'll fix this in the core.

Until then, you edit override file (\override\controllers\front\ContactController.php) and replace all occurrence of Module::isEnabled('NoCaptchaRecaptcha') with Module::isEnabled('nocaptcharecaptcha')  

  • Thanks 3

Share this post


Link to post
Share on other sites

Oh I was just trying to solve it too ^^

Thanks for the solution! I can confirm, that it works again!

Share this post


Link to post
Share on other sites

I also confirm it is working!

Just to know how it works... If I am using bleeding edge and you correct the core, should I also change (revert) again this override file, or  no need to touch anything because bleeding edge update core but also modify (revert) this file to its original/correct state? I hope I explained myself clearly 😅

Share this post


Link to post
Share on other sites

I believe, it doesn't matter at all. I guess, he will fix the core, so that lower/uppercase won't matter anymore. Next time when you update the module, the override will probably be replaced, but I am not 100% sure about it.

Share this post


Link to post
Share on other sites
4 minutes ago, rubben1985 said:

I also confirm it is working!

Just to know how it works... If I am using bleeding edge and you correct the core, should I also change (revert) again this override file, or  no need to touch anything because bleeding edge update core but also modify (revert) this file to its original/correct state? I hope I explained myself clearly 😅

CoreUpdater will fix code in core only, it will not touch your module.

But, in this case, you won't have to do anything. This issue will just be fixed 'twice' 🙂

  • Like 1

Share this post


Link to post
Share on other sites
On 1/21/2020 at 6:12 PM, datakick said:

I'll fix this in the core.

Not sure whether this is a good idea. NoCaptchaRecaptcha and nocaptcharecaptcha are two distinct modules, one can have both installed. Any attempt to deviate from the name of the directory on disk might well lead to unexpected behavior in other places.

Share this post


Link to post
Share on other sites

So I encountered the same problem now with TB 1.1.0 and latest "nocaptcharecaptcha" module... 
I can confirm that Datakick's fix above works. For those of us with an existing override, this post will help.

Module Update Request please:
Can Datakick's fix please be incorporated into the module and pushed as update?
https://github.com/thirtybees/nocaptcharecaptcha/issues/5

New users / developers will continue to experience this problem until it's been patched.
Also, I'm sure (from what I've seen in this thread) that there are merchants that think this is working when it's actually not.
 

Edited by Theo

Share this post


Link to post
Share on other sites
38 minutes ago, Theo said:

Module Update Request please:
Can Datakick's fix please be incorporated into the module and pushed as update?
https://github.com/thirtybees/nocaptcharecaptcha/issues/5
 

No, it can't. The issue you are referencing is about something very different, it has nothing to do with this new problem on bleeding edge. 

I've filed a new issue to track this new problem 

https://github.com/thirtybees/nocaptcharecaptcha/issues/8

Note that while we can fix this in the module, we need to fix the core as well

  • Thanks 1

Share this post


Link to post
Share on other sites

Create an account or sign in to comment

You need to be a member in order to leave a comment

Create an account

Sign up for a new account in our community. It's easy!

Register a new account

Sign in

Already have an account? Sign in here.

Sign In Now

×
×
  • Create New...