- Issue created by @kaszarobert
- Status changed to Postponed: needs info
about 1 year ago 11:58am 24 November 2023 - π©π°Denmark ressa Copenhagen
Maybe you can add some more details, like the riddle itself, and which Drupal version you are using?
- Status changed to Closed: works as designed
about 1 year ago 12:16pm 24 November 2023 - πΈπ°Slovakia kaszarobert
Found the problem: the site we were working on runs on a Drupal 9.5, with riddler version 2.0.5, with an older captcha 8.x-1.10. And the older captcha module does not have this class, it uses globals instead. So, no problem here.
- Status changed to Needs work
about 1 year ago 12:52pm 24 November 2023 - π©π°Denmark ressa Copenhagen
Thanks, but shouldn't we raise the minimum CATPCHA version in composer.json for 2.x to match when the class was added?
It currently says:
"require": { "drupal/captcha": "^1.1" }
Or are you perhaps using a custom module?
- Status changed to Needs review
about 1 year ago 1:19pm 24 November 2023 - πΈπ°Slovakia kaszarobert
Oh, I didn't even think about that. Yes, it would prevent such confusions definitely if we set dependencies and their versions perfectly. Now, let me see... according to their codebase, CaptchaConstants was introduced in version 8.x-1.7, so we should raise that then.
I opened a MR with this.
- πΈπ°Slovakia kaszarobert
One thing I didn't check is if the other branches, such as 3.x do have this issue, too.
- π©π°Denmark ressa Copenhagen
Thanks, but I don't see the MR ... or did you want to check the 3.x thing first?
- Merge request !32Issue #3403987: At least captcha 1.7 should be required for the module to work β (Merged) created by kaszarobert
- last update
about 1 year ago 33 pass - πΈπ°Slovakia kaszarobert
Yeah, you're right, I only created a fork and commit there, I forgot to press create Merge request button. Sorry for that. But better late than never.
- π©π°Denmark ressa Copenhagen
I also often forget to create the MR, or change to "Needs review" :)
- Status changed to Fixed
6 months ago 7:23am 9 August 2024 -
Anybody β
committed d459f9f2 on 2.x authored by
kaszarobert β
Issue #3403987 by kaszarobert, Anybody: Increase CAPTCHA module version...
-
Anybody β
committed d459f9f2 on 2.x authored by
kaszarobert β
Automatically closed - issue fixed for 2 weeks with no activity.